this post was submitted on 01 Aug 2025
66 points (97.1% liked)

Programming

21903 readers
392 users here now

Welcome to the main community in programming.dev! Feel free to post anything relating to programming here!

Cross posting is strongly encouraged in the instance. If you feel your post or another person's post makes sense in another community cross post into it.

Hope you enjoy the instance!

Rules

Rules

  • Follow the programming.dev instance rules
  • Keep content related to programming in some way
  • If you're posting long videos try to add in some form of tldr for those who don't want to watch videos

Wormhole

Follow the wormhole through a path of communities !webdev@programming.dev



founded 2 years ago
MODERATORS
top 6 comments
sorted by: hot top controversial new old
[–] deegeese@lemmy.dbzer0.com 48 points 1 day ago* (last edited 1 day ago) (2 children)

I gasped when I saw this:

A bit of discussion indicated that the trigger for the CPU spikes both times was our CEO logging in. We re-deployed to get a clean start, permanently banned him from the service, and moved on.

This is like finding a live grenade under your bed and putting it under the rug.

They found a way to reproduce a system killing bug, and instead of taking the time to understand it, they threw away their test case.

[–] BlazeDaley@lemmy.world 33 points 1 day ago

They contained the impact. Root causing or “understanding” should come after impact mitigation. If needed find a safe way to reproduce the bug without customer impact.

We reverted the refactoring, deployed, un-banned the CEO, and set about analysis.

[–] FizzyOrange@programming.dev 8 points 1 day ago

Yeah me too but if you keep reading they didn't actually "move on" in the way that it sounds.

[–] vhstape@lemmy.sdf.org 27 points 1 day ago

Well done. More and more companies are deploying LLM-written code in production environments. Might as well be honest about the results so we can learn what does and doesn't work.

[–] skip0110@lemmy.zip 11 points 1 day ago (1 children)

Why are we using tools that can't parse the comment and code via syntax for refactoring?

[–] spartanatreyu@programming.dev 23 points 1 day ago

The first problem is they're letting AI touch their code.

The second problem is they're relying on a human to pick up changes in moved code while using git's built-in diff tools. There's a whole bunch of studies that show how git's diff algorithms are terrible, and how swapping to newer diff algos improves things considerably.

TL;DR on the studies:

  • Only supporting add/remove/move operations is really bad.
  • Adding syntax awareness to understand if differences in indentation should be brought to a reviewer's attention, improves code and makes code reviews more accurate. (But this is hard because it's language dependent)
  • Adding extra operations (indent/deindent/move/rename-symbol/comment/un-comment/etc...) makes code review easier, faster and more accurate. (But again, most of this requires syntax awareness.

There's also a bunch of alternative diff algos you can use, but the best ones are paid, and the free ones have fewer features. See: