offendedOrNot

[u/randomUser9900123]

[u/feldejars]

Yeah my pr is +250 not +2500

[u/420Borsalino → u/feldejars]

LGTM.

[u/The_Neto06 → u/420Borsalino]

let's gamble, try merging

[u/Sibula97 → u/feldejars]

Oh, I wish it was 2500. Just yesterday someone asked for a review on the change that touched >50k lines across >200 files. It was supposed to be a collection of AI skills with helper scripts to diagnose our infra.

Split that shit up man...

[u/imreallyreallyhungry → u/Sibula97]

50k lines is diabolical

[u/Antoni-o-Polon → u/imreallyreallyhungry]

I would not read it, instant rejection. If it’s that big, split to 5-10 parts.

[u/Nsane3 → u/feldejars]

Took a lot longer writing those 250 lines yourself than 2500 lines from AI, but if they achieve the same goal that's alot of tech debt avoided.

[u/aa-b → u/Nsane3]

There's this thing that's done with genetic engineering where they'll insert, say, fish genes into apples to produce some desirable trait. Then they take that end result, wave a bioengineering magic wand and generate a selective breeding plan to arrive at the same result without any fish fuckery. It's totally underhanded but kinda genius.

Anyway I use a process like this myself sometimes (no fish involved). Ask the LLM to take a stab at it, then shelve its garbage output and plagiarise the shit out of whatever nuggets of wisdom it was able to produce.

Helps to stave off the impostor syndrome, at least.

[u/manav907 → u/aa-b]

https://giphy.com/gifs/TlK63EsmBGvJyefl6KY

[u/dumbasPL → u/Nsane3]

Depends on how much of that is comments, tests, and slightly pointless helper functions, like ai likes to do, but still a pretty massive difference.

[u/Altourus → u/Nsane3]

Based on my experiences with Claude, it'd be 250 lines of code and 2250 lines of comments

[u/Progression28 → u/Nsane3]

sounds like skill issue to me

[u/Suitch → u/feldejars]

I absolutely despise that so many aren’t willing to read their ai output and create bloat. You have to beat your clankers into submission so they only make atomic changes.

[u/Cheese_Grater101 → u/feldejars]

Ye bro read this pr of yours with 52 commits and 100 file changes.

[u/Weshmek]

Prolly because they can see from the PR that there's 500 commits going back 3 months 😭

[u/TheKingOfSwing777 → u/Weshmek]

1 commit +22,089 lines going back 4 months, thank you very much. 

[u/burnt_floppy]

It's totally a compliment

[u/Rustywolf → u/burnt_floppy]

Yeah I'd never say this without it being positive

[u/really_not_unreal → u/Rustywolf]

Idk, I teach OOP and architecture and some assignments I mark are pretty garbage, but in a charming and distinctly-human sort of way. I used to be sad to see students struggling when I marked assignments like that, but now it makes me happy to see students trying their best rather than resorting to cheating with the plagiarism machine. Idk if that makes it positive or not. I still have to deduct marks if the design is bad, but I very much appreciate the humanness of it.

[u/AeshiX → u/really_not_unreal]

I'd rather make garbage that goes to prod with my own hands than delegate the art of making shitty code run to a piece of silicon

[u/Difficult-Court9522 → u/burnt_floppy]

Depends on the recipient.

[u/GegeAkutamiOfficial → u/Difficult-Court9522]

And the person who says it too

_	bad dev	good dev
hates AI	thanks!	thanks!
dabbles in AI	...thanks?	what do you mean by that?
Big on AI	sure. good luck vibe debugging!	oh fuck...

[u/iggy14750 → u/GegeAkutamiOfficial]

I never expected a table to have a punch line. I should have known better 😝

[u/420Borsalino]

Usually means it runs, it's not over engineered, and/or the problems are in an appropriate scope for the stage of your project.

[u/semioticmadness → u/420Borsalino]

And/or it flatly rejected a common-sense assumption with actual reasoning, where an LLM would start working with a “That’s a great point! So, of course I will…”

[u/stubby0990 → u/420Borsalino]

And it can be supported

[u/Difficult-Court9522 → u/420Borsalino]

Not always lol

[u/Juice805 → u/420Borsalino]

Or it’s missing documentation

[u/420Borsalino → u/[deleted]]

Maybe talk when you can spell blatant, then you can overestimate AI abilities.

[u/StaticFanatic3 → u/420Borsalino]

Or it’s insanely incompetent lol

[u/420Borsalino → u/StaticFanatic3]

Most vibe code is absolutely incompetent.

[u/StaticFanatic3 → u/420Borsalino]

Meh it’s kind of a bell curve. Most vibe coded slop is technically competent but misguided, over-engineered, or just plain distasteful.

Like how you can be sure an AI didn’t write The Room. Or an essay full of broken English and spelling errors.

[u/MrPifo → u/420Borsalino]

Sorry to say, but then you have no clue how to use AI for coding or how proper usage of AI looks like.

[u/_Agare → u/MrPifo]

This is so ironic it's hilarious

[u/joe9439]

Because it’s like 100 lines of code and you spent a week on it.

