agents-relay 2.0.160 → 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/dist/planner.js

@@ -476,6 +476,10 @@ export class CommandObjectivePlanner {
 }
 const DEFAULT_PLANNER_AGENT = join(dirname(fileURLToPath(import.meta.url)), '..', 'skills', 'agents-relay', 'agents', 'planner.agent.md');
 const DEFAULT_TASK_JOB_CONTRACT = join(dirname(fileURLToPath(import.meta.url)), '..', 'skills', 'agents-relay', 'contracts', 'task-job-contract.md');
+export function buildPlannerCodexArgs(storeDirectory, input) {
+    return ['exec', '--json', '--sandbox', 'workspace-write', '--add-dir', resolvePath(dirname(storeDirectory)),
+        '-c', 'sandbox_workspace_write.network_access=true', '--', input];
+}
 function parsePlannerAgentMessage(line) {
     try {
         const value = JSON.parse(line);
@@ -504,7 +508,7 @@ export class AgentObjectivePlanner {
         const prompt = renderAgentPrompt(await buildReferenceFirstPrompt({ role: 'planner', adapter: 'codex', system: (effectivePlanner ? `[Effective planner contract SHA-256: ${effectivePlanner.hash}]\n\n` : '') + instructions.trim() + '\n\n## Shared Task / Job Contract\n\n' + contract.trim(), user: resolvedContext + '## Durable planner input\n\n' + plannerInput(context) }), 'codex');
         const runAgent = (input) => new Promise((resolve, reject) => {
             // Plan apply writes unified relay.sqlite beside AGENTS_RELAY_STORE_DIR.
-            const child = spawnOwnedProcess(this.command, ['exec', '--json', '--sandbox', 'workspace-write', '--add-dir', resolvePath(dirname(relayStoreDirectory())), '--', input], { invocation: 'planner.agent', signal, correlation: { jobId: context.job.id }, spawn: { cwd: context.executionContext?.project.path, stdio: ['ignore', 'pipe', 'pipe'] } });
+            const child = spawnOwnedProcess(this.command, buildPlannerCodexArgs(relayStoreDirectory(), input), { invocation: 'planner.agent', signal, correlation: { jobId: context.job.id }, spawn: { cwd: context.executionContext?.project.path, stdio: ['ignore', 'pipe', 'pipe'] } });
             let stdoutBuffer = '', finalMessage = '', stderrTail = '';
             const consume = (line) => {
                 const message = parsePlannerAgentMessage(line);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agents-relay",
-  "version": "2.0.160",
+  "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",