@skilldeck/core 0.1.0 → 0.1.1

package/lib/cli.mjs

@@ -7,6 +7,7 @@ export const COMMON_OPTIONS = {
   agent: { type: 'string', short: 'a', default: 'claude' },
   project: { type: 'boolean', short: 'p', default: false },
   force: { type: 'boolean', short: 'f', default: false },
+  skill: { type: 'string', short: 's', multiple: true },
   'dry-run': { type: 'boolean', default: false },
   help: { type: 'boolean', short: 'h', default: false },
   version: { type: 'boolean', short: 'v', default: false },
@@ -15,6 +16,7 @@ export const COMMON_OPTIONS = {
 export const OPTIONS_HELP = `Options:
   -a, --agent <name>   claude (default), codex, agents, all, or a comma list
   -p, --project        install into ./.claude (or ./.codex, ./.agents) instead of your home
+  -s, --skill <name>   only these skills from the pack (repeat, or comma-separate)
   -f, --force          replace skill folders that SkillDeck didn't install
       --dry-run        show what would happen without writing anything
   -h, --help           show help
@@ -35,14 +37,15 @@ export function parse(argv, extra = {}) {
 }
 
 export function toOpts(values) {
-  return { agent: values.agent, project: values.project, force: values.force, dryRun: values['dry-run'] }
+  const skills = (values.skill ?? []).flatMap((s) => s.split(',')).map((s) => s.trim()).filter(Boolean)
+  return { agent: values.agent, project: values.project, force: values.force, dryRun: values['dry-run'], skills: skills.length ? skills : undefined }
 }
 
 export function printInstall(pack, results) {
   for (const r of results) {
     const verb = r.dryRun ? 'would install' : r.upgraded ? 'updated' : 'installed'
     console.log(`${green('✓')} ${bold(pack.name)}@${pack.version} ${verb} for ${r.label} ${dim('→ ' + tilde(r.skillsDir))}`)
-    console.log(dim(`  skills: ${r.skills.join(', ')}`))
+    console.log(dim(`  skills: ${r.skills.join(', ')}${r.partial ? ' (selected)' : ''}`))
     if (r.hook) console.log(dim(`  hook:   ${r.hook.event} in ${tilde(r.hook.settingsFile)}`))
     if (r.note) console.log(yellow(`  note:   ${r.note}`))
   }
@@ -50,7 +53,7 @@ export function printInstall(pack, results) {
 
 export function printUninstall(name, results) {
   for (const r of results) {
-    if (r.removed) console.log(`${green('✓')} removed ${bold(name)} from ${r.label}${r.hook ? dim(' (and its hook)') : ''}`)
+    if (r.removed) console.log(`${green('✓')} removed ${bold(r.partial ? `${name}: ${r.skills.join(', ')}` : name)} from ${r.label}${r.hook ? dim(' (and its hook)') : ''}`)
     else console.log(dim(`- ${name} is not installed for ${r.label}`))
   }
 }

package/lib/install.mjs

@@ -15,14 +15,37 @@ function hookCommandFor(target, pack) {
   return target.hookCommand(`${pack.name}/${pack.hook.script}`)
 }
 
-/** Install a pack for each requested agent. Throws before writing anything if a conflict is found. */
+/** Skills a hook depends on (e.g. the bootstrap skill it injects). The hook is installed only with them. */
+const hookSkills = (pack) => (pack.hook?.include ?? []).filter((p) => p.startsWith('skills/')).map((p) => p.split('/')[1])
+
+/** Validate an optional `skills` subset against what the pack ships. */
+function selectSkills(pack, wanted) {
+  if (!wanted?.length) return { selected: pack.skills, partial: false }
+  const unknown = wanted.filter((s) => !pack.skills.includes(s))
+  if (unknown.length) throw new Error(`${pack.name} has no skill named ${unknown.join(', ')}. Available: ${pack.skills.join(', ')}`)
+  const selected = pack.skills.filter((s) => wanted.includes(s))
+  return { selected, partial: selected.length < pack.skills.length }
+}
+
+function removeHookFiles(hook) {
+  removeHook(hook.settingsFile, { event: hook.event, command: hook.command })
+  fs.rmSync(hook.dir, { recursive: true, force: true })
+  const parent = path.dirname(hook.dir)
+  if (fs.existsSync(parent) && fs.readdirSync(parent).length === 0) fs.rmdirSync(parent)
+}
+
+/**
+ * Install a pack (or, with `skills`, just some of its skills) for each requested agent.
+ * Throws before writing anything if a conflict is found.
+ */
 export function install(pack, opts = {}) {
   const { force = false, dryRun = false } = opts
+  const { selected, partial } = selectSkills(pack, opts.skills)
   const agents = expandAgents(opts.agent)
   const plans = agents.map((agent) => {
     const target = resolveTarget(agent, opts)
     const manifest = readManifest(target.skillsDir)
-    const conflicts = pack.skills.filter((s) => {
+    const conflicts = selected.filter((s) => {
       const owner = ownerOf(manifest, s)
       return owner !== pack.name && (owner || fs.existsSync(path.join(target.skillsDir, s)))
     })
@@ -35,26 +58,32 @@ export function install(pack, opts = {}) {
     throw new Error(`These skill folders already exist and weren't installed by ${pack.name}:\n${lines.join('\n')}\nRe-run with --force to replace them.`)
   }
 
+  const wantsHook = Boolean(pack.hook) && (!partial || hookSkills(pack).every((s) => selected.includes(s)))
+
   return plans.map(({ target, manifest }) => {
     const previous = manifest.packs[pack.name]
-    const result = { agent: target.agent, label: target.label, skillsDir: target.skillsDir, skills: pack.skills, hook: null, upgraded: Boolean(previous), dryRun }
+    const result = { agent: target.agent, label: target.label, skillsDir: target.skillsDir, skills: selected, partial, hook: null, upgraded: Boolean(previous), dryRun }
 
     if (!dryRun) {
-      for (const skill of pack.skills) {
+      for (const skill of selected) {
         const dst = path.join(target.skillsDir, skill)
         fs.rmSync(dst, { recursive: true, force: true })
         fs.mkdirSync(target.skillsDir, { recursive: true })
         copyTree(path.join(pack.root, 'skills', skill), dst)
       }
-      // Drop skills a previous version shipped but this one doesn't.
-      for (const old of previous?.skills ?? []) {
-        if (!pack.skills.includes(old)) fs.rmSync(path.join(target.skillsDir, old), { recursive: true, force: true })
+      // A full install drops skills a previous version shipped but this one doesn't.
+      if (!partial) {
+        for (const old of previous?.skills ?? []) {
+          if (!pack.skills.includes(old)) fs.rmSync(path.join(target.skillsDir, old), { recursive: true, force: true })
+        }
       }
     }
 
-    const entry = { version: pack.version, skills: pack.skills, installedAt: new Date().toISOString() }
+    // Partial installs add to what's already there.
+    const skills = partial ? pack.skills.filter((s) => selected.includes(s) || previous?.skills?.includes(s)) : pack.skills
+    const entry = { version: pack.version, skills, installedAt: new Date().toISOString() }
 
-    if (pack.hook && target.hooks) {
+    if (wantsHook && target.hooks) {
       const hookDir = path.join(target.hookRoot, pack.name)
       const command = hookCommandFor(target, pack)
       if (!dryRun) {
@@ -66,9 +95,11 @@ export function install(pack, opts = {}) {
         }
         addHook(target.settingsFile, { event: pack.hook.event, matcher: pack.hook.matcher, command })
       }
-      entry.hook = { event: pack.hook.event, command, dir: hookDir, settingsFile: target.settingsFile }
+      entry.hook = { event: pack.hook.event, command, dir: hookDir, settingsFile: target.settingsFile, skills: hookSkills(pack) }
       result.hook = entry.hook
-    } else if (pack.hook) {
+    } else if (previous?.hook) {
+      entry.hook = previous.hook
+    } else if (wantsHook) {
       result.note = `${target.label} has no hook system: skills installed, but the session-start bootstrap won't run automatically.`
     }
 
@@ -80,7 +111,7 @@ export function install(pack, opts = {}) {
   })
 }
 
-/** Remove exactly what the manifest says this pack installed. */
+/** Remove exactly what the manifest says this pack installed (or, with `skills`, only those skills). */
 export function uninstall(packName, opts = {}) {
   const agents = expandAgents(opts.agent)
   return agents.map((agent) => {
@@ -88,18 +119,27 @@ export function uninstall(packName, opts = {}) {
     const manifest = readManifest(target.skillsDir)
     const entry = manifest.packs[packName]
     if (!entry) return { agent, label: target.label, removed: false }
+
+    const installed = entry.skills ?? []
+    const wanted = opts.skills?.length ? opts.skills : installed
+    const removing = installed.filter((s) => wanted.includes(s))
+    if (removing.length === 0) return { agent, label: target.label, removed: false }
+    const remaining = installed.filter((s) => !removing.includes(s))
+    // The hook goes with the last skill, or with the skill it injects.
+    const hook = entry.hook ?? null
+    const dropHook = Boolean(hook) && (remaining.length === 0 || (hook.skills ?? []).some((s) => removing.includes(s)))
+
     if (!opts.dryRun) {
-      for (const s of entry.skills ?? []) fs.rmSync(path.join(target.skillsDir, s), { recursive: true, force: true })
-      if (entry.hook) {
-        removeHook(entry.hook.settingsFile, { event: entry.hook.event, command: entry.hook.command })
-        fs.rmSync(entry.hook.dir, { recursive: true, force: true })
-        const parent = path.dirname(entry.hook.dir)
-        if (fs.existsSync(parent) && fs.readdirSync(parent).length === 0) fs.rmdirSync(parent)
+      for (const s of removing) fs.rmSync(path.join(target.skillsDir, s), { recursive: true, force: true })
+      if (dropHook) removeHookFiles(hook)
+      if (remaining.length === 0) delete manifest.packs[packName]
+      else {
+        entry.skills = remaining
+        if (dropHook) delete entry.hook
       }
-      delete manifest.packs[packName]
       writeManifest(target.skillsDir, manifest)
     }
-    return { agent, label: target.label, removed: true, skills: entry.skills, hook: entry.hook ?? null }
+    return { agent, label: target.label, removed: true, skills: removing, partial: remaining.length > 0, hook: dropHook ? hook : null }
   })
 }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@skilldeck/core",
-  "version": "0.1.0",
+  "version": "0.1.1",
   "description": "Installer library for SkillDeck skill packs: copies SKILL.md folders into Claude Code, Codex or .agents and wires hooks.",
   "type": "module",
   "exports": {