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

一次PR发现3个隐患:AI写的代码,谁在把关?

0
分享至

那个周三下午,我把一连串改动塞进了一次提交里:重写落地页、适配移动端、新增六篇博客、刷新站点地图。PR提交得很痛快,但痛快之后,那种“谁都没仔细看就合并”的不安很快就浮了上来。

AI辅助写代码的速度一骑绝尘,可一个尖锐的问题随之浮现:代码在抵达真实用户之前,谁来为它做最后的检查?写代码的工具显然不会,因为它已经认为任务完成了——这不是工具的错,任谁写完东西,都还需要另一个人审一眼。


有人说,让人类同事做审查就够了。但在这个提交通常塞了多处改动的节奏里,人工逐行复查不仅费力,还容易漏过跨文件的一致性问题。于是我开始尝试另一种方案:用AI来审查AI生成的代码。Qodo就是那时被我找到的,它不像写代码的AI那样急于收工,而是专门找那些被遗漏的安全问题、跨文件逻辑冲突和持续集成失效点。

连上仓库之后,Qodo做的第一件事就是在PR #138下面贴出一行总结:3个缺陷,0条规则违规,1条合规规则已应用。没有额外配置,它已经索引过整个代码库,这为后面一个关键发现埋下了伏笔——那是只对比差异的工具根本看不到的暗坑。

第一个问题藏着一种AI生成界面里常被遗漏的隐患:在一个a标签上用了target="_blank",却没配上可靠的rel属性。没补上那个卫士,新打开的页面就能通过window.opener拿到原页面的控制权,悄无声息地把原标签页重定向到别的网站。这种攻击叫作反向标签劫持,人工审查偶尔会漏掉,而AI生成代码时几乎从不主动加上那层防护,仿佛它天然不觉得这是个事儿。

第二个问题是站点地图路由断裂:一个本该指向sitemap.xml的链接返回了404。单看这个PR的改动很难察觉,因为路由定义散落在别的文件里。Qodo没有只看差异,而是把改动放回整个代码库里验了一次连通性,才揪出这个跨文件的不一致。如果我用的是只看diff的审查工具,这个断点大概率会流到线上。

第三个是依赖冲突直接把构建打断了。PR里引入了一个库,版本号跟锁文件里另一个包的要求刚好顶住。CI日志里报了错,但直到Qodo把堆栈追溯和代码改动并排标出来,我才意识到那个冲突点就在这次提交里。

这三个发现串在一起,刚好把一场辩论的双方摆上了桌:正方说AI写代码效率极高,应该让人从琐碎审查里解脱出来;反方则担心没有可靠审查环节,就等于把生产环境的安全交给运气。从这次经验来看,双方都没错——错的是把写代码和审代码混为一谈。写代码的AI是油门,但一辆车不能只有油门,还要有刹车和仪表盘。

Qodo在这个PR里扮演的就是那套仪表盘:它不参与生成,而是专门检查安全把控、跨文件一致性和CI就绪状态。这跟现有的文档机器人、安全扫描机器人角色完全不同——那些工具各自为战,而一个理解代码库整体结构的审查工具,才能把碎片化的风险拼成完整的画面。好坏代码的评判,不只在提交的几行之间,更在那些改动的涟漪能扩散到的所有角落。

如果你也在把大量生成代码直接推向生产,不妨在代码审查环节划分出两个清晰的工种:生成工具负责“造出来”,审查工具负责“查明白”。这层分工一旦确立,AI辅助开发的节奏才能真正既快又稳。

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

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.

相关推荐
热点推荐
这才是黑马!3战全胜,进6球丢0球独一档,40岁门神追平梅西C罗

这才是黑马!3战全胜,进6球丢0球独一档,40岁门神追平梅西C罗

萌兰聊个球
2026-06-25 13:27:04
菲律宾声称其主力护卫舰在黄岩岛附近海域与中国海空力量发生对峙,国防部:中方在黄岩岛开展包括科考在内的任何活动,都是主权国家的权力

菲律宾声称其主力护卫舰在黄岩岛附近海域与中国海空力量发生对峙,国防部:中方在黄岩岛开展包括科考在内的任何活动,都是主权国家的权力

每日经济新闻
2026-06-25 17:26:11
广德车祸事件进展:知情人称3岁男孩“脑死亡”,36岁女司机曝光

广德车祸事件进展:知情人称3岁男孩“脑死亡”,36岁女司机曝光

老猫观点
2026-06-24 17:12:27
“走个面”再升级!网友深扒:半只脚踏入美国的人,教我们抓特务

“走个面”再升级!网友深扒:半只脚踏入美国的人,教我们抓特务

火山詩话
2026-06-25 05:08:44
当“太妹”遇到“穷鬼”,于是就有了“13岁女孩被强奸却不立案”

