菜单
代码审查工作流。对照需求和开发计划检查代码，确保功能完整、代码规范、无技术债。在"/review"、"/代码审查"、"review代码"时使用。
用 Codex 或 Claude 帮你安装 复制这段 Prompt，粘贴到 Codex、Claude 或其他助手里，让它检查 Skill 页面并帮你完成安装。
基于 SOC 职业分类
开发计划工作流。基于需求文档制定详细开发计划，拆解任务，明确依赖关系和文件修改范围。在"/dev-plan"、"/开发计划"、"制定计划"时使用。
需求讨论工作流。引导用户描述需求，使用 Critical Thinking 讨论，拆解功能，生成需求文档。在"/requirement"、"/需求讨论"、"讨论需求"时使用。
测试执行工作流。创建测试计划，编写测试脚本，执行测试，生成测试报告。在"/test"、"/测试"、"运行测试"时使用。
| name | code-review |
| description | 代码审查工作流。对照需求和开发计划检查代码，确保功能完整、代码规范、无技术债。在"/review"、"/代码审查"、"review代码"时使用。 |
开发流程第四阶段：代码 Review
核心目标：确保功能完整、代码规范、无技术债
docs/requirements/feature-xxx.mddocs/dev-plans/feature-xxx.md读取相关文档：
建立检查清单：
对照需求文档：
对照开发计划：
命名规范：
代码结构：
注释和文档：
错误处理：
临时方案：
性能问题：
安全问题：
保存到 docs/reviews/feature-xxx.md
# [Feature 名称] Review 报告
> Review 日期：YYYY-MM-DD
> 需求文档：[feature-xxx.md](../requirements/feature-xxx.md)
> 开发计划：[feature-xxx.md](../dev-plans/feature-xxx.md)
> 状态：待修复 / 已通过
## 总体评价
[一句话总结 Review 结果]
## 功能完整性
### 需求覆盖
| 功能 | 状态 | 备注 |
|-----|------|------|
| F1: xxx | ✅ 已实现 | - |
| F2: xxx | ⚠️ 部分实现 | 缺少 xxx |
| F3: xxx | ❌ 未实现 | - |
### 任务完成
| 任务 | 状态 | 备注 |
|-----|------|------|
| T1: xxx | ✅ 已完成 | - |
| T2: xxx | ✅ 已完成 | - |
## 代码规范
### 命名规范
- [x] 变量命名清晰
- [ ] 函数命名：`handleXxx` 建议改为 `onXxx`
### 代码结构
- [x] 函数长度合理
- [ ] 重复代码：`src/a.ts:20` 和 `src/b.ts:30` 逻辑重复
### 注释文档
- [x] 复杂逻辑有注释
- [ ] 公共 API 缺少 JSDoc
## 技术债
### 临时方案
| 位置 | 问题 | 建议 |
|-----|------|------|
| `src/xxx.ts:42` | TODO 注释未处理 | 实现或移除 |
| `src/yyy.ts:15` | 硬编码的 URL | 移到配置文件 |
### 性能问题
- 无明显性能问题
### 安全问题
- 无明显安全问题
## 问题清单
| ID | 严重程度 | 问题描述 | 位置 | 状态 |
|----|---------|---------|------|------|
| R1 | 🔴 高 | F2 功能未完整实现 | - | ⬜ 待修复 |
| R2 | 🟡 中 | 重复代码需要抽取 | src/a.ts:20 | ⬜ 待修复 |
| R3 | 🟢 低 | 建议添加 JSDoc | src/api.ts | ⬜ 待修复 |
## 修复建议
### R1: F2 功能未完整实现
**问题**：缺少 xxx 功能
**建议**：在 `src/xxx.ts` 中添加 xxx 逻辑
**参考**：需求文档 F2 描述
### R2: 重复代码需要抽取
**问题**：`src/a.ts:20` 和 `src/b.ts:30` 逻辑重复
**建议**：抽取为公共函数 `utils/xxx.ts`
## 修复记录
| ID | 修复时间 | Commit | 状态 |
|----|---------|--------|------|
| R1 | - | - | ⬜ 待修复 |
| R2 | - | - | ⬜ 待修复 |
进入下一阶段（问题修复）：
跳过修复阶段（直接进入测试）：