核心用法
Iterative Code Review Skill 采用并行多代理架构实现深度代码审查。核心流程为:每轮审查同时spawn 3个独立Reviewer(功能正确性、代码质量、安全性),汇总问题后由单一Fixer修复,循环迭代直至连续两轮无P0-P2级别问题。
Pre-flight Checks(6项强制检查):
1. 模型推荐:自动推荐代码能力最强模型(如bailian/glm-5),用户确认或更换,必须启用thinking: "high"
2. maxSpawnDepth验证:≥1方可继续
3. 变更规模检测:按文件数(<10/10-50/>50)动态调整超时时间(4-20分钟)
4. 新增代码识别:git diff识别新增文件,重点审查安全性和边界条件
5. PR历史检查:读取commit history避免重复报告已修复问题
6. Review模式选择:多commit PR询问用户选择审查范围
退出标准:连续两轮无P0/P1/P2问题(P3不重置计数器),或达到MAX_ROUNDS=10。Final Round强制采用Full Review模式,必须验证编译和测试。
显著优点
- 高检出率:3个独立Reviewer并行,覆盖功能、质量、安全三维度
- 自动化闭环:review-fix-review循环无需用户介入,强制自动执行
- 智能规模适配:根据变更量动态调整超时,避免资源浪费或审查不足
- 模型优化:自动推荐最强代码模型,降低用户决策成本
- 防重复机制:PR历史检查避免无效重复报告
潜在局限
- 资源消耗高:每轮3+1个子代理,大型PR可能触达10轮上限,token成本显著
- Final Round强制全量:大仓库Final Round全量审查可能超时或成本过高
- P3问题可能遗漏:P3级别"尽量修复"且不影响退出计数,低优先级问题可能未处理
- 模型依赖:代码能力评估依赖预设优先级列表,新型号可能未及时更新
- 无增量优化:每次Full Review需重新分析全部代码,缺乏变更聚焦优化
适合人群
- 需要严格代码质量把关的团队(金融、安全关键领域)
- 缺乏专职代码审查人员的小型开发团队
- 大型PR或涉及安全敏感变更的审查场景
- 需要标准化、可复现审查流程的组织
常规风险
- 循环超限风险:复杂遗留代码可能10轮未收敛,强制终止但问题未根治
- Fixer引入新问题:单一Fixer修复可能产生回归,依赖Final Round验证
- 超时配置不当:用户未按规模调整导致审查中断或草率完成
- git状态依赖:pre-flight检查依赖正确配置的origin/develop,误配导致范围错误