skillstogether 0.1.11 → 0.1.12

package/dist/index.js

@@ -2712,7 +2712,8 @@ async function uninstallSkill(organizationSlug, skillSlug, scope, skipConfirm) {
 // src/commands/update.ts
 import * as p9 from "@clack/prompts";
 import { Command as Command8 } from "commander";
-import { writeFileSync as writeFileSync3 } from "fs";
+import { rmSync as rmSync2, writeFileSync as writeFileSync3 } from "fs";
+import { basename as basename2, dirname as dirname3 } from "path";
 import pc10 from "picocolors";
 function generateUpdatedContent(skill, organizationSlug) {
   const frontmatter = generateFrontmatter({
@@ -2730,6 +2731,25 @@ function generateUpdatedContent(skill, organizationSlug) {
   );
   return frontmatter + "\n" + normalizedContent;
 }
+function removeInstalledSkillPath(skillPath) {
+  try {
+    if (basename2(skillPath) === "SKILL.md") {
+      rmSync2(dirname3(skillPath), { recursive: true, force: true });
+    } else {
+      rmSync2(skillPath, { force: true });
+      try {
+        rmSync2(dirname3(skillPath), { recursive: false });
+      } catch {
+      }
+    }
+    return { success: true };
+  } catch (error) {
+    return {
+      success: false,
+      error: formatErrorMessage(error)
+    };
+  }
+}
 var updateCommand = new Command8("update").description("Update installed skills to latest version").argument("[target]", "Organization slug").option("--global", "Update only globally installed skills").option("--project", "Update only project-installed skills").option(
   "--skill <slug>",
   "Update a specific skill by slug (already org-prefixed, e.g. acme-onboarding)"
@@ -2818,6 +2838,7 @@ var updateCommand = new Command8("update").description("Update installed skills
     const checkSpinner = p9.spinner();
     checkSpinner.start("Checking for updates...");
     const skillsToUpdate = [];
+    const skillsToRemove = [];
     const errors = [];
     for (const [org, skills] of byOrg) {
       try {
@@ -2826,6 +2847,7 @@ var updateCommand = new Command8("update").description("Update installed skills
           installedAt: s2.installedAt
         }));
         const result = await checkForUpdates(org, skillsInfo);
+        const remoteSlugs = new Set(result.updates.map((update) => update.slug));
         for (const update of result.updates) {
           if (update.hasUpdate) {
             const unique = skills.find((s2) => s2.slug === update.slug);
@@ -2838,12 +2860,17 @@ var updateCommand = new Command8("update").description("Update installed skills
             }
           }
         }
+        for (const unique of skills) {
+          if (!remoteSlugs.has(unique.slug)) {
+            skillsToRemove.push({ unique });
+          }
+        }
       } catch (error) {
         errors.push(`${org}: ${formatErrorMessage(error)}`);
       }
     }
     checkSpinner.stop("Update check complete");
-    if (skillsToUpdate.length === 0) {
+    if (skillsToUpdate.length === 0 && skillsToRemove.length === 0) {
       p9.log.success("All skills are up to date!");
       if (errors.length > 0) {
         console.log();
@@ -2857,19 +2884,38 @@ var updateCommand = new Command8("update").description("Update installed skills
       p9.outro(pc10.dim("Done"));
       return;
     }
-    const totalInstances = skillsToUpdate.reduce(
+    const totalUpdatedInstances = skillsToUpdate.reduce(
       (sum, s2) => sum + s2.unique.paths.length,
       0
     );
-    console.log();
-    const updateMsg = skillsToUpdate.length === totalInstances ? `${pc10.yellow(skillsToUpdate.length.toString())} skill${skillsToUpdate.length !== 1 ? "s" : ""} will be updated:` : `${pc10.yellow(skillsToUpdate.length.toString())} skill${skillsToUpdate.length !== 1 ? "s" : ""} (${totalInstances} installations) will be updated:`;
-    p9.log.info(updateMsg);
-    for (const { unique, remoteVersion } of skillsToUpdate) {
-      const agentNames = unique.agents.map((a) => a.name).join(", ");
-      const scopeBadge = unique.scope === "global" ? pc10.yellow("(global)") : pc10.dim("(project)");
-      console.log(
-        `  ${pc10.yellow("\u2191")} ${pc10.white(`${unique.organization}/${unique.slug}`)} ${pc10.dim(`v${unique.version || "?"}`)} \u2192 ${pc10.green(`v${remoteVersion}`)} ${pc10.dim(`[${agentNames}]`)} ${scopeBadge}`
-      );
+    const totalRemovedInstances = skillsToRemove.reduce(
+      (sum, s2) => sum + s2.unique.paths.length,
+      0
+    );
+    const totalAffectedInstances = totalUpdatedInstances + totalRemovedInstances;
+    if (skillsToUpdate.length > 0) {
+      console.log();
+      const updateMsg = skillsToUpdate.length === totalUpdatedInstances ? `${pc10.yellow(skillsToUpdate.length.toString())} skill${skillsToUpdate.length !== 1 ? "s" : ""} will be updated:` : `${pc10.yellow(skillsToUpdate.length.toString())} skill${skillsToUpdate.length !== 1 ? "s" : ""} (${totalUpdatedInstances} installations) will be updated:`;
+      p9.log.info(updateMsg);
+      for (const { unique, remoteVersion } of skillsToUpdate) {
+        const agentNames = unique.agents.map((a) => a.name).join(", ");
+        const scopeBadge = unique.scope === "global" ? pc10.yellow("(global)") : pc10.dim("(project)");
+        console.log(
+          `  ${pc10.yellow("\u2191")} ${pc10.white(`${unique.organization}/${unique.slug}`)} ${pc10.dim(`v${unique.version || "?"}`)} \u2192 ${pc10.green(`v${remoteVersion}`)} ${pc10.dim(`[${agentNames}]`)} ${scopeBadge}`
+        );
+      }
+    }
+    if (skillsToRemove.length > 0) {
+      console.log();
+      const removeMsg = skillsToRemove.length === totalRemovedInstances ? `${pc10.red(skillsToRemove.length.toString())} skill${skillsToRemove.length !== 1 ? "s" : ""} deleted from remote will be removed locally:` : `${pc10.red(skillsToRemove.length.toString())} skill${skillsToRemove.length !== 1 ? "s" : ""} deleted from remote will be removed locally (${totalRemovedInstances} installations):`;
+      p9.log.warn(removeMsg);
+      for (const { unique } of skillsToRemove) {
+        const agentNames = unique.agents.map((a) => a.name).join(", ");
+        const scopeBadge = unique.scope === "global" ? pc10.yellow("(global)") : pc10.dim("(project)");
+        console.log(
+          `  ${pc10.red("\u2193")} ${pc10.white(`${unique.organization}/${unique.slug}`)} ${pc10.dim(`[${agentNames}]`)} ${scopeBadge}`
+        );
+      }
     }
     console.log();
     if (options.dryRun) {
@@ -2878,7 +2924,15 @@ var updateCommand = new Command8("update").description("Update installed skills
       return;
     }
     if (!options.yes) {
-      const confirmMsg = skillsToUpdate.length === totalInstances ? `Update ${skillsToUpdate.length} skill${skillsToUpdate.length !== 1 ? "s" : ""}?` : `Update ${skillsToUpdate.length} skill${skillsToUpdate.length !== 1 ? "s" : ""} (${totalInstances} installations)?`;
+      let confirmMsg;
+      if (skillsToUpdate.length > 0 && skillsToRemove.length > 0) {
+        const noun = totalAffectedInstances === 1 ? "installation" : "installations";
+        confirmMsg = `Apply ${skillsToUpdate.length} update${skillsToUpdate.length !== 1 ? "s" : ""} and remove ${skillsToRemove.length} deleted skill${skillsToRemove.length !== 1 ? "s" : ""} (${totalAffectedInstances} ${noun})?`;
+      } else if (skillsToUpdate.length > 0) {
+        confirmMsg = skillsToUpdate.length === totalUpdatedInstances ? `Update ${skillsToUpdate.length} skill${skillsToUpdate.length !== 1 ? "s" : ""}?` : `Update ${skillsToUpdate.length} skill${skillsToUpdate.length !== 1 ? "s" : ""} (${totalUpdatedInstances} installations)?`;
+      } else {
+        confirmMsg = skillsToRemove.length === totalRemovedInstances ? `Remove ${skillsToRemove.length} deleted skill${skillsToRemove.length !== 1 ? "s" : ""} locally?` : `Remove ${skillsToRemove.length} deleted skill${skillsToRemove.length !== 1 ? "s" : ""} locally (${totalRemovedInstances} installations)?`;
+      }
       const confirmed = await p9.confirm({
         message: confirmMsg,
         initialValue: true
@@ -2889,11 +2943,15 @@ var updateCommand = new Command8("update").description("Update installed skills
       }
     }
     const updateSpinner = p9.spinner();
-    updateSpinner.start("Updating skills...");
+    updateSpinner.start("Applying changes...");
     let updatedSkills = 0;
     let updatedInstances = 0;
-    let failed = 0;
+    let updateFailed = 0;
     const updateErrors = [];
+    let removedSkills = 0;
+    let removedInstances = 0;
+    let removalFailed = 0;
+    const removalErrors = [];
     for (const { unique } of skillsToUpdate) {
       try {
         const skill = await fetchSkill(unique.organization, unique.slug);
@@ -2904,25 +2962,56 @@ var updateCommand = new Command8("update").description("Update installed skills
         }
         updatedSkills++;
       } catch (error) {
-        failed++;
+        updateFailed++;
         updateErrors.push(
           `${unique.organization}/${unique.slug}: ${formatErrorMessage(error)}`
         );
       }
     }
-    updateSpinner.stop("Update complete");
+    for (const { unique } of skillsToRemove) {
+      let allInstancesRemoved = true;
+      for (const path of unique.paths) {
+        const result = removeInstalledSkillPath(path);
+        if (result.success) {
+          removedInstances++;
+        } else {
+          allInstancesRemoved = false;
+          removalErrors.push(
+            `${unique.organization}/${unique.slug} (${path}): ${result.error}`
+          );
+        }
+      }
+      if (allInstancesRemoved) {
+        removedSkills++;
+      } else {
+        removalFailed++;
+      }
+    }
+    updateSpinner.stop("Changes applied");
     if (updatedSkills > 0) {
       const summaryMsg = updatedSkills === updatedInstances ? `Updated ${pc10.green(updatedSkills.toString())} skill${updatedSkills !== 1 ? "s" : ""}` : `Updated ${pc10.green(updatedSkills.toString())} skill${updatedSkills !== 1 ? "s" : ""} (${updatedInstances} installations)`;
       p9.log.success(summaryMsg);
     }
-    if (failed > 0) {
+    if (updateFailed > 0) {
       p9.log.error(
-        `Failed to update ${pc10.red(failed.toString())} skill${failed !== 1 ? "s" : ""}`
+        `Failed to update ${pc10.red(updateFailed.toString())} skill${updateFailed !== 1 ? "s" : ""}`
       );
       for (const error of updateErrors) {
         console.log(`  ${pc10.red("\u2022")} ${error}`);
       }
     }
+    if (removedSkills > 0) {
+      const summaryMsg = removedSkills === removedInstances ? `Removed ${pc10.green(removedSkills.toString())} deleted skill${removedSkills !== 1 ? "s" : ""}` : `Removed ${pc10.green(removedSkills.toString())} deleted skill${removedSkills !== 1 ? "s" : ""} (${removedInstances} installations)`;
+      p9.log.success(summaryMsg);
+    }
+    if (removalFailed > 0) {
+      p9.log.error(
+        `Failed to remove ${pc10.red(removalFailed.toString())} deleted skill${removalFailed !== 1 ? "s" : ""}`
+      );
+      for (const error of removalErrors) {
+        console.log(`  ${pc10.red("\u2022")} ${error}`);
+      }
+    }
     p9.outro(pc10.dim("Done"));
   }
 );

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "skillstogether",
-  "version": "0.1.11",
+  "version": "0.1.12",
   "description": "CLI tool to install skills",
   "keywords": [
     "cli",