@light-merlin-dark/skill-sync 0.1.1 → 0.1.3

package/CHANGELOG.md

@@ -1,5 +1,13 @@
 # Changelog
 
+## 0.1.3
+- Makefile release: robust GitHub release notes generation via `--notes-file` and correct escaping of `awk $0`.
+
+## 0.1.2
+- Report installed “orphan” skills during `skill-sync check` (helps explain why a harness skill UI didn’t update)
+- Add `make release` target (pre-publish → npm publish → git tag + push → GitHub release create/update)
+- Minor: manager skill naming support (codex-style capitalized display)
+
 ## 0.1.1
 - Add source-topology diagnostics for duplicate skill slugs in configured project roots
 - Surface resolved duplicates as warnings and unresolved duplicates as blocking errors in `check` and `sources`

package/SKILL.md

@@ -73,6 +73,9 @@ skill-sync harnesses --json
 ## Safety Rules
 
 - Prefer `check` before `sync`.
-- Do not overwrite unmanaged conflicts by hand; inspect why they exist first.
+- If `check` reports a `conflict` due to an existing *unmanaged* install (common case: a skill folder already exists in a harness root like `~/.hermes/skills/<skill>`), resolve by either:
+  - removing the unmanaged directory/file and re-running `sync`, or
+  - restoring via `skill-sync backup restore <backup-id>`.
+  Do not leave mixed symlink + real directories behind.
 - Use `--home` for isolated testing against a fake home directory.
 - Use `--projects-root` when you need to constrain discovery to a specific source tree.

package/dist/index.js

@@ -1288,12 +1288,52 @@ function buildSyncPlan(skills, harnesses, config, state, sourceDiagnostics) {
     }
     harnessPlan.entries.sort((a, b) => a.destinationPath.localeCompare(b.destinationPath));
   }
+  const orphanSkills = [];
+  for (const harnessPlan of harnessPlans) {
+    const desiredSet = desiredByHarness.get(harnessPlan.harness.id) || new Set;
+    let children = [];
+    try {
+      children = readdirSync3(harnessPlan.harness.rootPath);
+    } catch {
+      continue;
+    }
+    for (const child of children) {
+      const destinationPath = join5(harnessPlan.harness.rootPath, child);
+      if (desiredSet.has(destinationPath)) {
+        continue;
+      }
+      if (state.managedEntries[destinationPath]) {
+        continue;
+      }
+      const inspection = inspectEntry(destinationPath);
+      if (!inspection.exists) {
+        continue;
+      }
+      let hasSkillMd = false;
+      if (inspection.type === "directory") {
+        hasSkillMd = existsSync6(join5(destinationPath, "SKILL.md"));
+      } else if (inspection.type === "symlink" && inspection.resolvedTarget) {
+        hasSkillMd = existsSync6(join5(inspection.resolvedTarget, "SKILL.md"));
+      }
+      if (!hasSkillMd) {
+        continue;
+      }
+      orphanSkills.push({
+        harnessId: harnessPlan.harness.id,
+        harnessRoot: harnessPlan.harness.rootPath,
+        installName: child,
+        destinationPath,
+        inspection
+      });
+    }
+  }
   return {
     harnesses: harnessPlans,
     changes,
     conflicts,
     ok,
-    sourceDiagnostics: sourceDiagnostics || { warnings: [], errors: [] }
+    sourceDiagnostics: sourceDiagnostics || { warnings: [], errors: [] },
+    orphanSkills: orphanSkills.length ? orphanSkills : undefined
   };
 }
 function buildPlannedEntry(skill, harness, installName, destinationPath, state, config) {
@@ -1450,6 +1490,16 @@ function print(value, json) {
 function renderPlan(plan) {
   const lines = [];
   appendSourceDiagnostics(lines, plan.sourceDiagnostics);
+  if (plan.orphanSkills && plan.orphanSkills.length > 0) {
+    if (lines.length > 0) {
+      lines.push("");
+    }
+    lines.push("Orphan installed skills:");
+    for (const orphan of plan.orphanSkills) {
+      const resolved = orphan.inspection.type === "symlink" ? orphan.inspection.resolvedTarget || orphan.inspection.linkTarget : undefined;
+      lines.push(`- ${orphan.harnessId}/${orphan.installName}  ${orphan.destinationPath}${resolved ? ` -> ${resolved}` : ""}`);
+    }
+  }
   const counts = countPlanActions(plan);
   if (lines.length > 0) {
     lines.push("");

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@light-merlin-dark/skill-sync",
-  "version": "0.1.1",
+  "version": "0.1.3",
   "description": "Sync local repo-backed agent skills across Codex, Claude Code, Cursor, Gemini, Hermes, and other harnesses.",
   "type": "module",
   "main": "dist/index.js",