网易首页 > 网易号 > 正文 申请入驻

AI写的PR动辄600行,为什么逐行读diff抓不到问题?

0
分享至

审PR已经成了我一周里最难受的环节。不是因为代码烂,大部分读起来都还行。是量太大了,而且很多是AI写的。作者一个下午就能产出一个600行的PR,而理解它的人还是我。


后来我想明白一件事:逐行读diff,根本抓不住AI代码真正会犯的错。

出问题的从来不是某一行。行本身通常没问题,滑掉的是结构:

  • 新写了一个helper,而两个文件夹之外早就有一个一模一样的
  • 函数能跑,但完全不管代码库其他地方是怎么做同一件事的
  • 改动今天是对的,等数据量涨起来那天就是灾难
  • 测试是绿的,但它测的根本不是该测的东西

这些在diff里一个都看不见。只有把改动周围的代码一起看,才会浮出来。

所以我现在审PR,问的问题变了。不再是"这一行对不对",而是:

  • 它合不合群?跟代码库现有的写法一致吗?
  • 谁在调它?这一改会崩掉什么?
  • 测了吗?测的是要紧的那件事吗?
  • 扛得住吗?不只是"能跑",是数据涨了还站不站得住?
  • 是不是已经有了?我们是不是在重复造一个现成的东西?

这五个问题,在diff页面上一个都答不了。但在IDE里,每一个都只差一次点击:跳转到定义、查找引用、打开测试。

于是我做了个工具,把整个review搬进编辑器。

Review Lens是JetBrains IDE的免费插件,支持IntelliJ IDEA、PyCharm、WebStorm、GoLand。选一个PR,它就在你已经开着的窗口里打开。

你像读自己的代码一样读它:完整文件,PR的改动标在行号槽里。跳转定义、查找引用、跑测试,全都照常可用。

讨论会自己找上门。每个人在GitHub上的评论就挂在对应的代码行下面,你直接在原地回复,不用开浏览器。

阅读进度不会丢。读一个文件勾一个,勾选状态和GitHub同步;你读完之后文件又被改过,会被标出来。

只提交一次。点完成审阅,你的评论和结论作为一次review发到GitHub。

手头的工作是安全的。打开PR前你正在做的事会被搁到一边,审完再回来。

它不只用来审别人的代码。先读自己的PR:在请人审之前,打开自己的PR,一个commit一个commit地过一遍。你会看到它是怎么长出来的,也能在审阅者之前抓住结构上的错误。对任何交付AI辅助代码的人,这是我最推荐的习惯。

跟进新推送也一样。你审完之后如果有人又推了东西,PR会提示你,你只需要审新增的部分。

整个仓库一眼看完。所有开着的PR都在IDE里的一个列表里,等你的排在最前面,每个PR的评论点一下就到位。

想试的话,在IntelliJ IDEA、PyCharm、WebStorm或GoLand(2026.1+)里:Settings → Plugins → Marketplace → 搜索"Review Lens"。需要装好git,并用GitHub CLI登录(gh auth login)。插件在JetBrains Marketplace上架,源码在GitHub,Apache 2.0协议。

最后想问问:你在AI写的PR里最常抓到的结构错误是什么?我先来——重复的helper,每次都是。

特别声明:以上内容(如有图片或视频亦包括在内)为自媒体平台“网易号”用户上传并发布,本平台仅提供信息存储服务。

Notice: The content above (including the pictures and videos if any) is uploaded and posted by a user of NetEase Hao, which is a social media platform and only provides information storage services.

相关推荐
热点推荐
男子家中漏水花6800元找人上门补漏,维修人员灌了100多斤胶水,结果房顶被钻穿,下雨后漏水问题加重;包工头承认错误:退4000余元

男子家中漏水花6800元找人上门补漏,维修人员灌了100多斤胶水,结果房顶被钻穿,下雨后漏水问题加重;包工头承认错误:退4000余元

北青网-北京青年报
2026-10-20 14:37:08
Skip开怼奥尼尔:凯斯勒12中12就成最强中锋?文班13中1

Skip开怼奥尼尔:凯斯勒12中12就成最强中锋?文班13中1

星河漫山野
2026-10-22 13:03:14
美媒曝希罗猛烈抨击孩子母亲!两人曾超甜蜜 她还被称为翘臀女王

美媒曝希罗猛烈抨击孩子母亲!两人曾超甜蜜 她还被称为翘臀女王

Emily说个球
2026-10-22 10:18:07
难怪五胡乱华后北方只剩4万汉人,你不看胡人有多残暴,连人都吃

难怪五胡乱华后北方只剩4万汉人,你不看胡人有多残暴,连人都吃

史笔似尘钩
2025-02-10 23:23:05
开拓者124-83大胜太阳!利拉德25分,杨瀚森5+2,创NBA生涯里程碑

