@lw I care. If there will be a lot of commits from "passer-by committers", who don't want to understand how things work and don't care about code they are changing — because they just want to obtain a stripe "Look ma, I committed to the FreeBSD" and nothing more — then we are doomed
With LLM-conmits it is pretty easy to overload reviewers
Also, it makes questions about developer's intent — they used LLM because they don't have fun from programming and/or debugging (so, why contributing to FOSS?)? Or they were pushed to release new features faster (like in corporation)? Or smth likewise?
@evgandr @piero there is no policy from core on LLM-generated commits yet. we hope they will produce such a policy, but i would put money on that policy *not* prohibiting LLM-generated code.
i don't have any insider knowledge there, that's just based on my understand of how FreeBSD developers view LLMs from talking to them.
code which is bad or which the submitter doesn't understand should not pass review regardless of whether it's LLM-generated or not.
@evgandr people already do that with typo fixes. fixing a typo is an easy way to get a commit in, and i'm sure some people have done that just so they can say "i contributed to FreeBSD!"... and honestly, i don't really have a problem with that. i might find their motivation silly, but they're still providing useful changes.
also, since we've started allowing PRs on GitHub, there has been a steady stream of bad changes, which (from what i can tell) don't have anything to do with LLMs, it's just the result of opening up your development process. so i'm sure that will continue regardless of the project's stance of LLMs.
as far as developer's intent goes, FreeBSD is used at many corporations who may have a policy that developers are required to use LLMs. i don't agree with this, but it's a fact of life, at least for now. i don't know if we want to blanket reject contributions from such corporations.
@lw Yes, the "fix some typos and commas" is a well-known way to start contributing to opensource. But I'm afraid (and pretty sure) that LLM usage gives the same people the false feeling that they able made some good changes in the more complex parts of source code so they may to make a PR – because they believe that LLM is a real AI and able to understand some ideas, incorporated in the code base.
E.g. this PR: https://lore.kernel.org/linux-mm/cbd0a[email protected]/
As a result, I think, we will receive more and more meaningless but convincingly looking PRs, which will lead to the repeat of XZ backdoor story.