yinqd3 请求代码审查
概述
代码变更完成后请求代码审查。使用 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：修复与重新审查
- 严重/重要问题必须修复
- 修复后重新审查变更部分
- 全部通过后标记审查完成
审查严重程度
| 严重程度 | 行动 |
|---|---|
| 严重 | 阻塞合并，必须修复 |
| 重要 | 应该修复，强烈建议 |
| 次要 | 可以改进，不阻塞 |