@omdp/dsh-connector 0.3.4 → 0.3.6

package/README.md

@@ -1,14 +1,14 @@
 # @omdp/dsh-connector
 
-**MCP 服务器 + 用户 Skills + 魔搭市场浏览三合一设置页**（`v0.3.4`）。适合需要在 DSH 里频繁增删改 MCP server / skills、又不想手改 `cordis.patch.yml` 的用户。
+**MCP 服务器 + 用户 Skills + 魔搭市场浏览三合一设置页**（`v0.3.6`）。适合需要在 DSH 里频繁增删改 MCP server / skills、又不想手改 `cordis.patch.yml` 的用户。
 
 ## Requirements
 
 - DeepSeek Harness 带 `web` profile GUI（`npx @deepseek-ai/dsh web`）
 - Node.js `^22.19` 或 `>=24`
 - `@deepseek-ai/schemastery` `3.18.1` / `3.18.2` / `3.18.4`（peer 枚举，供工具过滤的 Config 声明用；无则过滤静默全放行）
-- 已实测 DSH **0.1.5-rc.1**（源码级核查，2026-09-10）、**0.1.5-rc.2** 与 **0.1.6-alpha.1**（源码级 + 运行时冒烟：`/connector/api/*` 正常服务、设置页渲染，2026-09-15，见 `docs/plugin-compatibility.md`）、**0.1.7-rc.1**（0.3.3 适配：修掉 import 期崩溃 + 迁移到 settings 托管配置，2026-09-24）
-- **`@deepseek-ai/dsh` peer 声明 `0.1.7-rc.1`**（0.3.4 起，逐版本枚举；规范见 `AGENTS.md` 规范 3）
+- 已实测 DSH **0.1.5-rc.1**（源码级核查，2026-09-10）、**0.1.5-rc.2** 与 **0.1.6-alpha.1**（源码级 + 运行时冒烟：`/connector/api/*` 正常服务、设置页渲染，2026-09-15，见 `docs/plugin-compatibility.md`）、**0.1.7-rc.1**（0.3.3 适配：修掉 import 期崩溃 + 迁移到 settings 托管配置，2026-09-24）、**0.1.7-rc.2**（0.3.5：rc.1→rc.2 tarball 逐文件 diff 零 API 变化 + rc.2 真机回归实测通过；0.3.6 再修 CRLF 解析 + 取当前 profile 补丁路径，2026-09-25）
+- **`@deepseek-ai/dsh` peer 声明 `0.1.7-rc.1 || 0.1.7-rc.2`**（0.3.4 起逐版本枚举，0.3.5 追加 rc.2；规范见 `AGENTS.md` 规范 3）
 
 ## 兼容性门禁（0.3.4 起声明）
 
@@ -16,7 +16,7 @@
 
 ```json
 "peerDependencies": {
-  "@deepseek-ai/dsh": "0.1.7-rc.1"
+  "@deepseek-ai/dsh": "0.1.7-rc.1 || 0.1.7-rc.2"
 }
 ```
 
@@ -24,11 +24,12 @@
 
 | 运行中的 DSH | 行为 |
 |---|---|
-| **`0.1.7-rc.1`**（本次实测） | ✅ `evaluatePluginCompatibility()` 返回 `undefined` ⇒ 正常加载，与 0.3.3 行为一致 |
+| **`0.1.7-rc.1`**（实测，2026-09-24） | ✅ `evaluatePluginCompatibility()` 返回 `undefined` ⇒ 正常加载，与 0.3.3 行为一致 |
+| **`0.1.7-rc.2`**（实测，2026-09-25） | ✅ 同上：rc.1→rc.2 tarball 逐文件 diff 显示 shell/settings/credentials 的 `lib/` **零变化**（仅 README/package.json 版本号）、app-boot 的变更全部与插件无关；另在 scratch profile（`dsh@0.1.7-rc.2` + 本插件 0.3.4）真机回归 `/connector/api/mcp/filters` → 200 |
 | **`0.1.7` 及更新的、未实测版本** | ⛔ 门禁拦下：启动时**整个 bundle 被跳过**（stderr 打 `skipping profile bundle "@omdp/dsh-connector"`），安装时该行被置灰 |
 | **`≤0.1.6` 及 `0.1.7-alpha.x`** | ➖ **不受影响**：那些版本里**根本没有这个门禁**（经解包 npm tarball 逐版核对，`0.1.5-rc.2`/`0.1.5-rc.3`/`0.1.6-alpha.1`/`0.1.6-alpha.2`/`0.1.7-alpha.1`/`0.1.7-alpha.2` 的 `dsh-app-boot` 里 `evaluatePluginCompatibility` 出现 **0 次**，只有 `0.1.7-rc.1` 出现 4 次），旧运行时读到这条 peer 只是「不认识的声明」，照常加载 |
 
