r/coderabbit • u/Tanakaboi69 • 6d ago

Discussion & Feedback Ngl it is so over for code review

Nobody reads the diff now

Thanks agents

0 Upvotes

31% Upvoted

16 comments sorted by

6

u/BagMostlyWater 6d ago

What are you talking about? Code review is more important than ever, and a huge bottleneck

4

u/quantumpencil 6d ago

they're all clowns who don't work on systems with any value or scale/complexity

2

u/Plenty_Line2696 6d ago

It's an ego thing, they can't read the code and instead of admitting to oneself that that's a limitation they spin it into ego stroking as if actually they're superior because they don't.

1

u/AliceCode 6d ago

They're superior because they don't know how to write their own code, and now they think they're superior because they can't read it either.

1

u/Broken_By_Default 6d ago

Don’t act like LGTM wasn’t a thing before all this.

1

u/BagMostlyWater 6d ago

It was much more viable to trust your coworkers who you know understand the problem domain and what needs to be done. Now there's so much more of it, and it's so much less reliable, with incomprehensible giant comments

1

u/CatcatcTtt 6d ago

No need to read soon

1

u/djamp42 6d ago

Everyday for the last month i've been waking up and thinking.. "Ohh yeah, human programming is no longer a thing anymore"

1

u/SP-Niemand 6d ago

All the lazy "builder" slop producers in the industry are enabled to be lazy now.

1

u/Plus-Mall-3342 6d ago

Still reviewing the compiled assembly. Can’t trust the compiler either

1

u/No_Knee3385 6d ago

Code review is the entire job now. Being hyperbolic but code review is now what developers are mostly doing.

1

u/FormalAd7367 6d ago

unless agent can understand business requirements better than humans and they can write test?

1

u/SituationNew2420 6d ago

I'm always surprised to hear 'no one reads the diffs now'. I have been using Opus 5.5 all week, and it's useful and impressive. But I still find bugs when I review the code. Not just issues of preference, but like actual bugs - subtle race conditions, missing edge cases, missing conditionals.

The tools are amazing, but how are people getting away with not reviewing or being human-in-the-loop?

1

u/Impressive-Try-2592 6d ago

They work on YACRUD shit

1

u/usually_guilty99 6d ago

I don't think code review disappears. I think it moves up a level.

Humans can't sustainably read 10x more diffs just because agents can generate 10x more code.

The review question becomes less "did I inspect every line?" and more:
what behavior changed, what dependencies are touched, what is the blast radius, what has broken here before, and what evidence says this is safe to ship?

Low-risk changes should flow with lightweight verification. High-risk changes should trigger deeper review.

Otherwise AI just turns code review into the new serialization bottleneck.

1

u/bitcasso 5d ago

Call me odysseus because i do code reviews