Skip to content

fix(cli): fail unmatched skill selectors - #17

Merged
cyhhao merged 1 commit into
mainfrom
fix/issue-16-skill-not-found
May 31, 2026
Merged

fix(cli): fail unmatched skill selectors#17
cyhhao merged 1 commit into
mainfrom
fix/issue-16-skill-not-found

Conversation

@cyhhao

@cyhhao cyhhao commented May 31, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Treat source@name as an explicit skill request when no skills match.
  • Return the same SKILL_NOT_FOUND JSON envelope used by --skill.
  • Add regression coverage for install request selector resolution.

Tests

  • npx tsx --test src/install-request.test.ts
  • npm test
  • npm run build
  • npx tsx src/cli.ts add gh:openclaw/openclaw@tavily -y --json
  • npx tsx src/cli.ts add gh:openclaw/openclaw --skill tavily -y --json
  • npx tsx src/cli.ts add gh:openclaw/openclaw@tavily --list --json

Fixes #16.

@github-actions github-actions Bot added the fix label May 31, 2026
@cyhhao
cyhhao merged commit 73211dd into main May 31, 2026
1 check passed
@cyhhao
cyhhao deleted the fix/issue-16-skill-not-found branch May 31, 2026 12:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

add <source>@<name> reports ok:true and installs nothing when <name> matches no skill (should error like --skill)

1 participant