Post #3621384
2026-07-03 11:47 UTC
@hughsie@mastodon.social on the fence about the usefulness of the "include fix" clause.
Do you think you'd really get meaningful fixes out of it that you could merge as-is? Cause in reality, I'd expect many of the non-obvious bugs that remain in highly scrutinized projects like fwupd, to require fixes that carry a lot of potential side-effects.
And it may actually be harder to reason about those in a review for a fix that was spit out by an LLM, as opposed to thinking through the problem from the start.
Then again, if your test suite is good enough, a fix that passes all tests should be fine to merge I guess 🤔
Replies (1)
-
@hughsie@mastodon.social 2026-07-03 13:24
@wall_e@ioc.exchange @wall_e@ioc.exchange usually the LLM has a lot more internal context at the point of filing the issue than what's formatted in the report -- at least from my limited experience it usually does a half-decent job at fixing the bug and adding tests -- even if a human has to refactor and move things around for maintainability.