前端代码评审(从代码爆炸到评审革新,Cursor与Graphite的强强联合)

前端代码评审(从代码爆炸到评审革新,Cursor与Graphite的强强联合)
从代码爆炸到评审革新,Cursor与Graphite的强强联合

财经摆渡人

精研出品

破浪前行,共探财富新局


大家好欢迎收看【古今财鉴】

12月19日,AI编程圈扔出个重磅消息Cursor宣布收购代码评审工具Graphite。

这事儿乍一看有点突然,一个做AI编程工具的,为啥非要花大价钱买个评审工具?如果你最近在用AI写代码,可能就懂这背后的门道。

AI编程工具这两年跑得飞快。

不管是写前端页面还是后端逻辑,对着AI助手说几句需求,代码唰唰就出来了。

工程师们的产出量,跟以前比简直是两个量级。

但有个环节,却被远远甩在了后面代码评审。

代码评审,AI编程时代的“堵车现场”

以前工程师一天写500行代码,评审可能花1小时。

现在AI帮忙,一天写1000多行都轻松,评审时间却没见少。

攒了一周的代码堆到评审环节,评审的人看着屏幕直发懵。

这就像高速路突然变成单车道,不堵才怪。

更麻烦的是评审质量。

大家都忙,打开同事的代码,扫两眼没大问题就回个“LGTM”(看着还行),成了不少人的常态。

本来想仔细看看逻辑漏洞,后来发现手头还有三四个评审等着,只能草草了事。

这种“走过场”的评审,在AI生成代码的时代更危险AI写的代码有时看着漂亮,细究起来全是坑。

Cursor自己也遇到了这问题。

去年有个叫Theo的T3Chat创始人吐槽,用Cursor写代码时,明明AI提示“没问题”,结果上线后bug一堆。

后来Cursor团队不得不进入“硬核修bug模式”,天天加班改代码。

那阵子圈内都调侃,“Cursor这是买个codereview公司来给自己修bug啊。

”虽是玩笑,却戳中了痛点AI能写代码,但管不住质量,等于白搭。

这问题咋来的?说白了,AI把“写代码”的门槛降低了,却把“保证代码质量”的压力全推给了评审环节。

以前是人写代码、人评审,节奏能对上;现在AI写代码、人评审,节奏彻底乱了。

Graphite的出现,就是想把这节奏重新捋顺。

Graphite的“疏通秘籍”,从Meta学来的stackeddiffs

Graphite是2021年由几个前Apple、Meta的工程师搞起来的。

一开始他们想做移动应用QA工具,后来发现代码评审才是真痛点。

2024年3月拿了5200万美元B轮融资,服务的公司包括Shopify、Snowflake这些大厂,数万名工程师在用。

能被Cursor看上,靠的就是他们从Meta学来的“独门秘籍”stackeddiffs。

前端代码评审(从代码爆炸到评审革新,Cursor与Graphite的强强联合)

这东西听着玄乎,其实就是把传统的代码提交方式换了个思路。

以前用Git,大家习惯搞个PR(PullRequest),攒一堆代码一次性提交评审。

stackeddiffs不这么玩,它把最小单位切成单个commit(提交)。

比如开发一个功能,今天改登录逻辑,单独提交一个commit评审;明天改支付接口,再提交一个。

每个commit独立评审、独立合并,不用等整个功能做完。

本来想按老规矩攒个大PR,后来发现拆成小commit后,评审的人压力小多了。

上午提交的代码,下午就能审完,有问题马上改,不用等一周后全忘了当时咋想的。

而且小步快跑,代码冲突也少了就像开车,频繁小转弯总比一把大转弯容易控制。

光有方法还不够,Graphite的AI评审也有讲究。

它有个死规矩,最多评审100行代码。

超过这个数,AI直接“罢工”。

为啥?因为代码越长,AI越容易漏掉细节。

而且它给的评论不是空话,比如“这里可以优化”,而是具体到“把for循环换成map函数,可读性更高”。

最有意思的是它的“沉默原则”不确定的地方绝对不说话。

这“不瞎指挥”的态度,比很多爱抢话的AI工具强多了。

从“写代码”到“管代码”,收购背后的野心

Cursor收购Graphite,可不是简单买个工具。

Graphite会保持独立运营,但会和Cursor深度集成。

想象一下,你用Cursor在本地写代码,写完按个键,代码自动切成小commit,直接推到Graphite的评审系统;评审通过后,自动合并到主分支。

整个流程从“写代码-等评审-改代码”变成“写一点-审一点-合一点”,中间几乎没停顿。

他们还打算把Graphite的AIReviewer和Cursor的Bugbot捏到一起。

Bugbot负责在写代码时抓小bug,AIReviewer负责在评审时看逻辑漏洞。

这俩要是真能配合好,说不定能成业内最强的“代码守门员”。

这事儿的意义不止于两家公司。

代码评审这东西,从IBM的FaganReview到Linux的邮件patch,再到GitHub的PR,几十年没变过核心玩法。

现在AI来了,逼着行业不得不改。

Graphite挖出来的stackeddiffs,其实是把Git里藏了多年的“堆叠视角”重新捡了起来Git本来就能处理多个commit堆叠,但大家以前没用好。

未来的开发流程可能会变成这样,资深工程师出设计文档,AI根据文档写代码,人负责拆成小commit,Graphite的AI和人一起评审,最后合并上线。

agent(智能体)在中间跑腿,人掌控开头和收尾。

这种模式下,“设计文档先行”可能会重新流行毕竟AI能写代码,但写不出靠谱的架构逻辑。

说到底,AI能帮我们写代码,但把好最后一关的,还得靠人加上靠谱的工具。

Cursor收购Graphite,更像是给AI编程时代的“代码质量”买了份保险。

至于stackeddiffs能不能成为新潮流,还得看更多公司用不用。

但至少现在,有人已经开始动手解决那个最刺眼的“堵车点”了。

这事儿,可能才刚开始。

文章版权声明:除非注明,否则均为边学边练网络文章,版权归原作者所有

相关阅读