skilllib 1.0.1 → 1.1.0

package/README.md

@@ -47,6 +47,7 @@ Most agents can't turn off a global skill for just one repo. Claude Code can, bu
 - **📦 Per-repo installs, pinned like dependencies.** Each repo lists its skills and versions in `skilllib.json`. `sync` reproduces them exactly, and `update` moves to the newest.
 - **🤝 One copy, every agent.** skilllib puts one real copy in the repo and links it into whichever folders your other agents read.
 - **🧹 A cleanup wizard for global skills.** For each global skill, pick the repos that should keep it, keep it global, or delete it. Usage numbers and descriptions help you decide.
+- **✓ Global on purpose.** Some skills belong everywhere. Mark them once and skilllib stops warning about them.
 - **🔎 See everything in a repo:**
   - where each skill comes from (the repo, your library, global, a plugin, claude.ai…)
   - which agents load it
@@ -119,7 +120,8 @@ A repo has two lists:
 |---|---|---|
 | *From your skills* | Versioned skills you added from your library | Remove it from this repo |
 | *The repo's own* | Skills committed to the repo (e.g. your team's `.agents/skills`) | Copy it into your skills, so other repos can use it |
-| *⚠ Global · yours* | Your global skills: they load in every repo | Open the cleanup wizard |
+| *⚠ Global · yours, not reviewed* | Your global skills you haven't decided about: they load in every repo | Open the cleanup wizard |
+| *✓ Global · yours, on purpose* | Your global skills you chose to keep global | Open the cleanup wizard |
 | *Global · from vendors* | Plugins, claude.ai skills, Cursor built-ins | – (manage them at the source) |
 
 **Add skills** shows your skills that this repo doesn't have yet. `space` adds one.
@@ -171,13 +173,23 @@ Edit a skill once in **Your skills** (Enter → *Edit SKILL.md*). Every repo tha
 Open **Global → ⚠ Clean up…**, or press `space` on any of your global skills. For each skill, `space` cycles through:
 
 - **◉ move:** installed into the repos you tick on the right. Repos that already have it, or where Claude Code used it, are pre-ticked.
-- **○ keep global**
+- **○ keep global**: skilllib remembers this choice, and the next cleanup skips the skill
 - **✕ delete** (`d`): removed without keeping a copy in your library
 
 Each skill shows how often Claude Code used it, and the panel below shows its description and where it was used. Review, apply, and your globals are gone: moved ones are in your library and in the right repos. Originals go to a backup you can restore from **Health**.
 
 You can also delete one skill (Enter → **Delete**) or a whole group (Enter → **Delete all N**) straight from Global.
 
+### Keeping a skill global on purpose
+
+Some skills are useful in every repo, for example a commit-message skill. Mark them so skilllib stops asking about them:
+
+- Enter on the skill (in Global or in a repo) → **✓ Keep global on purpose**. Enter on a group → **Keep all N global on purpose**.
+- In the cleanup wizard, every skill you leave on **○ keep global** is marked when you apply.
+- From the command line: `skilllib global keep <name>`.
+
+A marked skill shows **✓ global** instead of **⚠ loaded globally**, and has its own group in a repo. The cleanup wizard hides it (`k` shows it again), and review prompts tell the agent that you chose to keep it global. The mark only records your decision: no files move, and agents load the skill as before. Duplicate warnings (⧉) still show. To undo, use Enter → **Unmark** or `skilllib global unkeep <name>`.
+
 ### Ask an agent to review your skills
 
 Not sure what to keep? **Copy review prompt** puts a ready-to-paste prompt on your clipboard (also saved to `~/.skilllib/review-prompt.md`). For each skill, it includes:
@@ -277,6 +289,7 @@ skilllib link [--all]           make every skill here usable by all your agents
 
 skilllib list | show <name>     your library
 skilllib import <dir>...        add skill folders to your library (--global: all your global skills)
+skilllib global [keep|unkeep]   your global skills; mark the ones you keep global on purpose
 skilllib new <name> [desc]      create a skill
 
 skilllib usage [--days N]       which skills Claude Code used
