projecta-rrr 1.19.0 → 1.19.1

package/CHANGELOG.md

@@ -4,6 +4,12 @@ All notable changes to RRR will be documented in this file.
 
 Format follows [Keep a Changelog](https://keepachangelog.com/en/1.1.0/).
 
+## [1.19.1] - 2026-02-11
+
+### Fixed
+
+- **Install failure** - Postinstall script referenced wrong path for `register-mcp.js` (`rrr/scripts/` instead of `scripts/`)
+
 ## [1.19.0] - 2026-02-05
 
 ### Added

package/package.json

@@ -1,12 +1,12 @@
 {
   "name": "projecta-rrr",
-  "version": "1.19.0",
+  "version": "1.19.1",
   "description": "A meta-prompting, context engineering and spec-driven development system for Claude Code by Projecta.ai",
   "bin": {
     "projecta-rrr": "bin/install.js"
   },
   "scripts": {
-    "postinstall": "node bin/install.js --hud-only --global && node rrr/scripts/register-mcp.js",
+    "postinstall": "node bin/install.js --hud-only --global && node scripts/register-mcp.js",
     "watch": "node watcher/index.js",
     "watch:verbose": "node watcher/index.js --verbose",
     "watch:claude-code": "node watcher/watchers/claude-code.js --verbose",

package/rrr/lib/install-roots.js

@@ -58,33 +58,28 @@ function detectAllCandidateRoots(options = {}) {
     : path.join(os.homedir(), '.claude');
 
   const candidates = [
-    // Current canonical root
+    // Current canonical root (where Claude Code discovers commands)
     {
       path: path.join(claudeDir, 'commands', 'rrr'),
       type: 'canonical',
       canonical: true
     },
+    // Stale: rrr/commands/rrr (inside RRR skill dir or source repo clone)
+    // This is NOT the canonical location - Claude Code does NOT discover commands here
+    {
+      path: path.join(claudeDir, 'rrr', 'commands', 'rrr'),
+      type: 'stale-skill-dir',
+      canonical: false
+    },
     // Legacy: nested rrr/rrr/commands/rrr (from older installer)
     {
       path: path.join(claudeDir, 'rrr', 'rrr', 'commands', 'rrr'),
       type: 'legacy-nested',
       canonical: false
-    },
-    // Legacy: commands/rrr at root level (very old installer)
-    {
-      path: path.join(claudeDir, 'commands', 'rrr'),
-      type: 'legacy-root',
-      canonical: true // Same path as canonical, but conceptually legacy
-    },
+    }
     // NOTE: We intentionally do NOT check ~/.claude/commands/ as a duplicate candidate.
     // That's the PARENT directory containing canonical commands/rrr/. Quarantining it
     // would delete all commands including the canonical ones. See bug fix 2026-01-23.
-    // Project-local install (if config points to project)
-    {
-      path: path.join(claudeDir, 'commands', 'rrr'),
-      type: 'project-local',
-      canonical: false
-    }
   ];
 
   // Filter out duplicates and add actual existence check

package/scripts/doctor-rrr.js

@@ -173,10 +173,13 @@ function findCandidateRoots(baseDir, options = {}) {
           }
         }
 
-        // Continue scanning subdirectories (skip hidden and node_modules)
+        // Continue scanning subdirectories (skip hidden, node_modules, and git repos)
         if (entry.isDirectory() && !entry.name.startsWith('.') && entry.name !== 'node_modules') {
           // Skip scanning into other .claude directories that aren't under our target
           if (entry.name === 'claude' && dir !== baseDir) continue;
+          // Skip directories that are git repos (source repo clones, not installs)
+          // Exception: don't skip the base dir itself
+          if (currentDepth > 0 && fs.existsSync(path.join(fullPath, '.git'))) continue;
           scanDir(fullPath, currentDepth + 1);
         }
       }
@@ -200,10 +203,12 @@ function scoreCandidate(candidate) {
   let versionSource = null;
 
   // A) Canonical priority boost
-  if (rootPath === path.join(CLAUDE_DIR, 'rrr', 'commands', 'rrr')) {
+  // ~/.claude/commands/rrr is the TRUE canonical location (where Claude Code discovers commands)
+  // ~/.claude/rrr/commands/rrr is inside the RRR library/skill dir and should NOT be preferred
+  if (rootPath === path.join(CLAUDE_DIR, 'commands', 'rrr')) {
     score += 1000;
-  } else if (rootPath === path.join(CLAUDE_DIR, 'commands', 'rrr')) {
-    score += 900;
+  } else if (rootPath === path.join(CLAUDE_DIR, 'rrr', 'commands', 'rrr')) {
+    score += 500; // Inside rrr skill dir - likely a source repo clone, NOT the real canonical
   } else if (/\/rrr\/rrr\/commands\/rrr/.test(rootPath)) {
     score += 100;
   }
@@ -261,17 +266,19 @@ function getParentInstallRoot(candidatePath) {
   const normalized = path.normalize(candidatePath);
 
   if (normalized === path.join(CLAUDE_DIR, 'rrr', 'commands', 'rrr')) {
+    // This is inside the RRR skill dir or source repo clone - quarantine just the commands/rrr subdir
     return {
-      quarantinePath: path.join(CLAUDE_DIR, 'rrr'),
-      shouldQuarantineParent: true,
-      description: '~/.claude/rrr'
+      quarantinePath: normalized,
+      shouldQuarantineParent: false,
+      description: '~/.claude/rrr/commands/rrr (stale skill-dir commands)'
     };
   } else if (normalized === path.join(CLAUDE_DIR, 'commands', 'rrr')) {
-    // Don't quarantine ~/.claude itself, just the commands folder
+    // This is the canonical location - should NEVER be quarantined
+    // But if it is being quarantined, only quarantine the rrr subdir, not the parent commands/
     return {
-      quarantinePath: path.join(CLAUDE_DIR, 'commands'),
+      quarantinePath: normalized,
       shouldQuarantineParent: false,
-      description: '~/.claude/commands (commands folder only)'
+      description: '~/.claude/commands/rrr (canonical)'
     };
   } else if (/\/rrr\/rrr\/commands\/rrr/.test(normalized)) {
     return {
@@ -448,8 +455,9 @@ function runDiagnostics() {
   const scored = candidates.map(c => scoreCandidate(c));
 
   // Apply canonical override rule
+  // The TRUE canonical path is ~/.claude/commands/rrr (where Claude Code discovers commands)
   let selected = null;
-  const canonicalPath = path.join(CLAUDE_DIR, 'rrr', 'commands', 'rrr');
+  const canonicalPath = path.join(CLAUDE_DIR, 'commands', 'rrr');
   const canonicalExists = scored.find(c => c.path === canonicalPath && isHealthy(c));
 
   if (canonicalExists) {