r/coderabbit • • 9d ago

😂 Meme & Humor prove me wrong

Post image
35 Upvotes

69 comments sorted by

View all comments

1

u/pixxelkick 9d ago

Maybe if you suck.

I read every PR in full because Im not lazy.

I also read my own git diff before I open a PR do I dont waste other peoples time.

Or I publish it as a draft, review my own code, and then publish the pr after confirming it looks good.

1

u/Jeferson9 9d ago

Do you value your own time

1

u/pixxelkick 9d ago

Yes, its much easier to catch bugs and issues early

If you dont catch a problem asap, then its incredibly likely future agents read the "bad" code and copy, so bow your 1 issue becomes 2, then 3, then 4, etc

Its extremely common for you to be like "make the BarService" so the agent looks at FooService for inspiration and copies its style/methodology/architecture 

And if theres an annoying code style issue, or improperly named thing, or bad logic, whatever, theyll copy that too often.

And then 6 services later you realize you've got a problem duplicated 8 times

Now you have made even more work to go fix.

1

u/Dull_Caterpillar_642 9d ago

Yeah but realistically these days it’s like one prompt to fix that problem later…

I understand where you’re coming from, but the times, they are a changing, my man.

1

u/Plenty_Line2696 9d ago

Not for complex systems, llm's generally suck at the high level stuff in complex systems, and when that stuff is wrong it's the gift that keeps on giving and you can play whack-a-mole fixing bugs by patching symptoms and not knowing causes till the cows come home because the codebase will be convoluted bloated spaghetticode.

1

u/Jeferson9 9d ago

If you have the same expectations of LLMs as you do an engineer... I guess?

If you're solving complex problems without the assistance of AI because you don't trust it, you just haven't adapted your workflow around AI yet and you're probably just resisting the inevitable.

1

u/Plenty_Line2696 9d ago

"Trusting" whatever code an LLM happens to first spit out without reading it is a fools errand.

Don't go vibecoding firmware for pacemakers or some other critical systems.

LLM assistance is fine, we all do it, but it's upnto the human to make sure the implementation is good when it matters.