bulltrackers-module 1.0.143 → 1.0.144

package/functions/computation-system/helpers/orchestration_helpers.js

@@ -55,7 +55,21 @@ async function checkRootDataAvailability(dateStr, config, dependencies, earliest
             // We return null to skip the entire day
             return null;
         }
-        return {portfolioRefs, insightsData,socialData,historyRefs,status: { hasPortfolio, hasInsights, hasSocial, hasHistory }};
+        
+        /**
+         * --- THIS IS THE FIX ---
+         * Rename keys to match what streamAndProcess (Stage 8) expects.
+         * 'insightsData' is renamed to 'todayInsights'.
+         * 'socialData' is renamed to 'todaySocialPostInsights'.
+         */
+        return {
+            portfolioRefs, 
+            todayInsights: insightsData,      // <-- FIX
+            todaySocialPostInsights: socialData, // <-- FIX
+            historyRefs,
+            status: { hasPortfolio, hasInsights, hasSocial, hasHistory }
+        };
+
     } catch (err) { logger.log('ERROR', `[PassRunner] Error checking data for ${dateStr}`, { errorMessage: err.message }); return null; }
 }
 
@@ -224,6 +238,7 @@ async function loadHistoricalData(date, calcs, config, deps, rootData) {
 async function streamAndProcess(dateStr, state, passName, config, deps, rootData) { 
     const { logger, calculationUtils } = deps;
     // --- MODIFIED: yesterdayInsights/Social are now loaded by loadHistoricalData ---
+    // --- THIS WILL NOW WORK, as rootData contains 'todayInsights' from Stage 3 ---
     const { todayInsights, yesterdayInsights, todaySocialPostInsights, yesterdaySocialPostInsights, todayHistoryData, yesterdayHistoryData, yesterdayPortfolios } = rootData;
     
     // --- NEW: Check if streaming is even needed ---
@@ -240,6 +255,7 @@ async function streamAndProcess(dateStr, state, passName, config, deps, rootData
         if(cat==='socialPosts'||cat==='insights') {
             if (firstUser) {
                 logger.log('INFO', `[${passName}] Running non-streaming calc: ${name}`);
+                // --- This 'args' array will now receive the correct data ---
                 let args=[null,null,null,{...context, userType: 'n/a'},todayInsights,yesterdayInsights,todaySocialPostInsights,yesterdaySocialPostInsights,todayHistoryData,yesterdayHistoryData];
                 // Pass historical data if needed
                 if(calc.manifest.isHistorical) {
@@ -269,6 +285,7 @@ async function streamAndProcess(dateStr, state, passName, config, deps, rootData
                 // --- MODIFIED: Skip social/insights here, they ran above ---
                 if(isSocialOrInsights) continue; 
                 
+                // --- This 'args' array will now receive the correct data ---
                 let args=[p,null,uid,context,todayInsights,yesterdayInsights,todaySocialPostInsights,yesterdaySocialPostInsights,todayHistoryData,yesterdayHistoryData]; 
                 
                 if(isHistorical){ 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "bulltrackers-module",
-  "version": "1.0.143",
+  "version": "1.0.144",
   "description": "Helper Functions for Bulltrackers.",
   "main": "index.js",
   "files": [