Skip to content

feat(add): accept bare skill names without --skill - #7

Open
nisargpatel7042lva wants to merge 1 commit into
ContextVM:mainfrom
nisargpatel7042lva:feat/add-positional-skill-names
Open

nisargpatel7042lva wants to merge 1 commit into
ContextVM:mainfrom
nisargpatel7042lva:feat/add-positional-skill-names

Conversation

@nisargpatel7042lva

Copy link
Copy Markdown

Summary

npx cvmi add server-dev currently treats server-dev as a git source and fails. The intended command is a skill name, matching npx cvmi add with no args for the interactive picker.

Bare identifiers (overview, server-dev) are now treated as --skill values and install from the embedded skills repo. Real sources are unchanged: owner/repo, ./path, and URLs.

--skill still works as an alias.

Testing

  • pnpm exec vitest run src/add.test.ts src/cli.test.ts — 41 passed
  • pnpm test — 200 passed

Made with Cursor

Treat identifiers like overview as skills so npx cvmi add server-dev works. Git sources still use owner/repo, URLs, or paths.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant