skilllib 1.11.5 → 1.11.7

package/dist/{chunk-TE2EDZ7L.js → chunk-PKQPUIN3.js}

@@ -1931,7 +1931,7 @@ function setPluginOn(p, on) {
   }
   const r = setPluginIn(settingsFileOf(p), p.id, on);
   if (!r.ok) return { ok: false, message: `${p.id}: ${r.reason}` };
-  if (p.scope === "local" && p.projectPath) return { ...done, message: done.message + ignoreLocalSettings(p.projectPath) };
+  if (p.scope === "local" && p.projectPath) return { ...done, message: `${done.message}; it's in .claude/settings.local.json${ignoreLocalSettings(p.projectPath)}` };
   return done;
 }
 function uninstallPlugin(p) {
@@ -1966,27 +1966,16 @@ function removePlugin(plugin, how) {
   if (plugin.settingsOnly && how === "delete") return { ok: false, message: `${plugin.id} isn't installed, only turned on in your settings: turn it off instead` };
   const lost = unplaced(plugin, how === "delete" ? "uninstall" : "disable");
   if (lost) return lost;
-  const viaSettings = plugin.synced || plugin.settingsOnly || !claudeBinary() && how === "off" && !plugin.projectPath;
-  if (!viaSettings && !claudeBinary()) {
-    return { ok: false, message: `${plugin.id}: nothing changed; Claude Code's CLI didn't run (tried \`claude\` on your PATH and ~/.local/bin/claude)` };
-  }
-  const wasOn = enabledPlugins(plugin.projectPath)[plugin.id] === true;
+  const uninstall = how === "delete" && !plugin.synced;
+  if (uninstall && !claudeBinary()) return { ok: false, message: `${plugin.id}: nothing changed; Claude Code's CLI didn't run (tried \`claude\` on your PATH and ~/.local/bin/claude)` };
   const yours = new Set(machineSkills().filter((m) => m.movable).map((m) => m.name));
   const imported = plugin.skills.filter((dir) => !yours.has(basename4(dir)) && latestVersion(basename4(dir)) === null).map((dir) => importSkill(dir).name);
-  const scope = plugin.scope ? ["--scope", plugin.scope] : [];
-  const r = viaSettings ? (
-    // Synced and settings-only plugins aren't installed, so `claude plugin` can't touch them: their switch is enabledPlugins.
-    setPluginIn(join11(claudeDir(), "settings.json"), plugin.id, false)
-  ) : (
-    // --keep-data, as uninstallPlugin: its saved data stays, so a restore brings it back as it was.
-    claude(["plugin", how === "delete" ? "uninstall" : "disable", plugin.id, ...how === "delete" ? ["--keep-data"] : [], ...scope], plugin.projectPath)
-  );
   const saved = imported.length ? `; ${imported.join(", ")} copied into Your skills` : "";
-  if (!r.ok) return { ok: false, message: `${plugin.id}: ${r.reason}${saved}` };
-  if (how === "delete" && !plugin.synced) recordRemovedPlugin(plugin.id, plugin.scope ?? "user", plugin.projectPath, wasOn);
-  const others = how === "delete" ? (installedPluginsJson()[plugin.id] ?? []).filter((e) => (e.scope ?? "user") !== (plugin.scope ?? "user") || e.projectPath !== plugin.projectPath) : [];
+  const r = uninstall ? uninstallPlugin(plugin) : setPluginOn(plugin, false);
+  if (!r.ok) return { ok: false, message: `${r.message}${saved}` };
+  const others = uninstall ? (installedPluginsJson()[plugin.id] ?? []).filter((e) => (e.scope ?? "user") !== (plugin.scope ?? "user") || e.projectPath !== plugin.projectPath) : [];
   const left = others.length ? `; its ${others.map((e) => `${e.scope ?? "user"} install${e.projectPath ? ` in ${basename4(e.projectPath)}` : ""}`).join(", ")} stays` : "";
-  return { ok: true, message: `${plugin.id} ${how === "delete" && !plugin.synced ? `removed (${plugin.scope ?? "user"} install)` : "turned off"}${saved}${left}` };
+  return { ok: true, message: uninstall ? `${plugin.id} removed (${plugin.scope ?? "user"} install)${saved}${left}` : `${r.message}${saved}` };
 }
 function recordRemovedPlugin(id, scope, projectPath, on = true) {
   const list2 = readState(removedFile()) ?? [];
@@ -2034,6 +2023,9 @@ function entriesOf(dirs, name) {
     return [{ dir, path, link: link2, real, hash: null }];
   });
 }
+function uniqueReals(entries) {
+  return entries.filter((e, i, all) => !e.link && all.findIndex((o) => !o.link && o.real === e.real) === i);
+}
 function hashReals(reals) {
   if (reals.length > 1) for (const r of reals) r.hash = treeHash(r.path);
 }
@@ -2074,15 +2066,16 @@ function planProjectTidy(root, { keep = {}, enabled = enabledHarnesses() } = {})
   let git2;
   const skillsLock = projectSkillsLock(root);
   const dirs = repoTidyDirs(root).map(abs);
