mdpush 1.2.0 → 1.3.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "mdpush",
-  "version": "1.2.0",
+  "version": "1.3.1",
   "type": "module",
   "engines": {
     "node": ">=20.0.0"

package/src/commands/skill-command.js

@@ -86,9 +86,37 @@ export async function skillPushCommand(dir, options) {
   if (failed) process.exit(1)
 }
 
-/** Install a published skill into ~/.claude/skills/<slug> (or --dest). */
-export async function skillInstallCommand(slug, options) {
-  const manifestRes = await apiRequest(`/api/skills/${slug}/manifest`)
+/**
+ * Install published skills into ~/.claude/skills/<slug> (or --dest).
+ * Accepts several slugs (space or comma separated); each installs
+ * independently — one failure doesn't abort the rest (exit 1 at the end).
+ */
+export async function skillInstallCommand(slugs, options) {
+  const list = [...new Set(slugs.flatMap(s => s.split(',')).map(s => s.trim()).filter(Boolean))]
+  if (list.length > 1 && options.dest) {
+    console.log(chalk.red('--dest is ambiguous with multiple skills — install them one at a time.'))
+    process.exit(2)
+  }
+
+  const failedSlugs = []
+  for (const slug of list) {
+    try {
+      await installOneSkill(slug, options)
+    } catch (err) {
+      console.log(chalk.red(`✖ ${slug}: ${err.message}`))
+      failedSlugs.push(slug)
+    }
+  }
+  if (list.length > 1) {
+    console.log(`\n${chalk.green(`${list.length - failedSlugs.length} skill(s) installed`)}${failedSlugs.length ? `, ${chalk.red(`${failedSlugs.length} failed (${failedSlugs.join(', ')})`)}` : ''}`)
+  }
+  if (failedSlugs.length) process.exit(1)
+}
+
+/** Install one skill. `?source=cli-install` marks a real install so the
+ * server bumps install_count (detail-page manifest fetches don't). */
+async function installOneSkill(slug, options) {
+  const manifestRes = await apiRequest(`/api/skills/${slug}/manifest?source=cli-install`)
   const manifest = await manifestRes.json()
 
   const dest = path.resolve(options.dest || path.join(os.homedir(), '.claude', 'skills', slug))
@@ -115,15 +143,63 @@ export async function skillInstallCommand(slug, options) {
     }
   }
   console.log(`\n${chalk.green(`${success} installed`)}${failed ? `, ${chalk.red(`${failed} failed`)}` : ''}`)
-  if (failed) process.exit(1)
+  // Throw (not exit) so a batch install can continue with the next slug.
+  if (failed) throw new Error(`${failed} file(s) failed`)
 }
 
-/** List published skills. */
-export async function skillListCommand() {
+/** Fetch + print the catalog table. Returns the skills array (or []). */
+async function printCatalog({ numbered = false } = {}) {
   const res = await apiRequest('/api/skills')
   const skills = await res.json()
-  if (skills.length === 0) return console.log(chalk.yellow('No skills published.'))
-  for (const s of skills) {
-    console.log(`${chalk.cyan(s.slug.padEnd(24))} ${s.description || chalk.gray('(no description)')}`)
+  if (skills.length === 0) {
+    console.log(chalk.yellow('No skills published.'))
+    return []
+  }
+  skills.forEach((s, i) => {
+    const idx = numbered ? chalk.gray(`${String(i + 1).padStart(3)}. `) : ''
+    const cat = (s.category || '—').padEnd(10)
+    const installs = String(s.installCount ?? 0).padStart(4)
+    const desc = (s.description || '(no description)').slice(0, 80)
+    console.log(`${idx}${chalk.cyan(s.slug.padEnd(24))} ${chalk.magenta(cat)} ${chalk.gray(`⤓${installs}`)}  ${desc}`)
+  })
+  return skills
+}
+
+/** List published skills (slug · category · installs · description). */
+export async function skillListCommand() {
+  await printCatalog()
+}
+
+/**
+ * Interactive browse (`mdpush skill` with no subcommand) — numbered catalog,
+ * pick by number/comma list, installs the selection. Zero-dependency prompt
+ * via node:readline; falls back to plain list when stdin isn't a TTY.
+ */
+export async function skillBrowseCommand() {
+  const skills = await printCatalog({ numbered: true })
+  if (skills.length === 0 || !process.stdin.isTTY) return
+
+  const readline = await import('node:readline/promises')
+  const rl = readline.createInterface({ input: process.stdin, output: process.stdout })
+  // Race against 'close' — an EOF on stdin (Ctrl-D, piped input running dry)
+  // would otherwise leave the question promise pending forever.
+  const answer = (await Promise.race([
+    rl.question(chalk.blue('\nInstall which? (numbers, comma-separated — empty to exit): ')),
+    new Promise(resolve => rl.once('close', () => resolve('')))
+  ])).trim()
+  rl.close()
+  // A TTY stdin touched by readline keeps the event loop alive after close —
+  // exit explicitly on every path (install failures already exit(1) upstream).
+  if (!answer) process.exit(0)
+
+  const picks = [...new Set(answer.split(/[\s,]+/))]
+    .map(n => skills[parseInt(n, 10) - 1])
+    .filter(Boolean)
+    .map(s => s.slug)
+  if (picks.length === 0) {
+    console.log(chalk.yellow('No valid selection.'))
+    process.exit(2)
   }
+  await skillInstallCommand(picks, {})
+  process.exit(0)
 }

package/src/index.js

@@ -1,14 +1,18 @@
+import { createRequire } from 'module'
 import { program } from 'commander'
+
+// Single-source the CLI version from package.json (was hardcoded and stale)
+const pkg = createRequire(import.meta.url)('../package.json')
 import { loginCommand } from './commands/login-command.js'
 import { pushCommand } from './commands/push-command.js'
 import { configCommand } from './commands/config-command.js'
 import { logoutCommand } from './commands/logout-command.js'
-import { skillPushCommand, skillInstallCommand, skillListCommand } from './commands/skill-command.js'
+import { skillPushCommand, skillInstallCommand, skillListCommand, skillBrowseCommand } from './commands/skill-command.js'
 
 program
   .name('mdpush')
   .description('Push markdown, text, and HTML artifact files to sielay')
-  .version('1.1.0')
+  .version(pkg.version)
 
 program.command('login')
   .description('Authenticate with server')
@@ -25,6 +29,7 @@ program.command('push')
 
 const skill = program.command('skill')
   .description('Publish and install Claude Code skill bundles (projects with kind=skill)')
+  .action(skillBrowseCommand) // bare `mdpush skill` → interactive browse
 
 skill.command('push')
   .description('Push a skill directory (must contain SKILL.md) to the server')
@@ -33,9 +38,9 @@ skill.command('push')
   .action(skillPushCommand)
 
 skill.command('install')
-  .description('Install a published skill into ~/.claude/skills/<slug>')
-  .argument('<slug>', 'Skill slug (see: mdpush skill list)')
-  .option('-d, --dest <path>', 'Destination directory (default: ~/.claude/skills/<slug>)')
+  .description('Install published skills into ~/.claude/skills/<slug>')
+  .argument('<slugs...>', 'Skill slug(s) — space or comma separated (see: mdpush skill list)')
+  .option('-d, --dest <path>', 'Destination directory (single skill only)')
   .action(skillInstallCommand)
 
 skill.command('list')