在我们这个奇特的智能体化工程新世界里,有些行为已经演变成了典型的“反模式”。
将未经审查的代码甩给协作伙伴
这种反模式非常普遍,且极其令人沮丧。
永远不要提交一份连你自己都没审计过的拉取请求(PR)。
如果你提交了一个包含数百行(甚至数千行)由智能体生成的代码的 PR,而你自己却没有下功夫确保这些代码能够正常运行,那么你实际上是在把本属于你的工作转嫁给他人。
既然如此,别人完全可以自己去调教智能体。你作为一个人的价值体现在哪儿呢?
当你把代码提交审核时,你必须确信它已经准备就绪,值得别人投入时间去审阅。第一轮代码审查是你的责任,而不是你应该“外包”给别人的杂活。
一个优秀的“智能体化工程” PR 应当具备以下特征:
-
代码确实可用,且你对此深信不疑: 你的职责是交付可运行的代码。
-
改动范围足够小,可以被高效评审: 这样不会给评审者带来过重的认知负担。多个小 PR 优于一个大 PR;利用编程智能体帮你处理 Git 的繁琐操作,将代码拆分成独立的提交(Commit)是非常容易的。
-
包含额外的上下文信息: 这项改动的更高层目标是什么?链接到相关的 Issue 或技术规范会非常有帮助。
-
认真核对 PR 描述: 智能体能写出看起来煞有介事的 PR 描述,但你也必须审核它们!指望别人阅读连你自己都没读过、没验证过的文字,是非常不礼貌的行为。
考虑到向别人“倾倒”未经审查的代码是如此轻而易举,我建议你在 PR 中提供一些你亲自投入过精力的证明。例如:关于你如何进行手动测试的笔记、对特定实现方案的选择说明,甚至是功能运行的截图或录屏。这些细节能有力地证明:评审者钻研你的代码细节绝不会是在浪费时间。
原文:Agentic Engineering Patterns by Simon Willison
关注公众号「Python之禅」,回复「1024」免费获取Python资源