skillbox 0.3.3 → 0.3.4

package/dist/cli.js

@@ -12,7 +12,7 @@ import { registerRemove } from "./commands/remove.js";
 import { registerStatus } from "./commands/status.js";
 import { registerUpdate } from "./commands/update.js";
 const program = new Command();
-program.name("skillbox").description("Local-first, agent-agnostic skills manager").version("0.3.3");
+program.name("skillbox").description("Local-first, agent-agnostic skills manager").version("0.3.4");
 registerAdd(program);
 registerAgent(program);
 registerConfig(program);

package/dist/commands/add-repo.js

@@ -4,7 +4,7 @@ import { loadConfig } from "../lib/config.js";
 import { parseRepoRef } from "../lib/github.js";
 import { loadIndex, saveIndex, sortIndex, upsertSkill } from "../lib/index.js";
 import { recordInstallPaths } from "../lib/installs.js";
-import { printInfo, printJson } from "../lib/output.js";
+import { printFailure, printInfo, printJson, printSkipped, printSuccess, startSpinner, stopSpinner, } from "../lib/output.js";
 import { buildProjectAgentPaths } from "../lib/project-paths.js";
 import { fetchRepoFile, listRepoSkills, normalizeRepoRef, writeRepoSkillDirectory, } from "../lib/repo-skills.js";
 import { ensureProjectRegistered, resolveRuntime } from "../lib/runtime.js";
@@ -84,16 +84,27 @@ export async function handleRepoInstall(input, options) {
     }
     const summary = { installed: [], updated: [], skipped: [], failed: [] };
     const index = await loadIndex();
