@poncho-ai/browser 0.6.3 → 0.6.5

package/.turbo/turbo-build.log

@@ -1,5 +1,5 @@
 
-> @poncho-ai/browser@0.6.3 build /home/runner/work/poncho-ai/poncho-ai/packages/browser
+> @poncho-ai/browser@0.6.5 build /home/runner/work/poncho-ai/poncho-ai/packages/browser
 > tsup src/index.ts --format esm --dts
 
 CLI Building entry: src/index.ts
@@ -7,8 +7,8 @@
 CLI tsup v8.5.1
 CLI Target: es2022
 ESM Build start
-ESM dist/index.js 44.07 KB
-ESM ⚡️ Build success in 79ms
+ESM dist/index.js 45.97 KB
+ESM ⚡️ Build success in 73ms
 DTS Build start
-DTS ⚡️ Build success in 4573ms
-DTS dist/index.d.ts 13.72 KB
+DTS ⚡️ Build success in 4746ms
+DTS dist/index.d.ts 13.69 KB

package/CHANGELOG.md

@@ -1,5 +1,31 @@
 # @poncho-ai/browser
 
+## 0.6.5
+
+### Patch Changes
+
+- [#42](https://github.com/cesr/poncho-ai/pull/42) [`e58a984`](https://github.com/cesr/poncho-ai/commit/e58a984efaa673b649318102bbf735fb4c2f9172) Thanks [@cesr](https://github.com/cesr)! - Add continuation model and fire-and-forget subagents
+
+  **Continuation model**: Agents no longer send a synthetic `"Continue"` user message between steps. Instead, the harness injects a transient signal when needed, and the full internal message chain is preserved across continuations so the LLM never loses context. `RunInput` gains `disableSoftDeadline` and `RunResult` gains `continuationMessages`.
+
+  **Fire-and-forget subagents**: Subagents now run asynchronously in the background. `spawn_subagent` returns immediately with a subagent ID; results are delivered back to the parent conversation as a callback once the subagent completes. Subagents cannot spawn their own subagents. The web UI shows results in a collapsible disclosure and reconnects the live event stream automatically when the parent agent resumes.
+
+  **Bug fixes**:
+  - Fixed a race condition where concurrent runs on the same harness instance could assign a subagent or browser tab to the wrong parent conversation (shared `_currentRunConversationId` field replaced with per-run `ToolContext.conversationId`).
+  - Fixed Upstash KV store silently dropping large values by switching from URL-path encoding to request body format for `SET`/`SETEX` commands.
+  - Fixed empty assistant content blocks causing Anthropic `text content blocks must be non-empty` errors.
+
+  **Client**: Added `getConversationStatus()` and `waitForSubagents` option on `sendMessage()`.
+
+- Updated dependencies [[`e58a984`](https://github.com/cesr/poncho-ai/commit/e58a984efaa673b649318102bbf735fb4c2f9172)]:
+  - @poncho-ai/sdk@1.6.0
+
+## 0.6.4
+
+### Patch Changes
+
+- [`b5af10a`](https://github.com/cesr/poncho-ai/commit/b5af10a2f7b0023c683f14cd465105f8ddfff0ee) Thanks [@cesr](https://github.com/cesr)! - Fix browser cookie restore failing with "Invalid parameters" by sanitizing Playwright-format cookies to CDP-compatible format before calling Network.setCookies. Falls back to per-cookie restore when batch call fails.
+
 ## 0.6.3
 
 ### Patch Changes

package/dist/index.d.ts

@@ -166,7 +166,7 @@ declare class BrowserSession {
     private emitStatus;
 }
 
-declare function createBrowserTools(getSession: () => BrowserSession, getConversationId: () => string): ToolDefinition[];
+declare function createBrowserTools(getSession: () => BrowserSession): ToolDefinition[];
 
 /**
  * Returns a realistic Chrome user-agent string for the host OS.

package/dist/index.js

@@ -186,6 +186,24 @@ async function getBrowserManagerCtor() {
   return BrowserManagerCtor;
 }
 var MAX_TABS = 8;
+var VALID_SAME_SITE = ["Strict", "Lax", "None"];
+function sanitizeCookieForCDP(c) {
+  const name = typeof c.name === "string" ? c.name : "";
+  const value = typeof c.value === "string" ? c.value : "";
+  if (!name) return null;
+  const out = { name, value };
+  if (typeof c.domain === "string" && c.domain) out.domain = c.domain;
+  if (typeof c.path === "string") out.path = c.path;
+  if (typeof c.secure === "boolean") out.secure = c.secure;
+  if (typeof c.httpOnly === "boolean") out.httpOnly = c.httpOnly;
+  if (typeof c.expires === "number" && c.expires > 0) {
+    out.expires = c.expires;
+  }
+  if (typeof c.sameSite === "string" && VALID_SAME_SITE.includes(c.sameSite)) {
+    out.sameSite = c.sameSite;
+  }
+  return out;
+}
 var SAME_TAB_INIT_SCRIPT = `
 (() => {
   // Override window.open to navigate in-place
@@ -836,6 +854,10 @@ var BrowserSession = class {
     tab.frameListeners.add(listener);
     return () => {
       tab.frameListeners.delete(listener);
+      if (tab.frameListeners.size === 0 && this._screencastConversation === conversationId) {
+        this.stopScreencast().catch(() => {
+        });
+      }
     };
   }
   onStatus(conversationId, listener) {
@@ -941,8 +963,28 @@ var BrowserSession = class {
       if (!json) return;
       const state = JSON.parse(json);
       if (state.cookies?.length) {
-        await cdp.send("Network.setCookies", { cookies: state.cookies });
-        console.log(`[poncho][browser] Restored ${state.cookies.length} cookies`);
+        const sanitized = state.cookies.map(sanitizeCookieForCDP).filter((c) => c !== null);
+        if (sanitized.length) {
+          try {
+            await cdp.send("Network.setCookies", { cookies: sanitized });
+          } catch {
+            let restored = 0;
+            for (const cookie of sanitized) {
+              try {
+                await cdp.send("Network.setCookies", { cookies: [cookie] });
+                restored++;
+              } catch {
+              }
+            }
+            if (restored > 0) {
+              console.log(`[poncho][browser] Restored ${restored}/${sanitized.length} cookies (batch failed, fell back to individual)`);
+            } else {
+              console.warn("[poncho][browser] Could not restore any cookies");
+            }
+            return;
+          }
+          console.log(`[poncho][browser] Restored ${sanitized.length} cookies`);
+        }
       }
       if (state.origins?.length) {
         const entries = {};
@@ -1033,7 +1075,7 @@ var BrowserSession = class {
 };
 
 // src/tools.ts
-function createBrowserTools(getSession, getConversationId) {
+function createBrowserTools(getSession) {
   return [
     {
       name: "browser_open",
@@ -1048,9 +1090,9 @@ function createBrowserTools(getSession, getConversationId) {
         },
         required: ["url"]
       },
-      handler: async (input) => {
+      handler: async (input, context) => {
         const session = getSession();
-        const cid = getConversationId();
+        const cid = context.conversationId ?? "__default__";
         const url = String(input.url ?? "");
         if (!url) throw new Error("url is required");
         const result = await session.open(cid, url);
@@ -1067,9 +1109,9 @@ function createBrowserTools(getSession, getConversationId) {
         type: "object",
         properties: {}
       },
-      handler: async () => {
+      handler: async (_input, context) => {
         const session = getSession();
-        const snapshot = await session.snapshot(getConversationId());
+        const snapshot = await session.snapshot(context.conversationId ?? "__default__");
         return { snapshot };
       }
     },
@@ -1086,11 +1128,11 @@ function createBrowserTools(getSession, getConversationId) {
         },
         required: ["ref"]
       },
-      handler: async (input) => {
+      handler: async (input, context) => {
         const session = getSession();
         const ref = String(input.ref ?? "");
         if (!ref) throw new Error("ref is required");
-        await session.click(getConversationId(), ref);
+        await session.click(context.conversationId ?? "__default__", ref);
         return { clicked: ref };
       }
     },
@@ -1111,12 +1153,12 @@ function createBrowserTools(getSession, getConversationId) {
         },
         required: ["text"]
       },
-      handler: async (input) => {
+      handler: async (input, context) => {
         const session = getSession();
         const text = String(input.text ?? "");
         if (!text) throw new Error("text is required");
         const exact = input.exact === true;
-        await session.clickText(getConversationId(), text, exact);
+        await session.clickText(context.conversationId ?? "__default__", text, exact);
         return { clicked: text, exact };
       }
     },
@@ -1133,11 +1175,11 @@ function createBrowserTools(getSession, getConversationId) {
         },
         required: ["script"]
       },
-      handler: async (input) => {
+      handler: async (input, context) => {
         const session = getSession();
         const script = String(input.script ?? "");
         if (!script) throw new Error("script is required");
-        const result = await session.executeJs(getConversationId(), script);
+        const result = await session.executeJs(context.conversationId ?? "__default__", script);
         return { result: result ?? null };
       }
     },
@@ -1158,12 +1200,12 @@ function createBrowserTools(getSession, getConversationId) {
         },
         required: ["ref", "text"]
       },
-      handler: async (input) => {
+      handler: async (input, context) => {
         const session = getSession();
         const ref = String(input.ref ?? "");
         const text = String(input.text ?? "");
         if (!ref) throw new Error("ref is required");
-        await session.type(getConversationId(), ref, text);
+        await session.type(context.conversationId ?? "__default__", ref, text);
         return { typed: text, into: ref };
       }
     },
@@ -1174,9 +1216,9 @@ function createBrowserTools(getSession, getConversationId) {
         type: "object",
         properties: {}
       },
-      handler: async () => {
+      handler: async (_input, context) => {
         const session = getSession();
-        const result = await session.content(getConversationId());
+        const result = await session.content(context.conversationId ?? "__default__");
         return { url: result.url, title: result.title, text: result.text };
       }
     },
@@ -1187,9 +1229,9 @@ function createBrowserTools(getSession, getConversationId) {
         type: "object",
         properties: {}
       },
-      handler: async () => {
+      handler: async (_input, context) => {
         const session = getSession();
-        const base64 = await session.screenshot(getConversationId());
+        const base64 = await session.screenshot(context.conversationId ?? "__default__");
         const filePart = {
           type: "file",
           data: base64,
@@ -1217,11 +1259,11 @@ function createBrowserTools(getSession, getConversationId) {
         },
         required: ["direction"]
       },
-      handler: async (input) => {
+      handler: async (input, context) => {
         const session = getSession();
         const direction = String(input.direction ?? "down");
         const amount = typeof input.amount === "number" ? input.amount : void 0;
-        await session.scroll(getConversationId(), direction, amount);
+        await session.scroll(context.conversationId ?? "__default__", direction, amount);
         return { scrolled: direction, amount: amount ?? "viewport" };
       }
     },
@@ -1237,10 +1279,10 @@ function createBrowserTools(getSession, getConversationId) {
           }
         }
       },
-      handler: async (input) => {
+      handler: async (input, context) => {
         const session = getSession();
         const url = input.url ? String(input.url) : void 0;
-        const { cleared } = await session.clearCookies(getConversationId(), url);
+        const { cleared } = await session.clearCookies(context.conversationId ?? "__default__", url);
         return { cleared, scope: url ?? "all" };
       }
     },
@@ -1251,9 +1293,9 @@ function createBrowserTools(getSession, getConversationId) {
         type: "object",
         properties: {}
       },
-      handler: async () => {
+      handler: async (_input, context) => {
         const session = getSession();
-        await session.closeTab(getConversationId());
+        await session.closeTab(context.conversationId ?? "__default__");
         return { closed: true };
       }
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@poncho-ai/browser",
-  "version": "0.6.3",
+  "version": "0.6.5",
   "description": "Browser automation for Poncho agents, powered by agent-browser",
   "repository": {
     "type": "git",
@@ -21,7 +21,7 @@
   },
   "dependencies": {
     "agent-browser": "^0.15.1",
-    "@poncho-ai/sdk": "1.5.0"
+    "@poncho-ai/sdk": "1.6.0"
   },
   "devDependencies": {
     "tsup": "^8.0.0",

package/src/session.ts

@@ -62,6 +62,37 @@ async function getBrowserManagerCtor(): Promise<new () => BrowserManagerInstance
 
 const MAX_TABS = 8;
 
+const VALID_SAME_SITE = ["Strict", "Lax", "None"];
+
+/**
+ * Map a Playwright-format cookie to a CDP-compatible CookieParam, stripping
+ * unknown fields and fixing values that newer Chrome versions reject.
+ */
+function sanitizeCookieForCDP(c: Record<string, unknown>): Record<string, unknown> | null {
+  const name = typeof c.name === "string" ? c.name : "";
+  const value = typeof c.value === "string" ? c.value : "";
+  if (!name) return null;
+
+  const out: Record<string, unknown> = { name, value };
+
+  if (typeof c.domain === "string" && c.domain) out.domain = c.domain;
+  if (typeof c.path === "string") out.path = c.path;
+  if (typeof c.secure === "boolean") out.secure = c.secure;
+  if (typeof c.httpOnly === "boolean") out.httpOnly = c.httpOnly;
+
+  // Playwright uses -1 for session cookies; CDP expects the field to be
+  // absent for session cookies.
+  if (typeof c.expires === "number" && c.expires > 0) {
+    out.expires = c.expires;
+  }
+
+  if (typeof c.sameSite === "string" && VALID_SAME_SITE.includes(c.sameSite)) {
+    out.sameSite = c.sameSite;
+  }
+
+  return out;
+}
+
 /**
  * Init script that forces new-tab navigations (window.open, target="_blank")
  * to open in the current tab. Runs before page scripts on every navigation.
@@ -779,7 +810,12 @@ export class BrowserSession {
       this.tabs.set(conversationId, tab);
     }
     tab.frameListeners.add(listener);
-    return () => { tab!.frameListeners.delete(listener); };
+    return () => {
+      tab!.frameListeners.delete(listener);
+      if (tab!.frameListeners.size === 0 && this._screencastConversation === conversationId) {
+        this.stopScreencast().catch(() => {});
+      }
+    };
   }
 
   onStatus(conversationId: string, listener: StatusListener): () => void {
@@ -896,8 +932,29 @@ export class BrowserSession {
         origins?: Array<{ origin: string; localStorage: Array<{ name: string; value: string }> }>;
       };
       if (state.cookies?.length) {
-        await cdp.send("Network.setCookies", { cookies: state.cookies });
-        console.log(`[poncho][browser] Restored ${state.cookies.length} cookies`);
+        const sanitized = state.cookies
+          .map(sanitizeCookieForCDP)
+          .filter((c): c is Record<string, unknown> => c !== null);
+        if (sanitized.length) {
+          try {
+            await cdp.send("Network.setCookies", { cookies: sanitized });
+          } catch {
+            let restored = 0;
+            for (const cookie of sanitized) {
+              try {
+                await cdp.send("Network.setCookies", { cookies: [cookie] });
+                restored++;
+              } catch { /* skip this cookie */ }
+            }
+            if (restored > 0) {
+              console.log(`[poncho][browser] Restored ${restored}/${sanitized.length} cookies (batch failed, fell back to individual)`);
+            } else {
+              console.warn("[poncho][browser] Could not restore any cookies");
+            }
+            return;
+          }
+          console.log(`[poncho][browser] Restored ${sanitized.length} cookies`);
+        }
       }
       if (state.origins?.length) {
         const entries: Record<string, Array<{ name: string; value: string }>> = {};

package/src/tools.ts

@@ -1,11 +1,10 @@
-import type { ToolDefinition, FileContentPart } from "@poncho-ai/sdk";
+import type { ToolContext, ToolDefinition, FileContentPart } from "@poncho-ai/sdk";
 import type { BrowserSession } from "./session.js";
 
 type BrowserToolInput = Record<string, unknown>;
 
 export function createBrowserTools(
   getSession: () => BrowserSession,
-  getConversationId: () => string,
 ): ToolDefinition[] {
   return [
     {
@@ -22,9 +21,9 @@ export function createBrowserTools(
         },
         required: ["url"],
       },
-      handler: async (input: BrowserToolInput) => {
+      handler: async (input: BrowserToolInput, context: ToolContext) => {
         const session = getSession();
-        const cid = getConversationId();
+        const cid = context.conversationId ?? "__default__";
         const url = String(input.url ?? "");
         if (!url) throw new Error("url is required");
         const result = await session.open(cid, url);
@@ -44,9 +43,9 @@ export function createBrowserTools(
         type: "object",
         properties: {},
       },
-      handler: async () => {
+      handler: async (_input: BrowserToolInput, context: ToolContext) => {
         const session = getSession();
-        const snapshot = await session.snapshot(getConversationId());
+        const snapshot = await session.snapshot(context.conversationId ?? "__default__");
         return { snapshot };
       },
     },
@@ -65,11 +64,11 @@ export function createBrowserTools(
         },
         required: ["ref"],
       },
-      handler: async (input: BrowserToolInput) => {
+      handler: async (input: BrowserToolInput, context: ToolContext) => {
         const session = getSession();
         const ref = String(input.ref ?? "");
         if (!ref) throw new Error("ref is required");
-        await session.click(getConversationId(), ref);
+        await session.click(context.conversationId ?? "__default__", ref);
         return { clicked: ref };
       },
     },
@@ -94,12 +93,12 @@ export function createBrowserTools(
         },
         required: ["text"],
       },
-      handler: async (input: BrowserToolInput) => {
+      handler: async (input: BrowserToolInput, context: ToolContext) => {
         const session = getSession();
         const text = String(input.text ?? "");
         if (!text) throw new Error("text is required");
         const exact = input.exact === true;
-        await session.clickText(getConversationId(), text, exact);
+        await session.clickText(context.conversationId ?? "__default__", text, exact);
         return { clicked: text, exact };
       },
     },
@@ -121,11 +120,11 @@ export function createBrowserTools(
         },
         required: ["script"],
       },
-      handler: async (input: BrowserToolInput) => {
+      handler: async (input: BrowserToolInput, context: ToolContext) => {
         const session = getSession();
         const script = String(input.script ?? "");
         if (!script) throw new Error("script is required");
-        const result = await session.executeJs(getConversationId(), script);
+        const result = await session.executeJs(context.conversationId ?? "__default__", script);
         return { result: result ?? null };
       },
     },
@@ -148,12 +147,12 @@ export function createBrowserTools(
         },
         required: ["ref", "text"],
       },
-      handler: async (input: BrowserToolInput) => {
+      handler: async (input: BrowserToolInput, context: ToolContext) => {
         const session = getSession();
         const ref = String(input.ref ?? "");
         const text = String(input.text ?? "");
         if (!ref) throw new Error("ref is required");
-        await session.type(getConversationId(), ref, text);
+        await session.type(context.conversationId ?? "__default__", ref, text);
         return { typed: text, into: ref };
       },
     },
@@ -167,9 +166,9 @@ export function createBrowserTools(
         type: "object",
         properties: {},
       },
-      handler: async () => {
+      handler: async (_input: BrowserToolInput, context: ToolContext) => {
         const session = getSession();
-        const result = await session.content(getConversationId());
+        const result = await session.content(context.conversationId ?? "__default__");
         return { url: result.url, title: result.title, text: result.text };
       },
     },
@@ -182,9 +181,9 @@ export function createBrowserTools(
         type: "object",
         properties: {},
       },
-      handler: async () => {
+      handler: async (_input: BrowserToolInput, context: ToolContext) => {
         const session = getSession();
-        const base64 = await session.screenshot(getConversationId());
+        const base64 = await session.screenshot(context.conversationId ?? "__default__");
         const filePart: FileContentPart = {
           type: "file",
           data: base64,
@@ -213,11 +212,11 @@ export function createBrowserTools(
         },
         required: ["direction"],
       },
-      handler: async (input: BrowserToolInput) => {
+      handler: async (input: BrowserToolInput, context: ToolContext) => {
         const session = getSession();
         const direction = String(input.direction ?? "down") as "up" | "down";
         const amount = typeof input.amount === "number" ? input.amount : undefined;
-        await session.scroll(getConversationId(), direction, amount);
+        await session.scroll(context.conversationId ?? "__default__", direction, amount);
         return { scrolled: direction, amount: amount ?? "viewport" };
       },
     },
@@ -238,10 +237,10 @@ export function createBrowserTools(
           },
         },
       },
-      handler: async (input: BrowserToolInput) => {
+      handler: async (input: BrowserToolInput, context: ToolContext) => {
         const session = getSession();
         const url = input.url ? String(input.url) : undefined;
-        const { cleared } = await session.clearCookies(getConversationId(), url);
+        const { cleared } = await session.clearCookies(context.conversationId ?? "__default__", url);
         return { cleared, scope: url ?? "all" };
       },
     },
@@ -253,9 +252,9 @@ export function createBrowserTools(
         type: "object",
         properties: {},
       },
-      handler: async () => {
+      handler: async (_input: BrowserToolInput, context: ToolContext) => {
         const session = getSession();
-        await session.closeTab(getConversationId());
+        await session.closeTab(context.conversationId ?? "__default__");
         return { closed: true };
       },
     },