@victortomaili/skill-cli 0.1.0 → 0.1.1

package/CHANGELOG.md

@@ -7,13 +7,15 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
-### Added
-- "Star the repo" call to action in `skill --help` (with the `gh repo star` command) and the README.
-
 ### Planned
 - Cursor adapter (`.cursor/rules` format) for `init -g` bootstrap.
 - Per-agent hook adapters for automatic `/X` triggering (push model).
 
+## [0.1.1] - 2026-07-07
+
+### Fixed
+- `skill trigger <name>` now loads a skill by exact name when no `triggers:` keyword matches. Skills without a `triggers:` field (e.g. description-triggered skills imported from the `skills` / `vercel-labs` ecosystem) were previously unreachable via `trigger`. A passive skill matched by name now shows an enable hint (`skill enable` / `skill cat`) instead of a dead-end "No active skill" message.
+
 ## [0.1.0] - 2026-07-07
 
 ### Added
@@ -46,5 +48,6 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - Windows `npx` spawn runs with `shell:false` + an args array and rejects sources
   containing shell metacharacters (`& | < > ^`).
 
-[Unreleased]: https://github.com/victortomaili/skill-cli/compare/v0.1.0...HEAD
+[Unreleased]: https://github.com/victortomaili/skill-cli/compare/v0.1.1...HEAD
+[0.1.1]: https://github.com/victortomaili/skill-cli/compare/v0.1.0...v0.1.1
 [0.1.0]: https://github.com/victortomaili/skill-cli/releases/tag/v0.1.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@victortomaili/skill-cli",
-  "version": "0.1.0",
+  "version": "0.1.1",
   "description": "Cross-agent skill manager — one global store, clean agent folders, activation via skill.config",
   "license": "MIT",
   "author": "Victor Tomaili <victor@tomaili.com>",

package/src/cli.js

@@ -29,7 +29,7 @@ ${c.bold('Activation')}
 ${c.bold('Usage (agent)')}
   ${c.cyan('skill show')} ${c.gray('<name>')}         metadata + path + triggers
   ${c.cyan('skill cat')} ${c.gray('<name>')}          dump content to context
-  ${c.cyan('skill trigger')} ${c.gray('<keyword>')}   /X trigger: single→content, multi→candidates
+  ${c.cyan('skill trigger')} ${c.gray('<keyword|name>')}   /X trigger or skill name → content
 
 ${c.bold('Maintenance')}
   ${c.cyan('skill update')} ${c.gray('[name|--all]')} refresh store from source

package/src/commands/trigger.js

@@ -4,43 +4,69 @@ import { readGlobalConfig, readProjectConfig, computeEffective } from '../lib/co
 import { normalizeTrigger } from '../lib/frontmatter.js'
 import { trunc } from '../lib/format.js'
 
-// /X trigger: keyword match across ACTIVE skills.
-//   single  → dump skill content (drops into context)
-//   multi   → candidate list (name + description); agent picks with `skill cat <name>`
-//   none    → informational
+// Dump a single skill's body into context. Shared by the trigger-keyword and
+// exact-name code paths so both render identically.
+function loadSingle(skill, via) {
+  const s = readSkill(skill.name)
+  if (!s) { console.error(c.red('Skill not readable: ' + skill.name)); process.exit(1) }
+  console.log(c.green('▼ skill: ') + c.bold(s.name) + c.gray('  (' + via + ')'))
+  console.log(c.gray('━'.repeat(56)))
+  console.log(s.body.trim())
+}
+
+// `skill trigger <keyword|name>`:
+//   1. keyword matches a trigger across ACTIVE skills
+//        single  → dump content (drops into context)
+//        multi   → candidate list (name + description); pick with `skill cat <name>`
+//   2. no trigger match → exact NAME match among active skills. Lets you load
+//      description-triggered skills that have no `triggers:` field (e.g. skills
+//      imported from the `skills` / vercel-labs ecosystem) by name.
+//   3. name matches only a PASSIVE skill → hint how to activate (or read via cat).
+//   4. nothing → informational.
 export function cmdTrigger(args) {
   const keyword = normalizeTrigger(args[0] || '')
   if (!keyword) {
-    console.error(c.red('Usage: skill trigger <keyword>   (e.g. /research)'))
+    console.error(c.red('Usage: skill trigger <keyword|name>   (e.g. /research, web-design-guidelines)'))
     process.exit(1)
   }
 
   const installed = listStore()
   const effective = computeEffective(installed, readGlobalConfig(), readProjectConfig())
-  // Search ONLY active skills — a passive skill cannot trigger.
+
+  // 1) trigger-keyword match across ACTIVE skills — a passive skill cannot trigger.
   const candidates = installed.filter(s => effective.includes(s.name) && s.triggers.includes(keyword))
 
-  if (candidates.length === 0) {
-    console.log(c.yellow('No active skill has the ') + c.cyan('/' + keyword) + c.yellow(' trigger.'))
-    console.log(c.gray('  Run skill list to see active skills and their triggers.'))
+  if (candidates.length >= 2) {
+    console.log(c.yellow('/' + keyword) + ' matched ' + c.bold(candidates.length + ' skills') + ' — pick one:')
+    console.log()
+    for (const s of candidates) {
+      console.log('  ' + c.cyan(s.name.padEnd(20)) + c.gray('  ' + trunc(s.description, 54)))
+    }
+    console.log()
+    console.log(c.gray('Pick: ') + 'skill cat <name>')
     return
   }
-
   if (candidates.length === 1) {
-    const s = readSkill(candidates[0].name)
-    if (!s) { console.error(c.red('Skill not readable: ' + candidates[0].name)); process.exit(1) }
-    console.log(c.green('▼ skill: ') + c.bold(s.name) + c.gray('  (triggered: /' + keyword + ')'))
-    console.log(c.gray('━'.repeat(56)))
-    console.log(s.body.trim())
+    loadSingle(candidates[0], 'triggered: /' + keyword)
+    return
+  }
+
+  // 2) no trigger match → exact-name fallback among ACTIVE skills.
+  const byName = installed.filter(s => effective.includes(s.name) && s.name.toLowerCase() === keyword)
+  if (byName.length === 1) {
+    loadSingle(byName[0], 'name: ' + byName[0].name)
     return
   }
 
-  console.log(c.yellow('/' + keyword) + ' matched ' + c.bold(candidates.length + ' skills') + ' — pick one:')
-  console.log()
-  for (const s of candidates) {
-    console.log('  ' + c.cyan(s.name.padEnd(20)) + c.gray('  ' + trunc(s.description, 54)))
+  // 3) name matches only a PASSIVE skill → it's installed but not active.
+  const passive = installed.find(s => !effective.includes(s.name) && s.name.toLowerCase() === keyword)
+  if (passive) {
+    console.log(c.yellow('Skill ') + c.bold(passive.name) + c.yellow(' is installed but not active.'))
+    console.log(c.gray('  Enable: ') + c.cyan('skill enable [-g] ' + passive.name) + c.gray('  · read now: ') + c.cyan('skill cat ' + passive.name))
+    return
   }
-  console.log()
-  console.log(c.gray('Pick: ') + 'skill cat <name>')
-}
 
+  // 4) nothing matched.
+  console.log(c.yellow('No active skill has the ') + c.cyan('/' + keyword) + c.yellow(' trigger.'))
+  console.log(c.gray('  Run skill list to see active skills and their triggers.'))
+}