skillio 0.1.15 → 0.1.16

package/dist/cli.js

@@ -861,9 +861,8 @@ var listCommand = defineCommand({
         render: () => {
           if (rows.lock.totalCount === 0)
             return "";
-          if (orphans.length === 0)
-            return green("All skills onboard!");
-          return orphans.map((n) => red(n.name)).join(" ");
+          const orphanSet = new Set(orphans.map((n) => n.name));
+          return rows.lock.names.map((n) => orphanSet.has(n.name) ? red(n.name) : n.name).join(" ");
         }
       }
     ];
@@ -900,7 +899,6 @@ var listCommand = defineCommand({
 });
 
 // src/commands/remove.ts
-import { existsSync as existsSync3, lstatSync as lstatSync3 } from "node:fs";
 import { homedir as homedir2 } from "node:os";
 import { dirname as dirname2, join as join3, resolve as resolve3 } from "node:path";
 
@@ -988,17 +986,24 @@ function createConfirmer(opts = {}) {
 }
 
 // src/utils/fs-rm.ts
-import { existsSync as existsSync2, lstatSync as lstatSync2, readdirSync, rmSync } from "node:fs";
+import { lstatSync as lstatSync2, readdirSync, rmSync } from "node:fs";
 import { join as join2, resolve as resolve2 } from "node:path";
 function isInside(target, root) {
   const t = resolve2(target);
   const r = resolve2(root);
   return t === r || t.startsWith(`${r}/`);
 }
