# Code Review

> 代码审查和质量检查。调度 code-reviewer sub-agent 审查代码，检查是否符合计划和编码标准。

- Skill: `herxinsasa/code-review` (Agent Skill)
- Install (CLI): `npx skillmds@latest add herxinsasa/code-review`
- Raw SKILL.md: https://api.skillmd.com/api/skills/herxinsasa/code-review/raw
- Safety review: pending
- Works with: Claude Code, Claude.ai, OpenAI Codex
- Category: AI & ML
- Author: herxinsasa (https://skillmd.com/u/herxinsasa)
- Updated: 2026-09-17
- Page: https://skillmd.com/skills/herxinsasa/code-review

---


# Code Review — 代码审查

## 职责

审查代码质量和风险，确保实现符合计划要求和编码标准。将审查任务分派给独立的 `code-reviewer` sub-agent。

## 触发场景

- Dev Builder 完成开发后自动触发
- 用户说"审查代码"、"代码审查"、"review"
- commit 前希望检查代码质量

## 工作流程

### Step 1: 确定审查范围
与用户确认审查范围：
1. 全量审查还是特定文件？
2. 审查重点是什么？（安全性/性能/架构/常规/视觉一致性？）
3. 是否参考了 Dev-Plan.md 中的技术方案？
4. 如果有设计交付物（Design-Brief.md 交付物表），需要对照检查视觉一致性

### Step 2: 调度审查
创建 `code-reviewer` sub-agent 实例，传入：
- 审查范围（文件列表或变更范围）
- 审查重点
- 相关的计划文档（Dev-Plan.md）
- 设计规范（Design-Brief.md，如有）
- 设计交付物路径（Design-Brief.md「设计交付物」表格，如有）
- 项目编码规范

### Step 3: 处理审查结果
根据审查结论采取对应行动：

| 结论 | 处理方式 |
|------|----------|
| 通过 | 通知用户，可继续下一步 |
| 有条件通过 | 修复 Important 级别问题后继续 |
| 不通过 | 修复 Critical 问题后重新审查 |

### Step 4: 输出审查报告
将审查结果记录到审查报告中。

### Step 5: 写入审查完成标记
审查通过后，更新 `.claude/.review-status.json`：

```json
{
  "last_review": "<ISO 8601 timestamp>",
  "reviewed_files": ["file1.ts", "file2.ts"],
  "reviewer": "code-reviewer",
  "conclusion": "PASS|CONDITIONAL_PASS|FAIL"
}
```

此文件由 `stop-gate` hook 读取，用于判断是否允许 agent 停止。

## 输出
- 代码审查报告（问题清单、严重级别、修改建议）
- 如果需要修复，标记待修复项

## 审查检查清单

code-reviewer 在审查时，如果任务涉及 UI 实现，**必须**检查以下视觉一致性项：

```
□ 色值与 Design-Brief.md §1.3 配色方案一致
□ 圆角/阴影/间距与 Design-Brief.md §1.6 组件视觉规范一致
□ 布局与对应模块 UE（§2.x）一致
□ 若有设计源文件（.pen / Figma MCP）→ 读取结构化数据，比对关键尺寸
□ 若仅有 PNG → 以 Design-Brief 文字规范为准，PNG 仅作辅助参照
□ CSS 变量 / Token 是否与设计规范的色值对应（禁止 hardcode 裸色值）
```

## 参考资源
- `.claude/agents/code-reviewer.md` — 审查者 sub-agent 定义
- [code-review-skill](../../references/code-review-skill/SKILL.md) — 多技术栈审查规则
- [requesting-code-review](../../references/superpowers/requesting-code-review.md) — 审查请求流程
- [code-reviewer](../../references/superpowers/code-reviewer.md) — 审查者角色参考