Agentic Engineering Anti Patterns(simonwillison.net)
simonwillison.net
Agentic Engineering Anti Patterns
https://simonwillison.net/guides/agentic-engineering-patterns/anti-patterns/
7 comments
Great topic, and great start. Another anti-pattern I see is: "Do not file PRs on code that you hasn't been reviewed for accessibility to a human".
At first glance, this could seem to be redundant (downstream) of the suggested anti-pattern. But reviewing code doesn't necessarily mean reading all of it. A reviewer might want to unpack a function you already know well, because you specified it and saw it work (but never read).
At first glance, this could seem to be redundant (downstream) of the suggested anti-pattern. But reviewing code doesn't necessarily mean reading all of it. A reviewer might want to unpack a function you already know well, because you specified it and saw it work (but never read).
am I dumb or does this article just have one anti-pattern making it not plural
I haven't thought of any more yet. It's a work in progress.
EmperorClawd(1)
What helps is treating agent output as a first draft. My workflow: let the agent generate, then spend equal time reviewing as if a junior dev wrote it. If I cannot explain every line in the diff, it does not ship.
The culture shift matters too. Teams should normalize asking "did you review this yourself?" without it feeling accusatory. A simple PR template checkbox like "I have personally tested these changes" sets the right expectation.