@openmrs/esm-config 11.0.1-pre.5776 → 11.0.1-pre.5799

package/.turbo/turbo-build.log

@@ -1,3 +1,3 @@
-[0] Successfully compiled: 8 files with swc (260.2ms)
+[0] Successfully compiled: 8 files with swc (270.54ms)
 [0] swc --strip-leading-paths src -d dist exited with code 0
 [1] tsc --project tsconfig.build.json exited with code 0

package/dist/module-config/module-config.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"module-config.d.ts","sourceRoot":"","sources":["../../src/module-config/module-config.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,MAAM,EAAE,YAAY,EAAE,YAAY,EAAuB,MAAM,UAAU,CAAC;AAkWxF;;;;;;;;;;;GAWG;AACH,wBAAgB,kBAAkB,CAAC,UAAU,EAAE,MAAM,EAAE,MAAM,EAAE,YAAY,QAS1E;AAED;;;;;;;;;GASG;AACH,wBAAgB,8BAA8B,CAAC,UAAU,EAAE,MAAM,QAKhE;AAED;;;;;;;GAOG;AACH,wBAAgB,kBAAkB,CAAC,UAAU,EAAE,MAAM,QAKpD;AAED;;;;;;;;GAQG;AACH,wBAAgB,4BAA4B,CAAC,SAAS,EAAE,MAAM,QAK7D;AAED;;;;;;;;;;;;;;;;GAgBG;AACH,wBAAgB,2BAA2B,CAAC,aAAa,EAAE,MAAM,EAAE,MAAM,EAAE,YAAY,QAetF;AAED;;;;;;;;GAQG;AACH,wBAAgB,OAAO,CAAC,MAAM,EAAE,MAAM,EAAE,UAAU,SAAa,QAK9D;AAiCD;;;;;;;;GAQG;AACH,wBAAgB,SAAS,CAAC,CAAC,GAAG,MAAM,CAAC,MAAM,EAAE,GAAG,CAAC,EAAE,UAAU,EAAE,MAAM,GAAG,OAAO,CAAC,CAAC,CAAC,CAMjF;AAED;;;;;;;;;;;;;;GAcG;AACH,wBAAgB,uBAAuB,CACrC,UAAU,EAAE,MAAM,EAClB,QAAQ,CAAC,EAAE,MAAM,EACjB,WAAW,CAAC,EAAE,MAAM,GACnB,OAAO,CAAC,KAAK,CAAC,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC,CAAC,CAAC,CA2BxD;AAED;;;;;;;;;GASG;AACH,wBAAgB,aAAa,CAAC,MAAM,EAAE,YAAY,EAAE,cAAc,EAAE,YAAY,EAAE,cAAc,EAAE,MAAM,UAKvG;AAsnBD;;;;;GAKG;AACH,wBAAgB,iBAAiB,CAAC,OAAO,CAAC,EAAE,MAAM,QAUjD;AAED;;;;;;;GAOG;AACH,wBAAgB,iBAAiB,SAYhC"}
+{"version":3,"file":"module-config.d.ts","sourceRoot":"","sources":["../../src/module-config/module-config.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,MAAM,EAAE,YAAY,EAAE,YAAY,EAAuB,MAAM,UAAU,CAAC;AAkWxF;;;;;;;;;;;GAWG;AACH,wBAAgB,kBAAkB,CAAC,UAAU,EAAE,MAAM,EAAE,MAAM,EAAE,YAAY,QAS1E;AAED;;;;;;;;;GASG;AACH,wBAAgB,8BAA8B,CAAC,UAAU,EAAE,MAAM,QAKhE;AAED;;;;;;;GAOG;AACH,wBAAgB,kBAAkB,CAAC,UAAU,EAAE,MAAM,QAMpD;AAED;;;;;;;;GAQG;AACH,wBAAgB,4BAA4B,CAAC,SAAS,EAAE,MAAM,QAK7D;AAED;;;;;;;;;;;;;;;;GAgBG;AACH,wBAAgB,2BAA2B,CAAC,aAAa,EAAE,MAAM,EAAE,MAAM,EAAE,YAAY,QAetF;AAED;;;;;;;;GAQG;AACH,wBAAgB,OAAO,CAAC,MAAM,EAAE,MAAM,EAAE,UAAU,SAAa,QAK9D;AAiCD;;;;;;;;GAQG;AACH,wBAAgB,SAAS,CAAC,CAAC,GAAG,MAAM,CAAC,MAAM,EAAE,GAAG,CAAC,EAAE,UAAU,EAAE,MAAM,GAAG,OAAO,CAAC,CAAC,CAAC,CAMjF;AAED;;;;;;;;;;;;;;GAcG;AACH,wBAAgB,uBAAuB,CACrC,UAAU,EAAE,MAAM,EAClB,QAAQ,CAAC,EAAE,MAAM,EACjB,WAAW,CAAC,EAAE,MAAM,GACnB,OAAO,CAAC,KAAK,CAAC,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC,CAAC,CAAC,CA2BxD;AAED;;;;;;;;;GASG;AACH,wBAAgB,aAAa,CAAC,MAAM,EAAE,YAAY,EAAE,cAAc,EAAE,YAAY,EAAE,cAAc,EAAE,MAAM,UAKvG;AAioBD;;;;;GAKG;AACH,wBAAgB,iBAAiB,CAAC,OAAO,CAAC,EAAE,MAAM,QAUjD;AAED;;;;;;;GAOG;AACH,wBAAgB,iBAAiB,SAYhC"}

package/dist/module-config/module-config.js

@@ -178,8 +178,8 @@ function computeExtensionConfigs(configState, extensionState, tempConfigState, s
     const configs = {};
     let changed = false;
     let entryCount = 0;
-    // We assume that the module schema has already been defined, since the extension
-    // it contains is mounted.
+    // An extension can be mounted before its module loads, so the module schema may still be the
+    // implicit one; `computeExtensionConfig` holds back structural validation until it has started.
     for (let extension of extensionState.mountedExtensions){
         const previous = oldConfigs[extension.slotName]?.[extension.extensionId];
         let entry;
@@ -286,6 +286,10 @@ function computeExtensionConfigs(configState, extensionState, tempConfigState, s
             moduleLoaded: {
                 ...state.moduleLoaded,
                 [moduleName]: true
+            },
+            moduleStarted: {
+                ...state.moduleStarted,
+                [moduleName]: true
             }
         }));
 }
