refactor: move skill payload into skills/vault-onboarding/ - #5
Merged
Conversation
Adopt the agent-skills monorepo layout: SKILL.md, references/, and assets/ now live under skills/vault-onboarding/, isolating the installable payload from repo tooling. Previously the skills CLI treated the repo root as the skill and installed everything — .github/, scripts/, package files included. skills/ is on the CLI's priority search path, so npx skills add aliasunder/vault-onboarding is unchanged, and release zips become the skill directory's contents. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
aliasunder
added a commit
that referenced
this pull request
Aug 14, 2026
The release workflows still zipped SKILL.md, references/, and assets/ from the repo root, but PR #5 moved the skill payload into skills/vault-onboarding/. zip exits 0 when some input paths are unmatched, so v0.1.1 silently shipped a zip containing only README.md and LICENSE. Build from the payload directory and add an unzip guard that fails the step if SKILL.md is missing from the archive. Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
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.
Summary
Adopts the agent-skills monorepo layout:
SKILL.md,references/, andassets/now live underskills/vault-onboarding/, isolating the installable skill payload from repo tooling.Why
The skills CLI checks the repo root for a
SKILL.mdfirst and, when it finds one, treats the entire repo as the skill — the currently installed copy includes.github/,scripts/,package.json,bun.lock, CHANGELOG, and every OSS file. With the payload inskills/vault-onboarding/(a directory on the CLI's priority search path, and the same layout asaliasunder/agent-skillsandvercel-labs/skillsitself), only the skill's own files install. It also makes future release zips trivial: archive the skill directory's contents (the CLI expectsSKILL.mdat archive root).Changes
git mvofSKILL.md,references/(7 files),assets/(19 files) →skills/vault-onboarding/— 100% renames, no content changesscripts/validate-structure.ts: newSKILL_DIRconstant; all payload checks rebased onto itAGENTS.md: structure diagram + prose updated for the new layoutNot changed
npx skills add aliasunder/vault-onboarding— discovery findsskills/vault-onboarding/natively, install command and installed name (from frontmattername) are identicalreferences/…andassets/…pointers — relative within the skill dirVerification
bun run validate— all 56 structural checks pass against the new layout🤖 Generated with Claude Code