r/ClaudeCode • u/ElkEmpty6424 • 22h ago
Built with Claude War story: the hardest part of reviewing AI-generated PRs isn't the volume. it's that the author already left the building.
I review most of my team's PRs. These past few months, we've gone full agentic. Someone fires off Claude Code or Codex, and what lands in the PR is just the diff and PR desc. I've finally put words to why reviewing that feels worse than reviewing a teammate's own work, even when the diff is smaller.
When a colleague actually wrote the code themselves, review had a safety net. If something looked weird I could just ask them: why this way? what did you try first? which edge cases did you weigh? Half of "review" was really loading their reasoning into my head. Then the diff was easy to read.
Now the code came out of an agent session I never saw. The agent made a bunch of decisions on its own, and I don't know which ones. So I can't find the boundary between "my colleague actually thought about this and there's a good reason the code is like this" and "the agent just did it and I should double check." The day it really landed: I had to review a monster 14k-line PR top to bottom like a novel. I had no mental model of where to start, what to look at, or what my teammate had actually thought through. That's not review. It's like inheriting a stranger's code after the stranger has already left the building: the agent that made the calls is gone and I don't have access to its reasoning.
The review bots don't fix it. They flag individual lines, while what I actually care about is understanding: what is this change doing to the system I'm responsible for? Nowadays these bots are like one model writing and another model reviewing, and no one is looking at it.
"just write better specs" doesn't fix it. We do spec-driven development and the specs are good (at least I think so). But a spec is what I asked for; it isn't control over what actually ships. The code in the PR is what goes to production, and I'm the one responsible for it. Staying in control of my own codebase means checking the real thing, not just the intent we wrote up front.
I don't think the answer is "review faster." Or "don't review." Writing got cheap. Reading a change, understanding it, and being able to answer for it is our job, and it's super expensive now. The tools aren't good enough.
Curious how people are actually handling this. Are you:
- reading every AI PR top to bottom (and how are you not underwater)
- trusting tests/CI and full YOLO no review
- making the agent write up its own reasoning in the PR description
- something else?
Full disclosure, because I don't want to pretend I'm just musing: we ended up building something for this. It pulls the agent's session up next to the diff. That's why the problem is on my mind. But I care more about whether it's as real for you as it is for us than about pitching it. So, honestly: is losing that grip on your own codebase something you've made peace with, or something you'd actually pay to make go away? Genuinely curious, I keep assuming everyone just learns to live with it, but maybe not.
