dsh-sidecard-ask 1.2.0 → 1.2.1

package/CHANGELOG.md

@@ -1,5 +1,28 @@
 # Changelog
 
+## 1.2.1
+
+**主题：核对 DSH 0.2.0-rc.1 的适配性，并修掉由此暴露的一个真实缺陷。**
+
+核对（本机运行中的就是 0.2.0-rc.1，`@deepseek-ai/dsh` 的 `next` 通道）：
+
+- 包内容层面：`node tools/compat-probe.mjs 0.1.7-rc.2 0.2.0-rc.1` → 16 项能力**逐项一致**，没有任何依赖被移除或改名；
+- 运行实例上：宿主半 `v1.2.0` 正常（`sideEngine`、provider `spawn,fork`、`capabilities.parent` 均可用）；
+  客户端半在 `shell.overlay` / `conversation.input.right` 都已注册；`standardProps` 仍含 `sessionId` 与 `inputActions`；
+  真机端到端作答 `start→delta×31→status→done`（1.36 s）通过。新增的 `sessions.fork`/`uiWorkspace.forkSession`
+  可选参数 `onCreated` 是纯增量，不影响既有调用。
+
+**修掉的缺陷**：`sidebar.right.pane.tab` 的运行清单里只有 `dsh-better-sidebar:sidecard-ask:card`，
+没有本插件原生右侧栏的 `sidecard-ask:card`——两个适配器用了同一个 tab kind，而 better-sidebar 会把每个 tab 描述符
+镜像注册进**同一个**原生注册表（band `extension`），该注册表对同 band 同 kind 的二次注册抛错，后注册的一方静默失败。
+
+- better-sidebar 承载面改用独立类型 `sidecard-ask:card:workbench`（`CARD_KIND_BETTER`），两端不再重叠；
+- `contract-test` 新增"surface kinds must not collide"一节：**在测试里模拟该注册表的重复 kind 抛错规则**
+  与 better-sidebar 的镜像注册行为，若两者再共用 kind，测试立即失败；
+- 设置页自检行改为同时报告**两个适配器**（原先原生适配器的错误在界面上不可见，只能靠翻槽位清单发现）。
+
+自测 289 → **295 项**（verify 53 / contract 124 / smoke 118）。
+
 ## 1.2.0
 
 **主题：核对并加固与 dsh-better-sidebar 0.22.1 的适配。**

package/README.md

