agentskillsdk 0.1.4 → 0.1.6

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agentskillsdk",
-  "version": "0.1.4",
+  "version": "0.1.6",
   "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,5 +1,12 @@
 import chalk from 'chalk';
 
+// --- strip ANSI for width calculation ---
+
+const ANSI_RE = /\x1b\[[0-9;]*m/g;
+function stripAnsi(str) {
+  return str.replace(ANSI_RE, '');
+}
+
 /**
  * Interactive select prompt with arrow-key navigation.
  * Up/Down to move, Enter to confirm.
@@ -13,57 +20,70 @@ export function selectPrompt(question, choices, { defaultIndex = 0 } = {}) {
   return new Promise((resolve) => {
     let selected = defaultIndex;
     const { stdin, stdout } = process;
+    const cols = stdout.columns || 80;
+
+    /** 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));
+    }
+
+    /** Total physical lines used by the rendered choices. */
+    function totalPhysicalLines(lines) {
+      return lines.reduce((sum, l) => sum + physicalLines(l), 0);
+    }
 
     function render() {
-      // Move cursor up to overwrite previous render (all choice lines + 1 blank line below question)
-      // On first render we just printed the question, so we only clear choice lines after that
-      const lines = choices.map((c, i) => {
+      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}`;
+        return `      ${marker} ${label}${hint}`;
       });
-      return lines.join('\n');
     }
 
-    // Print question
-    stdout.write(`\n  ${question}\n\n`);
+    // Hide cursor
+    stdout.write('\x1b[?25l');
+
+    // Print question (4-space indent)
+    stdout.write(`\n    ${question}\n\n`);
 
     // Initial render
-    stdout.write(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') {
-        stdin.setRawMode(wasRaw ?? false);
-        stdin.removeListener('data', onData);
-        stdin.pause();
+        cleanup();
         stdout.write('\n');
         process.exit(0);
       }
 
       // Enter
       if (key === '\r' || key === '\n') {
-        stdin.setRawMode(wasRaw ?? false);
-        stdin.removeListener('data', onData);
-        stdin.pause();
-        // Clear the choices and rewrite with final selection
-        stdout.write(`\x1b[${choices.length}A`); // move up
-        stdout.write('\x1b[J'); // clear to end
-        const final = choices.map((c, i) => {
-          if (i === selected) {
-            return `    ${chalk.cyan('\u276f')} ${chalk.cyan(c.label)}`;
-          }
-          return '';
-        }).filter(Boolean).join('\n');
+        // Move cursor to start of first choice line, then clear
+        const upCount = totalPhysicalLines(prevLines) - 1;
+        if (upCount > 0) stdout.write(`\x1b[${upCount}A`);
+        stdout.write('\r\x1b[J'); // col 0, clear to end
+        const final = `      ${chalk.cyan('\u276f')} ${chalk.cyan(choices[selected].label)}`;
         stdout.write(final + '\n');
+        cleanup();
         resolve(choices[selected].value);
         return;
       }
@@ -79,10 +99,12 @@ export function selectPrompt(question, choices, { defaultIndex = 0 } = {}) {
         return; // ignore other keys
       }
 
-      // Redraw: move cursor up N lines, clear, rewrite
-      stdout.write(`\x1b[${choices.length}A`);
-      stdout.write('\x1b[J');
-      stdout.write(render());
+      // Redraw: move cursor to start of first choice line, clear, rewrite
+      const upCount = totalPhysicalLines(prevLines) - 1;
+      if (upCount > 0) stdout.write(`\x1b[${upCount}A`);
+      stdout.write('\r\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}`));
 }