Remix.run Logo
simonw 3 hours ago

I've found that access to coding agents has helped me be far less tolerant of bad code patterns that can be refactored.

Refactoring used to have a very real cost - it was substantial amounts of time that would have to be carved away from working on new features.

Now I can spot a potential refactor, fire off a prompt in an asynchronous coding agent (or on a worktree or whatever), then come back 20 minutes later and either accept it, poke it a bit, or abandon it. Costs me almost nothing.

rafaelRiv 3 hours ago | parent | next [-]

And for me, it has created a giant pile of unmaintainable code from my coworkers because of stupid management people that think they can code.

Refactoring was never a substantial amounts of time for me before llms. Before I could spot a potential refactor and refactor it in 20 minutes and less. Never abandon it. Just constant improvement to the point the previous tech startup that I was working for just drive from itself (I am still paid a a few hours per months for it)

Kiro 2 hours ago | parent | next [-]

I've never met a single developer saying refactors are easy. It's always a minefield.

In the only scenario where that could actually be true you wouldn't have coworkers creating unmaintainable code with LLMs now.

SoftTalker 2 hours ago | parent [-]

If you were fortunate enough to be working in a project with very good test coverage, refactors were easier (maybe not easy) in the sense that you were at least fairly confident that nothing broke.

But I've personally never worked on code with test coverage that good. Refactors were always risky.

simonw 3 hours ago | parent | prev [-]

Does that code at least come with tests?

nijave 2 hours ago | parent [-]

Gotta get IT to slip superpowers or at least the TDD skill into MDM/enterprise controls

boron1006 2 hours ago | parent | prev | next [-]

Do you work with other, potentially unmotivated devs/managers a lot? I wonder how much of this is an incentive issue. I don’t think refactoring itself is as much of an issue than working with 2-3 other engineers that DGAF and a manager who is only looking at LOC to determine who to promote.

But it also doesn’t mean these things aren’t problems, they’re obviously huge problems.

teaearlgraycold 3 hours ago | parent | prev [-]

All sorts of efforts that used to be put off indefinitely can now be handled largely by LLMs. Yesterday I took a codebase (~50 source files) and spawned a sub-agent (GLM 5.3 Flash) for every single file. Each file was analyzed for test coverage issues, inconsistencies between comments and implementations, and all call sites against the implementation. Then issues were aggregated. I reviewed the list manually, had Opus and K3 review as well to prune the list, and then had a commit made for every minor issue found. 70 commits with maybe 30 minutes of manual work.

Not exactly a refactor, but high degrees of consistency are what I strive for in a codebase. LLMs get confused by inconsistencies, as do humans.

nottorp 2 hours ago | parent [-]

> then had a commit made for every minor issue found

Interesting choice of words. Not a fix. Not a refactor. A commit.

teaearlgraycold an hour ago | parent [-]

Not sure what you’re getting at

Sivart13 10 minutes ago | parent [-]

Test coverage for its own sake is not always a worthwhile goal.