代码审查技能

通过并行独立子代理审查并在每一步获得用户确认的迭代式代码审查。

⚠️ 用户控制 vs 自动化

默认行为（安全模式）：

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

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

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

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

预检检查

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

详见 references/preflight.md。

工作流程

轮次结构

┌─────────────────────────────────────────────────────────┐
│ 审查轮次 N │
│ │
│ ┌─────────┐ ┌─────────┐ ┌─────────┐ │
│ │审查者1 │ │审查者2 │ │审查者3 │ ← 并行 3 个 │
│ └────┬────┘ └────┬────┘ └────┬────┘ │
│ └────────────┼────────────┘ │
│ ▼ │
│ ┌──────────────┐ │
│ │ 汇总问题列表 │ │
│ └──────┬───────┘ │
│ ▼ │
│ ┌──────────────┐ │
│ │ 用户确认 │ ← 是否继续修复？ │
│ └──────┬───────┘ │
│ ▼ │
│ ┌──────────────┐ │
│ │ 修复者 │ ← 用户同意后才执行 │
│ └──────────────┘ │
└─────────────────────────────────────────────────────────┘

审查者关注点

详见 references/workflow.md。

退出标准

• - 连续 两轮 无 >= 严重级别阈值 的问题
• 或达到最大轮次限制
• 用户决定结束

最终轮次特殊要求

最终轮次必须采用完整审查模式！

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

问题严重级别

关键要点

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

安全边界

✅ 允许：读取代码、运行只读命令、生成子代理分析、报告问题

⚠️ 需确认：修改文件、git commit、npm install/build、生成修复者

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

参考资料

• - automation.md - 自动化偏好配置详解
• preflight.md - 预检检查详细步骤
• workflow.md - 工作流程详细说明
• checklist.md - 审查清单
• issue-severity.md - 问题严重级别
• final-round.md - 最终轮次详细指南