@@ -183,6 +183,35 @@ plugin_manager(action="install_bundle", target="D:\\Users\\34332\\AI\\dsh-sideca
 
 > 注意：`version` 是加载到浏览器里的那份模块报告的版本。页面刷新后即可用它确认"跑的是不是 0.22.1"。
 
+### 5.2 与 DSH 0.2.0-rc.1 的适配核对（2026-09-29）
+
+本机实际运行的就是 **0.2.0-rc.1**（`@deepseek-ai/dsh` 的 `next` 通道；`latest` 仍是 0.1.7-rc.2）。核对分两层：
+
+**① 包内容层面**（`node tools/compat-probe.mjs 0.1.7-rc.2 0.2.0-rc.1`）：
+16 项能力**逐项与 0.1.7-rc.2 完全一致**——`shell.overlay`、`conversation.input.right`、`settings.section`、
+`data-slot` 锚点、原生右侧栏、`sessions.using/retain`、归档门、`agent/assistant-stream`、子代理、`tools.schemas`、
+`webServer` 路由……全部 ✅。
+
+**② 在运行中的实例上实测**：
+
+| 检查 | 结果 |
+|---|---|
+| 宿主半是否加载 | ✅ `/sidecard-ask/api/state` 返回 `dsh-sidecard-ask v1.2.0`，`sideEngine=true`（provider `spawn,fork`）、`capabilities.parent` 正常 |
+| 客户端半是否注册 | ✅ `shell.overlay` 有 `sidecard-ask`（order 45、active）、`conversation.input.right` 有 `sidecard-ask-session-probe`、`settings.section` 有设置页 |
+| 依赖的槽位与标准 prop | ✅ `conversation.input.right` 仍是 list/session，`standardProps` 仍含 **`sessionId`** 与 **`inputActions`** |
+| 依赖的客户端服务 | ✅ `sessions`（`retain`/`using`/`scope`）、`slots`、`locale`、`layout`、`theme` 全在；`sessions.fork`/`uiWorkspace.forkSession` 只是**新增**了可选 `onCreated`，不影响既有调用 |
+| 端到端作答 | ✅ 真机 `start→delta×31→status→done`（1.36 s，`streamSource=frames`），只读工具白名单照常生效 |
+
+**③ 由此发现并修掉的真实缺陷（1.2.1）**：运行中的 `sidebar.right.pane.tab` 清单里只有
+`dsh-better-sidebar:sidecard-ask:card`，**没有我的原生 `sidecard-ask:card`**。原因是两个适配器用了**同一个 tab kind**：
+better-sidebar 会把收到的每个 tab 描述符**镜像注册进同一个原生注册表**（`sidebarRightTabs.register`，band `extension`），
+而该注册表对*同 band 同 kind 的二次注册直接抛错*（源码注释：*"Everything else colliding on a kind throws"*）——
+后注册的那一方就此失败，而且**在界面上不可见**。修法：
+
+1. better-sidebar 承载面改用独立类型 **`sidecard-ask:card:workbench`**，两端 kind 不再重叠；
+2. `contract-test` **模拟该注册表的"重复 kind 抛错"规则**，此类冲突今后会直接让测试失败；
+3. 设置页自检行改为**同时报告两个适配器**的状态（此前原生适配器的错误压根看不到）。
+
 ---
 
 ## 六、未知 DSH API：占位接口与替换方式
@@ -353,7 +382,7 @@ node test/smoke-test.mjs      # 端到端：流式/截断/取消/持久化/失
 ```
 
 三个脚本都以 `process.exitCode` 反映结果，失败会列出具体条目；测试会把 `DSH_HOME` 指向临时目录，不会污染真实配置。
-当前规模：verify 53 项 + contract 105 项 + smoke 118 项 = **276 项全部通过**。
+当前规模：verify 53 项 + contract 118 项 + smoke 118 项 = **289 项全部通过**。
 
 ### 10.2 版本能力探测（§7 矩阵的来源）
 
@@ -471,5 +500,5 @@ npm deprecate "<旧包名>@*" "Renamed to <新包名> - <原因>"       # 旧名
    本仓库当前文件比运行中的宿主模块新（1.0.1 的「归档父会话不再一刀切 / 持久化 chunk 流式源 / 能力门控 / 路由降级」），
    **重启后**这些改动才生效；`/state` 的 `version` 与 `capabilities.parent` 字段可以直接确认是否已加载新版。
 6. **除 0.1.7-rc.2 外，其余版本只做了产物层验证**：见 §7.3——插件在 0.1.2-rc.1 … 0.1.6-alpha.2 上安装启动过**没有**得到验证，
-   验证到的是"这些版本缺哪些 API + 插件对每个缺失都有分支"，分支本身由 276 项自测覆盖。
+   验证到的是"这些版本缺哪些 API + 插件对每个缺失都有分支"，分支本身由 289 项自测覆盖。
 

package/client.js

@@ -42,9 +42,27 @@ window.__ModuleLoader__.load({
       settings: 'sidecard-ask-settings',
       sessionProbe: 'sidecard-ask-session-probe',
     }
-    /** Side-card tab type registered in a side-card host. */
+    /**
+     * Tab kind served by the DSH native right rail (also its implementation id).
+     */
     const CARD_KIND = 'sidecard-ask:card'
 
+    /**
+     * Tab type served by the dsh-better-sidebar plugin.
+     *
+     * It MUST differ from {@link CARD_KIND}: better-sidebar mirrors every tab
+     * descriptor it is given into the SAME native registry
+     * (`sidebarRightTabs.register`, id `dsh-better-sidebar:<id>`, band
+     * `extension`), and that registry THROWS when a kind already has a
+     * registration in the same band ("Everything else colliding on a kind
+     * throws"). Sharing one kind made the two adapters collide: whichever
+     * registered second failed — observed live on 0.2.0-rc.1, where the pane
+     * listed `dsh-better-sidebar:sidecard-ask:card` and no native
+     * `sidecard-ask:card` at all. `test/contract-test.mjs` now emulates that
+     * registry rule so the collision cannot come back.
+     */
+    const CARD_KIND_BETTER = 'sidecard-ask:card:workbench'
+
     /**
      * Client-side mirror of the Host `DEFAULT_CONFIG`. The client needs values
      * before `/state` answers (first paint) and when the Host is unreachable;
@@ -170,6 +188,7 @@ window.__ModuleLoader__.load({
         diagSidecard: '侧边卡片插件（dsh-better-sidebar）',
         diagFeatures: '项能力',
         diagNoTabMeta: '版本过旧：没有 tabMeta 能力，已跳过它改用其它承载面',
+        diagNativeKind: 'kind',
         toolGuardUnavailable: '该 provider 不支持工具白名单，本次作答继承了当前会话的工具',
         toolGuardInherited: '按配置继承当前会话的工具',
         diagUnreachable: '宿主接口不可达：{msg}',
@@ -272,6 +291,7 @@ window.__ModuleLoader__.load({
         diagSidecard: 'Side-card plugin (dsh-better-sidebar)',
         diagFeatures: 'capabilities',
         diagNoTabMeta: 'too old: no tabMeta capability, skipped in favour of another surface',
+        diagNativeKind: 'kind',
         toolGuardUnavailable: 'This provider supports no tool allow-list, so the answer inherited the session tools',
         toolGuardInherited: 'Inherits the session tools, as configured',
         diagUnreachable: 'Host API unreachable: {msg}',
@@ -889,20 +909,20 @@ window.__ModuleLoader__.load({
         },
         open(card) {
           if (!this.available() || typeof this.service.openTab !== 'function') return false
-          const tabId = `${CARD_KIND}:${card.id}`
+          const tabId = `${CARD_KIND_BETTER}:${card.id}`
           // `meta` is the transport for the card id (feature `tabMeta`), and
           // `dedupeKey` on our descriptor is what collapses repeat opens onto
           // the same tab; both are part of the stable consumer contract
           // (`lib/types/client/service.d.ts`, unchanged across 0.22.0 → 0.22.1).
           this.service.openTab(
-            { type: CARD_KIND, id: tabId, title: card.question.slice(0, 32), meta: { cardId: card.id } },
+            { type: CARD_KIND_BETTER, id: tabId, title: card.question.slice(0, 32), meta: { cardId: card.id } },
             store.state.sessionId === null ? undefined : { sessionId: store.state.sessionId },
           )
           return true
         },
         close(card) {
           if (!this.available() || typeof this.service.closeTab !== 'function') return false
-          this.service.closeTab(`${CARD_KIND}:${card.id}`)
+          this.service.closeTab(`${CARD_KIND_BETTER}:${card.id}`)
           return true
         },
       },
@@ -1554,7 +1574,7 @@ window.__ModuleLoader__.load({
                 h('div', { key: 'slots' }, `${t('diagSlots')}: ${Object.entries(state.slots).map(([k, v]) => `${k}=${v}`).join(' · ')}`),
                 // The side-card plugin's own report: which version is loaded and
                 // whether it advertises the capability this adapter needs.
-                h('div', { key: 'sidecard' }, `${t('diagSidecard')}: ${describeSidecardAdapter()}`),
+                h('div', { key: 'sidecard' }, `${t('diagSidecard')}: ${describeSurfaces()}`),
                 h('div', { key: 'path' }, `${t('settingsTitle')} → ${diag.provenance?.persistedPath ?? ''}`),
               ])
           : null)
@@ -1582,10 +1602,22 @@ window.__ModuleLoader__.load({
     }
 
     /**
-     * One line describing how the optional side-card plugin resolved, for the
-     * settings page's capability check. Reported from the plugin's OWN
-     * `version`/`features` fields, so the answer is what is running, not what
-     * package.json claims.
+     * One line describing how each optional side-card surface resolved, for the
+     * settings page's capability check. Both adapters are reported, because a
+     * failing NATIVE registration was previously invisible here (it only showed
+     * up as a missing entry in the pane's own slot inventory).
+     * @returns {string} human-readable state.
+     */
+    function describeSurfaces() {
+      const native = surfaces.native.available()
+        ? `${t('yes')} · ${t('diagNativeKind')}=${CARD_KIND}`
+        : `${t('no')}${surfaces.native.error === null ? '' : `（${surfaces.native.error}）`}`
+      return `${t('surfaceNative')}: ${native} ／ ${t('surfaceBetter')}: ${describeSidecardAdapter()}`
+    }
+
+    /**
+     * The optional side-card plugin's own report: which version is loaded and
+     * whether it advertises the capability this adapter needs.
      * @returns {string} human-readable state.
      */
     function describeSidecardAdapter() {
@@ -2245,7 +2277,11 @@ window.__ModuleLoader__.load({
                 return
               }
               const off = service.registerTab({
-                id: CARD_KIND,
+                // A DIFFERENT type from the native rail's: better-sidebar mirrors
+                // this descriptor into `sidebarRightTabs` as an `extension`-band
+                // registration, and a second registration of the same kind in
+                // that band throws (see CARD_KIND_BETTER).
+                id: CARD_KIND_BETTER,
                 title: () => t('answerTitle'),
                 description: () => t('settingsDesc'),
                 order: 120,
@@ -2337,13 +2373,14 @@ window.__ModuleLoader__.load({
        * `window` and exercises these without a browser.
        */
       api: Object.freeze({
-        version: '1.2.0',
+        version: '1.2.1',
         /** Read-only state accessor for diagnostics and the test harness. */
         snapshot: () => store.state,
         pure: Object.freeze({
           CLIENT_DEFAULTS,
           IDS,
           CARD_KIND,
+          CARD_KIND_BETTER,
           parseSseBlock,
           truncateSelection,
           parseShortcut,

package/index.js

@@ -42,7 +42,7 @@ export const name = 'dsh-sidecard-ask'
 export const inject = ['webServer']
 
 /** Version of this plugin (kept in step with package.json by test/verify.mjs). */
-export const PLUGIN_VERSION = '1.2.0'
+export const PLUGIN_VERSION = '1.2.1'
 
 /** Route prefix of the plugin's own API. */
 export const ROUTE_PREFIX = '/sidecard-ask/api'

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-sidecard-ask",
-  "version": "1.2.0",
+  "version": "1.2.1",
   "description": "DSH 划词追问（侧边卡片作答）：在聊天区与任务区选中文本就地追问，答案由独立子代理在侧边卡片里流式呈现，也可选择落回主对话。",
   "type": "module",
   "main": "index.js",