fix(skills): install all default sandbox skills (build-os ref + resilient install)
Merged
SW
4mo ago+18+18+01234567890123456789-10-10-01234567890123456789
Part of recoupable/chat#1833 (prerequisite item). Base: main.
Problem
Task sandboxes shipped with only 2 of 6 default skills. Two bugs:
DEFAULT_GLOBAL_SKILL_REFSlistedrecoup-platform-build-workspace(renamed torecoup-platform-build-os) →npx skills add404s.installGlobalSkillsthrew on the first failure, so that bad ref aborted the loop and every skill after it (all threerecoup-roster-*) never installed.
Verified on the Apache→OneRPM run (chat#1833): loading recoup-roster-list-artists returned "Skill not found. Available: recoup-platform-api-access, recoup-platform-email-helper."
Fix
defaultGlobalSkillRefs.ts:recoup-platform-build-workspace→recoup-platform-build-os.installGlobalSkills.ts: best-effort per skill —console.error+continueinstead ofthrow, so one bad ref can't block the rest.
Tests (TDD)
- New RED→GREEN test:
installGlobalSkillscontinues past a failed skill, still attempts the rest, logs the failure, never throws. - Full
lib/skillssuite green (49),tsc+ eslint clean.
🤖 Generated with Claude Code
Summary by cubic
Ensure all default sandbox skills install by fixing a renamed ref and making installs resilient. One bad ref no longer blocks the rest.
- Bug Fixes
- Updated
DEFAULT_GLOBAL_SKILL_REFS:recoup-platform-build-workspace→recoup-platform-build-osto stopnpx skills add404s. - Made
installGlobalSkillsbest-effort: never throws; logs and continues so later skills (e.g.,recoup-roster-*) still install.
- Updated
Summary by CodeRabbit
-
New Features
- Updated the default platform setup to include the OS build skill instead of the workspace build skill.
-
Bug Fixes
- Skill installation now continues even if one item fails, allowing the rest of the setup to complete.
- Failed installs are now reported in the console for easier troubleshooting.