【时间之外】AI写了2000行代码，你敢点提交吗？

目录场景模拟真相有点扎心怎么解决小PR到底香在哪几个避坑提醒现在的主要工作不是写代码了但是AI的盛行让我依然关注代码的优化前面有些文章研究了如何用AI帮忙写代码。参见这个系列【开发心得】代码找茬6-吃力不讨好-CSDN博客今天继续讨论AI写代码的事情现在AI已经进化到可以自行写代码的阶段了。但是也带来了一些问题。场景模拟你让AI帮你写个功能它交了一份2000行的PR动了12个文件后端前端测试全覆盖。你盯着diff看了三秒眼花了。算了信任测试点击Approve。恭喜你刚刚把一个定时炸弹送上了生产环境。真相有点扎心但是真相是AI写的代码缺陷率是人类的1.7倍。没看错不是少1.7倍是多1.7倍。而且大PR越大藏的坑越深。据Salesforce的数据AIPR平均干到1000行20个文件review时间直接拉满——然后就 plateau 了。啥意思评审人已经放弃治疗了。怎么解决别让AI一上来就写代码。先让它写计划。具体操作你先写个短spec别偷懒丢给AI让它读完spec后出一个拆分计划给它一个硬指标每个PR不超过100行每个PR只干一件事能独立编译、独立过CI重构和新功能必须分家混在一起就是给reviewer挖坑计划你先审不通过就打回去重拆每个PR开了就让CI跑起来别囤着AI默认不会拆你不逼它它就给你一坨。小PR到底香在哪SmartBear分析了2500个PR结论很粗暴50行的PR比200行的PR多交付40%的代码还更少bug。大PR小PR审查时间天小时回滚成本拆弹专家一键撤销合并冲突家常便饭几乎没有CI信号堵死全队即时反馈审查质量reviewer已读不回认真看完一句话小PR是给人看的大PR是给AI自嗨的。几个避坑提醒⚠️100行是 建议还是看你的代码审查能力和耐心。但是改个名动了50个文件不算大。⚠️永远别把重构和功能改塞一个PR。reviewer分不清这是新特性还是你手滑了。⚠️分支别活太久。活一周就跟main打架活两周就成远古化石。⚠️如果你们review一个小PR要两天……先去修你的review SLA别推小PR了推了也白推。记住AI不会主动 incremental delivery。你不管它它就给你造个巨无霸然后拍拍屁股走了。剩下那个2000行的PR是你的。是你点的Approve。是你的锅。