Menú
Use when the user asks to review code, check for issues, or says "review", "审查", "检查代码"
Instalar con Codex o Claude Copia este prompt, pégalo en Codex, Claude u otro asistente, y deja que revise la página de la skill y la instale por ti.
Basado en la clasificación ocupacional SOC
| name | code-review |
| description | Use when the user asks to review code, check for issues, or says "review", "审查", "检查代码" |
| user-invocable | true |
按优先级结构化审查代码。聚焦真实问题 — 正确性、安全性、可维护性，而不是风格吹毛求疵。
核心原则： 审查优先级：Bug > 安全 > 性能 > 可读性 > 风格。
违反优先级顺序就是违反代码审查的精神。
在验证正确性和安全性之前，不得评论代码风格
如果你还没有检查 Bug 和安全问题，就不应该评论命名和格式。
适用于任何代码审查场景：
特别是当：
不要跳过审查：
必须按顺序完成每个阶段。
在查看任何代码细节之前：
确定范围和意图
quick-review.sh 快速查看 diff 概况阅读周边代码
检查 diff 大小
500 行：建议拆分 PR
严格按顺序逐项检查：
完整清单见 checklist.md，简要版本：
各语言常见反模式详见 common-issues.md（TypeScript、Java、Python）。
按严重性分组，而非按文件分组：
## 代码审查: <范围>
### 严重（必须修复）
- [Bug] <文件:行号> — 描述 + 修复建议
- [安全] <文件:行号> — 描述 + 修复建议
### 警告（建议修复）
- [性能] <文件:行号> — 描述 + 建议
- [异常] <文件:行号> — 描述 + 建议
### 建议（可以改进）
- [风格] <文件:行号> — 描述
- [DRY] <文件:行号> — 描述
### 亮点
- <值得肯定的好模式>
### 总结
<1-2 句话: 整体评价，通过/需要修改>
规则：
如果你发现自己在想：
以上任何一条都意味着：停下来。回到阶段一。
| 借口 | 现实 |
|---|---|
| "测试过了，所以是对的" | 测试也有盲区。独立审查逻辑。 |
| "只是风格问题" | 如果你只发现了风格问题，说明你没认真看。 |
| "文件太多审不完" | 让作者拆 PR。不要囫囵吞枣。 |
| "我信任这个开发者" | 信任不能替代审查。看代码。 |
| "只是日志/注释" | 日志可能泄露密钥。注释可能误导。 |
| "没时间仔细审查" | 敷衍审查 = 漏掉 Bug = 以后花更多时间修。 |
| 阶段 | 核心活动 | 完成标准 |
|---|---|---|
| 1. 上下文 | 读范围、周边代码、diff 大小 | 理解意图 |
| 2. 检查清单 | 正确性 → 安全 → 性能 → 可维护性 | 所有类别已检查 |
| 3. 输出 | 按严重性分组的结构化报告 | 给出明确结论 |
本目录下可用的资源：
checklist.md — 按优先级组织的完整审查清单common-issues.md — 按语言分类的常见反模式（TypeScript、Java、Python）quick-review.sh — 快速 diff 分析脚本（文件统计、风险检测）代码审查研究数据：