# Requesting Code Review
> 代码变更完成后请求审查。派发规范审查+代码质量审查两个子代理，汇总结果。触发词：审查代码、review、code review、检查代码。
- Skill: `yinqd3/requesting-code-review` (Agent Skill)
- Install (CLI): `npx skillmds@latest add yinqd3/requesting-code-review`
- Raw SKILL.md: https://api.skillmd.com/api/skills/yinqd3/requesting-code-review/raw
- Safety review: pending
- Works with: Claude Code, Claude.ai, OpenAI Codex
- Category: Coding & Dev Tools
- Author: yinqd3 (https://skillmd.com/u/yinqd3)
- Updated: 2026-09-17
- Page: https://skillmd.com/skills/yinqd3/requesting-code-review
---
# 请求代码审查
## 概述
代码变更完成后请求代码审查。使用 WorkBuddy `Agent` 工具派发审查子代理。
**开始时声明：** "我将用 requesting-code-review 技能审查代码变更。"
## 审查流程
```
代码变更 → 规范审查子代理 → 代码质量审查子代理 → 汇总报告 → 修复 → 重新审查
```
## 步骤 1：准备审查上下文
```bash
# 获取变更范围
git diff --stat ..
git diff ..
```
## 步骤 2：派发规范审查子代理
使用 `Agent` 工具派发 general-purpose 子代理：
```
你是做规范合规审查的子代理。只审查实现是否匹配计划中的规格要求。
计划文件: [路径]
GIT 范围: [base..head]
审查:
- 每个规格要求都有实现吗？
- 实现的行为和规格描述一致吗？
- 有规格没要求但实现了的功能吗？（YAGNI 违反）
报告:
- PASS 或 FAIL
- 任何规格遗漏或偏差
- 不要评论代码风格或质量
```
## 步骤 3：派发代码质量审查子代理
```
你是做代码质量审查的子代理。
GIT 范围: [base..head]
描述: [变更概述]
审查:
- DRY 违反（重复代码）
- 死代码（永远不会执行的代码）
- YAGNI 违反（过度工程）
- 命名问题
- 缺少错误处理
- 测试覆盖不足
严重程度: 严重 / 重要 / 次要
报告: 批准，或按严重程度列出问题。
```
## 步骤 4：汇总结果
将两个审查报告汇总给用户，标注严重程度和处理建议。
## 步骤 5：修复与重新审查
- 严重/重要问题必须修复
- 修复后重新审查变更部分
- 全部通过后标记审查完成
## 审查严重程度
| 严重程度 | 行动 |
|:---|:---|
| 严重 | 阻塞合并，必须修复 |
| 重要 | 应该修复，强烈建议 |
| 次要 | 可以改进，不阻塞 |