@openmrs/esm-state 10.0.1-pre.5299 → 10.0.1-pre.5316

package/.turbo/turbo-build.log

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

package/dist/state.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"state.d.ts","sourceRoot":"","sources":["../src/state.ts"],"names":[],"mappings":"AAIA,OAAO,KAAK,EAAE,QAAQ,EAAE,MAAM,iBAAiB,CAAC;AAmBhD,KAAK,WAAW,GAAG,MAAM,GAAG,gBAAgB,CAAC;AAE7C;;;;;;;;GAQG;AACH,wBAAgB,iBAAiB,CAAC,CAAC,EAAE,IAAI,EAAE,MAAM,EAAE,YAAY,EAAE,CAAC,EAAE,WAAW,GAAE,WAAoB,GAAG,QAAQ,CAAC,CAAC,CAAC,CAwBlH;AAED;;;;;;;GAOG;AACH,wBAAgB,mBAAmB,CAAC,CAAC,EAAE,IAAI,EAAE,MAAM,EAAE,KAAK,EAAE,QAAQ,CAAC,CAAC,CAAC,GAAG,QAAQ,CAAC,CAAC,CAAC,CAsBpF;AAED;;;;;;;;GAQG;AACH,wBAAgB,cAAc,CAAC,CAAC,EAC9B,IAAI,EAAE,MAAM,EACZ,aAAa,CAAC,EAAE,CAAC,EACjB,mBAAmB,GAAE,WAAoB,GACxC,QAAQ,CAAC,CAAC,CAAC,CAab;AAID;;;;;;;;;;;;;GAaG;AACH,wBAAgB,WAAW,CAAC,CAAC,EAAE,CAAC,GAAG,CAAC,EAAE,KAAK,EAAE,QAAQ,CAAC,CAAC,CAAC,EAAE,MAAM,EAAE,CAAC,KAAK,EAAE,CAAC,KAAK,IAAI,GAAG,MAAM,IAAI,CAAC;AAClG,wBAAgB,WAAW,CAAC,CAAC,EAAE,CAAC,EAC9B,KAAK,EAAE,QAAQ,CAAC,CAAC,CAAC,EAClB,MAAM,EAAE,CAAC,KAAK,EAAE,CAAC,KAAK,CAAC,EACvB,MAAM,EAAE,CAAC,QAAQ,EAAE,CAAC,KAAK,IAAI,GAC5B,MAAM,IAAI,CAAC"}
+{"version":3,"file":"state.d.ts","sourceRoot":"","sources":["../src/state.ts"],"names":[],"mappings":"AAIA,OAAO,KAAK,EAAE,QAAQ,EAAE,MAAM,iBAAiB,CAAC;AA2ChD,KAAK,WAAW,GAAG,MAAM,GAAG,gBAAgB,CAAC;AAE7C;;;;;;;;GAQG;AACH,wBAAgB,iBAAiB,CAAC,CAAC,EAAE,IAAI,EAAE,MAAM,EAAE,YAAY,EAAE,CAAC,EAAE,WAAW,GAAE,WAAoB,GAAG,QAAQ,CAAC,CAAC,CAAC,CAwBlH;AAED;;;;;;;GAOG;AACH,wBAAgB,mBAAmB,CAAC,CAAC,EAAE,IAAI,EAAE,MAAM,EAAE,KAAK,EAAE,QAAQ,CAAC,CAAC,CAAC,GAAG,QAAQ,CAAC,CAAC,CAAC,CAsBpF;AAED;;;;;;;;GAQG;AACH,wBAAgB,cAAc,CAAC,CAAC,EAC9B,IAAI,EAAE,MAAM,EACZ,aAAa,CAAC,EAAE,CAAC,EACjB,mBAAmB,GAAE,WAAoB,GACxC,QAAQ,CAAC,CAAC,CAAC,CAab;AAID;;;;;;;;;;;;;GAaG;AACH,wBAAgB,WAAW,CAAC,CAAC,EAAE,CAAC,GAAG,CAAC,EAAE,KAAK,EAAE,QAAQ,CAAC,CAAC,CAAC,EAAE,MAAM,EAAE,CAAC,KAAK,EAAE,CAAC,KAAK,IAAI,GAAG,MAAM,IAAI,CAAC;AAClG,wBAAgB,WAAW,CAAC,CAAC,EAAE,CAAC,EAC9B,KAAK,EAAE,QAAQ,CAAC,CAAC,CAAC,EAClB,MAAM,EAAE,CAAC,KAAK,EAAE,CAAC,KAAK,CAAC,EACvB,MAAM,EAAE,CAAC,QAAQ,EAAE,CAAC,KAAK,IAAI,GAC5B,MAAM,IAAI,CAAC"}

