Refactor skill distribution; remove bumpy ai command#127
Merged
Conversation
Move the canonical add-change agent skill to the repo root (skills/) as a single source of truth, synced into the package on build/prepack (gitignored copy) so it ships version-pinned in the npm tarball and via the Claude Code plugin. Mirrors dmno-dev/varlock#808. Remove the bumpy ai command: its file-copying targets (opencode/cursor/codex) forked the skill into tool-specific dirs that drifted from canonical and were silently broken in the published package, while the claude target just wrapped claude plugin install. Skill is now installed via the plugin or referenced directly from node_modules.
|
The changes in this PR will be included in the next version bump.
|
Nothing at build time consumes the package-local skills/ copy now that the ai command is gone — prepack (which fires on npm pack/publish) is sufficient.
Now that the canonical skill lives at the repo root, npx skills add and gh skill install resolve it. List all install paths in the README.
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
1 participant
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.
Mirrors the varlock skill-distribution change (dmno-dev/varlock#808) and drops the now-redundant
bumpy aicommand.Skill distribution
skills/add-change/SKILL.mdto the repo root as a single source of truth.build/prepack— newpackages/bumpy/scripts/sync-skill.tscopies the canonical skill into the package (gitignored), so it ships version-pinned in the npm tarball (files: ["skills"]) and via the Claude Code plugin (.claude-plugin/plugin.json).This is mostly structural — the skill was already npm-bundled; this just gives it one canonical source instead of a committed copy inside the package.
Removing
bumpy aiThe
bumpy ai setupcommand is removed:opencode,cursor,codex) forked a stripped-down copy of the skill into tool-specific dirs that drifted from canonical — and had been silently broken in the published package (the bundled CLI resolved../../skillsinstead of../skills, so 3 of the 4 targets failed). Nobody reported it, which says how much it was used.claudetarget was a thin wrapper aroundclaude plugin install @varlock/bumpy.The skill is now installed via the Claude Code plugin (
claude plugin install @varlock/bumpy) or referenced directly fromnode_modules/@varlock/bumpy/skills/add-change/SKILL.md. README anddocs/cli.mdupdated accordingly.Versioned as a patch — the command was a one-time setup helper for new users and was already broken for most targets.
Verification
bun run build→ greenbun run test→ 353 pass, 0 failoxlint+oxfmt --check→ cleanbumpy ai ...→ nowUnknown command: ai;--helpis clean