-**为什么故意只写 `0.1.7-rc.1`（而不是写老版本）**：老版本运行时压根不执行这个检查，写进去纯属装饰、无法被验证；而**未实测的新版本必须被拦住**——这正是 `AGENTS.md` 规范 3 的要求（`peerDependencies` 只精确枚举实测过的版本，禁止开放范围，未核查的版本宁可报 unmet peer 也不得预先放行）。门禁被触发时是**优雅跳过**（插件不加载、DSH 照常启动），符合本插件「硬保证 = DSH 不会因插件崩溃」的抗崩溃设计。
+**为什么只写精确枚举（而不是开放范围，也不写未实测的老版本）**：`≤0.1.6` 的运行时压根不执行这个检查，写进去纯属装饰、无法被验证；而**未实测的新版本必须被拦住**——这正是 `AGENTS.md` 规范 3 的要求（`peerDependencies` 只精确枚举实测过的版本，禁止开放范围，未核查的版本宁可报 unmet peer 也不得预先放行）。门禁被触发时是**优雅跳过**（插件不加载、DSH 照常启动），符合本插件「硬保证 = DSH 不会因插件崩溃」的抗崩溃设计。
 
 > 升级 DSH 后若设置页突然看不到 Connector 标签，先看启动日志有没有 `skipping profile bundle`——那就是门禁在提醒你「该插件尚未针对这个 DSH 版本做兼容核查」，核查通过后把该版本追加进上面的枚举并同步更新 `docs/plugin-compatibility.md`。
 
@@ -321,6 +322,32 @@ skills/mcp 列表与详情、证书/Hosted 标识、安装命令、记录来源
 
 ## 变更记录
 
+- **0.3.6**（2026-09-25）：**修两个真机 bug —— CRLF 补丁文件解析 + 编辑错 profile 的补丁文件**。
+  1. **CRLF 兼容**：`parseMcpServers()` 的键值正则 `^\s+(\w+):\s*(.*)$` 在 CRLF 文件上必然失配
+     （JS 里 `.*` 不匹配 `\r`、`$` 也不匹配 `\r` 之前的位置），于是每行的
+     `transport`/`serverName`/`command`/`url` 全部落进 `preserve` 桶：API 返回
+     `transport:"" serverName:""`，UI 的「工具过滤」区因 `if (!props.serverName) return null`
+     **静默消失**、transport 徽标退化成 `stdio`。修复：解析前用 `toLf()` 归一化换行（连同
+     `stripQuotes`/`stripListScalar` 的 `$` 正则一起恢复），写回时也统一成 LF。
+     验证：同一份配置文本按 LF / CRLF 两种形态跑解析，修复前 CRLF 五项断言全红、修复后 14/14 通过。
+  2. **profile 补丁路径**：`patchPath()` 曾硬编码 `profiles/web/cordis.patch.yml`，在桌面端
+     （跑 `profiles/desktop`）会去改 **另一个 profile** 的文件——写下去对自己的 MCP 配置毫无影响，
+     UI 提示还把错误路径显示给用户。修复：0.1.7+ 从 `ctx.get('profileContext').patchPath` 取当前
+     profile 的真实路径（`dsh-app-boot` 的 `ProfileContext` 契约），无该服务（0.1.5/0.1.6）时保留
+     历史路径；`GET /api/mcp` 一并回传 `patchPath`，设置页提示改成显示真实路径。
+  触发场景：用户手改（或编辑器保存）`cordis.patch.yml` 使其变成 CRLF，桌面端点开
+  「Connector → MCP 服务器」就只剩 4 张光秃秃的卡片。详见
+  `notes/2026-09-25/debug/connector-crlf-patch-parser-and-profile-path.md`。
+- **0.3.5**（2026-09-25）：**追加 DSH `0.1.7-rc.2` 支持**（peer 枚举 `0.1.7-rc.1 || 0.1.7-rc.2`，代码零改动）。
+  背景：DSH 桌面版（DeepSeek Harness desktop）0.1.7-rc.2 上线后，门禁把只声明 `0.1.7-rc.1` 的 0.3.4 拦下
+  （`skipping profile bundle`，插件列表显示「异常」）。核查：rc.1→rc.2 npm tarball 逐文件 diff——
+  `dsh-shell`/`dsh-settings`/`dsh-credentials` 的 `lib/` **逐字节零变化**（仅 README/package.json），
+  `dsh-llm` 仅新增内容类型与错误码（不改既有签名），`dsh-app-boot` 的变更全部与插件无关
+  （`skippedBundles` 诊断重构、`generateConfigSchema` 内部签名）；再用 rc.2 的
+  `evaluatePluginCompatibility()` 执行新声明 → `undefined`（放行）。回归实测：scratch profile
+  （`dsh@0.1.7-rc.2` + 插件 0.3.4 + 精确版本豁免）真机启动 → `/connector/api/mcp/filters` 200。
+  **教训**：`0.1.7-rc.1` 这类 rc 枚举在 rc.2 发布当天就会过时——rc 系列每个新 rc 都要重新核查并追加。
+
 - **0.3.4**（2026-09-24）：**声明 DSH 版本支持 + 修复工具过滤被静默清空**。
   1. **`@deepseek-ai/dsh` peer 声明 `0.1.7-rc.1`**（逐版本枚举，语义见上方「## 兼容性门禁」）。
      用 DSH 真实的 `evaluatePluginCompatibility()` 逐一验证：`0.1.7-rc.1` → 正常加载；

