dsh-browser-playwright-codex 0.7.7 → 0.8.0

package/lib/tool.js

@@ -109,6 +109,11 @@ export function apply(ctx, config) {
             throw new BrowserError('DISABLED', 'dsh-browser-playwright-codex is disabled; open Settings → Plugins → 浏览器 and turn it on to use the browser');
         }
         return ctx.browser.acquire(ownerFor(exec), exec.signal).then((session) => {
+            // Let the session's page read honour this call's cancellation signal:
+            // a stuck page no longer hangs the whole tool call (issue #5).
+            const withSet = session;
+            if (typeof withSet.setSignal === 'function')
+                withSet.setSignal(exec.signal);
             publishLogins(session);
             return session;
         });
@@ -133,13 +138,18 @@ export function apply(ctx, config) {
             waitFor: {
                 type: 'string',
                 enum: ['load', 'domcontentloaded', 'networkidle'],
-                description: 'Load condition to wait for. Defaults to load.',
+                description: 'Load condition to wait for. Defaults to domcontentloaded; load never fires on long-polling or ' +
+                    'download pages and hangs the call (issue #5).',
             },
         },
         output: { schema: SNAPSHOT_SCHEMA, render: renderSnapshotValue },
         async execute(args, exec) {
             const session = await acquire(exec);
-            const waitUntil = args.waitFor ?? 'load';
+            // domcontentloaded, not load: a page with an endless subresource (long
+            // poll, streaming download, ad frame that never settles) never reaches
+            // 'load', so the default burned the whole navigation timeout on pages
+            // that were perfectly usable already (issue #5).
+            const waitUntil = args.waitFor ?? 'domcontentloaded';
             return snapshotValue(await session.navigate(args.url, waitUntil, exec.signal));
         },
     }));
@@ -430,13 +440,18 @@ export function apply(ctx, config) {
             waitFor: {
                 type: 'string',
                 enum: ['load', 'domcontentloaded', 'networkidle'],
-                description: 'Load condition to wait for. Defaults to load.',
+                description: 'Load condition to wait for. Defaults to domcontentloaded; load never fires on long-polling or ' +
+                    'download pages and hangs the call (issue #5).',
             },
         },
         output: { schema: SNAPSHOT_SCHEMA, render: renderSnapshotValue },
         async execute(args, exec) {
             const session = await acquire(exec);
-            const waitUntil = args.waitFor ?? 'load';
+            // domcontentloaded, not load: a page with an endless subresource (long
+            // poll, streaming download, ad frame that never settles) never reaches
+            // 'load', so the default burned the whole navigation timeout on pages
+            // that were perfectly usable already (issue #5).
+            const waitUntil = args.waitFor ?? 'domcontentloaded';
             return snapshotValue(await session.openTab(args.url, waitUntil, exec.signal));
         },
     }));

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-browser-playwright-codex",
-  "version": "0.7.7",
+  "version": "0.8.0",
   "description": "Browser automation for DeepSeek Harness, one plugin: fill in forms, scrape lists and work across tabs in real pages, using the logins you already have. Runs in a visible window that stays quiet when minimised; native dialogs are parked instead of confirmed, CAPTCHA pages are flagged with a note that says stop and hand the page back, password / one-time-code / card fields are refused by browser_fill, and the Settings card (enable/disable, page-reading mode, capability gates) now lives inside this package.",
   "license": "MIT",
   "type": "module",
@@ -57,10 +57,12 @@
     "verify": "npm run build && node scripts/dsh-browser-verify.mjs",
     "verify:live": "npm run build && node scripts/dsh-browser-verify.mjs --live",
     "doctor": "node scripts/doctor.mjs",
+    "swarm:patch": "node scripts/swarm-patch.mjs",
+    "swarm:check": "node scripts/swarm-patch.mjs --check",
     "cost": "node node_modules/tsx/dist/cli.mjs scripts/measure-prompt-cost.mjs",
     "format": "node node_modules/prettier/bin/prettier.cjs --write \"src/**/*.ts\" \"tests/**/*.ts\" \"scripts/**/*.mjs\"",
     "format:check": "node node_modules/prettier/bin/prettier.cjs --check \"src/**/*.ts\" \"tests/**/*.ts\" \"scripts/**/*.mjs\"",
-    "prepublishOnly": "npm run build && npm run typecheck && npm test && npm run verify && npm run preflight",
+    "prepublishOnly": "npm run build && npm run typecheck && npm test && npm run swarm:check && npm run verify && npm run preflight",
     "preflight": "node scripts/preflight.mjs ."
   },
   "dependencies": {