+  const realRoot = realpathOrNull(root) ?? root;
   for (const name of namesIn(dirs)) {
     const entries = entriesOf(dirs, name);
-    const external = entries.find((e) => e.link && e.real && !(e.real + sep6).startsWith(realpathOrNull(root) + sep6));
+    const external = entries.find((e) => e.link && e.real && !(e.real + sep6).startsWith(realRoot + sep6));
     if (external) {
       const differs = entries.some((e) => e !== external && e.real && existsSync11(join12(e.real, "SKILL.md")) && treeHash(e.real) !== treeHash(external.real));
       report.skipped.push({ name, reason: `${rel(external.path)} links outside the repo`, ...differs ? { differs } : {} });
       continue;
     }
-    const reals = entries.filter((e, i, all) => !e.link && all.findIndex((o) => !o.link && realpathOrNull(o.path) === realpathOrNull(e.path)) === i);
+    const reals = uniqueReals(entries);
     if (reals.length === 0) continue;
     hashReals(reals);
     const dep = skills[name];
@@ -2146,7 +2139,7 @@ function planGlobalTidy({ keep = {}, enabled = enabledHarnesses() } = {}) {
       report.skipped.push({ name, reason: "a Claude Code plugin kept in a skills folder" });
       continue;
     }
-    const reals = entries.filter((e, i, all) => !e.link && all.findIndex((o) => !o.link && realpathOrNull(o.path) === realpathOrNull(e.path)) === i);
+    const reals = uniqueReals(entries);
     if (reals.length < 2) continue;
     hashReals(reals);
     const kept = reals.find((r) => r.dir === keep[name]);

package/dist/index.js

@@ -31,7 +31,7 @@ import {
   unhide,
   update,
   usage
-} from "./chunk-TE2EDZ7L.js";
+} from "./chunk-PKQPUIN3.js";
 
 // src/index.ts
 import { readFileSync } from "fs";
@@ -138,10 +138,10 @@ async function main() {
   if (!["--version", "-v", "version", "help", "--help", "-h"].includes(command ?? "")) refreshAgentSkill();
   switch (command) {
     case void 0:
-      return process.stdin.isTTY && process.stdout.isTTY ? (await import("./tui-GFQUHPEC.js")).tui() : overview(args);
+      return process.stdin.isTTY && process.stdout.isTTY ? (await import("./tui-B5PF3J3E.js")).tui() : overview(args);
     case "ui":
       if (!process.stdin.isTTY || !process.stdout.isTTY) throw new Error("`skilllib ui` needs an interactive terminal. Agents and scripts: use `skilllib status --json`.");
-      return (await import("./tui-GFQUHPEC.js")).tui();
+      return (await import("./tui-B5PF3J3E.js")).tui();
     case "status":
       return status(args);
     case "add":

package/dist/{tui-GFQUHPEC.js → tui-B5PF3J3E.js}

@@ -84,7 +84,7 @@ import {
   versionHistory,
   visibleProjects,
   withoutForceHint
-} from "./chunk-TE2EDZ7L.js";
+} from "./chunk-PKQPUIN3.js";
 
 // src/tui/index.tsx
 import { render } from "ink";
@@ -1743,7 +1743,8 @@ function App({ initial, reload, loadUsage: loadUsage2 }) {
       if (isNameChar(input, key)) return setModal({ ...m, query: m.query + input.toLowerCase(), cursor: 0 });
     }
   }
-  const toastLines = toast && modal?.kind !== "confirm" ? wrap(toast, termCols - 2, 3).map((l) => " " + l) : [];
+  const toastMax = Math.max(1, Math.min(Math.max(3, Math.floor(termRows / 3)), termRows - 8 - (sidebar ? 0 : 1)));
+  const toastLines = toast && modal?.kind !== "confirm" ? wrap(toast, termCols - 2, toastMax).map((l) => " " + l) : [];
   const bottomH = Math.max(1, toastLines.length);
   const confirmLines = modal?.kind === "confirm" ? wrap(modal.fix.preview, columns - 6, 6) : [];
   const confirmH = modal?.kind === "confirm" ? confirmLines.length + 2 : 0;
@@ -1786,10 +1787,14 @@ function App({ initial, reload, loadUsage: loadUsage2 }) {
     }), here: onHelp }
   ];
   const sideAt = sideRows.findIndex((r) => r.here);
+  const brand = " \u25C6 skilllib  ";
+  const tabsWidth = (short) => PLACES.reduce((n, [p, label]) => n + (short && p === "library" ? 6 : label.length) + 2 + (p === "health" && healthCount ? ` ${healthCount}`.length : 0), 0);
+  const showBrand = brand.length + tabsWidth(false) <= columns;
+  const shortTabs = !showBrand && tabsWidth(false) > columns;
   const topBar = /* @__PURE__ */ jsx2(Box2, { height: 1, width: columns, justifyContent: "space-between", children: /* @__PURE__ */ jsxs2(Box2, { children: [
-    /* @__PURE__ */ jsx2(Text2, { color: color.accent, bold: true, children: " \u25C6 skilllib  " }),
+    /* @__PURE__ */ jsx2(Text2, { color: color.accent, bold: true, children: showBrand ? brand : "" }),
     PLACES.map(([p, label]) => /* @__PURE__ */ jsxs2(Text2, { backgroundColor: place === p ? color.accent : void 0, color: place === p ? "#0B1020" : color.muted, bold: place === p, children: [
-      ` ${label}`,
+      ` ${shortTabs && p === "library" ? "Skills" : label}`,
       /* @__PURE__ */ jsx2(Text2, { color: place === p ? "#0B1020" : healthWorst ? SEV[healthWorst.severity].color : color.green, children: p === "health" && healthCount ? ` ${healthCount}` : "" }),
       " "
     ] }, p))

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "skilllib",
-  "version": "1.11.5",
+  "version": "1.11.7",
   "description": "Manage your agent skills like dependencies. One library for all your skills; each repo gets only the ones it needs, and every agent can see them. Works with Claude Code, Cursor, Codex, Zed and Grok.",
   "keywords": [
     "agent-skills",