| name | requesting-code-review |
| description | 代码变更完成后请求审查。派发规范审查+代码质量审查两个子代理,汇总结果。触发词:审查代码、review、code review、检查代码。 |
| agent_created | true |
请求代码审查
概述
代码变更完成后请求代码审查。使用 WorkBuddy Agent 工具派发审查子代理。
开始时声明: "我将用 requesting-code-review 技能审查代码变更。"
审查流程
代码变更 → 规范审查子代理 → 代码质量审查子代理 → 汇总报告 → 修复 → 重新审查
步骤 1:准备审查上下文
git diff --stat <base>..<head>
git diff <base>..<head>
步骤 2:派发规范审查子代理
使用 Agent 工具派发 general-purpose 子代理:
你是做规范合规审查的子代理。只审查实现是否匹配计划中的规格要求。
计划文件: [路径]
GIT 范围: [base..head]
审查:
- 每个规格要求都有实现吗?
- 实现的行为和规格描述一致吗?
- 有规格没要求但实现了的功能吗?(YAGNI 违反)
报告:
- PASS 或 FAIL
- 任何规格遗漏或偏差
- 不要评论代码风格或质量
步骤 3:派发代码质量审查子代理
你是做代码质量审查的子代理。
GIT 范围: [base..head]
描述: [变更概述]
审查:
- DRY 违反(重复代码)
- 死代码(永远不会执行的代码)
- YAGNI 违反(过度工程)
- 命名问题
- 缺少错误处理
- 测试覆盖不足
严重程度: 严重 / 重要 / 次要
报告: 批准,或按严重程度列出问题。
步骤 4:汇总结果
将两个审查报告汇总给用户,标注严重程度和处理建议。
步骤 5:修复与重新审查
- 严重/重要问题必须修复
- 修复后重新审查变更部分
- 全部通过后标记审查完成
审查严重程度
| 严重程度 | 行动 |
|---|
| 严重 | 阻塞合并,必须修复 |
| 重要 | 应该修复,强烈建议 |
| 次要 | 可以改进,不阻塞 |