@poncho-ai/browser 0.6.3 → 0.6.4

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.4 build /Users/cesar/Dev/latitude/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.65 KB
+ESM ⚡️ Build success in 112ms
 DTS Build start
-DTS ⚡️ Build success in 4573ms
+DTS ⚡️ Build success in 1828ms
 DTS dist/index.d.ts 13.72 KB

package/.turbo/turbo-test.log

@@ -0,0 +1,12 @@
+
+> @poncho-ai/browser@0.6.2 test /Users/cesar/Dev/latitude/poncho-ai/packages/browser
+> vitest --passWithNoTests
+
+
+ RUN  v1.6.1 /Users/cesar/Dev/latitude/poncho-ai/packages/browser
+
+No test files found, exiting with code 0
+include: **/*.{test,spec}.?(c|m)[jt]s?(x)
+
+exclude:  **/node_modules/**, **/dist/**, **/cypress/**, **/.{idea,git,cache,output,temp}/**, **/{karma,rollup,webpack,vite,vitest,jest,ava,babel,nyc,cypress,tsup,build,eslint,prettier}.config.*
+watch exclude:  **/node_modules/**, **/dist/**

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # @poncho-ai/browser
 
+## 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.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 = {};

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@poncho-ai/browser",
-  "version": "0.6.3",
+  "version": "0.6.4",
   "description": "Browser automation for Poncho agents, powered by agent-browser",
   "repository": {
     "type": "git",

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 }>> = {};