library-skills 0.0.12 → 0.0.14

package/README.md

@@ -47,9 +47,13 @@ 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 ask where to install them and add them 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.
 

package/package.json

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

package/ts/dist/cli.d.ts

@@ -53,6 +53,7 @@ interface GlobalOptions {
     check?: boolean;
     all?: boolean;
     skill?: string[];
+    copy?: boolean;
 }
 interface ListOptions {
     installed?: boolean;
@@ -82,6 +83,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 +118,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

@@ -696,7 +696,14 @@ function installSkill(skill, targetDir, options = {}) {
   if (options.copy) {
     cpSync(source, dest, { recursive: true });
   } else {
-    symlinkSync(getSymlinkTarget({ source, dest }), dest, "dir");
+    try {
+      symlinkSync(getSymlinkTarget({ source, dest }), dest, "dir");
+    } catch (error) {
+      const message = error instanceof Error ? error.message : String(error);
+      throw new InstallError(
+        `Could not create symlink: ${message}. Use --copy if your system does not support symlinks.`
+      );
+    }
   }
   return dest;
 }
@@ -1138,6 +1145,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);
@@ -1438,6 +1459,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) => ({
@@ -1487,6 +1509,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,
@@ -1520,9 +1618,12 @@ function installSelected({
 async function sync(options) {
   const context = getProjectContext();
   const result = scanContext(context);
-  let targets = options.yes || options.check || options.claude ? getTargetDirs(context.projectRoot, {
-    includeClaude: options.claude
-  }) : getDefaultInstallTargetDirs(context.projectRoot);
+  let targets = syncTargetDirs({
+    projectRoot: context.projectRoot,
+    includeClaude: options.claude,
+    yes: options.yes,
+    check: options.check
+  });
   printContext(context);
   console.log();
   printWarnings(result.warnings);
@@ -1560,10 +1661,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,
@@ -1590,11 +1704,15 @@ async function sync(options) {
     const installedCount = installSelected({
       skills: selected,
       targets,
-      projectRoot: context.projectRoot
+      projectRoot: context.projectRoot,
+      copy: options.copy
     });
     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();
@@ -1728,8 +1846,8 @@ 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."
-  ).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(
+    "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("--copy", "Copy files instead of creating symlinks").option(
     "-s, --skill <name>",
     "Install a specific discovered skill by name",
     collect,
@@ -1792,7 +1910,13 @@ var testing = {
   installSelected,
   installedStatuses,
   listCommand,
+  removableStatuses,
+  removeSelected,
+  repairSelected,
+  repairableStatuses,
   scanCommand,
+  selectStatusesInteractive,
+  syncTargetDirs,
   sync,
   topLevelSkills,
   displayPath,

package/ts/dist/installer.cjs

@@ -91,7 +91,14 @@ function installSkill(skill, targetDir, options = {}) {
   if (options.copy) {
     (0, import_node_fs.cpSync)(source, dest, { recursive: true });
   } else {
-    (0, import_node_fs.symlinkSync)(getSymlinkTarget({ source, dest }), dest, "dir");
+    try {
+      (0, import_node_fs.symlinkSync)(getSymlinkTarget({ source, dest }), dest, "dir");
+    } catch (error) {
+      const message = error instanceof Error ? error.message : String(error);
+      throw new InstallError(
+        `Could not create symlink: ${message}. Use --copy if your system does not support symlinks.`
+      );
+    }
   }
   return dest;
 }

package/ts/dist/installer.js

@@ -67,7 +67,14 @@ function installSkill(skill, targetDir, options = {}) {
   if (options.copy) {
     cpSync(source, dest, { recursive: true });
   } else {
-    symlinkSync(getSymlinkTarget({ source, dest }), dest, "dir");
+    try {
+      symlinkSync(getSymlinkTarget({ source, dest }), dest, "dir");
+    } catch (error) {
+      const message = error instanceof Error ? error.message : String(error);
+      throw new InstallError(
+        `Could not create symlink: ${message}. Use --copy if your system does not support symlinks.`
+      );
+    }
   }
   return dest;
 }