refacil-sdd-ai 5.2.1 → 5.2.3

package/bin/cli.js

@@ -382,6 +382,39 @@ function checkUpdate() {
 
   cleanLegacySettingsHooks(projectRoot);
 
+  // Self-healing: if selected-ides.json exists but global skills/hooks are missing, restore them
+  // Covers the case where the Claude Code desktop app overwrites settings.json and wipes SDD hooks,
+  // or where skills were deleted by any means while the package is still installed.
+  try {
+    const home = os.homedir();
+    const sddSelectedIDEs = readSelectedIDEs(home);
+    if (sddSelectedIDEs && sddSelectedIDEs.length > 0) {
+      const dirMap = {
+        '.claude': globalClaudeDir(home),
+        '.cursor': globalCursorDir(home),
+        '.opencode': globalOpenCodeDir(home),
+        '.codex': globalCodexDir(home),
+      };
+      const missingSkills = sddSelectedIDEs.some(
+        (ide) => dirMap[ide] && !fs.existsSync(path.join(dirMap[ide], 'skills')),
+      );
+      if (missingSkills) {
+        installSkills(packageRoot, home, sddSelectedIDEs);
+        installAgents(packageRoot, home, sddSelectedIDEs);
+        writeGlobalVersion(getPackageVersion(packageRoot));
+        process.stdout.write('[refacil-sdd-ai] Self-healed: global skills and agents restored.\n');
+      }
+      // Always verify hooks are present for all selected IDEs — idempotent, only writes if missing
+      for (const ide of ['.claude', '.cursor', '.opencode', '.codex']) {
+        if (sddSelectedIDEs.includes(ide)) {
+          installHooks(ide, home, projectRoot);
+        }
+      }
+    }
+  } catch (_) {
+    // Tolerant — self-healing must never break session startup
+  }
+
   // Step 1: update the global package if a newer version is available on npm
   try {
     const latest = execSync('npm view refacil-sdd-ai version', {

package/bin/postinstall.js

@@ -0,0 +1,20 @@
+#!/usr/bin/env node
+'use strict';
+
+const fs = require('fs');
+const path = require('path');
+const os = require('os');
+
+// Only auto-reinstall if a previous `init` was done — indicated by selected-ides.json
+const selPath = path.join(os.homedir(), '.refacil-sdd-ai', 'selected-ides.json');
+if (!fs.existsSync(selPath)) process.exit(0);
+
+const { execFileSync } = require('child_process');
+try {
+  execFileSync(process.execPath, [path.join(__dirname, 'cli.js'), 'update'], {
+    stdio: 'inherit',
+    timeout: 120000,
+  });
+} catch (_) {
+  // Non-fatal: postinstall must never break npm install
+}

package/lib/hooks.js

@@ -326,6 +326,12 @@ function uninstallOpenCodePlugin(homeDir) {
  * @param {string} projectRoot
  */
 function removeProjectLevelHooks(projectRoot) {
+  // Safety guard: never strip hooks from the global ~/.claude/settings.json.
+  // If projectRoot is the home directory (findProjectRoot() fallback), skip entirely.
+  const resolvedRoot = require('path').resolve(projectRoot);
+  const resolvedHome = require('path').resolve(require('os').homedir());
+  if (resolvedRoot === resolvedHome) return;
+
   const sddMarkers = ['_sdd', '_sdd_compact', '_sdd_review', '_sdd_notify'];
 
   // .claude/settings.json

package/lib/installer.js

@@ -300,6 +300,13 @@ function installAgents(packageRoot, homeDir, ideDirs) {
  * @param {string} projectRoot
  */
 function removeProjectLevelArtifacts(projectRoot) {
+  // Safety guard: never remove artifacts from the home directory itself.
+  // If projectRoot resolves to home (e.g. findProjectRoot() fallback when cwd is ~),
+  // projectRoot/.claude/ would be ~/ .claude/ — i.e. the global installation dirs.
+  const resolvedRoot = path.resolve(projectRoot);
+  const resolvedHome = path.resolve(os.homedir());
+  if (resolvedRoot === resolvedHome) return 0;
+
   const ideDirs = ['.claude', '.cursor', '.opencode'];
   const subDirs = ['skills', 'agents'];
   let removed = 0;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "refacil-sdd-ai",
-  "version": "5.2.1",
+  "version": "5.2.3",
   "description": "SDD-AI: Specification-Driven Development with AI — development methodology using AI with Claude Code, Cursor, OpenCode and Codex",
   "bin": {
     "refacil-sdd-ai": "./bin/cli.js"
@@ -38,6 +38,7 @@
     "node": ">=20.0.0"
   },
   "scripts": {
+    "postinstall": "node ./bin/postinstall.js",
     "test": "node --test test/hooks.test.js test/installer.test.js test/ignore-files.test.js test/methodology-migration-pending.test.js test/sdd.test.js test/config.test.js test/refactor-integrar-openspec-nativo.test.js test/refactor-rutas-refacil-sdd.test.js test/refactor-agents-english.test.js test/remove-openspec-legacy.test.js test/find-project-root.test.js test/opencode-installer.test.js test/opencode-plugin.test.js test/toml-converter.test.js test/testing-policy-sync.test.js test/session-repo-sync.test.js"
   },
   "dependencies": {

package/skills/verify/SKILL.md

@@ -37,16 +37,6 @@ If you already have a `changeName`, run `refacil-sdd-ai sdd status <changeName>
 
 If **this session** inspects the change directory before or after delegating, apply **`refacil-prereqs/METHODOLOGY-CONTRACT.md` §8**.
 
-### Step 0.6: Verify validator agent is installed (blocking — CA-12)
-
-Before doing anything else, check that `.claude/agents/refacil-validator.md` exists (read by explicit path or `ls -la .claude/agents/refacil-validator.md`).
-
-**If the file does NOT exist**, stop immediately:
-```
-El agente `refacil-validator` no está instalado. Ejecuta `/refacil:update` y reinicia la sesión antes de volver a correr `/refacil:verify`.
-```
-Do not continue and do not escalate to any other agent.
-
 ### Step 1: Build briefing for the sub-agent (reduces validator tool calls)
 
 Before invoking the sub-agent, extract the context that the validator would otherwise calculate on its own:
@@ -107,7 +97,8 @@ The sub-agent:
 
 Show the user the **combined report** (everything before the `refacil-verify-result` block). Do not show the JSON block — it is internal metadata.
 
-**If the sub-agent failed to load** (tool error, agent type not found, or no response at all): stop immediately and do NOT escalate to any other agent:
+**If the sub-agent failed to load** (tool error, agent type not found, or no response at all): stop immediately and do NOT escalate to any other agent. If the failure is due to a missing install, `refacil-sdd-ai update` (or `init`) + restart the session — same as other skills that delegate to sub-agents.
+
 ```
 The validator sub-agent could not be loaded — retry or run `/refacil:verify` again.
 ```
@@ -173,5 +164,4 @@ Do you want me to apply these corrections? (yes/no)
 - **Sub-agent failsafe (CA-01)**: if the validator fails to load (tool error) or returns no response — stop and inform the user. Do NOT escalate to any other agent.
 - **Unstructured output (CA-02)**: if the validator responds but without a `refacil-verify-result` block — show the raw report and stop. Do NOT re-invoke another agent.
 - **SCOPE_ERROR (CR-03)**: if the validator returns `SCOPE_ERROR: <reason>` — propagate and ask for clarification. CA-01 does NOT apply here.
-- **Agent missing (CA-12)**: checked in Step 0.6 — stop before delegating if `.claude/agents/refacil-validator.md` is absent.
 - **Flow continuity**: if the result is APPROVED and the user confirms affirmatively, immediately invoke the **Skill tool** with `skill: "refacil:review"`. (See `METHODOLOGY-CONTRACT.md §5`.)