@openmrs/esm-config 10.0.1-pre.5738 → 10.0.1-pre.5741

package/.turbo/turbo-build.log

@@ -1,3 +1,3 @@
-[0] Successfully compiled: 8 files with swc (172.56ms)
+[0] Successfully compiled: 8 files with swc (143.43ms)
 [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

@@ -89,7 +89,9 @@ export declare function getConfig<T = Record<string, any>>(moduleName: string):
  * @param moduleName The name of the module providing the translation context.
  * @param slotName Optional extension slot name to include slot-specific overrides.
  * @param extensionId Optional extension ID to include extension-specific overrides.
- * @returns A Promise resolving to an array of translation override objects.
+ * @returns A Promise resolving to an array of translation override objects. It always settles: a
+ *   module the config system has not been told about, or an extension that is not mounted,
+ *   contributes no overrides.
  *
  * @internal
  */

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;AA4VxF;;;;;;;;;;;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;;;;;;;;;;;;GAYG;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,CAoBxD;AAED;;;;;;;;;GASG;AACH,wBAAgB,aAAa,CAAC,MAAM,EAAE,YAAY,EAAE,cAAc,EAAE,YAAY,EAAE,cAAc,EAAE,MAAM,UAKvG;AA6nBD;;;;;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,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"}

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

@@ -2,7 +2,7 @@
 import { Type } from "../types.js";
 import { isArray, isBoolean, isUuid, isNumber, isObject, isString } from "../validators/type-validators.js";
 import { validator } from "../validators/validator.js";
-import { configExtensionStore, configInternalStore, getConfigStore, getExtensionConfig, getExtensionSlotsConfigStore, getExtensionsConfigStore, clearConfigDerivationError, implementerToolsConfigStore, invalidateImplementerToolsConfig, recordConfigDerivationError, setImplementerToolsConfigRecomputer, temporaryConfigStore } from "./state.js";
+import { configExtensionStore, configInternalStore, getConfigStore, getExtensionConfigFromStore, getExtensionSlotsConfigStore, getExtensionsConfigStore, clearConfigDerivationError, implementerToolsConfigStore, invalidateImplementerToolsConfig, recordConfigDerivationError, setImplementerToolsConfigRecomputer, temporaryConfigStore } from "./state.js";
 /**
  * Deep merge two objects, with right-side values taking precedence.
  * Arrays are replaced entirely (not merged by index), matching the
@@ -36,6 +36,9 @@ function mergeDeepReplace(left, right) {
  * which creates stores.
  */ // Store unsubscribe functions to allow cleanup (e.g., in tests or hot module reloading)
 const configSubscriptions = [];
+/**
+ * Track displayed validation messages so they only get displayed once.
+ */ const displayedValidationMessages = new Set();
 /**
  * Recomputes all configuration derived stores based on current state of input stores.
  * Called whenever any input store (configInternalStore, temporaryConfigStore, configExtensionStore) changes.
@@ -73,8 +76,9 @@ const configSubscriptions = [];
 function setupConfigSubscriptions() {
     // Registered here rather than at module load so that `resetConfigSystem` restores it.
     setImplementerToolsConfigRecomputer(()=>computeImplementerToolsConfig(configInternalStore.getState(), temporaryConfigStore.getState()));
-    // Initial computation
-    recomputeAllConfigs();
+    // Guarded: a configuration the user saved through the implementer tools must not be able to stop
+    // the framework from loading.
+    recomputeAllConfigsSafely();
     // Subscribe to all input stores with a single handler
     // This ensures we only recompute once even if multiple stores change simultaneously
     configSubscriptions.push(configInternalStore.subscribe(recomputeAllConfigsSafely), temporaryConfigStore.subscribe(recomputeAllConfigsSafely), configExtensionStore.subscribe(recomputeAllConfigsSafely));
@@ -398,15 +402,24 @@ function computeExtensionConfigs(configState, extensionState, tempConfigState, s
  * @param moduleName The name of the module providing the translation context.
  * @param slotName Optional extension slot name to include slot-specific overrides.
  * @param extensionId Optional extension ID to include extension-specific overrides.
- * @returns A Promise resolving to an array of translation override objects.
+ * @returns A Promise resolving to an array of translation override objects. It always settles: a
+ *   module the config system has not been told about, or an extension that is not mounted,
+ *   contributes no overrides.
  *
  * @internal
  */ export function getTranslationOverrides(moduleName, slotName, extensionId) {
+    // A module's store only ever loads its overrides once the module is registered with the config
+    // system, so waiting on one that is not registered would leave the i18next namespace pending.
     const promises = [
-        whenStoreReady(getConfigStore(moduleName), (state)=>Boolean(state.translationOverridesLoaded && state.config), (state)=>state.config['Translation overrides'] ?? {})
+        moduleName in configInternalStore.getState().schemas ? whenStoreReady(getConfigStore(moduleName), (state)=>Boolean(state.translationOverridesLoaded && state.config), (state)=>state.config['Translation overrides'] ?? {}) : Promise.resolve({})
     ];
     if (slotName && extensionId) {
-        promises.push(whenStoreReady(getExtensionConfig(slotName, extensionId), (state)=>Boolean(state.loaded && state.config), (state)=>state.config['Translation overrides'] ?? {}));
+        // Read once rather than waited for: an extension's config entry exists only while that extension
+        // is registered as mounted, so a wait started in the gap never settles and leaves the i18next
+        // namespace pending.
+        const extensionConfig = getExtensionConfigFromStore(getExtensionsConfigStore().getState(), slotName, extensionId);
+        const overrides = extensionConfig.loaded && extensionConfig.config ? extensionConfig.config['Translation overrides'] ?? {} : {};
+        promises.push(Promise.resolve(overrides));
     }
     return Promise.all(promises);
 }
@@ -920,13 +933,8 @@ function hasObjectSchema(elementsSchema) {
 function isOrdinaryObject(value) {
     return typeof value === 'object' && !Array.isArray(value) && value !== null;
 }
-/** Keep track of which validation errors we have displayed. Each one should only be displayed once. */ let displayedValidationMessages = new Set();
 function logError(keyPath, message) {
     const key = `${keyPath}:::${message}`;
-    // technically, this should not be possible, but because of how things wind-up transpiled, this isn't impossible
-    if (!displayedValidationMessages) {
-        displayedValidationMessages = new Set();
-    }
     if (!displayedValidationMessages.has(key)) {
         console.error(message);
         displayedValidationMessages.add(key);

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

@@ -108,19 +108,6 @@ export interface ExtensionsConfigStore {
  * @internal
  */
 export declare function getExtensionsConfigStore(): StoreApi<ExtensionsConfigStore>;
-/**
- * The read-only part of a store's API.
- * @internal
- */
-export type ReadableStore<T> = Pick<StoreApi<T>, 'getInitialState' | 'getState' | 'subscribe'>;
-/**
- * A read-only view of one extension instance's config within a slot. Read-only because these
- * configs are derived from the config system's inputs, so a write here would be overwritten by the
- * next recomputation; provide a config or set a temporary one instead.
- *
- * @internal
- */
-export declare function getExtensionConfig(slotName: string, extensionId: string): ReadableStore<Omit<ConfigStore, 'translationOverridesLoaded'>>;
 /** @internal */
 export declare function getExtensionConfigFromStore(state: ExtensionsConfigStore, slotName: string, extensionId: string): ConfigStore;
 /** @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":"AAEA,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;;;GAGG;AACH,MAAM,MAAM,aAAa,CAAC,CAAC,IAAI,IAAI,CAAC,QAAQ,CAAC,CAAC,CAAC,EAAE,iBAAiB,GAAG,UAAU,GAAG,WAAW,CAAC,CAAC;AAE/F;;;;;;GAMG;AACH,wBAAgB,kBAAkB,CAChC,QAAQ,EAAE,MAAM,EAChB,WAAW,EAAE,MAAM,GAClB,aAAa,CAAC,IAAI,CAAC,WAAW,EAAE,4BAA4B,CAAC,CAAC,CAmChE;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;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"}

package/dist/module-config/state.js

@@ -1,5 +1,4 @@
 import { createGlobalStore, getGlobalStore, registerGlobalStore } from "@openmrs/esm-state";
-import { shallowEqual } from "@openmrs/esm-utils";
 import { createStore } from "zustand/vanilla";
 const configInternalStoreInitialValue = {
     providedConfigs: [],
@@ -74,39 +73,6 @@ function initializeConfigStore() {
         configs: {}
     });
 }
-/**
- * A read-only view of one extension instance's config within a slot. Read-only because these
- * configs are derived from the config system's inputs, so a write here would be overwritten by the
- * next recomputation; provide a config or set a temporary one instead.
- *
- * @internal
- */ export function getExtensionConfig(slotName, extensionId) {
-    if (typeof slotName !== 'string' || typeof extensionId !== 'string' || slotName === '__proto__' || extensionId === '__proto__' || slotName === 'constructor' || extensionId === 'constructor' || slotName === 'prototype' || extensionId === 'prototype') {
-        throw new Error('Attempted to call `getExtensionConfig()` with invalid argument');
-    }
-    const extensionConfigStore = getExtensionsConfigStore();
-    const selector = (configStore)=>configStore.configs[slotName]?.[extensionId];
-    return {
-        getInitialState () {
-            return selector(extensionConfigStore.getInitialState());
-        },
-        getState () {
-            return selector(extensionConfigStore.getState()) ?? {
-                loaded: false,
-                config: null
-            };
-        },
-        subscribe (listener) {
-            return extensionConfigStore.subscribe((state, prevState)=>{
-                const newState = selector(state);
-                const oldState = selector(prevState);
-                if (!shallowEqual(newState, oldState)) {
-                    listener(newState, oldState);
-                }
-            });
-        }
-    };
-}
 /** @internal */ export function getExtensionConfigFromStore(state, slotName, extensionId) {
     const extensionConfig = state.configs[slotName]?.[extensionId];
     return extensionConfig ?? {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@openmrs/esm-config",
-  "version": "10.0.1-pre.5738",
+  "version": "10.0.1-pre.5741",
   "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": "^10.0.1-pre.5738",
-    "@openmrs/esm-state": "^10.0.1-pre.5738",
-    "@openmrs/esm-utils": "^10.0.1-pre.5738",
+    "@openmrs/esm-globals": "^10.0.1-pre.5741",
+    "@openmrs/esm-state": "^10.0.1-pre.5741",
+    "@openmrs/esm-utils": "^10.0.1-pre.5741",
     "single-spa": "6.x"
   },
   "devDependencies": {
-    "@openmrs/esm-globals": "10.0.1-pre.5738",
-    "@openmrs/esm-state": "10.0.1-pre.5738",
-    "@openmrs/esm-utils": "10.0.1-pre.5738",
+    "@openmrs/esm-globals": "10.0.1-pre.5741",
+    "@openmrs/esm-state": "10.0.1-pre.5741",
+    "@openmrs/esm-utils": "10.0.1-pre.5741",
     "@swc/cli": "0.8.1",
     "@swc/core": "1.15.21",
     "@types/lodash-es": "^4.17.12",

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

@@ -10,7 +10,8 @@ import {
   configExtensionStore,
   configInternalStore,
   getConfigStore,
-  getExtensionConfig,
+  getExtensionConfigFromStore,
+  getExtensionsConfigStore,
   implementerToolsConfigStore,
   temporaryConfigStore,
 } from './state';
@@ -1314,7 +1315,7 @@ describe('extension config', () => {
           extensionId,
         })),
       });
-    const bar = (extensionId: string) => getExtensionConfig('barSlot', extensionId).getState().config.bar;
+    const bar = (extensionId: string) => currentExtensionConfig('barSlot', extensionId).config.bar;
 
     mount(['fooExt#a']);
     expect(bar('fooExt#a')).toBe('barry');
@@ -1343,7 +1344,7 @@ describe('extension config', () => {
           { slotModuleName: 'slot-mod', extensionModuleName, slotName: 'barSlot', extensionId: 'fooExt' },
         ],
       });
-    const bar = () => getExtensionConfig('barSlot', 'fooExt').getState().config.bar;
+    const bar = () => currentExtensionConfig('barSlot', 'fooExt').config.bar;
 
     mount('ext-mod');
     expect(bar()).toBe('barry');
@@ -1356,7 +1357,7 @@ describe('extension config', () => {
     const moduleLevelConfig = { 'ext-mod': { bar: 'qux' } };
     updateConfigExtensionStore();
     Config.provide(moduleLevelConfig);
-    const result = getExtensionConfig('barSlot', 'fooExt').getState().config;
+    const result = currentExtensionConfig('barSlot', 'fooExt').config;
     expect(result).toStrictEqual({
       bar: 'qux',
       baz: 'bazzy',
@@ -1379,7 +1380,7 @@ describe('extension config', () => {
       },
     };
     Config.provide(configureConfig);
-    const result = getExtensionConfig('barSlot', 'fooExt#id0').getState().config;
+    const result = currentExtensionConfig('barSlot', 'fooExt#id0').config;
     expect(result).toStrictEqual({
       bar: 'qux',
       baz: 'quiz',
@@ -1404,7 +1405,7 @@ describe('extension config', () => {
       },
     };
     Config.provide(configureConfig);
-    const result = getExtensionConfig('barSlot', 'fooExt#id6').getState().config;
+    const result = currentExtensionConfig('barSlot', 'fooExt#id6').config;
     expect(result).toStrictEqual({
       bar: 'qux',
       baz: 'quiz',
@@ -1433,7 +1434,7 @@ describe('extension config', () => {
       },
     };
     Config.provide(configureConfig);
-    const result = getExtensionConfig('barSlot', 'fooExt#id7').getState().config;
+    const result = currentExtensionConfig('barSlot', 'fooExt#id7').config;
     expect(result).toStrictEqual({
       bar: 'from-other',
       baz: 'from-owner',
@@ -1461,7 +1462,7 @@ describe('extension config', () => {
     // Composing `add` across modules would resurrect entries another module had removed, so the
     // last module to name the slot still supplies it whole.
     expect(getExtensionSlotsConfigStore().getState().slots['barSlot'].config.add).toStrictEqual(['fooExt#id8']);
-    expect(getExtensionConfig('barSlot', 'fooExt#id8').getState().config).toStrictEqual({
+    expect(currentExtensionConfig('barSlot', 'fooExt#id8').config).toStrictEqual({
       bar: 'from-first',
       baz: 'from-last',
       'Display conditions': { expression: undefined, privileges: [] },
@@ -1474,12 +1475,12 @@ describe('extension config', () => {
     temporaryConfigStore.setState({
       config: { 'other-mod': { extensionSlots: { barSlot: { configure: { 'fooExt#id9': { baz: 'temporary' } } } } } },
     });
-    expect(getExtensionConfig('barSlot', 'fooExt#id9').getState().config.baz).toBe('temporary');
+    expect(currentExtensionConfig('barSlot', 'fooExt#id9').config.baz).toBe('temporary');
 
     // The derived slot config is not part of the extension config cache key, so a released override
     // is only picked up because that cache also keys on the temporary config it was derived from.
     temporaryConfigStore.setState({ config: {} });
-    expect(getExtensionConfig('barSlot', 'fooExt#id9').getState().config.baz).toBe('bazzy');
+    expect(currentExtensionConfig('barSlot', 'fooExt#id9').config.baz).toBe('bazzy');
   });
 
   it('validates the extension configure config, with module config schema', () => {
@@ -1506,7 +1507,7 @@ describe('extension config', () => {
     Config.registerModuleLoad('fooExt');
     const extensionAtBaseConfig = { fooExt: { qux: 'quxolotl' } };
     Config.provide(extensionAtBaseConfig);
-    const result = getExtensionConfig('barSlot', 'fooExt').getState().config;
+    const result = currentExtensionConfig('barSlot', 'fooExt').config;
     expect(result).toStrictEqual({
       qux: 'quxolotl',
       'Display conditions': { expression: undefined, privileges: [] },
@@ -1531,7 +1532,7 @@ describe('extension config', () => {
       },
     };
     Config.provide(configureConfig);
-    const result = getExtensionConfig('barSlot', 'fooExt#id2').getState().config;
+    const result = currentExtensionConfig('barSlot', 'fooExt#id2').config;
     expect(result).toStrictEqual({
       qux: 'quxotic',
       'Display conditions': { expression: undefined, privileges: [] },
@@ -1651,6 +1652,18 @@ describe('promise-based config accessors', () => {
     return counter;
   }
 
+  /**
+   * Rejects if `promise` has not settled within half a second. i18next never retries a namespace
+   * read, so a translation-override wait that never settles leaves every component under that
+   * namespace suspended for good; this turns that hang into a failure.
+   */
+  function settled<T>(promise: Promise<T>) {
+    return Promise.race([
+      promise,
+      new Promise<never>((_, reject) => setTimeout(() => reject(new Error('never settled')), 500)),
+    ]);
+  }
+
   it('getConfig does not subscribe at all when the config has already loaded', async () => {
     Config.defineConfigSchema('leak-module', { foo: { _default: 'qux' } });
     Config.registerModuleLoad('leak-module');
@@ -1692,8 +1705,46 @@ describe('promise-based config accessors', () => {
 
     expect(subscriptions.live).toBe(0);
   });
+
+  it('getTranslationOverrides settles for an extension that is not mounted', async () => {
+    Config.defineConfigSchema('unmounted-ext-module', { foo: { _default: 'qux' } });
+    Config.registerModuleLoad('unmounted-ext-module');
+
+    const overrides = Config.getTranslationOverrides('unmounted-ext-module', 'absent-slot', 'absent-ext');
+
+    await expect(settled(overrides)).resolves.toStrictEqual([{}, {}]);
+  });
+
+  it('getTranslationOverrides settles for a module the config system has not been told about', async () => {
+    const subscriptions = countSubscriptions('unregistered-module');
+
+    await expect(settled(Config.getTranslationOverrides('unregistered-module'))).resolves.toStrictEqual([{}]);
+    expect(subscriptions.live).toBe(0);
+  });
+
+  it("getTranslationOverrides returns a mounted extension's overrides", async () => {
+    Config.defineConfigSchema('ext-mod', {});
+    Config.registerModuleLoad('ext-mod');
+    Config.provide({
+      'slot-mod': {
+        extensionSlots: {
+          barSlot: { configure: { fooExt: { 'Translation overrides': { fr: { hello: 'Bonjour' } } } } },
+        },
+      },
+    });
+    updateConfigExtensionStore('fooExt');
+
+    await expect(Config.getTranslationOverrides('ext-mod', 'barSlot', 'fooExt')).resolves.toStrictEqual([
+      {},
+      { fr: { hello: 'Bonjour' } },
+    ]);
+  });
 });
 
+function currentExtensionConfig(slotName: string, extensionId: string) {
+  return getExtensionConfigFromStore(getExtensionsConfigStore().getState(), slotName, extensionId);
+}
+
 function updateConfigExtensionStore(extensionId = 'fooExt') {
   configExtensionStore.setState({
     mountedExtensions: [

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

@@ -16,7 +16,7 @@ import {
   configExtensionStore,
   configInternalStore,
   getConfigStore,
-  getExtensionConfig,
+  getExtensionConfigFromStore,
   getExtensionSlotsConfigStore,
   getExtensionsConfigStore,
   clearConfigDerivationError,
@@ -66,6 +66,11 @@ function mergeDeepReplace<T extends object>(left: T, right: Partial<T>): T {
 // Store unsubscribe functions to allow cleanup (e.g., in tests or hot module reloading)
 const configSubscriptions: Array<() => void> = [];
 
+/**
+ * Track displayed validation messages so they only get displayed once.
+ */
+const displayedValidationMessages = new Set<string>();
+
 /**
  * Recomputes all configuration derived stores based on current state of input stores.
  * Called whenever any input store (configInternalStore, temporaryConfigStore, configExtensionStore) changes.
@@ -119,8 +124,9 @@ function setupConfigSubscriptions() {
     computeImplementerToolsConfig(configInternalStore.getState(), temporaryConfigStore.getState()),
   );
 
-  // Initial computation
-  recomputeAllConfigs();
+  // Guarded: a configuration the user saved through the implementer tools must not be able to stop
+  // the framework from loading.
+  recomputeAllConfigsSafely();
 
   // Subscribe to all input stores with a single handler
   // This ensures we only recompute once even if multiple stores change simultaneously
@@ -526,7 +532,9 @@ export function getConfig<T = Record<string, any>>(moduleName: string): Promise<
  * @param moduleName The name of the module providing the translation context.
  * @param slotName Optional extension slot name to include slot-specific overrides.
  * @param extensionId Optional extension ID to include extension-specific overrides.
- * @returns A Promise resolving to an array of translation override objects.
+ * @returns A Promise resolving to an array of translation override objects. It always settles: a
+ *   module the config system has not been told about, or an extension that is not mounted,
+ *   contributes no overrides.
  *
  * @internal
  */
@@ -535,22 +543,29 @@ export function getTranslationOverrides(
   slotName?: string,
   extensionId?: string,
 ): Promise<Array<Record<string, Record<string, string>>>> {
+  // A module's store only ever loads its overrides once the module is registered with the config
+  // system, so waiting on one that is not registered would leave the i18next namespace pending.
   const promises = [
-    whenStoreReady(
-      getConfigStore(moduleName),
-      (state) => Boolean(state.translationOverridesLoaded && state.config),
-      (state) => (state.config!['Translation overrides'] ?? {}) as Record<string, Record<string, string>>,
-    ),
+    moduleName in configInternalStore.getState().schemas
+      ? whenStoreReady(
+          getConfigStore(moduleName),
+          (state) => Boolean(state.translationOverridesLoaded && state.config),
+          (state) => (state.config!['Translation overrides'] ?? {}) as Record<string, Record<string, string>>,
+        )
+      : Promise.resolve({}),
   ];
 
   if (slotName && extensionId) {
-    promises.push(
-      whenStoreReady(
-        getExtensionConfig(slotName, extensionId),
-        (state) => Boolean(state.loaded && state.config),
-        (state) => (state.config!['Translation overrides'] ?? {}) as Record<string, Record<string, string>>,
-      ),
-    );
+    // Read once rather than waited for: an extension's config entry exists only while that extension
+    // is registered as mounted, so a wait started in the gap never settles and leaves the i18next
+    // namespace pending.
+    const extensionConfig = getExtensionConfigFromStore(getExtensionsConfigStore().getState(), slotName, extensionId);
+    const overrides =
+      extensionConfig.loaded && extensionConfig.config
+        ? ((extensionConfig.config['Translation overrides'] ?? {}) as Record<string, Record<string, string>>)
+        : {};
+
+    promises.push(Promise.resolve(overrides));
   }
 
   return Promise.all(promises);
@@ -1192,15 +1207,8 @@ function hasObjectSchema(elementsSchema: unknown): elementsSchema is ConfigSchem
 function isOrdinaryObject(value) {
   return typeof value === 'object' && !Array.isArray(value) && value !== null;
 }
-/** Keep track of which validation errors we have displayed. Each one should only be displayed once. */
-let displayedValidationMessages = new Set<string>();
-
 function logError(keyPath: string, message: string) {
   const key = `${keyPath}:::${message}`;
-  // technically, this should not be possible, but because of how things wind-up transpiled, this isn't impossible
-  if (!displayedValidationMessages) {
-    displayedValidationMessages = new Set<string>();
-  }
 
   if (!displayedValidationMessages.has(key)) {
     console.error(message);

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

@@ -0,0 +1,47 @@
+// The config system validates the temporary config restored from local storage while the module is
+// still initializing. This covers that path, which is the only one where `logError` can run before the
+// rest of the module body has been evaluated.
+import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
+
+const temporaryConfigKey = 'openmrs:temporaryConfig';
+
+beforeEach(() => {
+  vi.resetModules();
+  localStorage.clear();
+});
+
+afterEach(() => {
+  localStorage.clear();
+  vi.restoreAllMocks();
+});
+
+describe('loading with a temporary config saved by the implementer tools', () => {
+  it('reports an invalid extension slot config instead of failing to load', async () => {
+    const error = vi.spyOn(console, 'error').mockImplementation(() => {});
+    localStorage.setItem(
+      temporaryConfigKey,
+      JSON.stringify({ '@openmrs/esm-some-app': { extensionSlots: { 'a-slot': { notAKey: ['x'] } } } }),
+    );
+
+    // The failure this covers happened here, at module evaluation: validation ran before the module
+    // body reached the declaration `logError` reads, so the whole framework failed to initialize with
+    // "can't access lexical declaration 'displayedValidationMessages' before initialization".
+    await expect(import('./module-config')).resolves.toBeDefined();
+
+    expect(error.mock.calls.flat().join(' ')).toContain("contains invalid keys 'notAKey'");
+  });
+
+  it('still loads when the temporary config is malformed in a way validation cannot describe', async () => {
+    vi.spyOn(console, 'error').mockImplementation(() => {});
+    localStorage.setItem(
+      temporaryConfigKey,
+      JSON.stringify({ '@openmrs/esm-some-app': { extensionSlots: { 'a-slot': { add: 'not-an-array' } } } }),
+    );
+
+    await expect(import('./module-config')).resolves.toBeDefined();
+  });
+
+  it('loads normally when there is no temporary config', async () => {
+    await expect(import('./module-config')).resolves.toBeDefined();
+  });
+});

package/src/module-config/state.ts

@@ -1,5 +1,4 @@
 import { createGlobalStore, getGlobalStore, registerGlobalStore } from '@openmrs/esm-state';
-import { shallowEqual } from '@openmrs/esm-utils';
 import { type StoreApi } from 'zustand';
 import { createStore } from 'zustand/vanilla';
 import type { Config, ConfigObject, ConfigSchema, ExtensionSlotConfig, ProvidedConfig } from '../types';
@@ -178,59 +177,6 @@ export function getExtensionsConfigStore() {
   });
 }
 
-/**
- * The read-only part of a store's API.
- * @internal
- */
-export type ReadableStore<T> = Pick<StoreApi<T>, 'getInitialState' | 'getState' | 'subscribe'>;
-
-/**
- * A read-only view of one extension instance's config within a slot. Read-only because these
- * configs are derived from the config system's inputs, so a write here would be overwritten by the
- * next recomputation; provide a config or set a temporary one instead.
- *
- * @internal
- */
-export function getExtensionConfig(
-  slotName: string,
-  extensionId: string,
-): ReadableStore<Omit<ConfigStore, 'translationOverridesLoaded'>> {
-  if (
-    typeof slotName !== 'string' ||
-    typeof extensionId !== 'string' ||
-    slotName === '__proto__' ||
-    extensionId === '__proto__' ||
-    slotName === 'constructor' ||
-    extensionId === 'constructor' ||
-    slotName === 'prototype' ||
-    extensionId === 'prototype'
-  ) {
-    throw new Error('Attempted to call `getExtensionConfig()` with invalid argument');
-  }
-
-  const extensionConfigStore = getExtensionsConfigStore();
-  const selector = (configStore: ExtensionsConfigStore) => configStore.configs[slotName]?.[extensionId];
-
-  return {
-    getInitialState() {
-      return selector(extensionConfigStore.getInitialState());
-    },
-    getState() {
-      return selector(extensionConfigStore.getState()) ?? { loaded: false, config: null };
-    },
-    subscribe(listener) {
-      return extensionConfigStore.subscribe((state, prevState) => {
-        const newState = selector(state);
-        const oldState = selector(prevState);
-
-        if (!shallowEqual(newState, oldState)) {
-          listener(newState, oldState);
-        }
-      });
-    },
-  };
-}
-
 /** @internal */
 export function getExtensionConfigFromStore(state: ExtensionsConfigStore, slotName: string, extensionId: string) {
   const extensionConfig = state.configs[slotName]?.[extensionId];