install-agent-skill 1.2.8 → 1.3.0

package/bin/add-skill.v2.js

@@ -1,4 +1,4 @@
-#!/usr/bin/env node
+#!/usr/bin/env node
 /**
  * install-agent-skill
  * Vercel-Style CLI - Full Port v2.0
@@ -70,11 +70,11 @@ const c = {
 
 // --- UI Helpers ---
 function step(text, icon = S.diamond, color = "gray") {
-    console.log(` ${c.gray(S.branch)}  ${c[color](icon)}  ${text}`);
+    console.log(`${c.gray(S.branch)}  ${c[color](icon)}  ${text}`);
 }
-function stepLine() { console.log(` ${c.gray(S.branch)}`); }
-function fatal(msg) { console.log(` ${c.gray(S.branch)}  ${c.red(S.cross)}  ${c.red(msg)}`); process.exit(1); }
-function success(msg) { console.log(` ${c.gray(S.branch)}  ${c.green(S.check)}  ${c.green(msg)}`); }
+function stepLine() { console.log(`${c.gray(S.branch)}`); }
+function fatal(msg) { console.log(`${c.gray(S.branch)}  ${c.red(S.cross)}  ${c.red(msg)}`); process.exit(1); }
+function success(msg) { console.log(`${c.gray(S.branch)}  ${c.green(S.check)}  ${c.green(msg)}`); }
 function outputJSON(data) { if (JSON_OUTPUT) console.log(JSON.stringify(data, null, 2)); }
 
 // --- Core Helpers ---
@@ -231,18 +231,18 @@ async function runInit() {
 async function runList() {
     stepLine();
     step(c.bold("Installed Skills"), S.diamondFilled, "cyan");
-    console.log(` ${c.gray(S.branch)}  ${c.dim("Location: " + resolveScope())}`);
+    console.log(`${c.gray(S.branch)}  ${c.dim("Location: " + resolveScope())}`);
     stepLine();
     const skills = getInstalledSkills();
     if (skills.length === 0) { step(c.dim("No skills installed."), S.diamond); stepLine(); return; }
     if (JSON_OUTPUT) { outputJSON({ skills }); return; }
     for (const s of skills) {
         const icon = s.hasSkillMd ? c.green(S.check) : c.yellow(S.diamond);
-        console.log(` ${c.gray(S.branch)}  ${icon}  ${c.bold(s.name)} ${c.dim("v" + s.version)} ${c.dim("(" + formatBytes(s.size) + ")")}`);
-        if (s.description && VERBOSE) console.log(` ${c.gray(S.branch)}      ${c.dim(s.description.substring(0, 60))}`);
+        console.log(`${c.gray(S.branch)}  ${icon}  ${c.bold(s.name)} ${c.dim("v" + s.version)} ${c.dim("(" + formatBytes(s.size) + ")")}`);
+        if (s.description && VERBOSE) console.log(`${c.gray(S.branch)}      ${c.dim(s.description.substring(0, 60))}`);
     }
     stepLine();
-    console.log(` ${c.gray(S.branch)}  ${c.dim("Total: " + skills.length + " skill(s)")}`);
+    console.log(`${c.gray(S.branch)}  ${c.dim("Total: " + skills.length + " skill(s)")}`);
     stepLine();
 }
 
@@ -361,7 +361,7 @@ async function runInstall(spec) {
     for (const sn of selectedSkills) {
         const src = path.join(tmp, sn), dest = path.join(targetScope, sn);
         if (fs.existsSync(dest)) fs.rmSync(dest, { recursive: true, force: true });
-        fs.cpSync(src, dest, { recursive: true });
+        await fs.promises.cp(src, dest, { recursive: true });
         const hash = merkleHash(dest);
         fs.writeFileSync(path.join(dest, ".skill-source.json"), JSON.stringify({ repo: `${org}/${repo}`, skill: sn, ref: ref || null, checksum: hash, installedAt: new Date().toISOString() }, null, 2));
     }
@@ -441,7 +441,7 @@ function runVerify() {
         if (actual !== m.checksum) { step(`${name}: ${c.red("checksum mismatch")}`, S.cross, "red"); issues++; }
         else step(`${name}: ${c.green("OK")}`, S.check, "green");
     }
-    stepLine(); console.log(` ${c.gray(S.branch)}  ${issues ? c.red(issues + " issue(s)") : c.green("All verified")}`); stepLine();
+    stepLine(); console.log(`${c.gray(S.branch)}  ${issues ? c.red(issues + " issue(s)") : c.green("All verified")}`); stepLine();
     if (issues && STRICT) process.exit(1);
 }
 
@@ -464,7 +464,7 @@ function runDoctor() {
         } else if (lock && !lock.skills[name]) { step(`${name}: ${c.yellow("not in lock")}`, S.diamond, "yellow"); STRICT ? errors++ : warnings++; }
         else step(`${name}: ${c.green("healthy")}`, S.check, "green");
     }
-    stepLine(); console.log(` ${c.gray(S.branch)}  Errors: ${errors}, Warnings: ${warnings}`); stepLine();
+    stepLine(); console.log(`${c.gray(S.branch)}  Errors: ${errors}, Warnings: ${warnings}`); stepLine();
     if (STRICT && errors) process.exit(1);
 }
 
@@ -481,17 +481,17 @@ function runCache(sub) {
         const rs = fs.existsSync(REGISTRY_CACHE) ? getDirSize(REGISTRY_CACHE) : 0;
         const bs = fs.existsSync(BACKUP_DIR) ? getDirSize(BACKUP_DIR) : 0;
         step(c.bold("Cache Info"), S.diamondFilled, "cyan");
-        console.log(` ${c.gray(S.branch)}  Location: ${CACHE_ROOT}`);
-        console.log(` ${c.gray(S.branch)}  Registries: ${formatBytes(rs)}`);
-        console.log(` ${c.gray(S.branch)}  Backups: ${formatBytes(bs)}`);
-        console.log(` ${c.gray(S.branch)}  Total: ${formatBytes(getDirSize(CACHE_ROOT))}`);
+        console.log(`${c.gray(S.branch)}  Location: ${CACHE_ROOT}`);
+        console.log(`${c.gray(S.branch)}  Registries: ${formatBytes(rs)}`);
+        console.log(`${c.gray(S.branch)}  Backups: ${formatBytes(bs)}`);
+        console.log(`${c.gray(S.branch)}  Total: ${formatBytes(getDirSize(CACHE_ROOT))}`);
         stepLine(); return;
     }
     if (sub === "backups") {
         const backups = listBackups();
         if (backups.length === 0) { step("No backups found", S.diamond); return; }
         step(c.bold("Backups"), S.diamondFilled, "cyan"); stepLine();
-        backups.forEach(b => console.log(` ${c.gray(S.branch)}  ${b.name} (${formatBytes(b.size)})`));
+        backups.forEach(b => console.log(`${c.gray(S.branch)}  ${b.name} (${formatBytes(b.size)})`));
         stepLine(); return;
     }
     fatal(`Unknown cache subcommand: ${sub}`);
@@ -511,11 +511,11 @@ function runValidate(skillName) {
         if (!fs.existsSync(smp)) errors.push("Missing SKILL.md");
         else { const m = parseSkillMdFrontmatter(smp); if (!m.description) errors.push("Missing description"); else if (m.description.length < 50) warnings.push("Description too short"); }
         const status = errors.length > 0 ? c.red("FAIL") : warnings.length > 0 ? c.yellow("WARN") : c.green("PASS");
-        console.log(` ${c.gray(S.branch)}  ${status}  ${c.bold(skill.name)}`);
-        if (VERBOSE || errors.length || warnings.length) { errors.forEach(e => console.log(` ${c.gray(S.branch)}      ${c.red("ERROR: " + e)}`)); warnings.forEach(w => console.log(` ${c.gray(S.branch)}      ${c.yellow("WARN: " + w)}`)); }
+        console.log(`${c.gray(S.branch)}  ${status}  ${c.bold(skill.name)}`);
+        if (VERBOSE || errors.length || warnings.length) { errors.forEach(e => console.log(`${c.gray(S.branch)}      ${c.red("ERROR: " + e)}`)); warnings.forEach(w => console.log(`${c.gray(S.branch)}      ${c.yellow("WARN: " + w)}`)); }
         totalErrors += errors.length; totalWarnings += warnings.length;
     }
-    stepLine(); console.log(` ${c.gray(S.branch)}  Total: ${skillsToValidate.length}, Errors: ${totalErrors}, Warnings: ${totalWarnings}`); stepLine();
+    stepLine(); console.log(`${c.gray(S.branch)}  Total: ${skillsToValidate.length}, Errors: ${totalErrors}, Warnings: ${totalWarnings}`); stepLine();
     if (STRICT && totalErrors > 0) process.exit(1);
 }
 
@@ -524,20 +524,20 @@ function runAnalyze(skillName) {
     const scope = resolveScope(), skillDir = path.join(scope, skillName);
     if (!fs.existsSync(skillDir)) fatal(`Skill not found: ${skillName}`);
     stepLine(); step(c.bold(`Skill Analysis: ${skillName}`), S.diamondFilled, "cyan");
-    console.log(` ${c.gray(S.branch)}  ${c.dim("Path: " + skillDir)}`); stepLine();
+    console.log(`${c.gray(S.branch)}  ${c.dim("Path: " + skillDir)}`); stepLine();
     const smp = path.join(skillDir, "SKILL.md");
     if (fs.existsSync(smp)) {
         const m = parseSkillMdFrontmatter(smp);
-        console.log(` ${c.gray(S.branch)}  ${c.cyan("SKILL.md Frontmatter:")}`);
-        console.log(` ${c.gray(S.branch)}    Name: ${m.name || c.dim("(not set)")}`);
-        console.log(` ${c.gray(S.branch)}    Description: ${m.description ? m.description.substring(0, 60) : c.red("(MISSING)")}`);
-        if (m.tags) console.log(` ${c.gray(S.branch)}    Tags: ${m.tags.join(", ")}`);
+        console.log(`${c.gray(S.branch)}  ${c.cyan("SKILL.md Frontmatter:")}`);
+        console.log(`${c.gray(S.branch)}    Name: ${m.name || c.dim("(not set)")}`);
+        console.log(`${c.gray(S.branch)}    Description: ${m.description ? m.description.substring(0, 60) : c.red("(MISSING)")}`);
+        if (m.tags) console.log(`${c.gray(S.branch)}    Tags: ${m.tags.join(", ")}`);
         stepLine();
     }
     const structure = detectSkillStructure(skillDir);
-    console.log(` ${c.gray(S.branch)}  ${c.cyan("Structure:")}`);
+    console.log(`${c.gray(S.branch)}  ${c.cyan("Structure:")}`);
     const items = [["resources", structure.hasResources], ["examples", structure.hasExamples], ["scripts", structure.hasScripts], ["constitution", structure.hasConstitution], ["doctrines", structure.hasDoctrines]];
-    items.forEach(([n, has]) => console.log(` ${c.gray(S.branch)}    ${has ? c.green(S.check) : c.dim("○")} ${has ? c.bold(n) : c.dim(n)}`));
+    items.forEach(([n, has]) => console.log(`${c.gray(S.branch)}    ${has ? c.green(S.check) : c.dim("○")} ${has ? c.bold(n) : c.dim(n)}`));
     stepLine();
     let score = 0;
     if (fs.existsSync(smp)) score += 20;
@@ -548,7 +548,7 @@ function runAnalyze(skillName) {
     if (fs.existsSync(path.join(skillDir, ".skill-source.json"))) score += 10;
     if (structure.hasConstitution || structure.hasDoctrines) score += 15;
     const scoreColor = score >= 80 ? c.green : score >= 50 ? c.yellow : c.red;
-    console.log(` ${c.gray(S.branch)}  ${c.cyan("Antigravity Score:")} ${scoreColor(score + "/100")}`);
+    console.log(`${c.gray(S.branch)}  ${c.cyan("Antigravity Score:")} ${scoreColor(score + "/100")}`);
     stepLine();
 }
 
@@ -558,9 +558,9 @@ function runInfo(name) {
     stepLine();
     if (fs.existsSync(localDir)) {
         step(`${c.bold(name)} ${c.green("(installed)")}`, S.diamondFilled, "cyan");
-        console.log(` ${c.gray(S.branch)}  ${c.dim("Path: " + localDir)}`);
+        console.log(`${c.gray(S.branch)}  ${c.dim("Path: " + localDir)}`);
         const mf = path.join(localDir, ".skill-source.json");
-        if (fs.existsSync(mf)) { const m = JSON.parse(fs.readFileSync(mf, "utf-8")); console.log(` ${c.gray(S.branch)}  Repo: ${m.repo || "local"}`); console.log(` ${c.gray(S.branch)}  Installed: ${formatDate(m.installedAt)}`); }
+        if (fs.existsSync(mf)) { const m = JSON.parse(fs.readFileSync(mf, "utf-8")); console.log(`${c.gray(S.branch)}  Repo: ${m.repo || "local"}`); console.log(`${c.gray(S.branch)}  Installed: ${formatDate(m.installedAt)}`); }
         stepLine(); return;
     }
     step(`Skill not installed: ${name}`, S.diamond, "yellow"); stepLine();

package/bin/lib/commands/install.js

@@ -47,8 +47,7 @@ export async function run(spec) {
         await execAsync(`git clone --depth=1 ${url} "${tmp}"`);
         if (ref) await execAsync(`git -C "${tmp}" checkout ${ref}`);
     } catch {
-        s.stop("Failed to clone");
-        step(c.red("Failed to clone"), S.cross, "red");
+        s.fail("Failed to clone");
         fs.rmSync(tmp, { recursive: true, force: true });
         return;
     }

package/bin/lib/commands/update.js

@@ -41,15 +41,15 @@ export async function run(skillName) {
         const spec = `${meta.repo}#${meta.skill}${meta.ref ? "@" + meta.ref : ""}`;
 
         if (DRY) {
-            s.stop();
+            s.stop("Dry run analysis complete");
             step(`Would update: ${skillName}`);
         } else {
+            s.stop("Preparing update...");
             // Dynamically import install command
             const { run: install } = await import("./install.js");
             await install(spec);
-            s.stop();
         }
     } catch (err) {
-        s.stop(`Failed: ${err.message}`);
+        s.fail(`Failed: ${err.message}`);
     }
 }

package/bin/lib/ui.js

@@ -3,9 +3,46 @@
  */
 
 import kleur from "kleur";
