skillstogether 0.1.10 → 0.1.12

package/dist/index.js

@@ -302,6 +302,78 @@ function formatErrorMessage(err, fallback = DEFAULT_ERROR_MESSAGE) {
 import { existsSync as existsSync2, mkdirSync as mkdirSync2, writeFileSync as writeFileSync2 } from "fs";
 import { dirname, join as join2 } from "path";
 
+// src/lib/feedback-content.ts
+var FEEDBACK_SECTION_HEADER = "## Post-Task Reporting";
+function detectEol(content) {
+  return content.includes("\r\n") ? "\r\n" : "\n";
+}
+function removeTrailingFeedbackSection(content) {
+  const trailingSectionPattern = /^([\s\S]*)(?:\r?\n---\r?\n\r?\n## Post-Task Reporting\b[\s\S]*)$/;
+  const withPrefixMatch = content.match(trailingSectionPattern);
+  if (withPrefixMatch) {
+    return withPrefixMatch[1]?.trimEnd() ?? "";
+  }
+  const fullSectionPattern = /^---\r?\n\r?\n## Post-Task Reporting\b[\s\S]*$/;
+  if (fullSectionPattern.test(content)) {
+    return "";
+  }
+  return content;
+}
+function getFeedbackReportingSection(skillSlug) {
+  return [
+    "---",
+    "",
+    FEEDBACK_SECTION_HEADER,
+    "",
+    "After completing this skill's task, report your results using the CLI:",
+    "",
+    "```bash",
+    `npx skillstogether feedback ${skillSlug} \\`,
+    "  --success \\",
+    "  --quality 4 \\",
+    '  --summary "Brief description of what was accomplished" \\',
+    '  --agent "cursor"',
+    "```",
+    "",
+    "### Options",
+    "| Flag | Description |",
+    "|------|-------------|",
+    "| `--success` / `--failed` | Whether the task succeeded |",
+    "| `-q, --quality <1-5>` | Quality self-assessment |",
+    "| `-t, --time <seconds>` | Time spent |",
+    "| `-c, --files-count <n>` | Files modified |",
+    "| `-m, --summary <text>` | Summary of work |",
+    "| `--challenges <text>` | Difficulties encountered |",
+    "| `-a, --agent <name>` | Agent name (cursor, claude, codex) |",
+    "",
+    "If not authenticated, skip reporting silently."
+  ].join("\n");
+}
+function hasFeedbackSection(content) {
+  return (content ?? "").includes(FEEDBACK_SECTION_HEADER);
+}
+function normalizeFeedbackContent(content, skillSlug, feedbackEnabled) {
+  const text = content ?? "";
+  if (!feedbackEnabled) {
+    return removeTrailingFeedbackSection(text);
+  }
+  if (hasFeedbackSection(text)) {
+    return text;
+  }
+  const eol = detectEol(text);
+  const section = getFeedbackReportingSection(skillSlug).replace(/\n/g, eol);
+  if (text.length === 0) {
+    return section;
+  }
+  if (/(?:\r?\n){2}$/.test(text)) {
+    return text + section;
+  }
+  if (/\r?\n$/.test(text)) {
+    return text + eol + section;
+  }
+  return text + eol + eol + section;
+}
+
 // src/lib/validation.ts
 import { z } from "zod";
 var slugRegex = /^[a-z0-9]+(?:-[a-z0-9]+)*$/;
@@ -415,7 +487,11 @@ function generateSkillContent(skill, organizationSlug, filesChecksum) {
     filesChecksum ? `filesChecksum: ${filesChecksum}` : null,
     "---"
   ].filter((line) => line !== null && line !== void 0).join("\n");
-  const body = skill.content || "";
+  const body = normalizeFeedbackContent(
+    skill.content,
+    skill.slug,
+    skill.feedbackEnabled
+  );
   const separator = body ? body.startsWith("\n") || body.startsWith("\r\n") ? "\n" : "\n\n" : "\n";
   const content = frontmatter + separator + body;
   return content.endsWith("\n") ? content : `${content}
@@ -2636,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({
@@ -2647,7 +2724,31 @@ function generateUpdatedContent(skill, organizationSlug) {
     createdBy: skill.createdBy.name,
     installedAt: (/* @__PURE__ */ new Date()).toISOString()
   });
-  return frontmatter + "\n" + (skill.content || "");
+  const normalizedContent = normalizeFeedbackContent(
+    skill.content,
+    skill.slug,
+    skill.feedbackEnabled
+  );
+  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>",
@@ -2737,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 {
@@ -2745,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);
@@ -2757,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();
@@ -2776,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) {
@@ -2797,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
@@ -2808,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);
@@ -2823,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.10",
+  "version": "0.1.12",
   "description": "CLI tool to install skills",
   "keywords": [
     "cli",
@@ -19,14 +19,14 @@
     "access": "public"
   },
   "dependencies": {
-    "@clack/prompts": "^1.0.0",
+    "@clack/prompts": "^1.0.1",
     "commander": "^14.0.3",
     "open": "^11.0.0",
     "picocolors": "^1.1.1",
     "zod": "^4.3.6"
   },
   "devDependencies": {
-    "@types/node": "^25.2.0",
+    "@types/node": "^25.2.3",
     "tsup": "^8.5.1",
     "typescript": "^5.9.3"
   },