Skip to content

AI tooling must be disclosed for contributions #8289

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

Merged
merged 1 commit into from
Aug 19, 2025
Merged

Conversation

mitchellh
Copy link
Contributor

@mitchellh mitchellh commented Aug 19, 2025

I think, at this stage of AI, it is a common courtesy to disclose this.

In a perfect world, AI assistance would produce equal or higher quality work than any human. That isn't the world we live in today, and in many cases it's generating slop. I say this despite being a fan of and using them successfully myself (with heavy supervision)! I think the major issue is inexperienced human drivers of AI that aren't able to adequately review their generated code. As a result, they're pull requesting code that I'm sure they would be ashamed of if they knew how bad it was.

The disclosure is to help maintainers assess how much attention to give a PR. While we aren't obligated to in any way, I try to assist inexperienced contributors and coach them to the finish line, because getting a PR accepted is an achievement to be proud of. But if it's just an AI on the other side, I don't need to put in this effort, and it's rude to trick me into doing so.

I'm a fan of AI assistance and use AI tooling myself. But, we need to be responsible about what we're using it for and respectful to the humans on the other side that may have to review or maintain this code.

(In the spirit of this PR... none of this PR was AI generated. lol.)

Verified

This commit was signed with the committer’s verified signature.
mitchellh Mitchell Hashimoto
Copy link

@yawaramin yawaramin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Broadly agree with this; maybe a pull request template can be added to make it explicit: https://docs.github.com/en/communities/using-templates-to-encourage-useful-issues-and-pull-requests/creating-a-pull-request-template-for-your-repository

Another advantage of a PR template is it can have a checklist for other stuff, eg Developer Certificate of Origin.

@mitchellh mitchellh merged commit e71c238 into main Aug 19, 2025
80 of 84 checks passed
@mitchellh mitchellh deleted the push-luuksxnzkwzq branch August 19, 2025 22:52
@github-actions github-actions bot added this to the 1.2.0 milestone Aug 19, 2025
@amadeus
Copy link

amadeus commented Aug 20, 2025

gotta put a stop 2️⃣ da slop

@00-kat 00-kat added the documentation Improvements or additions to documentation label Aug 20, 2025
@hyz0906 hyz0906 mentioned this pull request Aug 20, 2025
@tobi
Copy link

tobi commented Aug 20, 2025

I think this is an opportunity for ai tools, too. GitHub should publish a standard for a special AI byline that all ai tools can write to. Anytime you use an ai tool, it adds itself to a .git staging file and the next commit lists all the involved tools appended to the message similar to co authors.

Then GitHub lists and links to the tools. This way the tools get some exposure, but maintainers also can require this standard to be followed. Everyone wins. And the tools can stop spamming the co-authors byline which is the current viral loop which is a bit obnoxious.

devin-ai-integration bot added a commit to antiwork/.github that referenced this pull request Aug 20, 2025

Unverified

This commit is not signed, but one or more authors requires that any commit attributed to them is signed.
Added comprehensive AI Assistance Notice section based on ghostty-org/ghostty#8289
- Requires disclosure of AI tool usage in PRs
- Provides clear examples and guidelines
- Explains reasoning for transparency policy

Co-Authored-By: Sahil Lavingia <sahil@gumroad.com>
@stuaxo
Copy link

stuaxo commented Aug 20, 2025

It seems sensible: if AI tools were used, the contributors should have reviewed, revised and understand what they are contributing.

@zanellig
Copy link

LGTM

@silva96
Copy link

silva96 commented Aug 20, 2025

Does this apply to tab completions? Most of the tab completions are just finishing what the user wanted to write anyways ...

@chrishas35
Copy link

Does this apply to tab completions? Most of the tab completions are just finishing what the user wanted to write anyways ...

this is covered in the doc change.

@ghostty-org ghostty-org locked as resolved and limited conversation to collaborators Aug 21, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

10 participants