PR #42 · Claude Code Provider 审查报告

• 契约 3 项(getCapabilities / getControlCommands / createRunner)+ id/kind/transport —— KEEP createRunner 干净(normalize → 拒 load replay → 取 providerSessionId → 委托)。
• 另挂 8 个非契约 public 方法(getMetadata/getAvailability/discoverSlashCommands/discoverAgents/discoverModels/readMcpDiagnostic/discoverNativeSessions/importNativeSession)。

• kill 升级(stdin.end → SIGTERM → SIGKILL 打进程组;Windows taskkill;ESRCH 吞;unref);早退窗口探测;写背压经 'error' 监听;CVE-aware 的 Windows .cmd 处理 —— 都是真功夫。
• M5:argv 硬编码 --permission-mode bypassPermissions(386 行)。起步过门有据,但到 set_permission_mode 落地前那段窗口无沙箱。

• M2 双发:partial 发 message.delta + 末尾又发带全文的 message.completed,都不带 itemId 配对 → 消费者整条重复。
• M3:usage 只读 input/output_tokens,丢 cache/context/model。
• N3:只读 tool_result.content,丢消息级富结构 tool_use_result(stdout/stderr/interrupted)。
• N5(待核):is_plan / occurredAt 是逆向字段。

// translator.ts 现状:同一段助手文本发两遍,且都没有 itemId 可配对
emit({ type: 'message.delta',     text: '正在分析…' })  // 流式增量,逐段到达
emit({ type: 'message.delta',     text: '…完成' })
emit({ type: 'message.completed', text: '正在分析……完成' }) // ✗ 末尾再带一遍全文
// 消费者把 delta 拼完,又收到 completed 全文 → 整条消息渲染两遍;
// 修法:completed 不重复携带全文,或两边都带 itemId 让消费者可配对去重

• 核保留 状态机 + 竞态/kill/自愈 = 长活双向进程 + 带内协议的必要适配,非 gold-plating。
• 删 M7 buildClaudeHooks hooks 表(多余 + 半接线 + 偏离实测,发 hooks:null)。
• 重做 N1 图像输入 —— capabilities.imageInput 恒 'unknown'、probe 没建 → 任何图像 turn 必抛,~80 行处理代码永跑不到。image 是契约 AgentInput.parts 一等成员(#50)→ 优先补 probe 接通;确认 CLI 不支持再删。
• 重做 M8 默认权限不能 bypass · N2 AskUserQuestion 回填按问题文本+保数组 · M10 空问题别进 input 状态机 · M4 过滤中断哨兵 · ExitPlanMode→永久 bypass 改掉 · parseLine 非 JSON 别杀进程。
• 待核 session-id 缓冲疑似冗余(session_id 第一条就到、早于业务事件)· N6/N7 中断 2s 偏激 / cancelled 误标。

// runner.ts:65 —— 能力探针初始化之后,全仓没有任何代码再更新它
imageInput: 'unknown',

// runner.ts:1436 —— 于是图像轮永远走进 throw,后面 ~80 行图像处理永跑不到
if (capabilities.imageInput !== 'supported') {
  throw unsupportedOperation('Claude stream-json image input is not supported.')
}
// image 是契约 AgentInput.parts 一等成员(#50)→ 修法是把 probe 建起来接通,不是留死路

• 命令定义 + semantic 映射 —— 干净,可留。
• 删 /context /cost 正则刮取人类可读输出 —— 投机。
• N4 待核 整个 slash 面赌「把 /compact 当 user 消息发,CLI 会当命令解释」,实测无确认 —— 若被当聊天文本则整面非功能。

// commands.ts:193-328 —— 把 /context、/cost 的「人类可读输出」用正则刮成数字(~135 行,示意)
readLabeledNumber(rawText, /Context size:\s*([\d,]+)/i)   // ✗ CLI 文案改一个词就全失效
// 命令定义与 semantic 映射保留;这套解析删 —— 这类数据该走结构化口子(usage 事件),不是刮终端文案

• 留 argv 冲突校验(ADAPTER_OWNED_FLAGS / DISALLOWED_FLAG_PATTERNS)—— 防 extraArgs 破协议,有用。
• 提升契约 自造的 ClaudeProviderConfigFieldMetadata + 14 条 configFields 与契约 AgentConfigField 干同一件事(描述字段给 UI 渲染)→ 改填 AgentAvailability.sessionConfig,删平行类型(契约本来就有这个口子)。
• 重做 M8 DEFAULT_PERMISSION_MODE='bypassPermissions' 但 capability 声明 approval → 矛盾,默认应审批开启。
• 重做 M9 getClaudeProviderAvailability() 直接 available:true —— 假阳性,必须真探针(paseo isAvailable() 是真去 resolve binary)。
• N9 extraArgs / advancedRawArgs 两字段做同一件事,冗余。

// config.ts:129 —— 默认权限模式
const DEFAULT_PERMISSION_MODE: ClaudePermissionMode = 'bypassPermissions' // ✗ 默认全放行
// 同时 getCapabilities() 声明了 'approval' —— 名义上有审批能力,默认配置却整个绕过它

// config.ts:326 —— 可用性上报
getClaudeProviderAvailability() {
  return { available: true, ... } // ✗ 从不探测 claude 是否安装 → 没装的用户到 createRunner 才炸
}

• 直接接 BASELINE_MODELS + getClaudeAvailabilityModelOptions()(~90 行)是活的 → 接进 AgentAvailability.models / 注册机制 ProviderModule.models。
• 扩契约 自定义 slash 命令发现 当前契约没方法(getControlCommands() 只管静态内置)→ 需接口层补统一 discovery 方法,实现挪到其后。
• agent markdown / MCP 诊断扫描:已拍定 defer(抽象层无对应建模,spec 0008 D12)—— 本期不入契约、不接线。

• 扩契约 原生 session 发现 契约缺方法 → 需接口层补统一 discovery 方法,返回统一 AgentNativeSession。
• 已有口子 发现后的导入→续:产出 AgentPersistenceHandle 喂 createRunner({resume}) —— 这段契约本来就有。
• 质量保留意见 逆向 Claude 私有、不保证的磁盘格式,天生脆 —— 要做这功能某种逆向不可避免,但必须版本兜底 + 失败降级,不能当稳定契约。

// sessions.ts 逆向的磁盘布局(Claude 私有格式,无兼容承诺):
~/.claude.json                                    // 全局索引:项目 → 会话清单
~/.claude/projects/<path-slug>/<sessionId>.jsonl  // transcript;文件名即 session id(已实测核对)
// 能力(把已有 CLI 会话导入 Eyrie 续聊)是真产品功能;但私有格式必须版本兜底 + 失败降级,
// 且产出要走统一契约出口(AgentPersistenceHandle),不能自成第二套持久化权威