agentskillsdk 0.1.3 → 0.1.5

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agentskillsdk",
-  "version": "0.1.3",
+  "version": "0.1.5",
   "description": "Install agent skills from agentskills.dk",
   "type": "module",
   "bin": {

package/src/commands/add.js

@@ -38,7 +38,7 @@ export async function addCommand(skillName, options) {
       (options.skill ? ` (skill: ${chalk.bold(options.skill)})` : ''));
   } else {
     installName = skillName;
-    const spinner = ora({ text: `Looking up ${chalk.bold(skillName)}...`, indent: 2 }).start();
+    const spinner = ora({ text: `Looking up ${chalk.bold(skillName)}...`, indent: 4 }).start();
     try {
       skill = await fetchSkill(skillName);
     } catch (err) {
@@ -85,14 +85,14 @@ export async function addCommand(skillName, options) {
     scope = 'project';
   } else {
     scope = await selectPrompt('Install scope:', [
-      { label: 'Project', hint: '(.claude/skills/ in current directory)', value: 'project' },
-      { label: 'Global', hint: '(~/.claude/skills/ for all projects)', value: 'global' },
+      { label: 'Project', hint: '(local .claude/skills/)', value: 'project' },
+      { label: 'Global', hint: '(~/.claude/skills/)', value: 'global' },
     ]);
   }
 
   // --- download ---
   const destDir = agent.path(installName, { cwd, scope });
-  const spinner = ora({ text: 'Downloading skill files...', indent: 2 }).start();
+  const spinner = ora({ text: 'Downloading skill files...', indent: 4 }).start();
   try {
     await downloadSkill(skill, destDir);
   } catch (err) {

package/src/lib/prompt.js

@@ -1,8 +1,15 @@
 import chalk from 'chalk';
-import { createInterface } from 'node:readline';
+
+// --- strip ANSI for width calculation ---
+
+const ANSI_RE = /\x1b\[[0-9;]*m/g;
+function stripAnsi(str) {
+  return str.replace(ANSI_RE, '');
+}
 
 /**
- * Enhanced select prompt with arrow indicator and hints.
+ * Interactive select prompt with arrow-key navigation.
+ * Up/Down to move, Enter to confirm.
  *
  * @param {string} question
  * @param {{ label: string, hint?: string, value: any }[]} choices
@@ -10,33 +17,96 @@ import { createInterface } from 'node:readline';
  * @returns {Promise<any>} selected value
  */
 export function selectPrompt(question, choices, { defaultIndex = 0 } = {}) {
-  const rl = createInterface({ input: process.stdin, output: process.stdout });
-
-  console.log(`\n  ${question}\n`);
-  choices.forEach((c, i) => {
-    const marker = i === defaultIndex ? chalk.cyan('>') : ' ';
-    const num = `${i + 1})`;
-    const label = i === defaultIndex ? chalk.cyan(`${num} ${c.label}`) : `${num} ${c.label}`;
-    const hint = c.hint ? chalk.dim(` ${c.hint}`) : '';
-    console.log(`    ${marker} ${label}${hint}`);
-  });
+  return new Promise((resolve) => {
+    let selected = defaultIndex;
+    const { stdin, stdout } = process;
+    const cols = stdout.columns || 80;
 
-  const defaultDisplay = defaultIndex + 1;
+    /** Count how many physical terminal lines a string occupies. */
+    function physicalLines(str) {
+      const w = stripAnsi(str).length;
+      return Math.max(1, Math.ceil(w / cols));
+    }
 
-  return new Promise((resolve) => {
-    rl.question(`\n  Choice ${chalk.dim(`[${defaultDisplay}]`)}: `, (answer) => {
-      rl.close();
-      const trimmed = answer.trim();
-      if (trimmed === '') {
-        resolve(choices[defaultIndex].value);
+    /** Total physical lines used by the rendered choices. */
+    function totalPhysicalLines(lines) {
+      return lines.reduce((sum, l) => sum + physicalLines(l), 0);
+    }
+
+    function render() {
+      return choices.map((c, i) => {
+        const marker = i === selected ? chalk.cyan('\u276f') : ' ';
+        const label = i === selected ? chalk.cyan(c.label) : c.label;
+        const hint = c.hint ? chalk.dim(` ${c.hint}`) : '';
+        return `      ${marker} ${label}${hint}`;
+      });
+    }
+
+    // Hide cursor
+    stdout.write('\x1b[?25l');
+
+    // Print question (4-space indent)
+    stdout.write(`\n    ${question}\n\n`);
+
+    // Initial render
+    let prevLines = render();
+    stdout.write(prevLines.join('\n'));
+
+    // Enter raw mode for keypress detection
+    const wasRaw = stdin.isRaw;
+    stdin.setRawMode(true);
+    stdin.resume();
+
+    function cleanup() {
+      stdin.setRawMode(wasRaw ?? false);
+      stdin.removeListener('data', onData);
+      stdin.pause();
+      // Restore cursor visibility
+      stdout.write('\x1b[?25h');
+    }
+
+    function onData(buf) {
+      const key = buf.toString();
+
+      // Ctrl+C
+      if (key === '\x03') {
+        cleanup();
+        stdout.write('\n');
+        process.exit(0);
+      }
+
+      // Enter
+      if (key === '\r' || key === '\n') {
+        // Move up by physical lines to clear previous render
+        const upCount = totalPhysicalLines(prevLines);
+        stdout.write(`\x1b[${upCount}A`);
+        stdout.write('\x1b[J'); // clear to end
+        const final = `      ${chalk.cyan('\u276f')} ${chalk.cyan(choices[selected].label)}`;
+        stdout.write(final + '\n');
+        cleanup();
+        resolve(choices[selected].value);
         return;
       }
-      const idx = parseInt(trimmed, 10) - 1;
-      if (idx >= 0 && idx < choices.length) {
-        resolve(choices[idx].value);
+
+      // Arrow keys come as escape sequences: \x1b[A (up), \x1b[B (down)
+      if (key === '\x1b[A' || key === '\x1b[D') {
+        // Up or Left
+        selected = (selected - 1 + choices.length) % choices.length;
+      } else if (key === '\x1b[B' || key === '\x1b[C') {
+        // Down or Right
+        selected = (selected + 1) % choices.length;
       } else {
-        resolve(choices[defaultIndex].value);
+        return; // ignore other keys
       }
-    });
+
+      // Redraw: move cursor up by physical lines, clear, rewrite
+      const upCount = totalPhysicalLines(prevLines);
+      stdout.write(`\x1b[${upCount}A`);
+      stdout.write('\x1b[J');
+      prevLines = render();
+      stdout.write(prevLines.join('\n'));
+    }
+
+    stdin.on('data', onData);
   });
 }

package/src/lib/ui.js

@@ -20,12 +20,12 @@ export function box(lines, { borderColor, padding = 2 } = {}) {
 
   const colorize = borderColor ? chalk[borderColor].bind(chalk) : (s) => s;
 
-  const top = colorize(`  \u250c${'─'.repeat(inner)}\u2510`);
-  const bottom = colorize(`  \u2514${'─'.repeat(inner)}\u2518`);
+  const top = colorize(`    \u250c${'─'.repeat(inner)}\u2510`);
+  const bottom = colorize(`    \u2514${'─'.repeat(inner)}\u2518`);
   const rowLines = lines.map(l => {
     const visible = stripAnsi(l).length;
     const rightPad = ' '.repeat(maxWidth - visible);
-    return `  ${colorize('\u2502')}${pad}${l}${rightPad}${pad}${colorize('\u2502')}`;
+    return `    ${colorize('\u2502')}${pad}${l}${rightPad}${pad}${colorize('\u2502')}`;
   });
 
   return [top, ...rowLines, bottom].join('\n');
@@ -72,9 +72,9 @@ export function printCompletionSummary({ skillName, scope, installPath, agentNam
 // --- step / error output ---
 
 export function step(text) {
-  console.log(chalk.green('  \u2713') + ` ${text}`);
+  console.log(chalk.green('    \u2713') + ` ${text}`);
 }
 
 export function error(text) {
-  console.error(chalk.red(`  ${text}`));
+  console.error(chalk.red(`    ${text}`));
 }