审查关卡:合并前的一份清单
审查关卡(review gate)就是一份在改动上线前必须为真的简短清单。自己过一遍,或者让 AI 逐项确认:
- 我读了 diff,并理解改了什么、为什么改
- 改动被限定在我实际要求的范围内
- 测试通过(且新行为有对应的测试)
- 类型检查和 linter 都是干净的
- 没有遗留密钥、调试日志或被注释掉的代码
- 没有安全检查被悄悄移除
- 我自己试过这个改动,至少跑过 happy path
如果有哪一项你勾不上,那你就还没做完。
这份清单刻意保持简短,因为一个你会跳过的关卡,根本就不算关卡。尤其是当你跑得很快、改动"看起来没问题"时,会有一种直接放行的诱惑。对任何要面向用户的东西,请抵抗它。一个有用的小技巧,是让 AI 替你来填——把清单贴过去,然后说:"针对你刚做的改动,把这份审查关卡逐条走一遍。每一项,给出 PASS 或 FAIL,并附上一行证据。" AI 没运行过测试就没法诚实地写下"测试通过",而强迫它逐行给出证据,会把一句含糊的"看着不错"变成一个你可以审计的东西。