Code Review Skill

Iterative code review through parallel independent subagent reviews with user confirmation at each step.

⚠️ 用户控制 vs 自动化

默认行为（安全模式）：

• ✅ 每一步都需要用户确认
• ❌ 不会自动修改代码

可选：自动化模式（需手动启用）：

用户可通过配置文件启用 autoFix 和 autoContinue。详见 references/automation.md。

┌─────────────────────────────────────────────────────────┐
│  默认：安全模式                                          │
│  - 每步都需要用户确认                                    │
│  - 适合：重要项目、首次使用                               │
│                                                          │
│  可选：自动化模式（需要手动配置启用）                      │
│  - autoFix=true: 发现问题后自动修复                       │
│  - autoContinue=true: 修复后自动继续下一轮                │
│  ⚠️ 警告：会自动修改代码！                                │
└─────────────────────────────────────────────────────────┘

Pre-flight Checks

开始 Review 前，必须执行以下检查：

详见 references/preflight.md。

Workflow

Round Structure

┌─────────────────────────────────────────────────────────┐
│  Review Round N                                          │
│                                                          │
│  ┌─────────┐  ┌─────────┐  ┌─────────┐                  │
│  │Reviewer1│  │Reviewer2│  │Reviewer3│  ← 并行 3 个      │
│  └────┬────┘  └────┬────┘  └────┬────┘                  │
│       └────────────┼────────────┘                        │
│                    ▼                                     │
│            ┌──────────────┐                              │
│            │ 汇总问题列表  │                              │
│            └──────┬───────┘                              │
│                   ▼                                      │
│            ┌──────────────┐                              │
│            │  用户确认    │  ← 是否继续修复？            │
│            └──────┬───────┘                              │
│                   ▼                                      │
│            ┌──────────────┐                              │
│            │   Fixer      │  ← 用户同意后才执行          │
│            └──────────────┘                              │
└─────────────────────────────────────────────────────────┘

Reviewer 关注点

详见 references/workflow.md。

退出标准

• 连续 两轮 无 >= severityThreshold 的问题
• 或达到 maxRounds 限制
• 用户决定结束

Final Round 特殊要求

Final Round 必须采用 Full Review 模式！

1. 必须 Full Review - 不是 Delta Review
2. 必须验证编译和测试 - npm run build + npm test
3. 使用更长超时 - 全量审查需要更多时间
4. 审查所有历史修复 - 确认所有 Round 的问题都已修复

Issue Severity

Key Points

1. 用户控制 - 默认每一步都需要用户确认
2. 自动化可选 - 通过配置启用自动修复
3. PR 历史检查 - 避免重复发现已修复的问题
4. Final Round 必须 Full Review - 全量审核
5. 3 个 reviewer 并行 - 最大化问题发现
6. MAX_ROUNDS = 10 - 防止无限循环

Safety Boundaries

✅ 允许：读取代码、运行只读命令、Spawn subagent 分析、报告问题

⚠️ 需确认：修改文件、git commit、npm install/build、Spawn Fixer

❌ 禁止：未经同意修改代码、自主运行多轮修复、自主提交

References

• automation.md - 自动化偏好配置详解
• preflight.md - Pre-flight Checks 详细步骤
• workflow.md - Workflow 详细说明
• checklist.md - 审查清单
• issue-severity.md - Issue 严重级别
• final-round.md - Final Round 详细指南