chore: Add CLAUDE.md and commit-msg hook#15
Conversation
Replicate the contribution guidelines and attribution-enforcing commit-msg hook from kagenti/.github (PR #79) onto this repo. - CLAUDE.md: DCO sign-off, Assisted-By attribution policy, PR description attribution, scope discipline - scripts/hooks/commit-msg: enforces Assisted-By over Co-authored-by Assisted-By: Claude Opus 4.6 <noreply@anthropic.com> Signed-off-by: Gloire Rubambiza <gloire@ibm.com>
rubambiza
force-pushed
the
chore/add-claude-md
branch
from
00837c9 to
f3d1757
Compare
rubambiza
added
the
ready-for-ai-review
clawgenti
left a comment
clawgenti
left a comment
There was a problem hiding this comment.
Adds CLAUDE.md and scripts/hooks/commit-msg to replicate contribution guidelines from kagenti/.github — clean, well-scoped PR with a single signed-off commit and passing CI.
One nit: the commit's Assisted-By trailer (Claude Opus 4.6 <noreply@anthropic.com>) differs from the format CLAUDE.md itself documents (Assisted-By: Claude (Anthropic AI) <noreply@anthropic.com>). Not a blocker, but the hook enforces the canonical form, so leading by example in this PR's commit would be nice.
Reviewed by clawgenti using github:pr-review
| if [ "$HAS_AI_TRAILER" = true ]; then | ||
| # Ensure there's a blank line before the trailer | ||
| echo "" >> "$COMMIT_MSG_FILE" | ||
| echo "Assisted-By: Claude (Anthropic AI) <noreply@anthropic.com>" >> "$COMMIT_MSG_FILE" |
There was a problem hiding this comment.
nit: This line uses Claude (Anthropic AI) but the commit that introduces this PR uses Claude Opus 4.6 as the Assisted-By value. CLAUDE.md (line 24) documents Claude (Anthropic AI) <noreply@anthropic.com> as the canonical form — the hook is correct, but the introducing commit itself doesn't follow it. Consider amending to match.
There was a problem hiding this comment.
Fair call out. The Assisted-By: Claude Opus 4.6 <noreply@anthropic.com> is due to my own local hooks. I will modify the local ones to match the CLAUDE.md one so that the trailers are not tied to a particular model for future commits.
rubambiza
added
documentation
pdettori
left a comment
pdettori
left a comment
There was a problem hiding this comment.
Clean replication of the contribution-guidelines setup from kagenti/.github.
CLAUDE.md: DCO,Assisted-Byattribution policy, PR-description attribution, and scope discipline — all consistent with the org standard.scripts/hooks/commit-msg: usesset -euo pipefail,mktemp, and portableawkfor trailing-blank-line cleanup; logic is sound. Opt-in viagit config core.hooksPath, runs only on commit — no concerns.
DCO passing. LGTM.
Assisted-By: Claude Code
3 participants
Summary
Replicate the contribution-guidelines setup from
kagenti/.github(merged in PR #79) onto this repo:Assisted-By, neverCo-authored-by), PR description attribution, and scope discipline.kagenti/.github): enforces the attribution policy automatically by stripping AICo-authored-by/Made-withtrailers and replacing them withAssisted-By.Install the hook locally with:
Closes #13
Assisted-By: Claude Code