Skip to content

feat(git): replace Co-Authored-By with Assisted-by trailer#27

Merged
cblecker merged 1 commit intomainfrom
feat/assisted-by-trailer
Mar 6, 2026
Merged

feat(git): replace Co-Authored-By with Assisted-by trailer#27
cblecker merged 1 commit intomainfrom
feat/assisted-by-trailer

Conversation

@cblecker
Copy link
Owner

@cblecker cblecker commented Mar 6, 2026

Summary

  • Replace Co-Authored-By trailer with Assisted-by: Claude:<model-id> format, following the Linux kernel's Documentation/process/coding-assistants.rst standard for AI attribution
  • Add safety rule: AI must never add Signed-off-by tags (only humans can certify the DCO)
  • Bump git plugin version from 1.0.0 to 1.1.0

Test plan

  • Run bash git/scripts/git-instructions.sh and verify output contains Assisted-by trailer and Signed-off-by safety rule
  • Run bash git/scripts/git-instructions.sh | grep -i co-authored-by and verify no matches
  • Run claude plugin validate ./git and verify it passes
  • Run npx markdownlint-cli2 --config .markdownlint-cli2.jsonc "git/**/*.md" and verify 0 errors
  • Run shellcheck git/scripts/git-instructions.sh and verify no warnings

Summary by CodeRabbit

  • Documentation

    • Updated commit message attribution standards with new trailer format
    • Established safety guidelines for commit signing practices
  • Chores

    • Plugin version bumped to 1.1.0

Adopt the Linux kernel's Assisted-by standard for AI attribution
and add a safety rule preventing AI from adding Signed-off-by tags,
since only humans can certify the DCO.

Assisted-by: Claude:claude-opus-4-6
@coderabbitai
Copy link

coderabbitai bot commented Mar 6, 2026

Walkthrough

The pull request bumps the Claude plugin version to 1.1.0 and updates AI attribution standards across documentation and scripts, replacing "Co-Authored-By" trailers with "Assisted-by" format per Linux kernel conventions and adding safety rules preventing DCO sign-off additions.

Changes

Cohort / File(s) Summary
Plugin Version Bump
git/.claude-plugin/plugin.json
Updated version field from 1.0.0 to 1.1.0.
Attribution and Safety Standards
git/CLAUDE.md, git/scripts/git-instructions.sh
Replaced Co-Authored-By trailers with Assisted-by format (Assisted-by: Claude:). Added safety rule prohibiting Signed-off-by tag additions and removed references to previous Co-Authored-By convention.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Poem

🐰 A hop and a bound, our Claude takes the stage,
With "Assisted-by" now on the page,
No sign-offs sneaked in, safety rules tight,
Version 1.1 shines ever so bright! 🌟

🚥 Pre-merge checks | ✅ 2
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the main change: replacing Co-Authored-By with Assisted-by trailer across git plugin files.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feat/assisted-by-trailer

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

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

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

🧹 Nitpick comments (1)
git/scripts/git-instructions.sh (1)

181-181: Add the DCO failure path explicitly.

This tells Claude what not to do, but not how to recover when a commit-msg hook or CI requires a sign-off. Add one sentence telling it to stop and ask the human to add Signed-off-by themselves instead of retrying the commit.

Suggested wording
- - NEVER add Signed-off-by tags to commits. Only humans can certify the Developer Certificate of Origin (DCO). The human submitter is responsible for adding their own Signed-off-by tag.
+ - NEVER add Signed-off-by tags to commits. Only humans can certify the Developer Certificate of Origin (DCO). The human submitter is responsible for adding their own Signed-off-by tag.
+ - If a hook or repository policy requires Signed-off-by, stop and tell the user they must add it themselves; do not retry with a generated sign-off.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@git/scripts/git-instructions.sh` at line 181, Update the guidance line that
reads "NEVER add Signed-off-by tags to commits. Only humans can certify the
Developer Certificate of Origin (DCO). The human submitter is responsible for
adding their own Signed-off-by tag." to explicitly include the DCO failure path:
instruct the agent, when a commit-msg hook or CI reports a missing
Signed-off-by, to stop retrying and prompt the human submitter to add the
Signed-off-by themselves rather than attempting to modify commits; reference the
existing sentence "NEVER add Signed-off-by tags to commits." when making this
single-sentence addition so the behavior is clearly prescribed.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In `@git/scripts/git-instructions.sh`:
- Line 181: Update the guidance line that reads "NEVER add Signed-off-by tags to
commits. Only humans can certify the Developer Certificate of Origin (DCO). The
human submitter is responsible for adding their own Signed-off-by tag." to
explicitly include the DCO failure path: instruct the agent, when a commit-msg
hook or CI reports a missing Signed-off-by, to stop retrying and prompt the
human submitter to add the Signed-off-by themselves rather than attempting to
modify commits; reference the existing sentence "NEVER add Signed-off-by tags to
commits." when making this single-sentence addition so the behavior is clearly
prescribed.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 1dcf572d-290e-4cda-ae82-1669a05ccb45

📥 Commits

Reviewing files that changed from the base of the PR and between 07394d5 and f6a3153.

📒 Files selected for processing (3)
  • git/.claude-plugin/plugin.json
  • git/CLAUDE.md
  • git/scripts/git-instructions.sh

@cblecker cblecker merged commit 4fac223 into main Mar 6, 2026
7 checks passed
@cblecker cblecker deleted the feat/assisted-by-trailer branch March 6, 2026 22:41
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