Article

AI Vibe Coding 05:AI优先代码审查流水线(先机审,再人审)

路线阶段:AI Vibe Coding 第 5 章。
本章目标:把代码评审从“靠经验抽查”升级为“机器先筛 + 人工决断”的稳定流程。

学习目标

完成本章后,你应该能做到:

  1. 设计 AI 代码审查清单与风险等级。
  2. 将 AI 审查结果结构化输出到 PR 流程。
  3. 让 AI 先给修复建议,再进入人工评审。
  4. 用数据衡量“审查速度与缺陷拦截率”。

为什么要先机审

人工评审常见问题:

  1. PR 过大,审查疲劳。
  2. 低级问题反复出现(命名、空指针、越界)。
  3. 高风险路径(经济、奖励、安全)漏看。

AI 先机审能先过滤大量基础问题,让人把精力放在架构与业务风险。

审查维度清单

建议固定 8 类:

  1. 逻辑正确性
  2. 空值与边界处理
  3. 并发与状态一致性
  4. 性能热点
  5. 安全与权限
  6. 兼容与回滚
  7. 测试覆盖
  8. 可维护性

风险分级

P0:可能导致崩溃、资金/奖励错误、安全漏洞
P1:明显功能错误、重要边界缺失
P2:可维护性或性能中风险问题
P3:风格与可读性优化项

AI 审查输出必须标注风险等级。

AI 审查输出协议

{
  "summary": "发现 2个P1,3个P2",
  "findings": [
    {
      "severity": "P1",
      "file": "src/domain/reward/grant.ts",
      "line": 87,
      "issue": "重复领奖幂等校验缺失",
      "suggestion": "增加requestId去重并落库"
    }
  ],
  "tests": [
    "新增奖励重复提交测试",
    "补充异常路径回归测试"
  ]
}

PR 流程改造

  1. 开发提交 PR。
  2. 触发 AI 审查任务。
  3. 输出结构化评论到 PR。
  4. 开发先处理 P0/P1。
  5. 人工评审只审剩余关键点。

机审规则示例

规则1:涉及奖励发放必须检查幂等
规则2:涉及排行榜提交必须检查异常分数防护
规则3:涉及活动时间窗必须检查时区与边界
规则4:涉及高频Update必须检查每帧分配

自动修复建议

对 P2/P3 问题可让 AI 给 patch 建议:

  1. 无副作用重构
  2. 缺失日志补充
  3. 明确错误处理分支

但 P0/P1 默认不自动合并,必须人工确认。

审查质量度量

每周统计:

  1. AI 发现问题总数
  2. 人工确认有效率
  3. 机审误报率
  4. 漏检率(上线后回归)
  5. 平均评审时长

目标:

  1. 评审时长下降
  2. P0/P1 漏检下降

与现有流程联动

  1. 质量门禁:AI 审查不过禁止合并。
  2. 测试门禁:AI 建议的关键测试必须补齐。
  3. 发布门禁:高风险路径需额外审查确认。
  4. 埋点系统:记录机审命中与后续缺陷关联。

常见坑

坑 1:把 AI 审查当最终结论

AI 是筛查器,不是裁决者。最终责任仍在人工。

坑 2:输出不结构化

无法自动接入 PR 工作流,最后变成“看一段长文本”。

坑 3:规则漂移无人维护

规则应版本化,定期复盘更新。

本月作业

落地“机审先行”流程:

  1. 为团队前 20 条高风险规则建规则库。
  2. 在 1 周内让全部 PR 走机审。
  3. 输出机审有效率和误报率报告。

下一章:AI Vibe Coding 06《测试协同:AI 自动生成回归测试与边界样例》。