@@ -294,7 +307,7 @@ Flags: `--force` (overwrite local edits), `--allow-tracked` (link into git-track
 | `<repo>/skilllib.json` | The repo's skills and pinned versions. **Commit it.** |
 | `~/.skilllib/library/` | Your skills (master copies). Worth putting under git. |
 | `~/.skilllib/store/` | Every version of every skill, immutable |
-| `~/.skilllib/config.json` | Your agents, project folders, hidden projects |
+| `~/.skilllib/config.json` | Your agents, project folders, hidden projects, skills you keep global |
 | `~/.skilllib/trash/`, `global-backup/` | Everything skilllib removed, restorable from Health |
 | `~/.skilllib/review-prompt.md` | The last review prompt you copied |
 
@@ -328,7 +341,7 @@ npm link                                     # use your checkout as `skilllib`
 1. Bump `version` in `package.json` and merge to `main`.
 2. Tag and push:
    ```bash
-   git tag v1.0.1 && git push --tags
+   git tag v1.1.0 && git push --tags
    ```
 
 CI then tests on Linux, macOS and Windows, publishes to npm through [trusted publishing](https://docs.npmjs.com/trusted-publishers) (no token; provenance is automatic), and creates a GitHub release.

package/dist/{chunk-K5RHLQBH.js → chunk-HLCJQVHB.js}

@@ -373,6 +373,19 @@ function allowTrackedLinks(root) {
   const config = readConfig();
   writeConfig({ ...config, agentsDirOk: [.../* @__PURE__ */ new Set([...config.agentsDirOk ?? [], root])].sort() });
 }
+function keptGlobal() {
+  return new Set(readConfig().keepGlobal ?? []);
+}
+function setKeepGlobal(names, keep) {
+  const kept = keptGlobal();
+  const before = kept.size;
+  for (const n of names) {
+    if (keep) kept.add(n);
+    else kept.delete(n);
+  }
+  if (kept.size === before) return;
+  writeConfig({ ...readConfig(), keepGlobal: [...kept].sort() });
+}
 
 // src/sources.ts
 import { existsSync as existsSync6, lstatSync, mkdirSync as mkdirSync4, readdirSync as readdirSync3, readFileSync as readFileSync5, readlinkSync, realpathSync as realpathSync2, writeFileSync as writeFileSync4 } from "fs";
@@ -550,8 +563,7 @@ function librarySkills() {
   return skillDirsIn(libraryDir()).map((dir) => ({
     name: basename3(dir),
     description: readSkillInfo(dir).description,
-    dir,
-    hash: treeHash(dir) ?? ""
+    dir
   }));
 }
 function librarySkillDir(name) {
@@ -850,7 +862,9 @@ function unloadGlobal(path, links = [], { requireLibrary = true } = {}) {
   const broken = !existsSync7(join8(path, "SKILL.md"));
   if (requireLibrary && !broken && latestVersion(name) === null) return { ok: false, reason: "import it into the library first" };
   for (const link2 of links) if (isLink(link2)) stash(link2, "global-backup");
-  return { ok: true, movedTo: stash(path, "global-backup") };
+  const movedTo = stash(path, "global-backup");
+  if (!globalSkillDirs().some((d) => entryExists(join8(d, name)))) setKeepGlobal([name], false);
+  return { ok: true, movedTo };
 }
 function createSkill(name, description) {
   if (!/^[a-z0-9][a-z0-9-]*$/.test(name)) return { ok: false, reason: "use lowercase letters, digits and dashes" };
@@ -1499,6 +1513,28 @@ function link(args) {
     if (res.blocked.length) warn(`${label}git tracks ${res.blocked.join(", ")}; re-run with --allow-tracked to link there too`);
   }
 }
+function globalCommand(args) {
+  const [action, ...names] = args.positional;
+  const yours = machineSkills().filter((m) => m.movable && !m.broken);
+  if (action === "keep" || action === "unkeep") {
+    const unknown = names.filter((n) => !yours.some((m) => m.name === n));
+    if (!names.length || unknown.length) {
+      error(names.length ? `Not one of your global skills: ${unknown.join(", ")}` : `Usage: skilllib global ${action} <name>...`);
+      process.exit(1);
+    }
+    setKeepGlobal(names, action === "keep");
+    success(action === "keep" ? `Keeping ${names.join(", ")} global on purpose` : `${names.join(", ")} will show up in cleanup again`);
+  } else if (action) {
+    error("Usage: skilllib global [keep|unkeep <name>...]");
+    process.exit(1);
+  }
+  const kept = keptGlobal();
+  if (!yours.length) return info("None of your skills load globally.");
+  for (const m of yours) info(`${kept.has(m.name) ? green("\u2713") : yellow("\u26A0")} ${m.name.padEnd(32)} ${dim(tildify(m.path))}`);
+  const unreviewed = yours.filter((m) => !kept.has(m.name)).length;
+  if (unreviewed) info(dim(`
+${unreviewed} not reviewed. Keep one global on purpose: skilllib global keep <name>`));
+}
 
 export {
   userHome,
@@ -1523,6 +1559,8 @@ export {
   harnessesChosen,
   setHarnesses,
   allowTrackedLinks,
+  keptGlobal,
+  setKeepGlobal,
   machineSkills,
   libraryOrigins,
   librarySkills,
@@ -1538,7 +1576,6 @@ export {
   syncProject,
   updateProject,
   importSkill,
-  projectsUsing,
   deleteLibrarySkill,
   deleteGlobal,
   unloadGlobal,
@@ -1569,5 +1606,6 @@ export {
   usage,
   show,
   harnessesCommand,
-  link
+  link,
+  globalCommand
 };

package/dist/index.js

@@ -4,6 +4,7 @@ import {
   doctor,
   error,
   folders,
+  globalCommand,
   harnessesCommand,
   importSkills,
   link,
@@ -21,7 +22,7 @@ import {
   unhide,
   update,
   usage
-} from "./chunk-K5RHLQBH.js";
+} from "./chunk-HLCJQVHB.js";
 
 // src/index.ts
 import { readFileSync } from "fs";
@@ -36,6 +37,7 @@ Set up:
   skilllib harnesses [id...]     Show or set the agents you use (claude-code cursor codex zed)
   skilllib import <dir>...       Add skill folders to the library (--force replaces)
   skilllib import --global       Copy ~/.claude/skills into the library
+  skilllib global [keep|unkeep <name>...]  Your global skills; mark ones you keep global on purpose
 
 In a project:
   skilllib status                This project's skills and their status
@@ -73,9 +75,9 @@ async function main() {
   const args = parseArgs(rest);
   switch (command) {
     case void 0:
-      return process.stdin.isTTY && process.stdout.isTTY ? (await import("./tui-D42E6QPE.js")).tui() : status(args);
+      return process.stdin.isTTY && process.stdout.isTTY ? (await import("./tui-YF7BQ7CD.js")).tui() : status(args);
     case "ui":
-      return (await import("./tui-D42E6QPE.js")).tui();
+      return (await import("./tui-YF7BQ7CD.js")).tui();
     case "status":
       return status(args);
     case "add":
@@ -115,6 +117,8 @@ async function main() {
     case "new":
     case "create":
       return newSkill(args);
+    case "global":
+      return globalCommand(args);
     case "restore":
       return restore(args);
     case "doctor":

package/dist/{tui-D42E6QPE.js → tui-YF7BQ7CD.js}

@@ -17,6 +17,7 @@ import {
   importSkill,
   installDirs,
   isProjectCandidate,
+  keptGlobal,
   latestVersion,
   libraryOrigins,
   librarySkillDir,
@@ -27,7 +28,6 @@ import {
   machineSkills,
   projectOfFactory,
   projectStatus,
-  projectsUsing,
   readConfig,
   readSkillInfo,
   relinkDependency,
@@ -37,6 +37,7 @@ import {
   scanUsage,
   setHarnesses,
   setHidden,
+  setKeepGlobal,
   skilllibHome,
   summarize,
   syncProject,
@@ -47,7 +48,7 @@ import {
   versionHistory,
   visibilityOf,
   visibleProjects
-} from "./chunk-K5RHLQBH.js";
+} from "./chunk-HLCJQVHB.js";
 
 // src/tui/index.tsx
 import { render } from "ink";
@@ -250,7 +251,8 @@ function loadSnapshot() {
     name: s.name,
     description: s.description,
     origin: origins[s.name] ?? inferred(s.name),
-    projects: projectsUsing(s.name).filter((p) => projects.includes(p)),
+    // Managed status entries are the manifest, already read above; no need to re-read every manifest per skill.
+    projects: projects.filter((p) => statusOf(p).some((x) => x.managed && x.name === s.name)).sort(),
     version: latestVersion(s.name)?.version ?? null
   }));
   return {
@@ -263,7 +265,8 @@ function loadSnapshot() {
     backups: listBackups(),
     roots: readConfig().roots,
     harnesses: enabledHarnesses(),
-    harnessesChosen: harnessesChosen()
+    harnessesChosen: harnessesChosen(),
+    keptGlobal: keptGlobal()
   };
 }
 function projectRows(root, library, status = projectStatus(root)) {
@@ -351,6 +354,7 @@ var NEXT_MODE = { move: "keep", keep: "delete", delete: "move" };
 var plural = (n, word) => `${n} ${word}${n === 1 ? "" : "s"}`;
 function CleanupWizard({
   skills,
+  kept,
   projects,
   usesIn,
   usesTotal,
@@ -367,11 +371,19 @@ function CleanupWizard({
   const [step, setStep] = useState("choose");
   const [focus, setFocus] = useState("skills");
   const [modes, setModes] = useState(
-    () => new Map(skills.map((s) => [s.path, !initialSkill || s.name === initialSkill ? "move" : "keep"]))
+    () => new Map(skills.map((s) => [s.path, s.name === initialSkill || !initialSkill && !kept.has(s.name) ? "move" : "keep"]))
   );
   const modeOf = (s) => modes.get(s.path) ?? "keep";
-  const setMode = (s, mode) => setModes((prev) => new Map(prev).set(s.path, mode));
-  const [skillIndex, setSkillIndex] = useState(() => Math.max(0, skills.findIndex((s) => s.name === initialSkill)));
+  const [touched, setTouched] = useState(() => new Set(initialSkill ? [] : skills.map((s) => s.path)));
+  const setMode = (s, mode) => {
+    setModes((prev) => new Map(prev).set(s.path, mode));
+    setTouched((prev) => new Set(prev).add(s.path));
+  };
+  const [showKept, setShowKept] = useState(false);
+  const shown = (on) => skills.filter((s) => on || !kept.has(s.name) || s.name === initialSkill);
+  const list = shown(showKept);
+  const hiddenKept = skills.length - shown(false).length;
+  const [skillIndex, setSkillIndex] = useState(() => Math.max(0, list.findIndex((s) => s.name === initialSkill)));
   const [projectIndex, setProjectIndex] = useState(0);
   const [status, setStatus] = useState("");
   const [targets, setTargets] = useState(
@@ -379,9 +391,10 @@ function CleanupWizard({
       skills.map((s) => [s.path, new Set(projects.filter((p) => installedIn(s.name).includes(p) || (usesIn(s.name, p) ?? 0) > 0))])
     )
   );
-  const skill = skills[skillIndex];
+  const skill = list[skillIndex];
   const chosen = skills.filter((s) => modeOf(s) === "move");
   const toDelete = skills.filter((s) => modeOf(s) === "delete");
+  const toKeep = skills.filter((s) => modeOf(s) === "keep" && touched.has(s.path) && !kept.has(s.name));
   const totalInstalls = chosen.reduce((n, s) => n + (targets.get(s.path)?.size ?? 0), 0);
   const orphans = chosen.filter((s) => (targets.get(s.path)?.size ?? 0) === 0);
   const toggleTarget = (s, project) => setTargets((prev) => {
@@ -417,7 +430,9 @@ function CleanupWizard({
       if (r.ok) unloaded++;
       else problems.push(`${s.name}: ${r.reason}`);
     }
-    onDone({ unloaded, installs, deleted, problems });
+    setKeepGlobal(toKeep.map((s) => s.name), true);
+    setKeepGlobal([...chosen, ...toDelete].map((s) => s.name).filter((n) => kept.has(n)), false);
+    onDone({ unloaded, installs, deleted, kept: toKeep.length, problems });
   };
   inputRef.current = (input, key) => {
     if (key.ctrl && input === "c") return onCancel();
@@ -427,14 +442,19 @@ function CleanupWizard({
       return;
     }
     if (key.escape) return onCancel();
-    if (input === "p") return setStatus(onReviewPrompt(skills));
-    if (key.return) return chosen.length || toDelete.length ? setStep("review") : void 0;
+    if (input === "p") return setStatus(onReviewPrompt(list));
+    if (input === "k" && hiddenKept) {
+      const next = shown(!showKept);
+      setShowKept(!showKept);
+      return setSkillIndex(Math.max(0, next.findIndex((s) => s.path === skill?.path)));
+    }
+    if (key.return) return chosen.length || toDelete.length || toKeep.length ? setStep("review") : void 0;
     if (key.tab || key.rightArrow || key.leftArrow) {
       return setFocus(key.leftArrow ? "skills" : key.rightArrow ? "projects" : focus === "skills" ? "projects" : "skills");
     }
     if (focus === "skills") {
       if (key.upArrow) return setSkillIndex((i) => Math.max(0, i - 1));
-      if (key.downArrow) return setSkillIndex((i) => Math.min(skills.length - 1, i + 1));
+      if (key.downArrow) return setSkillIndex((i) => Math.min(list.length - 1, i + 1));
       if (input === " " && skill) return setMode(skill, NEXT_MODE[modeOf(skill)]);
       if (input === "d" && skill) return setMode(skill, modeOf(skill) === "delete" ? "keep" : "delete");
     } else {
@@ -460,8 +480,9 @@ function CleanupWizard({
       /* @__PURE__ */ jsxs2(Box2, { flexDirection: "column", borderStyle: "round", borderColor: color.accent, paddingX: 2, paddingY: 1, height: bodyHeight + 2, children: [
         /* @__PURE__ */ jsx2(Text2, { color: color.text, children: [
           chosen.length ? `${plural(chosen.length, "skill")} ${chosen.length === 1 ? "moves" : "move"} into your projects (${plural(totalInstalls, "install")})` : "",
-          toDelete.length ? `${plural(toDelete.length, "skill")} ${toDelete.length === 1 ? "is" : "are"} deleted` : ""
-        ].filter(Boolean).join(" \xB7 ") + ". None of them will load in every repo anymore." }),
+          toDelete.length ? `${plural(toDelete.length, "skill")} ${toDelete.length === 1 ? "is" : "are"} deleted` : "",
+          toKeep.length ? `${plural(toKeep.length, "skill")} ${toKeep.length === 1 ? "stays" : "stay"} global on purpose` : ""
+        ].filter(Boolean).join(" \xB7 ") + "." }),
         /* @__PURE__ */ jsx2(Text2, { color: color.faint, children: "Moved skills are copied into Your skills first. Every original goes to a backup you can restore from Health." }),
         /* @__PURE__ */ jsx2(Text2, { children: " " }),
         lines.slice(0, Math.max(1, bodyHeight - 7)).map(({ s, where }) => /* @__PURE__ */ jsxs2(Text2, { wrap: "truncate-end", children: [
@@ -471,6 +492,7 @@ function CleanupWizard({
         ] }, s.path)),
         lines.length > bodyHeight - 7 ? /* @__PURE__ */ jsx2(Text2, { color: color.faint, children: `\u2026 and ${lines.length - (bodyHeight - 7)} more` }) : null,
         toDelete.length ? /* @__PURE__ */ jsx2(Text2, { color: color.red, wrap: "truncate-end", children: `\u2715 delete (not kept in Your skills): ${toDelete.map((s) => s.name).join(", ")}` }) : null,
+        toKeep.length ? /* @__PURE__ */ jsx2(Text2, { color: color.muted, wrap: "truncate-end", children: `\u2713 keep global, and skip next cleanup: ${toKeep.map((s) => s.name).join(", ")}` }) : null,
         /* @__PURE__ */ jsx2(Box2, { flexGrow: 1 }),
         orphans.length ? /* @__PURE__ */ jsx2(Text2, { color: color.yellow, children: `${plural(orphans.length, "skill")} won't be in any project \u2014 add them later from "Add skills".` }) : null
       ] }),
@@ -490,7 +512,7 @@ function CleanupWizard({
   const detailHeight = height >= 26 ? 7 : height >= 20 ? 5 : 0;
   const listHeight = bodyHeight - detailHeight;
   const usesText = (n) => n === null ? "\u2026" : n === 0 ? "unused" : `${n} use${n === 1 ? "" : "s"}`;
-  const skillRows = skills.map((s) => {
+  const skillRows = list.map((s) => {
     const mode = modeOf(s);
     const n = targets.get(s.path)?.size ?? 0;
     return {
@@ -500,7 +522,7 @@ function CleanupWizard({
         { text: s.name, grow: true, color: mode === "keep" ? color.muted : mode === "delete" ? color.red : void 0 },
         { text: usesText(usesTotal(s.name)).padStart(9), width: 9, color: usesTotal(s.name) ? color.text : color.faint },
         {
-          text: mode === "keep" ? "  keep global" : mode === "delete" ? "  delete" : n ? `  \u2192 ${plural(n, "project")}` : "  \u2192 no project",
+          text: mode === "keep" ? kept.has(s.name) ? "  \u2713 kept global" : "  keep global" : mode === "delete" ? "  delete" : n ? `  \u2192 ${plural(n, "project")}` : "  \u2192 no project",
           width: 15,
           color: mode === "keep" ? color.faint : mode === "delete" ? color.red : n ? color.green : color.yellow
         }
@@ -526,19 +548,19 @@ function CleanupWizard({
     ] }),
     /* @__PURE__ */ jsxs2(Box2, { height: 2, paddingX: 1, flexDirection: "column", children: [
       /* @__PURE__ */ jsx2(Text2, { color: color.text, wrap: "truncate-end", children: "These load in every repo, so every agent reads them all the time. Pick the projects that should keep each one." }),
-      /* @__PURE__ */ jsx2(Text2, { color: color.faint, wrap: "truncate-end", children: status ? `\u2713 ${status}` : "\u25C9 move into the ticked projects \xB7 \u25CB keep global \xB7 \u2715 delete. Pre-ticked: projects that have it or used it. p: ask an agent." })
+      /* @__PURE__ */ jsx2(Text2, { color: color.faint, wrap: "truncate-end", children: status ? `\u2713 ${status}` : "\u25C9 move into the ticked projects \xB7 \u25CB keep global (remembered) \xB7 \u2715 delete. Pre-ticked: projects that have it or used it. p: ask an agent." })
     ] }),
     /* @__PURE__ */ jsxs2(Box2, { height: listHeight, children: [
       /* @__PURE__ */ jsx2(
         ListPanel,
         {
-          title: `Global skills \xB7 ${chosen.length} move \xB7 ${toDelete.length} delete \xB7 uses ${usageDays}d`,
+          title: `Global skills \xB7 ${chosen.length} move \xB7 ${toDelete.length} delete${hiddenKept && !showKept ? ` \xB7 ${hiddenKept} kept hidden` : ""} \xB7 uses ${usageDays}d`,
           focused: focus === "skills",
           width: leftWidth,
           height: listHeight,
           rows: skillRows,
           selected: skillIndex,
-          empty: "None of your skills load globally. \u{1F389}"
+          empty: hiddenKept ? `All reviewed: you keep ${plural(hiddenKept, "skill")} global on purpose. k shows them.` : "None of your skills load globally. \u{1F389}"
         }
       ),
       /* @__PURE__ */ jsx2(
@@ -571,6 +593,7 @@ function CleanupWizard({
           ["space", focus === "skills" ? "move / keep / delete" : "tick project"],
           ...focus === "skills" ? [["d", "delete"]] : [],
           ["tab", "switch side"],
+          ...hiddenKept ? [["k", showKept ? "hide kept" : `show kept (${hiddenKept})`]] : [],
           ["enter", `review (${plural(totalInstalls, "install")})`],
           ["p", "ask an agent"],
           ["esc", "cancel"]
@@ -602,7 +625,8 @@ var HELP_TOPICS = [
           "tab           switch lists inside a project",
           "esc           clear the search, then go back to the left pane",
           "ctrl+r        reload        ctrl+c   quit",
-          "p             in the cleanup wizard: copy a prompt asking an agent to review the skills"
+          "p             in the cleanup wizard: copy a prompt asking an agent to review the skills",
+          "k             in the cleanup wizard: show / hide the skills you keep global on purpose"
         ]
       },
       {
@@ -610,7 +634,8 @@ var HELP_TOPICS = [
         lines: [
           "\u273B \u2B21 \u25CE \u2124       Claude Code, Cursor, Codex, Zed \u2014 lit when that agent loads the skill here",
           "\xB2             Cursor reaches it through two folders (it reads several)",
-          "\u26A0 loaded globally   loads in every repo on this machine",
+          "\u26A0 loaded globally   loads in every repo on this machine, and you haven't decided about it yet",
+          "\u2713 global      loads in every repo, and you keep it global on purpose (Enter \u2192 Keep global on purpose)",
           "\u29C9 N copies    the same skill reaches agents from N places \u2014 keep one",
           "\u25CF in sync \xB7 \u2191 update \xB7 \u270E edited \xB7 \u25CB only here",
           "\u2713 committed   in git, so teammates get it      \xB1 changed   committed, with local edits",
@@ -851,6 +876,7 @@ function buildReviewPrompt(skills, ctx) {
       `- Location: ${tildify(s.path)}${s.links.length ? ` (also linked from ${s.links.map(tildify).join(", ")})` : ""}`,
       `- Loaded by: ${s.loadedBy}`,
       ...s.vendor ? [`- Vendor-managed: ${s.vendor}`] : [],
+      ...s.keptGlobal ? ["- I marked this as global on purpose: recommend changing that only for a strong reason (unused, duplicated, low quality)"] : [],
       `- Usage (Claude Code transcripts only): ${usage}`,
       `- Installed in projects: ${s.installedIn.length ? s.installedIn.join(", ") : "none"}`,
       `- Other copies agents can see: ${s.otherCopies.length ? s.otherCopies.join("; ") : "none"}`,
@@ -1163,7 +1189,8 @@ function App() {
   const edit = async (file) => {
     const editor = process.env.VISUAL || process.env.EDITOR || (process.platform === "win32" ? "notepad" : "vi");
     await suspendTerminal(() => {
-      spawnSync2(`${editor} "${file}"`, { shell: true, stdio: "inherit" });
+      if (process.platform === "win32") spawnSync2(`${editor} "${file}"`, { shell: true, stdio: "inherit" });
+      else spawnSync2("/bin/sh", ["-c", `${editor} "$1"`, "sh", file], { stdio: "inherit" });
     });
     reload();
     say(`Saved ${tildify2(file)}`);
@@ -1469,6 +1496,7 @@ function App() {
     const dupes = (name) => copiesOf(name).length;
     const globalItem = (m) => {
       const inLib = libraryNames.has(m.name);
+      const kept = m.movable && snapshot.keptGlobal.has(m.name);
       const alsoLocal = rows.some((r) => r.installed && r.name === m.name);
       const moveHere = () => confirm(
         `Use ${m.name} only where you add it? It's ${inLib ? "" : "imported into the library, "}added to ${where}, and stops loading globally \u2014 other projects lose it until you add it there. Restorable from Health.`,
@@ -1482,6 +1510,7 @@ function App() {
       );
       const actions = m.movable ? [
         { label: "Choose which repos keep it\u2026", hint: "cleanup wizard", run: () => setWizard({ initialSkill: m.name }) },
+        keepAction(m),
         { label: `Use only in ${where}`, hint: "add here, stop loading everywhere", run: moveHere },
         { label: "Delete", hint: "not kept in Your skills \xB7 restorable from Health", run: () => confirmDeleteGlobal([m]) },
         ...inLib ? [] : [
@@ -1510,9 +1539,9 @@ function App() {
             ...harnessChips(enabledIds.map((id) => ({ id, paths: m.harnesses.includes(id) ? 1 : 0 })), true),
             { text: "", width: 8 },
             {
-              text: `  ${dupes(m.name) > 1 ? `\u29C9 ${dupes(m.name)} copies` : m.movable ? "\u26A0 loaded globally" : "vendor"}`,
+              text: `  ${dupes(m.name) > 1 ? `\u29C9 ${dupes(m.name)} copies` : kept ? "\u2713 global" : m.movable ? "\u26A0 loaded globally" : "vendor"}`,
               width: 19,
-              color: dupes(m.name) > 1 ? color.red : m.movable ? color.yellow : color.faint
+              color: dupes(m.name) > 1 ? color.red : kept ? color.green : m.movable ? color.yellow : color.faint
             },
             { text: usesText(m.name, root).padStart(4), color: color.muted }
           ]
@@ -1525,7 +1554,7 @@ function App() {
           meta: [
             `loaded in every project by ${m.harnesses.map(harnessName).join(", ")}`,
             `${m.kind}: ${m.origin}`,
-            m.movable ? "yours: the cleanup wizard can scope it to projects" : "managed by the vendor"
+            kept ? "yours: kept global on purpose" : m.movable ? "yours: the cleanup wizard can scope it to projects" : "managed by the vendor"
           ],
           description: dupes(m.name) > 1 ? `Loaded ${dupes(m.name)} times here: ${copiesOf(m.name).join(" \xB7 ")}. Agents may see it twice \u2014 keep one.` : sourceGroup(m)?.about
         }
@@ -1579,8 +1608,10 @@ function App() {
     const fromLibrary = local.filter((r) => r.managed || r.state === "untracked copy of library skill");
     const repoOwn = local.filter((r) => !fromLibrary.includes(r));
     const yours = globals.filter((m) => m.movable);
+    const keptYours = yours.filter((m) => snapshot.keptGlobal.has(m.name));
+    const unreviewed = yours.filter((m) => !snapshot.keptGlobal.has(m.name));
     const vendor = globals.filter((m) => !m.movable);
-    const cleanupItem = yours.length ? [
+    const cleanupItem = unreviewed.length ? [
       {
         key: "__cleanup",
         search: "clean up global",
@@ -1589,7 +1620,7 @@ function App() {
           key: "__cleanup",
           cells: [
             { text: "\u26A0 ", color: color.yellow },
-            { text: `${plural2(yours.length, "skill")} of yours load in every repo \u2014 choose where each should live\u2026`, grow: true, color: color.yellow }
+            { text: `${plural2(unreviewed.length, "skill")} of yours load in every repo \u2014 choose where each should live\u2026`, grow: true, color: color.yellow }
           ]
         },
         primary: { label: "clean up", run: () => setWizard({}) },
@@ -1633,28 +1664,14 @@ function App() {
           ]
         },
         {
-          title: `\u26A0 Global \xB7 yours, loaded in every repo (${yours.length})`,
-          items: [...cleanupItem, ...yours.map(globalItem)],
-          groupActions: (members) => [
-            {
-              label: `Copy review prompt for all ${members.length}`,
-              hint: "for an agent to decide keep / move / delete",
-              run: () => copyReviewPrompt(snapshot.machine.filter((m) => members.some((x) => x.skill?.path === m.path)).map(reviewOfGlobal), `These are global skills (group ${members[0]?.group?.label ?? ""}).`)
-            },
-            {
-              label: `Copy all ${members.length} to Your skills`,
-              hint: "they keep loading globally",
-              run: () => act(() => {
-                const added = members.map((m) => importSkill(m.skill.path)).filter((r) => r.status === "added").length;
-                return ok(`${plural2(added, "skill")} imported into your library`);
-              })
-            },
-            {
-              label: `Delete all ${members.length}`,
-              hint: "not kept in Your skills \xB7 restorable from Health",
-              run: () => confirmDeleteGlobal(snapshot.machine.filter((m) => members.some((x) => x.skill?.path === m.path)))
-            }
-          ]
+          title: `\u26A0 Global \xB7 yours, not reviewed, loaded in every repo (${unreviewed.length})`,
+          items: [...cleanupItem, ...unreviewed.map(globalItem)],
+          groupActions: (members) => [...keepGroupActions(members), ...yourGlobalActions(members)]
+        },
+        {
+          title: `\u2713 Global \xB7 yours, on purpose (${keptYours.length})`,
+          items: keptYours.map(globalItem),
+          groupActions: (members) => [...keepGroupActions(members), ...yourGlobalActions(members)]
         },
         {
           title: `Global \xB7 from vendors: plugins, claude.ai, built-in (${vendor.length})`,
@@ -1794,6 +1811,8 @@ function App() {
   }
   function globalSections() {
     const yours = snapshot.machine.filter((m) => m.movable);
+    const unreviewed = yours.filter((m) => !m.broken && !snapshot.keptGlobal.has(m.name));
+    const keptCount = yours.filter((m) => !m.broken && snapshot.keptGlobal.has(m.name)).length;
     const importAll = {
       label: "Copy all of yours to Your skills",
       run: () => {
@@ -1832,12 +1851,16 @@ function App() {
         key: "__cleanup",
         cells: [
           { text: "\u26A0 ", color: color.yellow },
-          { text: `Clean up: choose which repos keep each of your ${yours.length} global skills\u2026`, grow: true, color: color.yellow }
+          { text: `Clean up: choose which repos keep each of your ${unreviewed.length} unreviewed global skills\u2026`, grow: true, color: color.yellow }
         ]
       },
       primary: { label: "clean up", run: () => setWizard({}) },
       actions: [{ label: "Open the cleanup wizard", run: () => setWizard({}) }],
-      preview: { title: "Clean up global skills", meta: ["every agent reads global skills in every repo"], description: "Install each skill only where it's needed, then stop loading it everywhere. Originals are kept in backup." }
+      preview: {
+        title: "Clean up global skills",
+        meta: ["every agent reads global skills in every repo", ...keptCount ? [`skips the ${keptCount} you keep global on purpose (k shows them)`] : []],
+        description: "Install each skill only where it's needed, keep it global on purpose, or delete it. Originals are kept in backup."
+      }
     };
     const grouped = groups.map(([title, test]) => ({
       title,
@@ -1848,6 +1871,7 @@ function App() {
           run: () => copyReviewPrompt(snapshot.machine.filter((m) => members.some((x) => x.skill?.path === m.path)).map(reviewOfGlobal), `These are global skills (group ${members[0]?.group?.label ?? ""}).`)
         },
         ...title.startsWith("Yours") ? [
+          ...keepGroupActions(members),
           {
             label: `Delete all ${members.length}`,
             hint: "not kept in Your skills \xB7 restorable from Health",
@@ -1878,7 +1902,7 @@ function App() {
           actions.push({ label: "Remove the broken link", run: unload });
         } else {
           if (!inLib) actions.push({ label: "Copy to Your skills", hint: "keeps loading globally", run: importIt });
-          if (m.movable) actions.push({ label: "Stop loading it globally", hint: "use it per project instead", run: unload });
+          if (m.movable) actions.push(keepAction(m), { label: "Stop loading it globally", hint: "use it per project instead", run: unload });
           primary = m.movable ? { label: "choose repos\u2026", run: () => setWizard({ initialSkill: m.name }) } : !inLib ? { label: "copy to your skills", run: importIt } : void 0;
           if (m.movable) {
             actions.unshift({ label: "Choose which repos keep it\u2026", hint: "cleanup wizard", run: () => setWizard({ initialSkill: m.name }) });
@@ -1900,6 +1924,7 @@ function App() {
               { text: m.name.padEnd(30).slice(0, 30), color: m.broken ? color.red : void 0 },
               { text: m.broken ? "broken link" : m.origin, grow: true, color: m.broken ? color.red : kindColor[m.kind] },
               ...harnessChips(enabledIds.map((id) => ({ id, paths: m.harnesses.includes(id) ? 1 : 0 })), !m.broken),
+              { text: m.movable && snapshot.keptGlobal.has(m.name) ? " \u2713 kept" : "       ", color: color.green },
               { text: inLib ? " \u2713 library" : "          ", color: color.green },
               { text: usesText(m.name).padStart(5), color: color.muted }
             ]
@@ -1912,7 +1937,7 @@ function App() {
             meta: [
               m.harnesses.length ? `loaded everywhere by ${m.harnesses.map(harnessName).join(", ")}` : "not loaded by your harnesses",
               `${m.kind}: ${m.origin}`,
-              m.movable ? "yours to manage" : "managed by the vendor",
+              m.movable ? snapshot.keptGlobal.has(m.name) ? "yours \xB7 kept global on purpose" : "yours to manage" : "managed by the vendor",
               tildify2(m.path) + (m.links.length ? ` (+ ${m.links.length} link${m.links.length > 1 ? "s" : ""})` : "")
             ],
             description: m.broken ? "Broken link: it points at a folder that no longer exists, so it loads nothing." : void 0
@@ -1951,7 +1976,48 @@ function App() {
         description: "The prompt includes each skill's description, a SKILL.md excerpt, how it was installed (vendor, skills.sh, plugin\u2026), which agents load it, usage per project, duplicates and your project list. Paste it into Claude Code, Cursor or Codex; it replies with keep / move / delete per skill."
       }
     };
-    return [{ title: "", items: [...yours.some((m) => !m.broken) ? [launcher] : [], reviewAll] }, ...grouped];
+    return [{ title: "", items: [...unreviewed.length ? [launcher] : [], reviewAll] }, ...grouped];
+  }
+  function markKept(names, keep) {
+    act(() => {
+      setKeepGlobal(names, keep);
+      const label = names.length === 1 ? names[0] : plural2(names.length, "skill");
+      return ok(keep ? `${label} kept global on purpose \u2014 cleanup skips ${names.length === 1 ? "it" : "them"}` : `${label} back in cleanup`);
+    });
+  }
+  function keepAction(m) {
+    return snapshot.keptGlobal.has(m.name) ? { label: "Unmark: not global on purpose", hint: "cleanup asks about it again", run: () => markKept([m.name], false) } : { label: "\u2713 Keep global on purpose", hint: "stops the \u26A0 and skips it in cleanup", run: () => markKept([m.name], true) };
+  }
+  function keepGroupActions(members) {
+    const names = [...new Set(members.flatMap((m) => m.skill ? [m.skill.name] : []))];
+    const unkept = names.filter((n) => !snapshot.keptGlobal.has(n));
+    const kept = names.filter((n) => snapshot.keptGlobal.has(n));
+    return [
+      ...unkept.length ? [{ label: `\u2713 Keep all ${unkept.length} global on purpose`, hint: "cleanup skips them", run: () => markKept(unkept, true) }] : [],
+      ...kept.length ? [{ label: `Unmark all ${kept.length}`, hint: "cleanup asks about them again", run: () => markKept(kept, false) }] : []
+    ];
+  }
+  function yourGlobalActions(members) {
+    return [
+      {
+        label: `Copy review prompt for all ${members.length}`,
+        hint: "for an agent to decide keep / move / delete",
+        run: () => copyReviewPrompt(snapshot.machine.filter((m) => members.some((x) => x.skill?.path === m.path)).map(reviewOfGlobal), `These are global skills (group ${members[0]?.group?.label ?? ""}).`)
+      },
+      {
+        label: `Copy all ${members.length} to Your skills`,
+        hint: "they keep loading globally",
+        run: () => act(() => {
+          const added = members.map((m) => importSkill(m.skill.path)).filter((r) => r.status === "added").length;
+          return ok(`${plural2(added, "skill")} imported into your library`);
+        })
+      },
+      {
+        label: `Delete all ${members.length}`,
+        hint: "not kept in Your skills \xB7 restorable from Health",
+        run: () => confirmDeleteGlobal(snapshot.machine.filter((m) => members.some((x) => x.skill?.path === m.path)))
+      }
+    ];
   }
   function confirmDeleteGlobal(skills) {
     if (skills.length === 0) return;
@@ -2000,7 +2066,8 @@ function App() {
       usesByProject: u.byProject,
       installedIn: projectsWith(m.name),
       otherCopies: snapshot.machine.filter((x) => x.name === m.name && x.path !== m.path).map((x) => `${x.kind} ${x.origin} (${tildify2(x.path)})`),
-      inYourSkills: lib ? `yes, v${lib.version}` : null
+      inYourSkills: lib ? `yes, v${lib.version}` : null,
+      keptGlobal: m.movable && snapshot.keptGlobal.has(m.name)
     };
   }
   function reviewOfLibrary(l) {
@@ -2495,6 +2562,7 @@ function App() {
         CleanupWizard,
         {
           skills: yours,
+          kept: snapshot.keptGlobal,
           projects: snapshot.projects,
           usesIn: (skill, project) => usage ? usage.byProject.get(project)?.get(skill) ?? 0 : null,
           usesTotal: (skill) => usage ? usage.total.get(skill) ?? 0 : null,
@@ -2523,7 +2591,12 @@ function App() {
             reload();
             setToast({
               ok: r.problems.length === 0,
-              text: `${plural2(r.unloaded, "skill")} moved into projects (${plural2(r.installs, "install")})${r.deleted ? ` \xB7 ${r.deleted} deleted` : ""}${r.problems.length ? ` \xB7 ${r.problems.length} kept global: ${r.problems[0]}` : ""}`
+              text: [
+                r.unloaded || r.installs ? `${plural2(r.unloaded, "skill")} moved into projects (${plural2(r.installs, "install")})` : "",
+                r.deleted ? `${r.deleted} deleted` : "",
+                r.kept ? `${plural2(r.kept, "skill")} kept global on purpose` : "",
+                r.problems.length ? `${r.problems.length} left global: ${r.problems[0]}` : ""
+              ].filter(Boolean).join(" \xB7 ")
             });
           }
         }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "skilllib",
-  "version": "1.0.1",
+  "version": "1.1.0",
   "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 and Zed.",
   "keywords": [
     "agent-skills",