Skip to content

fix(rules): drop Co-Authored-By trailers from agent commits - #217

Merged
shunkakinoki merged 1 commit into
mainfrom
fix/no-coauthor-trailers
Oct 20, 2026
Merged

shunkakinoki merged 1 commit into
mainfrom
fix/no-coauthor-trailers

Conversation

@shunkakinoki

@shunkakinoki shunkakinoki commented Oct 20, 2026 •

Copy link
Copy Markdown
Owner

Reverts the rule from #213 that told every agent to add a Co-Authored-By trailer. Squash merges were collecting model and agent names (OpenCode, MiniMax, DeepSeek, kamino*) as co-authors.

  • commit-push command/rule, commit-lint and pr-create skills now forbid Co-Authored-By trailers

🤖 Generated with Claude Code


Summary by cubic

Removes the Co-Authored-By trailer requirement from agent-authored commits. Squash merges were accumulating model and agent names (OpenCode, MiniMax, DeepSeek, kamino*) as co-authors, so the rule is now flipped to forbid the trailer in commands/commit-push.md, rules/commit-push.md, skills/commit-lint/SKILL.md, and skills/pr-create/SKILL.md. AI attribution stays in the PR description or body instead.

Written for commit e01e438. Summary will update on new commits.

View guided diff Turn on auto-fix

@coderabbitai

coderabbitai Bot commented Oct 20, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 42e64d66-e92d-450f-bfa6-b7a22415e135
📥 Commits

Reviewing files that changed from the base of the PR and between 412021d and e01e438.

📒 Files selected for processing: 4
  • commands/commit-push.md
  • rules/commit-push.md
  • skills/commit-lint/SKILL.md
  • skills/pr-create/SKILL.md

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

📝 Summary

Summary by CodeRabbit

  • Documentation
    • Updated commit and pull request guidelines to prohibit Co-Authored-By trailers on solo-authored commits.
    • Removed the previous requirement to include an AI co-author trailer.
    • Clarified that issue references should continue using the specified format, such as Closes #123.

Four commit guidance documents now prohibit Co-Authored-By trailers. The commit-push guidance retains the Closes #123 issue reference format.

Changes: Commit authorship guidance

Layer / File(s) Summary
Update commit trailer policy
commands/commit-push.md, rules/commit-push.md, skills/commit-lint/SKILL.md, skills/pr-create/SKILL.md
The guidelines replace the requirement for an AI co-author trailer with a prohibition on Co-Authored-By trailers. The commit-push rule retains the Closes #123 issue reference format.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Bug fix

Merge Risk: ⚪ Minimal · up to e01e4

The commit guidance consistently prohibits Co-Authored-By trailers and is ready to merge.

Pre-merge checks: Passed 5
Check name Status Explanation
Title check Passed The title clearly identifies the main change: removing Co-Authored-By trailers from agent commits.
Description check Passed The description directly explains the rule changes and the reason for removing Co-Authored-By trailers.
Docstring Coverage Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (4 skipped: 4 unsupported.)

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

❤️ Share

A rabbit checks each commit line,
No co-author trailer will it find.
Solo authorship marks the page,
Issue references stay in place.
Then hops away, content and kind.

Comment @coderabbitai help to get the list of available commands.

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

No issues found across 4 files

View guided diff | Turn on auto-fix | Re-trigger cubic

@shunkakinoki
shunkakinoki merged commit 75f4a30 into main Oct 20, 2026
4 checks passed
@shunkakinoki
shunkakinoki deleted the fix/no-coauthor-trailers branch October 20, 2026 03:31
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.

1 participant