skild 0.4.2 → 0.4.3

package/dist/index.js

@@ -128,16 +128,6 @@ async function promptPassword(question) {
     stdin.on("keypress", onKeypress);
   });
 }
-async function promptConfirm(question, options = {}) {
-  const defaultValue = options.defaultValue ?? false;
-  const suffix = defaultValue ? " (Y/n)" : " (y/N)";
-  const answer = await promptLine(`${question}${suffix}`);
-  const v = answer.trim().toLowerCase();
-  if (!v) return defaultValue;
-  if (v === "y" || v === "yes") return true;
-  if (v === "n" || v === "no") return false;
-  return defaultValue;
-}
 
 // src/commands/install-discovery.ts
 import fs from "fs";
@@ -233,10 +223,6 @@ function printJson(value) {
   process.stdout.write(`${JSON.stringify(value, null, 2)}
 `);
 }
-function previewDiscovered(found, limit = 12) {
-  const preview = found.slice(0, limit).map((s) => `  - ${s.relPath}`).join("\n");
-  return `${preview}${found.length > limit ? "\n  ..." : ""}`;
-}
 function asDiscoveredSkills(discovered, toSuggestedSource, toMaterializedDir) {
   return discovered.map((d) => ({
     relPath: d.relPath,
@@ -244,24 +230,258 @@ function asDiscoveredSkills(discovered, toSuggestedSource, toMaterializedDir) {
     materializedDir: toMaterializedDir ? toMaterializedDir(d) : void 0
   }));
 }
+function buildSkillTree(found) {
+  const root = { id: "", name: ".", children: [], leafIndices: [] };
+  const byId = /* @__PURE__ */ new Map([["", root]]);
+  for (let i = 0; i < found.length; i += 1) {
+    const relPath = found[i].relPath;
+    const parts = relPath === "." ? ["."] : relPath.split("/").filter(Boolean);
+    let currentId = "";
+    let current = root;
+    current.leafIndices.push(i);
+    for (const part of parts) {
+      const nextId = currentId ? `${currentId}/${part}` : part;
+      let node = byId.get(nextId);
+      if (!node) {
+        node = { id: nextId, name: part, children: [], leafIndices: [] };
+        byId.set(nextId, node);
+        current.children.push(node);
+      }
+      node.leafIndices.push(i);
+      current = node;
+      currentId = nextId;
+    }
+  }
+  return root;
+}
+function getNodeState(node, selected) {
+  const total = node.leafIndices.length;
+  if (total === 0) return "none";
+  let selectedCount = 0;
+  for (const idx of node.leafIndices) {
+    if (selected.has(idx)) selectedCount += 1;
+  }
+  if (selectedCount === 0) return "none";
+  if (selectedCount === total) return "all";
+  return "partial";
+}
+function renderSkillTree(root, selected) {
+  const items = [];
+  const stack = [];
+  for (let i = root.children.length - 1; i >= 0; i -= 1) {
+    stack.push({ node: root.children[i], depth: 0 });
+  }
+  while (stack.length) {
+    const current = stack.pop();
+    items.push(current);
+    const children = current.node.children;
+    for (let i = children.length - 1; i >= 0; i -= 1) {
+      stack.push({ node: children[i], depth: current.depth + 1 });
+    }
+  }
+  return items;
+}
+function parseSelectionInput(input, maxIndex) {
+  const trimmed = input.trim().toLowerCase();
+  if (!trimmed) return { command: "done" };
+  if (["q", "quit", "exit", "cancel"].includes(trimmed)) return { command: "cancel" };
+  if (["a", "all"].includes(trimmed)) return { command: "all" };
+  if (["n", "none"].includes(trimmed)) return { command: "none" };
+  if (["i", "invert"].includes(trimmed)) return { command: "invert" };
+  const tokens = trimmed.split(/[,\s]+/).filter(Boolean);
+  const indices = [];
+  for (const token of tokens) {
+    if (/^\d+$/.test(token)) {
+      const value = Number(token);
+      if (value < 1 || value > maxIndex) return null;
+      indices.push(value);
+      continue;
+    }
+    if (/^\d+-\d+$/.test(token)) {
+      const [startRaw, endRaw] = token.split("-");
+      const start = Number(startRaw);
+      const end = Number(endRaw);
+      if (!Number.isFinite(start) || !Number.isFinite(end) || start < 1 || end < 1 || start > maxIndex || end > maxIndex) {
+        return null;
+      }
+      const from = Math.min(start, end);
+      const to = Math.max(start, end);
+      for (let i = from; i <= to; i += 1) indices.push(i);
+      continue;
+    }
+    return null;
+  }
+  return { indices };
+}
+async function promptSkillSelection(found, options) {
+  const root = buildSkillTree(found);
+  const selected = /* @__PURE__ */ new Set();
+  if (options.defaultAll) {
+    for (let i = 0; i < found.length; i += 1) selected.add(i);
+  }
+  while (true) {
+    console.log(chalk2.cyan("\nSelect skills to install"));
+    console.log(chalk2.dim("Toggle by number. Commands: a=all, n=none, i=invert, enter=continue, q=cancel."));
+    const display = renderSkillTree(root, selected);
+    for (let i = 0; i < display.length; i += 1) {
+      const item = display[i];
+      const state = getNodeState(item.node, selected);
+      const box = state === "all" ? "[x]" : state === "partial" ? "[-]" : "[ ]";
+      const indent = "  ".repeat(item.depth);
+      const idx = String(i + 1).padStart(2, " ");
+      console.log(`${idx} ${box} ${indent}${item.node.name}`);
+    }
+    const answer = await promptLine("Selection");
+    const parsed = parseSelectionInput(answer, display.length);
+    if (!parsed) {
+      console.log(chalk2.yellow('Invalid selection. Use numbers like "1,3-5" or a/n/i.'));
+      continue;
+    }
+    if (parsed.command === "done") break;
+    if (parsed.command === "cancel") return null;
+    if (parsed.command === "all") {
+      selected.clear();
+      for (let i = 0; i < found.length; i += 1) selected.add(i);
+      continue;
+    }
+    if (parsed.command === "none") {
+      selected.clear();
+      continue;
+    }
+    if (parsed.command === "invert") {
+      const next = /* @__PURE__ */ new Set();
+      for (let i = 0; i < found.length; i += 1) {
+        if (!selected.has(i)) next.add(i);
+      }
+      selected.clear();
+      for (const idx of next) selected.add(idx);
+      continue;
+    }
+    if (parsed.indices) {
+      for (const index of parsed.indices) {
+        const item = display[index - 1];
+        if (!item) continue;
+        const state = getNodeState(item.node, selected);
+        if (state === "all") {
+          for (const leaf of item.node.leafIndices) selected.delete(leaf);
+        } else {
+          for (const leaf of item.node.leafIndices) selected.add(leaf);
+        }
+      }
+    }
+  }
+  if (selected.size === 0) return null;
+  const selectedFound = [];
+  for (let i = 0; i < found.length; i += 1) {
+    if (selected.has(i)) selectedFound.push(found[i]);
+  }
+  return selectedFound;
+}
+async function promptPlatformSelection(defaultAll) {
+  const selected = /* @__PURE__ */ new Set();
+  if (defaultAll) {
+    for (let i = 0; i < PLATFORMS.length; i += 1) selected.add(i);
+  }
+  while (true) {
+    console.log(chalk2.cyan("\nSelect platforms to install to"));
+    console.log(chalk2.dim("Toggle by number. Commands: a=all, n=none, i=invert, enter=continue, q=cancel."));
+    for (let i = 0; i < PLATFORMS.length; i += 1) {
+      const label = PLATFORMS[i];
+      const box = selected.has(i) ? "[x]" : "[ ]";
+      const idx = String(i + 1).padStart(2, " ");
+      console.log(`${idx} ${box} ${label}`);
+    }
+    const answer = await promptLine("Selection");
+    const parsed = parseSelectionInput(answer, PLATFORMS.length);
+    if (!parsed) {
+      console.log(chalk2.yellow('Invalid selection. Use numbers like "1,3-5" or a/n/i.'));
+      continue;
+    }
+    if (parsed.command === "done") break;
+    if (parsed.command === "cancel") return null;
+    if (parsed.command === "all") {
+      selected.clear();
+      for (let i = 0; i < PLATFORMS.length; i += 1) selected.add(i);
+      continue;
+    }
+    if (parsed.command === "none") {
+      selected.clear();
+      continue;
+    }
+    if (parsed.command === "invert") {
+      const next = /* @__PURE__ */ new Set();
+      for (let i = 0; i < PLATFORMS.length; i += 1) {
+        if (!selected.has(i)) next.add(i);
+      }
+      selected.clear();
+      for (const idx of next) selected.add(idx);
+      continue;
+    }
+    if (parsed.indices) {
+      for (const index of parsed.indices) {
+        const idx = index - 1;
+        if (idx < 0 || idx >= PLATFORMS.length) continue;
+        if (selected.has(idx)) selected.delete(idx);
+        else selected.add(idx);
+      }
+    }
+  }
+  if (selected.size === 0) return null;
+  const chosen = [];
+  for (let i = 0; i < PLATFORMS.length; i += 1) {
+    if (selected.has(i)) chosen.push(PLATFORMS[i]);
+  }
+  return chosen;
+}
+function printSelectedSkills(found) {
+  const names = found.map((skill) => skill.relPath);
+  if (names.length === 0) return;
+  const preview = names.slice(0, 20).map((name) => `  - ${name}`).join("\n");
+  console.log(chalk2.dim(`
+Selected skills (${names.length}):`));
+  console.log(preview);
+  if (names.length > 20) {
+    console.log(chalk2.dim(`  ... and ${names.length - 20} more`));
+  }
+}
 async function install(source, options = {}) {
   const scope = options.local ? "project" : "global";
   const auth = loadRegistryAuth();
   const registryUrlForDeps = options.registry || auth?.registryUrl;
-  const all = Boolean(options.all);
   const jsonOnly = Boolean(options.json);
   const recursive = Boolean(options.recursive);
   const yes = Boolean(options.yes);
   const maxDepth = parsePositiveInt(options.depth, 6);
   const maxSkills = parsePositiveInt(options.maxSkills, 200);
-  const platform = options.target || "claude";
-  if (all && options.target) {
+  const interactive = Boolean(process.stdin.isTTY && process.stdout.isTTY) && !jsonOnly;
+  const requestedAll = Boolean(options.all);
+  const requestedTarget = options.target;
+  if (requestedAll && options.target) {
     const message = "Invalid options: use either --all or --target, not both.";
     if (jsonOnly) printJson({ ok: false, error: message });
     else console.error(chalk2.red(message));
     process.exitCode = 1;
     return;
   }
+  let targets = [];
+  if (requestedAll) {
+    targets = [...PLATFORMS];
+  } else if (requestedTarget) {
+    targets = [requestedTarget];
+  } else if (yes) {
+    targets = [...PLATFORMS];
+  } else if (interactive) {
+    const selectedTargets = await promptPlatformSelection(true);
+    if (!selectedTargets) {
+      console.error(chalk2.red("No platforms selected."));
+      process.exitCode = 1;
+      return;
+    }
+    targets = selectedTargets;
+  } else {
+    targets = ["claude"];
+  }
+  const allPlatformsSelected = targets.length === PLATFORMS.length;
   let resolvedSource = source.trim();
   try {
     if (looksLikeAlias(resolvedSource)) {
@@ -277,12 +497,10 @@ async function install(source, options = {}) {
     process.exitCode = 1;
     return;
   }
-  const targets = all ? [...PLATFORMS] : [platform];
   const results = [];
   const errors = [];
   let effectiveRecursive = recursive;
   let recursiveSkillCount = null;
-  const interactive = Boolean(process.stdin.isTTY && process.stdout.isTTY) && !jsonOnly;
   async function installOne(inputSource, materializedDir) {
     for (const targetPlatform of targets) {
       try {
@@ -301,14 +519,14 @@ async function install(source, options = {}) {
       }
     }
   }
-  async function maybeEnableRecursiveAndInstall(found, spinner) {
-    if (found.length === 0) return false;
+  async function resolveDiscoveredSelection(found, spinner) {
+    if (found.length === 0) return null;
     if (found.length > maxSkills) {
       const message = `Found more than ${maxSkills} skills. Increase --max-skills to proceed.`;
       if (jsonOnly) printJson({ ok: false, error: "TOO_MANY_SKILLS", message, source, resolvedSource, maxSkills });
       else console.error(chalk2.red(message));
       process.exitCode = 1;
-      return true;
+      return null;
     }
     if (!effectiveRecursive) {
       if (jsonOnly) {
@@ -322,31 +540,39 @@ async function install(source, options = {}) {
           found: foundOutput
         });
         process.exitCode = 1;
-        return true;
+        return null;
       }
       if (spinner) spinner.stop();
-      const headline = found.length === 1 ? `No SKILL.md found at root. Found 1 skill:
-${previewDiscovered(found)}
-` : `No SKILL.md found at root. Found ${found.length} skills:
-${previewDiscovered(found)}
+      const headline = found.length === 1 ? `No SKILL.md found at root. Found 1 skill.
+` : `No SKILL.md found at root. Found ${found.length} skills.
 `;
       console.log(chalk2.yellow(headline));
-      const question = found.length === 1 ? "Install the discovered skill?" : "Install all discovered skills?";
-      const confirm = yes || interactive && await promptConfirm(question, { defaultValue: found.length === 1 });
-      if (!confirm) {
+      if (!interactive && !yes) {
         console.log(chalk2.dim(`Tip: rerun with ${chalk2.cyan("skild install <source> --recursive")} to install all.`));
         process.exitCode = 1;
-        return true;
+        return null;
+      }
+      if (!yes) {
+        const selected = await promptSkillSelection(found, { defaultAll: true });
+        if (!selected) {
+          console.log(chalk2.red("No skills selected."));
+          process.exitCode = 1;
+          return null;
+        }
+        printSelectedSkills(selected);
+        effectiveRecursive = true;
+        if (spinner) spinner.start();
+        recursiveSkillCount = selected.length;
+        return selected;
       }
       effectiveRecursive = true;
-      if (spinner) spinner.start();
     }
     recursiveSkillCount = found.length;
-    return false;
+    return found;
   }
   try {
     const spinner = jsonOnly ? null : createSpinner(
-      all ? `Installing ${chalk2.cyan(source)} to ${chalk2.dim("all platforms")} (${scope})...` : `Installing ${chalk2.cyan(source)} to ${chalk2.dim(platform)} (${scope})...`
+      allPlatformsSelected ? `Installing ${chalk2.cyan(source)} to ${chalk2.dim("all platforms")} (${scope})...` : targets.length > 1 ? `Installing ${chalk2.cyan(source)} to ${chalk2.dim(`${targets.length} platforms`)} (${scope})...` : `Installing ${chalk2.cyan(source)} to ${chalk2.dim(targets[0])} (${scope})...`
     );
     let cleanupMaterialized = null;
     let materializedRoot = null;
@@ -374,10 +600,10 @@ ${previewDiscovered(found)}
               return;
             }
             const found = asDiscoveredSkills(discovered, (d) => path2.join(maybeLocalRoot, d.relPath));
-            const didReturn = await maybeEnableRecursiveAndInstall(found, spinner);
-            if (didReturn) return;
-            if (spinner) spinner.text = `Installing ${chalk2.cyan(source)} \u2014 discovered ${found.length} skills...`;
-            for (const skill of found) {
+            const selected = await resolveDiscoveredSelection(found, spinner);
+            if (!selected) return;
+            if (spinner) spinner.text = `Installing ${chalk2.cyan(source)} \u2014 discovered ${selected.length} skills...`;
+            for (const skill of selected) {
               if (spinner) spinner.text = `Installing ${chalk2.cyan(skill.relPath)} (${scope})...`;
               await installOne(skill.suggestedSource, skill.materializedDir);
             }
@@ -407,10 +633,10 @@ ${previewDiscovered(found)}
               (d) => deriveChildSource(resolvedSource, d.relPath),
               (d) => d.absDir
             );
-            const didReturn = await maybeEnableRecursiveAndInstall(found, spinner);
-            if (didReturn) return;
-            if (spinner) spinner.text = `Installing ${chalk2.cyan(source)} \u2014 discovered ${found.length} skills...`;
-            for (const skill of found) {
+            const selected = await resolveDiscoveredSelection(found, spinner);
+            if (!selected) return;
+            if (spinner) spinner.text = `Installing ${chalk2.cyan(source)} \u2014 discovered ${selected.length} skills...`;
+            for (const skill of selected) {
               if (spinner) spinner.text = `Installing ${chalk2.cyan(skill.relPath)} (${scope})...`;
               await installOne(skill.suggestedSource, skill.materializedDir);
             }
@@ -421,11 +647,21 @@ ${previewDiscovered(found)}
       if (cleanupMaterialized) cleanupMaterialized();
     }
     if (jsonOnly) {
-      if (!all && !effectiveRecursive) {
+      if (!effectiveRecursive && targets.length === 1) {
         if (errors.length) printJson({ ok: false, error: errors[0]?.error || "Install failed." });
         else printJson(results[0] ?? null);
       } else {
-        printJson({ ok: errors.length === 0, source, resolvedSource, scope, recursive: effectiveRecursive, all, recursiveSkillCount, results, errors });
+        printJson({
+          ok: errors.length === 0,
+          source,
+          resolvedSource,
+          scope,
+          recursive: effectiveRecursive,
+          all: allPlatformsSelected,
+          recursiveSkillCount,
+          results,
+          errors
+        });
       }
       process.exitCode = errors.length ? 1 : 0;
       return;
@@ -433,7 +669,7 @@ ${previewDiscovered(found)}
     if (errors.length === 0) {
       const displayName = results[0]?.canonicalName || results[0]?.name || source;
       spinner.succeed(
-        effectiveRecursive ? `Installed ${chalk2.green(String(recursiveSkillCount ?? results.length))}${chalk2.dim(" skills")} to ${chalk2.dim(`${targets.length} platforms`)}` : all ? `Installed ${chalk2.green(displayName)} to ${chalk2.dim(`${results.length} platforms`)}` : `Installed ${chalk2.green(displayName)} to ${chalk2.dim(results[0]?.installDir || "")}`
+        effectiveRecursive ? `Installed ${chalk2.green(String(recursiveSkillCount ?? results.length))}${chalk2.dim(" skills")} to ${chalk2.dim(`${targets.length} platforms`)}` : targets.length > 1 ? `Installed ${chalk2.green(displayName)} to ${chalk2.dim(`${results.length} platforms`)}` : `Installed ${chalk2.green(displayName)} to ${chalk2.dim(results[0]?.installDir || "")}`
       );
     } else {
       const attempted = results.length + errors.length;
@@ -441,9 +677,9 @@ ${previewDiscovered(found)}
         effectiveRecursive ? `Install had failures (${errors.length}/${attempted} installs failed)` : `Failed to install ${chalk2.red(source)} to ${errors.length}/${targets.length} platforms`
       );
       process.exitCode = 1;
-      if (!all && errors[0]) console.error(chalk2.red(errors[0].error));
+      if (!effectiveRecursive && targets.length === 1 && errors[0]) console.error(chalk2.red(errors[0].error));
     }
-    if (!effectiveRecursive && !all && results[0]) {
+    if (!effectiveRecursive && targets.length === 1 && results[0]) {
       const record = results[0];
       if (record.hasSkillMd) logger.installDetail("SKILL.md found \u2713");
       else logger.installDetail("Warning: No SKILL.md found", true);
@@ -452,7 +688,7 @@ ${previewDiscovered(found)}
       } else if (record.skill?.validation?.ok) {
         logger.installDetail(`Validation: ${chalk2.green("ok")}`);
       }
-    } else if (effectiveRecursive || all) {
+    } else if (effectiveRecursive || targets.length > 1) {
       for (const r of results.slice(0, 60)) {
         const displayName = r.canonicalName || r.name;
         const suffix = r.hasSkillMd ? chalk2.green("\u2713") : chalk2.yellow("\u26A0");

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "skild",
-  "version": "0.4.2",
+  "version": "0.4.3",
   "description": "The npm for Agent Skills — Discover, install, manage, and publish AI Agent Skills with ease.",
   "type": "module",
   "main": "./dist/index.js",
@@ -37,7 +37,7 @@
     "commander": "^12.1.0",
     "ora": "^8.0.1",
     "tar": "^7.4.3",
-    "@skild/core": "^0.4.2"
+    "@skild/core": "^0.4.3"
   },
   "devDependencies": {
     "@types/node": "^20.10.0",