harveyz-skill 0.6.2 → 0.8.1

package/bin/cli.js

@@ -7,13 +7,13 @@ import os from 'os'
 import path from 'path'
 import { fileURLToPath } from 'url'
 import {
-  getAllSkillItems, getAllToolItems,
+  getAllSkillItems, getAllToolItems, getAllHookItems, checkHookInstalled,
   checkInstalled, checkToolInstalled, scopeSummary,
   resolveSkills, resolveSkillsByName, resolveTools, resolveToolsByName,
   TOOL_BUNDLE_CHOICES,
 } from '../lib/bundles.js'
 import { buildTargetChoices, resolveTargets, TARGETS } from '../lib/targets.js'
-import { installSkills, installTools } from '../lib/installer.js'
+import { installSkills, installTools, installHooks, uninstallHook } from '../lib/installer.js'
 
 const __dirname = path.dirname(fileURLToPath(import.meta.url))
 const require = createRequire(import.meta.url)
@@ -21,25 +21,31 @@ const { version } = require('../package.json')
 
 const args = process.argv.slice(2)
 const subcommand = args[0]
+const jsonFlag = args.includes('--json')
 
 // ── Help ─────────────────────────────────────────────────────────────────────
 function printHelp() {
   console.log(`
-  ${chalk.bold('hskill')} — skill manager for Claude Code, Cursor, and Codex  v${version}
+  ${chalk.bold('hskill')} — skill manager for Claude Code, Cursor, Codex, OpenClaw, and Hermes  v${version}
 
   ${chalk.cyan('Usage:')}
-    hskill                         interactive install
+    hskill                         interactive install (requires TTY + fzf)
     hskill install                 interactive install (explicit)
     hskill install --bundle <b>    install a skill bundle
     hskill install --skill <s>     install specific skill(s)
     hskill install --tool <t>      install shell tool(s)
-    hskill install --target <t>    set target (claude/cursor/codex/all)
+    hskill install --target <t>    set target (claude/cursor/codex/openclaw/hermes/all)
     hskill install --scope <s>     set scope: user (default) or project
     hskill install --force         overwrite existing installs
-    hskill list                    list available skills and bundles
-    hskill status                  show install status for all skills and tools
-    hskill outdated                list skills and tools with available updates
-    hskill info <name>             show install detail for a skill or tool
+    hskill list [--json]           list available skills and bundles
+    hskill status [--json]         show install status for all skills and tools
+    hskill outdated [--json]       list skills and tools with available updates
+    hskill info <name> [--json]    show install detail for a skill or tool
+    hskill hooks list [--json]                        list hooks and install status
+    hskill hooks install [--name <n>] [--scope <s>]  install hook (scope: user|project)
+    hskill hooks install [--project <path>]           target project dir (for project scope)
+    hskill hooks install [--force]                    overwrite existing
+    hskill hooks uninstall <name> [--scope <s>]       remove hook
     hskill update                  update hskill to the latest version
     hskill version                 show version
     hskill --help                  show this help
@@ -49,12 +55,72 @@ function printHelp() {
     hskill install --skill git-workflow-init --target claude --scope project
     hskill install --tool p-launch
     hskill status
+    hskill status --json
     hskill outdated
     hskill info git-workflow-init
+
+  ${chalk.cyan('Agent / CI usage:')}
+    Use --json for machine-readable output on status, list, outdated, info.
+    Set NO_COLOR=1 to suppress ANSI color codes in all output.
+    Interactive mode (no flags) requires a TTY; use --bundle/--skill/--tool in scripts.
 `)
 }
 
 if (args[0] === '--help' || args[0] === '-h') {
+  if (jsonFlag || args.includes('--json')) {
+    console.log(JSON.stringify({
+      name: 'hskill',
+      version,
+      description: 'Skill manager for Claude Code, Cursor, Codex, OpenClaw, and Hermes',
+      agent_notes: 'Interactive mode requires TTY. Use --json for machine-readable output. Set NO_COLOR=1 to suppress ANSI codes.',
+      commands: [
+        {
+          name: 'install',
+          description: 'Install skills or shell tools',
+          interactive_fallback: 'Requires TTY + fzf when no flags given',
+          note: '--skill and --tool are mutually exclusive; use --bundle to install both',
+          flags: [
+            { name: '--bundle', arg: '<name>',   description: 'Install a skill bundle (comma-separated)' },
+            { name: '--skill',  arg: '<name>',   description: 'Install specific skill(s) (comma-separated)' },
+            { name: '--tool',   arg: '<name>',   description: 'Install shell tool(s) (comma-separated)' },
+            { name: '--target', arg: '<target>', description: 'Install target', enum: ['claude','cursor','codex','openclaw','hermes','all'] },
+            { name: '--scope',  arg: '<scope>',  description: 'Install scope', enum: ['user','project'], default: 'user' },
+            { name: '--force',  description: 'Overwrite existing installs' },
+          ],
+        },
+        {
+          name: 'list',
+          description: 'List available skills and bundles',
+          flags: [{ name: '--json', description: 'Machine-readable output' }],
+        },
+        {
+          name: 'status',
+          description: 'Show install status for all skills and tools',
+          flags: [{ name: '--json', description: 'Machine-readable output' }],
+        },
+        {
+          name: 'outdated',
+          description: 'List skills and tools with available updates',
+          flags: [{ name: '--json', description: 'Machine-readable output' }],
+        },
+        {
+          name: 'info',
+          description: 'Show install detail for a skill or tool',
+          args: ['<name>'],
+          flags: [{ name: '--json', description: 'Machine-readable output' }],
+        },
+        {
+          name: 'update',
+          description: 'Update hskill to the latest version via npm',
+        },
+        {
+          name: 'version',
+          description: 'Print version and exit',
+        },
+      ],
+    }, null, 2))
+    process.exit(0)
+  }
   printHelp()
   process.exit(0)
 }
