Skip to content

fix(npm): bundle auto-discovers skills (ships style-deep) #22

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
aglumova merged 1 commit into from
Jun 27, 2026
Merged

fix(npm): bundle auto-discovers skills (ships style-deep) #22

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions CHANGELOG.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,9 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0

## [Unreleased]

### Fixed
- 0.36.0 shipped without the `mastermind-style-deep` skill in the default install — it was missing from the `stage-npm-share.sh` bundle allowlist, so `mastermind install` never copied it to `~/.claude/skills/`. The bundle now auto-discovers every skill under `skills/` instead of a hand-maintained allowlist (`extras/` stays excluded as a separate tree), so the skill ships and future skills can't be forgotten.

## [0.36.0] - 2026-06-27

### Added
Expand Down
34 changes: 11 additions & 23 deletions scripts/stage-npm-share.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -19,30 +19,18 @@ mkdir -p "$SHARE/agents" "$SHARE/skills"
# Subagents — flat `.md` files.
cp "$REPO_ROOT"/agents/subagents/*.md "$SHARE/agents/"

# Core skills — explicit allowlist. Add here only skills that belong in the
# default install (intake → plan → execute → audit loop).
CORE_SKILLS=(
skills/workflow/mastermind-task-planning
skills/workflow/mastermind-task-executor
skills/workflow/mastermind-codegraph-research
skills/workflow/mastermind-structured-report-contract
skills/workflow/mastermind-critical-review
skills/prompt-engineering/mastermind-prompt-refiner
skills/debugging/mastermind-investigation-ledger
skills/security/mastermind-agent-security-review
skills/coding/no-ai-slop-comments
)

for skill_dir in "${CORE_SKILLS[@]}"; do
src="$REPO_ROOT/$skill_dir"
name="$(basename "$skill_dir")"
if [ ! -d "$src" ]; then
echo "ERROR: core skill not found: $src" >&2
exit 1
fi
cp -R "$src" "$SHARE/skills/$name"
done
# Core skills — every skill under `skills/` ships in the default install. Non-core
# artifacts live in `extras/` (a separate tree), which this never scans, so adding
# a skill under `skills/` ships it automatically — no allowlist to forget.
while IFS= read -r skill_md; do
src="$(dirname "$skill_md")"
cp -R "$src" "$SHARE/skills/$(basename "$src")"
done < <(find "$REPO_ROOT/skills" -name SKILL.md | sort)

agents_n=$(find "$SHARE/agents" -name '*.md' | wc -l | tr -d ' ')
skills_n=$(find "$SHARE/skills" -mindepth 1 -maxdepth 1 -type d | wc -l | tr -d ' ')
if [ "$agents_n" -eq 0 ] || [ "$skills_n" -eq 0 ]; then
echo "ERROR: staged $agents_n subagents + $skills_n skills — discovery found nothing" >&2
exit 1
fi
echo "staged $agents_n subagents + $skills_n skills → npm/mastermind/share/"