@openmrs/esm-react-utils 10.0.1-pre.5255 → 10.0.1-pre.5295

package/.turbo/turbo-build.log

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

package/dist/Extension.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"Extension.d.ts","sourceRoot":"","sources":["../src/Extension.tsx"],"names":[],"mappings":"AAEA,OAAO,KAAqD,MAAM,OAAO,CAAC;AAI1E,MAAM,MAAM,cAAc,GAAG,KAAK,CAAC,cAAc,CAAC,cAAc,CAAC,GAAG;IAClE,KAAK,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC;CACjC,CAAC;AAEF;;;;;;;;GAQG;AACH,eAAO,MAAM,SAAS,EAAE,KAAK,CAAC,EAAE,CAAC,cAAc,CA+E9C,CAAC"}
+{"version":3,"file":"Extension.d.ts","sourceRoot":"","sources":["../src/Extension.tsx"],"names":[],"mappings":"AAEA,OAAO,KAAqD,MAAM,OAAO,CAAC;AAI1E,MAAM,MAAM,cAAc,GAAG,KAAK,CAAC,cAAc,CAAC,cAAc,CAAC,GAAG;IAClE,KAAK,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC;CACjC,CAAC;AAEF;;;;;;;;GAQG;AACH,eAAO,MAAM,SAAS,EAAE,KAAK,CAAC,EAAE,CAAC,cAAc,CAuI9C,CAAC"}

package/dist/Extension.js

@@ -13,58 +13,100 @@ import { ComponentContext } from "./index.js";
     const { extension } = useContext(ComponentContext);
     const parcel = useRef(null);
     const updatePromise = useRef(Promise.resolve());