-import { intro, outro, spinner, multiselect, select, confirm, isCancel, cancel, text } from "@clack/prompts";
+import { intro, outro, multiselect, select, confirm, isCancel, cancel, text } from "@clack/prompts";
+import ora from "ora";
 
-export { intro, outro, spinner, multiselect, select, confirm, isCancel, cancel, text };
+export { intro, outro, multiselect, select, confirm, isCancel, cancel, text };
+
+/**
+ * Create a spinner
+ */
+export function spinner() {
+    return {
+        _s: null,
+        start(msg) {
+            this._s = ora({
+                text: msg,
+                prefixText: ` ${c.gray(S.branch)}  `,
+                color: "magenta",
+                spinner: "dots"
+            }).start();
+        },
+        stop(msg) {
+            if (this._s) {
+                this._s.stopAndPersist({
+                    symbol: c.green(S.check),
+                    text: msg
+                });
+            }
+        },
+        fail(msg) {
+            if (this._s) {
+                this._s.stopAndPersist({
+                    symbol: c.red(S.cross),
+                    text: msg
+                });
+            }
+        },
+        message(msg) {
+            if (this._s) this._s.text = msg;
+        }
+    };
+}
 
 // --- Symbols ---
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "install-agent-skill",
-  "version": "1.2.8",
+  "version": "1.3.0",
   "description": "Enterprise-grade Agent Skill Manager with Antigravity Skills support, Progressive Disclosure detection, and semantic routing validation",
   "license": "MIT",
   "author": "DataGuruIn <contact@dataguruin.com>",