@dealdeploy/skl 1.0.0 → 1.1.0

package/add.ts

@@ -1,24 +1,24 @@
 import { existsSync, lstatSync, mkdirSync, rmSync } from "fs";
 import { join } from "path";
+import { createCliRenderer } from "@opentui/core";
 import {
-  createCliRenderer,
-  BoxRenderable,
-  TextRenderable,
-  ScrollBoxRenderable,
-  TextAttributes,
-  type KeyEvent,
-} from "@opentui/core";
-import { catalogDir, addToLock, downloadSkillFiles, type LockEntry } from "./lib.ts";
+  catalogDir,
+  addToLock,
+  downloadSkillFiles,
+  findSkillEntries,
+  parseRepoArg,
+  type LockEntry,
+  type TreeEntry,
+} from "./lib.ts";
+import { createAddTui, type AddSkillEntry } from "./add-tui.ts";
 
 const CATALOG = catalogDir();
 
-let repo = process.argv[3] ?? "";
-if (repo) {
-  const urlMatch = repo.match(/github\.com\/([^/]+\/[^/]+)/);
-  if (urlMatch) repo = urlMatch[1]!.replace(/\.git$/, "");
-}
-if (!repo || !/^[^/]+\/[^/]+$/.test(repo)) {
-  console.error("Usage: skl add owner/repo");
+const allFlag = process.argv.includes("--all");
+const repoArg = process.argv.slice(3).find((a) => !a.startsWith("-")) ?? "";
+const repo = parseRepoArg(repoArg);
+if (!repo) {
+  console.error("Usage: skl add owner/repo [--all]");
   process.exit(1);
 }
 
@@ -47,27 +47,6 @@ function skillExists(name: string): boolean {
   }
 }
 
-// ── Colors ───────────────────────────────────────────────────────────
-
-const C = {
-  bg: "#1a1a2e",
-  rowBg: "#1a1a2e",
-  rowAltBg: "#1f1f38",
-  cursorBg: "#2a2a5a",
-  accentBg: "#3a3a7a",
-  border: "#444477",
-  fg: "#ccccdd",
-  fgDim: "#666688",
-  checked: "#66ff88",
-  unchecked: "#555566",
-  warning: "#ffaa44",
-  accent: "#8888ff",
-  title: "#aaaaff",
-  footer: "#888899",
-  statusOk: "#66ff88",
-  statusErr: "#ff6666",
-};
-
 // ── Fetch phase ─────────────────────────────────────────────────────
 
 console.log(`Fetching ${repo}...`);