+    const isUnmounted = useRef(false);
+    const isRendering = useRef(false);
+    const latestState = useRef(state);
+    // Takes the parcel to tear down rather than reading `parcel.current`, which can be reassigned
+    // between scheduling this and running it.
+    const unmountParcel = useCallback((target)=>{
+        if (!target) {
+            return;
+        }
+        const unmountWhenMounted = ()=>{
+            if (target.getStatus() === 'MOUNTED') {
+                target.unmount();
+            }
+        };
+        switch(target.getStatus()){
+            case 'MOUNTED':
+                target.unmount();
+                break;
+            case 'UPDATING':
+                // The rejection is reported by whoever owns `updatePromise`; here it only matters that a
+                // failed update still runs the teardown, or the parcel is left broken and mounted.
+                updatePromise.current?.then(unmountWhenMounted, unmountWhenMounted);
+                break;
+            default:
+                // Any other status: the parcel either hasn't finished coming up or is already gone.
+                // `mountPromise` has settled or will, and `unmountWhenMounted` re-checks the status.
+                target.mountPromise.then(unmountWhenMounted, ()=>{});
+        }
+    }, []);
+    const applyUpdate = useCallback((target, nextState)=>{
+        if (!target.update || target.getStatus() === 'UNMOUNTING') {
+            return;
+        }
+        // Every later update chains onto this promise, so it has to settle even when an update fails.
+        updatePromise.current = Promise.all([
+            target.mountPromise,
+            updatePromise.current
+        ]).then(()=>{
+            if (parcel.current?.getStatus() === 'MOUNTED' && parcel.current.update) {
+                return parcel.current.update({
+                    ...nextState
+                });
+            }
+        }).catch((err)=>{
+            // A parcel torn down while its update was in flight rejects, and that race is expected.
+            // We use the status to distinguish reject reasons as the message isn't reliable
+            const status = parcel.current?.getStatus();
+            if (status !== 'UNMOUNTING' && status !== 'NOT_MOUNTED' && status !== 'UNLOADING') {
+                console.error(`The extension '${extension?.extensionId}' failed to update`, err);
+            }
+        });
+    }, []);
     const ref = useCallback((node)=>{
-        if (extension?.extensionSlotName && extension.extensionSlotModuleName && extension.extensionSlotModuleName && !parcel.current) {
-            renderExtension(node, extension.extensionSlotName, extension.extensionSlotModuleName, extension.extensionId, undefined, state).then((newParcel)=>{
-                parcel.current = newParcel;
-            });
+        // React detaches a ref by calling it with null. If we render something in response to this,
+        // React ignores it and we're left tracking a parcel with no visible DOM, so skip conditions
+        // we can't handle.
+        if (!node || parcel.current || isRendering.current || !extension?.extensionSlotName || !extension.extensionSlotModuleName) {
+            return;
         }
+        isRendering.current = true;
+        renderExtension(node, extension.extensionSlotName, extension.extensionSlotModuleName, extension.extensionId, undefined, state).then((newParcel)=>{
+            isRendering.current = false;
+            parcel.current = newParcel;
+            // Loading an extension's bundle can outlast the component that asked for it: the cleanup
+            // effect has already run and saw no parcel, so teardown has to happen here instead, or the
+            // parcel mounts into a detached node and its rendering record outlives the page.
+            if (isUnmounted.current) {
+                unmountParcel(newParcel);
+                return;
+            }
+            // Ensure we update the state of the parcel to the latest version we've received
+            if (newParcel && latestState.current !== state) {
+                applyUpdate(newParcel, latestState.current);
+            }
+        }, // Cleared so a later reattach can try again; `renderExtension` reports its own failures.
+        ()=>{
+            isRendering.current = false;
+        });
     }, []);
     useEffect(()=>{
+        // Reset on every run, not just the first: StrictMode mounts, tears down and mounts again, so
+        // a flag only ever set by the cleanup would still read "unmounted" for the live component.
+        isUnmounted.current = false;
         return ()=>{
-            if (parcel && parcel.current) {
-                const status = parcel.current.getStatus();
-                switch(status){
-                    case 'MOUNTING':
-                        parcel.current.mountPromise.then(()=>{
-                            if (parcel.current?.getStatus() === 'MOUNTED') {
-                                parcel.current.unmount();
-                            }
-                        });
-                        break;
-                    case 'MOUNTED':
-                        parcel.current.unmount();
-                        break;
-                    case 'UPDATING':
-                        if (updatePromise.current) {
-                            updatePromise.current.then(()=>{
-                                if (parcel.current?.getStatus() === 'MOUNTED') {
-                                    parcel.current.unmount();
-                                }
-                            });
-                        }
-                }
-            }
+            isUnmounted.current = true;
+            unmountParcel(parcel.current);
         };
     }, []);
     useEffect(()=>{
-        if (parcel.current && parcel.current.update && parcel.current.getStatus() !== 'UNMOUNTING') {
-            Promise.all([
-                parcel.current.mountPromise,
-                updatePromise.current
-            ]).then(()=>{
-                if (parcel?.current?.getStatus() === 'MOUNTED' && parcel.current.update) {
-                    updatePromise.current = parcel.current.update({
-                        ...state
-                    }).catch((err)=>{
-                        // if we were trying to update but the component was unmounted
-                        // while this was happening, ignore the error
-                        if (!(err instanceof Error) || !err.message.includes('minified message #32') || parcel.current?.getStatus() === 'MOUNTED') {
-                            throw err;
-                        }
-                    });
-                }
-            });
+        // Recorded before the parcel is checked, so that a state that arrives before there is a parcel
+        // to receive it is still the one applied once there is.
+        latestState.current = state;
+        if (parcel.current) {
+            applyUpdate(parcel.current, state);
         }
     }, [
         state

package/dist/index.d.ts

@@ -10,6 +10,8 @@ export * from './useAbortController';
 export * from './useAppContext';
 export * from './useAssignedExtensionIds';
 export * from './useAssignedExtensions';
+export * from './useCandidateExtensions';
+export * from './useShallowStableValue';
 export * from './useAttachments';
 export * from './useBodyScrollLock';
 export * from './useConfig';

package/dist/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAAA,cAAc,oBAAoB,CAAC;AACnC,cAAc,oBAAoB,CAAC;AACnC,cAAc,aAAa,CAAC;AAC5B,cAAc,iBAAiB,CAAC;AAChC,cAAc,gBAAgB,CAAC;AAC/B,cAAc,6BAA6B,CAAC;AAC5C,cAAc,kBAAkB,CAAC;AACjC,cAAc,yBAAyB,CAAC;AACxC,cAAc,sBAAsB,CAAC;AACrC,cAAc,iBAAiB,CAAC;AAChC,cAAc,2BAA2B,CAAC;AAC1C,cAAc,yBAAyB,CAAC;AACxC,cAAc,kBAAkB,CAAC;AACjC,cAAc,qBAAqB,CAAC;AACpC,cAAc,aAAa,CAAC;AAC5B,cAAc,0BAA0B,CAAC;AACzC,cAAc,mBAAmB,CAAC;AAClC,cAAc,eAAe,CAAC;AAC9B,cAAc,uBAAuB,CAAC;AACtC,cAAc,uBAAuB,CAAC;AACtC,cAAc,6BAA6B,CAAC;AAC5C,cAAc,oBAAoB,CAAC;AACnC,cAAc,wBAAwB,CAAC;AACvC,cAAc,yBAAyB,CAAC;AACxC,cAAc,qBAAqB,CAAC;AACpC,cAAc,kBAAkB,CAAC;AACjC,cAAc,kBAAkB,CAAC;AACjC,cAAc,mBAAmB,CAAC;AAClC,cAAc,mBAAmB,CAAC;AAClC,cAAc,qBAAqB,CAAC;AACpC,cAAc,iBAAiB,CAAC;AAChC,cAAc,cAAc,CAAC;AAC7B,cAAc,mBAAmB,CAAC;AAClC,cAAc,gBAAgB,CAAC;AAC/B,cAAc,qBAAqB,CAAC;AACpC,cAAc,gBAAgB,CAAC;AAC/B,OAAO,EAAE,kBAAkB,EAAE,KAAK,wBAAwB,EAAE,MAAM,sBAAsB,CAAC;AACzF,OAAO,EAAE,kBAAkB,EAAE,KAAK,wBAAwB,EAAE,MAAM,sBAAsB,CAAC;AACzF,OAAO,EAAE,oBAAoB,EAAE,KAAK,0BAA0B,EAAE,MAAM,wBAAwB,CAAC;AAC/F,cAAc,iBAAiB,CAAC;AAChC,cAAc,iBAAiB,CAAC;AAChC,cAAc,qBAAqB,CAAC;AACpC,cAAc,cAAc,CAAC;AAC7B,cAAc,gCAAgC,CAAC;AAC/C,cAAc,2BAA2B,CAAC;AAC1C,OAAO,EAAE,UAAU,EAAE,MAAM,cAAc,CAAC;AAC1C,cAAc,YAAY,CAAC;AAC3B,cAAc,iBAAiB,CAAC;AAChC,cAAc,YAAY,CAAC;AAC3B,cAAc,wBAAwB,CAAC;AACvC,cAAc,iBAAiB,CAAC"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAAA,cAAc,oBAAoB,CAAC;AACnC,cAAc,oBAAoB,CAAC;AACnC,cAAc,aAAa,CAAC;AAC5B,cAAc,iBAAiB,CAAC;AAChC,cAAc,gBAAgB,CAAC;AAC/B,cAAc,6BAA6B,CAAC;AAC5C,cAAc,kBAAkB,CAAC;AACjC,cAAc,yBAAyB,CAAC;AACxC,cAAc,sBAAsB,CAAC;AACrC,cAAc,iBAAiB,CAAC;AAChC,cAAc,2BAA2B,CAAC;AAC1C,cAAc,yBAAyB,CAAC;AACxC,cAAc,0BAA0B,CAAC;AACzC,cAAc,yBAAyB,CAAC;AACxC,cAAc,kBAAkB,CAAC;AACjC,cAAc,qBAAqB,CAAC;AACpC,cAAc,aAAa,CAAC;AAC5B,cAAc,0BAA0B,CAAC;AACzC,cAAc,mBAAmB,CAAC;AAClC,cAAc,eAAe,CAAC;AAC9B,cAAc,uBAAuB,CAAC;AACtC,cAAc,uBAAuB,CAAC;AACtC,cAAc,6BAA6B,CAAC;AAC5C,cAAc,oBAAoB,CAAC;AACnC,cAAc,wBAAwB,CAAC;AACvC,cAAc,yBAAyB,CAAC;AACxC,cAAc,qBAAqB,CAAC;AACpC,cAAc,kBAAkB,CAAC;AACjC,cAAc,kBAAkB,CAAC;AACjC,cAAc,mBAAmB,CAAC;AAClC,cAAc,mBAAmB,CAAC;AAClC,cAAc,qBAAqB,CAAC;AACpC,cAAc,iBAAiB,CAAC;AAChC,cAAc,cAAc,CAAC;AAC7B,cAAc,mBAAmB,CAAC;AAClC,cAAc,gBAAgB,CAAC;AAC/B,cAAc,qBAAqB,CAAC;AACpC,cAAc,gBAAgB,CAAC;AAC/B,OAAO,EAAE,kBAAkB,EAAE,KAAK,wBAAwB,EAAE,MAAM,sBAAsB,CAAC;AACzF,OAAO,EAAE,kBAAkB,EAAE,KAAK,wBAAwB,EAAE,MAAM,sBAAsB,CAAC;AACzF,OAAO,EAAE,oBAAoB,EAAE,KAAK,0BAA0B,EAAE,MAAM,wBAAwB,CAAC;AAC/F,cAAc,iBAAiB,CAAC;AAChC,cAAc,iBAAiB,CAAC;AAChC,cAAc,qBAAqB,CAAC;AACpC,cAAc,cAAc,CAAC;AAC7B,cAAc,gCAAgC,CAAC;AAC/C,cAAc,2BAA2B,CAAC;AAC1C,OAAO,EAAE,UAAU,EAAE,MAAM,cAAc,CAAC;AAC1C,cAAc,YAAY,CAAC;AAC3B,cAAc,iBAAiB,CAAC;AAChC,cAAc,YAAY,CAAC;AAC3B,cAAc,wBAAwB,CAAC;AACvC,cAAc,iBAAiB,CAAC"}

package/dist/index.js

@@ -10,6 +10,8 @@ export * from "./useAbortController.js";
 export * from "./useAppContext.js";
 export * from "./useAssignedExtensionIds.js";
 export * from "./useAssignedExtensions.js";
+export * from "./useCandidateExtensions.js";
+export * from "./useShallowStableValue.js";
 export * from "./useAttachments.js";
 export * from "./useBodyScrollLock.js";
 export * from "./useConfig.js";

package/dist/useAssignedExtensionIds.d.ts

@@ -1,6 +1,6 @@
 /**
  * Gets the assigned extension ids for a given extension slot name.
- * Does not consider if offline or online.
+ *
  * @param slotName The name of the slot to get the assigned IDs for.
  *
  * @deprecated Use `useAssignedExtensions`

package/dist/useAssignedExtensionIds.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"useAssignedExtensionIds.d.ts","sourceRoot":"","sources":["../src/useAssignedExtensionIds.ts"],"names":[],"mappings":"AAKA;;;;;;GAMG;AACH,wBAAgB,uBAAuB,CAAC,QAAQ,EAAE,MAAM,YAavD"}
+{"version":3,"file":"useAssignedExtensionIds.d.ts","sourceRoot":"","sources":["../src/useAssignedExtensionIds.ts"],"names":[],"mappings":"AAIA;;;;;;GAMG;AACH,wBAAgB,uBAAuB,CAAC,QAAQ,EAAE,MAAM,YAIvD"}

package/dist/useAssignedExtensionIds.js

@@ -1,21 +1,14 @@
-/** @module @category Extension */ import { useEffect, useState } from "react";
-import { getExtensionStore } from "@openmrs/esm-extensions";
-import { isEqual } from "lodash-es";
+/** @module @category Extension */ import { useMemo } from "react";
+import { useAssignedExtensions } from "./useAssignedExtensions.js";
 /**
  * Gets the assigned extension ids for a given extension slot name.
- * Does not consider if offline or online.
+ *
  * @param slotName The name of the slot to get the assigned IDs for.
  *
  * @deprecated Use `useAssignedExtensions`
  */ export function useAssignedExtensionIds(slotName) {
-    const [ids, setIds] = useState([]);
-    useEffect(()=>{
-        return getExtensionStore().subscribe((state)=>{
-            const newIds = state.slots[slotName]?.assignedExtensions.map((e)=>e.id) ?? [];
-            if (!isEqual(newIds, ids)) {
-                setIds(newIds);
-            }
-        });
-    }, []);
-    return ids;
+    const assignedExtensions = useAssignedExtensions(slotName);
+    return useMemo(()=>assignedExtensions.map((extension)=>extension.id), [
+        assignedExtensions
+    ]);
 }

package/dist/useAssignedExtensions.d.ts

@@ -1,6 +1,18 @@
+import { type ExtensionSlotCustomState } from '@openmrs/esm-extensions';
 /**
  * Gets the assigned extensions for a given extension slot name.
+ *
+ * The reactive form of `getAssignedExtensions`, and it answers the same thing: display conditions
+ * are always applied, so the result is what should actually be displayed. Pass `state` whenever you
+ * know it: the same slot can be rendered in several places at once with different state, so
+ * conditions are resolved against the state of this rendering. Omitting it resolves them against
+ * the session alone, hiding any extension whose condition depends on state.
+ *
+ * The returned array is a copy, so sorting or filtering it in place can't corrupt the extension
+ * store. Its reference is stable for as long as the slot's extensions, the state and the session are.
+ *
  * @param slotName The name of the slot to get the assigned extensions for.
+ * @param state The state of this rendering of the slot.
  */
-export declare function useAssignedExtensions(slotName: string): import("@openmrs/esm-extensions").AssignedExtension[];
+export declare function useAssignedExtensions(slotName: string, state?: ExtensionSlotCustomState): import("@openmrs/esm-extensions").AssignedExtension[];
 //# sourceMappingURL=useAssignedExtensions.d.ts.map

package/dist/useAssignedExtensions.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"useAssignedExtensions.d.ts","sourceRoot":"","sources":["../src/useAssignedExtensions.ts"],"names":[],"mappings":"AAGA;;;GAGG;AACH,wBAAgB,qBAAqB,CAAC,QAAQ,EAAE,MAAM,yDAGrD"}
+{"version":3,"file":"useAssignedExtensions.d.ts","sourceRoot":"","sources":["../src/useAssignedExtensions.ts"],"names":[],"mappings":"AAGA,OAAO,EAAE,KAAK,wBAAwB,EAAyB,MAAM,yBAAyB,CAAC;AAO/F;;;;;;;;;;;;;;GAcG;AACH,wBAAgB,qBAAqB,CAAC,QAAQ,EAAE,MAAM,EAAE,KAAK,CAAC,EAAE,wBAAwB,yDAYvF"}

package/dist/useAssignedExtensions.js

@@ -1,8 +1,35 @@
-/** @module @category Extension */ import { useExtensionSlotStore } from "./useExtensionSlotStore.js";
+/** @module @category Extension */ import { useMemo } from "react";
+import { sessionStore } from "@openmrs/esm-api";
+import { getAssignedExtensions } from "@openmrs/esm-extensions";
+import { useShallowStableValue } from "./useShallowStableValue.js";
+import { useExtensionSlotStore } from "./useExtensionSlotStore.js";
+import { useStore } from "./useStore.js";
+const selectSession = (state)=>state.session;
 /**
  * Gets the assigned extensions for a given extension slot name.
+ *
+ * The reactive form of `getAssignedExtensions`, and it answers the same thing: display conditions
+ * are always applied, so the result is what should actually be displayed. Pass `state` whenever you
+ * know it: the same slot can be rendered in several places at once with different state, so
+ * conditions are resolved against the state of this rendering. Omitting it resolves them against
+ * the session alone, hiding any extension whose condition depends on state.
+ *
+ * The returned array is a copy, so sorting or filtering it in place can't corrupt the extension
+ * store. Its reference is stable for as long as the slot's extensions, the state and the session are.
+ *
  * @param slotName The name of the slot to get the assigned extensions for.
- */ export function useAssignedExtensions(slotName) {
-    const slotStore = useExtensionSlotStore(slotName);
-    return slotStore?.assignedExtensions ?? [];
+ * @param state The state of this rendering of the slot.
+ */ export function useAssignedExtensions(slotName, state) {
+    // Subscribes so that this re-runs when the slot's extensions change; `getAssignedExtensions` reads
+    // the same store, so the value below is what it is about to return.
+    const { candidateExtensions } = useExtensionSlotStore(slotName);
+    // Display conditions may refer to `session`, so they have to be re-evaluated when it changes.
+    const session = useStore(sessionStore, selectSession);
+    const stableState = useShallowStableValue(state);
+    return useMemo(()=>getAssignedExtensions(slotName, stableState), [
+        slotName,
+        stableState,
+        session,
+        candidateExtensions
+    ]);
 }

package/dist/useCandidateExtensions.d.ts

@@ -0,0 +1,11 @@
+/**
+ * The reactive form of `getCandidateExtensions`: everything assigned to a slot, with no display
+ * condition evaluated. Intended for tools that present a slot's configuration rather than render it.
+ *
+ * Use {@link useAssignedExtensions} to decide what to display.
+ *
+ * @param slotName The name of the slot to get the candidate extensions for.
+ * @internal
+ */
+export declare function useCandidateExtensions(slotName: string): import("@openmrs/esm-extensions").AssignedExtension[];
+//# sourceMappingURL=useCandidateExtensions.d.ts.map

package/dist/useCandidateExtensions.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"useCandidateExtensions.d.ts","sourceRoot":"","sources":["../src/useCandidateExtensions.ts"],"names":[],"mappings":"AAGA;;;;;;;;GAQG;AACH,wBAAgB,sBAAsB,CAAC,QAAQ,EAAE,MAAM,yDAEtD"}

package/dist/useCandidateExtensions.js

@@ -0,0 +1,12 @@
+/** @module @category Extension */ import { useExtensionSlotStore } from "./useExtensionSlotStore.js";
+/**
+ * The reactive form of `getCandidateExtensions`: everything assigned to a slot, with no display
+ * condition evaluated. Intended for tools that present a slot's configuration rather than render it.
+ *
+ * Use {@link useAssignedExtensions} to decide what to display.
+ *
+ * @param slotName The name of the slot to get the candidate extensions for.
+ * @internal
+ */ export function useCandidateExtensions(slotName) {
+    return useExtensionSlotStore(slotName).candidateExtensions;
+}

package/dist/useExtensionSlot.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"useExtensionSlot.d.ts","sourceRoot":"","sources":["../src/useExtensionSlot.ts"],"names":[],"mappings":"AACA,OAAO,EACL,KAAK,wBAAwB,EAG9B,MAAM,yBAAyB,CAAC;AAIjC,gBAAgB;AAChB,wBAAgB,gBAAgB,CAAC,QAAQ,EAAE,MAAM,EAAE,KAAK,CAAC,EAAE,wBAAwB;;;;EA0BlF"}
+{"version":3,"file":"useExtensionSlot.d.ts","sourceRoot":"","sources":["../src/useExtensionSlot.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,KAAK,wBAAwB,EAAyB,MAAM,yBAAyB,CAAC;AAI/F,gBAAgB;AAChB,wBAAgB,gBAAgB,CAAC,QAAQ,EAAE,MAAM,EAAE,KAAK,CAAC,EAAE,wBAAwB;;;;EAoBlF"}

package/dist/useExtensionSlot.js

@@ -1,26 +1,21 @@
-import { useContext, useEffect, useRef } from "react";
-import { registerExtensionSlot, updateExtensionSlotState } from "@openmrs/esm-extensions";
+import { useContext, useEffect } from "react";
+import { registerExtensionSlot } from "@openmrs/esm-extensions";
 import { ComponentContext } from "./ComponentContext.js";
 import { useAssignedExtensions } from "./useAssignedExtensions.js";
 /** @internal */ export function useExtensionSlot(slotName, state) {
     const { moduleName } = useContext(ComponentContext);
-    const isInitialRender = useRef(true);
     if (!moduleName) {
         throw Error('ComponentContext has not been provided. This should come from @openmrs/esm-react-utils.');
     }
     useEffect(()=>{
-        registerExtensionSlot(moduleName, slotName, state);
-        isInitialRender.current = false;
-    }, []);
-    useEffect(()=>{
-        if (!isInitialRender.current) {
-            updateExtensionSlotState(slotName, state);
-        }
+        registerExtensionSlot(moduleName, slotName);
     }, [
-        slotName,
-        state
+        moduleName,
+        slotName
     ]);
-    const extensions = useAssignedExtensions(slotName);
+    // `state` must be local to the rendering rather than written to the store as state is
+    // render-specific
+    const extensions = useAssignedExtensions(slotName, state);
     return {
         extensions,
         extensionSlotName: slotName,

package/dist/useExtensionSlotStore.d.ts

@@ -1,4 +1,3 @@
-/** @module @category Extension */
 import { type ExtensionSlotState } from '@openmrs/esm-extensions';
 export declare const useExtensionSlotStore: (slot: string) => ExtensionSlotState;
 //# sourceMappingURL=useExtensionSlotStore.d.ts.map

package/dist/useExtensionSlotStore.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"useExtensionSlotStore.d.ts","sourceRoot":"","sources":["../src/useExtensionSlotStore.ts"],"names":[],"mappings":"AAAA,kCAAkC;AAClC,OAAO,EAAE,KAAK,kBAAkB,EAA0C,MAAM,yBAAyB,CAAC;AAG1G,eAAO,MAAM,qBAAqB,GAAI,MAAM,MAAM,uBACiD,CAAC"}
+{"version":3,"file":"useExtensionSlotStore.d.ts","sourceRoot":"","sources":["../src/useExtensionSlotStore.ts"],"names":[],"mappings":"AAEA,OAAO,EAEL,KAAK,kBAAkB,EAGxB,MAAM,yBAAyB,CAAC;AAcjC,eAAO,MAAM,qBAAqB,GAAI,MAAM,MAAM,uBAIjD,CAAC"}

package/dist/useExtensionSlotStore.js

@@ -1,3 +1,20 @@
-/** @module @category Extension */ import { getExtensionStore } from "@openmrs/esm-extensions";
+/** @module @category Extension */ import { useCallback } from "react";
+import { getExtensionStore } from "@openmrs/esm-extensions";
 import { useStore } from "./useStore.js";
-export const useExtensionSlotStore = (slot)=>useStore(getExtensionStore(), (state)=>state.slots?.[slot]);
+/**
+ * Stands in for a slot that has not been registered or attached to. Shared rather than built per
+ * call, so the snapshot stays reference-stable across renders — and frozen because that sharing
+ * means an in-place sort or push would otherwise corrupt every unregistered slot in the page.
+ */ const emptyCandidates = [];
+Object.freeze(emptyCandidates);
+const emptySlotState = {
+    candidateExtensions: emptyCandidates
+};
+Object.freeze(emptySlotState);
+export const useExtensionSlotStore = (slot)=>{
+    // Memoized so that `useStore`'s snapshot function is stable across renders.
+    const select = useCallback((state)=>state.slots?.[slot] ?? emptySlotState, [
+        slot
+    ]);
+    return useStore(getExtensionStore(), select);
+};

package/dist/useShallowStableValue.d.ts

@@ -0,0 +1,13 @@
+/**
+ * Returns `value` with a reference that only changes when its contents do, so that it can be used
+ * as a dependency of `useMemo`, `useEffect` and friends.
+ *
+ * This exists for props that are almost always written as object literals — `state={{ patientUuid }}`
+ * — which get a new identity on every render while meaning the same thing. Comparison is shallow,
+ * so a value nested more than one level deep still reads as a change.
+ *
+ * @param value The value to stabilize
+ * @returns `value`, or the last value this hook returned if the two are shallowly equal
+ */
+export declare function useShallowStableValue<T>(value: T): T;
+//# sourceMappingURL=useShallowStableValue.d.ts.map

package/dist/useShallowStableValue.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"useShallowStableValue.d.ts","sourceRoot":"","sources":["../src/useShallowStableValue.ts"],"names":[],"mappings":"AAIA;;;;;;;;;;GAUG;AACH,wBAAgB,qBAAqB,CAAC,CAAC,EAAE,KAAK,EAAE,CAAC,GAAG,CAAC,CAQpD"}

package/dist/useShallowStableValue.js

@@ -0,0 +1,19 @@
+/** @module @category Utility */ import { useRef } from "react";
+import { shallowEqual } from "@openmrs/esm-utils";
+/**
+ * Returns `value` with a reference that only changes when its contents do, so that it can be used
+ * as a dependency of `useMemo`, `useEffect` and friends.
+ *
+ * This exists for props that are almost always written as object literals — `state={{ patientUuid }}`
+ * — which get a new identity on every render while meaning the same thing. Comparison is shallow,
+ * so a value nested more than one level deep still reads as a change.
+ *
+ * @param value The value to stabilize
+ * @returns `value`, or the last value this hook returned if the two are shallowly equal
+ */ export function useShallowStableValue(value) {
+    const stable = useRef(value);
+    if (!shallowEqual(stable.current, value)) {
+        stable.current = value;
+    }
+    return stable.current;
+}

package/dist/useStore.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"useStore.d.ts","sourceRoot":"","sources":["../src/useStore.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,QAAQ,EAAE,MAAM,SAAS,CAAC;AAExC,MAAM,MAAM,cAAc,CAAC,CAAC,IAAI,CAAC,KAAK,EAAE,CAAC,EAAE,GAAG,IAAI,EAAE,GAAG,EAAE,KAAK,OAAO,CAAC,CAAC,CAAC,CAAC;AAEzE,KAAK,qBAAqB,CAAC,CAAC,IAAI,MAAM,CAAC,MAAM,EAAE,cAAc,CAAC,CAAC,CAAC,CAAC,CAAC;AAElE,MAAM,MAAM,OAAO,CAAC,CAAC,IAAI,CAAC,CAAC,KAAK,EAAE,QAAQ,CAAC,CAAC,CAAC,KAAK,qBAAqB,CAAC,CAAC,CAAC,CAAC,GAAG,qBAAqB,CAAC,CAAC,CAAC,CAAC;AAEvG,MAAM,MAAM,YAAY,CAAC,CAAC,EAAE,CAAC,SAAS,OAAO,CAAC,CAAC,CAAC,IAAI,CAAC,SAAS,qBAAqB,CAAC,CAAC,CAAC,GAClF,eAAe,CAAC,CAAC,CAAC,GAClB,CAAC,SAAS,CAAC,KAAK,EAAE,QAAQ,CAAC,CAAC,CAAC,KAAK,qBAAqB,CAAC,CAAC,CAAC,GACxD,eAAe,CAAC,qBAAqB,CAAC,CAAC,CAAC,CAAC,GACzC,KAAK,CAAC;AAIZ,KAAK,IAAI,CAAC,CAAC,SAAS,CAAC,GAAG,IAAI,EAAE,GAAG,EAAE,KAAK,GAAG,IAAI,CAAC,SAAS,CAAC,QAAQ,KAAA,EAAE,GAAG,QAAQ,EAAE,MAAM,YAAY,KAAK,GAAG,GACvG,CAAC,GAAG,IAAI,EAAE,YAAY,KAAK,IAAI,GAC/B,KAAK,CAAC;AAGV,KAAK,eAAe,CAAC,CAAC,SAAS,MAAM,CAAC,MAAM,EAAE,CAAC,GAAG,IAAI,EAAE,GAAG,EAAE,KAAK,GAAG,CAAC,IAAI;KACvE,IAAI,IAAI,MAAM,CAAC,GAAG,IAAI,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC;CACjC,CAAC;AA0BF,iBAAS,QAAQ,CAAC,CAAC,EAAE,KAAK,EAAE,QAAQ,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC;AAC5C,iBAAS,QAAQ,CAAC,CAAC,EAAE,CAAC,EAAE,KAAK,EAAE,QAAQ,CAAC,CAAC,CAAC,EAAE,MAAM,EAAE,CAAC,KAAK,EAAE,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC;AACxE,iBAAS,QAAQ,CAAC,CAAC,EAAE,CAAC,EAAE,CAAC,SAAS,OAAO,CAAC,CAAC,CAAC,EAC1C,KAAK,EAAE,QAAQ,CAAC,CAAC,CAAC,EAClB,MAAM,EAAE,SAAS,EACjB,OAAO,EAAE,CAAC,GACT,CAAC,GAAG,YAAY,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC;AAC1B,iBAAS,QAAQ,CAAC,CAAC,EAAE,CAAC,EAAE,CAAC,SAAS,OAAO,CAAC,CAAC,CAAC,EAC1C,KAAK,EAAE,QAAQ,CAAC,CAAC,CAAC,EAClB,MAAM,EAAE,CAAC,KAAK,EAAE,CAAC,KAAK,CAAC,EACvB,OAAO,EAAE,CAAC,GACT,CAAC,GAAG,YAAY,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC;AA2B1B;;;;;GAKG;AACH,iBAAS,mBAAmB,CAAC,CAAC,EAAE,CAAC,SAAS,OAAO,CAAC,CAAC,CAAC,EAAE,KAAK,EAAE,QAAQ,CAAC,CAAC,CAAC,EAAE,OAAO,EAAE,CAAC,GAAG,CAAC,GAAG,YAAY,CAAC,CAAC,EAAE,CAAC,CAAC,CAE5G;AAED;;;GAGG;AACH,iBAAS,cAAc,CAAC,CAAC,EAAE,KAAK,EAAE,QAAQ,CAAC,CAAC,CAAC;QACtB,CAAC;KACJ,CAAC,SAAS,OAAO,CAAC,CAAC,CAAC,WAAW,CAAC,GAAG,CAAC,GAAG,YAAY,CAAC,CAAC,EAAE,CAAC,CAAC;KACzD,CAAC,SAAS,OAAO,CAAC,CAAC,CAAC,YAAY,CAAC,GAAG,CAAC,GAAG,YAAY,CAAC,CAAC,EAAE,CAAC,CAAC;EAe7E;AAED,OAAO,EAAE,cAAc,EAAE,QAAQ,EAAE,mBAAmB,EAAE,CAAC"}
+{"version":3,"file":"useStore.d.ts","sourceRoot":"","sources":["../src/useStore.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,QAAQ,EAAE,MAAM,SAAS,CAAC;AAExC,MAAM,MAAM,cAAc,CAAC,CAAC,IAAI,CAAC,KAAK,EAAE,CAAC,EAAE,GAAG,IAAI,EAAE,GAAG,EAAE,KAAK,OAAO,CAAC,CAAC,CAAC,CAAC;AAEzE,KAAK,qBAAqB,CAAC,CAAC,IAAI,MAAM,CAAC,MAAM,EAAE,cAAc,CAAC,CAAC,CAAC,CAAC,CAAC;AAElE,MAAM,MAAM,OAAO,CAAC,CAAC,IAAI,CAAC,CAAC,KAAK,EAAE,QAAQ,CAAC,CAAC,CAAC,KAAK,qBAAqB,CAAC,CAAC,CAAC,CAAC,GAAG,qBAAqB,CAAC,CAAC,CAAC,CAAC;AAEvG,MAAM,MAAM,YAAY,CAAC,CAAC,EAAE,CAAC,SAAS,OAAO,CAAC,CAAC,CAAC,IAAI,CAAC,SAAS,qBAAqB,CAAC,CAAC,CAAC,GAClF,eAAe,CAAC,CAAC,CAAC,GAClB,CAAC,SAAS,CAAC,KAAK,EAAE,QAAQ,CAAC,CAAC,CAAC,KAAK,qBAAqB,CAAC,CAAC,CAAC,GACxD,eAAe,CAAC,qBAAqB,CAAC,CAAC,CAAC,CAAC,GACzC,KAAK,CAAC;AAIZ,KAAK,IAAI,CAAC,CAAC,SAAS,CAAC,GAAG,IAAI,EAAE,GAAG,EAAE,KAAK,GAAG,IAAI,CAAC,SAAS,CAAC,QAAQ,KAAA,EAAE,GAAG,QAAQ,EAAE,MAAM,YAAY,KAAK,GAAG,GACvG,CAAC,GAAG,IAAI,EAAE,YAAY,KAAK,IAAI,GAC/B,KAAK,CAAC;AAGV,KAAK,eAAe,CAAC,CAAC,SAAS,MAAM,CAAC,MAAM,EAAE,CAAC,GAAG,IAAI,EAAE,GAAG,EAAE,KAAK,GAAG,CAAC,IAAI;KACvE,IAAI,IAAI,MAAM,CAAC,GAAG,IAAI,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC;CACjC,CAAC;AA8BF,iBAAS,QAAQ,CAAC,CAAC,EAAE,KAAK,EAAE,QAAQ,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC;AAC5C,iBAAS,QAAQ,CAAC,CAAC,EAAE,CAAC,EAAE,KAAK,EAAE,QAAQ,CAAC,CAAC,CAAC,EAAE,MAAM,EAAE,CAAC,KAAK,EAAE,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC;AACxE,iBAAS,QAAQ,CAAC,CAAC,EAAE,CAAC,EAAE,CAAC,SAAS,OAAO,CAAC,CAAC,CAAC,EAC1C,KAAK,EAAE,QAAQ,CAAC,CAAC,CAAC,EAClB,MAAM,EAAE,SAAS,EACjB,OAAO,EAAE,CAAC,GACT,CAAC,GAAG,YAAY,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC;AAC1B,iBAAS,QAAQ,CAAC,CAAC,EAAE,CAAC,EAAE,CAAC,SAAS,OAAO,CAAC,CAAC,CAAC,EAC1C,KAAK,EAAE,QAAQ,CAAC,CAAC,CAAC,EAClB,MAAM,EAAE,CAAC,KAAK,EAAE,CAAC,KAAK,CAAC,EACvB,OAAO,EAAE,CAAC,GACT,CAAC,GAAG,YAAY,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC;AA8B1B;;;;;GAKG;AACH,iBAAS,mBAAmB,CAAC,CAAC,EAAE,CAAC,SAAS,OAAO,CAAC,CAAC,CAAC,EAAE,KAAK,EAAE,QAAQ,CAAC,CAAC,CAAC,EAAE,OAAO,EAAE,CAAC,GAAG,CAAC,GAAG,YAAY,CAAC,CAAC,EAAE,CAAC,CAAC,CAE5G;AAED;;;GAGG;AACH,iBAAS,cAAc,CAAC,CAAC,EAAE,KAAK,EAAE,QAAQ,CAAC,CAAC,CAAC;QACtB,CAAC;KACJ,CAAC,SAAS,OAAO,CAAC,CAAC,CAAC,WAAW,CAAC,GAAG,CAAC,GAAG,YAAY,CAAC,CAAC,EAAE,CAAC,CAAC;KACzD,CAAC,SAAS,OAAO,CAAC,CAAC,CAAC,YAAY,CAAC,GAAG,CAAC,GAAG,YAAY,CAAC,CAAC,EAAE,CAAC,CAAC;EAe7E;AAED,OAAO,EAAE,cAAc,EAAE,QAAQ,EAAE,mBAAmB,EAAE,CAAC"}

package/dist/useStore.js

@@ -14,7 +14,12 @@ function bindActions(store, actions) {
     }
     return bound;
 }
-const defaultSelectFunction = ()=>(x)=>x;
+/**
+ * A single shared identity function, rather than one built per call: `getSnapshot` below takes
+ * the selector as a dependency, so a fresh default each render would leave it unstable for every
+ * caller that doesn't pass a selector of its own.
+ */ const identitySelect = (x)=>x;
+const defaultSelectFunction = ()=>identitySelect;
 function useStore(store, select = defaultSelectFunction(), actions) {
     // Use useSyncExternalStore to subscribe synchronously during render
     // This ensures React can properly track all state updates
@@ -28,14 +33,20 @@ function useStore(store, select = defaultSelectFunction(), actions) {
         select
     ]);
     const state = useSyncExternalStore(subscribe, getSnapshot, getSnapshot);
-    let boundActions = useMemo(()=>actions ? bindActions(store, actions) : {}, [
+    let boundActions = useMemo(()=>actions ? bindActions(store, actions) : null, [
         store,
         actions
     ]);
-    return {
-        ...state,
-        ...boundActions
-    };
+    // Returned as-is when there are no actions to merge in, so the reference stays stable for as
+    // long as the selected value does. That value is not a copy — it is whatever the selector
+    // returned, usually the store's own state.
+    return useMemo(()=>boundActions ? {
+            ...state,
+            ...boundActions
+        } : state, [
+        state,
+        boundActions
+    ]);
 }
 /**
  *
@@ -53,13 +64,16 @@ function useStore(store, select = defaultSelectFunction(), actions) {
         const subscribe = useCallback((callback)=>store.subscribe(callback), []);
         const getSnapshot = useCallback(()=>store.getState(), []);
         const state = useSyncExternalStore(subscribe, getSnapshot, getSnapshot);
-        let boundActions = useMemo(()=>actions ? bindActions(store, actions) : {}, [
+        let boundActions = useMemo(()=>actions ? bindActions(store, actions) : null, [
             actions
         ]);
-        return {
-            ...state,
-            ...boundActions
-        };
+        return useMemo(()=>boundActions ? {
+                ...state,
+                ...boundActions
+            } : state, [
+            state,
+            boundActions
+        ]);
     }
     return useStore;
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@openmrs/esm-react-utils",
-  "version": "10.0.1-pre.5255",
+  "version": "10.0.1-pre.5295",
   "license": "MPL-2.0",
   "description": "React utilities for OpenMRS.",
   "type": "module",
@@ -60,17 +60,17 @@
     "single-spa-react": "^6.0.2"
   },
   "peerDependencies": {
-    "@openmrs/esm-api": "^10.0.1-pre.5255",
-    "@openmrs/esm-config": "^10.0.1-pre.5255",
-    "@openmrs/esm-context": "^10.0.1-pre.5255",
-    "@openmrs/esm-emr-api": "^10.0.1-pre.5255",
-    "@openmrs/esm-error-handling": "^10.0.1-pre.5255",
-    "@openmrs/esm-extensions": "^10.0.1-pre.5255",
-    "@openmrs/esm-feature-flags": "^10.0.1-pre.5255",
-    "@openmrs/esm-globals": "^10.0.1-pre.5255",
-    "@openmrs/esm-navigation": "^10.0.1-pre.5255",
-    "@openmrs/esm-state": "^10.0.1-pre.5255",
-    "@openmrs/esm-utils": "^10.0.1-pre.5255",
+    "@openmrs/esm-api": "^10.0.1-pre.5295",
+    "@openmrs/esm-config": "^10.0.1-pre.5295",
+    "@openmrs/esm-context": "^10.0.1-pre.5295",
+    "@openmrs/esm-emr-api": "^10.0.1-pre.5295",
+    "@openmrs/esm-error-handling": "^10.0.1-pre.5295",
+    "@openmrs/esm-extensions": "^10.0.1-pre.5295",
+    "@openmrs/esm-feature-flags": "^10.0.1-pre.5295",
+    "@openmrs/esm-globals": "^10.0.1-pre.5295",
+    "@openmrs/esm-navigation": "^10.0.1-pre.5295",
+    "@openmrs/esm-state": "^10.0.1-pre.5295",
+    "@openmrs/esm-utils": "^10.0.1-pre.5295",
     "dayjs": "1.x",
     "i18next": "25.x",
     "react": "18.x",
@@ -80,17 +80,17 @@
     "swr": "2.x"
   },
   "devDependencies": {
-    "@openmrs/esm-api": "10.0.1-pre.5255",
-    "@openmrs/esm-config": "10.0.1-pre.5255",
-    "@openmrs/esm-context": "10.0.1-pre.5255",
-    "@openmrs/esm-emr-api": "10.0.1-pre.5255",
-    "@openmrs/esm-error-handling": "10.0.1-pre.5255",
-    "@openmrs/esm-extensions": "10.0.1-pre.5255",
-    "@openmrs/esm-feature-flags": "10.0.1-pre.5255",
-    "@openmrs/esm-globals": "10.0.1-pre.5255",
-    "@openmrs/esm-navigation": "10.0.1-pre.5255",
-    "@openmrs/esm-state": "10.0.1-pre.5255",
-    "@openmrs/esm-utils": "10.0.1-pre.5255",
+    "@openmrs/esm-api": "10.0.1-pre.5295",
+    "@openmrs/esm-config": "10.0.1-pre.5295",
+    "@openmrs/esm-context": "10.0.1-pre.5295",
+    "@openmrs/esm-emr-api": "10.0.1-pre.5295",
+    "@openmrs/esm-error-handling": "10.0.1-pre.5295",
+    "@openmrs/esm-extensions": "10.0.1-pre.5295",
+    "@openmrs/esm-feature-flags": "10.0.1-pre.5295",
+    "@openmrs/esm-globals": "10.0.1-pre.5295",
+    "@openmrs/esm-navigation": "10.0.1-pre.5295",
+    "@openmrs/esm-state": "10.0.1-pre.5295",
+    "@openmrs/esm-utils": "10.0.1-pre.5295",
     "@swc/cli": "0.8.1",
     "@swc/core": "1.15.21",
     "@vitest/coverage-v8": "^4.1.2",