人工审查是用户阅读智能体生成的代码并作出判断。阅读 diff 或被修改的文件才算审查;只阅读智能体对自己工作的描述不算——叙述不是产物。描述是由被审查方自己编写的二手来源;diff 才是一手来源,审查意味着真正阅读它。
智能体提高了代码产量,因此审查会成为瓶颈。一种有效思路是分层使用不同审查策略。自动化检查捕获机械性故障,自动化审查发现能够明确描述的问题,人工审查则保留给只有你能判断的内容——这项改动是否正确、方案是否适合代码库,以及它是否根本就不该存在。
审查越早,成本也越低。在工作开始前阅读计划,或在过程中查看一份小 diff,只需几分钟;一次 AFK 运行结束后再挖掘完整分支,则耗时得多。把审查节点放在哪里,是一项人工介入决策,而不是事后才考虑的问题。
避免:只说“代码审查”,因为无法区分人工审查还是自动化审查。
用法:
“我对 AFK 输出做了人工审查。”
“你读了 diff,还是只看了摘要?”
“读了 diff。摘要说删除的是死代码,结果那个函数其实被生成文件调用了。”