代码评审这件事最近多了个新姿势:先看画面,再读代码。有人把这套做法完整讲了一遍,起因是前一天总有人问屏幕上那张图到底是什么。
这套流程的逻辑很清楚。智能体开出来的合并请求越来越多,光看文字描述不够踏实,UIs 的起点得是一张截图。可就算截图对上了。人还是得弄明白底下那层到底改了什么,脑子里那张系统结构图不能丢。
真正把这件事接住的是 ColdteaAI 的 PR Lens 和 Jev 的配合。前者先把系统的结构摊在画布上铺开,后者负责把合并请求里的每张截图跟被改动的组件一一对上号。评审的人从看得见的画面入手,先看清这次改动落在系统的哪块地界上,再照着影响读代码,读之前心里就已经有了一张图。
顺手记一笔给同行的,这套思路其实不只适用于代码。凡是画面先于文字的场合都能照搬,比如设计稿评审。事故复盘,先摆图再讲理永远比干讲理省力。
我自己拿的看法是,这套做法摸到了代码评审的真痛点。人不是看不懂代码。是懒得在脑内重建上下文。把结构图先摆出来等于省掉了最耗神的那一步。至于截图和组件的匹配准不准,那得在真实的合并请求上多试几轮才知道。仓库在这,github.com/coldteadotai/p… ,团队里天天审代码的可以先试试水。
话题来源 @OhansEmmanuel
603.4K阅读 ❤️1026 x.com/…↗ 已改写,非原文转载
24 浏览 0 评论
0 反应














