agents-relay 1.0.113 → 1.0.115

package/dist/planner.js

@@ -11,9 +11,9 @@ export function planningSpec(task) {
         throw new Error(`Initial task ${task.id} requires a supported adapter`);
     return { id: task.id, input: task.input, priority: task.priority, storySize: task.storySize, projectName: task.projectName, agentName: task.agentName, agentGraph: task.agentGraph, dependencies: task.dependencies, capabilities: task.capabilities, workerRequirements: task.workerRequirements, adapter: task.adapter, output: task.output ?? { kind: 'task_pr' }, routing: task.routing, continuation: task.continuation, maxAttempts: task.maxAttempts, timeoutMs: task.timeoutMs };
 }
-function record(value) {
+function record(value, field = 'result') {
     if (!value || typeof value !== 'object' || Array.isArray(value))
-        throw new Error('Planner result must be an object');
+        throw new Error(`Planner ${field} must be an object`);
     return value;
 }
 function optionalString(value, field) {
@@ -33,7 +33,7 @@ function stringArray(value, field, fallback = []) {
 function workerRequirements(value, field) {
     if (value === undefined || value === null)
         return undefined;
-    const root = record(value);
+    const root = record(value, field);
     const capabilities = stringArray(root.capabilities, `${field}.capabilities`);
     const optional = (name) => optionalString(root[name], `${field}.${name}`);
     const labelsValue = root.labels;
@@ -77,14 +77,14 @@ export function normalizePlannerPriority(value, field) {
 function routing(value, field) {
     if (value === undefined || value === null)
         return undefined;
-    const route = record(value);
+    const route = record(value, field);
     for (const key of ['provider', 'model', 'decidedBy', 'decidedAt'])
         if (typeof route[key] !== 'string' || String(route[key]).length === 0)
             throw new Error(`Planner ${field}.${key} is required`);
     return { provider: route.provider, model: route.model, profile: optionalString(route.profile, `${field}.profile`), reasoning: optionalString(route.reasoning, `${field}.reasoning`), cwd: optionalString(route.cwd, `${field}.cwd`), projectId: optionalString(route.projectId, `${field}.projectId`), decidedBy: route.decidedBy, decidedAt: route.decidedAt };
 }
 export function parsePlannerResult(value) {
-    const root = record(value);
+    const root = record(value, 'result');
     const status = root.objective_status;
     if (status !== 'in_progress' && status !== 'satisfied')
         throw new Error('Planner objective_status must be in_progress or satisfied');
@@ -94,7 +94,7 @@ export function parsePlannerResult(value) {
         throw new Error('Planner next_tasks must be an array');
     const ids = new Set();
     const next_tasks = root.next_tasks.map((item, index) => {
-        const task = record(item);
+        const task = record(item, `next_tasks[${index}]`);
         const id = task.id;
         if (typeof id !== 'string' || id.length === 0)
             throw new Error(`Planner next_tasks[${index}].id is required`);
@@ -106,7 +106,7 @@ export function parsePlannerResult(value) {
         const adapter = task.adapter;
         if (adapter !== undefined && !['codex', 'chatgpt'].includes(String(adapter)))
             throw new Error(`Planner next_tasks[${index}].adapter must be codex or chatgpt when provided`);
-        const output = task.output === undefined ? undefined : record(task.output);
+        const output = task.output === undefined ? undefined : record(task.output, `next_tasks[${index}].output`);
         if (!output || (output.kind !== 'task_pr' && output.kind !== 'file'))
             throw new Error(`Planner next_tasks[${index}].output is required`);
         if (output.kind === 'file' && (typeof output.path !== 'string' || output.path.length === 0))

package/dist/reconciler.js

@@ -292,7 +292,7 @@ export class Reconciler extends ReconcilerReviewBase {
         const contextTask = job.tasks.find(task => task.kind === 'planner') ?? job.tasks[0];
         if (!contextTask)
             return;
-        const executionContext = await this.resolvedContext(job, contextTask, 'planner');
+        const executionContext = await this.plannerContext(job, contextTask);
         for (const planner of job.tasks.filter(task => task.kind === 'planner' && task.state === 'SUCCEEDED' && task.plannerResult)) {
             const children = planner.plannerResult?.next_tasks.map(spec => this.plannerChild(job, planner, spec, executionContext)) ?? [];
             for (const child of children) {
@@ -313,6 +313,17 @@ export class Reconciler extends ReconcilerReviewBase {
             runtimes: this.options.adapters.map(adapter => ({ name: adapter.name, id: adapter.id, capabilities: [...adapter.capabilities] })),
         });
     }
+    async plannerContext(job, task) {
+        let projectPath = this.options.projectPathForJob ? await this.options.projectPathForJob(job) : task.routing?.cwd ?? process.cwd();
+        if (this.store.pullRequestHead) {
+            const head = await this.store.pullRequestHead();
+            if (!head)
+                throw new Error(`PR #${job.prNumber} has no resolvable head ref`);
+            const key = `planner-${task.id}-${head.sha.slice(0, 12)}`;
+            projectPath = await prepareManagedExecutionWorktree(projectPath, job.repository, job.prNumber, head, key);
+        }
+        return this.resolvedContext(job, task, 'planner', projectPath);
+    }
     async launch(job, task) {
         const adapter = this.options.adapters.find(x => x.name === task.adapter);
         if (!adapter) {
@@ -443,7 +454,7 @@ export class Reconciler extends ReconcilerReviewBase {
         this.plannerLive.set(task.id, { executionId: executionId, controller, timer });
         let promise;
         try {
-            const executionContext = await this.resolvedContext(job, task, 'planner');
+            const executionContext = await this.plannerContext(job, task);
             const context = { job, tasks: job.tasks, objective: job.objective ?? job.description ?? job.title, previousPlannerTaskId: task.parentTaskId, executionContext, retryFeedback: task.plannerRetryFeedback };
             task.input = plannerInput(context);
             await this.store.saveTask(task);
@@ -483,7 +494,7 @@ export class Reconciler extends ReconcilerReviewBase {
         }
         try {
             const result = parsePlannerResult(raw);
-            const executionContext = await this.resolvedContext(job, task, 'planner');
+            const executionContext = await this.plannerContext(job, task);
             const children = result.next_tasks.map(spec => this.plannerChild(job, task, spec, executionContext));
             for (const child of children) {
                 const existing = job.tasks.find(item => item.id === child.id);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agents-relay",
-  "version": "1.0.113",
+  "version": "1.0.115",
   "description": "Durable async agent jobs coordinated through GitHub pull requests",
   "type": "module",
   "bin": {

package/skills/agents-relay/agents/planner.agent.md

@@ -87,3 +87,36 @@ For `objective_status: "in_progress"`, `assessment` must be Markdown containing:
 For `objective_status: "satisfied"`, `assessment` may be a concise evidence-based completion assessment without a graph.
 
 Each item in next_tasks contains id, input, output, and storySize. It may also contain priority, projectName, agentName, dependencies, capabilities, workerRequirements, maxAttempts, and timeoutMs. Adapter/routing fields are selected by Relay after planning; do not invent them.
+
+### Exact `next_tasks` serialization
+
+Use these exact serialized forms:
+
+- `output` is always an object, never a string:
+  - PR-backed work: `{"kind":"task_pr"}`
+  - File artifact: `{"kind":"file","path":"runs/2026-09-26/example/output.md"}`
+- `agentGraph`, when present, is one Mermaid graph **string**, for example `"graph LR\n  build-log-editor --> build-log-producer"`. Do not emit an object with `nodes` or `edges`.
+- `priority`, when present, is exactly `"P0"`, `"P1"`, `"P2"`, or `"P3"`; otherwise omit it.
+- `storySize` is exactly `"S"`, `"M"`, or `"L"`.
+
+A valid non-satisfied response:
+
+```json
+{
+  "objective_status": "in_progress",
+  "assessment": "```mermaid\nflowchart LR\n  A[Evidence] --> B[Content package]\n```\nOne daily package is the durable Task boundary.",
+  "next_tasks": [
+    {
+      "id": "episode-2026-09-26",
+      "input": "Produce and verify the dated content package.",
+      "output": {"kind": "file", "path": "runs/2026-09-26/package.md"},
+      "storySize": "L",
+      "priority": "P1",
+      "agentName": "build-log-editor",
+      "agentGraph": "graph LR\n  build-log-editor --> build-log-producer"
+    }
+  ]
+}
+```
+
+Return this JSON object directly. Do not wrap the whole result in another JSON string.