dsh-browser-playwright-codex 0.6.6 → 0.7.1

package/CHANGELOG.md

@@ -3,6 +3,45 @@
 Bilingual by intent: the version headings and the summary of each release are given in both
 languages, the individual entries follow the language the surrounding code and docs use.
 
+## 0.7.1 — 2026-10-19
+
+一句话摘要：**只修文档与一处口径** —— 外部评分（8.9/10）点名的问题：CHANGELOG 的数字手写写错、对比表把五个闸门写成三个、有头模式的 viewport 仍然不生效却没标注。行为零变化。
+
+### Added
+
+- **设置卡新增第三个工具开关：诊断工具**（`browser_console_messages` / `browser_network_requests`）。与 `evaluate` / `cdp` 同一套机制：`POST /api/browser-playwright/gates` 现在接受 `diagnostics`，home 补丁块的整行 config 一并写入，`tool.ts` 把**实际生效值**（而不是猜的）发布给卡片；卡片上显示"当前：N 个开着"也把这一项算进去。
+- **「已登录的站点」默认折叠**：标题行直接给出「N 个域 · M 个 cookie · 点击展开」，点开才渲染说明与名单（分组仍各自折叠）。设置页不再被这份名单占掉大半屏。
+
+### Fixed
+
+- **CHANGELOG 0.7.0 的数字按实测改正**：常驻是 **16 工具 / 11,179 字符**（此前手写成 11,005 字符，token 数也跟着漂），对比基线 18 工具 / 12,755 字符保持不变，字符降幅 **−1,576（−12.4%）**。这是连续第三个版本出现"主文档被守卫钉住、CHANGELOG 叙事手写漂移"——本轮起新增加一条守卫：**最新一节 CHANGELOG 不许出现手写的 `≈ N token/轮` 数字**，要写就写实测命令与结果表。
+- **对比与优势.md 的闸门清单**："五个闸门"此前只列了 `evaluate` / `cdp` / `extract` 三个名字，补上 `console_messages` / `network_requests`，并把它与全量注册的字符差从旧的 2,853 更正为实测 4,603 字符（token 换算见 README 成本表）。
+
+### Changed
+
+- **有头模式的 `viewport` 标注**：`browser-lifecycle.ts:44` 用 `viewport: null` 跟随真实窗口，所以 `launch.viewport` 只在 `headless: true` 时生效；README 配置示例与 Known Limitations 现在明写这一点（此前"配了也不生效"没有任何提示，被外部评分连续四次点出）。
+
+## 0.7.0 — 2026-10-19
+
+一句话摘要：**两个插件合成一个** —— 设置卡片并入本包；加载行从三行收成一行（这是 `dsh-client-modules` 允许本包带 client 模块的前提），独立的 `dsh-browser-toggle` 退役。
+
+### Breaking
+
+- **一个包一行 loader**：`cordis.patch.yml` 现在只插 `- id: browser` / `name: 'dsh-browser-playwright-codex'`，三块配置收进嵌套的 `service` / `playwright` / `tool`。旧的 `browser-playwright`、`browser-tool` 行不再被读取；loader 整体替换 config，迁移时三节都要写全（设置卡会自动重写 home 补丁层里的那一块）。
+- 版本跳到 0.7.0，因为配置形状变了。
+
+### Added
+
+- **设置卡随本包发布**：`dsh.client` + `./client` 导出，卡片仍在 `/api/browser-playwright/*` 上提供总开关、页面识别方式、两个强力工具闸门与已登录站点名单。
+- **单 loader 行守卫**（tests/plugin-entry.test.ts）：`cordis.patch.yml` 不得再出现本包的子路径行，否则卡片会重新变成非法。
+
+### Changed
+
+- **两个诊断工具改为门控**：`browser_console_messages` / `browser_network_requests` 不再常驻 —— 新增 `tool.diagnostics: false`（默认关），开启后才注册。常驻成本 **18 工具 12,755 字符 ≈ 3,543 token/轮 → 16 工具 11,179 字符 ≈ 3,105 token/轮（−438，−12.4%）**；全量注册仍是 21 工具 15,782 字符 ≈ 4,384。
+- **快照瘦身（运行时大头）**：无名无状态的 `generic` 包装层不再逐层输出（真实页面常把一个链接包在 7~8 层 div 里），显示用 `href` 去掉追踪参数（`utm_*` / `gclid` / `rsv_*` / `sa` 等）并截到 160 字符。百度结果页形状的样本：**6,169 → 2,557 字符（−58.6%）**，行数 108 → 24。refs 与点击行为不变 —— 省的是模型读到的东西。
+- 闸门写入块改为**整行 config**（新标记 `# >>> dsh-browser:`），并兼容旧的 `dsh-browser-toggle` 标记就地替换 —— 避免同一个 id 被两段 patch 争抢。
+- `scripts/shipped-gates.mjs` 按新行名读取出厂闸门；`scripts/build-client.mjs` 把 `client/client.js` 复制进 `lib/`。
+
 ## 0.6.6 — 2026-10-17
 
 一句话摘要：**四份独立审查（文档漂移 / 代码对抗 / 发行打包 / 中性冷读）查出的问题集中修掉**——两条安全缺陷、几处"假成功/假失败"、随包自检的假承诺、发布门禁漏洞与一批文档漂移。

