@zeniai/client-epic-state 5.0.34-betaRR0 → 5.0.34

package/lib/esm/init.js

@@ -57,7 +57,19 @@ export async function initializePusher(pusherClientSecret, options, headers, reI
     if (pusherInitPromise && !reInitialize) {
         return pusherInitPromise;
     }
-    pusherInitPromise = (async () => {
+    // The IIFE captures `currentInit` so that, when its `finally` runs, it can
+    // identity-check before clearing the module-level `pusherInitPromise`. This
+    // matters when a reInitialize=true call overwrites `pusherInitPromise`
+    // while we're still in flight: without the identity check, our finally
+    // would clobber the newer call's slot and the next concurrent caller
+    // would bypass the dedup guard. Capture is safe because the first action
+    // inside the try is `await import(...)`, which always yields control —
+    // so the finally runs in a future microtask, after `currentInit` has
+    // already been assigned below. We declare it as `let … | undefined` so
+    // TS's definite-assignment analysis (which doesn't model the await) accepts
+    // the closure read; at runtime it's always the IIFE promise by then.
+    let currentInit = undefined;
+    currentInit = (async () => {
         try {
             // Dynamic import so the browser-only bundle is loaded only when this
             // function actually runs (browser path). Works in both build outputs:
@@ -84,10 +96,13 @@ export async function initializePusher(pusherClientSecret, options, headers, reI
             return pusher;
         }
         finally {
-            pusherInitPromise = undefined;
+            if (pusherInitPromise === currentInit) {
+                pusherInitPromise = undefined;
+            }
         }
     })();
-    return pusherInitPromise;
+    pusherInitPromise = currentInit;
+    return currentInit;
 }
 const customHandler = async (params, headers, callback) => {
     const { socketId, channelName } = params;

package/lib/init.js

@@ -101,7 +101,19 @@ async function initializePusher(pusherClientSecret, options, headers, reInitiali
     if (pusherInitPromise && !reInitialize) {
         return pusherInitPromise;
     }
-    pusherInitPromise = (async () => {
+    // The IIFE captures `currentInit` so that, when its `finally` runs, it can
+    // identity-check before clearing the module-level `pusherInitPromise`. This
+    // matters when a reInitialize=true call overwrites `pusherInitPromise`
+    // while we're still in flight: without the identity check, our finally
+    // would clobber the newer call's slot and the next concurrent caller
+    // would bypass the dedup guard. Capture is safe because the first action
+    // inside the try is `await import(...)`, which always yields control —
+    // so the finally runs in a future microtask, after `currentInit` has
+    // already been assigned below. We declare it as `let … | undefined` so
+    // TS's definite-assignment analysis (which doesn't model the await) accepts
+    // the closure read; at runtime it's always the IIFE promise by then.
+    let currentInit = undefined;
+    currentInit = (async () => {
         try {
             // Dynamic import so the browser-only bundle is loaded only when this
             // function actually runs (browser path). Works in both build outputs:
@@ -128,10 +140,13 @@ async function initializePusher(pusherClientSecret, options, headers, reInitiali
             return pusher;
         }
         finally {
-            pusherInitPromise = undefined;
+            if (pusherInitPromise === currentInit) {
+                pusherInitPromise = undefined;
+            }
         }
     })();
-    return pusherInitPromise;
+    pusherInitPromise = currentInit;
+    return currentInit;
 }
 const customHandler = async (params, headers, callback) => {
     const { socketId, channelName } = params;