docs(claude): migrate workflows out of CLAUDE.md to skills - #355
Conversation
Move "Pre-commit Checks" (now a pointer to the existing /pre-commit skill), "Creating Issues", and "Key workflows" out of the always-loaded CLAUDE.md into two new on-demand skills (create-issue, key-workflows), cutting ~2.2k always-resident characters per session. Found and applied by the new /doctor command. Co-authored-by: Claude Sonnet 5 <noreply@anthropic.com>
WalkthroughAdds on-demand Claude skills for issue creation and key development workflows, updates ChangesClaude workflow guidance
Estimated code review effort: 2 (Simple) | ~10 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 2✅ Passed checks (2 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #355 +/- ##
===========================================
Coverage 100.00% 100.00%
Complexity 32 32
===========================================
Files 3 3
Lines 90 90
Branches 8 8
===========================================
Hits 90 90 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In @.claude/skills/create-issue/SKILL.md:
- Around line 1-6: Add a top-level Markdown heading immediately after the YAML
front matter in both .claude/skills/create-issue/SKILL.md (lines 1-6) and
.claude/skills/key-workflows/SKILL.md (lines 1-6), using headings such as
“Create an issue” and “Key workflows” respectively, while preserving the
existing content.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro Plus
Run ID: bb4eaabe-cf24-4884-886d-bbf480406336
📒 Files selected for processing (4)
.claude/skills/create-issue/SKILL.md.claude/skills/key-workflows/SKILL.mdCHANGELOG.mdCLAUDE.md
| --- | ||
| name: create-issue | ||
| description: Draft a GitHub Issue as the spec artifact for Spec-Driven Development (SDD) before implementing a feature or bug fix in this project. | ||
| --- | ||
|
|
||
| This project uses Spec-Driven Development (SDD): discuss in Plan mode first, create a GitHub Issue as the spec artifact, then implement. Always offer to draft an issue before writing code. |
There was a problem hiding this comment.
📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win
Resolve MD041 in both new skill files.
Both files begin their Markdown content with a paragraph after YAML front matter, triggering the same top-level-heading lint warning.
.claude/skills/create-issue/SKILL.md#L1-L6: add a heading such as# Create an issue..claude/skills/key-workflows/SKILL.md#L1-L6: add a heading such as# Key workflows.
🧰 Tools
🪛 markdownlint-cli2 (0.23.0)
[warning] 6-6: First line in a file should be a top-level heading
(MD041, first-line-heading, first-line-h1)
📍 Affects 2 files
.claude/skills/create-issue/SKILL.md#L1-L6(this comment).claude/skills/key-workflows/SKILL.md#L1-L6
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In @.claude/skills/create-issue/SKILL.md around lines 1 - 6, Add a top-level
Markdown heading immediately after the YAML front matter in both
.claude/skills/create-issue/SKILL.md (lines 1-6) and
.claude/skills/key-workflows/SKILL.md (lines 1-6), using headings such as
“Create an issue” and “Key workflows” respectively, while preserving the
existing content.
Source: Linters/SAST tools
1 participant
Summary
CLAUDE.mdwith a pointer to the existing/pre-commitskillCLAUDE.mdinto two new on-demand skills (create-issue,key-workflows), so their content only loads into context when actually needed instead of every sessionCLAUDE.mdper sessionFound and applied by the new
/doctorcommand, per The new rules of context engineering for Claude 5 generation models.No backing GitHub Issue — pure internal doc/context reorganization, no behavior change.
Test plan
CLAUDE.mdstill renders correctly and both skills load (name/descriptionfrontmatter present)🤖 Generated with Claude Code
This change is
Summary by CodeRabbit
/pre-commitcheck.