package/README.md

@@ -2,12 +2,14 @@
 
 > [![dsh.so 安装验证 · harness 0.2.0-rc.1](https://www.dsh.so/badge/install/dsh-browser-playwright-codex@0.2.0-rc.1.svg)](https://www.dsh.so/zh/artifact/dsh-browser-playwright-codex/) [![dsh.so 风险](https://www.dsh.so/badge/dsh-browser-playwright-codex.svg)](https://www.dsh.so/zh/artifact/dsh-browser-playwright-codex/)
 
-**一句话**：让 AI 用你**已经登录的账号**在真实网页上干活 —— 填表、抓取列表、多标签查后台；浏览器窗口就在你桌面上、你随时看得见。原生弹框会被挂起、**绝不替你点「确定」**；验证码/登录页会被识别出来并把情况报给你、请你本人接手（识别不出来时也会停下来问人，而不是硬闯）。
+**一句话**：让 AI 用你**已经登录的账号**在真实网页上干活 —— 填表、抓取列表、多标签查后台；浏览器窗口就在你桌面上、你随时看得见。原生弹框会被挂起、**绝不替你点「确定」**（机制级）；验证码/挑战页会被识别出来，并以提示要求模型停下交给你（**提示不是阻断**）。登录/二次验证**没有探测器**，靠模型从快照看出来并停下；密码、一次性验证码、卡号字段 `browser_fill` **直接拒绝**——这类字由你本人在窗口里输。
 
 **In one line**: the agent drives a **visible, profile-backed browser window** with the logins you
 already have — filling in forms, scraping lists and working across tabs in real pages — through
 accessibility snapshots with stable element refs. Native dialogs are parked and reported instead of
-auto-accepted; a login, 2FA or challenge page is handed back to you rather than guessed at. No
+auto-accepted. Challenge pages are detected and reported as a note that tells the agent to stop and hand
+the page back — a note, not a block. There is **no sign-in detector**, and `browser_fill` refuses
+password / one-time-code / card fields outright — you type those in the visible window. No
 CSS-selector guessing, no full-DOM dumps.
 
 ```sh
@@ -42,8 +44,8 @@ dsh plugin --profile <name> add github:miqian-nomad/dsh-browser-playwright-codex
 implementation of `browser_evaluate` (its dynamic evaluation), which this package ships switched off.
 [SECURITY.md](https://github.com/miqian-nomad/dsh-browser-playwright-codex/blob/main/SECURITY.md) says what each
 finding actually is, whether a capability gate makes it reachable (two of the warnings are not gate-controlled:
-the login-state write and the local test server in the acceptance script), and explains what the plugin can and
-cannot do with your logins (URL policy, CDP allow-list, dialog policy, no telemetry, and it never types
+cannot do with your logins (URL policy, CDP allow-list, dialog policy, no telemetry, and `browser_fill`
+refuses password / one-time-code / card fields). Counts move as the scanner and the code change — their own report is the
 passwords or 2FA codes). Counts move as the scanner and the code change — their own report is the
 live number.
 
@@ -88,8 +90,9 @@ Install it and ask for something that leaves a visible result — there is nothi
 > Go to my orders page on `<site>` and tell me the delivery address on the most recent order.
 
 > Fill in this contact form, but stop and show me the filled form before you submit it.
-
-What you should see: a real browser window on your desktop doing the clicking, and the page coming back to
+the model as a `ref`-numbered list. The first time it meets a verification page it **reports the page back
+to you** — take the window over by hand and it continues from where it left off (sign-ins have no detector:
+the agent is told to stop when it sees one).
 the model as a `ref`-numbered list. The first time it meets a sign-in, a 2FA prompt or a verification page
 it **reports the page back to you** — take the window over by hand and it continues from where it left off.
 Native dialogs are the one case that is enforced at the interface: they are parked and never auto-accepted.
@@ -116,7 +119,7 @@ Or pin a binary: `launch.executablePath` / `launch.channel` in the plugin config
 
 ## Tools
 
-The default `toolPrefix` is `browser_`. Every action returns a fresh snapshot, so refs always come from the latest result. The registered set is capability-gated: `browser_evaluate`, `browser_cdp` and `browser_extract` appear only when their capability is switched on (their schemas are resident in the system prompt on every turn, so a tool that could only answer "disabled" would be pure prompt cost). Count the live surface with `npm run cost` — the shipped `cordis.patch.yml` keeps **both power gates off** (since 0.4.7), so a profile mounting this bundle sees **18 tools**; 19 or 20 once your deployment opts into `evaluate` / `cdp`, 21 once `extract` is configured. How to opt in: paste the snippet under Known Limitations below, or use the settings card in `dsh-browser-toggle`.
+The default `toolPrefix` is `browser_`. Every action returns a fresh snapshot, so refs always come from the latest result. The registered set is capability-gated: `browser_evaluate`, `browser_cdp` and `browser_extract` appear only when their capability is switched on (their schemas are resident in the system prompt on every turn, so a tool that could only answer "disabled" would be pure prompt cost). Count the live surface with `npm run cost` — the shipped `cordis.patch.yml` keeps **both power gates off** (since 0.4.7), so a profile mounting this bundle sees **16 tools**; 17 or 18 once your deployment opts into `evaluate` / `cdp`, 19 with `diagnostics: true`, 21 once `extract` is configured. How to opt in: paste the snippet under Known Limitations below, or use the built-in Settings card.
 
 | Tool | Purpose |
 |---|---|
@@ -135,8 +138,8 @@ The default `toolPrefix` is `browser_`. Every action returns a fresh snapshot, s
 | `browser_dialog` | Answer the native dialog blocking the page: `{ accept: true }` confirms, `false` cancels; errors with `NO_DIALOG` when nothing is pending |
 | `browser_cdp` | Send one raw CDP command (DOM inspection / Input simulation allow-list only; disabled by default); **only registered when its capability is enabled** |
 | `browser_close` | Close the session's browser window; the profile keeps login state, so the next call reopens signed in |
-| `browser_console_messages` | Read the page console messages captured for the current page (levels, text, source) |
-| `browser_network_requests` | Read the network requests captured for the current page (method, URL, status, resource type) |
+| `browser_console_messages` | Read the page console messages captured for the current page (levels, text, source); **only registered when `diagnostics: true`** |
+| `browser_network_requests` | Read the network requests captured for the current page (method, URL, status, resource type); **only registered when `diagnostics: true`** |
 
 ### Parameter signatures
 
@@ -211,49 +214,51 @@ An actionable element with no text of its own is named from its descendants —
 All tunables are patchable through the profile's `cordis.patch.yml` (later layers win per row).
 
 ```yaml
+# ONE row for the whole capability (2026-10-19). A package with a client module (the
+# Settings card lives here) may only be mounted once, so the three concerns are nested.
 - id: browser
+  name: 'dsh-browser-playwright-codex'
   config:
-    provider: playwright                # explicit provider selection
-
-- id: browser-playwright
-  config:
-    launch:
-      channel: chrome                   # or chromium / msedge / edge
-      headless: false                   # visible window you can watch and rescue
-      persistent: true                  # profile-backed window, login survives close
-      # profileDir: ~/.dsh/browser-profiles/playwright by default
-      viewport: { width: 1280, height: 800 }
-      navigationTimeoutMs: 30000
-      ignoreHTTPSErrors: false
-    allowedDomains: []                  # e.g. ['github.com'] restricts navigation
-    idleTimeoutMs: 0                    # shipped value: never auto-close (schema default 600000 = 10 min)
-    maxSessions: 8                      # LRU eviction (persistent mode: one window)
-    snapshot:
-      engine: legacy                    # 'legacy' (default) or 'aria' (Playwright ariaSnapshot, refs aligned back)
-      maxNodes: 500                     # token budget per snapshot tree
-      maxNameLength: 120
-      maxTextLength: 300
-      diff: false                       # true = return "+/-/~" ref diffs instead of the full tree
-
-- id: browser-tool
-  config:
-    toolPrefix: browser_                # model-facing tool name prefix
-    allowEvaluate: false                # browser_evaluate gate
-    allowCdp: false                     # browser_cdp gate (DOM inspection / Input simulation allow-list)
-    registerDisabledTools: false        # register evaluate/cdp/extract even while off (discoverability vs resident prompt cost)
-    interactiveOnlyDefault: false       # true = browser_snapshot returns actionable elements only
-    maxWaitMs: 60000                    # browser_wait bound
-    extract:
-      provider: deepseek-official       # optional: enables browser_extract
-      model: deepseek-v4-flash
-      maxInputChars: 20000             # caps the page text handed to that model
-      maxOutputTokens: 2000
+    service:
+      provider: playwright                # explicit provider selection
+    playwright:
+      launch:
+        channel: chrome                   # or chromium / msedge / edge
+        headless: false                   # visible window you can watch and rescue
+        persistent: true                  # profile-backed window, login survives close
+        # profileDir: ~/.dsh/browser-profiles/playwright by default
+        viewport: { width: 1280, height: 800 }   # headful: IGNORED (the window tracks the desktop; only headless: true pins it)
+        navigationTimeoutMs: 30000
+        ignoreHTTPSErrors: false
+      allowedDomains: []                  # e.g. ['github.com'] restricts navigation
+      idleTimeoutMs: 0                    # shipped value: never auto-close (schema default 600000 = 10 min)
+      maxSessions: 8                      # LRU eviction (persistent mode: one window)
+      snapshot:
+        engine: legacy                    # 'legacy' (default) or 'aria' (Playwright ariaSnapshot, refs aligned back)
+        maxNodes: 500                     # token budget per snapshot tree
+        maxNameLength: 120
+        maxTextLength: 300
+        diff: false                       # true = return "+/-/~" ref diffs instead of the full tree
+    tool:
+      toolPrefix: browser_                # model-facing tool name prefix
+      allowEvaluate: false                # browser_evaluate gate
+      allowCdp: false                     # browser_cdp gate (DOM inspection / Input simulation allow-list)
+      registerDisabledTools: false        # register gated tools even while off (discoverability vs resident prompt cost)
+      diagnostics: false                  # true = register browser_console_messages / browser_network_requests
+      interactiveOnlyDefault: false       # true = browser_snapshot returns actionable elements only
+      maxWaitMs: 60000                    # browser_wait bound
+      extract:
+        provider: deepseek-official       # optional: enables browser_extract
+        model: deepseek-v4-flash
+        maxInputChars: 20000              # caps the page text handed to that model
+        maxOutputTokens: 2000
+```
 ```
 
 `snapshot.engine` picks the page-reading engine: `'legacy'` (the default) walks the injected DOM,
 `'aria'` uses Playwright's official `ariaSnapshot({ mode: 'ai' })` and aligns the result back onto the
 same `ref` ids; an aria tree that cannot be parsed falls back to legacy rather than handing the model an
-empty tree. It is a `snapshot` key, next to `maxNodes` / `maxNameLength` / `maxTextLength` / `diff`.
+empty tree. It lives under `playwright.snapshot`, next to `maxNodes` / `maxNameLength` / `maxTextLength` / `diff`.
 
 `browser_extract` needs an auxiliary LLM route (`extract.provider` + `extract.model`); without one it fails with an actionable error. With a route configured, `extract.maxInputChars` bounds the page text in that prompt, and the prompt names the cap when it truncates. A gated tool is **not registered at all** while its capability is off, so its schema never reaches the system prompt; set `registerDisabledTools: true` to keep the whole surface discoverable (the tool then answers with the error naming the switch). The shipped layer in [`cordis.patch.yml`](https://github.com/miqian-nomad/dsh-browser-playwright-codex/blob/main/cordis.patch.yml) keeps `allowEvaluate` and `allowCdp` **off**, matching the schema defaults and this sample: that file travels inside the package, so anything it enabled would be enabled for everyone who installs it. Opt in per deployment, in that deployment's own patch layer. `browser_evaluate` executes arbitrary page JavaScript; `browser_cdp` stays off until `allowCdp: true`, and even then only the allow-list in `lib/cdp-policy.js` passes — network, storage, cookie, security and arbitrary-JS (`Runtime.evaluate`) commands are rejected with `CDP_DENIED`.
 
@@ -278,8 +283,8 @@ turning them on in a deployment that shares prompts across many users.
 
 ### Prefer a switch to editing config?
 
-The companion package [`dsh-browser-toggle`](https://github.com/miqian-nomad/dsh-browser-toggle) puts
-this on a Settings card: enable/disable the browser, pick the page-reading mode, and turn the two
+The Settings card ships **inside this package** (it moved out of the companion package `dsh-browser-toggle`, which is retired) and puts
+this on one card: enable/disable the browser, pick the page-reading mode, and turn the two
 power tools on or off — no restart, no YAML. The card also lists **which sites this browser looks
 signed in to**. Two things make that list trustworthy: cookies are attributed **first-party** — only
 domains this browser actually visited are candidates, because a tracker's cookie belongs to the site
@@ -363,9 +368,9 @@ Measured with ``npm run cost`` (per-turn, resident in the system prompt):
 
 | Surface | Tools | Schema characters | ≈ tokens/turn |
 |---|---|---|---|
-| Schema defaults (`Config({})`, no patch layer) | 18 | 12,755 | ≈ 3,543 |
-| Shipped bundle layer (this package's `cordis.patch.yml`: both power gates off) | 18 | 12,755 | ≈ 3,543 |
-| Everything registered (`registerDisabledTools: true`, plus `extract`) | 21 | 15,608 | ≈ 4,336 |
+| Schema defaults (`Config({})`, no patch layer) | 16 | 11,179 | ≈ 3,105 |
+| Shipped bundle layer (this package's `cordis.patch.yml`: both power gates off) | 16 | 11,179 | ≈ 3,105 |
+| Everything registered (`registerDisabledTools: true`, plus `extract`) | 21 | 15,782 | ≈ 4,384 |
 
 This paragraph describes the 0.6.0 change; the table above is today's measurement. Three things changed
 there, and only one of them was about size. (1) The click family was rewritten so each rule is stated once
@@ -415,14 +420,18 @@ links below go to GitHub rather than to files inside the package:
 - **The package ships both power gates OFF, and that is deliberate** — `cordis.patch.yml` is part of the package, so anything it enables is enabled for *everyone who installs this*. It therefore sets `allowEvaluate: false` and `allowCdp: false` (the code defaults), and each deployment opts in through its **own profile patch** (`~/.dsh/profiles/<name>/cordis.patch.yml`). A package that shipped them on would be handing every installer a backdoor they never asked for. Snippet:
   ```yaml
   # dsh replaces an entry's config object wholesale — list every key you rely on.
-  - id: browser-tool
+  - id: browser
     config:
-      toolPrefix: 'browser_'
-      allowEvaluate: true    # registers browser_evaluate
-      allowCdp: true         # registers browser_cdp (needs a full DSH restart)
-      registerDisabledTools: false
-      maxWaitMs: 60000
+      service: { provider: playwright }
+      playwright: { launch: { headless: false, persistent: true }, idleTimeoutMs: 0, maxSessions: 8, snapshot: { diff: false } }
+      tool:
+        toolPrefix: 'browser_'
+        allowEvaluate: true    # registers browser_evaluate
+        allowCdp: true         # registers browser_cdp (needs a full DSH restart)
+        registerDisabledTools: false
+        maxWaitMs: 60000
   ```
+- **`launch.viewport` only applies headless** — the visible window is a real window: it tracks the desktop size (`browser-lifecycle.ts` passes `viewport: null` in headful), so setting 1280×800 there changes nothing. Pin the size by running `headless: true`, or resize the window itself.
 - **Chromium family only** — Firefox/WebKit channels are not probed; providers are swappable if another engine is needed.
 - **Extract needs a dedicated model route** — it does not reuse the main request's route; misconfiguration fails loudly at call time.
 - **JS-driven navigations are not statically checkable** — `allowedDomains` covers `browser_navigate`, `browser_open_tab`, every navigating click (`browser_click`: `ref`, `text`, `precise` and raw-coordinate modes) and `browser_switch_tab`; `mode: "hover"` reveals content without activating anything, so it is not checked against the URL policy. A button whose handler runs `location.href = …` can still leave the allowed hosts, and so can a `<form>` whose `action` points at another host (pressing its submit button is a navigation this check does not statically see), and a tab that is already sitting on a disallowed host stays open (the agent simply refuses to drive it); deploy an external network guard for hard isolation.
@@ -449,9 +458,13 @@ MIT
 
 （此列表只记 **minor 里程碑**；每个补丁版本见 [CHANGELOG.md](https://github.com/miqian-nomad/dsh-browser-playwright-codex/blob/main/CHANGELOG.md)。）
 
+### 0.7.0（2026-10-19）
+
+- **两个插件合成一个（破坏性）**：设置卡片并入本包（`dsh.client` + `./client`），`dsh-browser-toggle` 退役；`cordis.patch.yml` 从三行 loader 收成**一行** —— `- id: browser` 加嵌套的 `service` / `playwright` / `tool`。这是硬约束：`dsh-client-modules` 拒绝一个包被多个 loader 行挂载，所以带 client 的包只能有一个入口。
+
 ### 0.6.0（2026-10-16）
 
-- **工具面三合一（破坏性）**：`browser_click` / `browser_hover` / `browser_click_at` 合并成一个 `browser_click` —— `mode: "hover"` 是原来的悬停、`precise: true` 是原来的几何中点兜底、`x`+`y` 是裸坐标。常驻 prompt **净减 16.5%（约六分之一）**：现在是 **18 个工具 12,755 字符 ≈ 3,543 token/轮**，全量注册（含三个闸门工具）21 个；规则一条没丢，守卫改成"合并后的工具必须保住原来三个的全部规则"。
+- **工具面三合一（破坏性）**：`browser_click` / `browser_hover` / `browser_click_at` 合并成一个 `browser_click` —— `mode: "hover"` 是原来的悬停、`precise: true` 是原来的几何中点兜底、`x`+`y` 是裸坐标。常驻 prompt **净减 16.5%（约六分之一）**（当时的值；当前成本见上面的成本表），全量注册（含三个闸门工具）21 个；规则一条没丢，守卫改成"合并后的工具必须保住原来三个的全部规则"。
 - **`browser_fill` 接受 `value` 作 `text` 的别名**：模型按语义猜参数名不再第一次就撞墙；缺值时给的是指路错误（同时点出两种写法，并说明要点可见标签该用 `browser_click`）。
 - **设置页多了一节「已登录的站点」（只读）**：按域名列出"像登录态"的 cookie —— 只列你在浏览器里**真的访问过**的域（第三方统计/广告域排除），判据用 OWASP 的会话特征（httpOnly 且会话级/值像令牌），**名字与条数不再作为依据**。不显示 cookie 的名字和值、不写磁盘、**模型看不到**；同一家公司的域折叠成一行（如 `京东系（18 个域）`），每行写明"这个域名是什么"（离线对照表 → 网页自己的标题 → 域名，认不出不编）。
 - **`GET /state` 补上防伪头**：四个路由里唯一一个没校验 `x-dsh-plugin` 的；新增的 `GET /logins` 同样强制该头（这份名单是用户的登录站点）。

package/SECURITY.md

@@ -15,9 +15,12 @@ this package ships switched off** — it is not registered at all until a deploy
 
 **It drives your browser, as you.** The provider opens a visible, profile-backed Chromium-family
 window (`~/.dsh/browser-profiles/playwright` by default) and reuses the logins already in it. It does
-not have your passwords, and it never types them: when a login, a 2FA prompt or a captcha appears,
-the agent is supposed to stop and hand the page back to you.
-
+window (`~/.dsh/browser-profiles/playwright` by default) and reuses the logins already in it. It does
+not have your passwords, and `browser_fill` **refuses** credential fields (`type=password`,
+`autocomplete=current-password` / `new-password` / `one-time-code` / `cc-*`) — typing one is a
+mechanism-level no, not a promise. Challenge pages are detected and reported as a note; there is
+**no sign-in detector**, so a sign-in page is handled by the agent's discipline and you complete it in
+the visible window.
 **It is visible and interruptible.** The window lives on your desktop. You can watch every action,
 minimise it (the provider detects a minimised window and keeps working without raising it — see
 [FOCUS-STEALING.md](https://github.com/miqian-nomad/dsh-browser-playwright-codex/blob/main/FOCUS-STEALING.md)), or close it: login state is exported first and restored on

package/cordis.patch.yml

@@ -1,72 +1,80 @@
-# dsh-browser-playwright-codex bundle layer: the browser seam, the Playwright
-# provider, and the model-facing tool family. Applied as one insert over the
-# profile's entry list; rows are patchable by id like every other plugin.
+# dsh-browser-playwright-codex bundle layer — ONE row for the whole capability.
 #
-# The `name:` values are package specifiers resolved from the profile, so they
-# must track package.json#name — a rename that misses this file fails at load
-# with MODULE_NOT_FOUND (2026-09-30: the 0.4.0 rename to *-codex did exactly
-# that; verified `require.resolve('dsh-browser-playwright/service')` fails in
-# both profiles while the *-codex specifier resolves).
+# History worth keeping: this file used to insert three rows (`browser` service,
+# `browser-playwright` provider, `browser-tool` tools) because the capability has
+# three configurable concerns. `dsh-client-modules` rejects distinct active Loader
+# sources that resolve to one package name, so a Settings card (which needs a
+# client module) could not live in such a package — it shipped as a second plugin,
+# dsh-browser-toggle. 2026-10-19: one row, three nested config sections, one plugin.
+#
+# The `name:` value is a package specifier resolved from the profile, so it must
+# track package.json#name — a rename that misses this file fails at load with
+# MODULE_NOT_FOUND (the 0.4.0 rename to *-codex did exactly that).
 - insert:
     - id: browser
-      name: 'dsh-browser-playwright-codex/service'
-      config:
-        provider: playwright
-    - id: browser-playwright
-      name: 'dsh-browser-playwright-codex/playwright'
-      config:
-        launch:
-          # Visible profile-backed window (persistent mode): the agent drives a
-          # browser the user can watch and rescue. Set true for headless
-          # servers. profileDir defaults to ~/.dsh/browser-profiles/playwright.
-          headless: false
-          persistent: true
-          viewport:
-            width: 1280
-            height: 800
-          navigationTimeoutMs: 30000
-          ignoreHTTPSErrors: false
-        # Idle auto-close: ms of no agent activity before the window closes
-        # itself (login state is saved first, so it reopens logged-in). 0 =
-        # never auto-close; the window stays until the user closes it or DSH
-        # exits. User choice: never auto-close (2026-09-04).
-        idleTimeoutMs: 0
-        maxSessions: 8
-        # Response-size lever (off by default): return "+/-/~" ref diffs between
-        # snapshots instead of the whole tree. Smaller returns on pages that
-        # re-render often; the model then has to hold the previous tree itself.
-        snapshot:
-          diff: false
-    - id: browser-tool
-      name: 'dsh-browser-playwright-codex/tool'
+      name: 'dsh-browser-playwright-codex'
       config:
-        # This file ships INSIDE the package, so it must not turn anything on for
-        # the people who install it. It used to: an earlier revision carried the
-        # author's own `allowEvaluate: true` / `allowCdp: true` (2026-09-04) into
-        # every download, which is how a package ends up handing each installer a
-        # backdoor they never asked for — and it is what a third-party scanner
-        # reads as "dynamic code execution, critical".
-        #
-        # Both are OFF here (matching the code defaults in src/tool.ts, and the
-        # README's own configuration example). Opt in per deployment, in YOUR
-        # profile patch — never in the package:
-        #
-        #   # ~/.dsh/profiles/<name>/cordis.patch.yml
-        #   # NOTE: dsh replaces an entry's config object wholesale, so list every
-        #   # key you rely on, not just the one you are changing.
-        #   - id: browser-tool
-        #     config:
-        #       toolPrefix: 'browser_'
-        #       allowEvaluate: true    # registers browser_evaluate
-        #       allowCdp: true         # registers browser_cdp (needs a DSH restart)
-        #       registerDisabledTools: false
-        #       interactiveOnlyDefault: false   # true = snapshot returns actionable elements only
-        #       maxWaitMs: 60000
-        #
-        # browser_evaluate runs arbitrary page JavaScript: it can read anything in
-        # the page and act on the user's logged-in sites. browser_cdp is CDP with
-        # an allow-list — only inspection and input simulation pass
-        # (lib/cdp-policy.js); network/storage/cookie/security commands never do.
-        # Neither tool is registered at all while its gate is false.
-        allowEvaluate: false
-        allowCdp: false
+        # Browser seam: which registered provider to drive.
+        service:
+          provider: playwright
+        # Provider: window, launch, snapshot and session settings.
+        playwright:
+          launch:
+            # Visible profile-backed window (persistent mode): the agent drives a
+            # browser the user can watch and rescue. Set true for headless servers.
+            # profileDir defaults to ~/.dsh/browser-profiles/playwright.
+            headless: false
+            persistent: true
+            viewport:
+              width: 1280
+              height: 800
+            navigationTimeoutMs: 30000
+            ignoreHTTPSErrors: false
+          # Idle auto-close: ms of no agent activity before the window closes
+          # itself (login state is saved first, so it reopens logged-in). 0 =
+          # never auto-close. User choice: never auto-close (2026-09-04).
+          idleTimeoutMs: 0
+          maxSessions: 8
+          # Response-size lever (off by default): return "+/-/~" ref diffs between
+          # snapshots instead of the whole tree.
+          snapshot:
+            diff: false
+        # Model-facing tools: prefix, capability gates and wait caps.
+        tool:
+          # This file ships INSIDE the package, so it must not turn anything on for
+          # the people who install it. It used to: an earlier revision carried the
+          # author's own allowEvaluate/allowCdp true (2026-09-04) into every
+          # download — a backdoor each installer never asked for, and what a
+          # third-party scanner reads as "dynamic code execution, critical".
+          #
+          # Both are OFF here (matching the code defaults in src/tool.ts). Opt in
+          # per deployment, in YOUR profile or home patch — never in the package:
+          #
+          #   - id: browser
+          #     name: 'dsh-browser-playwright-codex'
+          #     config:
+          #       service: { provider: playwright }
+          #       playwright: { launch: { headless: false, persistent: true }, idleTimeoutMs: 0, maxSessions: 8, snapshot: { diff: false } }
+          #       tool:
+          #         toolPrefix: 'browser_'
+          #         allowEvaluate: true    # registers browser_evaluate
+          #         diagnostics: true      # registers browser_console_messages / browser_network_requests
+          #         allowCdp: true         # registers browser_cdp
+          #         registerDisabledTools: false
+          #         interactiveOnlyDefault: false   # true = snapshot returns actionable elements only
+          #         maxWaitMs: 60000
+          #
+          # The loader replaces an entry's config object WHOLESALE, so a partial
+          # block resets every key you leave out. List all three sections.
+          #
+          # browser_evaluate runs arbitrary page JavaScript: it can read anything
+          # in the page and act on the user's logged-in sites. browser_cdp is CDP
+          # with an allow-list — only inspection and input simulation pass
+          # (lib/cdp-policy.js); network/storage/cookie/security commands never do.
+          # Neither tool is registered at all while its gate is false.
+          allowEvaluate: false
+          allowCdp: false
+          # Diagnostics tools (console messages, network requests) are OFF here:
+          # they debug a failed run; their schemas would otherwise sit in the
+          # system prompt every turn. Turn them on per deployment.
+          diagnostics: false