代码审查被AI搞崩之后，我给我们团队重建了一遍流程

我对比了纯人工的Code Review和AI自动审查两条路，实际跑了一遍，大概两周时间，把团队里那条快要堵死的PR流水线理清楚了。先说结论：AI没把写代码搞崩，它把代码审查搞崩了。

问题出在量上。以前一个PR三五处改动，人看一眼就过。现在AI写代码，一个PR动辄几百行，速度快了，量上来了，审查的人还是那几个。我团队里现在的情况是，PR排队时间从半天涨到两天，reviewer打开PR扫一眼就approve，因为真没时间细看。这比不看更危险，看起来有流程，实际上流程空转了。

素材里有一句话说得挺准：作者没写那行代码，审查者也没真读那行代码，那这行代码到底谁负责。

我决定把流程拆成三段，AI先筛，人再读，机器兜底。

第一步，接AI预审。我们在CI里挂了一个自动审查的工具，PR一提交，它先跑一遍静态检查、逻辑扫描和安全规则。我这边测下来，AI对风格问题和常识性bug抓得挺准，比如空指针、忘了处理null、硬编码的密钥，这些它一眼就能挑出来。速度和人也比不了，基本是秒级出结果。

第二步，人审关键路径。这一步是给开发者的规则：AI报出来的问题先看，但AI没报的，你要自己读diff。读不完几百行没关系，核心逻辑、数据流转、异常处理这几段必须看。我定了条硬规矩，diff超过三百行的PR必须拆，拆到人能真正读得完的粒度。

第三步，机器兜底。合并之前跑一遍完整测试和构建，覆盖率低于阈值直接拦下来。这一步是给流程上保险，防止人审和AI审都漏掉的东西悄悄上线。

踩坑环节。这流程跑了三天就出问题，AI误报太多，开发者在群里吵，说工具在教他写代码。我让负责的工程师把规则调了一下，安全类的问题全开，风格类的只留高置信度的，误报率降下来，大家才愿意用。第二个坑是，有些开发者看了AI的结论就直接approve，人审那步形同虚设。我把流程改成AI的comment必须逐条resolve，不处理完不能合入。第三个坑是覆盖率阈值设得太高，CI时间拉长，又堵了。后来调到大约现有水平加五个点，不折腾。

跑了一周多，效果是PR周转时间体感降了一半，reviewer的心理负担也小了。以前打开一个PR是「我要对每一行负责」，现在是「AI先过了一遍，我重点看它没看懂的」。这个心理转换很关键，人愿意认真读了。

有一篇讨论里说，AI审查的问题在于，它抓到的bug抓得更快，但它抓不到的，比以前更容易漏过去。我认同这个判断，所以人审那一步不能省。

学完这套，下一步可以试的是把AI审查的结论自动汇总成PR摘要，reviewer打开先看摘要再决定要不要深读。我们正准备往这个方向试。

说句预测的话：AI先审、人后读会成为团队标配，但「会读代码」这个能力反而会更值钱，因为机器替人筛掉了大部分噪音之后，剩下的才是真正需要人判断的东西。

---

本文编译自 Hacker News，原文：AI Broke Code Review and It's Breaking Your Team.
版权归原作者所有，本文为基于公开报道的编译与独立分析。