@@ -455,6 +459,9 @@ const extensionConfigCache = new Map();
  */ function computeExtensionConfig(slotModuleName, extensionModuleName, slotName, extensionId, configState, tempConfigState, slotConfig) {
     const extensionName = getExtensionNameFromId(extensionId);
     const extensionConfigSchema = configState.schemas[extensionName];
+    // Not `moduleLoaded`: `defineConfigSchema` sets that while `startupApp` is still running, before it
+    // reaches `defineExtensionConfigSchema`, so the extension would be checked against the module schema.
+    const extensionModuleStarted = Boolean(configState.moduleStarted[extensionModuleName]);
     // JSON string of these elements should be unique-per-extension
     const cacheKey = JSON.stringify([
         slotName,
@@ -463,7 +470,7 @@ const extensionConfigCache = new Map();
         extensionModuleName
     ]);
     const cached = extensionConfigCache.get(cacheKey);
-    if (cached && cached.extensionSchema === extensionConfigSchema && cached.extensionModuleSchema === configState.schemas[extensionModuleName] && cached.providedConfigs === configState.providedConfigs && cached.temporaryConfig === tempConfigState.config) {
+    if (cached && cached.extensionSchema === extensionConfigSchema && cached.extensionModuleSchema === configState.schemas[extensionModuleName] && cached.extensionModuleStarted === extensionModuleStarted && cached.providedConfigs === configState.providedConfigs && cached.temporaryConfig === tempConfigState.config) {
         return cached.config;
     }
     const nameOfSchemaSource = extensionConfigSchema ? extensionName : extensionModuleName;
@@ -478,13 +485,17 @@ const extensionConfigCache = new Map();
         configOverride
     ]);
     const schema = extensionConfigSchema ?? configState.schemas[extensionModuleName];
