@poncho-ai/harness 0.39.0 → 0.39.1

package/.turbo/turbo-build.log

@@ -1,5 +1,5 @@
 
-> @poncho-ai/harness@0.39.0 build /home/runner/work/poncho-ai/poncho-ai/packages/harness
+> @poncho-ai/harness@0.39.1 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            479.07 KB
 ESM dist/isolate-TCWTUVG4.js 47.34 KB
-ESM ⚡️ Build success in 251ms
+ESM dist/index.js            480.10 KB
+ESM ⚡️ Build success in 223ms
 DTS Build start
-DTS ⚡️ Build success in 8269ms
+DTS ⚡️ Build success in 7012ms
 DTS dist/index.d.ts 73.95 KB

package/CHANGELOG.md

@@ -1,5 +1,38 @@
 # @poncho-ai/harness
 
+## 0.39.1
+
+### Patch Changes
+
+- [`244a3a3`](https://github.com/cesr/poncho-ai/commit/244a3a310c6c52f9e8535b28fb25d77829583d3f) Thanks [@cesr](https://github.com/cesr)! - fix(harness): don't archive `browser_screenshot` / `browser_snapshot` payloads
+
+  The per-conversation `_toolResultArchive` had no size cap or eviction, and
+  browser tool results were being archived in full — base64 JPEG screenshots
+  (~50-500KB each) and accessibility-tree snapshots accumulated for the lifetime
+  of a conversation. Heavy browser sessions OOM'd `poncho dev` after ~80 minutes.
+
+  Skip archiving for view-once tool results (`browser_screenshot`,
+  `browser_snapshot`). The model consumes them in-step; they're never retrieved
+  after-the-fact, so archiving them only burns memory.
+
+- [`d6248c8`](https://github.com/cesr/poncho-ai/commit/d6248c8b6d22e0fd0becde9e31dff7c12c724d84) Thanks [@cesr](https://github.com/cesr)! - fix(cli, harness): unify turn-parameter assembly so `conversation_recall` works everywhere
+
+  The recall tool relies on three context parameters (`__conversationRecallCorpus`,
+  `__conversationListFn`, `__conversationFetchFn`) that were only injected for
+  user-initiated HTTP turns. Cron, reminder, messaging-adapter, chat-continuation,
+  subagent-callback, and tool-approval-resume runs all built their own
+  `runInput.parameters` object and silently omitted these — causing
+  `conversation_recall` to throw "not available in this environment" or return
+  empty results depending on the call mode.
+
+  Introduces a single `buildTurnParameters(conversation, opts)` helper in the CLI
+  that owns context-parameter assembly (recall functions, `__activeConversationId`,
+  `__ownerId`, messaging metadata, tool-result archive). HTTP, messaging, and
+  cron/reminder paths now go through it. The harness orchestrator's three
+  internal turn sites (chat continuation, subagent-callback resume, tool-approval
+  resume) now call the existing `hooks.buildRecallParams` so they pick up the
+  recall functions too.
+
 ## 0.39.0
 
 ### Minor Changes

package/dist/index.js

@@ -8039,6 +8039,10 @@ var SKILL_TOOL_NAMES = [
   "list_skill_scripts",
   "run_skill_script"
 ];
+var NON_ARCHIVABLE_TOOL_NAMES = /* @__PURE__ */ new Set([
+  "browser_screenshot",
+  "browser_snapshot"
+]);
 var isAbortError = (error) => {
   if (!error || typeof error !== "object") {
     return false;
@@ -8743,7 +8747,7 @@ var AgentHarness = class _AgentHarness {
         if (row.content.startsWith(TOOL_RESULT_TRUNCATED_PREFIX)) return row;
         if (this.shouldPreserveSkillToolResult(row)) return row;
         const toolResultId = row.tool_use_id;
-        if (!archive[toolResultId]) {
+        if (!archive[toolResultId] && !NON_ARCHIVABLE_TOOL_NAMES.has(row.tool_name)) {
           archive[toolResultId] = {
             toolResultId,
             conversationId,
@@ -10506,7 +10510,7 @@ ${textContent}` };
             });
             {
               const archive = this.archivedToolResultsByConversation.get(conversationId);
-              if (archive) {
+              if (archive && !NON_ARCHIVABLE_TOOL_NAMES.has(result2.tool)) {
                 const payload = JSON.stringify(result2.output ?? null);
                 archive[result2.callId] = {
                   toolResultId: result2.callId,
@@ -11500,12 +11504,18 @@ var AgentOrchestrator = class {
     }
   }
   async runChatContinuation(conversationId, conversation, continuationMessages, onYield) {
+    const recallParams = this.hooks?.buildRecallParams?.({
+      ownerId: conversation.ownerId,
+      tenantId: conversation.tenantId,
+      excludeConversationId: conversationId
+    }) ?? {};
     const execution = await executeConversationTurn({
       harness: this.harness,
       runInput: {
         conversationId,
         tenantId: conversation.tenantId ?? void 0,
         parameters: withToolResultArchiveParam({
+          ...recallParams,
           __activeConversationId: conversationId,
           __ownerId: conversation.ownerId
         }, conversation),
@@ -11618,6 +11628,11 @@ var AgentOrchestrator = class {
     const fullCheckpointWithResults = resumeToolResultMsg ? [...fullCheckpointMessages, resumeToolResultMsg] : fullCheckpointMessages;
     let draftRef;
     let execution;
+    const resumeRecallParams = this.hooks?.buildRecallParams?.({
+      ownerId: conversation.ownerId,
+      tenantId: conversation.tenantId,
+      excludeConversationId: conversationId
+    }) ?? {};
     try {
       execution = await executeConversationTurn({
         harness: this.harness,
@@ -11625,6 +11640,11 @@ var AgentOrchestrator = class {
           messages: fullCheckpointMessages,
           toolResults,
           conversationId,
+          parameters: withToolResultArchiveParam({
+            ...resumeRecallParams,
+            __activeConversationId: conversationId,
+            __ownerId: conversation.ownerId
+          }, conversation),
           abortSignal: abortController.signal
         }),
         initialContextTokens: conversation.contextTokens ?? 0,
@@ -12260,6 +12280,11 @@ ${resultBody}`,
       `[poncho][subagent-callback] conversation="${conversationId}" history_source=${historySelection.source}`
     );
     let execution;
+    const recallParams = this.hooks?.buildRecallParams?.({
+      ownerId: conversation.ownerId,
+      tenantId: conversation.tenantId,
+      excludeConversationId: conversationId
+    }) ?? {};
     try {
       execution = await executeConversationTurn({
         harness: this.harness,
@@ -12268,6 +12293,7 @@ ${resultBody}`,
           conversationId,
           tenantId: conversation.tenantId ?? void 0,
           parameters: withToolResultArchiveParam({
+            ...recallParams,
             __activeConversationId: conversationId,
             __ownerId: conversation.ownerId
           }, conversation),

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@poncho-ai/harness",
-  "version": "0.39.0",
+  "version": "0.39.1",
   "description": "Agent execution runtime - conversation loop, tool dispatch, streaming",
   "repository": {
     "type": "git",

package/src/harness.ts

@@ -142,6 +142,15 @@ const SKILL_TOOL_NAMES = [
   "run_skill_script",
 ] as const;
 
+// Tools whose results are "view-once" — the model consumes the payload in the
+// step where it's returned and never needs it retrieved later. Archiving them
+// caused unbounded heap growth on browser-heavy sessions (each screenshot
+// is ~50-500KB base64 and accumulated for the lifetime of the conversation).
+const NON_ARCHIVABLE_TOOL_NAMES = new Set<string>([
+  "browser_screenshot",
+  "browser_snapshot",
+]);
+
 const isAbortError = (error: unknown): boolean => {
   if (!error || typeof error !== "object") {
     return false;
@@ -969,7 +978,7 @@ export class AgentHarness {
         if (row.content.startsWith(TOOL_RESULT_TRUNCATED_PREFIX)) return row;
         if (this.shouldPreserveSkillToolResult(row)) return row;
         const toolResultId = row.tool_use_id;
-        if (!archive[toolResultId]) {
+        if (!archive[toolResultId] && !NON_ARCHIVABLE_TOOL_NAMES.has(row.tool_name)) {
           archive[toolResultId] = {
             toolResultId,
             conversationId,
@@ -3068,7 +3077,7 @@ Code is wrapped in an async IIFE — use \`return\` to return a value to the too
           });
           {
             const archive = this.archivedToolResultsByConversation.get(conversationId);
-            if (archive) {
+            if (archive && !NON_ARCHIVABLE_TOOL_NAMES.has(result.tool)) {
               const payload = JSON.stringify(result.output ?? null);
               archive[result.callId] = {
                 toolResultId: result.callId,

package/src/orchestrator/orchestrator.ts

@@ -190,12 +190,18 @@ export class AgentOrchestrator {
     continuationMessages: Message[],
     onYield?: (event: AgentEvent) => void | Promise<void>,
   ): Promise<void> {
+    const recallParams = this.hooks?.buildRecallParams?.({
+      ownerId: conversation.ownerId,
+      tenantId: conversation.tenantId,
+      excludeConversationId: conversationId,
+    }) ?? {};
     const execution = await executeConversationTurn({
       harness: this.harness,
       runInput: {
         conversationId,
         tenantId: conversation.tenantId ?? undefined,
         parameters: withToolResultArchiveParam({
+          ...recallParams,
           __activeConversationId: conversationId,
           __ownerId: conversation.ownerId,
         }, conversation),
@@ -332,6 +338,11 @@ export class AgentOrchestrator {
 
     let draftRef: TurnDraftState | undefined;
     let execution: ExecuteTurnResult | undefined;
+    const resumeRecallParams = this.hooks?.buildRecallParams?.({
+      ownerId: conversation.ownerId,
+      tenantId: conversation.tenantId,
+      excludeConversationId: conversationId,
+    }) ?? {};
 
     try {
       execution = await executeConversationTurn({
@@ -340,6 +351,11 @@ export class AgentOrchestrator {
           messages: fullCheckpointMessages,
           toolResults,
           conversationId,
+          parameters: withToolResultArchiveParam({
+            ...resumeRecallParams,
+            __activeConversationId: conversationId,
+            __ownerId: conversation.ownerId,
+          }, conversation),
           abortSignal: abortController.signal,
         }),
         initialContextTokens: conversation.contextTokens ?? 0,
@@ -1073,6 +1089,11 @@ export class AgentOrchestrator {
       `[poncho][subagent-callback] conversation="${conversationId}" history_source=${historySelection.source}`,
     );
     let execution: ExecuteTurnResult | undefined;
+    const recallParams = this.hooks?.buildRecallParams?.({
+      ownerId: conversation.ownerId,
+      tenantId: conversation.tenantId,
+      excludeConversationId: conversationId,
+    }) ?? {};
 
     try {
       execution = await executeConversationTurn({
@@ -1082,6 +1103,7 @@ export class AgentOrchestrator {
           conversationId,
           tenantId: conversation.tenantId ?? undefined,
           parameters: withToolResultArchiveParam({
+            ...recallParams,
             __activeConversationId: conversationId,
             __ownerId: conversation.ownerId,
           }, conversation),