logoalt Hacker News

0x20cowboy • today at 5:56 PM • 3 replies • view on HN

> Who is clinging to line-by-line review.

It really depends on the context. A react button background color - meh - the critical part of the backend where a new 300ms delay will cost the company millions then yeah.


Replies

keybrd-intrrpt • today at 6:20 PM

> the critical part of the backend where a new 300ms delay will cost the company millions then yeah.

If it is so critical, why don't we have tests and benchmarks for it?

Oh right, because "Greg the super-coder" never wrote any of this down and it lives in his head so he can be the arbiter of change.

It was toxic before agentic coding, it's still toxic now.

Document your systems. Add tests and benchmarks for what is important.

We've always known this, we have a dozen or more textbooks written about it. But we always gave it a pass because "Greg the super-coder" was producing more value than he was hurting.

That's no longer the case. Your best coder is now Opus or Astra

➕ show 2 replies
Rudybega • today at 6:45 PM

> he critical part of the backend where a new 300ms delay will cost the company millions then yeah.

If a performance regression like this isn't caught by an automated testing suite, you have made a horrible mistake.

➕ show 1 reply
dominotw • today at 6:05 PM

> new 300ms delay will cost the company millions

these might be 1% typically