当“太妹”遇到“穷鬼”,于是就有了“13岁女孩被强奸却不立案”

刘既济
2026-06-25 16:20:45
特控线是什么线?和一本线有什么区别?

特控线是什么线?和一本线有什么区别?

王姐懒人家常菜
2026-06-25 11:32:58
这不妥妥像爸爸!徐朵不再瞒曝私密事,徐帆净身出户传闻真相大白

这不妥妥像爸爸!徐朵不再瞒曝私密事,徐帆净身出户传闻真相大白

橙星文娱
2026-06-25 10:29:21
博主怒批《抓特务》:整个电影都是私货堆起来的,怎么过审的?

博主怒批《抓特务》:整个电影都是私货堆起来的,怎么过审的?

小徐讲八卦
2026-06-25 16:41:12
随着巴西3-0,摩洛哥4-2,世界杯死亡半区浮现:阿根廷队运气爆棚

随着巴西3-0,摩洛哥4-2,世界杯死亡半区浮现:阿根廷队运气爆棚

慢歌轻步谣
2026-06-25 12:42:50
981天,146场缺阵:内马尔14分钟告诉世界,什么叫“迟到的英雄”

981天,146场缺阵:内马尔14分钟告诉世界,什么叫“迟到的英雄”

新姐看世界
2026-06-25 15:41:55
“欠下”10000亿!雷军做什么都是错的

“欠下”10000亿!雷军做什么都是错的

鸣金网
2026-06-24 20:20:42
71岁卢卡申科长期离境!交往22岁嫩模,让他开始相信爱情?

71岁卢卡申科长期离境!交往22岁嫩模,让他开始相信爱情?

大江看潮
2026-06-25 08:32:25
悲催!57岁领导突发脑梗离世,办公室被清空翻新,仿佛他从未出现

悲催!57岁领导突发脑梗离世,办公室被清空翻新,仿佛他从未出现

火山詩话
2026-06-25 07:53:57
官方发布!四川省2026年普通高等学校艺术体育类招生录取控制分数线

官方发布!四川省2026年普通高等学校艺术体育类招生录取控制分数线

四川省教育厅
2026-06-25 19:42:58
4年前花21亿买下“废铁”,如今江苏首富让欧美船东跪着求订单

4年前花21亿买下“废铁”,如今江苏首富让欧美船东跪着求订单

毒sir财经
2026-06-25 15:07:38
给韩国队算分!世界杯小组第3最新形势:6队同分,比利时无缘前8

给韩国队算分!世界杯小组第3最新形势:6队同分,比利时无缘前8

球场没跑道
2026-06-25 12:07:52
日本强震震级上调至7.2级 已有4人受伤

日本强震震级上调至7.2级 已有4人受伤

中国网
2026-06-25 12:02:08
过期一天就绝对不能碰的10种危险食物,第一名你可能经常舍不得扔

过期一天就绝对不能碰的10种危险食物,第一名你可能经常舍不得扔

娱乐的硬糖吖
2026-06-24 17:00:21
形势有多严峻?网传现在很多医院出现经营困难,发不出工资…

形势有多严峻?网传现在很多医院出现经营困难,发不出工资…

慧翔百科
2026-06-25 17:32:14
白俄罗斯关闭信号中继器,卢卡申科准备离开白俄罗斯

白俄罗斯关闭信号中继器,卢卡申科准备离开白俄罗斯

山河路口
2026-06-25 09:05:47
2026-06-25 20:16:49
灰度测试中
灰度测试中
生活正在重构,目前还在灰度测试阶段,暂不全量发布。
262文章数 40关注度
往期回顾 全部

科技要闻

宇树机器人大降价

头条要闻

特朗普又点名抱怨意英德法西五国:我很失望

头条要闻

特朗普又点名抱怨意英德法西五国:我很失望

体育要闻

世界杯最动人一吻:我若离世 你就改嫁吧

娱乐要闻

这国产剧太装了,居然还热播第一?

财经要闻

又有纸尿裤送检后被检测出甲酰胺!

汽车要闻

东风奕派纳米06智趣版上市 指导价9.99万元起

态度原创

手机
时尚
旅游
数码
军事航空

手机要闻

vivo参展MWC26上海:X Fold6折叠屏新机首秀

最高级的夏季配色,来了!

旅游要闻

卡皮巴拉、羊驼、逗号鸟都来了!上博“美洲‘萌萌TA’奇妙夜”夏夜开逛

数码要闻

三星与LG将于越南生产iPhone 18 Pro OLED模块 供应量或超8000万片

军事要闻

特朗普:现在到了关注朝鲜问题的时候了

无障碍浏览 进入关怀版