library-skills 0.0.11 → 0.0.13

package/README.md

@@ -47,13 +47,19 @@ In JavaScript/TypeScript, you can install them with:
 $ npx library-skills
 ```
 
-This will scan the dependencies for the current project, find the installed libraries, and ask you which of their skills you want to install in the project.
+This will scan the dependencies for the current project, find the installed libraries, and show the current skill installation status.
 
-Then it will add them to the `.agents` directory as symbolic links, so when you update the libraries, the skills are updated too.
+It can install new skills, repair managed symlinks that point to old skill locations, and remove managed symlinks for packages or skills that disappeared.
+
+It will only remove managed symlinks, not hand-authored skill directories.
+
+Then it will ask where to install new skills and add them as symbolic links, so when you update the libraries, the skills are updated too.
+
+By default it selects `.agents/skills`, the agent-neutral target. If the project already has a `.claude/` directory, it selects `.claude/skills` too.
 
 /// tip
 
-If you are using Claude Code, add the `--claude` CLI Option to install the skills in the `.claude/skills` directory too, as Claude Code doesn't support the standard `.agents` directory.
+If you are using Claude Code, select `.claude/skills` when asked for installation targets, as Claude Code doesn't support the standard `.agents` directory. For non-interactive installs, add the `--claude` CLI option to install the skills in `.claude/skills` too.
 
 ///
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "library-skills",
-  "version": "0.0.11",
+  "version": "0.0.13",
   "description": "Library Skills, AI Agents using libraries, as intended, always up to date.",
   "type": "module",
   "exports": {

package/ts/dist/cli.d.ts

@@ -82,6 +82,23 @@ declare function installedStatuses({ targets, skills, }: {
     targets: InstallTarget[];
     skills: Skill[];
 }): InstalledStatus[];
+declare function syncTargetDirs({ projectRoot, includeClaude, yes, check, }: {
+    projectRoot: string;
+    includeClaude?: boolean;
+    yes?: boolean;
+    check?: boolean;
+}): InstallTarget[];
+declare function repairableStatuses(statuses: InstalledStatus[]): InstalledStatus[];
+declare function removableStatuses(statuses: InstalledStatus[]): InstalledStatus[];
+declare function selectStatusesInteractive(statuses: InstalledStatus[], action: "repair" | "remove"): Promise<InstalledStatus[]>;
+declare function repairSelected({ statuses, projectRoot, }: {
+    statuses: InstalledStatus[];
+    projectRoot: string;
+}): number;
+declare function removeSelected({ statuses, projectRoot, }: {
+    statuses: InstalledStatus[];
+    projectRoot: string;
+}): number;
 declare function installSelected({ skills, targets, projectRoot, copy, }: {
     skills: Skill[];
     targets: InstallTarget[];
@@ -100,7 +117,13 @@ declare const testing: {
     installSelected: typeof installSelected;
     installedStatuses: typeof installedStatuses;
     listCommand: typeof listCommand;
+    removableStatuses: typeof removableStatuses;
+    removeSelected: typeof removeSelected;
+    repairSelected: typeof repairSelected;
+    repairableStatuses: typeof repairableStatuses;
     scanCommand: typeof scanCommand;
+    selectStatusesInteractive: typeof selectStatusesInteractive;
+    syncTargetDirs: typeof syncTargetDirs;
     sync: typeof sync;
     topLevelSkills: typeof topLevelSkills;
     displayPath: typeof displayPath;

package/ts/dist/cli.js

@@ -660,6 +660,26 @@ function getTargetDirs(projectRoot, options = {}) {
   }
   return targets;
 }
+function getAllTargetDirs(projectRoot) {
+  return getTargetDirs(projectRoot, { includeClaude: true });
+}
+function getDefaultInstallTargetDirs(projectRoot) {
+  const [universal, claude] = getAllTargetDirs(projectRoot);
+  const selected = [];
+  if (existsSync3(join3(projectRoot, ".agents"))) {
+    selected.push(universal);
+  }
+  if (existsSync3(join3(projectRoot, ".claude"))) {
+    selected.push(claude);
+  }
+  return selected.length > 0 ? selected : [universal];
+}
+function getExistingTargetDirs(projectRoot, options = {}) {
+  if (options.includeClaude) {
+    return getTargetDirs(projectRoot, { includeClaude: true });
+  }
+  return getAllTargetDirs(projectRoot).filter((target) => isDirectory2(target.path));
+}
 function installSkill(skill, targetDir, options = {}) {
   const dest = join3(targetDir, skill.name);
   const source = resolve2(skill.skillDir);
@@ -1118,6 +1138,20 @@ function isDirectory4(path) {
 }
 
 // ts/src/cli.ts
+var ACTION_COLORS = {
+  install: "\x1B[1;32m",
+  repair: "\x1B[1;34m",
+  remove: "\x1B[1;31m",
+  reset: "\x1B[0m"
+};
+function printActionHeader(action) {
+  const labels = {
+    install: "Install new skills",
+    repair: "Repair installed skills",
+    remove: "Remove stale skills"
+  };
+  console.log(`${ACTION_COLORS[action]}${labels[action]}${ACTION_COLORS.reset}`);
+}
 function getProjectContext(cwd = process.cwd()) {
   const workspace = findUvWorkspace(cwd);
   const nodeWorkspace = findNodeWorkspace(cwd);
@@ -1304,6 +1338,19 @@ function findCollisions(skills) {
     [...counts.entries()].filter(([, count]) => count > 1).map(([name]) => name)
   );
 }
+function deduplicateSkills(skills) {
+  const seen = /* @__PURE__ */ new Set();
+  const unique = [];
+  for (const skill of skills) {
+    const key = resolve5(skill.skillDir);
+    if (seen.has(key)) {
+      continue;
+    }
+    seen.add(key);
+    unique.push(skill);
+  }
+  return unique;
+}
 function filterInstallableSkills({
   skills,
   selectedNames,
@@ -1405,6 +1452,7 @@ function printInstalledSkillsTable(statuses, projectRoot) {
   printStatusTable(installed, projectRoot);
 }
 async function selectSkillsInteractive(skills) {
+  printActionHeader("install");
   return checkbox({
     message: "Select skills to install (press Space to select, Enter to confirm):",
     choices: skills.map((skill) => ({
@@ -1413,6 +1461,34 @@ async function selectSkillsInteractive(skills) {
     }))
   });
 }
+async function selectTargetsInteractive({
+  projectRoot,
+  defaultTargets
+}) {
+  const defaultNames = new Set(defaultTargets.map((target) => target.name));
+  return checkbox({
+    message: "Select installation targets (press Space to select, Enter to confirm):",
+    choices: getAllTargetDirs(projectRoot).map((target) => ({
+      name: displayPath(target.path, projectRoot),
+      value: target,
+      checked: defaultNames.has(target.name)
+    })),
+    validate: (selected) => selected.length > 0 || "Please select at least one installation target."
+  });
+}
+async function selectInstallTargets({
+  projectRoot,
+  includeClaude,
+  interactive
+}) {
+  if (!interactive) {
+    return getTargetDirs(projectRoot, { includeClaude });
+  }
+  return selectTargetsInteractive({
+    projectRoot,
+    defaultTargets: getDefaultInstallTargetDirs(projectRoot)
+  });
+}
 async function selectInstalledSkillsInteractive(statuses) {
   const removable = statuses.filter((status) => status.type === "symlink");
   if (removable.length === 0) {
@@ -1426,6 +1502,82 @@ async function selectInstalledSkillsInteractive(statuses) {
     }))
   });
 }
+function syncTargetDirs({
+  projectRoot,
+  includeClaude,
+  yes,
+  check
+}) {
+  const targetsByName = new Map(
+    getExistingTargetDirs(projectRoot).map((target) => [target.name, target])
+  );
+  const defaultTargets = yes || check || includeClaude ? getTargetDirs(projectRoot, { includeClaude }) : getDefaultInstallTargetDirs(projectRoot);
+  for (const target of defaultTargets) {
+    targetsByName.set(target.name, target);
+  }
+  return [...targetsByName.values()];
+}
+function repairableStatuses(statuses) {
+  return statuses.filter(
+    (status) => status.type === "symlink" && (status.status === "broken" || status.status === "outdated") && status.skill !== null
+  );
+}
+function removableStatuses(statuses) {
+  return statuses.filter(
+    (status) => status.type === "symlink" && (status.status === "orphaned" || status.status === "broken" && status.skill === null)
+  );
+}
+async function selectStatusesInteractive(statuses, action) {
+  if (statuses.length === 0) {
+    return [];
+  }
+  printActionHeader(action);
+  return checkbox({
+    message: `Select skills to ${action} (press Space to select, Enter to confirm):`,
+    choices: statuses.map((status) => ({
+      name: `${status.name} [${status.target.name}]`,
+      value: status,
+      checked: true
+    }))
+  });
+}
+function repairSelected({
+  statuses,
+  projectRoot
+}) {
+  let repairedCount = 0;
+  for (const status of statuses) {
+    installSkill(status.skill, status.target.path);
+    console.log(
+      `Repaired: ${status.name} (${status.target.name}) -> ${displayPath(
+        status.path,
+        projectRoot
+      )}`
+    );
+    repairedCount++;
+  }
+  return repairedCount;
+}
+function removeSelected({
+  statuses,
+  projectRoot
+}) {
+  let removedCount = 0;
+  for (const status of statuses) {
+    if (uninstallSkill(status.name, status.target.path)) {
+      console.log(
+        `Removed ${status.status} symlink: ${status.name} (${status.target.name}) -> ${displayPath(
+          status.path,
+          projectRoot
+        )}`
+      );
+      removedCount++;
+    } else {
+      console.log(`Not found: ${status.name} (${status.target.name})`);
+    }
+  }
+  return removedCount;
+}
 function installSelected({
   skills,
   targets,
@@ -1459,8 +1611,11 @@ function installSelected({
 async function sync(options) {
   const context = getProjectContext();
   const result = scanContext(context);
-  const targets = getTargetDirs(context.projectRoot, {
-    includeClaude: options.claude
+  let targets = syncTargetDirs({
+    projectRoot: context.projectRoot,
+    includeClaude: options.claude,
+    yes: options.yes,
+    check: options.check
   });
   printContext(context);
   console.log();
@@ -1499,10 +1654,23 @@ async function sync(options) {
     }
     return;
   }
-  for (const status of drift) {
-    if (options.yes && (status.status === "broken" || status.status === "outdated") && status.skill) {
-      installSkill(status.skill, status.target.path);
-    }
+  const repairable = repairableStatuses(drift);
+  const removable = removableStatuses(drift);
+  if (drift.length > 0) {
+    console.log();
+    console.log("Some installed skills need attention.");
+    console.log("Select the skills to install, repair, or remove.");
+    console.log("Only managed symlinks will be changed.");
+  }
+  const selectedRepairs = options.yes ? repairable : await selectStatusesInteractive(repairable, "repair");
+  const selectedRemovals = options.yes ? removable : await selectStatusesInteractive(removable, "remove");
+  if (selectedRepairs.length > 0) {
+    console.log();
+    repairSelected({ statuses: selectedRepairs, projectRoot: context.projectRoot });
+  }
+  if (selectedRemovals.length > 0) {
+    console.log();
+    removeSelected({ statuses: selectedRemovals, projectRoot: context.projectRoot });
   }
   let selected = filterInstallableSkills({
     skills: candidateSkills,
@@ -1512,10 +1680,19 @@ async function sync(options) {
   if (selected.length === 0 && !options.yes && selectedNames.length === 0 && !options.all) {
     const newSkills = visibleStatuses.filter((status) => status.status === "new" && status.skill !== null).map((status) => status.skill);
     if (newSkills.length > 0) {
-      selected = await selectSkillsInteractive(newSkills);
+      selected = await selectSkillsInteractive(deduplicateSkills(newSkills));
     }
   }
   if (selected.length > 0) {
+    targets = await selectInstallTargets({
+      projectRoot: context.projectRoot,
+      includeClaude: options.claude,
+      interactive: !options.yes && !options.claude
+    });
+    if (targets.length === 0) {
+      console.log("No installation targets selected.");
+      return;
+    }
     console.log();
     const installedCount = installSelected({
       skills: selected,
@@ -1525,6 +1702,9 @@ async function sync(options) {
     console.log();
     console.log(`Installed ${installedCount} skill target(s).`);
   }
+  if (selectedRepairs.length === 0 && selectedRemovals.length === 0 && selected.length === 0) {
+    console.log("No changes needed.");
+  }
 }
 function scanCommand(options) {
   const context = getProjectContext();
@@ -1556,7 +1736,7 @@ function listCommand(options) {
     skills: result.skills,
     includeAll: Boolean(options.all)
   });
-  const targets = getTargetDirs(context.projectRoot, {
+  const targets = getExistingTargetDirs(context.projectRoot, {
     includeClaude: options.claude
   });
   const statuses = installedStatuses({ targets, skills: result.skills });
@@ -1596,9 +1776,6 @@ async function installCommand(options) {
     skills: result.skills,
     includeAll: Boolean(options.all) || selectedNames.length > 0
   });
-  const targets = getTargetDirs(context.projectRoot, {
-    includeClaude: options.claude
-  });
   printWarnings(result.warnings);
   let selected = filterInstallableSkills({
     skills,
@@ -1612,6 +1789,15 @@ async function installCommand(options) {
     console.log("No skills selected.");
     return;
   }
+  const targets = await selectInstallTargets({
+    projectRoot: context.projectRoot,
+    includeClaude: options.claude,
+    interactive: !options.yes && !options.claude
+  });
+  if (targets.length === 0) {
+    console.log("No installation targets selected.");
+    return;
+  }
   const installedCount = installSelected({
     skills: selected,
     targets,
@@ -1624,7 +1810,7 @@ async function installCommand(options) {
 async function removeCommand(skillNames, options) {
   const context = getProjectContext();
   const result = scanContext(context);
-  const targets = getTargetDirs(context.projectRoot, {
+  const targets = getExistingTargetDirs(context.projectRoot, {
     includeClaude: options.claude
   });
   const statuses = installedStatuses({ targets, skills: result.skills });
@@ -1652,7 +1838,7 @@ async function removeCommand(skillNames, options) {
 function createProgram() {
   const program = new Command().enablePositionalOptions();
   program.name("library-skills").description(
-    "Discover and install agent skills from installed library packages."
+    "Discover and reconcile agent skills from installed library packages."
   ).option("--claude", "Also manage .claude/skills alongside .agents/skills").option("-y, --yes", "Skip confirmation prompts").option("--check", "Validate only; exit 1 if installs drift").option("--all", "Install all newly discovered unmanaged skills").option(
     "-s, --skill <name>",
     "Install a specific discovered skill by name",
@@ -1716,7 +1902,13 @@ var testing = {
   installSelected,
   installedStatuses,
   listCommand,
+  removableStatuses,
+  removeSelected,
+  repairSelected,
+  repairableStatuses,
   scanCommand,
+  selectStatusesInteractive,
+  syncTargetDirs,
   sync,
   topLevelSkills,
   displayPath,

package/ts/dist/installer.cjs

@@ -23,6 +23,9 @@ __export(installer_exports, {
   CLAUDE_SKILLS_DIR: () => CLAUDE_SKILLS_DIR,
   InstallError: () => InstallError,
   UNIVERSAL_SKILLS_DIR: () => UNIVERSAL_SKILLS_DIR,
+  getAllTargetDirs: () => getAllTargetDirs,
+  getDefaultInstallTargetDirs: () => getDefaultInstallTargetDirs,
+  getExistingTargetDirs: () => getExistingTargetDirs,
   getTargetDirs: () => getTargetDirs,
   installSkill: () => installSkill,
   listInstalledSkills: () => listInstalledSkills,
@@ -52,6 +55,26 @@ function getTargetDirs(projectRoot, options = {}) {
   }
   return targets;
 }
+function getAllTargetDirs(projectRoot) {
+  return getTargetDirs(projectRoot, { includeClaude: true });
+}
+function getDefaultInstallTargetDirs(projectRoot) {
+  const [universal, claude] = getAllTargetDirs(projectRoot);
+  const selected = [];
+  if ((0, import_node_fs.existsSync)((0, import_node_path.join)(projectRoot, ".agents"))) {
+    selected.push(universal);
+  }
+  if ((0, import_node_fs.existsSync)((0, import_node_path.join)(projectRoot, ".claude"))) {
+    selected.push(claude);
+  }
+  return selected.length > 0 ? selected : [universal];
+}
+function getExistingTargetDirs(projectRoot, options = {}) {
+  if (options.includeClaude) {
+    return getTargetDirs(projectRoot, { includeClaude: true });
+  }
+  return getAllTargetDirs(projectRoot).filter((target) => isDirectory(target.path));
+}
 function installSkill(skill, targetDir, options = {}) {
   const dest = (0, import_node_path.join)(targetDir, skill.name);
   const source = (0, import_node_path.resolve)(skill.skillDir);
@@ -137,6 +160,9 @@ var testing = {
   CLAUDE_SKILLS_DIR,
   InstallError,
   UNIVERSAL_SKILLS_DIR,
+  getAllTargetDirs,
+  getDefaultInstallTargetDirs,
+  getExistingTargetDirs,
   getTargetDirs,
   installSkill,
   listInstalledSkills,

package/ts/dist/installer.d.cts

@@ -19,6 +19,11 @@ declare class InstallError extends Error {
 declare function getTargetDirs(projectRoot: string, options?: {
     includeClaude?: boolean;
 }): InstallTarget[];
+declare function getAllTargetDirs(projectRoot: string): InstallTarget[];
+declare function getDefaultInstallTargetDirs(projectRoot: string): InstallTarget[];
+declare function getExistingTargetDirs(projectRoot: string, options?: {
+    includeClaude?: boolean;
+}): InstallTarget[];
 declare function installSkill(skill: Skill, targetDir: string, options?: {
     copy?: boolean;
 }): string;
@@ -38,4 +43,4 @@ declare const testing: {
     resolveSymlink: typeof resolveSymlink;
 };
 
-export { CLAUDE_SKILLS_DIR, InstallError, type InstallTarget, type InstalledSkill, UNIVERSAL_SKILLS_DIR, getTargetDirs, installSkill, listInstalledSkills, testing, uninstallSkill };
+export { CLAUDE_SKILLS_DIR, InstallError, type InstallTarget, type InstalledSkill, UNIVERSAL_SKILLS_DIR, getAllTargetDirs, getDefaultInstallTargetDirs, getExistingTargetDirs, getTargetDirs, installSkill, listInstalledSkills, testing, uninstallSkill };

package/ts/dist/installer.d.ts

@@ -19,6 +19,11 @@ declare class InstallError extends Error {
 declare function getTargetDirs(projectRoot: string, options?: {
     includeClaude?: boolean;
 }): InstallTarget[];
+declare function getAllTargetDirs(projectRoot: string): InstallTarget[];
+declare function getDefaultInstallTargetDirs(projectRoot: string): InstallTarget[];
+declare function getExistingTargetDirs(projectRoot: string, options?: {
+    includeClaude?: boolean;
+}): InstallTarget[];
 declare function installSkill(skill: Skill, targetDir: string, options?: {
     copy?: boolean;
 }): string;
@@ -38,4 +43,4 @@ declare const testing: {
     resolveSymlink: typeof resolveSymlink;
 };
 
-export { CLAUDE_SKILLS_DIR, InstallError, type InstallTarget, type InstalledSkill, UNIVERSAL_SKILLS_DIR, getTargetDirs, installSkill, listInstalledSkills, testing, uninstallSkill };
+export { CLAUDE_SKILLS_DIR, InstallError, type InstallTarget, type InstalledSkill, UNIVERSAL_SKILLS_DIR, getAllTargetDirs, getDefaultInstallTargetDirs, getExistingTargetDirs, getTargetDirs, installSkill, listInstalledSkills, testing, uninstallSkill };

package/ts/dist/installer.js

@@ -31,6 +31,26 @@ function getTargetDirs(projectRoot, options = {}) {
   }
   return targets;
 }
+function getAllTargetDirs(projectRoot) {
+  return getTargetDirs(projectRoot, { includeClaude: true });
+}
+function getDefaultInstallTargetDirs(projectRoot) {
+  const [universal, claude] = getAllTargetDirs(projectRoot);
+  const selected = [];
+  if (existsSync(join(projectRoot, ".agents"))) {
+    selected.push(universal);
+  }
+  if (existsSync(join(projectRoot, ".claude"))) {
+    selected.push(claude);
+  }
+  return selected.length > 0 ? selected : [universal];
+}
+function getExistingTargetDirs(projectRoot, options = {}) {
+  if (options.includeClaude) {
+    return getTargetDirs(projectRoot, { includeClaude: true });
+  }
+  return getAllTargetDirs(projectRoot).filter((target) => isDirectory(target.path));
+}
 function installSkill(skill, targetDir, options = {}) {
   const dest = join(targetDir, skill.name);
   const source = resolve(skill.skillDir);
@@ -115,6 +135,9 @@ export {
   CLAUDE_SKILLS_DIR,
   InstallError,
   UNIVERSAL_SKILLS_DIR,
+  getAllTargetDirs,
+  getDefaultInstallTargetDirs,
+  getExistingTargetDirs,
   getTargetDirs,
   installSkill,
   listInstalledSkills,