chore: move the conventional-commit skill where Claude Code loads it - #1990
Merged
Conversation
The skill sat in .agents/skills, which Claude Code does not scan - it only reads .claude/skills - so it never loaded and commit bodies drifted into six-paragraph essays. Rewritten around a length budget (most commits need no body at all), an explicit list of what does not earn a line, and a footer rule that keeps issue numbers out of body prose, since a bare number in a sentence makes conventional-changelog list the issue twice.
Contributor
There was a problem hiding this comment.
Pull request overview
Moves and rewrites the conventional-commit Claude Skill so it is discoverable by Claude Code (which scans .claude/skills), and tightens the guidance to prevent overly long commit bodies and duplicated issue references in changelog output.
Changes:
- Add a new
.claude/skills/conventional-commit/SKILL.mdwith clearer length/structure rules and footer-only issue references. - Remove the old
.agents/skills/conventional-commit/SKILL.mdso the guidance lives only where Claude Code loads it.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
.claude/skills/conventional-commit/SKILL.md |
New, more prescriptive skill content (length budget, body/footers, examples) in the directory Claude Code scans. |
.agents/skills/conventional-commit/SKILL.md |
Removes the old skill file from the non-scanned location to avoid drift/duplication. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The skill sat in
.agents/skills, which Claude Code does not scan — it only reads.claude/skills— so it neverloaded and commit bodies drifted into six-paragraph essays.
Rewritten around a length budget (most commits need no body at all), an explicit list of what does not earn a line,
and a footer rule that keeps issue numbers out of body prose, since a bare number in a sentence makes
conventional-changelog list the issue twice.