package/client.js

@@ -412,7 +412,10 @@ window.__ModuleLoader__.load({
         ),
         err ? createElement('div', { className: 'pm_err' }, err) : null,
         createElement('div', { className: 'pm_meta' },
-          props.mcp.exists ? '编辑 profiles/web/cordis.patch.yml 中的 mcp-* 块。保存后重启 dsh 生效。Edit the mcp-* block in cordis.patch.yml; restart dsh to apply.' : '未找到 cordis.patch.yml。cordis.patch.yml not found.'),
+          props.mcp.exists
+            // host 回传真实路径（0.1.7+ 的 profileContext.patchPath）；老 host 没这个字段时退回笼统说法。
+            ? '编辑 ' + (props.mcp.patchPath || '本 profile 的 cordis.patch.yml') + ' 中的 mcp-* 块。保存后重启 dsh 生效。Edit the mcp-* block in cordis.patch.yml; restart dsh to apply.'
+            : '未找到 cordis.patch.yml。cordis.patch.yml not found.'),
         rows,
         editing === null
           ? createElement('button', { className: 'pm_btn pm_btn ghost', onClick: function () { setEditing({ id: 'mcp-new', customId: '', name: '', transport: 'stdio', serverName: '', url: '', command: '', args: '' }) } }, '＋ 添加 MCP 服务器 Add MCP Server')

package/index.js

@@ -95,6 +95,9 @@ let _legacyScope = null   // rc.x: settings.register(...) 返回的 scope
 let _legacySvc = null     // rc.x: settings 服务（读用 get，写用 update）
 let _settingsSvc = null   // 0.1.7+: ctx.settings（迁移写回用，见 ensureLegacyFiltersMigration）
 let _migrationAttempted = false
