@angelky7/dsh-browser-control 1.2.0 → 1.3.0

package/README.en.md

@@ -64,8 +64,8 @@ In one line: **for "use MY browser" tasks (logged-in Bilibili, Zhihu, admin pane
 
 | File | Purpose |
 |---|---|
-| [DSH-Browser-Control-v1.2.0.zip](https://github.com/angelky7/dsh-browser-control/releases/download/v1.2.0/DSH-Browser-Control-v1.2.0.zip) | Chrome extension (unzip and load; this release adds the three cookie commands — reinstall required) |
-| [dsh-browser-control-plugin-v1.2.0.zip](https://github.com/angelky7/dsh-browser-control/releases/download/v1.2.0/dsh-browser-control-plugin-v1.2.0.zip) | dsh plugin (offline fallback; online installs use Option A/B) |
+| [DSH-Browser-Control-v1.3.0.zip](https://github.com/angelky7/dsh-browser-control/releases/download/v1.3.0/DSH-Browser-Control-v1.3.0.zip) | Chrome extension (unzip and load; extension code is unchanged since 1.2.0 — no reinstall needed) |
+| [dsh-browser-control-plugin-v1.3.0.zip](https://github.com/angelky7/dsh-browser-control/releases/download/v1.3.0/dsh-browser-control-plugin-v1.3.0.zip) | dsh plugin (offline fallback; online installs use Option A/B) |
 
 ## Install the Chrome extension (30 seconds)
 
@@ -104,7 +104,7 @@ npm install @angelky7/dsh-browser-control
 
 ```bash
 # Straight from GitHub
-dsh plugin --profile web add "github:angelky7/dsh-browser-control#v1.2.0"
+dsh plugin --profile web add "github:angelky7/dsh-browser-control#v1.3.0"
 
 # Local checkout for debugging (note: the explicit file: prefix is required)
 dsh plugin --profile web add "file:D:\path\to\dsh-browser-control"

package/README.md

@@ -64,8 +64,8 @@ Chrome 扩展（CDP 驱动）
 
 | 文件 | 说明 |
 |---|---|
-| [DSH-Browser-Control-v1.2.0.zip](https://github.com/angelky7/dsh-browser-control/releases/download/v1.2.0/DSH-Browser-Control-v1.2.0.zip) | Chrome 扩展（解压后加载；这一版扩展新增了 cookie 三条命令，**必须重装**） |
-| [dsh-browser-control-plugin-v1.2.0.zip](https://github.com/angelky7/dsh-browser-control/releases/download/v1.2.0/dsh-browser-control-plugin-v1.2.0.zip) | dsh 插件（离线兜底，在线装直接用方式 A/B） |
+| [DSH-Browser-Control-v1.3.0.zip](https://github.com/angelky7/dsh-browser-control/releases/download/v1.3.0/DSH-Browser-Control-v1.3.0.zip) | Chrome 扩展（解压后加载；与 1.2.0 相比扩展代码无改动，已装 1.2.0 不必重装） |
+| [dsh-browser-control-plugin-v1.3.0.zip](https://github.com/angelky7/dsh-browser-control/releases/download/v1.3.0/dsh-browser-control-plugin-v1.3.0.zip) | dsh 插件（离线兜底，在线装直接用方式 A/B） |
 
 ## 安装 Chrome 扩展（30 秒）
 
@@ -104,7 +104,7 @@ npm install @angelky7/dsh-browser-control
 
 ```bash
 # 直接从 GitHub 安装
-dsh plugin --profile web add "github:angelky7/dsh-browser-control#v1.2.0"
+dsh plugin --profile web add "github:angelky7/dsh-browser-control#v1.3.0"
 
 # 本地目录调试（注意：必须显式 file: 前缀）
 dsh plugin --profile web add "file:D:\path\to\dsh-browser-control"

package/lib/index.js

@@ -939,6 +939,15 @@ function policyRefusal(command, flags, params) {
 * fix, so the model can tell the user what to do instead of hanging.
 * @module @deepseek-ai/dsh-browser-bridge
 */
+/** Navigation wait ceiling (same as browser_wait); the bridge gets this plus BRIDGE_MARGIN_MS. */
+const NAV_MAX_MS = 12e4;
+/**
+* Extra time handed to the bridge on top of the navigation budget. Without it a
+* caller-requested budget longer than the bridge default (60s) would be beaten by
+* the bridge, and the model would get `command_timeout` (effect unknown) instead
+* of a navigation timeout (nothing was touched) - two very different next steps.
+*/
+const BRIDGE_MARGIN_MS = 2e3;
 /** Cordis plugin name used by loader diagnostics. */
 const name = "browser-bridge";
 /** The tool registry this plugin contributes `browser_*` tools to. */
@@ -1236,6 +1245,10 @@ function applyBrowserTools(ctx, controller) {
 			tabId: {
 				type: "number",
 				description: "Target tab; defaults to the active tab."
+			},
+			timeoutMs: {
+				type: "number",
+				description: `How long to wait for the page load to settle: defaults to 15000, capped at ${NAV_MAX_MS}. Raise it for slow pages instead of retrying; drop it to 3000-5000 when you do not need the page to finish loading and will read/wait afterwards.`
 			}
 		},
 		output: {
@@ -1287,7 +1300,9 @@ function applyBrowserTools(ctx, controller) {
 		async execute(args, exec) {
 			const params = { url: args.url };
 			if (args.tabId !== void 0) params.tabId = args.tabId;
-			const raw = await controller.execute("nav", params, exec.signal);
+			const budgetMs = typeof args.timeoutMs === "number" && Number.isFinite(args.timeoutMs) ? Math.min(NAV_MAX_MS, Math.max(100, Math.trunc(args.timeoutMs))) : 15e3;
+			params.timeoutMs = budgetMs;
+			const raw = await controller.execute("nav", params, exec.signal, budgetMs + BRIDGE_MARGIN_MS);
 			const out = { tabId: raw.tabId };
 			if (typeof raw.url === "string") out.url = raw.url;
 			if (typeof raw.title === "string") out.title = raw.title;
@@ -2041,6 +2056,8 @@ function applyBrowserTools(ctx, controller) {
 			}
 		}
 	}));
+	/** Page-side ceiling of the eval race in the extension; the bridge budget adds slack on top. */
+	const EVAL_MAX_MS = 12e4;
 	ctx.tools.register(defineTool({
 		name: "browser_evaluate",
 		description: "Run JavaScript in the page and get the JSON result back as a string. Prefer read-only inspection.",
@@ -2053,6 +2070,10 @@ function applyBrowserTools(ctx, controller) {
 			tabId: {
 				type: "number",
 				description: "Target tab; defaults to the active tab."
+			},
+			timeoutMs: {
+				type: "number",
+				description: `Give up after this many milliseconds; defaults to 60000, capped at ${EVAL_MAX_MS}.`
 			}
 		},
 		output: {
@@ -2081,10 +2102,15 @@ function applyBrowserTools(ctx, controller) {
 			kind: "other"
 		}),
 		async execute(args, exec) {
-			const raw = await controller.execute("eval", args.tabId === void 0 ? { expression: args.expression } : {
-				expression: args.expression,
-				tabId: args.tabId
-			}, exec.signal);
+			const params = { expression: args.expression };
+			if (args.tabId !== void 0) params.tabId = args.tabId;
+			let bridgeCeiling;
+			if (typeof args.timeoutMs === "number" && Number.isFinite(args.timeoutMs)) {
+				const evalBudget = Math.min(EVAL_MAX_MS, Math.max(100, Math.trunc(args.timeoutMs)));
+				params.timeoutMs = evalBudget;
+				bridgeCeiling = evalBudget + 2e3;
+			}
+			const raw = await controller.execute("eval", params, exec.signal, bridgeCeiling);
 			let json;
 			try {
 				json = JSON.stringify(raw.value) ?? String(raw.value);

package/lib/types/index.js

@@ -24,6 +24,15 @@ import { defineTool } from '@deepseek-ai/dsh-tools';
 import { BridgeServer, cleanupArtifacts } from "./server.js";
 import { BridgeError, classify } from "./errors.js";
 import { policyRefusal } from "./policy.js";
+/** Navigation wait ceiling (same as browser_wait); the bridge gets this plus BRIDGE_MARGIN_MS. */
+const NAV_MAX_MS = 120_000;
+/**
+ * Extra time handed to the bridge on top of the navigation budget. Without it a
+ * caller-requested budget longer than the bridge default (60s) would be beaten by
+ * the bridge, and the model would get `command_timeout` (effect unknown) instead
+ * of a navigation timeout (nothing was touched) - two very different next steps.
+ */
+const BRIDGE_MARGIN_MS = 2_000;
 // Re-exported so the policy tables stay testable from the published bundle
 // (verify/verify_policy.mjs reconciles them against the extension's COMMANDS).
 export { LOCAL_DESTRUCTIVE_COMMANDS, PAGE_MUTATING_COMMANDS, READ_ONLY_ALLOWED, SCRIPT_COMMANDS, policyRefusal } from "./policy.js";
@@ -358,6 +367,7 @@ function applyBrowserTools(ctx, controller) {
         parameters: {
             url: { type: 'string', required: true, description: 'Absolute URL to open in the tab.' },
             tabId: { type: 'number', description: 'Target tab; defaults to the active tab.' },
+            timeoutMs: { type: 'number', description: `How long to wait for the page load to settle: defaults to 15000, capped at ${NAV_MAX_MS}. Raise it for slow pages instead of retrying; drop it to 3000-5000 when you do not need the page to finish loading and will read/wait afterwards.` },
         },
         output: {
             schema: {
@@ -397,7 +407,11 @@ function applyBrowserTools(ctx, controller) {
             const params = { url: args.url };
             if (args.tabId !== undefined)
                 params.tabId = args.tabId;
-            const raw = await controller.execute('nav', params, exec.signal);
+            const budgetMs = typeof args.timeoutMs === 'number' && Number.isFinite(args.timeoutMs)
+                ? Math.min(NAV_MAX_MS, Math.max(100, Math.trunc(args.timeoutMs)))
+                : 15_000;
+            params.timeoutMs = budgetMs;
+            const raw = await controller.execute('nav', params, exec.signal, budgetMs + BRIDGE_MARGIN_MS);
             // The declared schema is closed, so the fields describing a navigation that did
             // not arrive have to be carried over on purpose. The old shape check
             // (`siteUnreachable === true`) also missed the object the extension really
@@ -866,12 +880,15 @@ function applyBrowserTools(ctx, controller) {
             }
         },
     }));
+    /** Page-side ceiling of the eval race in the extension; the bridge budget adds slack on top. */
+    const EVAL_MAX_MS = 120_000;
     ctx.tools.register(defineTool({
         name: 'browser_evaluate',
         description: 'Run JavaScript in the page and get the JSON result back as a string. Prefer read-only inspection.',
         parameters: {
             expression: { type: 'string', required: true, description: 'JavaScript expression or statement sequence; awaited like a promise body.' },
             tabId: { type: 'number', description: 'Target tab; defaults to the active tab.' },
+            timeoutMs: { type: 'number', description: `Give up after this many milliseconds; defaults to 60000, capped at ${EVAL_MAX_MS}.` },
         },
         output: {
             schema: {
@@ -886,7 +903,19 @@ function applyBrowserTools(ctx, controller) {
         },
         presentCall: () => ({ card: 'generic', title: 'Evaluate in page', kind: 'other' }),
         async execute(args, exec) {
-            const raw = await controller.execute('eval', args.tabId === undefined ? { expression: args.expression } : { expression: args.expression, tabId: args.tabId }, exec.signal);
+            const params = { expression: args.expression };
+            if (args.tabId !== undefined)
+                params.tabId = args.tabId;
+            // The extension races the evaluation at this budget; give the bridge 2s of
+            // slack so the page-side timeout reports first. Omitted → send nothing and
+            // let the bridge default (60s) govern, exactly as before the knob existed.
+            let bridgeCeiling;
+            if (typeof args.timeoutMs === 'number' && Number.isFinite(args.timeoutMs)) {
+                const evalBudget = Math.min(EVAL_MAX_MS, Math.max(100, Math.trunc(args.timeoutMs)));
+                params.timeoutMs = evalBudget;
+                bridgeCeiling = evalBudget + 2_000;
+            }
+            const raw = await controller.execute('eval', params, exec.signal, bridgeCeiling);
             let json;
             try {
                 json = JSON.stringify(raw.value) ?? String(raw.value);

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@angelky7/dsh-browser-control",
   "description": "Local WebSocket bridge between dsh and the DSH Browser Control browser extension, with model-facing browser_* tools and a Settings-managed on/off switch",
-  "version": "1.2.0",
+  "version": "1.3.0",
   "type": "module",
   "main": "lib/index.js",
   "types": "lib/types/index.d.ts",
@@ -37,7 +37,7 @@
     "build": "tsc -p tsconfig.json && tsdown",
     "typecheck": "tsc -p tsconfig.json --noEmit",
     "check": "npm run typecheck && npm run build && publint",
-    "verify": "node verify/verify_config_patch.mjs && node verify/verify_peers.mjs && node verify/verify_surface.mjs && node verify/verify_tab_groups.mjs && node verify/verify_tabs_close_many.mjs && node verify/verify_cdp_retry.mjs && node verify/verify_snapshot_ax.mjs && node verify/verify_hard_lock.mjs && node verify/verify_policy.mjs && node verify/verify_cookies.mjs && node verify/verify_extension_e2e.mjs && node verify/verify_plugin.mjs legacy && node verify/verify_plugin.mjs conflict && node verify/verify_plugin.mjs new && node verify/verify_http.mjs && node verify/verify_errors.mjs && node verify/verify_hold.mjs && node verify/verify_protocol.mjs && node verify/verify_bundle_http.mjs && node verify/verify_nav_report.mjs && node verify/verify_nav_landing.mjs && node verify/verify_render_contract.mjs && node verify/verify_press_events.mjs"
+    "verify": "node verify/verify_config_patch.mjs && node verify/verify_peers.mjs && node verify/verify_surface.mjs && node verify/verify_tab_groups.mjs && node verify/verify_tabs_close_many.mjs && node verify/verify_cdp_retry.mjs && node verify/verify_snapshot_ax.mjs && node verify/verify_hard_lock.mjs && node verify/verify_policy.mjs && node verify/verify_cookies.mjs && node verify/verify_extension_e2e.mjs && node verify/verify_plugin.mjs legacy && node verify/verify_plugin.mjs conflict && node verify/verify_plugin.mjs new && node verify/verify_http.mjs && node verify/verify_errors.mjs && node verify/verify_hold.mjs && node verify/verify_protocol.mjs && node verify/verify_bundle_http.mjs && node verify/verify_nav_report.mjs && node verify/verify_nav_landing.mjs && node verify/verify_render_contract.mjs && node verify/verify_press_events.mjs && node verify/verify_nav_timeout.mjs"
   },
   "publishConfig": {
     "access": "public",