[gavmor]

Does it seem like even senior developers are forgetting this axiom? Or do we feel as though it's been obviated by LLM grokking swaths of text for us?

TBH I'm so arrogant, I always suspect there's redundant nonsense in any code module I haven't myself inspected. LLM code is no different.

[ambicapter]

> I always suspect there's redundant nonsense in any code module I haven't myself inspected.

I don't think this is arrogance in the sense that it's probably correct. It is however pretty easy to take that line of thinking into an arrogant attitude though, which is the real issue.

[axegon_]

Seniors are no different and that infuriates me even more. The few times I felt lazy and let an AI do a simple function for me, all hell broke loose. I'm starting to think that many people were never that senior to begin with: Writing the code accounts for very little of what development requires and is often the easy part. Understanding the problem and finding the sweet spot/optimal compromise, edge cases and how you can break it is what has always been difficult. Seeing github explode with slop and github(microsoft/openai) themselves push even harder should be a wake up call for anyone that understands what development is: not writing the code but having someone else go through it, analyze it, understand the problem you are trying to solve and why you made the decisions that you made - that pretty much always takes a lot more time than writing the actual code. And then I see someone push 20 commits in a day, each being 5000 lines, jam packed with emojis and other slop and tell me that they carefully reviewed all of them? Yeah, that's bullshit, mate.

[gavmor]

I'm easily pushing 20 commits a day, but I won't pretend to have reviewed it all, let alone carefully. What I did was design it all carefully.

But, for some projects, yes—I still do line-by-line code review with a colleague.

Then again, a lot of my efforts are explicit refactor aimed at reducing LOC and tidying the codebase with, eg DRY.

> The few times I felt lazy and let an AI do a simple function for me, all hell broke loose.

This is confusing, because LLMs are more than capable of implementing "a simple function." How did you spec it out?

[ambicapter]

I once worked at a fairly large corp that considered itself tech-forward (it was a retail ecommerce company), and at one point they just decided to demote all engineers one level because they somehow finally realized that everyone they had been calling "Senior" were definitely not at that level.