Skip to content

v0.7.18

Latest

Choose a tag to compare

@yzhao062 yzhao062 released this 23 Aug 01:19
· 13 commits to main since this release

/vet is a short alias for implement-review.

The skill is the one typed most often and its name is long. Renaming it was
rejected on measurement: the name appears 680 times in the source repo and 503
here, and is also carried by 26 consumer repositories, both published packages,
the IMPLEMENT_REVIEW_* environment variables, and the dispatch
state-directory prefix that auto-watch globs for. The deciding constraint is
that nothing prunes a skill directory that vanishes upstream. Removal happens
only through an explicit pack uninstall, so a rename would strand the old
name deployed in every consumer, still answering the old slash command with
content frozen at the release before the rename.

A pointer costs none of that. .claude/commands/vet.md declares
alias-of: implement-review and names the target skill's three lookup paths,
so the Claude Code slash surface and the documented lookup order resolve the
same SKILL.md. The canonical name does not move.

The pointer contract learned what an alias is, and learned it narrowly: the
key is read from the leading frontmatter block only, duplicates raise, every
pointer must resolve a skill that exists, and vet is pinned to
implement-review by name.

Delivery needed its own fix. The wheel globs .claude/commands/*.md, so an
unmapped pointer rides along inside the artifact and looks shipped while the
composer never writes it and no lock owns it for uninstall. Both manifests now
map vet.md, parity gates it, and a new test asserts that every pointer on
disk has a mapping.

See CHANGELOG.md for the full entry.