关于 review
review 是用于代码库工作流的审查技能,可识别计划或代码模式,组建匹配的审查面板并执行修复循环。它按严重级别处理发现,并在轮次上限后给出结论。
主要功能
- 01自动判断计划或代码审查
- 02按内容信号选择审查者
- 03按严重级别修复或记录发现
- 04循环复审并生成结论
- 05支持轮次上限与模式覆盖
适用场景
- 01检查计划文件并处理审查意见
- 02审查分支差异并修复问题
- 03将低于门槛的发现记录为问题
安装
复制下面的命令,在终端中运行。
npx skills add https:/ / github.com/ camacho/ ai-skills --skill review安装后在 Claude Code 中输入/review确认已加载。
详细介绍
/review
检测应审查计划还是代码,组建合适的评审小组,并运行审查与修复循环。
1. 检测模式
根据参数或上下文判断:
- 明确指定:
/review plan或/review code—— 使用指定模式。 ai-workspace/plans/中存在计划文件(非.done.md),且没有超出 main 的提交 —— 计划模式。git diff main...HEAD有输出 —— 代码模式。- 无法确定 —— 提问:“正在审查计划还是代码?”
模式后可选的覆盖项:
- 轮次上限:默认为 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-reviewer | architect-reviewer |
| AGENTS.md、config.toml、sync.sh、技能 | codex-specialist | codex-specialist |
| 身份验证、凭据、令牌、机密 | security-auditor | security-auditor |
| UI、CSS、组件、无障碍性 | ui-designer | design-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 问题。为控制这一情况,代码模式中的最后一轮修复轮次是终结性扫描:
- 应用本轮的 P3 修复。
- 提交:
fix: address R{N} review findings (P3 sweep)。 - 运行验证(类型检查 + 测试)。如果有任何失败,则将失败视为门槛为 P2 的新一轮问题——终结性扫描作废,循环按正常流程继续。
- 不要重新分派评审小组进行下一轮审查。
- 打印摘要并以结论 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 → 立即退出,并打印摘要。