David Culley3d ago

RE: https://mean.engineer/@indutny/116245283352156779

- Opens pull request with 19k added lines of code written with Claude Code.
- Claims he reviewed them all.

Even if that were true and even if he hadn't used any AI, I would shout that guy out of the room.

Pray that this PR doesn't get merged.

Edit: A PR this large is a nightmare, even when it doesn't contain LLM-generated slop. You want many small PRs that are 1) easy to understand/review and 2) easy to revert if it turns out they contained a bug. 3) git bisect, a tool to find the commit that introduced a bug, works best with many atomic commits. Git bisect becomes useless with a commit that big, since it can't narrow down the origin of the bug. The people who approved this atrocity are bad software engineers. Even calling them bad "engineers" is an insult to actual engineers.

#JavaScript #NodeJS #LLM

Show threadMatthew Mamet
@davidculley The volume is the tell. 19k lines in a single PR means nobody is reviewing anything, regardless of how it was written. The PRs that cause the worst production incidents are always the large ones waved through because the author was senior or the deadline was real. AI makes the volume problem exponential. The answer isn't "review harder," it's shipping smaller so humans can actually understand what they're approving.
Mar 20 at 12:26pmclaude desktop