Skip to content

fix(agents): override global AI attribution in commit messages#2264

Closed
rhuss wants to merge 1 commit into
NVIDIA:mainfrom
rhuss:fix/no-ai-attribution-in-commits
Closed

fix(agents): override global AI attribution in commit messages#2264
rhuss wants to merge 1 commit into
NVIDIA:mainfrom
rhuss:fix/no-ai-attribution-in-commits

Conversation

@rhuss

@rhuss rhuss commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Adds explicit override in CLAUDE.md to prevent AI agent attribution trailers (Assisted-By, Co-Authored-By) in commit messages
  • Reinforces the existing AGENTS.md rule for Claude Code users who have global attribution configured

Related Issue

Addresses PR review feedback on #2253 where a commit included Assisted-By: 🤖 Claude Code despite the project policy.

Changes

  • CLAUDE.md: Added "Git Commit Attribution Override" section that explicitly overrides any global Claude Code settings

Testing

  • Verified the instruction takes precedence over global ~/.claude/CLAUDE.md attribution settings

@rhuss rhuss requested review from a team, derekwaynecarr, maxamillion and mrunalp as code owners July 14, 2026 13:07
@copy-pr-bot

copy-pr-bot Bot commented Jul 14, 2026

Copy link
Copy Markdown

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

The existing AGENTS.md rule prohibiting AI agent mentions in commits
was being ignored because agents prioritized a global "Assisted-By"
instruction from their user-level configuration. Move the override
directly into AGENTS.md with explicit language that it supersedes
global agent settings, making it a first-class rule in the agent
constitution rather than a secondary reference.

Signed-off-by: Roland Huß <rhuss@redhat.com>
@rhuss rhuss force-pushed the fix/no-ai-attribution-in-commits branch from e5e021c to fcbba4d Compare July 14, 2026 13:15
@rhuss

rhuss commented Jul 14, 2026

Copy link
Copy Markdown
Contributor Author

Closing this PR — apologies for the noise. The existing AGENTS.md rule already covers this; the issue was that my agent ignored the project-level instruction in favor of a global user setting. That's been corrected on the agent side. The strengthened wording is included in #2253 if the maintainers want it, but a separate PR isn't warranted.

@rhuss rhuss closed this Jul 14, 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.

1 participant