代码变得容易生成以后,时间会花在读改动上。测试通过只说明跑过的例子过了。不是你写的改动,先看三处,再谈风格。

三处

  • 有没有改你没点名的文件。有,就先问它为什么。说不清,把那些文件退回。
  • 失败时怎么停。接口超时、校验失败、权限不够,要有明确结果,不能当成成功继续往下写。
  • 密钥、令牌、证件号、内部地址在不在改动里。在,就不能合。

风格、命名、多一行注释,放在这三处之后。先挑命名,会把真正的越权改动漏掉。

一次看不完就还不能合

改动长到你无法指出“哪一段是这次要的”,就拆开或拒绝。助手可以再改一版,但合并的人必须能说出这次改了什么。选它动哪一层,见 选 AI 编程工具先看它改的是哪一层。

这周合并前只做这一件事:在改动里指出无关文件、失败路径和有没有密钥。三处有一处说不清,就先不要合。

效率龙虾 会带着下面这段开聊

按文章《不是你写的改动,审查先看三处》把卡点收成可执行步骤:先做什么、别踩哪条、怎么验证。

用效率龙虾试这篇

本文侧重全链路风控方法论。落地时请用自身业务单据做回放验证,不要把示例阈值直接当生产策略。 相关:风控体检 · 方案资源

常见问题 FAQ

为什么生成的改动更要审?

它一次能改很多文件,而且写得像那么回事。没人读,错误会直接进主干。

先看哪三处?

有没有改你没点名的文件,失败时是停还是继续,密钥和证件号在不在改动里。

测试绿能不能合并?

不能单独作为理由。测试绿只覆盖你跑过的例子。没写的失败路径,测试不会替你看见。

改动太长读不完怎么办?

让它拆小,或先拒绝无关文件。一次看不完的改动,就还不能合。

和选工具有什么关系?

选工具决定它动的是哪一层。审查决定这一次动得对不对。两件都要有人做。

谁来审自己让助手写的代码?

你自己先看这三处。看不懂的部分找另一个人。两个人都看不懂,就不要合。

这篇看完可以接着做:工作灵感、发出前验收、试一版稿。