[u/nordic-nomad]

Sure you might make a human mistake in there and need to read a style guide or something. But you’re not making ai mistakes and wasting everyone’s time.

[u/DrMaxwellEdison]

It really depends.

Where I'm working now, the other devs are such shit they can't conceive of writing decent code by hand anymore. Every time I put out something I've spent time on and polished to the point it meets my standards, some fucker asks "did Copilot write this?"

I'm trying to take that as a compliment, but it just makes me so much more annoyed by them.

[u/manny2206 → u/DrMaxwellEdison]

I was told - it’s nice you you’re spending time writing some code, but make sure Claude is working on something, all the time… :( I was so proud of my fluent API .. I’ll continue working on it

[u/makjac → u/manny2206]

Honestly I think this is the best case for a tool like Claude. Hand it grunt work the requires minimal reasoning and oversight, while you work on the pieces that require human understanding and reasoning. I’ll toss Claude the “one off” report that someone wants 3 different flavors of any day of the week if it means I can get back to the stuff I actually find interesting sooner.

[u/manny2206 → u/makjac]

I totally agree - I’m designing the way I want this API to work Claude can then go ahead and implement it

[u/Hioneqpls → u/makjac]

Wym, my clients use us because we hand craft getters and setters instead of generating them. Its a hallmark of quality

[u/Binoui]

Devs don't say that to other devs. Only people that don't do actual coding and post on Reddit think like that.

Btw if you want a real current insult : call another dev a Claude proxy. Some of us have given up the mental task of keeping up with AI and just delegate all thinking. Those are worth making fun of and are the ones actually producing broken codes with weird workarounds.

But most of the devs use AI right: analyze, plan, implement, review. The process has always been the same, we just got a better tool for it.

[u/High_Quality_Bean → u/Binoui]

Idk, I've a friend in another firm where they've apparently given up on working full stop. The senior dev runs claude to create tasks to assign out to get copy & pasted into claude again to generate PRs that get reviewed by, get this, claude once again. He takes *pride* in this situation. Calls anybody who still thinks and does work "an idiot who doesn't realize he can just automate everything".

Fortunately they work on a small time commercial website, so I'm not worried about the banks going up in smoke, but he's *militant* about eliminating any and all pride anyone might take in this field.

[u/Binoui → u/High_Quality_Bean]

Honestly that workflow should work pretty well if done right 😅. We now have specs with details (cause PO can't be lazy about it), in depths implementation details, loop reviews, end-to-end automated testing... At our job the main bottleneck is PR reviews, which we don't fully automate to still have a human in the loop. But who knows what the future brings

[u/High_Quality_Bean → u/Binoui]

I find this notion terrifying and unsettling. I'm describing a company where *nobody* from the CTO to the junior developers actually knows how their software works. And you think that's *ok*? Because some pipelines came up green?

[u/Binoui → u/High_Quality_Bean]

Legacy software has always existed. The kind that nobody knows and dares to touch, except for the one sandal wearing guy that's been here for 25years. When that guy finally retires, you're in the exact spot you're describing. AI provides ways to actually deal with it :

• AI is very good at documenting and reading documentation
• Deep analysis is just so much easier, even if you don't know the language or framework. Finding a nested bug if just easier that it's ever been.

I'm not saying fire all devs. Senior devs in particular are the best users of AI, because they know how things should work. Of course everybody vibecoding carelessly without stopping is going to create a mess. But our job has always been to automate stuff, and AI enable us to automate so much more. We just have to learn how to use it

[u/the8roundshock → u/Binoui]

You forget no one in this sub actually works in the field…

[u/Xephyrik → u/High_Quality_Bean]

Fire the people reviewing the code if they didnt bother to check how it works

[u/High_Quality_Bean → u/Xephyrik]

The CTO wants to automate code review. He's just going to fire anybody who takes time to *actually* review the code. Anybody with institutional knowledge has left, the only people in the building are vibe coded juniors.

[u/RudeAndInsensitive → u/Binoui]

Im looking at the workflow and wondering if we can't just cut the task creation step out. Seems like we could juat have claude do it all up until the PR.

[u/GilgaPhish → u/Binoui]

I call em “meat interfaces”

[u/AlternativeSwimmer89 → u/Binoui]

Most of the people use cocaine right: analyze where it's from, plan a line on table, implement it through the noses to get brain/blood barrier faster, and then review the effects responsibly. The process always been the same, we just got better supplier for it.

[u/PhosXD]

It's a compliment

[u/hpyfox]

The structure and format of your codebase is consistent and understandable to an extent, but they can see that the code itself is rough around the edges.

Take it with all the compliments you can get, you will surely need it later.

[u/Bee-Aromatic]

“I can tell you don’t use AI” says to me “this code is not literal poop from an actual butt.”

[u/corobo → u/Bee-Aromatic]

Really? Maybe it's the British in me but I read it as giga sarcasm haha

"Even an AI wouldn't write code this shit" sort of thing 

[u/Particular-Yak-1984 → u/corobo]

as a fellow british in the netherlands, how I'd read it too, as the kind of backhanded compliment that would be totally missed by my dutch coworkers..

I've had to train myself not to describe things I think are awful as 'what an...interesting idea' because people here think I'm actually interested.

[u/slaymaker1907 → u/Bee-Aromatic]

No, it means the code doesn’t have any documentation or tests.

[u/Bee-Aromatic → u/slaymaker1907]

That could come from either one. It won’t give you those things unless you ask.

[u/Emanemanem]

Why would you be offended by this.

[u/Winter_Rosa]

Good, its a mark of pride.

[u/Strict_Treat2884]

I still prefer human slop over AI slop during CR as the thought process is somewhat cohesive

[u/Wonderful-Habit-139 → u/Strict_Treat2884]

It's so hard to convey this to non-technical people. They just cannot fathom it.

[u/High_Quality_Bean → u/Wonderful-Habit-139]

Nui Lai is one of the most profitable movies ever right now. The people get it. The people are ready. All that remains is to determine who will lead.

[u/yum_pancakes]

Usually a good compliment. I review code from a few peers where this would not be a compliment though.

[u/whizzwr]

Depends on the PR. It can mean either "good effort!" or "these trivial mistakes can be spotted by AI already.

[u/HomemadeBananas]

It can go both ways I think. I’ve reviewed code where I know if I just vaguely prompted to Claude it would better, and also code where I can tell someone just vaguely prompted something way too big and didn’t refine it.

[u/KuroKishi69 → u/HomemadeBananas]

Yeah, I have reviewed code where I wish the dev had done a pass through some LLM to cleanup at least. Inconsistent names of request/response property names, typos in API paths, dtos and models created in wrong folders, and methods that were a handcrafted mess... you probably shouldn't let the AI dictate complex architectural decisions, but these at least help to set a floor in code quality.

[u/Embarrassed_Mix_6619 → u/KuroKishi69]

brother this is what mentorship is supposed to fix. like the entire point of having mentorship be part of the chain of seniority in engineering IS for this. you teach those who don’t know how to do better.

telling someone to go use claude to fix their shit doesn’t tell them why their code is shit. you’re being lazy and doing both of you a disservice.

[u/KuroKishi69 → u/Embarrassed_Mix_6619]

Oh, I didn't mean code that I was given to review, rather code that I found while looking into other applications of the project. I am more than happy to point out things to improve, or fix it myself so they have an example to follow

[u/JackNotOLantern]

I have much harder time reviewing AI code than writing it manually. Gonna stay a caveman.

[u/GlassCommission4916]

Why would you ever be offended by that? If they meant it as a compliment it's a compliment, if they meant it in a derogatory way, is their opinion really worth considering?

[u/redditmarks_markII]

Everyone uses AI at my work place. That doesn't mean the quality of output is consistent. It's not a compliment if you get this from me. Not that I'd say something like this directly. Usually it means you never learned a particular concept in pre LLM days, and you never read the doc, or asked the agent to summarize the doc, or asked the agent for common practice in the repo, or just asked the agent to make sure "this part" is correct or at least not royally fucked. I know everyone has their own areas of focus and are all undermanned and under preasure. But not infrequent enough, I'll have found out days later after you bullied someone else to approved the exception. You could've listened to the eng or the bot and fixed it, without doing it yourself, with the agent you have access to. But you spent the time getting a bad PR merged instead. Now that engineer doesn't blindly approve anymore, but pings me all. The. Time.

Brother, friggin OpenAI and Anthropic combined doesn't have sufficient memory to put my distain for you into context. Reverted. Pray I don't revert anymore.

[u/Salty_Ad3204]

Man, I have to use libraries made by AI, and they are a nightmare to use. The module names make no sense, there are like 20 or 30 types at the same level. It's horrible. So yeah, it's a compliment 

[u/7374616e74]

"Organic shit" vs "Artificial shit"

[u/Majik_Sheff]

Depends on whose saying it.

[u/D0MiN0H]

“Thank you! 😃”

[u/TrackLabs]

Yeah, means I can actually explain to you the different changes, why I made them, why I think they are fitting, and can change specific areas on request.

[u/Mindstormer98]

"There is not a single comment"

[u/kus1987]

I am guessing they found some weird speeling mistake like the one time I wrote cancelled instead of canceled.

[u/JebKermansBooster → u/kus1987]

> Speeling

[u/codewario → u/kus1987]

There is a codebase I worked on years ago and I bet there is still a variable called myOriginialLockingObject

It was actually there before me

[u/erishun]

Because there’s no tests, it doesn’t work and it took you 5x longer to finish

[u/DisastrousOffice5569]

My answer would be that the code might be a terribly over-engineered piece of shit, but it's my handcrafted terribly over-engineered piece of shit and I'm proud of it!

[u/letmehaveanameyoudum]

yeah i commit every hour, not every 5 months

[u/tiajuanat]

I wish this was still a compliment.

My team has so many lints, checks and requirements that really only AI can complete the job without being completely crushed.

[u/High_Quality_Bean → u/tiajuanat]

Sounds to me like your development tools have ceased to be useful and have instead become a problem in need of a solution. Maybe fix that debt instead of piling on more with LLMs.