dsh-ntr 0.1.3 → 0.1.4

package/README.en.md

@@ -22,7 +22,11 @@ In one sentence: your host agent acts as an ACP client, negotiates capabilities
 
 This plugin is a DSH **bundle**: `package.json`'s `dsh.bundle.patch` declares the two patches shipped with the package — the plugin's own `cordis.patch.yml`, and the NTR-mode preset at `presets/acp/acp.patch.yml`. After `dsh plugin` installs it, the package joins the profile's `dsh.profile.bundles` and the patch layers take effect **automatically** — no config file needs editing by hand.
 
-### Option 1: from the npm registry
+### Option 1: from the official UI
+
+Open the **Plugins** page in DSH's sidebar, click **Add plugin**, enter `dsh-ntr`, and click install. Refresh the page and it's ready to use.
+
+### Option 2: from the npm registry
 
 ```bash
 dsh plugin --profile web add dsh-ntr@latest
@@ -32,7 +36,7 @@ Run the same command again to upgrade.
 
 > **Always pass `@latest`**: without it the package manager keeps reusing the version it wrote the first time, and `update` won't get you off it afterwards.
 
-### Option 2: via the [dsh-market](https://github.com/dsh-market/dsh-market) plugin market
+### Option 3: via the [dsh-market](https://github.com/dsh-market/dsh-market) plugin market
 
 First install the market itself:
 
@@ -42,9 +46,9 @@ dsh plugin --profile web add dshmarket
 
 Restart `dsh web` and open the **Plugins** page: on dsh 0.2.1+ the market is the **Plugin Market** tab next to **Installed**; on older hosts it lives under **Settings → Plugin Market**. Search for `dsh-ntr`, confirm the source, and install in one click — refresh the page and it's live; update checks and upgrades afterwards also happen in the market. The official DSH Desktop app ships with this market preset, so just search and install from the app's **Plugins** page.
 
-> The market's catalog comes from the curated [awesome-dsh-plugin](https://awesome-dsh-plugin.com) registry and refreshes daily. If `dsh-ntr` doesn't show up in search yet, install it from npm with Option 1 — nothing is lost.
+> The market's catalog comes from the curated [awesome-dsh-plugin](https://awesome-dsh-plugin.com) registry and refreshes daily. If `dsh-ntr` doesn't show up in search yet, install it from npm with Option 2 — nothing is lost.
 
-### Option 3: from source
+### Option 4: from source
 
 Requires Node.js ≥ 22.12.0.
 

package/README.md

@@ -22,7 +22,11 @@
 
 本插件是一个 DSH **bundle**：`package.json` 的 `dsh.bundle.patch` 声明了随包分发的两层补丁——插件本体的 `cordis.patch.yml`，与 NTR 模式 preset 的 `presets/acp/acp.patch.yml`。用 `dsh plugin` 安装后，包会加入 profile 的 `dsh.profile.bundles`，补丁层**自动生效**，无需手动编辑任何配置文件。
 
-### 方式一：从 npm registry 安装
+### 方式一：从官方界面安装
+
+打开 DSH 侧边栏的**插件**页，点**添加插件**，输入 `dsh-ntr`，点安装。装完刷新页面即可使用。
+
+### 方式二：从 npm registry 安装
 
 ```bash
 dsh plugin --profile web add dsh-ntr@latest
@@ -32,7 +36,7 @@ dsh plugin --profile web add dsh-ntr@latest
 
 > `add` 时**务必带 `@latest`**：不带的话包管理器会一直复用首次写入的版本号，之后连 `update` 也升不上去。
 
-### 方式二：经 [dsh-market](https://github.com/dsh-market/dsh-market) 插件市场安装
+### 方式三：经 [dsh-market](https://github.com/dsh-market/dsh-market) 插件市场安装
 
 先装市场本体：
 
@@ -42,9 +46,9 @@ dsh plugin --profile web add dshmarket
 
 重启 `dsh web`，打开**插件**页：dsh 0.2.1 起市场是「已安装」旁边的**插件市场**标签页；更早的版本在**设置 → 插件市场**。搜索 `dsh-ntr`，确认来源后一键安装，装完刷新页面即可使用；之后的更新检查与升级也在市场里完成。官方 DSH 桌面 App 自带这个市场，直接在 App 的**插件**页里搜索安装即可。
 
