Martinfowler iconMartinfowlerSep 2, 2026 ~5 min source read

Maybe We Shouldn't Be Reviewing All This Code

AI and larger diffs are making human-led code review a bottleneck. The author argues we offload the wrong responsibilities to code review and should move many feedback activities earlier or automate them, reserving human review for exceptions.

Maybe We Shouldn't Be Reviewing All This Code

Share this story

Send the public story page.

Useful takeaways from this story.

Code review has been acting as a multipurpose ceremony—quality gate, mentoring, knowledge transfer, ownership signal, and architectural alignment—which stretches it beyond its strengths.

Shift judgment left: do pairing, mob programming, team design sessions, trunk-based development, and automated checks to shorten feedback loops and capture human learning earlier.

Adopt review-by-exception: reserve human review for large blast-radius changes, security-sensitive boundaries, unfamiliar systems, or agreed low-confidence areas.

# Why reviewing every change doesn't scale

AI and coding agents are producing far more code than humans can practically inspect. Rachel Laycock points to striking reported numbers: at Meta, significant lines of code per human-landed diff rose 106% in a year, and DX's data shows median pull request size grew 64%. That matters because the old pattern—relying on human code review as the central safety and learning mechanism—is breaking under this load.

# What we've been asking code review to do

Teams have piled many responsibilities onto the pull request ceremony: catching bugs, enforcing style, checking security, transferring knowledge, mentoring juniors, distributing architectural understanding, and signaling ownership. That makes each review heavy and slow. When an agent or another developer produces lots of code quickly, every line can queue up waiting for senior attention, creating a bottleneck and delaying value.

# Move feedback earlier (shift left)

If feedback is valuable, place it closer to the decision it informs. Concrete alternatives to using the pull request as the primary learning and alignment moment:

  • Pair programming and mob programming for real-time knowledge transfer and mentoring.
  • Collective design sessions and whiteboard reviews before code is implemented so architectural trade-offs are discussed up front.
  • Trunk-based development and shorter-lived changes to avoid large, monolithic diffs.
  • Automated static analysis, linting, security scanning, and fitness functions for deterministic checks.
  • Encoding constraints as tests or fitness functions so CI can enforce architecture-level rules.

These practices shorten feedback loops and make the thinking of experienced engineers visible while the work is in progress, not after it's done.

# Keep human review, but change when and why it happens

Laycock does not say eliminate human review. She recommends review-by-exception: reserve human attention for the places where human judgment adds unique value. Examples include major architectural changes, anything crossing a sensitive security boundary, changes with a large blast radius, or work in unfamiliar critical systems. Those are the moments where a senior engineer or the team should inspect code together.

# Automate the rest

# Practical implications for teams

  • Rework team workflows so ownership and operation of software are collective, reducing the need for PRs to inform everyone after the fact.
  • Define explicit criteria for when human review is required so review effort focuses on high-impact cases.

# Bottom line

Code review still matters, but it should not be the default mechanism for every type of feedback teams want. Move learning and alignment earlier, automate deterministic checks, and use human review where judgment is essential. That approach addresses the practical bottleneck created by agent-driven code production while preserving the human benefits teams need.

More context around this story.

Avdi iconAvdiSep 2, 2026

Untitled story

Originally appeared on avdi.codes . We’ve spent years loading an extraordinary number of responsibilities onto the humble code review: quality gate, security check, architecture review, mentoring mechanism, knowledge-sharing system, ownership model. Rachel Laycock (rlaycock@thoughtworks.com): Maybe We Shouldn’t Be Revi

Pastel de Nata
Survivefrance iconSurvivefranceSep 21, 2026

Pastel de Nata

Rachel05: Only one ! Go on, be honest Honestly only one, I shouldn’t have any with my blood sugars! Rachel05: Are you like me, on a Seafood (See food) diet ??? I was a bit (OK a lot!) over Christmas, and beyond after stopping smoking early December so managed to put on 20 kg between 2 stopping smokings (other early las

Loading more related stories...

Keep reading in the app

Open the app view to save this story, compare related coverage, and continue from the same source.

Open in app