bug: `bili plugin install codex` 无法修复已存在的畸形 `[mcp_servers.bili]` 块（"already installed" 短路只比对 BILI_MCP_PROXY 行）

[ranxianglei]

来源: #636 (#636) 分析用户「升级到 0.1.95 后 codex 仍报 mcp_servers.bili.args expected a sequence」时发现

现象

用户 ~/.codex/config.toml 里存在一个 [mcp_servers.bili] 块，其中 args 被写成了字符串（整个列表被一对引号包住），例如：

[mcp_servers.bili]
command = "node"
args = '["C:\Users\<you>\AppData\Roaming\npm\node_modules\billion-context\dist\mcp.js"]'
env = { BILI_MCP_PROXY = "http://127.0.0.1:8787" }

这是合法 TOML（能 parse），但 codex 的 schema 要求 args 是数组（sequence），于是启动时报：

Error loading config.toml: invalid type: string "[...]", expected a sequence in `mcp_servers.bili.args`

问题（bug）

src/plugin-install.ts 的 codexInstall()（v0.1.95）在块已存在时，用只比对 BILI_MCP_PROXY 那一行来判断是否「已安装」：

if (block.includes(`BILI_MCP_PROXY = ${JSON.stringify(proxyOriginForInstall())}`)) return `codex: already installed (${file})`;

只要 env 行匹配当前 origin，就直接返回 already installed，完全不校验 / 修复 args（以及 command）。因此：

• 升级 bili 后重新 bili plugin install codex → 命中 already installed → 畸形 args 原样保留 → codex 依旧报错。
• 只有 bili plugin remove codex（纯文本切片删除，不看内容）+ 重新 install 才能修好。

也就是说，「升级 / 重装」这条用户最自然的自愈路径，对这种畸形块是无效的。

复现（v0.1.95 逻辑，零依赖）

1. 往 config.toml 写入上面那个畸形块（env 行 = 默认 origin，args = 字符串）。
2. 运行 bili plugin install codex → 输出 codex: already installed (...)，args 行未变（仍是字符串）。
3. 运行 bili plugin remove codex，再 bili plugin install codex → args 变为合法数组 ["…"]，codex 正常。

已用 v0.1.95 的 codexInstall / codexRemove 原文 + 少量 stub 实际跑过，确认第 2 步不修复、第 3 步修复。

影响

• 用户被卡住：升级 / 重装都自愈不了；且报错是 codex 的 serde 类型错误，完全没指向「config.toml 里 args 写成了字符串」，排查困难（疑似字符串错误 #636 就卡在这里）。
• plugin install 作为「安装 / 刷新」命令，在块已存在但畸形时静默 no-op，违背直觉。

建议修复

把「已安装」判定从「只比对 origin 行」改为「整块与规范块比对」：只有当现有块与 codexBlock() 生成的规范块一致时才返回 already installed，否则一律用规范块替换（自愈 args / command / 路径）。最简实现：块存在时直接以 codexBlock() 覆盖（内容确定、天然幂等），仅在字节一致时保留 already installed 文案。

顺带：codexInstall 命中畸形块时，可打印一句更明确的提示（如「检测到 [mcp_servers.bili].args 不是数组，已重写」），而不是让用户去啃 codex 的 serde 报错。

[ranxianglei]

继续

[ranxianglei]

🤖 Powered by ework · qwen3.8-27b

[bot] 🏷 已修复，PR: #652

验证：复现了 issue 描述——codexInstall() 的 already installed 短路只比对 BILI_MCP_PROXY 那一行（src/plugin-install.ts:378），只要 env 行命中当前 origin 就直接返回，完全不校验 args/command。用 v0.1.95 逻辑 + 畸形块（args 写成字符串、env 行 = 默认 origin）实测：旧代码返回 already installed、args 原样保留；只有 remove + install 才修好。判定成立。

修复（按 issue 建议的「最简实现」）：

• 「已安装」判定从「只比对 origin 行」改为「整块与 codexBlock() 规范块比对」（尾部空白归一化，兼容块在文件末尾 vs 块后紧跟其他 table 的换行差异）。
• 只有字节一致才返回 already installed；任何漂移（args / command / origin / 路径）一律用规范块原地重写，自愈畸形块。重写时按边界 splice，不扰动前后其他 table。
• 文案从 refreshed proxy origin 改为 refreshed [mcp_servers.bili]（因为现在刷新原因不止 origin）。

测试：

• tests/plugin-agent.test.ts 新增回归用例：env 行命中 origin 但 args 是字符串的块 → 第一次 install 必须 refreshed（args 变回合法数组），第二次 install 幂等返回 already installed。
• 用构建产物端到端验证：畸形块 → 首次 install 重写 args 为数组 + 真实 node 路径 → 二次 install 幂等。

Pre-flight（基于最新 master，v0.1.95 / kernel 0.0.59）：

• npm run typecheck ✅
• npm test：1239 pass / 1 fail。唯一失败 resolveClientCommand: codex/claude resolve to themselves 是干净 master 上就存在的环境相关失败（该测试假设 /usr/bin 下没有 codex/claude；本沙箱装了，resolveOnPath 返回全路径导致断言不符）——与本次改动无关。
• npm run build ✅

顺带说明：issue 里「命中畸形块时打印更明确提示」那条，当前实现统一走 refreshed [mcp_servers.bili] 文案（不解析具体是哪个字段畸形，避免脆弱的 TOML 解析）。如果你希望进一步精确到「args 不是数组，已重写」，我可以在 PR 里补一个轻量的字段级检测，等你拍板。