@hanzlaa/rcode 4.3.3 → 4.4.0

package/CONTRIBUTING.md

@@ -521,7 +521,7 @@ only surface at install time.
 ### For dashboard changes
 1. `node server/dashboard.js` starts cleanly
 2. Visit `http://localhost:7717` — all sections render
-3. Stop with `kill $(lsof -t -i:7717)`
+3. Stop with `kill $(ss -ltnp 'sport = :7717' | awk 'NR>1{match($6,/pid=([0-9]+)/,m); print m[1]}')`
 
 ### For new workflows
 1. Read the workflow instructions end-to-end as if executing it

package/README.md

@@ -2,11 +2,15 @@
 
 <div dir="rtl">طريقة رحال</div>
 
-<p align="center"><img src="brand/hero-receipt.png" alt="rcode benchmark output: 497 tests, 0 failing · 45 agents, 116 commands · 1 runtime dependency" width="760"></p>
+<p align="center"><img src="brand/dashboard-tour.gif" alt="rcode Diwan dashboard tour — cycling through the project cockpit (progress donut, current phase executing, milestone outlook), the specialist agent roster, the Kanban board (TODO / in-progress / done), and the phase-driven delivery view" width="860"></p>
 
-### 497 tests. Zero failing. Read the script.
+### An engineering org in a folder — and you can watch it work.
 
-*An engineering org in a folder — and every number below is reproducible with one command (`node benchmarks/facts.cjs`). Marketing rounds up; this repo ships the receipts.*
+*A roster of specialist agents, phase-driven delivery, and a persistent Memory Bank — running as plain files your IDE reads. Every number is reproducible with one command (`node benchmarks/facts.cjs`); marketing rounds up, this repo ships the receipts: **497 tests, 0 failing · 45 agents · 116 commands · 1 runtime dependency**.*
+
+<p align="center"><img src="brand/dashboard-team.png" alt="rcode agent roster grouped by function — Leadership (Sadiq / strategy, Waleed / CTO, Ahmed / delivery, Nasser / eng manager), Engineering (Omar, Haitham / frontend, Yousef / backend, Zayd / ML, Khalid / DevOps), Product (Hussain / PM, Mariam / marketing), each card showing its role and tools" width="860"></p>
+
+<p align="center"><sub>The <a href="docs/install.md">Diwan dashboard</a> (view-only, <code>node server/dashboard.js</code>) — your project's phases, sprints, agents, decisions, and live orchestration, read straight from the files in <code>.rcode/</code>.</sub></p>
 
 > **A curated composition of the best AI-development practices — shipped as files.** Surgical-change discipline, parallel-agent orchestration, persistent Memory Bank patterns, phase-driven planning — packaged as one workflow for Claude Code power users. No multi-agent harness. No vector DB. Your IDE keeps the methodology; the project keeps the memory.
 
@@ -123,6 +127,8 @@ The point isn't "I beat LangChain." The point is **you don't need LangChain for
 > node benchmarks/facts.cjs
 > ```
 
+<p align="center"><img src="brand/hero-receipt.png" alt="benchmarks/facts.cjs output — 497 tests, 0 failing · 45 agents · 116 commands · 1 runtime dependency, computed from files on disk" width="720"></p>
+
 | Metric | Value | Why it's not a vanity number |
 |---|---|---|
 | **Portable methodology corpus** | **94,481 lines** of markdown | A 10-person eng org's playbooks (agents + commands + workflows + skills + references) — as files you own and grep, not a SaaS you rent. |
@@ -146,7 +152,7 @@ pnpm dlx @hanzlaa/rcode install
 /rcode-init
 ```
 
-> **Don't have pnpm?** On Node 18+ / npm 11.x we recommend pnpm to avoid `npx` cache issues:
+> **Don't have pnpm?** We recommend pnpm to avoid peer-dependency resolution issues on npm 11.x and `npx` cache issues:
 > `npm install -g pnpm`
 
 `/rcode-init` detects your project state (fresh / existing / returning) and routes to the right first action. For a greenfield project it auto-routes to `/rcode-new-project`.

package/cli/doctor.js

