@lifeaitools/rdc-skills 0.9.6 → 0.9.8

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@lifeaitools/rdc-skills",
-  "version": "0.9.6",
-  "description": "RDC typed-agent dispatch skill suite for Claude Code — plan, build, review, overnight builds",
+  "version": "0.9.8",
+  "description": "RDC typed-agent dispatch skill suite for Claude Code \u00e2\u20ac\u201d plan, build, review, overnight builds",
   "keywords": [
     "claude-code",
     "claude-code-plugin",

package/scripts/install-rdc-skills.js

@@ -120,6 +120,62 @@ function buildPluginCache(cacheDir, version, gitSha) {
   }
 }
 
+// ── User-skills cleanup ───────────────────────────────────────────────────────
+// Older installer versions wrote skill files directly to ~/.claude/skills/user/.
+// Claude Code loads that directory AND the plugin cache, so any rdc skills left
+// there produce duplicate registrations and break the resolver.
+// This function nukes any entry whose frontmatter name starts with "rdc:".
+function cleanUserSkills(userSkillsDir) {
+  if (!fs.existsSync(userSkillsDir)) return 0;
+  let removed = 0;
+  for (const entry of fs.readdirSync(userSkillsDir, { withFileTypes: true })) {
+    const candidate = path.join(userSkillsDir, entry.name);
+    let skillFile = null;
+    if (entry.isDirectory()) {
+      // New format: <name>/SKILL.md or <name>/skill.md
+      for (const sf of ['SKILL.md', 'skill.md']) {
+        const p = path.join(candidate, sf);
+        if (fs.existsSync(p)) { skillFile = p; break; }
+      }
+    } else if (entry.isFile() && entry.name.endsWith('.md') && entry.name !== 'skill.md' && entry.name !== 'SKILL.md' && entry.name !== 'README.md') {
+      // Old format: rdc-deploy.md flat file
+      skillFile = candidate;
+    }
+    if (!skillFile) continue;
+    const fm = readFrontmatter(skillFile);
+    if (fm.name && fm.name.startsWith('rdc:')) {
+      try {
+        if (entry.isDirectory()) fs.rmSync(candidate, { recursive: true, force: true });
+        else fs.unlinkSync(candidate);
+        removed++;
+      } catch {}
+    }
+  }
+  return removed;
+}
+
+// ── Stale hook cleanup ────────────────────────────────────────────────────────
+// Remove ONLY explicitly orphaned hook files — hooks that were previously shipped
+// by rdc-skills and have since been removed from the project.
+// NEVER use "not in source = remove" logic: most hooks in ~/.claude/hooks/ are
+// not managed by rdc-skills (they come from other plugins or were written directly).
+function cleanStaleHooks(hooksDstDir) {
+  if (!fs.existsSync(hooksDstDir)) return 0;
+  // Explicit orphan list — add entries here when a hook is intentionally removed.
+  // Format: filename that should be deleted if it still exists.
+  const ORPHANED_HOOKS = [
+    'verify-rdc-skills.js', // removed in v0.9.7 — was checking for old flat-file format
+  ];
+  let removed = 0;
+  for (const f of ORPHANED_HOOKS) {
+    const p = path.join(hooksDstDir, f);
+    if (fs.existsSync(p)) {
+      try { fs.unlinkSync(p); removed++; } catch {}
+    }
+  }
+  return removed;
+}
+
 // ── Cache flush helper ────────────────────────────────────────────────────────
 function flushOldCaches(cacheBase, keepVersion) {
   if (!fs.existsSync(cacheBase)) return 0;
@@ -480,6 +536,20 @@ async function main() {
     warn('[0/6] git pull failed — installing from local copy');
   }
 
+  // 0.5a. User-skills cleanup — remove any rdc: skills from ~/.claude/skills/user/
+  // (older installer versions wrote there; plugin cache is the only authoritative source)
+  {
+    const userSkillsDir = path.join(claudeHome, 'skills', 'user');
+    const purged = cleanUserSkills(userSkillsDir);
+    if (purged > 0) ok(`[0.5a] Skills cleanup — removed ${purged} stale rdc: skill(s) from skills/user/`);
+  }
+
+  // 0.5b. Stale hook cleanup — remove hooks we no longer ship
+  {
+    const staleRemoved = cleanStaleHooks(hooksDst);
+    if (staleRemoved > 0) ok(`[0.5b] Hook cleanup — removed ${staleRemoved} orphaned hook file(s)`);
+  }
+
   // 0.5. Legacy cleanup — remove old commands/rdc/ (pre-plugin-system format)
   const legacyCommandsDir = path.join(claudeHome, 'commands', 'rdc');
   if (fs.existsSync(legacyCommandsDir)) {