docs: add AI attribution policy#848
Conversation
Documents the project's AI tool policy across contributor-facing locations: optional Assisted-by trailer for humans, default-on for AI assistants, and an Attribution checkbox in all PR templates. Assisted-by: Claude Code Signed-off-by: Alex Bozarth <ajbozart@us.ibm.com>
github-actions
bot
added
the
documentation
|
The PR description has been updated. Please fill out the template for your PR to be reviewed. |
psschwei
left a comment
psschwei
left a comment
There was a problem hiding this comment.
governance file is not updated
Add pointer to CONTRIBUTING.md attribution conventions in the existing 'Use of AI tools' paragraph. Assisted-by: Claude Code Signed-off-by: Alex Bozarth <ajbozart@us.ibm.com>
psschwei
left a comment
psschwei
left a comment
There was a problem hiding this comment.
These days I assume all code I see is AI generated. So I'm not sure the value of flagging it in a commit, but it also doesn't really hurt anything (as long as AGENTS.md will add it automatically, I don't want to have to go back and edit my commits)
The LF document we're basing this on (https://docs.kernel.org/process/coding-assistants.html#attribution) suggests the reason they are adding this line is to "track the evolving role of AI in the development process". Is that something we plan on tracking / care about?
A bit, but they're looking at tools and models in their trailers, whereas I've only put the AI Assistant name since IMHO I use multiple models on any given commit and tools come and go quickly. So to me this is more about automatically marking code as AI assisted as more of an reference |
|
Also if it was not clear, this update is less of a policy change and more of a clarification for agents: |
psschwei
left a comment
psschwei
left a comment
There was a problem hiding this comment.
LGTM
I would let at least one other person review before merging.
Remove redundant intro sentence and 'For AI assistants:' label — the whole document is addressed to AI assistants. Assisted-by: Claude Code Signed-off-by: Alex Bozarth <ajbozart@us.ibm.com>
planetf1
left a comment
planetf1
left a comment
There was a problem hiding this comment.
I suggest the ambiguity is resolved
- Standardize terminology to "AI coding assistants" throughout
(GOVERNANCE.md, CONTRIBUTING.md, PR templates, docs guide)
- Clarify AGENTS.md §7: human runs git commit -s; agents always use
Assisted-by, not Signed-off-by; add common name guidance with etc.
- Add §6→§7 cross-reference in AGENTS.md for commit guidance
- Fix stale "Section 7" reference in Feedback Loop (now Section 9)
- Expand docs/community/contributing-guide.md to include trailer
example and common name note; fix tense ("will add" → "add")
- Simplify PR template attribution checkbox to pure disclosure
Assisted-by: Claude Code
Signed-off-by: Alex Bozarth <ajbozart@us.ibm.com>
Signed-off-by: Alex Bozarth <ajbozart@us.ibm.com>
52ae72c
3 participants
Misc PR
Type of PR
Description
Adds an AI attribution policy for the project, partially adopting the
Linux kernel's
Assisted-bytrailer convention. AI tools add the trailerby default, so no extra action is required from human contributors.
Changes across five locations:
AGENTS.md— New Section 7 instructing AI assistants to addAssisted-by: <tool>by default and not to add aSigned-off-byin their own nameCONTRIBUTING.md— New "AI Tools" subsection under Commit Messagesdocumenting the policy for human contributors
docs/docs/community/contributing-guide.md— Lighter version of thesame for new contributors finding the website first
.github/PULL_REQUEST_TEMPLATE/*.md— Optional Attribution checkboxadded to all five PR templates
Testing
Attribution