mcpField
R
review

review

github.com/camacho测试与质量393 次安装
代码仓库

关于 review

review 是用于代码库工作流的审查技能,可识别计划或代码模式,组建匹配的审查面板并执行修复循环。它按严重级别处理发现,并在轮次上限后给出结论。

主要功能

  1. 01自动判断计划或代码审查
  2. 02按内容信号选择审查者
  3. 03按严重级别修复或记录发现
  4. 04循环复审并生成结论
  5. 05支持轮次上限与模式覆盖

适用场景

  1. 01检查计划文件并处理审查意见
  2. 02审查分支差异并修复问题
  3. 03将低于门槛的发现记录为问题

安装

复制下面的命令,在终端中运行。

$npx skills add https://github.com/camacho/ai-skills --skill review

安装后在 Claude Code 中输入/review确认已加载。

详细介绍

/review

检测应审查计划还是代码,组建合适的评审小组,并运行审查与修复循环。

1. 检测模式

根据参数或上下文判断:

  1. 明确指定:/review plan 或 /review code —— 使用指定模式。
  2. ai-workspace/plans/ 中存在计划文件(非 .done.md),且没有超出 main 的提交 —— 计划模式。
  3. git diff main...HEAD 有输出 —— 代码模式。
  4. 无法确定 —— 提问:“正在审查计划还是代码?”

模式后可选的覆盖项:

  • 轮次上限:默认为 3。“thorough”(5)或“quick”(1)。
  • 严重级别门槛:默认值因模式而异 —— 计划模式 = P2,代码模式 = P3(本地合并流程假设现在修复小问题比将其列为问题更省事)。“strict”强制设为 P1,“p2”强制设为 P2,“lenient”强制设为 P3。

2. 组建评审小组

计划模式(步骤 4)

主审:technical-editor(始终加入)。 目标:计划文件。

代码模式(步骤 6)

主审:code-reviewer(始终加入)。 目标:已更改的文件(git diff main...HEAD)。

条件审查员(两种模式均适用)

扫描目标中的信号,并添加匹配的审查员:

信号计划模式代码模式
架构、新模块、DI 模式architect-reviewerarchitect-reviewer
AGENTS.md、config.toml、sync.sh、技能codex-specialistcodex-specialist
身份验证、凭据、令牌、机密security-auditorsecurity-auditor
UI、CSS、组件、无障碍性ui-designerdesign-reviewer

没有匹配信号时,仅由主审执行。

如果 /assemble-panel 可用,则调用它以遵循 RETAIN、EXPAND、CONVERGE 和 ESCALATE_RECURRING 治理规则。不要重复其策略规则。

Codex/Cursor:读取 .agents/skills/assemble-panel/SKILL.md 并在此处应用。

3. 分派

  • 计划模式:通过 Skill 工具调用 /plan-review。传入评审小组和轮次上限。其内部 P0 循环计为 R1。
  • 代码模式:通过 Agent 工具分派评审小组。要求提供带严重级别标签的发现(P0-P3)。

未标记的发现:按 P1 处理。

4. 循环:修复、记录、重新审查

每轮(R1..R{cap}):

a. 根据严重级别门槛进行分类:

  • 修复(P0 至 P{gate}):立即修复。提交:fix: address R{N} review findings,并附带 Co-Authored-By 尾注。
  • 记录(P{gate+1}+):记录,但不修复。

b. 记录低于门槛的发现:

  • 计划:在计划文件中追加 ### Deferred Findings (R{N})。
  • 代码:执行 gh issue create --label review-finding --title "R{N}: <summary>"。每轮最多 10 个,总计最多 20 个。如果 gh 不可用,则追加到 .branch-context.md。

c–f. 评审小组治理 —— 调用 /assemble-panel 以遵循 RETAIN、EXPAND、CONVERGE、ESCALATE_RECURRING。

终结性 P3 扫描(代码模式,门槛 = P3)

P3 小问题会不断增加:修复 15 个 P3 问题后,产生的差异会重新进入评审小组,并发现另外 15 个 P3 问题。为控制这一情况,代码模式中的最后一轮修复轮次是终结性扫描:

  1. 应用本轮的 P3 修复。
  2. 提交:fix: address R{N} review findings (P3 sweep)。
  3. 运行验证(类型检查 + 测试)。如果有任何失败,则将失败视为门槛为 P2 的新一轮问题——终结性扫描作废,循环按正常流程继续。
  4. 不要重新分派评审小组进行下一轮审查。
  5. 打印摘要并以结论 APPROVE 退出。

只有同时满足以下所有条件时,该轮才是终结性扫描:门槛为 P3,没有未修复的 P0/P1/P2 发现,且本轮仅发现 P3 问题。P0–P2 发现始终按正常流程重新进入循环(终结性扫描仅适用于只有小问题的轮次)。

5. 摘要

Review Summary
───────────────────────────────────────
Mode:      plan | code
Target:    <file or diff summary>
Rounds:    <N of cap>
Gate:      P<N>

│ Round │ P0 │ P1 │ P2 │ P3+ │ Fixed │ Recorded │
│───────│────│────│────│─────│───────│──────────│

Verdict: APPROVE | ESCALATE
  • APPROVE —— 所有高于门槛的发现均已解决。下一步:
    • 计划模式 → 构建(步骤 5)
    • 代码模式 → 归档(步骤 7)
  • ESCALATE —— 达到轮次上限后仍有未解决的高于门槛的发现。提交给人工处理。

防护措施

  • 绝不修改受保护文件(参见 AGENTS.md)。
  • 硬性上限:即使请求更高,也最多 5 轮。
  • 绝不自动合并或自动批准。只给出结论,不执行操作。
  • 仅输出纯文本——可安全用于不同界面。

回退方案

  • /plan-review 不可用 → 直接分派 technical-editor + architect-reviewer。
  • /assemble-panel 不可用 → 回退到 [technical-editor, code-reviewer],gate=P2,cap=3。
  • gh 未通过身份验证 → 降级为 .branch-context.md。
  • 审阅者返回 DROP → 立即退出,并打印摘要。

评论

支持 Markdown