review-task

Review Task Command

功能说明

审查任务的代码实现，检查代码质量、规范合规性、测试覆盖等，输出审查报告。

⚠️ CRITICAL: 状态更新要求

执行此命令后，你必须立即更新任务状态。参见规则 7。

执行流程

1. 验证前置条件

检查必需文件：

• .ai-workspace/active/{task-id}/task.md - 任务文件
• .ai-workspace/active/{task-id}/implementation.md - 实现报告

注意：{task-id} 格式为 TASK-{yyyyMMdd-HHmmss}，例如 TASK-20260205-202013

如果任一文件不存在，提示用户先完成前置步骤。

2. 读取实现报告

仔细阅读 implementation.md，了解：

• 已修改的文件列表
• 实现的关键功能
• 测试情况
• 实现者标注的需要关注的点

3. 执行代码审查

按照 .agents/workflows/feature-development.yaml 中的 code-review 步骤：

必须审查的内容：

• 代码质量和编码规范（遵循 CLAUDE.md）
• Bug 和潜在问题检测
• 测试覆盖率和测试质量
• 错误处理和边界情况
• 性能和安全问题
• 代码注释和文档
• 与技术方案的一致性

审查原则：

1. 严格但公正：指出问题，但也认可优点
2. 具体明确：给出具体的文件和行号
3. 提供建议：不仅指出问题，还要提供改进建议
4. 分级处理：区分必须修复和建议优化

4. 调用专业审查工具（可选）

如果需要更深度的审查，可以调用：

方案1：快速审查（推荐用于日常 PR）

/code-review:code-review <pr-number>

• 5个并行 Sonnet 代理
• CLAUDE.md 规范合规性
• Bug 检测与历史上下文分析

方案2：深度审查（推荐用于重要功能）

/pr-review-toolkit:review-pr

• 6个专业审查代理
• 代码注释准确性、测试覆盖、错误处理、类型设计等多维度审查

5. 输出审查报告

创建 .ai-workspace/active/{task-id}/review.md，必须包含以下章节：

# 代码审查报告

## 审查概要

- **审查者**: {审查者}
- **审查时间**: {时间}
- **审查范围**: {文件数量和主要模块}
- **总体评价**: {通过/需要修改/不通过}

## 审查发现

### 🔴 必须修复（Blocker）

#### 1. {问题标题}
**文件**: `{file-path}:{line-number}`
**问题描述**: {详细描述}
**修复建议**: {具体建议}
**严重程度**: 高

### 🟡 建议修改（Major）

#### 1. {问题标题}
**文件**: `{file-path}:{line-number}`
**问题描述**: {详细描述}
**修复建议**: {具体建议}
**严重程度**: 中

### 🟢 优化建议（Minor）

#### 1. {优化点}
**文件**: `{file-path}:{line-number}`
**建议**: {优化建议}

## 优点与亮点

- {做得好的地方1}
- {做得好的地方2}

## 规范检查

### CLAUDE.md 合规性
- [ ] 编码规范
- [ ] 命名规范
- [ ] 注释规范
- [ ] 测试规范

### 代码质量指标
- 圈复杂度: {评估}
- 代码重复: {评估}
- 测试覆盖率: {百分比}

## 测试审查

### 测试覆盖
- 单元测试: {评价}
- 边界情况: {是否覆盖}
- 异常情况: {是否覆盖}

### 测试质量
- 测试命名: {评价}
- 断言充分性: {评价}
- 测试独立性: {评价}

## 安全审查

- SQL 注入风险: {检查结果}
- XSS 风险: {检查结果}
- 权限控制: {检查结果}
- 敏感信息泄露: {检查结果}

## 性能审查

- 算法复杂度: {评估}
- 数据库查询: {优化建议}
- 资源释放: {检查结果}

## 与方案的一致性

- [ ] 实现符合技术方案
- [ ] 未偏离设计意图
- [ ] 无计划外功能

## 总结与建议

### 是否批准
- [ ] ✅ 批准合并（无阻塞问题）
- [ ] ⚠️ 修改后批准（有建议修改项）
- [ ] ❌ 需要重大修改（有阻塞问题）

### 下一步行动
{根据审查结果给出下一步建议}