Improve configuration provider initialization routine (#1402)

Author: lars-reimann

examples/arithmetics/src/language-server/arithmetics-module.ts

@@ -69,5 +69,10 @@ export function createArithmeticsServices(context: DefaultSharedModuleContext):
     );
     shared.ServiceRegistry.register(arithmetics);
     registerValidationChecks(arithmetics);
+    if (!context.connection) {
+        // We don't run inside a language server
+        // Therefore, initialize the configuration provider instantly
+        shared.workspace.ConfigurationProvider.initialized({});
+    }
     return { shared, arithmetics };
 }

examples/domainmodel/src/language-server/domain-model-module.ts

@@ -54,5 +54,10 @@ export function createDomainModelServices(context: DefaultSharedModuleContext):
     );
     shared.ServiceRegistry.register(domainmodel);
     registerValidationChecks(domainmodel);
+    if (!context.connection) {
+        // We don't run inside a language server
+        // Therefore, initialize the configuration provider instantly
+        shared.workspace.ConfigurationProvider.initialized({});
+    }
     return { shared, domainmodel };
 }

examples/requirements/src/language-server/requirements-and-tests-lang-module.ts

@@ -52,5 +52,10 @@ export function createRequirementsAndTestsLangServices(context: DefaultSharedMod
     shared.ServiceRegistry.register(tests);
     registerRequirementsValidationChecks(requirements);
     registerTestsValidationChecks(tests);
+    if (!context.connection) {
+        // We don't run inside a language server
+        // Therefore, initialize the configuration provider instantly
+        shared.workspace.ConfigurationProvider.initialized({});
+    }
     return { shared, requirements, tests };
 }

examples/statemachine/src/language-server/statemachine-module.ts

@@ -66,5 +66,10 @@ export function createStatemachineServices(context: DefaultSharedModuleContext):
     );
     shared.ServiceRegistry.register(statemachine);
     registerValidationChecks(statemachine);
+    if (!context.connection) {
+        // We don't run inside a language server
+        // Therefore, initialize the configuration provider instantly
+        shared.workspace.ConfigurationProvider.initialized({});
+    }
     return { shared, statemachine };
 }

packages/generator-langium/templates/core/src/language/language-id-module.ts

@@ -59,5 +59,10 @@ export function create<%= LanguageName %>Services(context: DefaultSharedModuleCo
     );
     shared.ServiceRegistry.register(<%= LanguageName %>);
     registerValidationChecks(<%= LanguageName %>);
+    if (!context.connection) {
+        // We don't run inside a language server
+        // Therefore, initialize the configuration provider instantly
+        shared.workspace.ConfigurationProvider.initialized({});
+    }
     return { shared, <%= LanguageName %> };
 }

packages/langium/src/grammar/langium-grammar-module.ts

@@ -93,6 +93,12 @@ export function createLangiumGrammarServices(context: DefaultSharedModuleContext
     registerValidationChecks(grammar);
     registerTypeValidationChecks(grammar);
 
+    if (!context.connection) {
+        // We don't run inside a language server
+        // Therefore, initialize the configuration provider instantly
+        shared.workspace.ConfigurationProvider.initialized({});
+    }
+
     return { shared, grammar };
 }
 

packages/langium/test/workspace/configuration.test.ts

@@ -14,7 +14,6 @@ describe('ConfigurationProvider', () => {
     const grammarServices = createLangiumGrammarServices(EmptyFileSystem).grammar;
     const langId = grammarServices.LanguageMetaData.languageId;
     const configs = grammarServices.shared.workspace.ConfigurationProvider;
-    configs.initialized({});
     beforeEach(() => {
         (configs as any).settings = {};
     });