chore: add writing-changesets AI skill - #70
Conversation
|
📝 WalkthroughWalkthroughThis change adds a new documentation skill at Changes
Sequence Diagram(s)(omitted — changes are documentation and guidance updates, not new multi-component control flow) Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Poem
Important Pre-merge checks failedPlease resolve all errors before merging. Addressing warnings is optional. ❌ Failed checks (1 error)
✅ Passed checks (3 passed)
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. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In @.claude/skills/writing-changesets/SKILL.md:
- Around line 76-84: The breaking-change example in SKILL.md uses
`"@hypercerts-org/lexicon": major`, which contradicts the repo rule that 0.x
packages must use `minor`; update the markdown example so the bump type reads
`"@hypercerts-org/lexicon": minor` (i.e., replace the `major` token with `minor`
in the front-matter block) so agents and docs align with 0.x guidance.
🧹 Nitpick comments (1)
.claude/skills/writing-changesets/SKILL.md (1)
21-21: Clarify “build scripts” exception to avoid contradicting earlier guidance.
The line suggests skipping changesets for “build scripts,” but earlier guidance requires changesets for generation scripts that affect exports. Consider narrowing this to “build tooling changes that do not affect generated outputs or public API.”
20aa34d to
40cf978
Compare
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.