Requesting Code Review

请求代码审查

概述

代码变更完成后请求代码审查。使用 WorkBuddy Agent 工具派发审查子代理。

开始时声明： "我将用 requesting-code-review 技能审查代码变更。"

审查流程

代码变更 → 规范审查子代理 → 代码质量审查子代理 → 汇总报告 → 修复 → 重新审查

步骤 1：准备审查上下文

# 获取变更范围
git diff --stat ..
git diff ..

步骤 2：派发规范审查子代理

使用 Agent 工具派发 general-purpose 子代理：

你是做规范合规审查的子代理。只审查实现是否匹配计划中的规格要求。

计划文件: [路径]
GIT 范围: [base..head]

审查:
- 每个规格要求都有实现吗？
- 实现的行为和规格描述一致吗？
- 有规格没要求但实现了的功能吗？（YAGNI 违反）

报告:
- PASS 或 FAIL
- 任何规格遗漏或偏差
- 不要评论代码风格或质量

步骤 3：派发代码质量审查子代理

你是做代码质量审查的子代理。

GIT 范围: [base..head]
描述: [变更概述]

审查:
- DRY 违反（重复代码）
- 死代码（永远不会执行的代码）
- YAGNI 违反（过度工程）
- 命名问题
- 缺少错误处理
- 测试覆盖不足

严重程度: 严重 / 重要 / 次要
报告: 批准，或按严重程度列出问题。

步骤 4：汇总结果

将两个审查报告汇总给用户，标注严重程度和处理建议。

步骤 5：修复与重新审查

• 严重/重要问题必须修复
• 修复后重新审查变更部分
• 全部通过后标记审查完成

审查严重程度

| 严重程度 | 行动 | |:---|:---| | 严重 | 阻塞合并，必须修复 | | 重要 | 应该修复，强烈建议 | | 次要 | 可以改进，不阻塞 |

Source & license

This open-source skill is cataloged on AgentStack and links to its original source — we do not rehost the code.

• Author: yinqd3
• Source: yinqd3/workbuddy-skills
• License: MIT

Install and usage instructions live in the source repository linked above.