开拓者124-83大胜太阳!利拉德25分,杨瀚森5+2,创NBA生涯里程碑

球场没跑道
2026-10-22 12:35:21
和卡里克一起走人!曼联灾难水货该清理!8000 万豪砸彻底打水漂

和卡里克一起走人!曼联灾难水货该清理!8000 万豪砸彻底打水漂

澜归序
2026-10-22 09:30:25
当不成巴西总统?博索纳罗对华表态激起千层浪,头号敌人还有底牌

当不成巴西总统?博索纳罗对华表态激起千层浪,头号敌人还有底牌

经纬戎韬
2026-10-15 00:15:02
傻白甜女星酒局上陪大佬!

傻白甜女星酒局上陪大佬!

八卦疯叔
2026-10-22 09:45:06
影视飓风测评iPhone Duo:硬件结构强度非常高,不用担心坐弯或者坐断;但屏幕容易沾指纹,污渍很难擦干净

影视飓风测评iPhone Duo:硬件结构强度非常高,不用担心坐弯或者坐断;但屏幕容易沾指纹,污渍很难擦干净

第一财经资讯
2026-10-22 01:14:09
有一个长不大的妈妈是什么体验?网友:好时髦,我都没玩过!

有一个长不大的妈妈是什么体验?网友:好时髦,我都没玩过!

夜深爱杂谈
2026-10-14 21:30:47
伟大的绝杀!终结美国,力压日本,中国是冠军!人民日报发文赞誉

伟大的绝杀!终结美国,力压日本,中国是冠军!人民日报发文赞誉

大秦壁虎白话体育
2026-10-22 11:38:16
丰田“不讲武德”!中型SUV只卖11.38万,高配630km续航+激光雷达

丰田“不讲武德”!中型SUV只卖11.38万,高配630km续航+激光雷达

趣味萌宠的日常
2026-10-21 00:54:54
小米新车谍照曝光,对标尊界S800,不是迈巴赫,是中国自有的豪车

小米新车谍照曝光,对标尊界S800,不是迈巴赫,是中国自有的豪车

一文热点捕手
2026-10-20 21:02:37
悲惨!因辅导作业,情绪失控,母女相继跳楼,双双坠亡!

悲惨!因辅导作业,情绪失控,母女相继跳楼,双双坠亡!

闲侃闲侃
2026-06-25 07:43:49
杭州警方:对浙江盘石信息技术股份有限公司及其关联公司涉嫌合同诈骗立案侦查,对田某等人采取刑事强制措施

杭州警方:对浙江盘石信息技术股份有限公司及其关联公司涉嫌合同诈骗立案侦查,对田某等人采取刑事强制措施

环球网资讯
2026-10-22 11:37:21
韩国怒了!

韩国怒了!

陆弃
2026-10-21 11:26:06
赛力斯张兴海:安全是车企必须无条件扛起的第一责任

赛力斯张兴海:安全是车企必须无条件扛起的第一责任

贝壳财经
2026-10-21 21:42:07
毛不易在好友婚礼上发言,一句“结婚对咱们这个年龄来说,越来越不意味着什么”引争议

毛不易在好友婚礼上发言,一句“结婚对咱们这个年龄来说,越来越不意味着什么”引争议

韩小娱
2026-10-22 08:39:12
黄景瑜未成名地铁旧照,模样普通认不出

黄景瑜未成名地铁旧照,模样普通认不出

喜欢历史的阿繁
2026-10-20 02:15:38
国庆花坛5000盆鲜花赠市民

国庆花坛5000盆鲜花赠市民

北青网-北京青年报
2026-10-22 00:48:24
2026-10-22 13:44:49
Ping值焦虑
Ping值焦虑
有态度网友ytd
240文章数 109关注度
往期回顾 全部

科技要闻

腾讯云谷歌紧急行动!互联网安全出大事

头条要闻

牛弹琴:泽连斯基突然接受保留75%领土 释放一个信号

头条要闻

牛弹琴:泽连斯基突然接受保留75%领土 释放一个信号

体育要闻

热闹的NBA回来了,他却倒在新赛季大门口

娱乐要闻

杜淳王灿近况曝光:铁婚六年甜上热搜

财经要闻

为卖房,有房企花20万拍短剧

汽车要闻

有七座/续航提升 领克900L曝光还可选双色车身+大饼轮毂

态度原创

旅游
房产
健康
教育
时尚

旅游要闻

最是橙黄橘绿时,南京“最美600米”斑斓上线

房产要闻

2026国际新消费(医疗)旅游服务出海论坛在海口盛大举办

痘痘能根治吗?皮肤科医生说实话

教育要闻

日常英语:在公共场合注意自己播放的音量

中年男人,开始抢着打针回春

无障碍浏览 进入关怀版