@applicaster/quick-brick-core 15.1.3-rc.0 → 15.1.4-alpha.7394150354

package/App/ActionSetters/index.ts

@@ -20,10 +20,6 @@ const warnUndefinedContext = (context) =>
     data: { context },
   });
 
-type Subscription = {
-  [x: string]: () => any;
-};
-
 const getModuleContext = R.path(["module", "context"]);
 
 const getContexts = R.compose(
@@ -63,24 +59,43 @@ const useSubscribeToActionProviders = () => {
 export const useRiverInitialState = () => {
   const key = "getRiverInitialState";
   const actionProvidersSubscriptions = useSubscribeToActionProviders();
+  // Capture river-init promises once. Re-calling getRiverInitialState() on every
+  // context re-render causes thrash and can drop already-completed observables.
+  const capturedPromisesRef = React.useRef<Promise<unknown>[] | null>(null);
 
-  const subscriptions = React.useMemo(() => {
-    if (!actionProvidersSubscriptions.length) return undefined;
+  if (
+    capturedPromisesRef.current === null &&
+    actionProvidersSubscriptions.length > 0
+  ) {
+    const promises: Promise<unknown>[] = [];
+    let hasRiverInitMethod = false;
+
+    for (const item of actionProvidersSubscriptions) {
+      if (!item?.[key]) continue;
 
-    return actionProvidersSubscriptions.reduce((acc, item) => {
-      const riverInitialStatePromise = item[key]?.();
+      hasRiverInitMethod = true;
+      const promise = item[key]();
 
-      if (riverInitialStatePromise) {
-        acc.push(riverInitialStatePromise);
+      if (promise) {
+        promises.push(promise);
       }
+    }
 
-      return acc;
-    }, [] as Subscription[]);
-  }, [actionProvidersSubscriptions]);
+    // Lock when we have promises, or when no provider exposes river-init.
+    // If methods exist but all returned undefined (observable not created yet),
+    // keep null and retry on next render.
+    if (promises.length > 0 || !hasRiverInitMethod) {
+      capturedPromisesRef.current = promises;
+    }
+  }
 
-  if (!actionProvidersSubscriptions.length) return undefined;
+  if (!actionProvidersSubscriptions.length) {
+    return undefined;
+  }
 
-  return subscriptions;
+  return capturedPromisesRef.current === null
+    ? undefined
+    : capturedPromisesRef.current;
 };
 
 const getActionInitialStateSetters = (key) => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@applicaster/quick-brick-core",
-  "version": "15.1.3-rc.0",
+  "version": "15.1.4-alpha.7394150354",
   "description": "Core package for Applicaster's Quick Brick App",
   "main": "index.js",
   "types": "index.d.ts",
@@ -28,13 +28,13 @@
   },
   "homepage": "https://github.com/applicaster/quickbrick#readme",
   "dependencies": {
-    "@applicaster/applicaster-types": "15.1.3-rc.0",
-    "@applicaster/quick-brick-core-plugins": "15.1.3-rc.0",
-    "@applicaster/zapp-pipes-v2-client": "15.1.3-rc.0",
-    "@applicaster/zapp-react-native-bridge": "15.1.3-rc.0",
-    "@applicaster/zapp-react-native-redux": "15.1.3-rc.0",
-    "@applicaster/zapp-react-native-ui-components": "15.1.3-rc.0",
-    "@applicaster/zapp-react-native-utils": "15.1.3-rc.0",
+    "@applicaster/applicaster-types": "15.1.4-alpha.7394150354",
+    "@applicaster/quick-brick-core-plugins": "15.1.4-alpha.7394150354",
+    "@applicaster/zapp-pipes-v2-client": "15.1.4-alpha.7394150354",
+    "@applicaster/zapp-react-native-bridge": "15.1.4-alpha.7394150354",
+    "@applicaster/zapp-react-native-redux": "15.1.4-alpha.7394150354",
+    "@applicaster/zapp-react-native-ui-components": "15.1.4-alpha.7394150354",
+    "@applicaster/zapp-react-native-utils": "15.1.4-alpha.7394150354",
     "atob": "^2.1.2",
     "axios": "^0.28.0",
     "btoa": "^1.2.1",