+// 0.1.7+ 的 profileContext.patchPath：本进程真正在用的 profile 补丁文件。
+// 0.1.5/0.1.6 没有这个服务，回退到历史路径（见 patchPath）。
+let _profilePatchPath = null
 function readToolFilters() {
   try {
     // 优先 0.1.7+ 的活 Ref（.get() 每次返回最新值）
@@ -191,8 +194,13 @@ function resolveHome() {
     : join(homedir(), '.dsh')
 }
 
+// 本进程要编辑的 profile 补丁文件。
+// 0.1.7 起 dsh 通过 profileContext 服务提供当前 profile 的绝对路径（desktop / web /
+// 自定义 profile 都不同），必须用它；否则桌面端会去改 profiles/web 的文件 —— 写下去
+// 对桌面端自己的 MCP 配置毫无影响，还会误导 UI 提示（0.3.5 及以前的 bug）。
+// 0.1.5/0.1.6 没有 profileContext：退回历史硬编码路径保持老行为。
 function patchPath() {
-  return join(resolveHome(), 'profiles', 'web', 'cordis.patch.yml')
+  return _profilePatchPath ?? join(resolveHome(), 'profiles', 'web', 'cordis.patch.yml')
 }
 
 function skillsRoot() {
@@ -235,6 +243,13 @@ function stripQuotes(v) {
   return v
 }
 
+// Normalize any line ending (CRLF / lone CR) to LF. Every line-based parser here
+// anchors regexes with `$`, which in JS does not match before a `\r`, so a CRLF
+// profile patch silently loses fields instead of failing loudly.
+function toLf(text) {
+  return String(text).replace(/\r\n?/g, '\n')
+}
+
 // Extract the bare scalar from a YAML block-sequence item line like
 // `  - '--transport'` -> `--transport`. Handles quoted and unquoted values.
 function stripListScalar(line) {
@@ -265,7 +280,12 @@ const SKIP_KEYS = new Set(['name', 'config'])
  * (env blocks, `!!js` lines, any other keys) so a rewrite never drops them.
  */
 function parseMcpServers(blockText) {
-  const lines = blockText.split('\n')
+  // 先统一换行：CRLF 文件里每行都以 `\r` 结尾，而下面的键值正则
+  // `^\s+(\w+):\s*(.*)$` 中 `.*` 不匹配 `\r`、`$` 也不匹配 `\r` 之前的位置，
+  // 于是一整行的 transport/serverName/command/url 全部匹配失败、统统落进
+  // `preserve` —— serverName 空 → UI 工具过滤区静默消失（`if (!props.serverName) return null`）。
+  // 归一化后与 LF 文件完全同构，顺带让 stripQuotes/stripListScalar 的 `$` 正则重新生效。
+  const lines = toLf(blockText).split('\n')
   const servers = []
   let cur = null
   let curIndent = 0
@@ -1024,6 +1044,18 @@ async function readBody(req) {
 }
 
 export function apply(ctx, config) {
+  // ── 绑定本进程的 profile 补丁文件（0.1.7+）──
+  // profileContext 由 dsh 启动器提供，patchPath 就是当前 profile 的
+  // cordis.patch.yml 绝对路径；没有它就只能沿用历史硬编码路径。
+  try {
+    const profile = ctx.get('profileContext')
+    if (profile && typeof profile.patchPath === 'string' && profile.patchPath.length > 0) {
+      _profilePatchPath = profile.patchPath
+      console.info('[dsh-connector] profile patch file:', profile.name ? `${profile.name} -> ${profile.patchPath}` : profile.patchPath)
+      ctx.effect(() => () => { _profilePatchPath = null }, 'connector: release profile patch path')
+    }
+  } catch {}
+
   // ── 工具过滤三件套之 (0)：判定配置后端 + 快照缓存 ──
   // 0.1.7+：config.toolFilters 是 volatile Ref（.get() 每次返回最新值，编辑热生效）。
   // rc.x：无 volatile，退回老的 settings.register/get/update（0.3.0 的原始实现）。
@@ -1106,7 +1138,7 @@ export function apply(ctx, config) {
         }
         const found = findMcpBlock(text)
         const servers = found ? parseMcpServers(found.blockText) : []
-        return json(res, 200, { servers, exists: true, version: '0.1.1' })
+        return json(res, 200, { servers, exists: true, version: '0.1.2', patchPath: patchPath() })
       }
 
       // POST /api/mcp — replace server entries. Preserve env/!!js from the
@@ -1141,7 +1173,7 @@ export function apply(ctx, config) {
           const prev = existing.get(s.id)
           return prev ? { ...s, preserve: s.preserve || prev.preserve } : s
         })
-        const next = buildPatch(text, servers)
+        const next = toLf(buildPatch(text, servers))
         // Validate the entire resulting patch parses as YAML before writing.
         // cordis.patch.yml uses `!!js` tags; the parser tolerates them via
         // silent log level (values stay raw, nothing printed), so a successful

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@omdp/dsh-connector",
-  "version": "0.3.4",
+  "version": "0.3.6",
   "description": "Unified DeepSeek Harness connector: edit MCP servers (cordis.patch.yml), user skills (~/.dsh/skills) and browse the ModelScope MCP/Skills marketplaces from one Web UI settings page.",
   "type": "module",
   "main": "index.js",
@@ -46,7 +46,7 @@
     "jsdom": "^24.1.3"
   },
   "peerDependencies": {
-    "@deepseek-ai/dsh": "0.1.7-rc.1",
+    "@deepseek-ai/dsh": "0.1.7-rc.1 || 0.1.7-rc.2",
     "@deepseek-ai/schemastery": "3.18.1 || 3.18.2 || 3.18.4"
   }
 }