-    for (const skill of skills) {
-        if (!selected.includes(skill.name)) {
-            continue;
-        }
+    const showProgress = !options.json;
+    const selectedSkills = skills.filter((s) => selected.includes(s.name));
+    const total = selectedSkills.length;
+    if (showProgress && total > 0) {
+        printInfo(`Adding ${total} skill${total === 1 ? "" : "s"} from ${ref.owner}/${ref.repo}...\n`);
+    }
+    for (let i = 0; i < selectedSkills.length; i++) {
+        const skill = selectedSkills[i];
+        const progress = `(${i + 1}/${total})`;
         const alreadyInstalled = index.skills.some((entry) => entry.name === skill.name);
+        if (showProgress) {
+            startSpinner(`${skill.name} ${progress}`);
+        }
         try {
             const skillMarkdown = await fetchRepoFile(ref, ref.path ? `${ref.path}/${skill.skillFile}` : skill.skillFile);
             const parsed = parseSkillMarkdown(skillMarkdown);
             if (!parsed.description) {
                 summary.skipped.push(skill.name);
+                if (showProgress) {
+                    printSkipped(skill.name, "missing description");
+                }
                 continue;
             }
             await writeRepoSkillDirectory(ref, skill.path, skill.name);
@@ -124,56 +135,49 @@ export async function handleRepoInstall(input, options) {
             index.skills = nextIndex.skills;
             if (alreadyInstalled) {
                 summary.updated.push(skill.name);
+                if (showProgress) {
+                    printSuccess(skill.name, "updated");
+                }
             }
             else {
                 summary.installed.push(skill.name);
+                if (showProgress) {
+                    printSuccess(skill.name);
+                }
             }
         }
         catch (error) {
             const message = getErrorMessage(error, "unknown");
             summary.failed.push({ name: skill.name, reason: message });
+            if (showProgress) {
+                printFailure(skill.name, message);
+            }
         }
     }
+    if (showProgress) {
+        stopSpinner();
+    }
     await saveIndex(sortIndex(index));
     if (options.json) {
         printJson({ ok: true, command: "add", data: { repo: `${ref.owner}/${ref.repo}`, ...summary } });
         return;
     }
-    printInfo(`Skills Added from: ${ref.owner}/${ref.repo}`);
-    printInfo("");
-    printInfo("Source: git");
-    printInfo(`  ${ref.owner}/${ref.repo}${ref.path ? `/${ref.path}` : ""} (${ref.ref})`);
-    if (summary.installed.length > 0) {
-        printInfo("");
-        printInfo(`Installed (${summary.installed.length}):`);
-        for (const name of summary.installed) {
-            printInfo(`  ✓ ${name}`);
-        }
+    // Summary line
+    const added = summary.installed.length + summary.updated.length;
+    const failed = summary.failed.length;
+    const skipped = summary.skipped.length;
+    if (added > 0 && failed === 0 && skipped === 0) {
+        printInfo(`\nAdded ${added} skill${added === 1 ? "" : "s"} from ${ref.owner}/${ref.repo}.`);
     }
-    if (summary.updated.length > 0) {
-        printInfo("");
-        printInfo(`Updated (${summary.updated.length}):`);
-        for (const name of summary.updated) {
-            printInfo(`  ✓ ${name}`);
-        }
-    }
-    if (summary.skipped.length > 0) {
-        printInfo("");
-        printInfo(`Skipped (${summary.skipped.length}):`);
-        for (const name of summary.skipped) {
-            printInfo(`  - ${name} (missing description)`);
-        }
+    else if (added > 0) {
+        const parts = [];
+        if (failed > 0)
+            parts.push(`${failed} failed`);
+        if (skipped > 0)
+            parts.push(`${skipped} skipped`);
+        printInfo(`\nAdded ${added} skill${added === 1 ? "" : "s"} (${parts.join(", ")}).`);
     }
-    if (summary.failed.length > 0) {
-        printInfo("");
-        printInfo(`Failed (${summary.failed.length}):`);
-        for (const failure of summary.failed) {
-            printInfo(`  ✗ ${failure.name} (${failure.reason})`);
-        }
-    }
-    const total = summary.installed.length + summary.updated.length;
-    if (total === 0) {
-        printInfo("");
-        printInfo("No skills were added.");
+    else {
+        printInfo("\nNo skills were added.");
     }
 }

package/dist/commands/add.js

@@ -5,7 +5,7 @@ import { fetchText } from "../lib/fetcher.js";
 import { collect } from "../lib/fs-utils.js";
 import { loadIndex, saveIndex, sortIndex, upsertSkill } from "../lib/index.js";
 import { recordInstallPaths } from "../lib/installs.js";
-import { isJsonEnabled, printInfo, printJson } from "../lib/output.js";
+import { isJsonEnabled, printInfo, printJson, printSuccess, startSpinner, stopSpinner, } from "../lib/output.js";
 import { buildProjectAgentPaths } from "../lib/project-paths.js";
 import { ensureProjectRegistered, resolveRuntime } from "../lib/runtime.js";
 import { buildMetadata, inferNameFromUrl, parseSkillMarkdown } from "../lib/skill-parser.js";
@@ -34,17 +34,28 @@ export function registerAdd(program) {
                 });
                 return;
             }
+            const showProgress = !isJsonEnabled(options);
+            const inferred = inferNameFromUrl(url);
+            const displayName = options.name ?? inferred ?? "skill";
+            if (showProgress) {
+                startSpinner(`Adding ${displayName}`);
+            }
             const skillMarkdown = await fetchText(url);
             const parsed = parseSkillMarkdown(skillMarkdown);
-            const inferred = inferNameFromUrl(url);
             const skillName = options.name ?? inferred ?? parsed.name;
             if (!skillName) {
+                if (showProgress)
+                    stopSpinner();
                 throw new Error("Unable to infer skill name. Use --name to specify it.");
             }
             if (!parsed.name && !options.name) {
+                if (showProgress)
+                    stopSpinner();
                 throw new Error("Skill frontmatter missing name. Provide --name to continue.");
             }
             if (!parsed.description) {
+                if (showProgress)
+                    stopSpinner();
                 throw new Error("Skill frontmatter missing description. Convert the source into a valid skill.");
             }
             const metadata = buildMetadata(parsed, { type: "url", url }, skillName);
@@ -102,6 +113,7 @@ export function registerAdd(program) {
             });
             await saveIndex(sortIndex(nextIndex));
             if (isJsonEnabled(options)) {
+                stopSpinner();
                 printJson({
                     ok: true,
                     command: "add",
@@ -114,22 +126,8 @@ export function registerAdd(program) {
                 });
                 return;
             }
-            printInfo(`Skill Added: ${skillName}`);
-            printInfo("");
-            printInfo("Source: url");
-            printInfo(`  ${url}`);
-            if (installs.length > 0) {
-                printInfo("");
-                printInfo("Installed to:");
-                for (const install of installs) {
-                    const scopeLabel = install.scope === "project" ? `project:${install.projectRoot}` : "user";
-                    printInfo(`  ✓ ${scopeLabel}/${install.agent}`);
-                }
-            }
-            else {
-                printInfo("");
-                printInfo("No agent targets were updated.");
-            }
+            printSuccess(skillName);
+            printInfo(`\nAdded skill from ${url}.`);
         }
         catch (error) {
             handleCommandError(options, "add", error);

package/dist/commands/list.js

@@ -172,6 +172,14 @@ function filterByAgents(skills, agents) {
     const agentSet = new Set(agents);
     return skills.filter((skill) => skill.installs?.some((install) => install.agent && agentSet.has(install.agent)));
 }
+function filterUserScope(skills) {
+    return skills
+        .filter((skill) => skill.installs?.some((install) => install.scope === "user") ?? !skill.installs?.length)
+        .map((skill) => ({
+        ...skill,
+        installs: skill.installs?.filter((install) => install.scope === "user"),
+    }));
+}
 export function registerList(program) {
     program
         .command("list")
@@ -186,8 +194,9 @@ export function registerList(program) {
         const indexedSkills = options.agents
             ? filterByAgents(index.skills, runtime.agentList)
             : index.skills;
-        const allSkills = [...indexedSkills, ...globalSkills];
-        const enrichedSkills = await enrichWithSubcommands(allSkills);
+        const scopedSkills = options.global ? filterUserScope(indexedSkills) : indexedSkills;
+        const mergedSkills = [...scopedSkills, ...globalSkills];
+        const enrichedSkills = await enrichWithSubcommands(mergedSkills);
         if (isJsonEnabled(options)) {
             printJson({
                 ok: true,

package/dist/commands/update.js

@@ -4,7 +4,7 @@ import { loadConfig } from "../lib/config.js";
 import { fetchText } from "../lib/fetcher.js";
 import { loadIndex, saveIndex, sortIndex, upsertSkill } from "../lib/index.js";
 import { getInstallPaths } from "../lib/installs.js";
-import { isJsonEnabled, printInfo, printJson } from "../lib/output.js";
+import { isJsonEnabled, printFailure, printInfo, printJson, printSkipped, printSuccess, startSpinner, stopSpinner, } from "../lib/output.js";
 import { fetchRepoFile, normalizeRepoRef, writeRepoSkillDirectory } from "../lib/repo-skills.js";
 import { buildMetadata, parseSkillMarkdown } from "../lib/skill-parser.js";
 import { ensureSkillsDir, writeSkillFiles, writeSkillMetadata } from "../lib/skill-store.js";
@@ -21,31 +21,6 @@ function groupBySource(results) {
         failedCount: items.filter((r) => r.status === "failed").length,
     }));
 }
-function formatSourceHeader(group) {
-    const count = group.results.length;
-    const skillWord = count === 1 ? "skill" : "skills";
-    if (group.source === "local") {
-        return `${group.source} (${count} ${skillWord} - skipped)`;
-    }
-    if (group.failedCount > 0) {
-        return `${group.source} (${count} ${skillWord}, ${group.failedCount} failed)`;
-    }
-    return `${group.source} (${count} ${skillWord})`;
-}
-function printSourceGroup(group) {
-    printInfo(formatSourceHeader(group));
-    for (const result of group.results) {
-        if (result.status === "skipped") {
-            printInfo(`  - ${result.name}`);
-        }
-        else if (result.status === "failed") {
-            printInfo(`  ✗ ${result.name} (${result.error ?? "failed"})`);
-        }
-        else {
-            printInfo(`  ✓ ${result.name}`);
-        }
-    }
-}
 async function updateUrlSkill(skill, index, projectRoot, config) {
     if (!skill.source.url) {
         return;
@@ -129,39 +104,72 @@ export function registerUpdate(program) {
             const config = await loadConfig();
             const projectRoot = options.project ? path.resolve(options.project) : null;
             const results = [];
-            for (const skill of targets) {
+            const showProgress = !isJsonEnabled(options);
+            if (showProgress && targets.length > 0) {
+                printInfo(`Updating ${targets.length} skill${targets.length === 1 ? "" : "s"}...\n`);
+            }
+            const total = targets.length;
+            for (let i = 0; i < targets.length; i++) {
+                const skill = targets[i];
+                const progress = `(${i + 1}/${total})`;
                 if (skill.source.type === "url") {
+                    if (showProgress) {
+                        startSpinner(`${skill.name} ${progress}`);
+                    }
                     try {
                         await updateUrlSkill(skill, index, projectRoot, config);
                         results.push({ name: skill.name, source: "url", status: "updated" });
+                        if (showProgress) {
+                            printSuccess(skill.name);
+                        }
                     }
                     catch (err) {
+                        const errorMsg = err instanceof Error ? err.message : "unknown error";
                         results.push({
                             name: skill.name,
                             source: "url",
                             status: "failed",
-                            error: err instanceof Error ? err.message : "unknown error",
+                            error: errorMsg,
                         });
+                        if (showProgress) {
+                            printFailure(skill.name, errorMsg);
+                        }
                     }
                 }
                 else if (skill.source.type === "git") {
+                    if (showProgress) {
+                        startSpinner(`${skill.name} ${progress}`);
+                    }
                     try {
                         await updateGitSkill(skill, index, projectRoot, config);
                         results.push({ name: skill.name, source: "git", status: "updated" });
+                        if (showProgress) {
+                            printSuccess(skill.name);
+                        }
                     }
                     catch (err) {
+                        const errorMsg = err instanceof Error ? err.message : "unknown error";
                         results.push({
                             name: skill.name,
                             source: "git",
                             status: "failed",
-                            error: err instanceof Error ? err.message : "unknown error",
+                            error: errorMsg,
                         });
+                        if (showProgress) {
+                            printFailure(skill.name, errorMsg);
+                        }
                     }
                 }
                 else {
                     results.push({ name: skill.name, source: skill.source.type, status: "skipped" });
+                    if (showProgress) {
+                        printSkipped(skill.name, "skipped");
+                    }
                 }
             }
+            if (showProgress) {
+                stopSpinner();
+            }
             await saveIndex(sortIndex(index));
             const sourceGroups = groupBySource(results);
             const totalUpdated = results.filter((r) => r.status === "updated").length;
@@ -189,21 +197,15 @@ export function registerUpdate(program) {
                 printInfo("No skills to update.");
                 return;
             }
-            printInfo("Skill Update");
-            for (const group of sourceGroups) {
-                printInfo("");
-                printSourceGroup(group);
-            }
             // Summary line
-            printInfo("");
             if (totalFailed > 0) {
-                printInfo(`Updated ${totalUpdated} of ${totalTrackable} trackable skills (${totalFailed} failed).`);
+                printInfo(`\nUpdated ${totalUpdated} of ${totalTrackable} trackable skill${totalTrackable === 1 ? "" : "s"} (${totalFailed} failed).`);
             }
             else if (totalUpdated > 0) {
-                printInfo(`Updated ${totalUpdated} of ${totalTrackable} trackable skills.`);
+                printInfo(`\nUpdated ${totalUpdated} of ${totalTrackable} trackable skill${totalTrackable === 1 ? "" : "s"}.`);
             }
             else if (totalSkipped > 0 && totalTrackable === 0) {
-                printInfo("No trackable skills to update.");
+                printInfo("\nNo trackable skills to update.");
             }
         }
         catch (error) {

package/dist/lib/output.js

@@ -2,6 +2,48 @@ import chalk from "chalk";
 export function isJsonEnabled(options) {
     return Boolean(options.json);
 }
+// Progress indicator support
+const isTTY = process.stdout.isTTY ?? false;
+// Braille spinner frames (single character, smooth animation)
+const SPINNER_FRAMES = ["⠋", "⠙", "⠹", "⠸", "⠼", "⠴", "⠦", "⠧", "⠇", "⠏"];
+let spinnerInterval = null;
+let spinnerFrame = 0;
+export function startSpinner(message) {
+    if (!isTTY)
+        return;
+    spinnerFrame = 0;
+    const render = () => {
+        const frame = SPINNER_FRAMES[spinnerFrame % SPINNER_FRAMES.length];
+        process.stdout.write(`\r\x1b[K  ${frame} ${message}`);
+        spinnerFrame++;
+    };
+    render();
+    spinnerInterval = setInterval(render, 80);
+}
+export function stopSpinner() {
+    if (spinnerInterval) {
+        clearInterval(spinnerInterval);
+        spinnerInterval = null;
+    }
+    if (isTTY) {
+        process.stdout.write(`\r\x1b[K`);
+    }
+}
+export function printProgressResult(message) {
+    stopSpinner();
+    process.stdout.write(`${message}\n`);
+}
+// Common result formatters for progress output
+export function printSuccess(name, suffix) {
+    const msg = suffix ? `  ✓ ${name} (${suffix})` : `  ✓ ${name}`;
+    printProgressResult(msg);
+}
+export function printFailure(name, error) {
+    printProgressResult(`  ✗ ${name} (${error})`);
+}
+export function printSkipped(name, reason) {
+    printProgressResult(`  - ${name} (${reason})`);
+}
 export function printJson(result) {
     process.stdout.write(`${JSON.stringify(result, null, 2)}\n`);
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "skillbox",
-  "version": "0.3.3",
+  "version": "0.3.4",
   "description": "Local-first, agent-agnostic skills manager",
   "license": "MIT",
   "author": "Christian Anagnostou",
@@ -33,6 +33,9 @@
     "lint:ci": "oxlint --deny-warnings",
     "format": "oxfmt",
     "format:check": "oxfmt --check",
+    "test": "vitest run",
+    "test:watch": "vitest",
+    "test:ci": "vitest run --reporter=verbose --reporter=junit --outputFile=test-results.xml",
     "prepublishOnly": "npm run lint:ci && npm run format:check && npm run build"
   },
   "dependencies": {
@@ -43,9 +46,11 @@
   },
   "devDependencies": {
     "@types/node": "^20.11.19",
+    "execa": "^9.6.1",
     "oxfmt": "^0.16.0",
     "oxlint": "^0.16.0",
     "ts-node": "^10.9.2",
-    "typescript": "^5.4.2"
+    "typescript": "^5.4.2",
+    "vitest": "^4.0.18"
   }
 }