AI 评审爱给「建议换更现代的写法」这类无关紧要的意见，真正会造成事故的逻辑漏洞，它反而容易漏过去。把它当参考可以，当成标准不行。

有次我让它审一段扣库存的代码，它提了五条，全是「这里可以用 LINQ 简化」「变量名可以更语义化」这种。没一条说到点子上。真正的问题是并发下没加锁，两个请求同时扣会超卖——它一个字没提。我当时差点因为「AI 都看过了」就直接合了。

所以我现在把它的意见当 nits 看，真正要卡的是我自己的 checklist：边界、异常、并发、数据一致性。它适合找格式、命名、明显冗余，不适合判断业务正确性。新人尤其要小心，别把「AI 没提意见」误读成「没问题」。

一句提醒就够：

  • AI 的评审意见标成 nits，可改可不改
  • 并发、异常、数据一致性这类，永远由人来拍板