ci: require AI assistance disclosure in PRs#9260
Open
dapplion wants to merge 2 commits intosigp:unstablefrom
Open
ci: require AI assistance disclosure in PRs#9260dapplion wants to merge 2 commits intosigp:unstablefrom
dapplion wants to merge 2 commits intosigp:unstablefrom
Conversation
Add an `## AI Assistance Disclosure` section to the PR template asking contributors (1) whether AI was used and at what level, (2) to attest they have read and understand the diff, and (3) which tools they used. A new workflow validates the section is present and filled in. It runs on `pull_request_target` (read-only) so it works for fork PRs without needing a checkout. Bots and PRs labeled `skip-ai-disclosure` are exempt.
Drop the three-tier "trivial vs non-trivial" question — the boundary is fuzzy and unverifiable, and invites underreporting. Keep only: - Tools used (free text; "none" if no AI was used) - Author attestation that the diff was read and is understood Tools used implicitly answers yes/no, and the attestation is the only item enforceable in review.
Member
|
Yeah I like this model. I think our contributor guidelines still give us the flexibility to reject vibe-coded PRs as well if we suspect the author hasn't understood it or the review burden would be too great compared to the benefit |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Issue Addressed
N/A — process change.
Proposed Changes
## AI Assistance Disclosuresection toPULL_REQUEST_TEMPLATE.mdwith two required fields: tools used (free text;noneif not used) and an author attestation..github/workflows/pr-ai-disclosure.ymlto validate the section is present and filled in. Runs onpull_request_target(read-only, no checkout) so fork PRs are covered. Bots and theskip-ai-disclosurelabel are exempt.Goal: normalize disclosure rather than discourage AI use, and make contributors explicitly vouch for code they submit. To gate merges, add the check to branch protection on
unstableafter this lands.Earlier iteration of this PR had a three-tier "trivial / non-trivial" question; dropped because the boundary is fuzzy, unverifiable, and invites underreporting. The attestation is the only item enforceable in review anyway.
Additional Info
This PR's own description follows the new template — it's the first test of the workflow.
AI Assistance Disclosure
Tools used (required — write
noneif no AI was used): Claude Code (template wording and workflow scaffold; reviewed and edited by author)Attestation (required):