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

package/App/ActionSetters/index.ts

@@ -20,6 +20,10 @@ const warnUndefinedContext = (context) =>
     data: { context },
   });
 
+type Subscription = {
+  [x: string]: () => any;
+};
+
 const getModuleContext = R.path(["module", "context"]);
 
 const getContexts = R.compose(
@@ -59,43 +63,24 @@ 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);
-
-  if (
-    capturedPromisesRef.current === null &&
-    actionProvidersSubscriptions.length > 0
-  ) {
-    const promises: Promise<unknown>[] = [];
-    let hasRiverInitMethod = false;
 
-    for (const item of actionProvidersSubscriptions) {
-      if (!item?.[key]) continue;
+  const subscriptions = React.useMemo(() => {
+    if (!actionProvidersSubscriptions.length) return undefined;
 
-      hasRiverInitMethod = true;
-      const promise = item[key]();
+    return actionProvidersSubscriptions.reduce((acc, item) => {
+      const riverInitialStatePromise = item[key]?.();
 
-      if (promise) {
-        promises.push(promise);
+      if (riverInitialStatePromise) {
+        acc.push(riverInitialStatePromise);
       }
-    }
 
-    // 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;
-    }
-  }
+      return acc;
+    }, [] as Subscription[]);
+  }, [actionProvidersSubscriptions]);
 
-  if (!actionProvidersSubscriptions.length) {
-    return undefined;
-  }
+  if (!actionProvidersSubscriptions.length) return undefined;
 
-  return capturedPromisesRef.current === null
-    ? undefined
-    : capturedPromisesRef.current;
+  return subscriptions;
 };
 
 const getActionInitialStateSetters = (key) => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@applicaster/quick-brick-core",
-  "version": "15.1.4-alpha.7394150354",
+  "version": "15.1.4-rc.0",
   "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.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",
+    "@applicaster/applicaster-types": "15.1.4-rc.0",
+    "@applicaster/quick-brick-core-plugins": "15.1.4-rc.0",
+    "@applicaster/zapp-pipes-v2-client": "15.1.4-rc.0",
+    "@applicaster/zapp-react-native-bridge": "15.1.4-rc.0",
+    "@applicaster/zapp-react-native-redux": "15.1.4-rc.0",
+    "@applicaster/zapp-react-native-ui-components": "15.1.4-rc.0",
+    "@applicaster/zapp-react-native-utils": "15.1.4-rc.0",
     "atob": "^2.1.2",
     "axios": "^0.28.0",
     "btoa": "^1.2.1",