bulltrackers-module 1.0.307 → 1.0.308

package/functions/computation-system/WorkflowOrchestrator.js

@@ -1,6 +1,6 @@
 /**
  * FILENAME: computation-system/WorkflowOrchestrator.js
- * UPDATED: Historical Continuity is now a first-class "Temporal Dependency".
+ * UPDATED: Added missing groupByPass export.
  * Includes Content-Based Short-Circuiting for both Upstream and Historical dependencies.
  */
 
@@ -13,6 +13,19 @@ const { MetaExecutor }                                     = require('./executor
 
 const STATUS_IMPOSSIBLE_PREFIX = 'IMPOSSIBLE';
 
+/**
+ * [FIX] Groups manifest entries by their pass number.
+ * Required by the Dispatcher to identify current work-sets.
+ */
+function groupByPass(manifest) {
+    const passes = {};
+    manifest.forEach(calc => {
+        if (!passes[calc.pass]) passes[calc.pass] = [];
+        passes[calc.pass].push(calc);
+    });
+    return passes;
+}
+
 /**
  * [NEW] Core Short-Circuit Logic.
  * Checks if a dependency (either a different node or "yesterday's self") is satisfied.
@@ -76,6 +89,7 @@ function analyzeDateExecution(dateStr, calcsInPass, rootDataStatus, dailyStatus,
             const yesterday = new Date(dateStr + 'T00:00:00Z');
             yesterday.setUTCDate(yesterday.getUTCDate() - 1);
             
+            // Only block if yesterday is a valid data date. 
             if (yesterday >= DEFINITIVE_EARLIEST_DATES.absoluteEarliest) {
                 const check = isDependencyReady(calc.name, true, null, prevDailyStatus, manifestMap, stored);
                 if (!check.ready) isBlockedByHistory = true;
@@ -148,4 +162,4 @@ async function executeDispatchTask(dateStr, pass, targetComputation, config, dep
     return { date: dateStr, updates };
 }
 
-module.exports = { executeDispatchTask, analyzeDateExecution };
+module.exports = { executeDispatchTask, analyzeDateExecution, groupByPass };

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

@@ -48,7 +48,7 @@ async function handleComputationTask(message, config, dependencies) {
 
     logger.log('INFO', `[Worker] 📥 Task: ${computation} (${date}) [Tier: ${resourceTier}]`);
 
-    // 1. Audit Lease
+    // 1. Audit Lease (Lease on life for the monitor to see)
     await db.doc(ledgerPath).set({
         status: 'IN_PROGRESS',
         workerId: process.env.K_REVISION || os.hostname(),
@@ -84,6 +84,7 @@ async function handleComputationTask(message, config, dependencies) {
             composition: calcUpdate.composition
         };
 
+        // Mark ledger as completed
         await db.doc(ledgerPath).update({ status: 'COMPLETED', completedAt: new Date() });
         await recordRunAttempt(db, { date, computation, pass }, 'SUCCESS', null, metrics, triggerReason, resourceTier);
 
@@ -94,9 +95,9 @@ async function handleComputationTask(message, config, dependencies) {
         if (isDeterministic || (message.deliveryAttempt || 1) >= MAX_RETRIES) {
             await db.doc(ledgerPath).set({ status: 'FAILED', error: err.message, failedAt: new Date() }, { merge: true });
             await recordRunAttempt(db, { date, computation, pass }, 'FAILURE', { message: err.message, stage: err.stage || 'FATAL' }, { peakMemoryMB: heartbeat.getPeak() }, triggerReason, resourceTier);
-            return; // Don't throw, we've handled the permanent failure
+            return; // Exit without throwing to prevent endless Pub/Sub retries for dead logic
         }
-        throw err; // Trigger Pub/Sub retry
+        throw err; // Trigger Pub/Sub retry for non-deterministic transient errors
     }
 }
 

package/package.json

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