@@ -85,6 +151,14 @@ if (subcommand === 'list') {
     if (!byBundle[s.bundle]) byBundle[s.bundle] = []
     byBundle[s.bundle].push(s.path)
   }
+  if (jsonFlag) {
+    const bundles = {}
+    for (const [name, paths] of Object.entries(byBundle)) {
+      bundles[name] = { description: bundleMeta[name] ?? name, skills: paths }
+    }
+    console.log(JSON.stringify({ bundles, tools: tools.map(t => t.name) }, null, 2))
+    process.exit(0)
+  }
   for (const [name, paths] of Object.entries(byBundle)) {
     console.log(chalk.bold(name) + ' — ' + (bundleMeta[name] ?? name))
     for (const p of paths) console.log('  ' + p)
@@ -97,11 +171,20 @@ if (subcommand === 'list') {
   process.exit(0)
 }
 
+// ── Helper: resolve displayed version for a hook ──────────────────────────────
+// Priority: user-installed → project-installed → source version
+function resolveHookDisplayVersion(inst, sourceVersion) {
+  if (inst.user.version !== '—') return inst.user.version
+  if (inst.project.version !== '—') return inst.project.version
+  return sourceVersion ?? '—'
+}
+
 // ── Status / Outdated ─────────────────────────────────────────────────────────
 if (subcommand === 'status' || subcommand === 'outdated') {
   const outdatedOnly = subcommand === 'outdated'
   const skillItems   = getAllSkillItems()
   const toolItems    = getAllToolItems()
+  const hookItems    = getAllHookItems()
 
   function icon(status) {
     if (status === 'up-to-date') return chalk.green('✓')
@@ -122,6 +205,30 @@ if (subcommand === 'status' || subcommand === 'outdated') {
     return { name: t.toolName, version: t.version ?? '—', ...inst }
   })
 
+  if (jsonFlag) {
+    const targets = ['claude', 'cursor', 'codex', 'openclaw', 'hermes']
+    const jsonSkills = skillRows.map(r => ({
+      name: r.name,
+      version: r.version,
+      user:    Object.fromEntries(targets.map(t => [t, r.userDetail[t]])),
+      project: Object.fromEntries(targets.map(t => [t, r.projectDetail[t]])),
+    }))
+    const jsonTools = toolRows.map(r => ({ name: r.name, version: r.version, status: r.status }))
+    const jsonHooks = hookItems.map(h => {
+      const inst = checkHookInstalled(h.name)
+      return { name: h.name, description: h.description, user: inst.user, project: inst.project }
+    })
+    if (outdatedOnly) {
+      console.log(JSON.stringify({
+        skills: jsonSkills.filter(s => Object.values(s.user).some(v => v.status === 'update') || Object.values(s.project).some(v => v.status === 'update')),
+        tools:  jsonTools.filter(t => t.status === 'update'),
+      }, null, 2))
+    } else {
+      console.log(JSON.stringify({ skills: jsonSkills, tools: jsonTools, hooks: jsonHooks }, null, 2))
+    }
+    process.exit(0)
+  }
+
   if (outdatedOnly) {
     const outdatedSkills = skillRows.filter(r => r.userStatus === 'update' || r.projectStatus === 'update')
     const outdatedTools  = toolRows.filter(r => r.status === 'update')
@@ -131,7 +238,7 @@ if (subcommand === 'status' || subcommand === 'outdated') {
       process.exit(0)
     }
 
-    const targets = ['claude', 'cursor', 'codex']
+    const targets = ['claude', 'cursor', 'codex', 'openclaw', 'hermes']
 
     if (outdatedSkills.length) {
       console.log('\n  ' + chalk.bold('SKILLS WITH UPDATES'))
@@ -161,7 +268,7 @@ if (subcommand === 'status' || subcommand === 'outdated') {
   }
 
   // Full status table
-  const allNames = [...skillRows, ...toolRows].map(r => r.name)
+  const allNames = [...skillRows.map(r => r.name), ...toolRows.map(r => r.name), ...hookItems.map(h => h.name)]
   const allVers  = [...skillRows, ...toolRows].map(r => r.version)
   const nw = Math.max(...allNames.map(n => n.length), 4)
   const vw = Math.max(...allVers.map(v => v.length), 7)
@@ -183,6 +290,23 @@ if (subcommand === 'status' || subcommand === 'outdated') {
     console.log('  ' + r.name.padEnd(nw) + '  ' + chalk.dim(r.version.padEnd(vw)) + '  ' + icon(r.status))
   }
 
+  // ── hooks ──
+  if (hookItems.length > 0) {
+    console.log('')
+    console.log('  ' + chalk.bold('HOOKS') + chalk.dim(`  — ${hookItems.length} available`))
+    console.log(sep)
+    function hIcon(s) {
+      if (s === 'installed') return chalk.green('✓')
+      if (s === 'partial')   return chalk.yellow('~')
+      return chalk.dim('—')
+    }
+    for (const h of hookItems) {
+      const inst = checkHookInstalled(h.name)
+      const ver = resolveHookDisplayVersion(inst, h.version)
+      console.log('  ' + h.name.padEnd(nw) + '  ' + chalk.dim(ver.padEnd(vw)) + '  ' + hIcon(inst.user.status) + '       ' + hIcon(inst.project.status) + '  ' + chalk.dim(h.description))
+    }
+  }
+
   const installedSkills = skillRows.filter(r => r.userStatus !== 'none' || r.projectStatus !== 'none').length
   const installedTools  = toolRows.filter(r => r.status !== 'none').length
   const outdatedCount   = skillRows.filter(r => r.userStatus === 'update' || r.projectStatus === 'update').length
@@ -220,6 +344,24 @@ if (subcommand === 'info') {
     return chalk.dim('— not installed')
   }
 
+  if (jsonFlag) {
+    if (skill) {
+      const inst = checkInstalled(skill.skillName, skill.version ?? '—')
+      console.log(JSON.stringify({
+        name: skill.skillName, type: 'skill', version: skill.version ?? '—',
+        user:    Object.fromEntries(targets.map(t => [t, inst.user[t]])),
+        project: Object.fromEntries(targets.map(t => [t, inst.project[t]])),
+      }, null, 2))
+    } else {
+      const inst = checkToolInstalled(tool.toolName, tool.srcPath)
+      console.log(JSON.stringify({
+        name: tool.toolName, type: 'tool', version: tool.version ?? '—',
+        installed: inst,
+      }, null, 2))
+    }
+    process.exit(0)
+  }
+
   console.log('')
   if (skill) {
     const inst = checkInstalled(skill.skillName, skill.version ?? '—')
@@ -246,6 +388,115 @@ if (subcommand === 'info') {
   process.exit(0)
 }
 
+// ── Hooks ─────────────────────────────────────────────────────────────────────
+if (subcommand === 'hooks') {
+  const hooksSubcmd    = args[1]
+  const hookArgs       = args.slice(2)
+  const hookJsonFlag   = hooksSubcmd === '--json' || hookArgs.includes('--json')
+  const hookNameIdx    = hookArgs.indexOf('--name')
+  const hookScopeIdx   = hookArgs.indexOf('--scope')
+  const hookProjectIdx = hookArgs.indexOf('--project')
+  const hookForce      = hookArgs.includes('--force')
+  const hookNameArg    = hookNameIdx    !== -1 ? hookArgs[hookNameIdx    + 1] : undefined
+  const hookScopeArg   = hookScopeIdx   !== -1 ? hookArgs[hookScopeIdx   + 1] : 'user'
+  const hookProjectArg = hookProjectIdx !== -1 ? hookArgs[hookProjectIdx + 1] : process.cwd()
+
+  // Validate scope for install/uninstall commands
+  if (!['user', 'project'].includes(hookScopeArg) && ['install', 'uninstall'].includes(hooksSubcmd)) {
+    console.error(chalk.red(`  ✗ Invalid scope: "${hookScopeArg}". Use user or project.`))
+    process.exit(1)
+  }
+
+  // ── hooks list ──────────────────────────────────────────────────────────────
+  if (hooksSubcmd === 'list' || !hooksSubcmd || hooksSubcmd.startsWith('--')) {
+    const hookItems = getAllHookItems()
+    if (hookJsonFlag) {
+      const out = hookItems.map(h => {
+        const inst = checkHookInstalled(h.name)
+        const ver = resolveHookDisplayVersion(inst, h.version)
+        return { name: h.name, version: ver, description: h.description, event: h.event, user: inst.user, project: inst.project }
+      })
+      console.log(JSON.stringify({ hooks: out }, null, 2))
+      process.exit(0)
+    }
+    function hookIcon(s) {
+      if (s === 'installed') return chalk.green('✓')
+      if (s === 'partial')   return chalk.yellow('~')
+      return chalk.dim('—')
+    }
+    const nameWidth = Math.max(...hookItems.map(h => h.name.length), 4)
+    const verWidth = 7
+    console.log('')
+    console.log('  ' + chalk.bold('NAME'.padEnd(nameWidth)) + '  ' + chalk.bold('VER'.padEnd(verWidth)) + '  U  P  ' + chalk.bold('DESCRIPTION'))
+    console.log('  ' + '─'.repeat(nameWidth) + '  ' + '─'.repeat(verWidth) + '  ─  ─  ' + '─'.repeat(20))
+    for (const h of hookItems) {
+      const inst = checkHookInstalled(h.name)
+      const ver = resolveHookDisplayVersion(inst, h.version)
+      console.log('  ' + h.name.padEnd(nameWidth) + '  ' + chalk.dim(ver.padEnd(verWidth)) + '  ' + hookIcon(inst.user.status) + '  ' + hookIcon(inst.project.status) + '  ' + h.description)
+    }
+    console.log('')
+    console.log(chalk.dim(`  U=user  P=project  ${chalk.green('✓')}=installed  ${chalk.yellow('~')}=partial  ${chalk.dim('—')}=none`))
+    console.log('')
+    process.exit(0)
+  }
+
+  // ── hooks install ────────────────────────────────────────────────────────────
+  if (hooksSubcmd === 'install') {
+    const hookItems = getAllHookItems()
+    let toInstall
+
+    if (hookNameArg) {
+      const found = hookItems.find(h => h.name === hookNameArg)
+      if (!found) {
+        console.error(chalk.red(`  ✗ Unknown hook: "${hookNameArg}"`))
+        process.exit(1)
+      }
+      toInstall = [found]
+    } else if (!process.stdout.isTTY) {
+      toInstall = hookItems
+    } else {
+      const { checkbox } = await import('@inquirer/prompts')
+      const selected = await checkbox({
+        message: 'Select hooks to install:',
+        choices: hookItems.map(h => ({ name: `${h.name.padEnd(32)} ${h.description}`, value: h })),
+      })
+      if (!selected.length) {
+        console.log(chalk.dim('  · Nothing selected'))
+        process.exit(0)
+      }
+      toInstall = selected
+    }
+
+    const { installed, skipped, failed } = await installHooks(toInstall, hookScopeArg, hookProjectArg, hookForce)
+
+    if (hookJsonFlag) {
+      console.log(JSON.stringify({ installed, skipped, failed }, null, 2))
+      process.exit(failed.length ? 1 : 0)
+    } else {
+      if (installed.length) console.error(chalk.green.bold(`✔ Hooks installed (${hookScopeArg}):`), installed.join(', '))
+      for (const s of skipped) console.error(chalk.dim(`  · ${s.name} skipped (${s.reason})`))
+
+      for (const f of failed)  console.error(chalk.red(`  ✗ ${f.name} failed: ${f.reason}${f.detail ? ` — ${f.detail}` : ''}`))
+      process.exit(failed.length ? 1 : 0)
+    }
+  }
+
+  // ── hooks uninstall ──────────────────────────────────────────────────────────
+  if (hooksSubcmd === 'uninstall') {
+    const nameToRemove = args[2]
+    if (!nameToRemove || nameToRemove.startsWith('--')) {
+      console.error(chalk.red('  ✗ Usage: hskill hooks uninstall <name> [--scope user|project]'))
+      process.exit(1)
+    }
+    const { removed } = await uninstallHook(nameToRemove, hookScopeArg, hookProjectArg)
+    if (!removed) console.log(chalk.dim(`  · ${nameToRemove} was not installed in ${hookScopeArg} scope`))
+    process.exit(0)
+  }
+
+  console.error(chalk.red(`  ✗ Unknown hooks subcommand: "${hooksSubcmd}". Use list, install, or uninstall.`))
+  process.exit(1)
+}
+
 // ── Install ───────────────────────────────────────────────────────────────────
 // subcommand is 'install' or omitted (default behavior)
 const installArgs = subcommand === 'install' ? args.slice(1) : args
@@ -351,7 +602,161 @@ function fzfSelect() {
   })
 }
 
+// ── Print install summary (shared between interactive loop and non-interactive) ──
+function printSummary(skillSummary, toolSummary) {
+  if (skillSummary !== null) {
+    const anyInstalled = Object.values(skillSummary).some(r => r.installed.length > 0)
+    if (!anyInstalled) {
+      console.log(chalk.dim('  · No skills installed'))
+    } else {
+      console.log(chalk.green.bold('✔ Skills installed:'))
+      for (const [target, { installed }] of Object.entries(skillSummary)) {
+        if (installed.length > 0)
+          console.log(`  ${chalk.bold(target)} ← ${installed.join(', ')}`)
+      }
+    }
+    for (const [target, { skipped, failed }] of Object.entries(skillSummary)) {
+      for (const s of skipped) {
+        const detail = s.reason === 'up-to-date'
+          ? `up-to-date ${s.version}`
+          : `outdated ${s.installed} → ${s.available}, use --force`
+        console.log(chalk.dim(`  · ${target}/${s.name} skipped (${detail})`))
+      }
+      for (const f of failed) {
+        console.log(chalk.red(`  ✗ ${target}/${f.name} failed: ${f.reason}${f.detail ? ` — ${f.detail}` : ''}`))
+      }
+    }
+  }
+  if (toolSummary !== null) {
+    if (toolSummary.installed.length === 0 && !toolSummary.skipped.length && !toolSummary.failed.length) {
+      console.log(chalk.dim('  · No shell tools installed'))
+    } else {
+      if (toolSummary.installed.length > 0) {
+        console.log(chalk.green.bold('✔ Shell tools installed:'))
+        for (const name of toolSummary.installed) {
+          console.log(`  ${chalk.bold('~/.local/bin')} ← ${name}`)
+        }
+        console.log('')
+        console.log(chalk.yellow.bold('  ⚡ Reload your shell to apply changes:'))
+        console.log('')
+        console.log(`     ${chalk.bold.cyan('source ~/.zshrc')}`)
+        console.log('')
+      }
+      for (const s of toolSummary.skipped) {
+        console.log(chalk.dim(`  · ${s.name} skipped (${s.reason === 'already_exists' ? 'already exists — use --force to overwrite' : s.reason})`))
+      }
+      for (const f of toolSummary.failed) {
+        console.log(chalk.red(`  ✗ ${f.name} failed: ${f.reason}${f.detail ? ` — ${f.detail}` : ''}`))
+      }
+    }
+  }
+}
+
 try {
+  if (toolArg && skillArg) {
+    const msg = '--tool and --skill cannot be combined; use --bundle to install both'
+    if (jsonFlag) process.stderr.write(JSON.stringify({ error: true, message: msg }) + '\n')
+    else console.error(chalk.red('  ✗ ' + msg))
+    process.exit(1)
+  }
+
+  const isInteractive = !toolArg && !skillArg && !bundleArg
+
+  // ── Interactive mode: loop back to skill selector after each install ────────
+  if (isInteractive) {
+    if (!process.stdout.isTTY && !process.env.HSKILL_TEST_INTERACTIVE) {
+      console.error(chalk.red('  ✗ Interactive mode requires a TTY. Use --bundle, --skill, or --tool flags for non-interactive install.'))
+      console.error(chalk.dim('  Example: hskill install --bundle dev --target claude'))
+      process.exit(1)
+    }
+
+    while (true) {
+      const selected = fzfSelect()
+
+      if (!selected.length) {
+        console.log(chalk.dim('  · Nothing selected, exiting'))
+        break
+      }
+
+      const toolItems = selected.filter(s => s.kind === 'tool')
+      const seen = new Set()
+      const skillItems = selected.filter(s => s.kind === 'skill').filter(s => {
+        if (seen.has(s.skillName)) return false
+        seen.add(s.skillName); return true
+      })
+
+      if (!skillItems.length && !toolItems.length) continue
+
+      let skillSummary = null
+      if (skillItems.length > 0) {
+        // Scope selection
+        const scopeResult = spawnSync('fzf', [
+          '--prompt=  › ',
+          '--header=  Scope  ·  enter 确认  ·  esc 取消',
+          '--layout=reverse',
+          '--border=rounded',
+          '--color=header:italic:dim,prompt:cyan,pointer:cyan,hl:cyan,hl+:cyan:bold',
+        ], {
+          input: `user     — ~/.claude/skills/  (所有项目共享)\nproject  — .claude/skills/    (仅当前项目)`,
+          encoding: 'utf8',
+          stdio: ['pipe', 'pipe', 'inherit'],
+        })
+        if (!scopeResult.stdout.trim()) {
+          console.log(chalk.dim('  · Cancelled'))
+          break
+        }
+        const scope = scopeResult.stdout.trim().startsWith('project') ? 'project' : 'user'
+
+        // Target selection
+        const targetChoices = buildTargetChoices(scope)
+        const targetInput = targetChoices.map(c => c.name).join('\n') + '\nall      — all tools'
+        const targetResult = spawnSync('fzf', [
+          '--multi',
+          '--prompt=  › ',
+          '--header=  Install to  ·  tab 多选  ·  enter 确认  ·  esc 取消',
+          '--layout=reverse',
+          '--border=rounded',
+          '--color=header:italic:dim,prompt:cyan,pointer:cyan,hl:cyan,hl+:cyan:bold',
+        ], {
+          input: targetInput,
+          encoding: 'utf8',
+          stdio: ['pipe', 'pipe', 'inherit'],
+        })
+        if (!targetResult.stdout.trim()) {
+          console.log(chalk.dim('  · Cancelled'))
+          break
+        }
+        const selectedTargets = targetResult.stdout.trim().split('\n')
+          .map(l => l.trim().split(/\s+/)[0])
+
+        if (selectedTargets.length > 0) {
+          const targets = resolveTargets(selectedTargets, scope)
+          console.log('')
+          skillSummary = await installSkills(skillItems, targets, forceFlag)
+          console.log('')
+        }
+      }
+
+      let toolSummary = null
+      if (toolItems.length > 0) {
+        console.log('')
+        toolSummary = await installTools(
+          toolItems.map(t => ({ toolName: t.toolName, srcPath: t.srcPath })),
+          TARGETS.shell,
+          forceFlag,
+        )
+        console.log('')
+      }
+
+      printSummary(skillSummary, toolSummary)
+
+      // Loop back to skill selector automatically
+    }
+
+    process.exit(0)
+  }
+
+  // ── Non-interactive mode: resolve items from flags, install once ────────────
   let skillItems = []
   let toolItems  = []
 
@@ -367,21 +772,6 @@ try {
     const toolBundles  = bundles.filter(b =>  TOOL_BUNDLE_VALUES.has(b))
     if (skillBundles.length) skillItems = resolveSkills(skillBundles).map(s => ({ kind: 'skill', ...s }))
     if (toolBundles.length)  toolItems  = resolveTools(toolBundles).map(t => ({ kind: 'tool', ...t }))
-  } else {
-    const selected = fzfSelect()
-
-    if (!selected.length) {
-      console.log(chalk.dim('  · Nothing selected, exiting'))
-      process.exit(0)
-    }
-
-    toolItems = selected.filter(s => s.kind === 'tool')
-
-    const seen = new Set()
-    skillItems = selected.filter(s => s.kind === 'skill').filter(s => {
-      if (seen.has(s.skillName)) return false
-      seen.add(s.skillName); return true
-    })
   }
 
   if (!skillItems.length && !toolItems.length) {
@@ -390,10 +780,15 @@ try {
   }
 
   // ── Install skills ──────────────────────────────────────────────────────────
+  let skillSummary = null
   if (skillItems.length > 0) {
     // Resolve scope
     let scope = scopeArg ?? 'user'
     if (!scopeArg && !targetArg) {
+      if (!process.stdout.isTTY) {
+        console.error(chalk.red('  ✗ Interactive scope selection requires a TTY. Use --scope user|project.'))
+        process.exit(1)
+      }
       const scopeResult = spawnSync('fzf', [
         '--prompt=  › ',
         '--header=  Scope  ·  enter 确认  ·  esc 取消',
@@ -415,8 +810,12 @@ try {
     // Resolve target
     let selectedTargets
     if (targetArg) {
-      selectedTargets = targetArg === 'all' ? ['claude', 'cursor', 'codex'] : [targetArg]
+      selectedTargets = targetArg === 'all' ? ['claude', 'cursor', 'codex', 'openclaw', 'hermes'] : [targetArg]
     } else {
+      if (!process.stdout.isTTY) {
+        console.error(chalk.red('  ✗ Interactive target selection requires a TTY. Use --target claude|cursor|codex|openclaw|hermes|all.'))
+        process.exit(1)
+      }
       const targetChoices = buildTargetChoices(scope)
       const targetInput = targetChoices.map(c => c.name).join('\n') + '\nall      — all tools'
       const targetResult = spawnSync('fzf', [
@@ -442,43 +841,37 @@ try {
     if (selectedTargets.length > 0) {
       const targets = resolveTargets(selectedTargets, scope)
       console.log('')
-      const summary = await installSkills(skillItems, targets, forceFlag)
+      skillSummary = await installSkills(skillItems, targets, forceFlag)
       console.log('')
-      if (Object.keys(summary).length === 0) {
-        console.log(chalk.dim('  · No skills installed'))
-      } else {
-        console.log(chalk.green.bold('✔ Skills installed:'))
-        for (const [target, names] of Object.entries(summary)) {
-          console.log(`  ${chalk.bold(target)} ← ${names.join(', ')}`)
-        }
-      }
     }
   }
 
   // ── Install shell tools ─────────────────────────────────────────────────────
+  let toolSummary = null
   if (toolItems.length > 0) {
     console.log('')
-    const installed = await installTools(
+    toolSummary = await installTools(
       toolItems.map(t => ({ toolName: t.toolName, srcPath: t.srcPath })),
       TARGETS.shell,
       forceFlag,
     )
     console.log('')
-    if (installed.length === 0) {
-      console.log(chalk.dim('  · No shell tools installed'))
-    } else {
-      console.log(chalk.green.bold('✔ Shell tools installed:'))
-      for (const name of installed) {
-        console.log(`  ${chalk.bold('~/.local/bin')} ← ${name}`)
-      }
-      console.log('')
-      console.log(chalk.yellow.bold('  ⚡ Reload your shell to apply changes:'))
-      console.log('')
-      console.log(`     ${chalk.bold.cyan('source ~/.zshrc')}`)
-      console.log('')
-    }
+  }
+
+  // ── Output ──────────────────────────────────────────────────────────────────
+  if (jsonFlag) {
+    const out = {}
+    if (skillSummary !== null) out.skills = skillSummary
+    if (toolSummary  !== null) out.tools  = toolSummary
+    console.log(JSON.stringify(out, null, 2))
+  } else {
+    printSummary(skillSummary, toolSummary)
   }
 } catch (err) {
-  console.error(chalk.red('  ✗ ' + err.message))
+  if (jsonFlag) {
+    process.stderr.write(JSON.stringify({ error: true, message: err.message }) + '\n')
+  } else {
+    console.error(chalk.red('  ✗ ' + err.message))
+  }
   process.exit(1)
 }