Add chunk skills command with embedded chunk-review skill#9
Add chunk skills command with embedded chunk-review skill#9michael-webster merged 2 commits intomainfrom
Conversation
Add the chunk-review SKILL.md to skills/ (embedded into the binary at build time) and .claude/skills/ (installed to the agent config directory when running chunk skills install).
Introduce src/core/skills.ts and src/commands/skills.ts to implement install/status/list subcommands that manage skills in agent config directories (~/.claude/skills/, ~/.codex/skills/). Skill content is read from skills/ and embedded into the binary via Bun's text loader (--loader .md:text added to all build targets). Register commands in src/index.ts under the Commander.js program. Add 15 unit tests covering idempotency, outdated-skill updates, and per-agent skipping. Fix .gitignore to anchor chunk-* at root so the skills/ directory is not accidentally ignored.
michael-webster
force-pushed
the
webster/add-review-skill
branch
from
d38d6b9 to
a9c09d4
Compare
| // Skill content is imported at build time. Bun's text loader inlines the file | ||
| // content as a string literal in the compiled binary (requires --loader .md:text | ||
| // in build commands). | ||
| import chunkReviewContent from "../../skills/chunk-review/SKILL.md"; |
There was a problem hiding this comment.
could we just embed them from the .clude/skills directory? Or do we think the ones in there may diverge from the ones we embed
There was a problem hiding this comment.
we certainly could, i started doing that. i was a little concerned about embedding all of the skills in our binary. not that's it's secret, more i expect we end up with additional customizations. let me look around at other projects and how they handle it, bun might let us be selective on what actually get's included.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
| skills: SkillStatus[]; | ||
| } | ||
|
|
||
| const home = process.env.HOME ?? process.env.USERPROFILE ?? "~"; |
There was a problem hiding this comment.
os.homedir() would be safer here — the ~ fallback isn't a valid absolute path and would create a literal ~ directory in cwd.
— Claude Code (dm)
This comment was marked as off-topic.
This comment was marked as off-topic.
2 participants
Summary
chunk skillscommand withinstall,status, andlistsubcommandschunk-reviewskill in the binary using Bun's text loader (--loader .md:text)~/.claude/skills/and~/.codex/skills/for any agent whose config directory already existsskills/at the repo root as the canonical source for embedded skill files (.claude/skills/is kept for in-repo Claude Code usage).gitignoreanchor sochunk-*only ignores root-level binariesReproduction steps
Install skills and verify the file structure:
List bundled skills:
Test plan
bun test src/**/*.unit.test.ts— 23 tests pass (15 new coveringlistSkills,installSkills,getSkillsStatus)bun run typecheck— no errorsbun run lint— no warnings