fix: reconcile internal drift (counts + skill file naming) - #5
Draft
eddiebelaval wants to merge 1 commit into
Draft
fix: reconcile internal drift (counts + skill file naming)#5eddiebelaval wants to merge 1 commit into
eddiebelaval wants to merge 1 commit into
Conversation
- Normalize 24 lowercase skill.md -> SKILL.md so they load as Claude Code skills (each already has valid frontmatter) - Correct skill count to actual 325 dirs (was 327 in README, 328 in SPEC/VISION/BUILDING) - Fix patterns count to 2 (was 7) in README and SPEC - Refresh stale install.sh banner (was 318 skills | 56 commands | 23 agents) 10 multi-file skill collections (calendar, communication, etc.) still lack a SKILL.md and are left for the live-config sync.
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ 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 |
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
2 participants
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.
What
Fixes internal inconsistencies found while auditing the repo. This is the partial update that doesn't require access to the live
~/.claudeconfig — the full sync against the Mac Studio still needs to be run from a local Claude Code session (this cloud container can't reach it).Changes
Skill file naming (24 files)
skill.md→SKILL.mdin 24 skill directories. Each already has valid YAML frontmatter, but Claude Code only loadsSKILL.md, so these weren't being recognized as skills.Count reconciliation — the docs cited three different skill numbers (327 / 328) but there are 325 skill directories:
README.md: skills 327 → 325 (intro, table, section header, metrics); patterns 7 → 2SPEC.md: skills 328 → 325; behavioral patterns 7 → 2VISION.md: skills 328 → 325BUILDING.md: skills 328 → 325 (prose + metrics)install.sh: refreshed stale banner (318 skills | 56 commands | 23 agents→325 skills | 63 commands | 34 agents)Still open (needs live-config sync)
10 multi-file skill collections (
calendar,communication,compliance,deadline,deal,document,intelligence,knowledge,negotiation,party) have noSKILL.md— they're content collections, not loadable skills. Left untouched pending the real sync against the current system.https://claude.ai/code/session_01HF3QsquK7u3ZsXaU6cNoCZ
Generated by Claude Code