@dealdeploy/skl 0.1.7 → 0.2.0

package/index.ts

@@ -39,6 +39,10 @@ if (arg === "add") {
   await import("./add.ts");
   process.exit(0);
 }
+if (arg === "update") {
+  await import("./update.ts");
+  process.exit(0);
+}
 
 try {
 
@@ -95,8 +99,35 @@ function nonLibraryExists(dir: string, name: string): boolean {
   }
 }
 
+function isLocalDir(dir: string): boolean {
+  return dir !== GLOBAL_DIR && dir !== GLOBAL_CLAUDE_DIR;
+}
+
+function isEnabled(dir: string, name: string): boolean {
+  if (isLocalDir(dir)) return existsSync(join(dir, name));
+  return isLibraryLink(dir, name);
+}
+
+function isBlocked(dir: string, name: string): boolean {
+  if (isLocalDir(dir)) return false;
+  return nonLibraryExists(dir, name);
+}
+
 function toggle(dir: string, name: string): boolean {
   const full = join(dir, name);
+  if (isLocalDir(dir)) {
+    // Copy-based toggle for project-local dirs
+    if (existsSync(full)) {
+      rmSync(full, { recursive: true, force: true });
+      return true;
+    }
+    // Remove any broken symlink left from old behavior
+    try { rmSync(full, { force: true }); } catch {}
+    mkdirSync(dir, { recursive: true });
+    cpSync(join(LIBRARY, name), full, { recursive: true });
+    return true;
+  }
+  // Symlink-based toggle for global dirs
   if (isLibraryLink(dir, name)) {
     unlinkSync(full);
     return true;
@@ -145,14 +176,11 @@ function ensureLocalClaudeDir(): string {
 
 // ── Migration check ──────────────────────────────────────────────────
 {
+  // Only migrate global dirs — project-local dirs use copies, not symlinks
   const dirsToCheck: [string, string][] = [
     [GLOBAL_DIR, "~/.agents/skills"],
     [GLOBAL_CLAUDE_DIR, "~/.claude/skills"],
   ];
-  const ld = findLocalDir();
-  if (ld) dirsToCheck.push([ld, ld.replace(homedir(), "~")]);
-  const lcd = findLocalClaudeDir();
-  if (lcd) dirsToCheck.push([lcd, lcd.replace(homedir(), "~")]);
 
   const migrations: { dir: string; label: string; name: string; conflict: boolean }[] = [];
 
@@ -171,7 +199,7 @@ function ensureLocalClaudeDir(): string {
   if (migrations.length > 0) {
     console.log("\nFound skills not managed by library:");
     for (const m of migrations) {
-      console.log(`  ${m.label}/${m.name}${m.conflict ? "  (conflict — already in library)" : ""}`);
+      console.log(`  ${m.label}/${m.name}${m.conflict ? "  (already in library)" : ""}`);
     }
     process.stdout.write(`\nMigrate to ${LIBRARY.replace(homedir(), "~")} and replace with symlinks? [y/N] `);
 
@@ -181,33 +209,89 @@ function ensureLocalClaudeDir(): string {
 
     if (answer === "y" || answer === "yes") {
       let migrated = 0;
-      let skipped = 0;
       for (const m of migrations) {
         const src = join(m.dir, m.name);
         const dest = join(LIBRARY, m.name);
         if (m.conflict) {
-          console.log(`  skip ${m.name} (already exists in library)`);
-          skipped++;
-          continue;
-        }
-        try {
-          renameSync(src, dest);
-        } catch {
-          // cross-device fallback
-          cpSync(src, dest, { recursive: true });
+          // Library already has this skill — remove local and symlink to library
           rmSync(src, { recursive: true, force: true });
+          symlinkSync(dest, src);
+          console.log(`  replaced ${m.name} (library version kept)`);
+        } else {
+          try {
+            renameSync(src, dest);
+          } catch {
+            // cross-device fallback
+            cpSync(src, dest, { recursive: true });
+            rmSync(src, { recursive: true, force: true });
+          }
+          symlinkSync(dest, src);
+          console.log(`  migrated ${m.name}`);
         }
-        symlinkSync(dest, src);
-        console.log(`  migrated ${m.name}`);
         migrated++;
       }
-      console.log(`\n${migrated} migrated${skipped ? `, ${skipped} skipped` : ""}\n`);
+      console.log(`\n${migrated} migrated\n`);
     } else {
       console.log("Skipping migration.\n");
     }
   }
 }
 
+// ── Local symlink → copy conversion ─────────────────────────────────
+{
+  const localDirsToCheck: [string, string][] = [];
+  const ld = findLocalDir();
+  if (ld) localDirsToCheck.push([ld, ld.replace(homedir(), "~")]);
+  const lcd = findLocalClaudeDir();
+  if (lcd) localDirsToCheck.push([lcd, lcd.replace(homedir(), "~")]);
+
+  const localSymlinks: { dir: string; label: string; name: string; target: string }[] = [];
+
+  for (const [dir, label] of localDirsToCheck) {
+    try {
+      for (const name of readdirSync(dir)) {
+        const full = join(dir, name);
+        try {
+          if (lstatSync(full).isSymbolicLink()) {
+            localSymlinks.push({ dir, label, name, target: readlinkSync(full) });
+          }
+        } catch {}
+      }
+    } catch {}
+  }
+
+  if (localSymlinks.length > 0) {
+    console.log("\nFound symlinked skills in project-local dirs:");
+    for (const s of localSymlinks) {
+      console.log(`  ${s.label}/${s.name} → ${s.target}`);
+    }
+    process.stdout.write("\nConvert to local copies? (recommended for portability) [y/N] ");
+
+    const buf = new Uint8Array(100);
+    const n = readSync(0, buf);
+    const answer = new TextDecoder().decode(buf.subarray(0, n)).trim().toLowerCase();
+
+    if (answer === "y" || answer === "yes") {
+      let converted = 0;
+      for (const s of localSymlinks) {
+        const full = join(s.dir, s.name);
+        const libPath = join(LIBRARY, s.name);
+        unlinkSync(full);
+        if (existsSync(libPath)) {
+          cpSync(libPath, full, { recursive: true });
+          console.log(`  converted ${s.name}`);
+          converted++;
+        } else {
+          console.log(`  removed ${s.name} (not in library)`);
+        }
+      }
+      console.log(`\n${converted} converted\n`);
+    } else {
+      console.log("Skipping conversion.\n");
+    }
+  }
+}
+
 const allSkills = readdirSync(LIBRARY).sort();
 
 // ── State ───────────────────────────────────────────────────────────
@@ -297,14 +381,23 @@ const colHeaderRow = new BoxRenderable(renderer, {
   paddingLeft: 1,
 });
 
+let COL_W = 14;
+let NAME_W = 34;
+
+function calcWidths() {
+  const w = process.stdout.columns || 80;
+  COL_W = Math.max(5, Math.min(14, Math.floor((w - 20) / 4)));
+  NAME_W = Math.min(34, Math.max(15, w - COL_W * 4 - 1));
+}
+calcWidths();
+
 const colName = new TextRenderable(renderer, {
   id: "col-name",
   content: "Skill",
   fg: C.fgDim,
   attributes: TextAttributes.BOLD,
-  width: 34,
+  width: NAME_W,
 });
-const COL_W = 14;
 const colGlobal = new TextRenderable(renderer, {
   id: "col-global",
   content: "~/.agents",
@@ -382,9 +475,9 @@ for (let i = 0; i < allSkills.length; i++) {
 
   const nameText = new TextRenderable(renderer, {
     id: `name-${i}`,
-    content: `  ${skill}`,
+    content: `  ${ellipsize(skill, NAME_W - 3)}`,
     fg: C.fg,
-    width: 34,
+    width: NAME_W,
   });
 
   const gChecked = isLibraryLink(GLOBAL_DIR, skill);
@@ -405,8 +498,8 @@ for (let i = 0; i < allSkills.length; i++) {
     width: COL_W,
   });
 
-  const lChecked = localDir ? isLibraryLink(localDir, skill) : false;
-  const lBlocked = localDir ? nonLibraryExists(localDir, skill) : false;
+  const lChecked = localDir ? isEnabled(localDir, skill) : false;
+  const lBlocked = localDir ? isBlocked(localDir, skill) : false;
   const localText = new TextRenderable(renderer, {
     id: `local-${i}`,
     content: localDir ? checkboxStr(lChecked, lBlocked) : "  -",
@@ -414,8 +507,8 @@ for (let i = 0; i < allSkills.length; i++) {
     width: COL_W,
   });
 
-  const lcChecked = localClaudeDir ? isLibraryLink(localClaudeDir, skill) : false;
-  const lcBlocked = localClaudeDir ? nonLibraryExists(localClaudeDir, skill) : false;
+  const lcChecked = localClaudeDir ? isEnabled(localClaudeDir, skill) : false;
+  const lcBlocked = localClaudeDir ? isBlocked(localClaudeDir, skill) : false;
   const localClaudeText = new TextRenderable(renderer, {
     id: `local-claude-${i}`,
     content: localClaudeDir ? checkboxStr(lcChecked, lcBlocked) : "  -",
@@ -442,7 +535,7 @@ const footerSep = new TextRenderable(renderer, {
 
 const footer = new TextRenderable(renderer, {
   id: "footer",
-  content: " ↑↓ move  ←→/tab col  enter toggle  ^a all  ^e edit  ^d del  ↑search  esc quit",
+  content: " ↑↓ move  ←→/tab col  enter toggle  ^a all  ^e edit  ^d del  / search  q/esc quit",
   fg: C.footer,
   height: 1,
 });
@@ -513,7 +606,7 @@ function updateRow(i: number) {
   r.row.backgroundColor = isCursor ? C.cursorBg : baseBg;
 
   const pointer = isCursor ? "▸" : " ";
-  r.nameText.content = `${pointer} ${skill}`;
+  r.nameText.content = `${pointer} ${ellipsize(skill, NAME_W - 3)}`;
   r.nameText.fg = isCursor ? "#ffffff" : C.fg;
   r.nameText.attributes = isCursor ? TextAttributes.BOLD : TextAttributes.NONE;
 
@@ -533,8 +626,8 @@ function updateRow(i: number) {
   r.globalClaudeText.bg = gcActive ? C.accentBg : undefined;
   r.globalClaudeText.attributes = gcActive ? TextAttributes.BOLD : TextAttributes.NONE;
 
-  const lChecked = localDir ? isLibraryLink(localDir, skill) : false;
-  const lBlocked = localDir ? nonLibraryExists(localDir, skill) : false;
+  const lChecked = localDir ? isEnabled(localDir, skill) : false;
+  const lBlocked = localDir ? isBlocked(localDir, skill) : false;
   const lActive = isCursor && cursorCol === "local";
   if (localDir) {
     r.localText.content = checkboxStr(lChecked, lBlocked);
@@ -547,8 +640,8 @@ function updateRow(i: number) {
     r.localText.bg = lActive ? C.accentBg : undefined;
   }
 
-  const lcChecked = localClaudeDir ? isLibraryLink(localClaudeDir, skill) : false;
-  const lcBlocked = localClaudeDir ? nonLibraryExists(localClaudeDir, skill) : false;
+  const lcChecked = localClaudeDir ? isEnabled(localClaudeDir, skill) : false;
+  const lcBlocked = localClaudeDir ? isBlocked(localClaudeDir, skill) : false;
   const lcActive = isCursor && cursorCol === "localClaude";
   if (localClaudeDir) {
     r.localClaudeText.content = checkboxStr(lcChecked, lcBlocked);
@@ -571,6 +664,30 @@ function setStatus(msg: string, color: string) {
   }, 3000);
 }
 
+function relayout() {
+  calcWidths();
+  colName.width = NAME_W;
+  colGlobal.width = COL_W;
+  colGlobalClaude.width = COL_W;
+  colLocal.width = COL_W;
+  colLocal.content = ellipsize(localLabel, COL_W - 1);
+  colLocalClaude.width = COL_W;
+  colLocalClaude.content = ellipsize(localClaudeLabel, COL_W - 1);
+  for (let i = 0; i < allSkills.length; i++) {
+    const r = rows[i];
+    r.nameText.width = NAME_W;
+    r.globalText.width = COL_W;
+    r.globalClaudeText.width = COL_W;
+    r.localText.width = COL_W;
+    r.localClaudeText.width = COL_W;
+  }
+}
+
+process.stdout.on("resize", () => {
+  relayout();
+  refreshAll();
+});
+
 function refreshAll() {
   for (const i of filteredIndices) updateRow(i);
   updateSearchBar();
@@ -604,7 +721,7 @@ function toggleAllColumn(col: ColId) {
   // Determine intent: if majority of visible skills are linked, unlink all; otherwise link all
   let linkedCount = 0;
   for (const i of filteredIndices) {
-    if (isLibraryLink(dir, allSkills[i])) linkedCount++;
+    if (isEnabled(dir, allSkills[i])) linkedCount++;
   }
   const shouldLink = linkedCount <= filteredIndices.length / 2;
   let changed = 0;
@@ -612,7 +729,7 @@ function toggleAllColumn(col: ColId) {
 
   for (const i of filteredIndices) {
     const skill = allSkills[i];
-    const isLinked = isLibraryLink(dir, skill);
+    const isLinked = isEnabled(dir, skill);
     if (shouldLink && !isLinked) {
       if (toggle(dir, skill)) changed++;
       else skipped++;
@@ -658,11 +775,11 @@ function cancelPendingDelete() {
 function deleteSkill(idx: number) {
   const skill = allSkills[idx];
 
-  // Remove symlinks first
+  // Remove from all skill dirs (symlinks for global, copies for local)
   if (isLibraryLink(GLOBAL_DIR, skill)) unlinkSync(join(GLOBAL_DIR, skill));
   if (isLibraryLink(GLOBAL_CLAUDE_DIR, skill)) unlinkSync(join(GLOBAL_CLAUDE_DIR, skill));
-  if (localDir && isLibraryLink(localDir, skill)) unlinkSync(join(localDir, skill));
-  if (localClaudeDir && isLibraryLink(localClaudeDir, skill)) unlinkSync(join(localClaudeDir, skill));
+  if (localDir && existsSync(join(localDir, skill))) rmSync(join(localDir, skill), { recursive: true, force: true });
+  if (localClaudeDir && existsSync(join(localClaudeDir, skill))) rmSync(join(localClaudeDir, skill), { recursive: true, force: true });
 
   // Remove from library
   rmSync(join(LIBRARY, skill), { recursive: true, force: true });
@@ -791,6 +908,20 @@ renderer.keyInput.on("keypress", (key: KeyEvent) => {
     return;
   }
 
+  // ── "q" to quit ──
+  if (key.sequence === "q") {
+    renderer.destroy();
+    process.exit(0);
+  }
+
+  // ── "/" to focus search (vim-style) ──
+  if (key.sequence === "/") {
+    focusArea = "search";
+    refreshAll();
+    ensureVisible();
+    return;
+  }
+
   // ── Navigation & actions ──
   switch (key.name) {
     case "down":
@@ -835,7 +966,7 @@ renderer.keyInput.on("keypress", (key: KeyEvent) => {
       const { dir, label } = dirForCol(cursorCol);
       const ok = toggle(dir, skill);
       if (ok) {
-        const linked = isLibraryLink(dir, skill);
+        const linked = isEnabled(dir, skill);
         setStatus(
           linked ? `✓ ${skill} → ${label}` : `✗ ${skill} removed from ${label}`,
           linked ? C.statusOk : C.statusErr

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dealdeploy/skl",
-  "version": "0.1.7",
+  "version": "0.2.0",
   "description": "TUI skill manager for Claude Code agents",
   "module": "index.ts",
   "bin": {

package/update.ts

@@ -0,0 +1,87 @@
+#!/usr/bin/env bun
+
+import { readdirSync, existsSync, cpSync, rmSync, lstatSync, readlinkSync, readSync, unlinkSync } from "fs";
+import { join, resolve } from "path";
+import { homedir } from "os";
+
+const LIBRARY = join(homedir(), "dotfiles/skills");
+
+if (!existsSync(LIBRARY)) {
+  console.error(`skl: library not found at ${LIBRARY}`);
+  process.exit(1);
+}
+
+const localDirs: [string, string][] = [];
+const agentsDir = join(process.cwd(), ".agents/skills");
+if (existsSync(agentsDir)) localDirs.push([agentsDir, ".agents/skills"]);
+
+const claudeDir = join(process.cwd(), ".claude/skills");
+if (existsSync(claudeDir)) localDirs.push([claudeDir, ".claude/skills"]);
+
+if (localDirs.length === 0) {
+  console.log("No local skill directories found.");
+  process.exit(0);
+}
+
+type Entry = { dir: string; label: string; name: string; kind: "symlink" | "copy" };
+
+const symlinks: Entry[] = [];
+const copies: Entry[] = [];
+let skipped = 0;
+
+for (const [dir, label] of localDirs) {
+  for (const name of readdirSync(dir)) {
+    const localPath = join(dir, name);
+    const libPath = join(LIBRARY, name);
+    if (!existsSync(libPath)) {
+      skipped++;
+      continue;
+    }
+    const isSym = lstatSync(localPath).isSymbolicLink();
+    const entry: Entry = { dir, label, name, kind: isSym ? "symlink" : "copy" };
+    if (isSym) symlinks.push(entry);
+    else copies.push(entry);
+  }
+}
+
+// Handle symlinks → copies (requires confirmation)
+if (symlinks.length > 0) {
+  console.log("\nSymlinks to convert to local copies:");
+  for (const s of symlinks) {
+    console.log(`  ${s.label}/${s.name} → ${readlinkSync(join(s.dir, s.name))}`);
+  }
+  process.stdout.write(`\nReplace ${symlinks.length} symlink(s) with copies from library? [y/N] `);
+
+  const buf = new Uint8Array(100);
+  const n = readSync(0, buf);
+  const answer = new TextDecoder().decode(buf.subarray(0, n)).trim().toLowerCase();
+
+  if (answer === "y" || answer === "yes") {
+    for (const s of symlinks) {
+      const localPath = join(s.dir, s.name);
+      unlinkSync(localPath);
+      cpSync(join(LIBRARY, s.name), localPath, { recursive: true });
+      console.log(`  converted ${s.label}/${s.name}`);
+    }
+  } else {
+    console.log("Skipping symlink conversion.");
+  }
+}
+
+// Update existing copies
+if (copies.length > 0) {
+  console.log(`\nUpdating ${copies.length} local copies from library...`);
+  for (const c of copies) {
+    const localPath = join(c.dir, c.name);
+    rmSync(localPath, { recursive: true, force: true });
+    cpSync(join(LIBRARY, c.name), localPath, { recursive: true });
+    console.log(`  updated ${c.label}/${c.name}`);
+  }
+}
+
+if (symlinks.length === 0 && copies.length === 0) {
+  console.log("Nothing to update.");
+}
+if (skipped > 0) {
+  console.log(`${skipped} skipped (not in library)`);
+}