-    validateStructure(schema, combinedConfig, nameOfSchemaSource);
+    // only validate structure once the schema has loaded
+    if (extensionConfigSchema || extensionModuleStarted) {
+        validateStructure(schema, combinedConfig, nameOfSchemaSource);
+    }
     const config = setDefaults(schema, combinedConfig);
     runAllValidatorsInConfigTree(schema, config, nameOfSchemaSource);
     delete config.extensionSlots;
     extensionConfigCache.set(cacheKey, {
         extensionSchema: extensionConfigSchema,
         extensionModuleSchema: configState.schemas[extensionModuleName],
+        extensionModuleStarted,
         providedConfigs: configState.providedConfigs,
         temporaryConfig: tempConfigState.config,
         config

package/dist/module-config/state.d.ts

@@ -16,6 +16,12 @@ export interface ConfigInternalStore {
      * whether the schema has actually been defined).
      */
     moduleLoaded: Record<string, boolean>;
+    /**
+     * Whether the app shell has finished starting each module (run its `startupApp`). Unlike
+     * `moduleLoaded`, `defineConfigSchema` does not set this, so it stays false while `startupApp`
+     * is still defining the module's extension schemas.
+     */
+    moduleStarted: Record<string, boolean>;
 }
 /**
  * @internal

package/dist/module-config/state.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"state.d.ts","sourceRoot":"","sources":["../../src/module-config/state.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,KAAK,QAAQ,EAAE,MAAM,SAAS,CAAC;AAExC,OAAO,KAAK,EAAE,MAAM,EAAE,YAAY,EAAE,YAAY,EAAE,mBAAmB,EAAE,cAAc,EAAE,MAAM,UAAU,CAAC;AAExG;;;;GAIG;AACH,MAAM,WAAW,mBAAmB;IAClC,iDAAiD;IACjD,eAAe,EAAE,KAAK,CAAC,cAAc,CAAC,CAAC;IACvC,kEAAkE;IAClE,OAAO,EAAE,MAAM,CAAC,MAAM,EAAE,YAAY,CAAC,CAAC;IACtC;;;;OAIG;IACH,YAAY,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC;CACvC;AAQD;;GAEG;AACH,eAAO,MAAM,mBAAmB,+BAG/B,CAAC;AAEF;;;;GAIG;AACH,MAAM,WAAW,oBAAoB;IACnC,MAAM,EAAE,MAAM,CAAC;CAChB;AAED,gBAAgB;AAChB,eAAO,MAAM,oBAAoB,gCAE/B,CAAC;AAsBH;;;;;;GAMG;AACH,MAAM,WAAW,oBAAoB;IACnC,iBAAiB,EAAE,KAAK,CAAC,2BAA2B,CAAC,CAAC;CACvD;AAED,gBAAgB;AAChB,MAAM,WAAW,2BAA2B;IAC1C,cAAc,EAAE,MAAM,CAAC;IACvB,mBAAmB,EAAE,MAAM,CAAC;IAC5B,QAAQ,EAAE,MAAM,CAAC;IACjB,WAAW,EAAE,MAAM,CAAC;CACrB;AAED,gBAAgB;AAChB,eAAO,MAAM,oBAAoB,gCAE/B,CAAC;AAEH;;;;;GAKG;AACH,MAAM,WAAW,WAAW;IAC1B,MAAM,EAAE,YAAY,GAAG,IAAI,CAAC;IAC5B,MAAM,EAAE,OAAO,CAAC;IAChB,0BAA0B,EAAE,OAAO,CAAC;CACrC;AAUD;;;;;;;;GAQG;AACH,wBAAgB,cAAc,CAAC,UAAU,EAAE,MAAM,yBAGhD;AAED;;;GAGG;AACH,MAAM,WAAW,yBAAyB;IACxC,KAAK,EAAE;QACL,CAAC,QAAQ,EAAE,MAAM,GAAG;YAClB,MAAM,EAAE,mBAAmB,CAAC;YAC5B,MAAM,EAAE,OAAO,CAAC;SACjB,CAAC;KACH,CAAC;CACH;AAED,gBAAgB;AAChB,wBAAgB,4BAA4B,wCAI3C;AAED,gBAAgB;AAChB,wBAAgB,sBAAsB,CAAC,QAAQ,EAAE,MAAM;YAdzC,mBAAmB;YACnB,OAAO;EAepB;AAED,gBAAgB;AAChB,wBAAgB,+BAA+B,CAAC,KAAK,EAAE,yBAAyB,EAAE,QAAQ,EAAE,MAAM;YAnBpF,mBAAmB;YACnB,OAAO;EAqBpB;AAED,gBAAgB;AAChB,MAAM,WAAW,qBAAqB;IACpC,OAAO,EAAE;QACP,CAAC,QAAQ,EAAE,MAAM,GAAG;YAClB,CAAC,WAAW,EAAE,MAAM,GAAG,WAAW,CAAC;SACpC,CAAC;KACH,CAAC;CACH;AAED;;;GAGG;AACH,wBAAgB,wBAAwB,oCAIvC;AAED,gBAAgB;AAChB,wBAAgB,2BAA2B,CAAC,KAAK,EAAE,qBAAqB,EAAE,QAAQ,EAAE,MAAM,EAAE,WAAW,EAAE,MAAM,eAG9G;AAED,gBAAgB;AAChB,wBAAgB,wCAAwC,CACtD,KAAK,EAAE,mBAAmB,EAC1B,QAAQ,EAAE,MAAM,EAChB,WAAW,EAAE,MAAM,iBAIpB;AAED;;;GAGG;AACH,MAAM,WAAW,2BAA2B;IAC1C,MAAM,EAAE,MAAM,CAAC;IACf;;;OAGG;IACH,eAAe,CAAC,EAAE,MAAM,CAAC;CAC1B;AAkBD;;;GAGG;AACH,wBAAgB,mCAAmC,CAAC,SAAS,EAAE,MAAM,IAAI,QAExE;AAED;;;GAGG;AACH,wBAAgB,gCAAgC,SAM/C;AAuBD;;;;GAIG;AACH,wBAAgB,2BAA2B,CAAC,KAAK,EAAE,OAAO,QAIzD;AAED;;;;;GAKG;AACH,wBAAgB,0BAA0B,SAIzC;AAoCD;;;;;GAKG;AACH,eAAO,MAAM,2BAA2B,uCAGvC,CAAC"}
+{"version":3,"file":"state.d.ts","sourceRoot":"","sources":["../../src/module-config/state.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,KAAK,QAAQ,EAAE,MAAM,SAAS,CAAC;AAExC,OAAO,KAAK,EAAE,MAAM,EAAE,YAAY,EAAE,YAAY,EAAE,mBAAmB,EAAE,cAAc,EAAE,MAAM,UAAU,CAAC;AAExG;;;;GAIG;AACH,MAAM,WAAW,mBAAmB;IAClC,iDAAiD;IACjD,eAAe,EAAE,KAAK,CAAC,cAAc,CAAC,CAAC;IACvC,kEAAkE;IAClE,OAAO,EAAE,MAAM,CAAC,MAAM,EAAE,YAAY,CAAC,CAAC;IACtC;;;;OAIG;IACH,YAAY,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC;IACtC;;;;OAIG;IACH,aAAa,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC;CACxC;AASD;;GAEG;AACH,eAAO,MAAM,mBAAmB,+BAG/B,CAAC;AAEF;;;;GAIG;AACH,MAAM,WAAW,oBAAoB;IACnC,MAAM,EAAE,MAAM,CAAC;CAChB;AAED,gBAAgB;AAChB,eAAO,MAAM,oBAAoB,gCAE/B,CAAC;AAsBH;;;;;;GAMG;AACH,MAAM,WAAW,oBAAoB;IACnC,iBAAiB,EAAE,KAAK,CAAC,2BAA2B,CAAC,CAAC;CACvD;AAED,gBAAgB;AAChB,MAAM,WAAW,2BAA2B;IAC1C,cAAc,EAAE,MAAM,CAAC;IACvB,mBAAmB,EAAE,MAAM,CAAC;IAC5B,QAAQ,EAAE,MAAM,CAAC;IACjB,WAAW,EAAE,MAAM,CAAC;CACrB;AAED,gBAAgB;AAChB,eAAO,MAAM,oBAAoB,gCAE/B,CAAC;AAEH;;;;;GAKG;AACH,MAAM,WAAW,WAAW;IAC1B,MAAM,EAAE,YAAY,GAAG,IAAI,CAAC;IAC5B,MAAM,EAAE,OAAO,CAAC;IAChB,0BAA0B,EAAE,OAAO,CAAC;CACrC;AAUD;;;;;;;;GAQG;AACH,wBAAgB,cAAc,CAAC,UAAU,EAAE,MAAM,yBAGhD;AAED;;;GAGG;AACH,MAAM,WAAW,yBAAyB;IACxC,KAAK,EAAE;QACL,CAAC,QAAQ,EAAE,MAAM,GAAG;YAClB,MAAM,EAAE,mBAAmB,CAAC;YAC5B,MAAM,EAAE,OAAO,CAAC;SACjB,CAAC;KACH,CAAC;CACH;AAED,gBAAgB;AAChB,wBAAgB,4BAA4B,wCAI3C;AAED,gBAAgB;AAChB,wBAAgB,sBAAsB,CAAC,QAAQ,EAAE,MAAM;YAdzC,mBAAmB;YACnB,OAAO;EAepB;AAED,gBAAgB;AAChB,wBAAgB,+BAA+B,CAAC,KAAK,EAAE,yBAAyB,EAAE,QAAQ,EAAE,MAAM;YAnBpF,mBAAmB;YACnB,OAAO;EAqBpB;AAED,gBAAgB;AAChB,MAAM,WAAW,qBAAqB;IACpC,OAAO,EAAE;QACP,CAAC,QAAQ,EAAE,MAAM,GAAG;YAClB,CAAC,WAAW,EAAE,MAAM,GAAG,WAAW,CAAC;SACpC,CAAC;KACH,CAAC;CACH;AAED;;;GAGG;AACH,wBAAgB,wBAAwB,oCAIvC;AAED,gBAAgB;AAChB,wBAAgB,2BAA2B,CAAC,KAAK,EAAE,qBAAqB,EAAE,QAAQ,EAAE,MAAM,EAAE,WAAW,EAAE,MAAM,eAG9G;AAED,gBAAgB;AAChB,wBAAgB,wCAAwC,CACtD,KAAK,EAAE,mBAAmB,EAC1B,QAAQ,EAAE,MAAM,EAChB,WAAW,EAAE,MAAM,iBAIpB;AAED;;;GAGG;AACH,MAAM,WAAW,2BAA2B;IAC1C,MAAM,EAAE,MAAM,CAAC;IACf;;;OAGG;IACH,eAAe,CAAC,EAAE,MAAM,CAAC;CAC1B;AAkBD;;;GAGG;AACH,wBAAgB,mCAAmC,CAAC,SAAS,EAAE,MAAM,IAAI,QAExE;AAED;;;GAGG;AACH,wBAAgB,gCAAgC,SAM/C;AAuBD;;;;GAIG;AACH,wBAAgB,2BAA2B,CAAC,KAAK,EAAE,OAAO,QAIzD;AAED;;;;;GAKG;AACH,wBAAgB,0BAA0B,SAIzC;AAoCD;;;;;GAKG;AACH,eAAO,MAAM,2BAA2B,uCAGvC,CAAC"}

package/dist/module-config/state.js

@@ -3,7 +3,8 @@ import { createStore } from "zustand/vanilla";
 const configInternalStoreInitialValue = {
     providedConfigs: [],
     schemas: {},
-    moduleLoaded: {}
+    moduleLoaded: {},
+    moduleStarted: {}
 };
 /**
  * @internal

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@openmrs/esm-config",
-  "version": "11.0.1-pre.5776",
+  "version": "11.0.1-pre.5799",
   "license": "MPL-2.0",
   "description": "A configuration library for the OpenMRS Single-Spa framework.",
   "type": "module",
@@ -57,15 +57,15 @@
     "lodash-es": "^4.17.21"
   },
   "peerDependencies": {
-    "@openmrs/esm-globals": "^11.0.1-pre.5776",
-    "@openmrs/esm-state": "^11.0.1-pre.5776",
-    "@openmrs/esm-utils": "^11.0.1-pre.5776",
+    "@openmrs/esm-globals": "^11.0.1-pre.5799",
+    "@openmrs/esm-state": "^11.0.1-pre.5799",
+    "@openmrs/esm-utils": "^11.0.1-pre.5799",
     "single-spa": "6.x"
   },
   "devDependencies": {
-    "@openmrs/esm-globals": "11.0.1-pre.5776",
-    "@openmrs/esm-state": "11.0.1-pre.5776",
-    "@openmrs/esm-utils": "11.0.1-pre.5776",
+    "@openmrs/esm-globals": "11.0.1-pre.5799",
+    "@openmrs/esm-state": "11.0.1-pre.5799",
+    "@openmrs/esm-utils": "11.0.1-pre.5799",
     "@swc/cli": "0.8.1",
     "@swc/core": "1.15.21",
     "@types/lodash-es": "^4.17.12",

package/src/module-config/config-derivation-failure.test.ts

@@ -18,7 +18,7 @@ describe('a configuration that cannot be derived', () => {
   beforeEach(() => {
     consoleError = vi.spyOn(console, 'error').mockImplementation(() => {});
     temporaryConfigStore.setState({ config: {} });
-    configInternalStore.setState({ providedConfigs: [], schemas: {}, moduleLoaded: {} });
+    configInternalStore.setState({ providedConfigs: [], schemas: {}, moduleLoaded: {}, moduleStarted: {} });
     implementerToolsConfigStore.setState({ config: {}, derivationError: undefined });
     resetConfigSystem();
     defineConfigSchema('esm-probe', { thing: { _type: Type.Object, _default: {} } });

package/src/module-config/implementer-tools-config.test.ts

@@ -28,7 +28,7 @@ type ImplementerToolsTree = Record<string, ConfigNode>;
 describe('implementer tools config', () => {
   beforeEach(() => {
     temporaryConfigStore.setState({ config: {} });
-    configInternalStore.setState({ providedConfigs: [], schemas: {}, moduleLoaded: {} });
+    configInternalStore.setState({ providedConfigs: [], schemas: {}, moduleLoaded: {}, moduleStarted: {} });
     resetConfigSystem();
     defineConfigSchema('esm-flintstone', {
       label: { _type: Type.String, _default: 'default-label', _description: 'x' },

package/src/module-config/module-config.test.ts

@@ -1590,6 +1590,105 @@ describe('extension config', () => {
   });
 });
 
+// `renderExtension` registers an extension before awaiting its bundle, so its config is derived
+// while the module is known to the config system only by the implicit schema.
+describe('extension config of a module that has not loaded yet', () => {
+  beforeEach(() => {
+    console.error = vi.fn();
+  });
+
+  afterEach(resetAll);
+
+  const mount = (extensionModuleName: string, extensionId: string) =>
+    configExtensionStore.setState({
+      mountedExtensions: [{ slotModuleName: 'slot-mod', extensionModuleName, slotName: 'early-slot', extensionId }],
+    });
+
+  it('does not call a key unknown before the module defines its schema', () => {
+    Config.provide({ 'early-mod': { thud: 'provided' } });
+    Config.registerModuleWithConfigSystem('early-mod');
+    mount('early-mod', 'earlyExt');
+
+    expect(console.error).not.toHaveBeenCalled();
+
+    Config.defineConfigSchema('early-mod', { thud: { _default: 'default' } });
+    Config.registerModuleLoad('early-mod');
+
+    expect(console.error).not.toHaveBeenCalled();
+    expect(currentExtensionConfig('early-slot', 'earlyExt').config.thud).toBe('provided');
+  });
+
+  it('still calls a key unknown once the module has defined a schema without it', () => {
+    Config.provide({ 'late-mod': { thud: 'provided', plugh: 'unknown' } });
+    Config.registerModuleWithConfigSystem('late-mod');
+    mount('late-mod', 'lateExt');
+
+    expect(console.error).not.toHaveBeenCalled();
+
+    Config.defineConfigSchema('late-mod', { thud: { _default: 'default' } });
+    Config.registerModuleLoad('late-mod');
+
+    expect(console.error).toHaveBeenCalledTimes(1);
+    expect(console.error).toHaveBeenCalledWith(expect.stringMatching(/unknown config key 'late-mod.plugh' provided/i));
+  });
+
+  it('still calls a key unknown once a module without a schema has loaded', () => {
+    // Configured through the slot so that only the extension config, not the module config, sees it.
+    Config.provide({
+      'slot-mod': { extensionSlots: { 'early-slot': { configure: { bareExt: { xyzzy: 'unknown' } } } } },
+    });
+    Config.registerModuleWithConfigSystem('bare-mod');
+    mount('bare-mod', 'bareExt');
+
+    expect(console.error).not.toHaveBeenCalled();
+
+    Config.registerModuleLoad('bare-mod');
+
+    expect(console.error).toHaveBeenCalledWith(expect.stringMatching(/unknown config key 'bare-mod.xyzzy' provided/i));
+  });
+
+  it('still reports a value of the wrong type once the module has defined its schema', () => {
+    Config.provide({ 'typed-mod': { thud: 42 } });
+    Config.registerModuleWithConfigSystem('typed-mod');
+    mount('typed-mod', 'typedExt');
+
+    expect(console.error).not.toHaveBeenCalled();
+
+    // `defineConfigSchema` marks the module loaded, so the module config reports both errors here. The
+    // extension's own structure check waits for `registerModuleLoad`, and its repeats are deduplicated.
+    Config.defineConfigSchema('typed-mod', {
+      thud: { _type: Type.String, _default: 'default', _validators: [validator((v) => v !== 42, 'must not be 42')] },
+    });
+    Config.registerModuleLoad('typed-mod');
+
+    expect(console.error).toHaveBeenCalledTimes(2);
+    expect(console.error).toHaveBeenCalledWith(expect.stringMatching(/42 for typed-mod.thud: must be a string/));
+    expect(console.error).toHaveBeenCalledWith(expect.stringMatching(/42 for typed-mod.thud: must not be 42/));
+  });
+
+  it('does not call an extension schema key unknown while its module starts up', () => {
+    Config.provide({
+      'slot-mod': { extensionSlots: { 'early-slot': { configure: { flagsExt: { quux: 'x', corge: 'unknown' } } } } },
+    });
+    Config.registerModuleWithConfigSystem('flags-mod');
+    mount('flags-mod', 'flagsExt');
+    Config.defineConfigSchema('flags-mod', { thud: { _default: 'd' } });
+
+    expect(console.error).not.toHaveBeenCalled();
+
+    // The extension's own schema is checked as soon as it is defined, without waiting for startup.
+    Config.defineExtensionConfigSchema('flagsExt', { quux: { _default: 'd' } });
+
+    expect(console.error).toHaveBeenCalledTimes(1);
+    expect(console.error).toHaveBeenCalledWith(expect.stringMatching(/unknown config key 'flagsExt.corge' provided/i));
+    expect(currentExtensionConfig('early-slot', 'flagsExt').config.quux).toBe('x');
+
+    Config.registerModuleLoad('flags-mod');
+
+    expect(console.error).toHaveBeenCalledTimes(1);
+  });
+});
+
 describe('translation overrides', () => {
   it('allows obtaining translation overrides before schema is registered', async () => {
     console.error = vi.fn();

package/src/module-config/module-config.ts

@@ -275,8 +275,8 @@ function computeExtensionConfigs(
   let changed = false;
   let entryCount = 0;
 
-  // We assume that the module schema has already been defined, since the extension
-  // it contains is mounted.
+  // An extension can be mounted before its module loads, so the module schema may still be the
+  // implicit one; `computeExtensionConfig` holds back structural validation until it has started.
   for (let extension of extensionState.mountedExtensions) {
     const previous = oldConfigs[extension.slotName]?.[extension.extensionId];
     let entry: ConfigStore | { config: ConfigObject; loaded: true };
@@ -406,6 +406,7 @@ export function registerModuleLoad(moduleName: string) {
   configInternalStore.setState((state) => ({
     ...state,
     moduleLoaded: { ...state.moduleLoaded, [moduleName]: true },
+    moduleStarted: { ...state.moduleStarted, [moduleName]: true },
   }));
 }
 
@@ -602,6 +603,7 @@ export function processConfig(schema: ConfigSchema, providedConfig: ConfigObject
 interface ExtensionConfigCacheEntry {
   extensionSchema: ConfigSchema | undefined;
   extensionModuleSchema: ConfigSchema | undefined;
+  extensionModuleStarted: boolean;
   providedConfigs: ConfigInternalStore['providedConfigs'];
   temporaryConfig: Config;
   config: ConfigObject;
@@ -634,6 +636,9 @@ function computeExtensionConfig(
 ) {
   const extensionName = getExtensionNameFromId(extensionId);
   const extensionConfigSchema = configState.schemas[extensionName];
+  // Not `moduleLoaded`: `defineConfigSchema` sets that while `startupApp` is still running, before it
+  // reaches `defineExtensionConfigSchema`, so the extension would be checked against the module schema.
+  const extensionModuleStarted = Boolean(configState.moduleStarted[extensionModuleName]);
   // JSON string of these elements should be unique-per-extension
   const cacheKey = JSON.stringify([slotName, extensionId, slotModuleName, extensionModuleName]);
   const cached = extensionConfigCache.get(cacheKey);
@@ -642,6 +647,7 @@ function computeExtensionConfig(
     cached &&
     cached.extensionSchema === extensionConfigSchema &&
     cached.extensionModuleSchema === configState.schemas[extensionModuleName] &&
+    cached.extensionModuleStarted === extensionModuleStarted &&
     cached.providedConfigs === configState.providedConfigs &&
     cached.temporaryConfig === tempConfigState.config
   ) {
@@ -656,7 +662,12 @@ function computeExtensionConfig(
   const extensionConfig = mergeConfigsFor(nameOfSchemaSource, providedConfigs);
   const combinedConfig = mergeConfigs([extensionConfig, sharedOverride, configOverride]);
   const schema = extensionConfigSchema ?? configState.schemas[extensionModuleName];
-  validateStructure(schema, combinedConfig, nameOfSchemaSource);
+
+  // only validate structure once the schema has loaded
+  if (extensionConfigSchema || extensionModuleStarted) {
+    validateStructure(schema, combinedConfig, nameOfSchemaSource);
+  }
+
   const config = setDefaults(schema, combinedConfig);
   runAllValidatorsInConfigTree(schema, config, nameOfSchemaSource);
   delete config.extensionSlots;
@@ -664,6 +675,7 @@ function computeExtensionConfig(
   extensionConfigCache.set(cacheKey, {
     extensionSchema: extensionConfigSchema,
     extensionModuleSchema: configState.schemas[extensionModuleName],
+    extensionModuleStarted,
     providedConfigs: configState.providedConfigs,
     temporaryConfig: tempConfigState.config,
     config,

package/src/module-config/state.ts

@@ -19,12 +19,19 @@ export interface ConfigInternalStore {
    * whether the schema has actually been defined).
    */
   moduleLoaded: Record<string, boolean>;
+  /**
+   * Whether the app shell has finished starting each module (run its `startupApp`). Unlike
+   * `moduleLoaded`, `defineConfigSchema` does not set this, so it stays false while `startupApp`
+   * is still defining the module's extension schemas.
+   */
+  moduleStarted: Record<string, boolean>;
 }
 
 const configInternalStoreInitialValue = {
   providedConfigs: [],
   schemas: {},
   moduleLoaded: {},
+  moduleStarted: {},
 };
 
 /**