-> 市场的插件清单来自 [awesome-dsh-plugin](https://awesome-dsh-plugin.com) 精选目录，每天自动刷新。若搜索结果里暂时没有 `dsh-ntr`，用方式一从 npm 安装，不影响使用。
+> 市场的插件清单来自 [awesome-dsh-plugin](https://awesome-dsh-plugin.com) 精选目录，每天自动刷新。若搜索结果里暂时没有 `dsh-ntr`，用方式二从 npm 安装，不影响使用。
 
-### 方式三：从源码安装
+### 方式四：从源码安装
 
 需要 Node.js ≥ 22.12.0。
 

package/lib/bridge.js

@@ -12,7 +12,7 @@ import { delimiter, dirname, join } from 'node:path';
 import { Writable, Readable } from 'node:stream';
 import { accessSync, constants, existsSync, mkdirSync, readFileSync, realpathSync, renameSync, statSync, writeFileSync } from 'node:fs';
 import { client, methods, ndJsonStream, PROTOCOL_VERSION, RequestError, } from '@agentclientprotocol/sdk';
-import { AcpError, EventHub, PermissionLedger, Transcript, acpToolCallDelta, acpToolCallFromUpdate, buildAdapterModelId, childEnv, isAcpToolContinuation, isUnderPath, parseCatalogOptions, quoteCommandToken, randomUUID, resolveFsPath, resolveSessionCwd, selectForwardPrompt, acpPresetStatusFromRoster, splitAdapterModelId, updateContentToText, describeUnhandledUpdate, BOOKKEEPING_UPDATES, validateAgentDef, validateTagHalo, } from "./impl.js";
+import { AcpError, EventHub, PermissionLedger, Transcript, acpToolCallDelta, acpToolCallFromUpdate, buildAdapterModelId, childEnv, isAcpToolContinuation, isUnderPath, mergeCatalogShape, parseCatalogOptions, usableConfigOptions, quoteCommandToken, randomUUID, resolveFsPath, resolveSessionCwd, selectForwardPrompt, acpPresetStatusFromRoster, splitAdapterModelId, updateContentToText, describeUnhandledUpdate, BOOKKEEPING_UPDATES, validateAgentDef, validateTagHalo, } from "./impl.js";
 import { DEFAULT_TAG_HALO } from "./protocol.js";
 const STDERR_TAIL_LINES = 40;
 /** Single fs/read_text_file answer cap — agents reading huge files must page via `line`/`limit`. */
@@ -101,7 +101,13 @@ export class AgentBridge {
     constructor(def, hub, transcriptMax, 
     /** Called when the connection dies so the manager can settle outstanding prompts. */
     onConnectionLost, 
-    /** Observes `configOptions` from any real session so the manager's model catalog stays warm. */
+    /**
+     * Observes `configOptions` from any real session so the manager's model
+     * catalog stays warm. `shapeOnly` marks a per-session answer (a
+     * `session/set_config_option` response): only the option SHAPE belongs in
+     * the agent-level catalog, never its `currentValue` — see
+     * {@link BridgeManager.absorbCatalogShape}.
+     */
     onCatalog, 
     /** Observes `availableCommands` from any real session so the agent's slash menu survives its sessions. */
     onCommands, tuning, 
@@ -464,7 +470,14 @@ export class AgentBridge {
             protocolVersion: PROTOCOL_VERSION,
             // The fs handlers are implemented below behind the same workspace
             // sandbox that gates session cwds; terminal is not implemented.
-            clientCapabilities: { fs: { readTextFile: true, writeTextFile: true }, terminal: false },
+            // `session.configOptions.boolean` advertises the toggle form of
+            // `configOptions`; agents that gate `type: "boolean"` entries on it
+            // would otherwise never publish them.
+            clientCapabilities: {
+                fs: { readTextFile: true, writeTextFile: true },
+                terminal: false,
+                session: { configOptions: { boolean: {} } },
+            },
             clientInfo: { name: 'dsh-ntr', title: 'DSH NTR', version: '0.1.0' },
         }), 'initialize');
         this.initResult = init;
@@ -516,7 +529,8 @@ export class AgentBridge {
             throw new AcpError('agent_unreachable', 'acp: connection is not open');
         const resolved = resolveSessionCwd(this.def, cwd, p => realpathSync(p), extraRoots);
         const result = await this.withStartupTimeout(conn.agent.request(methods.agent.session.new, { cwd: resolved, mcpServers: [] }), 'session/new');
-        if (this.onCatalog !== undefined && result.configOptions !== undefined && result.configOptions !== null) {
+        // An empty `configOptions` answer is a non-answer — see `usableConfigOptions`.
+        if (this.onCatalog !== undefined && usableConfigOptions(result.configOptions)) {
             this.onCatalog(this.def.id, result.configOptions);
         }
         const session = {
@@ -541,7 +555,7 @@ export class AgentBridge {
                     },
                 }
                 : {}),
-            ...(result.configOptions !== undefined && result.configOptions !== null ? { configOptions: result.configOptions } : {}),
+            ...(usableConfigOptions(result.configOptions) ? { configOptions: result.configOptions } : {}),
         };
         this.sessions.set(session.sessionId, session);
         this.acpIndex.set(session.acpSessionId, session);
@@ -720,7 +734,7 @@ export class AgentBridge {
             this.acpIndex.set(session.acpSessionId, session);
             this.hub.broadcast({ type: 'session/reset', sessionId: session.sessionId, session: snapshot(session) });
             const result = await this.withStartupTimeout(conn.agent.request(methods.agent.session.load, { sessionId: session.acpSessionId, cwd: resolved, mcpServers: [] }), 'session/load');
-            if (this.onCatalog !== undefined && result.configOptions !== undefined && result.configOptions !== null) {
+            if (this.onCatalog !== undefined && usableConfigOptions(result.configOptions)) {
                 this.onCatalog(this.def.id, result.configOptions);
             }
             session.modes = result.modes !== undefined && result.modes !== null
@@ -733,7 +747,7 @@ export class AgentBridge {
                     })),
                 }
                 : session.modes;
-            if (Array.isArray(result.configOptions))
+            if (usableConfigOptions(result.configOptions))
                 session.configOptions = result.configOptions;
             this.sessions.set(session.sessionId, session);
             this.released.delete(session.sessionId);
@@ -825,15 +839,26 @@ export class AgentBridge {
         const conn = this.conn;
         if (conn === undefined)
             throw new AcpError('agent_unreachable', 'acp: connection is not open');
-        const result = await this.runCommand(() => conn.agent.request(methods.agent.session.setConfigOption, { sessionId: session.acpSessionId, configId, value }));
+        // A toggle must say so on the wire: the request union discriminates on
+        // `type`, and a bare boolean `value` is not a select value id.
+        const params = typeof value === 'boolean'
+            ? { sessionId: session.acpSessionId, configId, value, type: 'boolean' }
+            : { sessionId: session.acpSessionId, configId, value };
+        const result = await this.runCommand(() => conn.agent.request(methods.agent.session.setConfigOption, params));
         // The response carries the full authoritative option set; some agents
         // (pi-acp observed) never re-notify after a client-driven set. It feeds the
-        // SESSION snapshot only — never the agent-level catalog: `currentValue` in
-        // a per-session answer describes this session's pick, and promoting it to
-        // the catalog would turn the seat's own choice into `reasoning.defaultEffort`
-        // for every other conversation (and back onto itself next turn).
-        if (Array.isArray(result.configOptions))
+        // SESSION snapshot first — `currentValue` in a per-session answer describes
+        // this session's pick, and promoting it to the catalog would turn the
+        // seat's own choice into `reasoning.defaultEffort` for every other
+        // conversation (and back onto itself next turn). The option SHAPE still
+        // reaches the agent-level catalog (`shapeOnly`): this response is often the
+        // only place an agent publishes how its option set changes with the model
+        // (opencode offers `thought_level` just for models with variants), and the
+        // composer's thought chip is driven by the catalog.
+        if (usableConfigOptions(result.configOptions)) {
             session.configOptions = result.configOptions;
+            this.onCatalog?.(this.def.id, result.configOptions, true);
+        }
         this.publishMeta(session);
     }
     publishMeta(session) {
@@ -1115,7 +1140,10 @@ export class AgentBridge {
                 session.modes = { ...session.modes, currentModeId: update.currentModeId };
                 break;
             case 'config_option_update':
-                session.configOptions = update.configOptions ?? [];
+                // Empty answers never overwrite the last known set — see `usableConfigOptions`.
+                if (!usableConfigOptions(update.configOptions))
+                    break;
+                session.configOptions = update.configOptions;
                 this.onCatalog?.(this.def.id, session.configOptions);
                 break;
             case 'available_commands_update':
@@ -2864,8 +2892,19 @@ export class BridgeManager {
         if (option === undefined) {
             throw new AcpError('unknown_config', `acp: agent "${bound}" publishes no extra config option "${configId}"`);
         }
-        if (!option.choices.some(choice => choice.value === value)) {
-            throw new AcpError('bad_config', `acp: "${value}" is not a value of config option "${configId}"`);
+        if (option.kind === 'boolean') {
+            // The browser sends 'true' / 'false' from a checkbox row; the wire and
+            // the persisted pick both use the real boolean.
+            const toggle = typeof value === 'boolean'
+                ? value
+                : value === 'true' ? true : value === 'false' ? false : undefined;
+            if (toggle === undefined) {
+                throw new AcpError('bad_config', `acp: "${String(value)}" is not a boolean value of config option "${configId}"`);
+            }
+            value = toggle;
+        }
+        else if (typeof value !== 'string' || !option.choices.some(choice => choice.value === value)) {
+            throw new AcpError('bad_config', `acp: "${String(value)}" is not a value of config option "${configId}"`);
         }
         const current = this.chatAgentIntent.get(dshSessionId);
         const extraConfigs = current?.agentId === bound ? { ...current.extraConfigs } : {};
@@ -2962,12 +3001,17 @@ export class BridgeManager {
                     });
                     continue;
                 }
-                if (!option.choices.some(entry => entry.value === value)) {
+                // A toggle's truth is its boolean itself; a select's is membership in
+                // the choices the agent still offers.
+                const usable = option.kind === 'boolean'
+                    ? typeof value === 'boolean'
+                    : typeof value === 'string' && option.choices.some(entry => entry.value === value);
+                if (!usable) {
                     this.hub.broadcast({
                         type: 'agent/notice',
                         agentId: session.agentId,
                         level: 'error',
-                        text: `acp: agent "${session.agentId}" no longer offers "${value}" for config "${configId}"; keeping its own`,
+                        text: `acp: agent "${session.agentId}" no longer offers "${String(value)}" for config "${configId}"; keeping its own`,
                     });
                     continue;
                 }
@@ -3194,7 +3238,7 @@ export class BridgeManager {
                 continue;
             const configs = extraConfigs !== null && typeof extraConfigs === 'object' && !Array.isArray(extraConfigs)
                 ? Object.fromEntries(Object.entries(extraConfigs)
-                    .filter(([, value]) => typeof value === 'string' && value !== ''))
+                    .filter(([, value]) => typeof value === 'boolean' || (typeof value === 'string' && value !== '')))
                 : {};
             this.chatAgentIntent.set(dshSessionId, {
                 agentId,
@@ -3359,6 +3403,9 @@ export class BridgeManager {
      * current `parseCatalogOptions` has now seen this agent's real answer.
      */
     absorbCatalog(agentId, configOptions) {
+        // Empty / absurd answers never overwrite known state — see `usableConfigOptions`.
+        if (!usableConfigOptions(configOptions))
+            return this.catalogs.get(agentId) ?? { catalog: {}, updatedAt: 0 };
         const catalog = parseCatalogOptions(configOptions);
         this.catalogStale.delete(agentId);
         const existing = this.catalogs.get(agentId);
@@ -3386,6 +3433,30 @@ export class BridgeManager {
             this.broadcastExposure(agentId);
         return entry;
     }
+    /**
+     * Fold a per-session option answer's SHAPE into the agent's catalog.
+     *
+     * The counterpart of {@link absorbCatalog} for `session/set_config_option`
+     * responses: which options exist and which choices they offer is agent
+     * knowledge the seat must show (the composer's thought chip only renders
+     * when the catalog has `thought_level`, and some agents publish it only in
+     * the answer to a model change), while a per-session `currentValue` stays
+     * out of it. No auto-exposure and no staleness discharge either — this is
+     * one session's answer, not the agent-level observation those marks track.
+     */
+    absorbCatalogShape(agentId, configOptions) {
+        if (!usableConfigOptions(configOptions))
+            return;
+        const observed = parseCatalogOptions(configOptions);
+        const existing = this.catalogs.get(agentId);
+        const catalog = mergeCatalogShape(existing?.catalog, observed);
+        if (existing !== undefined && JSON.stringify(existing.catalog) === JSON.stringify(catalog))
+            return;
+        const entry = { catalog, updatedAt: Date.now() };
+        this.catalogs.set(agentId, entry);
+        this.persistCatalogs();
+        this.hub.broadcast({ type: 'catalog/updated', agentId, updatedAt: entry.updatedAt });
+    }
     /**
      * Remember the slash commands an agent declared in a real session.
      *
@@ -3418,11 +3489,12 @@ export class BridgeManager {
      * selection would resurrect a model the user deliberately unchecked every
      * time the agent repeats its catalog. The first observation has no baseline
      * (`before === undefined`), so it adds nothing; the user can still add the
-     * whole list with one click in the settings card.
+     * whole list with one click in the settings card. Discovery is ON unless the
+     * agent definition explicitly opts out with `autoDiscoverModels: false`.
      */
     autoExposeNewModels(agentId, next, before) {
         const def = this.defsMap.get(agentId);
-        if (def?.autoDiscoverModels !== true || before === undefined)
+        if (def === undefined || def.autoDiscoverModels === false || before === undefined)
             return false;
         const known = new Set((before.model?.choices ?? []).map(choice => choice.value));
         const exposed = new Set(def.exposedModels ?? []);
@@ -3444,7 +3516,7 @@ export class BridgeManager {
             type: 'agent/exposure',
             agentId,
             exposedModels: [...(def.exposedModels ?? [])],
-            autoDiscoverModels: def.autoDiscoverModels === true,
+            autoDiscoverModels: def.autoDiscoverModels !== false,
         });
     }
     catalogFor(agentId) {
@@ -3618,6 +3690,7 @@ export class BridgeManager {
                 extras: extraOptions.map(option => ({
                     configId: option.configId,
                     name: option.name,
+                    ...(option.kind === 'boolean' ? { kind: 'boolean' } : {}),
                     ...(option.currentValue === null ? {} : { currentValue: option.currentValue }),
                     choices: option.choices.map(choice => ({
                         id: choice.value,
@@ -3841,8 +3914,11 @@ export class BridgeManager {
         if (bridge === undefined) {
             bridge = this.makeBridge(def, (b, reason) => {
                 this.connectionLost(b, reason);
-            }, (observedAgentId, configOptions) => {
-                this.absorbCatalog(observedAgentId, configOptions);
+            }, (observedAgentId, configOptions, shapeOnly) => {
+                if (shapeOnly)
+                    this.absorbCatalogShape(observedAgentId, configOptions);
+                else
+                    this.absorbCatalog(observedAgentId, configOptions);
             }, (observedAgentId, commands) => {
                 this.absorbCommands(observedAgentId, commands);
             });
@@ -3970,16 +4046,17 @@ export class BridgeManager {
      * Toggle auto-discovery of newly published models. Lightweight like
      * `setExposedModels`: the def is mutated in place (live bridges share the
      * object) and persisted; no process is restarted, and the current selection
-     * is left exactly as it is.
+     * is left exactly as it is. On is the default, so only the opt-out
+     * (`false`) is written to the file.
      */
     async setAutoDiscoverModels(agentId, autoDiscoverModels) {
         const def = this.defsMap.get(agentId);
         if (def === undefined)
             throw new AcpError('unknown_agent', `acp: agent "${agentId}" is not configured`);
         if (autoDiscoverModels)
-            def.autoDiscoverModels = true;
-        else
             delete def.autoDiscoverModels;
+        else
+            def.autoDiscoverModels = false;
         this.persistAgents();
         this.broadcastExposure(agentId);
     }
@@ -4237,6 +4314,13 @@ export class BridgeManager {
         const extraConfigs = {};
         for (const option of catalog.extras ?? []) {
             const value = option.currentValue;
+            if (option.kind === 'boolean') {
+                // The catalog encodes a toggle as 'true' / 'false'; the pick stores the
+                // real boolean, exactly like an explicit checkbox click.
+                if (value === 'true' || value === 'false')
+                    extraConfigs[option.configId] = value === 'true';
+                continue;
+            }
             if (value !== null && value !== undefined && value !== '')
                 extraConfigs[option.configId] = value;
         }
@@ -4290,7 +4374,11 @@ export class BridgeManager {
                 const option = options.extras?.find(entry => entry.configId === configId);
                 if (option === undefined)
                     continue;
-                if (option.choices.length > 0 && !option.choices.some(choice => choice.value === value))
+                // The fresh session's own option set is the authority for BOTH forms:
+                // a toggle must still be a toggle, a select must still offer the value.
+                if (option.kind === 'boolean'
+                    ? typeof value !== 'boolean'
+                    : typeof value !== 'string' || (option.choices.length > 0 && !option.choices.some(choice => choice.value === value)))
                     continue;
                 extraConfigs[configId] = value;
                 changed = true;

package/lib/client/acp-ui.js

@@ -350,7 +350,7 @@ export function AcpMenu({ model, anchor, onPick, onClose }) {
     }, [model, onClose]);
     if (model === undefined)
         return null;
-    return (_jsxs("div", { ref: rootRef, className: "ntr-acp-menu", role: "menu", "aria-label": model.title, children: [_jsxs("div", { className: "ntr-acp-menu-head", children: [_jsx("span", { className: "ntr-acp-menu-title", children: model.title }), model.loading && _jsx("span", { className: "ntr-acp-menu-spin", "aria-hidden": true, children: "\u2026" })] }), model.note !== undefined && _jsx("div", { className: "ntr-acp-menu-note", children: model.note }), model.error !== undefined && _jsx("div", { className: "ntr-acp-menu-error", children: model.error }), _jsxs("div", { className: "ntr-acp-menu-body", children: [model.sections.map(section => (_jsxs("div", { className: "ntr-acp-menu-section", children: [section.title !== undefined && _jsx("div", { className: "ntr-acp-menu-section-title", children: section.title }), section.note !== undefined && _jsx("div", { className: "ntr-acp-menu-note", children: section.note }), section.items.map(item => (_jsxs("button", { type: "button", role: "menuitemradio", "aria-checked": item.active === true, className: `ntr-acp-menu-item${item.active === true ? ' is-active' : ''}`, disabled: item.disabled === true, onClick: () => { onPick(item.id, section.id); }, children: [item.dot !== undefined && _jsx("span", { className: "ntr-acp-dot", style: { background: item.dot }, "aria-hidden": true }), _jsx("span", { className: "ntr-acp-menu-item-label", children: item.label }), item.active === true && _jsx("span", { className: "ntr-acp-tick", "aria-hidden": true, children: "\u2713" }), item.detail !== undefined && _jsx("span", { className: "ntr-acp-menu-item-detail", children: item.detail })] }, `${section.id}/${item.id}`)))] }, section.id))), model.sections.every(section => section.items.length === 0) && (_jsx("div", { className: "ntr-acp-menu-empty", children: "\u2014" }))] })] }));
+    return (_jsxs("div", { ref: rootRef, className: "ntr-acp-menu", role: "menu", "aria-label": model.title, children: [_jsxs("div", { className: "ntr-acp-menu-head", children: [_jsx("span", { className: "ntr-acp-menu-title", children: model.title }), model.loading && _jsx("span", { className: "ntr-acp-menu-spin", "aria-hidden": true, children: "\u2026" })] }), model.note !== undefined && _jsx("div", { className: "ntr-acp-menu-note", children: model.note }), model.error !== undefined && _jsx("div", { className: "ntr-acp-menu-error", children: model.error }), _jsxs("div", { className: "ntr-acp-menu-body", children: [model.sections.map(section => (_jsxs("div", { className: "ntr-acp-menu-section", children: [section.title !== undefined && _jsx("div", { className: "ntr-acp-menu-section-title", children: section.title }), section.note !== undefined && _jsx("div", { className: "ntr-acp-menu-note", children: section.note }), section.items.map(item => (_jsxs("button", { type: "button", role: item.checkbox === true ? 'menuitemcheckbox' : 'menuitemradio', "aria-checked": item.active === true, className: `ntr-acp-menu-item${item.active === true ? ' is-active' : ''}`, disabled: item.disabled === true, onClick: () => { onPick(item.id, section.id); }, children: [item.dot !== undefined && _jsx("span", { className: "ntr-acp-dot", style: { background: item.dot }, "aria-hidden": true }), item.checkbox === true && (_jsx("span", { className: `ntr-acp-check${item.active === true ? ' is-on' : ''}`, "aria-hidden": true, children: item.active === true ? '✓' : '' })), _jsx("span", { className: "ntr-acp-menu-item-label", children: item.label }), item.active === true && item.checkbox !== true && _jsx("span", { className: "ntr-acp-tick", "aria-hidden": true, children: "\u2713" }), item.detail !== undefined && _jsx("span", { className: "ntr-acp-menu-item-detail", children: item.detail })] }, `${section.id}/${item.id}`)))] }, section.id))), model.sections.every(section => section.items.length === 0) && (_jsx("div", { className: "ntr-acp-menu-empty", children: "\u2014" }))] })] }));
 }
 // ---- Chips ----
 /**
@@ -490,6 +490,8 @@ const STYLE = `
 .ntr-acp-seat-sub { color: var(--dsw-alias-text-secondary, #6b7280); overflow: hidden; text-overflow: ellipsis; white-space: nowrap; }
 .ntr-acp-dot { width: 7px; height: 7px; border-radius: 50%; flex: none; }
 .ntr-acp-tick { margin-left: auto; color: var(--dsw-alias-state-business-primary, #2563eb); }
+.ntr-acp-check { display: inline-flex; align-items: center; justify-content: center; flex: none; width: 13px; height: 13px; border: 1px solid var(--dsw-alias-border-l2, #d1d5db); border-radius: 3px; font-size: 10px; line-height: 1; color: var(--dsw-alias-state-business-primary, #2563eb); }
+.ntr-acp-check.is-on { border-color: var(--dsw-alias-state-business-primary, #2563eb); }
 .ntr-acp-menu { position: absolute; bottom: calc(100% + 8px); left: 8px; z-index: 100; width: min(360px, 100%); max-height: min(420px, 60vh); display: flex; flex-direction: column; border: 1px solid var(--dsw-alias-border-l2, #d1d5db); border-radius: 10px; background: var(--dsw-alias-bg-layer-1, #fff); box-shadow: 0 12px 32px rgba(0, 0, 0, 0.16); font-size: 12px; overflow: hidden; }
 .ntr-acp-menu-head { padding: 8px 10px 6px; font-weight: 600; display: flex; align-items: center; gap: 6px; }
 .ntr-acp-menu-title { flex: 1; }

package/lib/client/panel.js

@@ -349,7 +349,7 @@ function payloadEquals(payload, current) {
         return false;
     if (exposureKey(payload.exposedModels) !== exposureKey(current.exposedModels))
         return false;
-    if ((payload.autoDiscoverModels === true) !== (current.autoDiscoverModels === true))
+    if ((payload.autoDiscoverModels !== false) !== (current.autoDiscoverModels !== false))
         return false;
     // An absent field means "use the host default" — which is what `current`
     // already resolved to (see `resolvedIdleTimeoutMs`), so it is not a change.
@@ -428,7 +428,7 @@ function AgentRow({ detected, activeAgent, acp, isDefault, synthesized, newCusto
         // Re-sync the editor only when the persisted filter actually changes, so
         // background resyncs never clobber in-progress checkbox edits.
     }, [serverExposureKey]);
-    const serverAutoDiscover = detected.currentConfig?.autoDiscoverModels === true;
+    const serverAutoDiscover = detected.currentConfig?.autoDiscoverModels !== false;
     useEffect(() => {
         // The switch saves itself through the lightweight RPC, so the host is the
         // truth; this keeps the form (and therefore a later full save) in step.
@@ -536,7 +536,7 @@ function initialForm(detected, activeAgent) {
         roots,
         envRows: Object.entries(config?.env ?? {}).map(([key, value]) => ({ key, value })),
         exposed: config?.exposedModels === undefined ? undefined : [...config.exposedModels],
-        autoDiscoverModels: config?.autoDiscoverModels === true,
+        autoDiscoverModels: config?.autoDiscoverModels !== false,
         // Prefer the host's resolved value: it is the one actually in effect.
         idleStr: idleText(activeAgent?.idleTimeoutMs ?? config?.idleTimeoutMs),
     };
@@ -590,7 +590,9 @@ function buildPayload(form, detected, newCustom) {
         ...(form.npmPackage.trim() === '' ? {} : { npmPackage: form.npmPackage.trim() }),
         env,
         ...(form.exposed === undefined ? {} : { exposedModels: form.exposed }),
-        ...(form.autoDiscoverModels ? { autoDiscoverModels: true } : {}),
+        // Auto-discovery is ON by default: only an unchecked switch records the
+        // opt-out, otherwise a full save would silently re-enable it.
+        ...(form.autoDiscoverModels ? {} : { autoDiscoverModels: false }),
         // `''` sends nothing: the host then applies its own default.
         ...(idleRaw === '' || !idleValid(idleRaw) ? {} : { idleTimeoutMs: Number(idleRaw) }),
     };

package/lib/client/session-controls.js

@@ -286,10 +286,14 @@ export function advancedOptions(catalog, choice) {
         return [];
     const picked = choice !== undefined && choice.agentId === catalog.agentId ? choice.extraConfigs : undefined;
     return extras.map(option => {
-        const value = picked?.[option.configId] ?? option.currentValue;
+        const raw = picked?.[option.configId] ?? option.currentValue;
+        // A toggle's catalog value is 'true' / 'false'; a conversation pick stores
+        // the real boolean. Both collapse to the string form the menu speaks.
+        const value = typeof raw === 'boolean' ? (raw ? 'true' : 'false') : raw;
         return {
             configId: option.configId,
             name: option.name,
+            ...(option.kind === 'boolean' ? { kind: 'boolean' } : {}),
             ...(value === undefined ? {} : { value }),
             choices: option.choices,
         };
@@ -694,16 +698,28 @@ function thoughtSection(context) {
  * and the pick would then be read as the primary model axis.
  */
 function advancedSections(context) {
-    return advancedOptions(context.catalog, context.choice).map(option => ({
-        id: `config:${option.configId}`,
-        title: option.name,
-        items: option.choices.map(choice => ({
-            id: choice.id,
-            label: choice.name,
-            ...(choice.description === undefined ? {} : { detail: choice.description }),
-            active: choice.id === option.value,
-        })),
-    }));
+    return advancedOptions(context.catalog, context.choice).map(option => {
+        if (option.kind === 'boolean') {
+            // A toggle is ONE checkbox row, not an On/Off pair: the row's check is
+            // the current value and the pick sends the OPPOSITE, which `onPick`
+            // forwards verbatim (the host normalizes 'true' / 'false' to a boolean).
+            const checked = option.value === 'true';
+            return {
+                id: `config:${option.configId}`,
+                items: [{ id: checked ? 'false' : 'true', label: option.name, active: checked, checkbox: true }],
+            };
+        }
+        return {
+            id: `config:${option.configId}`,
+            title: option.name,
+            items: option.choices.map(choice => ({
+                id: choice.id,
+                label: choice.name,
+                ...(choice.description === undefined ? {} : { detail: choice.description }),
+                active: choice.id === option.value,
+            })),
+        };
+    });
 }
 /**
  * REMOVED on purpose (2026-10-17): the agent's own `session/set_mode` list used

package/lib/client.js

@@ -1988,7 +1988,7 @@ window.__ModuleLoader__.load({
 								}),
 								section.items.map((item) => /* @__PURE__ */ (0, react_jsx_runtime.jsxs)("button", {
 									type: "button",
-									role: "menuitemradio",
+									role: item.checkbox === true ? "menuitemcheckbox" : "menuitemradio",
 									"aria-checked": item.active === true,
 									className: `ntr-acp-menu-item${item.active === true ? " is-active" : ""}`,
 									disabled: item.disabled === true,
@@ -2001,11 +2001,16 @@ window.__ModuleLoader__.load({
 											style: { background: item.dot },
 											"aria-hidden": true
 										}),
+										item.checkbox === true && /* @__PURE__ */ (0, react_jsx_runtime.jsx)("span", {
+											className: `ntr-acp-check${item.active === true ? " is-on" : ""}`,
+											"aria-hidden": true,
+											children: item.active === true ? "✓" : ""
+										}),
 										/* @__PURE__ */ (0, react_jsx_runtime.jsx)("span", {
 											className: "ntr-acp-menu-item-label",
 											children: item.label
 										}),
-										item.active === true && /* @__PURE__ */ (0, react_jsx_runtime.jsx)("span", {
+										item.active === true && item.checkbox !== true && /* @__PURE__ */ (0, react_jsx_runtime.jsx)("span", {
 											className: "ntr-acp-tick",
 											"aria-hidden": true,
 											children: "✓"
@@ -2331,6 +2336,8 @@ window.__ModuleLoader__.load({
 .ntr-acp-seat-sub { color: var(--dsw-alias-text-secondary, #6b7280); overflow: hidden; text-overflow: ellipsis; white-space: nowrap; }
 .ntr-acp-dot { width: 7px; height: 7px; border-radius: 50%; flex: none; }
 .ntr-acp-tick { margin-left: auto; color: var(--dsw-alias-state-business-primary, #2563eb); }
+.ntr-acp-check { display: inline-flex; align-items: center; justify-content: center; flex: none; width: 13px; height: 13px; border: 1px solid var(--dsw-alias-border-l2, #d1d5db); border-radius: 3px; font-size: 10px; line-height: 1; color: var(--dsw-alias-state-business-primary, #2563eb); }
+.ntr-acp-check.is-on { border-color: var(--dsw-alias-state-business-primary, #2563eb); }
 .ntr-acp-menu { position: absolute; bottom: calc(100% + 8px); left: 8px; z-index: 100; width: min(360px, 100%); max-height: min(420px, 60vh); display: flex; flex-direction: column; border: 1px solid var(--dsw-alias-border-l2, #d1d5db); border-radius: 10px; background: var(--dsw-alias-bg-layer-1, #fff); box-shadow: 0 12px 32px rgba(0, 0, 0, 0.16); font-size: 12px; overflow: hidden; }
 .ntr-acp-menu-head { padding: 8px 10px 6px; font-weight: 600; display: flex; align-items: center; gap: 6px; }
 .ntr-acp-menu-title { flex: 1; }
@@ -2777,7 +2784,7 @@ window.__ModuleLoader__.load({
 			if (!sameStringArray(payload.allowedCwdRoots ?? [], current.allowedCwdRoots ?? [])) return false;
 			if (!sameStringMap(payload.env ?? {}, current.env ?? {})) return false;
 			if (exposureKey(payload.exposedModels) !== exposureKey(current.exposedModels)) return false;
-			if (payload.autoDiscoverModels === true !== (current.autoDiscoverModels === true)) return false;
+			if (payload.autoDiscoverModels !== false !== (current.autoDiscoverModels !== false)) return false;
 			if (payload.idleTimeoutMs !== void 0 && payload.idleTimeoutMs !== current.idleTimeoutMs) return false;
 			return true;
 		}
@@ -3142,7 +3149,7 @@ window.__ModuleLoader__.load({
 					exposed: server === void 0 ? void 0 : [...server]
 				}));
 			}, [serverExposureKey]);
-			const serverAutoDiscover = detected.currentConfig?.autoDiscoverModels === true;
+			const serverAutoDiscover = detected.currentConfig?.autoDiscoverModels !== false;
 			(0, react.useEffect)(() => {
 				setForm((prev) => ({
 					...prev,
@@ -3396,7 +3403,7 @@ window.__ModuleLoader__.load({
 					value
 				})),
 				exposed: config?.exposedModels === void 0 ? void 0 : [...config.exposedModels],
-				autoDiscoverModels: config?.autoDiscoverModels === true,
+				autoDiscoverModels: config?.autoDiscoverModels !== false,
 				idleStr: idleText(activeAgent?.idleTimeoutMs ?? config?.idleTimeoutMs)
 			};
 		}
@@ -3444,7 +3451,7 @@ window.__ModuleLoader__.load({
 				...form.npmPackage.trim() === "" ? {} : { npmPackage: form.npmPackage.trim() },
 				env,
 				...form.exposed === void 0 ? {} : { exposedModels: form.exposed },
-				...form.autoDiscoverModels ? { autoDiscoverModels: true } : {},
+				...form.autoDiscoverModels ? {} : { autoDiscoverModels: false },
 				...idleRaw === "" || !idleValid(idleRaw) ? {} : { idleTimeoutMs: Number(idleRaw) }
 			};
 			if (form.rootsOpen) {
@@ -4794,10 +4801,12 @@ window.__ModuleLoader__.load({
 			if (catalog === void 0 || extras.length === 0) return [];
 			const picked = choice !== void 0 && choice.agentId === catalog.agentId ? choice.extraConfigs : void 0;
 			return extras.map((option) => {
-				const value = picked?.[option.configId] ?? option.currentValue;
+				const raw = picked?.[option.configId] ?? option.currentValue;
+				const value = typeof raw === "boolean" ? raw ? "true" : "false" : raw;
 				return {
 					configId: option.configId,
 					name: option.name,
+					...option.kind === "boolean" ? { kind: "boolean" } : {},
 					...value === void 0 ? {} : { value },
 					choices: option.choices
 				};
@@ -5140,16 +5149,30 @@ window.__ModuleLoader__.load({
 		* and the pick would then be read as the primary model axis.
 		*/
 		function advancedSections(context) {
-			return advancedOptions(context.catalog, context.choice).map((option) => ({
-				id: `config:${option.configId}`,
-				title: option.name,
-				items: option.choices.map((choice) => ({
-					id: choice.id,
-					label: choice.name,
-					...choice.description === void 0 ? {} : { detail: choice.description },
-					active: choice.id === option.value
-				}))
-			}));
+			return advancedOptions(context.catalog, context.choice).map((option) => {
+				if (option.kind === "boolean") {
+					const checked = option.value === "true";
+					return {
+						id: `config:${option.configId}`,
+						items: [{
+							id: checked ? "false" : "true",
+							label: option.name,
+							active: checked,
+							checkbox: true
+						}]
+					};
+				}
+				return {
+					id: `config:${option.configId}`,
+					title: option.name,
+					items: option.choices.map((choice) => ({
+						id: choice.id,
+						label: choice.name,
+						...choice.description === void 0 ? {} : { detail: choice.description },
+						active: choice.id === option.value
+					}))
+				};
+			});
 		}
 		/**
 		* REMOVED on purpose (2026-10-17): the agent's own `session/set_mode` list used