+function lstatOrNull(path) {
+  try {
+    return lstatSync2(path);
+  } catch {
+    return null;
+  }
+}
 function countFiles(path) {
-  if (!existsSync2(path))
+  const stat = lstatOrNull(path);
+  if (!stat)
     return 0;
-  const stat = lstatSync2(path);
   if (stat.isFile())
     return 1;
   if (!stat.isDirectory())
@@ -1029,7 +1034,7 @@ function rmSkillDir(path, opts) {
   if (!safe) {
     throw new Error(`Refusing to delete: "${path}" is outside allowed roots`);
   }
-  if (!existsSync2(path))
+  if (!lstatOrNull(path))
     return { removed: false, fileCount: 0 };
   const fileCount = countFiles(path);
   rmSync(path, { recursive: true, force: true });
@@ -1038,12 +1043,13 @@ function rmSkillDir(path, opts) {
 
 // src/commands/remove.ts
 function buildLocation(dir) {
-  if (!existsSync3(dir))
-    return { kind: "missing", folders: 0, files: 0 };
-  if (lstatSync3(dir).isSymbolicLink())
-    return { kind: "symlink", folders: 0, files: 0 };
+  const stat = lstatOrNull(dir);
+  if (!stat)
+    return { kind: "missing", subfolders: 0, files: 0 };
+  if (stat.isSymbolicLink())
+    return { kind: "symlink", subfolders: 0, files: 0 };
   const { folders, files } = countFoldersAndFiles(dir);
-  return { kind: "real", folders, files };
+  return { kind: "real", subfolders: folders, files };
 }
 function rootFor2(base, isGlobal, lockPath) {
   return isGlobal ? join3(homedir2(), base, "skills") : join3(dirname2(resolve3(lockPath)), base, "skills");
@@ -1072,53 +1078,63 @@ function header(targets, verb) {
 }
 function aggregateLocations(infos) {
   let folders = 0;
+  let subfolders = 0;
   let files = 0;
   let symlinks = 0;
   for (const info of infos) {
     if (info.kind === "real") {
-      folders += info.folders;
+      folders += 1;
+      subfolders += info.subfolders;
       files += info.files;
     } else if (info.kind === "symlink") {
       symlinks += 1;
     }
   }
-  return { folders, files, symlinks, found: infos.some((i) => i.kind !== "missing") };
+  return { folders, subfolders, files, symlinks, found: infos.some((i) => i.kind !== "missing") };
 }
-function diskLine(label, singleName, infos, variant) {
-  const displayLabel = singleName ? `${label}/${singleName}/` : label;
+function diskCell(infos, variant) {
   const agg = aggregateLocations(infos);
   if (!agg.found)
-    return `  ${displayLabel}  (not found)`;
+    return "not found";
   const parts = [];
   if (agg.folders > 0 || agg.files > 0) {
-    const text = `${plural(agg.folders, "folder")}, ${plural(agg.files, "file")}`;
+    const text = `${plural(agg.folders, "folder")}, ${plural(agg.subfolders, "subfolder")}, ${plural(agg.files, "file")}`;
     parts.push(variant === "summary" ? red(text) : green(text));
   }
   if (agg.symlinks > 0) {
     const text = plural(agg.symlinks, "symlink");
     parts.push(variant === "summary" ? red(text) : yellow(text));
   }
-  return `  ${displayLabel}  (${parts.join(", ")})`;
+  return parts.join(", ");
 }
-function lockLine(n, variant) {
-  const label = "skills-lock.json";
+function lockCell(n, variant) {
   if (n === 0)
-    return `  ${label}  (not in lock)`;
+    return "not in lock";
   if (variant === "kept")
-    return `  ${green(`${label}  (${plural(n, "line")} kept)`)}`;
-  return `  ${red(`${label}  (${plural(n, "line")})`)}`;
+    return green(`${plural(n, "line")} kept`);
+  return red(plural(n, "line"));
 }
 function printBlock(targets, scope, verb, lockLinesToRemove, diskVariant, lockVariant) {
   console.log(header(targets, verb));
-  const singleName = targets.length === 1 ? targets[0]?.name : undefined;
+  const rows = [];
   if (scope === "all" || scope === "agents-only") {
-    console.log(diskLine(".agents/skills", singleName, targets.map((t) => t.agents), diskVariant));
+    rows.push([
+      ".agents/skills/",
+      diskCell(targets.map((t) => t.agents), diskVariant)
+    ]);
   }
   if (scope === "all" || scope === "claude-only") {
-    console.log(diskLine(".claude/skills", singleName, targets.map((t) => t.claude), diskVariant));
+    rows.push([
+      ".claude/skills/",
+      diskCell(targets.map((t) => t.claude), diskVariant)
+    ]);
   }
   if (scope === "all" || scope === "lock-only") {
-    console.log(lockLine(lockLinesToRemove, lockVariant));
+    rows.push(["skills-lock.json", lockCell(lockLinesToRemove, lockVariant)]);
+  }
+  const labelWidth = Math.max(...rows.map(([label]) => label.length));
+  for (const [label, cell] of rows) {
+    console.log(`${label.padEnd(labelWidth)}  ${cell}`);
   }
 }
 var removeCommand = defineCommand({
@@ -1190,6 +1206,7 @@ var removeCommand = defineCommand({
     printBlock(targets, scope, "will be removed from:", lockLinesToRemove, "plan", "removed");
     const ask = createConfirmer();
     if (!yes) {
+      console.log("");
       const ok = await ask("Proceed?");
       if (!ok) {
         console.log("Aborted");
@@ -1211,7 +1228,11 @@ var removeCommand = defineCommand({
         removeSkillFromLock(lockPath, t.name);
       lockCleaned = true;
     } else if (scope === "all" && lockLinesToRemove > 0) {
-      const cleanLock = yes || await ask(`Clean skills-lock.json (${plural(lockLinesToRemove, "line")})?`);
+      let cleanLock = yes;
+      if (!yes) {
+        console.log("");
+        cleanLock = await ask(`Clean skills-lock.json (${plural(lockLinesToRemove, "line")})?`);
+      }
       if (cleanLock) {
         for (const t of targets)
           removeSkillFromLock(lockPath, t.name);
@@ -1224,7 +1245,7 @@ var removeCommand = defineCommand({
 });
 
 // src/commands/usage.ts
-import { existsSync as existsSync6 } from "node:fs";
+import { existsSync as existsSync4 } from "node:fs";
 import { join as join7 } from "node:path";
 
 // src/readers/claude.ts
@@ -1558,10 +1579,10 @@ function readClaudeUsage(options) {
 }
 
 // src/readers/codex.ts
-import { existsSync as existsSync5, readFileSync as readFileSync3 } from "node:fs";
+import { existsSync as existsSync3, readFileSync as readFileSync3 } from "node:fs";
 
 // src/utils/scope.ts
-import { existsSync as existsSync4 } from "node:fs";
+import { existsSync as existsSync2 } from "node:fs";
 import { homedir as homedir4 } from "node:os";
 import { dirname as dirname3, join as join6 } from "node:path";
 function detectScope(opts) {
@@ -1583,7 +1604,7 @@ function encodeClaudeProjectDir(absPath) {
 function findGitRoot(start) {
   let dir = start;
   while (true) {
-    if (existsSync4(join6(dir, ".git")))
+    if (existsSync2(join6(dir, ".git")))
       return dir;
     const parent = dirname3(dir);
     if (parent === dir)
@@ -1656,7 +1677,7 @@ function readCodexMentions(options) {
   const historyPath = expandHome(options.history ?? "~/.codex/history.jsonl");
   const counts = new Map;
   let linesRead = 0;
-  if (!existsSync5(historyPath))
+  if (!existsSync3(historyPath))
     return { counts, filesRead: 0, linesRead: 0 };
   for (const line of readFileSync3(historyPath, "utf8").split(`
 `)) {
@@ -1766,7 +1787,7 @@ async function runUsage(args) {
   });
   const claudeProjectsRoot = expandHome("~/.claude/projects");
   const claudeRoot = args.root ?? (scope.projectRoot ? join7(claudeProjectsRoot, encodeClaudeProjectDir(scope.projectRoot)) : claudeProjectsRoot);
-  const claudeRootMissing = !args.root && !!scope.projectRoot && !existsSync6(claudeRoot);
+  const claudeRootMissing = !args.root && !!scope.projectRoot && !existsSync4(claudeRoot);
   const lockPath = getLockPath(args.global);
   const skillUniverse = discoverSkills({
     isGlobal: args.global,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "skillio",
-  "version": "0.1.15",
+  "version": "0.1.16",
   "description": "Audit and manage AI agent skills for Claude Code and Codex",
   "license": "MIT",
   "author": "ihororlovskyi",