[Bug]0.1.6-aplha.1，MCP 工具重名会让会话恢复失败：failOnStartupError 把可容纳的错误升级成了致命错误

[Enderman112]

概要

挂载实验性 Browser Use 提供方后，一次 MCP 工具注册冲突会升级成致命的会话故障：所有新会话都无法启动/恢复。MCP 客户端自身对这类错误的默认处理是容纳（registrationFailure: "contain"），但启动同步路径在设置了 failOnStartupError 时会把它替换成 "throw"，于是一个工具重名就打死整个会话，而不是只让那个 MCP 服务器失效。

环境

项	值
dsh	0.1.6-alpha.1
@deepseek-ai/dsh-mcp-client	0.1.6-alpha.1
@deepseek-ai/dsh-tools	0.1.6-alpha.1
@playwright/mcp	0.0.80
系统	macOS 27.0（build 26A428），arm64
Node	v26.8.2
界面	dsh web

配置

按提供方 README 的最小配置挂载 Browser Use 核心服务 + Playwright MCP 提供方：

- name: '@deepseek-ai/dsh-browser-use'
- name: '@deepseek-ai/dsh-experimental-browser-use-playwright-mcp'
  config:
    mode: launch
    headless: true

（作为 profile 的 cordis.patch.yml 里的顶层 insert: 条目。组合本身没有问题——dsh web --dump-config 能看到两行都进去了，config 块也正确挂在了后端那一条下面。）

实际行为

服务启动后，第一次 MCP 同步抛出工具重复注册错误：

mcp-client(playwright-mcp): connection attempt failed: Error: tool "mcp__playwright-mcp__browser_close" is already registered (for a per-agent variant, register through that agent's `agent.ctx` instead)
    at NamedEntries.duplicateError (<dsh>/node_modules/@deepseek-ai/dsh-tools/lib/index.js:2634:59)
    at NamedEntries.insert (<dsh>/node_modules/@deepseek-ai/dsh-scope/lib/index.js:29:34)
    at layers.effect.label (<dsh>/node_modules/@deepseek-ai/dsh-tools/lib/index.js:2884:62)
    at ScopedLayers.<anonymous> (<dsh>/node_modules/@deepseek-ai/dsh-scope/lib/index.js:206:12)
    at ScopedLayers.effect (<dsh>/node_modules/@deepseek-ai/dsh-scope/lib/index.js:192:14)
    at Proxy.register (<dsh>/node_modules/@deepseek-ai/dsh-tools/lib/index.js:2884:22)
    at Object.apply (<dsh>/node_modules/@deepseek-ai/cordis/lib/index.js:120:36)
    at Proxy.wrappedRegister (<profile>/node_modules/dsh-context/lib/index.js:218:29)
    at Object.apply (<dsh>/node_modules/@deepseek-ai/cordis/lib/index.js:120:36)
    at syncTools (<profile>/node_modules/@deepseek-ai/dsh-mcp-client/lib/index.js:153:91)
    at process.processTicksAndRejections (node:internal/process/task_queues:104:5)
    at async <profile>/node_modules/@deepseek-ai/dsh-mcp-client/lib/index.js:521:16

紧接着会话直接失败，Web 界面完全打不开会话：

resume failed for session "<session-id>":
Error: mcp-client(playwright-mcp): initial connection or tool synchronization failed

为什么一个重名会变成致命错误

@deepseek-ai/dsh-mcp-client 其实已经把这类失败建模成可恢复的。syncTools 里：

try {
    for (const [publicName, definition] of definitions) disposers.set(publicName, ctx.tools.register(definition));
} catch (error) {
    for (const dispose of disposers.values()) dispose();
    ctx.logger.error(`mcp-client(${opts.serverName}): tool registration failed, no tools registered: ${String(error)}`);
    if (opts.registrationFailure === "throw") throw error;
    return new Map();
}

而该选项是这样选出来的：

const opts = { registrationFailure: "contain", serverName: config.serverName, toolCallTimeoutMs: config.toolCallTimeoutMs };
const startupOpts = config.failOnStartupError ? { ...opts, registrationFailure: "throw" } : opts;

启动同步走的是 startupOpts（enqueueSync(generation, startupOpts)），所以在设了 failOnStartupError 的情况下，这个"容器"被绕过，重复注册错误直接从连接建立过程抛出去 → 会话恢复失败。

期望行为

与可选 MCP 提供方发生的工具名冲突，应当让该提供方降级（其工具不可用 + 记录清晰错误），而不是让会话根本无法启动。具体可选：

• 启动同步期间，工具名冲突仍适用默认的 registrationFailure: "contain"；或
• failOnStartupError 只覆盖连接失败，不覆盖工具注册冲突；或
• 冲突以确定性方式解决（后注册者胜出 / 处置先前的注册），而不是抛出。

待确认的问题

• 我无法确定同一个工具名为什么会被注册两次。enqueueSync 明确串行化了所有同步，以防两次竞态的 dispose/register 交换，所以这个重复看起来不像是两次竞争同步导致的。"重复"本身的成因可能与上面这个"致命性"问题是两回事。
• dsh-context@0.52.2 通过其 tools.register 归属包装器出现在调用栈里。读过 createToolAttribution 的实现后，它只做包装并委托一次（带 WeakSet 防重入），所以很可能不是重复的来源。这里仅因它位于调用路径上、排查时可能相关才提一句。
• 相关：v0.1.6-alpha.1 的更新日志提到让可选插件的启动失败不影响其它插件（"可选插件启动失败不影响其他可用插件，必需插件失败时退出"）。这里看起来是一个仍然会升级成"必需插件失败"式退出的情形。

复现步骤

1. 安装 @deepseek-ai/dsh-browser-use + @deepseek-ai/dsh-experimental-browser-use-playwright-mcp（另需 playwright install chromium 装 Chromium）。
2. 按上面的配置挂载两者，重启 dsh web。
3. 打开 Web 界面，新建一个会话。

观察到的结果：服务端记录一次重复注册错误；会话恢复失败并报上述信息。

卸载该提供方（移除两个包并删掉挂载）后会话恢复正常启动——我就是这样确认相关性的。