code-review

按六类清单审查 diff，输出分级的「位置 → 问题 → 原因 → 改法」，自带 references/checklist.md。

触发条件（frontmatter.description）

审查代码变更（diff）或指定文件，按严重程度输出问题清单。当用户要求 review / 审查代码、检查 PR、评估改动风险、或问"这段代码有没有问题"时使用。不用于格式化、自动重写全文件。

目录结构

code-review/
├── SKILL.md
└── references/checklist.md

安装方式

Claude Code把技能目录复制到以下位置，重启会话后自动可用~/.claude/skills/<技能名>/项目级可放 <项目>/.claude/skills/
Claude.ai / Desktop下载 .zip 后在设置 → Capabilities → Skills 上传需要开启 Skills 功能
Hermes Agent放到技能目录，Hermes 启动时自动加载~/.hermes/skills/<技能名>/
其它 Agent任何支持 SKILL.md 约定（YAML frontmatter + 步骤 + 脚本）的框架都可直接使用本质是一个带说明文档的脚本目录

下载的 zip 解压后是 code-review/ 目录，直接整个放进去即可，不用改文件名。

技能包内容

---
name: code-review
description: 审查代码变更（diff）或指定文件，按严重程度输出问题清单。当用户要求 review / 审查代码、检查 PR、评估改动风险、或问"这段代码有没有问题"时使用。不用于格式化、自动重写全文件。
---

# 代码审查

## 什么时候用

- 用户给出 diff、PR 链接或文件路径，要求审查
- 用户问「这段代码有没有坑」「能合并吗」
- 提交前自检

**不要用**：用户只是要格式化、要重写整个文件、或问语法细节。

## 步骤

1. **取范围**：确认是 diff 还是完整文件；没有明确范围时先问，不要假设
2. **读上下文**：找同目录的类型定义、调用方、错误处理惯例（不要只看被改的几行）
3. **逐条过清单**：按 `references/checklist.md` 的六类检查
4. **分级**：🔴 阻断 / 🟡 建议 / 🟢 可选
5. **每条给改法**：位置 → 问题 → 为什么 → 具体替换代码
6. **给结论**：可合并 / 需修改 / 建议重新设计

## 输出格式

```
## 结论
（一行结论 + 一句话理由）

## 问题
### 🔴 阻断
1. 【文件:行】问题 → 原因 → 改法（代码块）
### 🟡 建议
### 🟢 可选（最多 3 条）
```

## 验证

- 每条问题都能定位到具体行，不是泛泛而谈
- 每条改法都给出可粘贴的代码
- 没有「建议加注释/加测试」这类空话

## 陷阱

- 只报真实问题，不要凑数——凑数会淹没关键问题
- 不确定的写法明确说「不确定，需确认」，不要猜
- 不要顺手重写整个文件，审查只给判断

SKILL.md 说明（Agent 读到后照此执行）

# 代码审查

## 什么时候用

- 用户给出 diff、PR 链接或文件路径，要求审查
- 用户问「这段代码有没有坑」「能合并吗」
- 提交前自检

**不要用**：用户只是要格式化、要重写整个文件、或问语法细节。

## 步骤

1. **取范围**：确认是 diff 还是完整文件；没有明确范围时先问，不要假设
2. **读上下文**：找同目录的类型定义、调用方、错误处理惯例（不要只看被改的几行）
3. **逐条过清单**：按 `references/checklist.md` 的六类检查
4. **分级**：🔴 阻断 / 🟡 建议 / 🟢 可选
5. **每条给改法**：位置 → 问题 → 为什么 → 具体替换代码
6. **给结论**：可合并 / 需修改 / 建议重新设计

## 输出格式

```
## 结论
（一行结论 + 一句话理由）

## 问题
### 🔴 阻断
1. 【文件:行】问题 → 原因 → 改法（代码块）
### 🟡 建议
### 🟢 可选（最多 3 条）
```

## 验证

- 每条问题都能定位到具体行，不是泛泛而谈
- 每条改法都给出可粘贴的代码
- 没有「建议加注释/加测试」这类空话

## 陷阱

- 只报真实问题，不要凑数——凑数会淹没关键问题
- 不确定的写法明确说「不确定，需确认」，不要猜
- 不要顺手重写整个文件，审查只给判断

← 返回技能库