package/dist/state.js

@@ -3,6 +3,22 @@ import { persist, createJSONStorage } from "zustand/middleware";
 import { createStore } from "zustand/vanilla";
 import { isTestEnvironment } from "./utils.js";
 const availableStores = {};
+/**
+ * Each copy of this module owns a separate `availableStores`, so a page running more than one copy
+ * has stores, config and extension state that cannot see each other. Nothing fails outright, which
+ * is what makes it expensive: the symptoms are missing state and "store already exists" errors far
+ * from the cause. The count lives on a global registry key rather than in module scope, since a
+ * second copy would not share module scope — that is the whole problem being detected.
+ *
+ * A hot reload re-evaluates this module and so also trips this, which is the honest answer: after a
+ * hot reload the page really is running two copies.
+ */ const instanceCountKey = Symbol.for('openmrs.esm-state.instanceCount');
+const globalCounters = globalThis;
+const instanceCount = (globalCounters[instanceCountKey] ?? 0) + 1;
+globalCounters[instanceCountKey] = instanceCount;
+if (instanceCount > 1 && !isTestEnvironment()) {
+    console.error(`Detected ${instanceCount} copies of @openmrs/esm-framework running on this page. Each copy keeps ` + 'its own store registry, so stores, configuration and extension state are not shared between ' + 'them. This usually means a frontend module loaded its own bundled framework instead of the one ' + 'the app shell provides.');
+}
 // spaEnv isn't available immediately. Wait a bit before making stores available
 // on window in development mode.
 globalThis.setTimeout?.(()=>{

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@openmrs/esm-state",
-  "version": "10.0.1-pre.5299",
+  "version": "10.0.1-pre.5316",
   "license": "MPL-2.0",
   "description": "Frontend stores & state management for OpenMRS",
   "type": "module",
@@ -58,12 +58,12 @@
     "zustand": "^4.5.5"
   },
   "peerDependencies": {
-    "@openmrs/esm-globals": "^10.0.1-pre.5299",
-    "@openmrs/esm-utils": "^10.0.1-pre.5299"
+    "@openmrs/esm-globals": "^10.0.1-pre.5316",
+    "@openmrs/esm-utils": "^10.0.1-pre.5316"
   },
   "devDependencies": {
-    "@openmrs/esm-globals": "10.0.1-pre.5299",
-    "@openmrs/esm-utils": "10.0.1-pre.5299",
+    "@openmrs/esm-globals": "10.0.1-pre.5316",
+    "@openmrs/esm-utils": "10.0.1-pre.5316",
     "@swc/cli": "0.8.1",
     "@swc/core": "1.15.21",
     "@vitest/coverage-v8": "^4.1.2",

package/src/state.ts

@@ -13,6 +13,30 @@ interface StoreEntity {
 
 const availableStores: Record<string, StoreEntity> = {};
 
+/**
+ * Each copy of this module owns a separate `availableStores`, so a page running more than one copy
+ * has stores, config and extension state that cannot see each other. Nothing fails outright, which
+ * is what makes it expensive: the symptoms are missing state and "store already exists" errors far
+ * from the cause. The count lives on a global registry key rather than in module scope, since a
+ * second copy would not share module scope — that is the whole problem being detected.
+ *
+ * A hot reload re-evaluates this module and so also trips this, which is the honest answer: after a
+ * hot reload the page really is running two copies.
+ */
+const instanceCountKey = Symbol.for('openmrs.esm-state.instanceCount');
+const globalCounters = globalThis as unknown as Record<symbol, number | undefined>;
+const instanceCount = (globalCounters[instanceCountKey] ?? 0) + 1;
+globalCounters[instanceCountKey] = instanceCount;
+
+if (instanceCount > 1 && !isTestEnvironment()) {
+  console.error(
+    `Detected ${instanceCount} copies of @openmrs/esm-framework running on this page. Each copy keeps ` +
+      'its own store registry, so stores, configuration and extension state are not shared between ' +
+      'them. This usually means a frontend module loaded its own bundled framework instead of the one ' +
+      'the app shell provides.',
+  );
+}
+
 // spaEnv isn't available immediately. Wait a bit before making stores available
 // on window in development mode.
 globalThis.setTimeout?.(() => {