@@ -67,10 +67,17 @@ function checkCompliance(filePath) {
   if (!/^## Examples/m.test(content)) missing.push('Examples');
 
   // Negative-boundary clause (component 1 of the 5-component standard).
-  // Parse the frontmatter description so folded-block YAML is normalized
-  // before the regex runs — "Do\n  NOT" becomes "Do NOT" after normalization.
+  // Three valid locations, checked in order:
+  //   a) frontmatter `not-for:` array with at least one entry (canonical YAML form)
+  //   b) a boundary phrase in the frontmatter `description` field
+  //   c) a "## Do NOT use" heading or "do not use/include" in the body
+  // Bug #874: only (b) and (c) were checked, missing skills that use (a).
+  const notForField = frontmatter['not-for'];
+  const hasNotForField =
+    Array.isArray(notForField) && notForField.length > 0;
   const desc = typeof frontmatter.description === 'string' ? frontmatter.description : '';
   const hasBoundary =
+    hasNotForField ||
     NEGATIVE_BOUNDARY_RE.test(desc) ||
     /##[^\n]*\bnot\b/i.test(body) ||
     /\bdo not (use|include)\b/i.test(body);
@@ -188,7 +195,18 @@ function runPreflight(cwd, packageRoot) {
   // 6. Agent manifest drift (only if .rcode/ is initialized — indicates installed editors)
   if (fs.existsSync(rcodeDir)) {
     const editors = [];
-    if (fs.existsSync(path.join(cwd, '.claude/skills'))) editors.push('claude');
+    // Detect Claude install via flat .claude/commands/rcode-*.md (installer writes
+    // slash commands flat, not under a subdirectory) or .claude/skills as a fallback
+    // for legacy layouts. Bug #873: checking only .claude/skills missed fresh installs.
+    const claudeCommandsDir = path.join(cwd, '.claude/commands');
+    const hasClaudeFlatCommands =
+      fs.existsSync(claudeCommandsDir) &&
+      fs.readdirSync(claudeCommandsDir).some(
+        (f) => f.startsWith('rcode-') && f.endsWith('.md'),
+      );
+    if (hasClaudeFlatCommands || fs.existsSync(path.join(cwd, '.claude/skills'))) {
+      editors.push('claude');
+    }
     if (fs.existsSync(path.join(cwd, '.cursor/rules'))) editors.push('cursor');
     if (fs.existsSync(path.join(cwd, '.windsurf/rules'))) editors.push('windsurf');
     if (fs.existsSync(path.join(cwd, '.antigravity/agents'))) editors.push('antigravity');

package/cli/index.js

@@ -64,6 +64,10 @@ Usage:
   init           Alias for install
   update         Refresh skill files (backs up .rcode/ state first)
   uninstall      Remove rcode from the current project
+                 --yes      Skip confirmation prompts
+                 --purge    Remove everything: .rcode/ state, .planning/ scaffolds,
+                            .claude/agents/rules/, .cursor/ rcode files,
+                            .gitignore block, and .git/hooks/pre-commit
   remove         Alias for uninstall
   nuke           Wipe ALL rcode/rcode installs everywhere (global packages,
                  binaries, ~/.claude/* rcode artifacts, ~/.rcode/, project artifacts)
@@ -116,7 +120,7 @@ Getting started:
   rcode tiers        # see the Golden Path
   rcode set-profile  # choose model profile (quality | balanced | budget)
 
-Documentation: https://github.com/hanzlahabib/rihal-code
+Documentation: https://github.com/hanzlahabib/rcode
   `.trim());
 }
 

package/cli/install.js

@@ -370,7 +370,7 @@ function printInstallHeader(targetVersion) {
     pc.cyan('│') + '   ' + dim('A persistent context-brain for your editor') + '             ' + pc.cyan('│'),
     pc.cyan('│') + '                                                           ' + pc.cyan('│'),
     pc.cyan('│') + '   ' + dim('version  ') + pc.green('v' + v) + '                                          ' + pc.cyan('│'),
-    pc.cyan('│') + '   ' + dim('docs     ') + 'github.com/hanzlahabib/rihal-code               ' + pc.cyan('│'),
+    pc.cyan('│') + '   ' + dim('docs     ') + 'github.com/hanzlahabib/rcode               ' + pc.cyan('│'),
     pc.cyan('│') + '   ' + dim('by       ') + 'Hanzla Habib' + '                                    ' + pc.cyan('│'),
     pc.cyan('│') + '                                                           ' + pc.cyan('│'),
     pc.cyan('╰───────────────────────────────────────────────────────────╯'),
@@ -726,7 +726,7 @@ function seedStarterPlanning(target, projectName) {
   // planning artifacts. See issues #670 #671 #676.
   const STUB_BANNER =
     `<!-- INSTALL STUB — overwritten by /rcode-new-project. Delete this file or run\n` +
-    `     /rcode-new-project before committing. See https://github.com/hanzlahabib/rihal-code/issues/670 -->\n\n`;
+    `     /rcode-new-project before committing. See https://github.com/hanzlahabib/rcode/issues/670 -->\n\n`;
 
   fs.writeFileSync(projectPath,
     STUB_BANNER +

package/cli/postinstall.js

@@ -134,7 +134,7 @@ More:
   rcode help         # all commands (grouped)
   rcode dashboard    # view-only Diwan on :7717
 
-Docs: https://github.com/hanzlahabib/rihal-code
+Docs: https://github.com/hanzlahabib/rcode
 `);
 }
 

package/cli/uninstall.js

@@ -949,17 +949,59 @@ async function runUninstall(args) {
   // Notice about what was intentionally preserved (#876 — never delete user data silently)
   const rcodeStillExists = plan.stateDir && fs.existsSync(path.join(cwd, '.rcode'));
   const planningStillExists = !opts.purge && fs.existsSync(path.join(cwd, '.planning'));
-  if (rcodeStillExists || planningStillExists) {
+  const agentsRulesStillExists = !opts.purge && fs.existsSync(path.join(cwd, '.claude/agents/rules'));
+  // Only flag .cursor/ as residue if rcode files are still inside it — the dir
+  // may exist for unrelated Cursor user config that was never touched by rcode.
+  const cursorStillExists = !opts.purge && (() => {
+    const rulesDir = path.join(cwd, '.cursor/rules');
+    if (!fs.existsSync(rulesDir)) return false;
+    try {
+      return fs.readdirSync(rulesDir).some(
+        (f) => f.startsWith('rcode-') || f === 'rcode.mdc' || f === 'rcode-method.mdc' || f === 'rcode',
+      );
+    } catch { return false; }
+  })();
+  const gitignoreBlockStillExists = !opts.purge && (() => {
+    try {
+      const gi = path.join(cwd, '.gitignore');
+      if (!fs.existsSync(gi)) return false;
+      const txt = fs.readFileSync(gi, 'utf8');
+      return txt.includes('# ===== rcode-managed gitignore block') || txt.includes('# >>> rcode >>>');
+    } catch { return false; }
+  })();
+  // Only flag pre-commit as residue if the rcode block is still present inside
+  // it — the hook file may exist for user-written scripts unrelated to rcode.
+  const preCommitStillExists = !opts.purge && (() => {
+    const hookPath = path.join(cwd, '.git/hooks/pre-commit');
+    if (!fs.existsSync(hookPath)) return false;
+    try {
+      return fs.readFileSync(hookPath, 'utf8').includes('# ===== rcode-managed pre-commit block =====');
+    } catch { return false; }
+  })();
+
+  if (rcodeStillExists || planningStillExists || agentsRulesStillExists || cursorStillExists || gitignoreBlockStillExists || preCommitStillExists) {
     console.log();
-    console.log(`ℹ  Preserved (your project data — not removed by default):`);
+    console.log(`ℹ  Preserved (use --purge to remove all of these):`);
     if (rcodeStillExists) {
-      console.log(`      .rcode/     phases, decisions, progress, config`);
-      console.log(`                  /rcode-init will detect this on reinstall`);
+      console.log(`      .rcode/                    project state, config, decisions`);
     }
     if (planningStillExists) {
-      console.log(`      .planning/  planning scaffolds (ROADMAP, STATE, PROJECT)`);
+      console.log(`      .planning/                 ROADMAP, STATE, PROJECT scaffolds`);
     }
-    console.log(`   To remove these on next uninstall: rcode uninstall --purge`);
+    if (agentsRulesStillExists) {
+      console.log(`      .claude/agents/rules/      agent rule files`);
+    }
+    if (cursorStillExists) {
+      console.log(`      .cursor/                   Cursor IDE integration`);
+    }
+    if (gitignoreBlockStillExists) {
+      console.log(`      .gitignore                 rcode-managed block still present`);
+    }
+    if (preCommitStillExists) {
+      console.log(`      .git/hooks/                pre-commit hook still present`);
+    }
+    console.log();
+    console.log(`   Full cleanup: rcode uninstall --purge`);
   }
 
   // IDE cache reload hint — Claude Code caches the slash-command list in memory.