GeezerVisonThe World Wide Web, restored to 800 pixels of glory. | ||||||||||||||||||||||||||||
| Home|About|Gaming cabinet|Archives|How it works|Starter sites | ||||||||||||||||||||||||||||
GeezerVison is showing github.com — feat(ai): reject commits whose author is an AI tool (CC017) by shenxianpeng · Pull Request #661 · commit-check/commit-check Conversation
|
| Author | Before, forbid / disclose | After, forbid / disclose |
|---|---|---|
google-labs-jules[bot] |
pass / pass | CC017 / CC017 |
openhands <[email protected]> |
pass / pass | CC017 / CC017 |
Jane Dev (aider) |
pass / pass | CC017 / CC017 |
claude[bot] |
pass / pass | CC017 / CC017 |
Copilot <[email protected]> |
pass / pass | CC017 / CC017 |
devin-ai-integration[bot] |
pass / pass | CC017 / CC017 |
Jane Dev, dependabot[bot], renovate[bot] |
pass | pass |
CC017 ai-author check failed ==> devin-ai-integration[bot] <158243242+devin-ai-integration[bot]@users.noreply.github.com>
An AI tool is the commit author: Devin
Suggest: Author the commit yourself: git commit --amend --reset-author, and disclose the tool with "Assisted-by: Devin"
Docs: https://commit-check.com/rules/#cc017
Under forbid, the suggestion stops after --reset-author.
Piped message, ignore_authors = ["Renovate Bot"], GIT_AUTHOR_NAME=Renovate Bot: before, the message was checked under the runner's user.name. After, it is skipped.
I ran the detector over every author in junit-framework's last 3,000 commits (93 identities, including renovate, dependabot, github-actions and pre-commit-ci) and over this org's repos: no matches.
pytest: 1563 passed, 100% line and branch coverage. The one failure, test_load_config_file_permission_error, also fails on main when run as root. ruff check and ruff format are clean.
TestFixOption now pins GIT_AUTHOR_NAME/GIT_AUTHOR_EMAIL, because on a machine whose git identity is an agent, the pending author is an AI tool. The README rule table now says CC013–CC017.
The site's rules page needs a CC017 section. It goes in the release docs PR, because commit-check.com documents released behaviour only.
Sorry, something went wrong.
There was an error while loading. Please reload this page.
The AI rules read only the message, so a commit authored by the agent itself, with the person at most a Co-authored-by line, passed both forbid and disclose: Jules (google-labs-jules[bot]), OpenHands (openhands <[email protected]>), Aider ("<name> (aider)"), claude[bot], Copilot and devin-ai-integration[bot] all did. CC017 ai-author reads the author of the commit under test and judges it as the same identity on a Co-authored-by line, so the two can't disagree about who is a tool. It is built under both forbid and disclose: a tool in the author's place takes a person's place even more than a co-author line does. A project that accepts agent-authored commits can list CC017 under warn. It reads an existing commit (--rev, or HEAD); a message not yet committed has no author and passes. The GitHub App identities of Claude, Cursor's background agent, Codex cloud and Amazon Q Developer are added, so they count as co-authors too.
A commit-msg hook or a piped message has no commit to read, so CC017 now reads the author git would record: git var GIT_AUTHOR_IDENT honours GIT_AUTHOR_NAME and GIT_AUTHOR_EMAIL the way git commit does, which is how Aider commits as "<name> (aider)", and falls back to user.name and user.email. With no identity at all, nothing is flagged.
There was an error while loading. Please reload this page.
|
Warning Review limit reachedYou've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Next included review available in 15 minutes. View limit details
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@ ## main #661 +/- ## ========================================= Coverage 100.00% 100.00% ========================================= Files 14 14 Lines 2462 2499 +37 Branches 478 484 +6 ========================================= + Hits 2462 2499 +37 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
Merging this PR will not alter performance1 warning
Comparing Footnotes
|
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
Successfully merging this pull request may close these issues.
There was an error while loading. Please reload this page.