@poncho-ai/harness 0.52.2 → 0.53.0

package/.turbo/turbo-build.log

@@ -1,5 +1,5 @@
 
-> @poncho-ai/harness@0.52.2 build /home/runner/work/poncho-ai/poncho-ai/packages/harness
+> @poncho-ai/harness@0.53.0 build /home/runner/work/poncho-ai/poncho-ai/packages/harness
 > node scripts/embed-docs.js && tsup src/index.ts --format esm --dts
 
 [embed-docs] Generated poncho-docs.ts with 4 topics
@@ -8,9 +8,9 @@
 CLI tsup v8.5.1
 CLI Target: es2022
 ESM Build start
-ESM dist/index.js            540.60 KB
 ESM dist/isolate-F2PPSUL6.js 53.82 KB
-ESM ⚡️ Build success in 204ms
+ESM dist/index.js            541.42 KB
+ESM ⚡️ Build success in 239ms
 DTS Build start
-DTS ⚡️ Build success in 7242ms
-DTS dist/index.d.ts 93.60 KB
+DTS ⚡️ Build success in 7708ms
+DTS dist/index.d.ts 94.04 KB

package/CHANGELOG.md

@@ -1,5 +1,37 @@
 # @poncho-ai/harness
 
+## 0.53.0
+
+### Minor Changes
+
+- [#145](https://github.com/cesr/poncho-ai/pull/145) [`bfa4976`](https://github.com/cesr/poncho-ai/commit/bfa4976ac8b05a300e22271e23c3bae4aadae2a8) Thanks [@cesr](https://github.com/cesr)! - events: add stable identity so streaming clients match instead of guess
+
+  Additive fields that let a streaming client reconstruct view-state by
+  identity rather than inferring structure from event order (the source of a
+  class of reconnect/subagent rendering bugs):
+  - `tool:started` / `tool:completed` / `tool:error` now carry `toolCallId`
+    (already in scope as `call.id` / `result.callId`). Clients match tool
+    pills by id instead of by tool name.
+  - `subagent:spawned|completed|error|stopped` now carry `parentToolCallId`
+    (the `spawn_subagent` tool call's id) and `task`; `completed`/`error`
+    also carry `resultText`. Clients attach subagent state to the spawning
+    tool's pill and render the result inline — no header-regex or
+    sequential-cursor pairing needed.
+  - `ToolContext` gains `toolCallId` so the `spawn_subagent` handler can
+    record which call produced the subagent (plumbed: tool-dispatcher →
+    spawn handler → `SubagentSpawnOptions.parentToolCallId` →
+    `subagentMeta.parentToolCallId` → the events above).
+  - `run:started` gains an optional `cause` field in the type
+    (`user|continuation|subagent_callback|approval_resume`); emission is
+    deferred to a later pass.
+
+  All fields are additive; older clients ignore them.
+
+### Patch Changes
+
+- Updated dependencies [[`bfa4976`](https://github.com/cesr/poncho-ai/commit/bfa4976ac8b05a300e22271e23c3bae4aadae2a8)]:
+  - @poncho-ai/sdk@1.15.0
+
 ## 0.52.2
 
 ### Patch Changes

package/dist/index.d.ts

@@ -199,6 +199,9 @@ interface Conversation {
          *  subagent's runs emit no telemetry (e.g. spawned from an incognito
          *  turn). Read by the orchestrator's runSubagent / continuation. */
         suppressTelemetry?: boolean;
+        /** The parent's `spawn_subagent` tool call id — echoed onto subagent:*
+         *  events so a client can attach subagent state to that tool's pill. */
+        parentToolCallId?: string;
     };
     channelMeta?: {
         platform: string;
@@ -1190,6 +1193,10 @@ interface SubagentManager {
         /** Inherit the parent run's telemetry choice — when true, the subagent
          *  run (and its re-runs) emit no telemetry. */
         suppressTelemetry?: boolean;
+        /** The id of the `spawn_subagent` tool call that produced this subagent,
+         *  so its events can carry `parentToolCallId` and a client can attach
+         *  subagent state to the spawning tool's pill. */
+        parentToolCallId?: string;
     }): Promise<SubagentSpawnResult>;
     sendMessage(subagentId: string, message: string): Promise<SubagentSpawnResult>;
     stop(subagentId: string): Promise<void>;

package/dist/index.js

@@ -8323,7 +8323,8 @@ var createSubagentTools = (manager) => [
         parentConversationId: conversationId,
         ownerId,
         tenantId: context.tenantId,
-        suppressTelemetry: context.suppressTelemetry
+        suppressTelemetry: context.suppressTelemetry,
+        parentToolCallId: context.toolCallId
       });
       return { subagentId, status: "running" };
     }
@@ -8585,7 +8586,7 @@ var ToolDispatcher = class {
       };
     }
     try {
-      const output = await definition.handler(call.input, context);
+      const output = await definition.handler(call.input, { ...context, toolCallId: call.id });
       if (context.abortSignal?.aborted) {
         return {
           callId: call.id,
@@ -11170,7 +11171,7 @@ ${textContent}` };
               return;
             }
             const runtimeToolName = exposedToolNames.get(call.name) ?? call.name;
-            yield pushEvent({ type: "tool:started", tool: runtimeToolName, input: call.input });
+            yield pushEvent({ type: "tool:started", tool: runtimeToolName, toolCallId: call.id, input: call.input });
             if (this.requiresApprovalForToolCall(runtimeToolName, call.input)) {
               approvalNeeded.push({
                 approvalId: `approval_${randomUUID5()}`,
@@ -11363,6 +11364,7 @@ ${textContent}` };
               yield pushEvent({
                 type: "tool:error",
                 tool: result2.tool,
+                toolCallId: result2.callId,
                 error: result2.error,
                 recoverable: true
               });
@@ -11404,6 +11406,7 @@ ${textContent}` };
               yield pushEvent({
                 type: "tool:completed",
                 tool: result2.tool,
+                toolCallId: result2.callId,
                 input: callInputMap.get(result2.callId),
                 output: result2.output,
                 duration: now() - batchStart,
@@ -12826,11 +12829,14 @@ var AgentOrchestrator = class {
       result: { status: "completed", response: responseText, steps: 0, tokens: { input: 0, output: 0, cached: 0 }, duration: 0 },
       timestamp: Date.now()
     };
-    await this.conversationStore.appendSubagentResult(conv.parentConversationId, pendingResult);
+    await this.appendSubagentResultReliable(conv.parentConversationId, pendingResult);
     await this.eventSink(conv.parentConversationId, {
       type: "subagent:completed",
       subagentId,
-      conversationId: subagentId
+      conversationId: subagentId,
+      task: conv.subagentMeta?.task ?? conv.title,
+      parentToolCallId: conv.subagentMeta?.parentToolCallId,
+      resultText: responseText
     });
     await this.triggerParentCallback(conv.parentConversationId);
   }
@@ -12906,9 +12912,11 @@ var AgentOrchestrator = class {
     let latestRunId = "";
     let runResult;
     let runError;
+    let parentToolCallId;
     try {
       const conversation = await this.conversationStore.getWithArchive(childConversationId);
       if (!conversation) throw new Error("Subagent conversation not found");
+      parentToolCallId = conversation.subagentMeta?.parentToolCallId;
       if (conversation.subagentMeta?.status === "stopped") return;
       conversation.lastActivityAt = Date.now();
       await this.conversationStore.update(conversation);
@@ -13126,7 +13134,10 @@ var AgentOrchestrator = class {
       await this.eventSink(parentConversationId, {
         type: "subagent:completed",
         subagentId: childConversationId,
-        conversationId: childConversationId
+        conversationId: childConversationId,
+        task,
+        parentToolCallId,
+        resultText: subagentResponse
       });
       this.triggerParentCallback(parentConversationId).catch(
         (err) => console.error(`[poncho][subagent] Parent callback failed:`, err instanceof Error ? err.message : err)
@@ -13157,7 +13168,9 @@ var AgentOrchestrator = class {
         type: "subagent:error",
         subagentId: childConversationId,
         conversationId: childConversationId,
-        error: errMsg
+        error: errMsg,
+        task,
+        parentToolCallId
       });
       this.triggerParentCallback(parentConversationId).catch(
         (err2) => console.error(`[poncho][subagent] Parent callback failed:`, err2 instanceof Error ? err2.message : err2)
@@ -13486,7 +13499,10 @@ ${resultBody}`,
       await this.eventSink(parentConversationId, {
         type: "subagent:completed",
         subagentId: conversationId,
-        conversationId
+        conversationId,
+        task,
+        parentToolCallId: conversation.subagentMeta?.parentToolCallId,
+        resultText: subagentResponse
       });
       if (parentConv) {
         if (this.isServerless) {
@@ -13530,7 +13546,9 @@ ${resultBody}`,
       await this.eventSink(conversation.parentConversationId, {
         type: "subagent:completed",
         subagentId: conversationId,
-        conversationId
+        conversationId,
+        task,
+        parentToolCallId: conversation.subagentMeta?.parentToolCallId
       });
       if (parentConv) {
         if (this.isServerless) {
@@ -13560,7 +13578,7 @@ ${resultBody}`,
           opts.tenantId ?? null,
           {
             parentConversationId: opts.parentConversationId,
-            subagentMeta: { task: opts.task, status: "running", suppressTelemetry: opts.suppressTelemetry },
+            subagentMeta: { task: opts.task, status: "running", suppressTelemetry: opts.suppressTelemetry, parentToolCallId: opts.parentToolCallId },
             messages: [{ role: "user", content: opts.task }]
           }
         );
@@ -13572,7 +13590,8 @@ ${resultBody}`,
           type: "subagent:spawned",
           subagentId: conversation.conversationId,
           conversationId: conversation.conversationId,
-          task: opts.task
+          task: opts.task,
+          parentToolCallId: opts.parentToolCallId
         });
         if (this.isServerless) {
           this.hooks.dispatchBackground("subagent-run", conversation.conversationId);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@poncho-ai/harness",
-  "version": "0.52.2",
+  "version": "0.53.0",
   "description": "Agent execution runtime - conversation loop, tool dispatch, streaming",
   "repository": {
     "type": "git",
@@ -34,7 +34,7 @@
     "mustache": "^4.2.0",
     "yaml": "^2.4.0",
     "zod": "^3.22.0",
-    "@poncho-ai/sdk": "1.14.0"
+    "@poncho-ai/sdk": "1.15.0"
   },
   "peerDependencies": {
     "esbuild": ">=0.17.0",

package/src/harness.ts

@@ -3344,7 +3344,7 @@ Code is wrapped in an async IIFE — use \`return\` to return a value to the too
           return;
         }
         const runtimeToolName = exposedToolNames.get(call.name) ?? call.name;
-        yield pushEvent({ type: "tool:started", tool: runtimeToolName, input: call.input });
+        yield pushEvent({ type: "tool:started", tool: runtimeToolName, toolCallId: call.id, input: call.input });
         if (this.requiresApprovalForToolCall(runtimeToolName, call.input)) {
           approvalNeeded.push({
             approvalId: `approval_${randomUUID()}`,
@@ -3563,6 +3563,7 @@ Code is wrapped in an async IIFE — use \`return\` to return a value to the too
           yield pushEvent({
             type: "tool:error",
             tool: result.tool,
+            toolCallId: result.callId,
             error: result.error,
             recoverable: true,
           });
@@ -3604,6 +3605,7 @@ Code is wrapped in an async IIFE — use \`return\` to return a value to the too
           yield pushEvent({
             type: "tool:completed",
             tool: result.tool,
+            toolCallId: result.callId,
             input: callInputMap.get(result.callId),
             output: result.output,
             duration: now() - batchStart,

package/src/orchestrator/orchestrator.ts

@@ -693,12 +693,15 @@ export class AgentOrchestrator {
       result: { status: "completed", response: responseText, steps: 0, tokens: { input: 0, output: 0, cached: 0 }, duration: 0 },
       timestamp: Date.now(),
     };
-    await this.conversationStore.appendSubagentResult(conv.parentConversationId, pendingResult);
+    await this.appendSubagentResultReliable(conv.parentConversationId, pendingResult);
 
     await this.eventSink(conv.parentConversationId, {
       type: "subagent:completed",
       subagentId,
       conversationId: subagentId,
+      task: conv.subagentMeta?.task ?? conv.title,
+      parentToolCallId: conv.subagentMeta?.parentToolCallId,
+      resultText: responseText,
     });
 
     await this.triggerParentCallback(conv.parentConversationId);
@@ -796,10 +799,14 @@ export class AgentOrchestrator {
     let latestRunId = "";
     let runResult: { status: "completed" | "error" | "cancelled"; response?: string; steps: number; duration: number; continuation?: boolean; continuationMessages?: Message[] } | undefined;
     let runError: { code?: string; message?: string } | undefined;
+    // The spawning tool call's id — echoed onto subagent:* events so the
+    // client can attach subagent state to that tool's pill.
+    let parentToolCallId: string | undefined;
 
     try {
       const conversation = await this.conversationStore.getWithArchive(childConversationId);
       if (!conversation) throw new Error("Subagent conversation not found");
+      parentToolCallId = conversation.subagentMeta?.parentToolCallId;
 
       if (conversation.subagentMeta?.status === "stopped") return;
 
@@ -1054,6 +1061,9 @@ export class AgentOrchestrator {
         type: "subagent:completed",
         subagentId: childConversationId,
         conversationId: childConversationId,
+        task,
+        parentToolCallId,
+        resultText: subagentResponse,
       });
 
       this.triggerParentCallback(parentConversationId).catch(err =>
@@ -1090,6 +1100,8 @@ export class AgentOrchestrator {
         subagentId: childConversationId,
         conversationId: childConversationId,
         error: errMsg,
+        task,
+        parentToolCallId,
       });
 
       this.triggerParentCallback(parentConversationId).catch(err2 =>
@@ -1476,6 +1488,9 @@ export class AgentOrchestrator {
         type: "subagent:completed",
         subagentId: conversationId,
         conversationId,
+        task,
+        parentToolCallId: conversation.subagentMeta?.parentToolCallId,
+        resultText: subagentResponse,
       });
 
       if (parentConv) {
@@ -1520,6 +1535,8 @@ export class AgentOrchestrator {
         type: "subagent:completed",
         subagentId: conversationId,
         conversationId,
+        task,
+        parentToolCallId: conversation.subagentMeta?.parentToolCallId,
       });
 
       if (parentConv) {
@@ -1553,7 +1570,7 @@ export class AgentOrchestrator {
           opts.tenantId ?? null,
           {
             parentConversationId: opts.parentConversationId,
-            subagentMeta: { task: opts.task, status: "running", suppressTelemetry: opts.suppressTelemetry },
+            subagentMeta: { task: opts.task, status: "running", suppressTelemetry: opts.suppressTelemetry, parentToolCallId: opts.parentToolCallId },
             messages: [{ role: "user", content: opts.task }],
           },
         );
@@ -1568,6 +1585,7 @@ export class AgentOrchestrator {
           subagentId: conversation.conversationId,
           conversationId: conversation.conversationId,
           task: opts.task,
+          parentToolCallId: opts.parentToolCallId,
         });
 
         if (this.isServerless) {

package/src/state.ts

@@ -79,6 +79,9 @@ export interface Conversation {
      *  subagent's runs emit no telemetry (e.g. spawned from an incognito
      *  turn). Read by the orchestrator's runSubagent / continuation. */
     suppressTelemetry?: boolean;
+    /** The parent's `spawn_subagent` tool call id — echoed onto subagent:*
+     *  events so a client can attach subagent state to that tool's pill. */
+    parentToolCallId?: string;
   };
   channelMeta?: {
     platform: string;

package/src/subagent-manager.ts

@@ -40,6 +40,10 @@ export interface SubagentManager {
     /** Inherit the parent run's telemetry choice — when true, the subagent
      *  run (and its re-runs) emit no telemetry. */
     suppressTelemetry?: boolean;
+    /** The id of the `spawn_subagent` tool call that produced this subagent,
+     *  so its events can carry `parentToolCallId` and a client can attach
+     *  subagent state to the spawning tool's pill. */
+    parentToolCallId?: string;
   }): Promise<SubagentSpawnResult>;
 
   sendMessage(subagentId: string, message: string): Promise<SubagentSpawnResult>;

package/src/subagent-tools.ts

@@ -46,6 +46,7 @@ export const createSubagentTools = (
         ownerId,
         tenantId: context.tenantId,
         suppressTelemetry: context.suppressTelemetry,
+        parentToolCallId: context.toolCallId,
       });
       return { subagentId, status: "running" };
     },

package/src/tool-dispatcher.ts

@@ -62,7 +62,10 @@ export class ToolDispatcher {
     }
 
     try {
-      const output = await definition.handler(call.input, context);
+      // Per-call context: stamp the current tool call's id so handlers that
+      // spawn further work (spawn_subagent) can record `parentToolCallId`.
+      // A fresh object — `context` is shared across a batch, don't mutate it.
+      const output = await definition.handler(call.input, { ...context, toolCallId: call.id });
       if (context.abortSignal?.aborted) {
         return {
           callId: call.id,