-
Notifications
You must be signed in to change notification settings - Fork 972
docs: add generative ai contribution guidelines #19427
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
2b4bd88
08bf2cb
dd148af
985bb3b
6c6c9f7
4388a74
706560b
bf983b8
5adc972
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,32 @@ | ||
# AI Contribution Guidelines | ||
|
||
This document defines rules for contributions where an AI system is the primary author of the code (i.e., most of the pull request was generated by AI). | ||
It applies to all Coder repositories and is a supplement to the [existing contributing guidelines](./CONTRIBUTING.md), not a replacement. | ||
|
||
For minor AI-assisted edits, suggestions, or completions where the human contributor is clearly the primary author, these rules do not apply — standard contributing guidelines are sufficient. | ||
|
||
## Disclosure | ||
|
||
Contributors must **disclose AI involvement** in the pull request description whenever these guidelines apply. | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I'm not 100% sure. I know we can make issue templates, never tried for PRs There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. It'd be nice to have a standard way to represent AI contributors. We could either have a separate AI PR template or put the AI splash in the template inside There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @Emyrk can we please include this? Right now folks won't know to look at your new doc; let's increase discoverability. |
||
|
||
## Human Ownership & Attribution | ||
|
||
- All pull requests must be opened under **user accounts linked to a human**, and not an application ("bot account"). | ||
- Contributors are personally accountable for the content of their PRs, regardless of how it was generated. | ||
|
||
## Verification & Evidence | ||
|
||
All AI-assisted contributions require **manual verification**. | ||
dannykopping marked this conversation as resolved.
Show resolved
Hide resolved
|
||
Contributions without verification evidence will be rejected. | ||
|
||
- Test your changes yourself. Don’t assume AI is correct. | ||
- Provide screenshots showing that the change works as intended. | ||
- For visual/UI changes: include before/after screenshots. | ||
- For CLI or backend changes: include terminal or api output. | ||
|
||
## Why These Rules Exist | ||
|
||
Traditionally, maintainers assumed that producing a pull request required more effort than reviewing it. | ||
With AI-assisted tools, the balance has shifted: generating code is often faster than reviewing it. | ||
|
||
Our guidelines exist to safeguard maintainers’ time, uphold contributor accountability, and preserve the overall quality of the project. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This decision is very much "vibes based" from the reviewer's perspective. As such, I anticipate that this may be the source of a number of arguments. Unfortunately, I don't have any suggestion for a better delineation.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I do agree, but I kept it subjective because I know we all use AI day to day in our coding. I did not want to encompass our usual PRs.
I do not like the idea of saying like "50% of lines" or something.