@@ -84,21 +63,9 @@ try {
 const treeJson = JSON.parse(
   await gh(`repos/${repo}/git/trees/${branch}?recursive=1`)
 );
-const tree: { path: string; type: string; sha: string }[] = treeJson.tree;
+const tree: TreeEntry[] = treeJson.tree;
 
-// Find SKILL.md at any depth
-const skillEntries = tree
-  .filter((e) => e.type === "blob" && e.path.endsWith("/SKILL.md"))
-  .map((e) => {
-    const prefix = e.path.replace(/\/SKILL\.md$/, "");
-    const name = prefix.split("/").pop()!;
-    // Find the tree SHA for this skill's directory
-    const dirEntry = tree.find(
-      (t) => t.path === prefix && t.type === "tree"
-    );
-    return { prefix, name, treeSHA: dirEntry?.sha ?? "" };
-  })
-  .sort((a, b) => a.name.localeCompare(b.name));
+const skillEntries = findSkillEntries(tree);
 
 if (skillEntries.length === 0) {
   console.error(`No skills found in ${repo} (no directories with SKILL.md)`);
@@ -107,206 +74,28 @@ if (skillEntries.length === 0) {
 
 // ── Classify skills ──────────────────────────────────────────────────
 
-type SkillEntry = { name: string; prefix: string; treeSHA: string; exists: boolean };
-const skills: SkillEntry[] = skillEntries.map(({ name, prefix, treeSHA }) => ({
+const skills: AddSkillEntry[] = skillEntries.map(({ name, prefix, treeSHA }) => ({
   name,
   prefix,
   treeSHA,
   exists: skillExists(name),
 }));
 
-const addableCount = skills.filter((s) => !s.exists).length;
-if (addableCount === 0) {
+const addable = skills.filter((s) => !s.exists);
+if (addable.length === 0) {
   console.log("All skills already exist in your catalog.");
   process.exit(0);
 }
 
-// ── State ────────────────────────────────────────────────────────────
-
-let cursor = 0;
-const checked = new Set<number>();
-let statusTimeout: ReturnType<typeof setTimeout> | null = null;
-let exitResolve: () => void;
-
-for (let i = 0; i < skills.length; i++) {
-  if (!skills[i]!.exists) { cursor = i; break; }
-}
-
-// ── Build TUI ────────────────────────────────────────────────────────
-
-const renderer = await createCliRenderer({ exitOnCtrlC: true });
-
-const outer = new BoxRenderable(renderer, {
-  id: "outer",
-  width: "100%",
-  height: "100%",
-  flexDirection: "column",
-  backgroundColor: C.bg,
-});
-
-const header = new TextRenderable(renderer, {
-  id: "header",
-  content: ` Add skills from ${repo}`,
-  fg: C.title,
-  attributes: TextAttributes.BOLD,
-  height: 1,
-});
-
-const sep = new TextRenderable(renderer, {
-  id: "sep",
-  content: "\u2500".repeat(60),
-  fg: C.border,
-  height: 1,
-});
-
-const scrollBox = new ScrollBoxRenderable(renderer, {
-  id: "skill-list",
-  flexGrow: 1,
-  width: "100%",
-});
-
-type RowRefs = {
-  row: BoxRenderable;
-  checkText: TextRenderable;
-  nameText: TextRenderable;
-};
-const rows: RowRefs[] = [];
-
-for (let i = 0; i < skills.length; i++) {
-  const skill = skills[i]!;
-
-  const row = new BoxRenderable(renderer, {
-    id: `row-${i}`,
-    flexDirection: "row",
-    height: 1,
-    width: "100%",
-    paddingLeft: 1,
-    backgroundColor: i % 2 === 0 ? C.rowBg : C.rowAltBg,
-  });
-
-  const checkText = new TextRenderable(renderer, {
-    id: `check-${i}`,
-    content: skill.exists ? "[*]" : "[ ]",
-    fg: skill.exists ? C.fgDim : C.unchecked,
-    width: 4,
-  });
-
-  const label = skill.exists ? `${skill.name} (exists)` : skill.name;
-  const nameText = new TextRenderable(renderer, {
-    id: `name-${i}`,
-    content: `  ${label}`,
-    fg: skill.exists ? C.fgDim : C.fg,
-    width: 40,
-  });
-
-  row.add(checkText);
-  row.add(nameText);
-  scrollBox.add(row);
-  rows.push({ row, checkText, nameText });
-}
-
-const footerSep = new TextRenderable(renderer, {
-  id: "footer-sep",
-  content: "\u2500".repeat(60),
-  fg: C.border,
-  height: 1,
-});
-
-const footer = new TextRenderable(renderer, {
-  id: "footer",
-  content: " j/k move  space toggle  a all  enter confirm  q cancel",
-  fg: C.footer,
-  height: 1,
-});
-
-const statusLine = new TextRenderable(renderer, {
-  id: "status",
-  content: "",
-  fg: C.statusOk,
-  height: 1,
-});
-
-outer.add(header);
-outer.add(sep);
-outer.add(scrollBox);
-outer.add(footerSep);
-outer.add(footer);
-outer.add(statusLine);
-renderer.root.add(outer);
-
-// ── Display helpers ──────────────────────────────────────────────────
-
-function updateRow(i: number) {
-  const skill = skills[i]!;
-  const r = rows[i]!;
-  const isCursor = cursor === i;
-
-  const baseBg = i % 2 === 0 ? C.rowBg : C.rowAltBg;
-  r.row.backgroundColor = isCursor ? C.cursorBg : baseBg;
-
-  if (skill.exists) {
-    r.checkText.content = "[*]";
-    r.checkText.fg = C.fgDim;
-    const pointer = isCursor ? "\u25b8" : " ";
-    r.nameText.content = `${pointer} ${skill.name} (exists)`;
-    r.nameText.fg = C.fgDim;
-    r.nameText.attributes = isCursor ? TextAttributes.BOLD : TextAttributes.NONE;
-  } else {
-    const isChecked = checked.has(i);
-    r.checkText.content = isChecked ? "[x]" : "[ ]";
-    r.checkText.fg = isCursor ? C.accent : (isChecked ? C.checked : C.unchecked);
-    const pointer = isCursor ? "\u25b8" : " ";
-    r.nameText.content = `${pointer} ${skill.name}`;
-    r.nameText.fg = isCursor ? "#ffffff" : C.fg;
-    r.nameText.attributes = isCursor ? TextAttributes.BOLD : TextAttributes.NONE;
-  }
-}
-
-function setStatus(msg: string, color: string) {
-  statusLine.content = ` ${msg}`;
-  statusLine.fg = color;
-  if (statusTimeout) clearTimeout(statusTimeout);
-  statusTimeout = setTimeout(() => {
-    statusLine.content = "";
-  }, 3000);
-}
-
-function selectedCount(): number {
-  return checked.size;
-}
-
-function updateHeader() {
-  header.content = ` Add skills from ${repo}  (${selectedCount()} selected)`;
-}
-
-function refreshAll() {
-  for (let i = 0; i < skills.length; i++) updateRow(i);
-  updateHeader();
-}
-
-function ensureVisible() {
-  scrollBox.scrollTo(Math.max(0, cursor - 2));
-}
-
-refreshAll();
-
-// ── Confirm & download ───────────────────────────────────────────────
-
-async function confirmAndDownload() {
-  const selected = [...checked].map((i) => skills[i]!);
-  if (selected.length === 0) {
-    setStatus("Nothing selected \u2014 use space to toggle", C.warning);
-    return;
-  }
-
-  renderer.destroy();
+// ── Download helper ─────────────────────────────────────────────────
 
+async function downloadSelected(selected: AddSkillEntry[]) {
   mkdirSync(CATALOG, { recursive: true });
+  let added = 0;
 
   for (const skill of selected) {
     const skillDir = join(CATALOG, skill.name);
 
-    // Remove broken remnant if present
     try {
       if (lstatSync(skillDir).isSymbolicLink()) {
         rmSync(skillDir, { force: true });
@@ -314,87 +103,60 @@ async function confirmAndDownload() {
     } catch {}
 
     mkdirSync(skillDir, { recursive: true });
+    try {
+      const fileCount = await downloadSkillFiles(
+        repo, branch, skill.prefix, skillDir, tree,
+      );
+
+      const entry: LockEntry = {
+        source: repo,
+        sourceUrl: `https://github.com/${repo}`,
+        skillPath: skill.prefix,
+        treeSHA: skill.treeSHA,
+        addedAt: new Date().toISOString(),
+      };
+      addToLock(skill.name, entry);
+
+      console.log(`  ${skill.name} — added (${fileCount} files)`);
+      added++;
+    } catch (e: any) {
+      console.log(`  ${skill.name} — failed (${e.message})`);
+    }
+  }
 
-    process.stdout.write(`  ${skill.name}...`);
-
-    const fileCount = await downloadSkillFiles(
-      repo!, branch, skill.prefix, skillDir, tree,
-    );
+  console.log(`\nAdded ${added}/${selected.length} skill(s) to ~/.skl/catalog/`);
+}
 
-    // Write lock entry
-    const entry: LockEntry = {
-      source: repo!,
-      sourceUrl: `https://github.com/${repo}`,
-      skillPath: skill.prefix,
-      treeSHA: skill.treeSHA,
-      addedAt: new Date().toISOString(),
-    };
-    addToLock(skill.name, entry);
+// ── --all: skip TUI ─────────────────────────────────────────────────
 
-    console.log(` done (${fileCount} files)`);
+if (allFlag) {
+  const skipped = skills.filter((s) => s.exists);
+  for (const s of skipped) {
+    console.log(`  ${s.name} — skipped (already exists)`);
   }
-
-  console.log(`\nAdded ${selected.length} skill(s) to ~/.skl/catalog/`);
-  exitResolve();
+  console.log(`Adding ${addable.length} skill(s)...`);
+  await downloadSelected(addable);
+  process.exit(0);
 }
 
-// ── Key handler ──────────────────────────────────────────────────────
+// ── TUI ─────────────────────────────────────────────────────────────
 
-renderer.keyInput.on("keypress", (key: KeyEvent) => {
-  const prevCursor = cursor;
+let exitResolve: () => void;
 
-  switch (key.name) {
-    case "j":
-    case "down":
-      if (cursor < skills.length - 1) cursor++;
-      break;
-    case "k":
-    case "up":
-      if (cursor > 0) cursor--;
-      break;
-    case "space": {
-      if (skills[cursor]!.exists) {
-        setStatus(`${skills[cursor]!.name} already exists`, C.warning);
-        break;
-      }
-      if (checked.has(cursor)) {
-        checked.delete(cursor);
-      } else {
-        checked.add(cursor);
-      }
-      break;
-    }
-    case "a": {
-      const addable = skills
-        .map((s, i) => (!s.exists ? i : -1))
-        .filter((i) => i >= 0);
-      const allChecked = addable.every((i) => checked.has(i));
-      if (allChecked) {
-        for (const i of addable) checked.delete(i);
-      } else {
-        for (const i of addable) checked.add(i);
-      }
-      for (const i of addable) updateRow(i);
-      updateHeader();
-      ensureVisible();
-      return;
-    }
-    case "return":
-      confirmAndDownload();
-      return;
-    case "q":
-    case "escape":
-      renderer.destroy();
-      exitResolve();
-      return;
-    default:
-      return;
-  }
+const renderer = await createCliRenderer({ exitOnCtrlC: true });
 
-  if (prevCursor !== cursor) updateRow(prevCursor);
-  updateRow(cursor);
-  updateHeader();
-  ensureVisible();
+createAddTui(renderer, {
+  repo,
+  skills,
+  async onConfirm(selected) {
+    renderer.destroy();
+    await downloadSelected(selected);
+    exitResolve();
+  },
+  onCancel() {
+    renderer.destroy();
+    exitResolve();
+  },
 });
 
 await new Promise<void>((resolve) => {

package/index.ts

@@ -4,7 +4,7 @@ import { createCliRenderer } from "@opentui/core";
 import { existsSync, rmSync, cpSync, mkdirSync, readdirSync } from "fs";
 import { join } from "path";
 import { homedir } from "os";
-import { getCatalogSkills, getLockEntry, removeFromLock, catalogDir } from "./lib.ts";
+import { getCatalogSkills, removeFromLock, catalogDir, buildAddArgs, parseSkillsListOutput } from "./lib.ts";
 import { createTui, type ColId } from "./tui.ts";
 
 // @ts-ignore - bun supports JSON imports
@@ -16,6 +16,21 @@ if (arg === "-v" || arg === "--version") {
   console.log(`skl ${VERSION}`);
   process.exit(0);
 }
+if (arg === "-h" || arg === "--help" || arg === "help") {
+  console.log(`skl ${VERSION} — TUI skill manager for Claude Code agents
+
+Usage:
+  skl                Open the interactive TUI
+  skl add <repo>     Add skills from a GitHub repo (interactive)
+  skl add <repo> --all  Add all skills from a repo (non-interactive)
+  skl update         Update all remote skills
+  skl help           Show this help message
+
+Options:
+  -h, --help         Show this help message
+  -v, --version      Show version`);
+  process.exit(0);
+}
 
 // ── Subcommand routing ───────────────────────────────────────────────
 if (arg === "add") {
@@ -59,10 +74,7 @@ async function listInstalled(global: boolean): Promise<Set<string>> {
     const out = await new Response(proc.stdout).text();
     const code = await proc.exited;
     if (code !== 0) return new Set();
-    const data = JSON.parse(out.trim());
-    if (Array.isArray(data)) {
-      return new Set(data.map((s: { name: string }) => s.name));
-    }
+    return new Set(parseSkillsListOutput(out.trim()));
   } catch {}
   return new Set();
 }
@@ -74,20 +86,6 @@ const [globalInstalled, localInstalled] = await Promise.all([
 ]);
 console.log(" done");
 
-// ── Build add/remove args ────────────────────────────────────────────
-
-function buildAddArgs(name: string, global: boolean): string[] {
-  const lockEntry = getLockEntry(name);
-  if (lockEntry) {
-    const args = ["add", lockEntry.sourceUrl, "--skill", name, "-y"];
-    if (global) args.push("-g");
-    return args;
-  }
-  const args = ["add", join(CATALOG, name), "-y"];
-  if (global) args.push("-g");
-  return args;
-}
-
 // ── Create TUI ──────────────────────────────────────────────────────
 
 const renderer = await createCliRenderer({ exitOnCtrlC: true });
@@ -101,7 +99,7 @@ const tui = createTui(renderer, {
   async onToggle(col: ColId, name: string, enable: boolean) {
     const isGlobal = col === "global";
     const args = enable
-      ? buildAddArgs(name, isGlobal)
+      ? buildAddArgs(CATALOG, name, isGlobal)
       : ["remove", name, "-y", ...(isGlobal ? ["-g"] : [])];
     const proc = Bun.spawn(["npx", "-y", "skills", ...args], {
       stdout: "pipe",