Skip to content

docs: AI contribution policy and PR disclosure levels#430

Open
Just-Insane wants to merge 5 commits intodevfrom
docs/ai-contribution-policy
Open

docs: AI contribution policy and PR disclosure levels#430
Just-Insane wants to merge 5 commits intodevfrom
docs/ai-contribution-policy

Conversation

@Just-Insane
Copy link
Contributor

@Just-Insane Just-Insane commented Mar 20, 2026

Summary

Updates the "Generative AI Usage" section in CONTRIBUTING.md to append a note about disclosing AI use in PRs. Also updates the PR template with a four-level AI disclosure checklist and fixes the stale Cinny link.

Changes

CONTRIBUTING.md

  • Keeps the original policy text intact
  • Appends: disclosure prompt pointing contributors to the PR template checklist

.github/PULL_REQUEST_TEMPLATE.md

  • Fixed stale link (was pointing to upstream Cinny repo, now points to Sable's own CONTRIBUTING.md)
  • Added "AI assistance level" section with four mutually-exclusive levels:
    • None — no AI tools used
    • Research only — AI for research/understanding, no generated code
    • AI-assisted — AI-suggested code, fully reviewed and understood
    • Primarily AI-generated — majority AI-generated, thoroughly reviewed

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

AI assistance level

  • None — No AI tools were used
  • Research only — AI used for research, brainstorming, or understanding existing code; no generated code included
  • AI-assisted — AI suggested code or logic that I reviewed, tested, and understand fully; I can explain every change
  • Primarily AI-generated — The majority of code was AI-generated; I have thoroughly reviewed it, verified correctness, and made it my own

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • My changes generate no new warnings

CONTRIBUTING.md Outdated
## AI-Assisted Development

We expect and appreciate authentic engagement in our community.
We welcome AI-assisted contributions. Tools like GitHub Copilot, Cursor, Claude, ChatGPT, and others are legitimate parts of a modern development workflow, and we don't ask you to hide their use.
Copy link
Contributor

Choose a reason for hiding this comment

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

hmmm that might be understood as us endorsing ai generated code.

i liked the previous version more.

Copy link
Contributor

Choose a reason for hiding this comment

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

agree, we should stay neutral.

Copy link
Contributor

Choose a reason for hiding this comment

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

yeah the new version somehow sounds aggressively pro-ai.

like we would endorse it, idk if that's what we want to do

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Disagree, this language is used to explicitly take a stance that can be pointed to when people make complaints.

The last section does need to be re-written though.

Copy link
Contributor

Choose a reason for hiding this comment

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

i think i just have to disagree. it’s too pro. we should be more neutral.

CONTRIBUTING.md Outdated
- **Add genuine value.** Wrapping a prompt in a PR without understanding or verifying the output is not a contribution. Maintainers can tell the difference and will close PRs that don't meet the bar.

Make sure you have added value based on your personal competency to your contributions. Just taking some input, feeding it to an AI and posting the result is not of value to the project. We reserve the right to rigorously reject seemingly AI generated low-value contributions.
When opening a pull request, **disclose your level of AI use** using the checklist in the PR template. This isn't gatekeeping — it's transparency that helps reviewers calibrate their review depth. A heavily AI-generated PR will get more scrutiny; that's expected and fine.
Copy link
Contributor

Choose a reason for hiding this comment

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

i would keep the part with adding value based on personal competency in there.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Contributor

Choose a reason for hiding this comment

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

ig i just liked the old phrasing more, idk

Clarified guidelines on AI-assisted contributions, emphasizing transparency in AI usage.
@linkpy
Copy link

linkpy commented Mar 20, 2026

Not a maintainer but a user of Sable. Im rather opposed of the use of generative AI. It includes a lot of issues (code quality, performance, and usage of licensed code are the big one). Sable has been doing fine without it, I dont really see what it would bring or add to the project (except the issues listed above).

@Just-Insane Just-Insane changed the title docs: pro-AI contribution policy and PR disclosure levels docs: AI contribution policy and PR disclosure levels Mar 20, 2026
@Just-Insane
Copy link
Contributor Author

Just-Insane commented Mar 20, 2026

Not a maintainer but a user of Sable. Im rather opposed of the use of generative AI. It includes a lot of issues (code quality, performance, and usage of licensed code are the big one). Sable has been doing fine without it, I dont really see what it would bring or add to the project (except the issues listed above).

Code quality, testing, reviews, etc are out of scope for this PR and will be addressed separately.

Additionally, there is AI generated code currently in Sable.

@Just-Insane Just-Insane requested review from dozro and hazre March 20, 2026 20:55
@Just-Insane Just-Insane marked this pull request as ready for review March 20, 2026 20:56
@Just-Insane Just-Insane requested a review from 7w1 as a code owner March 20, 2026 20:56
Copy link
Contributor

@dozro dozro left a comment

Choose a reason for hiding this comment

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

seems fine now, even tho i would’ve kept the heading as “restrictions on gen ai” but seems fine :3

@Rosy-iso
Copy link
Contributor

i think this works now

@Rosy-iso
Copy link
Contributor

wait why was this closed??

@Just-Insane
Copy link
Contributor Author

Just-Insane commented Mar 21, 2026

wait why was this closed??

Because I am done with this community after the last couple days. This place is toxic and I will not put up with that.

I'm sure you all can figure out a path forward, but I will no longer be interacting with this community.

If anyone wants to keep in touch, my mxid is: @evie:cloudhub.social

@Just-Insane
Copy link
Contributor Author

This may be okay to merge? But I can't support/update it going forward. You can merge it if you think it is useful.

@Just-Insane Just-Insane reopened this Mar 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants