Skip to content

chore(OMN-18426): add strip-ai-coauthor-trailer commit-msg hook - #210

Merged
jonahgabriel merged 3 commits into
devfrom
lakshman/strip-ai-coauthor-commit-msg-hook
Sep 16, 2026
Merged

chore(OMN-18426): add strip-ai-coauthor-trailer commit-msg hook#210
jonahgabriel merged 3 commits into
devfrom
lakshman/strip-ai-coauthor-commit-msg-hook

Conversation

@Patel230

@Patel230 Patel230 commented Sep 16, 2026 •
edited by onexbot-occ-writer Bot
Loading

Copy link
Copy Markdown
Contributor

Summary (OMN-18426)

  • Adds a strip-ai-coauthor-trailer commit-msg hook to .pre-commit-config.yaml
  • The hook uses perl to strip Co-Authored-By lines matching Claude/Anthropic noreply addresses from commit messages
  • Ensures all team members automatically strip AI attribution trailers via shared pre-commit config

Details

The hook runs at the commit-msg stage and strips lines matching the Anthropic noreply pattern. This is a shared enforcement of the same rule currently in personal global git hooks, making it permanent across the team.

Test plan

  • Hook added to .pre-commit-config.yaml
  • Verify pre-commit install --hook-type commit-msg picks up the new hook
  • Commit with a Co-Authored-By Claude trailer and verify it gets stripped

Evidence-Ticket: OMN-18426
Evidence-Source: OCC#9800

@Patel230 Patel230 changed the title chore: add strip-ai-coauthor-trailer commit-msg hook chore(OMN-18426): add strip-ai-coauthor-trailer commit-msg hook Sep 16, 2026
@onexbot-occ-writer onexbot-occ-writer Bot mentioned this pull request Sep 16, 2026
Merged
jonahgabriel pushed a commit to OmniNode-ai/onex_change_control that referenced this pull request Sep 16, 2026
@onexbot-occ-writer node-occ-companion-effect
evidence(OMN-18426): OCC companion for OmniNode-ai/omnibase_compat#210 (
d662daa 
#9800)

* evidence: OCC companion pass 1 for OmniNode-ai/omnibase_compat#210

* evidence: OCC companion self-bind for #9800

---------

Co-authored-by: node-occ-companion-effect <occ-companion-effect@omninode.ai>
@jonahgabriel
jonahgabriel merged commit f758710 into dev Sep 16, 2026
35 checks passed
@jonahgabriel
jonahgabriel deleted the lakshman/strip-ai-coauthor-commit-msg-hook branch September 16, 2026 10:55
This was referenced Sep 17, 2026
Merged
Merged
Merged
@Patel230

Patel230 commented Sep 20, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

Post-merge closeout for OMN-18426.

Merged 2026-09-16T10:55:21Z as f758710fe643.

What landed. Adds the strip-ai-coauthor-trailer hook on the commit-msg stage to this repository's .pre-commit-config.yaml.

Review. No review bodies or inline review comments were present on this pull request, so there is nothing outstanding to answer here.

One limit, recorded rather than left implicit. The entry this rollout shipped matches a single vendor address. The maintained machine-identity list now carries four — noreply@anthropic.com, claude@omninode.ai, cursoragent@cursor.com, codex@omninode.ai — and the branch-side gate reads that list directly, but the hooks in this repository and thirteen others have not been re-keyed to it. The shape of that re-key is the open question on the ticket, so nothing has been changed here on one lane's own reading.

Thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Patel230 @jonahgabriel