你们团队用AI做Code Review了吗?效果怎么样?

最近在团队里推AI辅助Code Review,试了半年了来说说感受。

目前主流的AI Review方式大概三种:

  1. PR评论机器人:GitHub Action自动触发,每个PR都过一遍
  2. IDE内Review:Cursor之类的编辑器内置功能
  3. 对话式Review:直接把代码丢给Claude Code聊

我们团队三种都在用,配合下来发现效率确实提升了不少。以前Review一个PR要半小时,现在AI先过一遍后人工只需要10-15分钟。

但也有明显局限:AI看不懂业务逻辑,架构层面的判断也做不了。它擅长的是找Bug模式、安全漏洞、代码规范这些。

想听听大家的实践经验——

  • 你们用的什么工具?
  • Prompt怎么设计的?
  • 有没有遇到AI Review噪音太多的问题?

我们团队从去年Q3开始用AI Review,说下实际感受。

工具方面:试过CodeRabbit、Sourcery、还有自己搭的GitHub Action + Claude API。最后留下来的是自建的方案,因为自定义程度最高,可以针对我们项目的特殊规范做定制。

Prompt设计的关键点:

  1. 明确告诉AI只关注P0/P1级问题(Bug、安全漏洞、严重性能问题)
  2. 代码风格类的东西全交给ESLint/Prettier,不让AI管
  3. 要求给出具体修改建议,不要只说这里有问题
  4. 给AI提供项目的技术栈和关键约定

噪音确实是最大的问题。刚开始的时候AI每个PR都能写20+条评论,开发都烦死了。后来调整了Prompt的优先级策略才好起来。

5 个赞

AI Review最擅长的几个方面:空指针、资源泄漏、SQL注入、XSS这些经典Bug模式。这些它识别得比人准,因为见过海量代码里的同类问题。

最不擅长的:业务逻辑判断。它看不懂你的折扣计算对不对,也判断不了状态转换是否合理。所以我们的策略是AI看技术质量,人看业务逻辑,各司其职。

5 个赞

我个人更喜欢对话式Review,就是把关键代码段丢给Claude Code问具体问题。比如这段并发代码有没有竞态条件、这个缓存策略会不会有一致性问题。

这种定向Review比PR机器人泛泛的扫描有价值得多,但缺点是需要Review的人自己有判断力,知道该问什么。

4 个赞

说个反面经验…我们之前AI Review的Prompt没设好,它把每个函数缺少注释都当成问题报出来。结果开发者对AI Review的评论免疫了,真正有价值的反馈也不看了。

教训就是:宁可漏报不要误报,建立信任比覆盖面更重要。

5 个赞

分享一个Prompt技巧:在系统提示里加一条如果没有发现严重问题只需回复LGTM。这样没问题的PR就不会收到一堆无意义的评论了。

还有一个实用的做法是让AI给问题分级:红色必须修复、黄色建议修改、绿色可以考虑。开发者可以优先处理红色的。

5 个赞

楼上说的分级方案我们也在用,确实好用。但补充一点:AI的分级不一定准,有时候它觉得是绿色的实际上是红色。所以最终还是需要人来把关。

AI Review最大的价值不是替代人,而是帮人提前过滤掉低级问题,让人把精力集中在更高层面的Review上。

3 个赞

好奇大家用Claude API做Review的话成本怎么样?一个月大概多少?我们团队一天大概10-15个PR,担心Token消耗太大。

2 个赞