agents-relay 2.0.161 → 2.0.162

package/dist/planner-child-recovery.js

@@ -3,15 +3,14 @@ import { attachResolutionOwnership } from './workflow/task-resolution.js';
 /** Recover persisted child definitions without inventing new task identities. */
 export async function recoverPlannerChildren(store, job, plannerContext) {
     for (const plan of (job.plans ?? []).filter(item => item.status === 'SUCCEEDED' && item.plannerResult)) {
-        const executionContext = await plannerContext(job, plan);
-        // Historical plans were validated on commit. Rechecking against later Task failures
-        // blocks recovery before the Troubleshooter graph can run.
-        const children = plan.plannerResult.next_tasks.map(spec => materializePlannerChild(job, plan, spec, executionContext));
-        for (const child of children) {
-            const existing = job.tasks.find(item => item.id === child.id);
-            if (existing)
+        for (const spec of plan.plannerResult.next_tasks) {
+            // A succeeded Plan is immutable historical evidence. Existing children may
+            // have accumulated execution and Resolution state since the Plan committed;
+            // do not rematerialize them against today's Task states.
+            if (job.tasks.some(item => item.id === spec.id))
                 continue;
-            const spec = plan.plannerResult.next_tasks.find(item => item.id === child.id);
+            const executionContext = await plannerContext(job, plan);
+            const child = materializePlannerChild(job, plan, spec, executionContext);
             if (spec?.resolution)
                 await store.saveTask(attachResolutionOwnership(job, child, spec.resolution, 'planner'));
             await store.appendTask(child);
@@ -21,7 +20,6 @@ export async function recoverPlannerChildren(store, job, plannerContext) {
     const historicalPlanners = job.tasks.filter(task => task.kind === 'planner' && task.state === 'SUCCEEDED' && task.plannerResult);
     if (historicalPlanners.length === 0)
         return;
-    const executionContext = await plannerContext(job, historicalPlanners[0]);
     for (const planner of historicalPlanners) {
         const result = planner.plannerResult;
         const plan = { jobId: job.id, id: planner.id, round: planner.planningRound ?? 1, source: planner.planningSource ?? 'model-planner', previousPlanId: planner.previousPlannerTaskId ?? null, trigger: planner.planTrigger ?? { source: 'continuation' }, status: 'SUCCEEDED', guidance: planner.plannerGuidance, objectiveStatus: result.objective_status, plannerResult: result, assessment: result.assessment, generatedTaskIds: result.next_tasks.map(item => item.id), headSha: planner.planTrigger?.currentHeadSha ?? job.headSha, attempt: planner.attempt, maxAttempts: planner.maxAttempts, leaseOwner: null, leaseExpiresAt: null, executionId: null, timeoutMs: planner.timeoutMs, manualPlanning: planner.manualPlanning, createdAt: planner.createdAt, updatedAt: planner.updatedAt };
@@ -31,10 +29,10 @@ export async function recoverPlannerChildren(store, job, plannerContext) {
         }
         else
             continue;
-        const children = result.next_tasks.map(spec => materializePlannerChild(job, plan, spec, executionContext));
-        for (const child of children)
-            if (!job.tasks.some(item => item.id === child.id)) {
-                const spec = result.next_tasks.find(item => item.id === child.id);
+        for (const spec of result.next_tasks)
+            if (!job.tasks.some(item => item.id === spec.id)) {
+                const executionContext = await plannerContext(job, planner);
+                const child = materializePlannerChild(job, plan, spec, executionContext);
                 if (spec?.resolution)
                     await store.saveTask(attachResolutionOwnership(job, child, spec.resolution, 'planner'));
                 await store.appendTask(child);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agents-relay",
-  "version": "2.0.161",
+  "version": "2.0.162",
   "description": "Durable async agent jobs coordinated through GitHub pull requests",
   "repository": {
     "type": "git",
@@ -19,7 +19,7 @@
     "test:publish": "npm run build && node --import ./test/test-store-env.mjs --test --test-concurrency=1 --test-force-exit test/*.test.js",
     "test:fast": "npm run build && node --import ./test/test-store-env.mjs --test --test-concurrency=2 test/workflow-lifecycle.test.js test/workflow-v2-lifecycle-parity.test.js test/workflow-effects.test.js test/relay-store.test.js test/local-authority.test.js test/local-jobid-routing.test.js test/local-sqlite-jobid.test.js test/no-github.test.js test/scheduled-jobs.test.js test/reconcile-coordinator.test.js test/service-publication.test.js test/github-webhook.test.js",
     "test:full": "npm run build && node --import ./test/test-store-env.mjs --test --test-concurrency=1 test/*.test.js && npm run test:verification:l3",
-    "test:verification:l1": "npm run build && node --test test/verification/harness.test.mjs test/verification/contract.test.mjs test/verification/graph-inventory.test.mjs test/verification/dashboard-drilldown.test.mjs test/verification/evidence-matrix.test.mjs test/verification/task-resolution-classification.test.mjs test/verification/planner-contract.test.mjs",
+    "test:verification:l1": "npm run build && node --test test/verification/harness.test.mjs test/verification/contract.test.mjs test/verification/graph-inventory.test.mjs test/verification/dashboard-drilldown.test.mjs test/verification/evidence-matrix.test.mjs test/verification/task-resolution-classification.test.mjs test/verification/planner-contract.test.mjs test/planner-child-recovery.test.js",
     "test:verification:l3": "npm run build && node --test --test-concurrency=1 test/verification/harness.test.mjs test/verification/l3-sqlite.test.mjs test/verification/l3-ingress.test.mjs test/verification/l3-replay.test.mjs test/verification/objective-authority.test.mjs test/verification/l3-convergence.test.mjs test/verification/l3-review-convergence.test.mjs test/verification/l3-human-escalation.test.mjs test/verification/l3-escalation-notifier.test.mjs test/verification/l3-unknown-effect.test.mjs test/verification/l3-cli-actions.test.mjs test/verification/l3-planner-round.test.mjs test/verification/l3-planner-cli.test.mjs test/verification/l3-task-resolution.test.mjs test/verification/full-recovery-gates.test.mjs test/verification/production-wiring.test.mjs test/verification/l3-full-path.test.mjs test/verification/l3-projection-recovery.test.mjs",
     "test:verification": "npm run test:verification:l1 && npm run test:verification:l2 && npm run test:verification:l3 && npm run test:verification:cross-a && node --test test/workflow-production-outbox.test.js && npm run test:verification:matrix",
     "e2e": "node e2e/runner/run.mjs",