feat(config): add config compiler plugin that includes default:/menu into schema

Author: lotem

src/rime/config/default_config_plugin.cc

@@ -0,0 +1,31 @@
+//
+// Copyright RIME Developers
+// Distributed under the BSD License
+//
+#include <boost/algorithm/string.hpp>
+#include <rime/config/config_compiler_impl.h>
+#include <rime/config/config_cow_ref.h>
+#include <rime/config/plugins.h>
+
+namespace rime {
+
+bool DefaultConfigPlugin::ReviewCompileOutput(
+    ConfigCompiler* compiler, an<ConfigResource> resource) {
+  return true;
+}
+
+bool DefaultConfigPlugin::ReviewLinkOutput(
+    ConfigCompiler* compiler, an<ConfigResource> resource) {
+  if (!boost::ends_with(resource->resource_id, ".schema"))
+    return true;
+  auto target = Cow(resource, "menu");
+  Reference reference{"default", "menu", true};
+  if (!IncludeReference{reference}
+        .TargetedAt(target).Resolve(compiler)) {
+    LOG(ERROR) << "failed to include section " << reference;
+    return false;
+  }
+  return true;
+}
+
+}  // namespace rime

src/rime/config/plugins.h

@@ -27,6 +27,12 @@ class AutoPatchConfigPlugin : public ConfigCompilerPlugin {
   Review ReviewLinkOutput;
 };
 
+class DefaultConfigPlugin : public ConfigCompilerPlugin {
+ public:
+  Review ReviewCompileOutput;
+  Review ReviewLinkOutput;
+};
+
 class LegacyPresetConfigPlugin : public ConfigCompilerPlugin {
  public:
   Review ReviewCompileOutput;

src/rime/core_module.cc

@@ -22,6 +22,7 @@ static void rime_core_initialize() {
 
   auto config = new ConfigComponent;
   config->InstallPlugin(new AutoPatchConfigPlugin);
+  config->InstallPlugin(new DefaultConfigPlugin);
   config->InstallPlugin(new LegacyPresetConfigPlugin);
   config->InstallPlugin(new LegacyDictionaryConfigPlugin);
   r.Register("config", config);

src/rime/schema.cc

@@ -31,15 +31,7 @@ void Schema::FetchUsefulConfigItems() {
   if (!config_->GetString("schema/name", &schema_name_)) {
     schema_name_ = schema_id_;
   }
-  if (!config_->GetInt("menu/page_size", &page_size_) &&
-      schema_id_ != ".default") {
-    // not defined in schema, use default setting
-    the<Config> default_config(
-        Config::Require("config")->Create("default"));
-    if (default_config) {
-      default_config->GetInt("menu/page_size", &page_size_);
-    }
-  }
+  config_->GetInt("menu/page_size", &page_size_);
   config_->GetString("menu/alternative_select_keys", &select_keys_);
 }