@hanzlaa/rcode 4.3.3 → 4.4.1

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. **45 agents · 116 commands · 1 runtime dependency** — test count and CI status tracked by the badge above.*
+
+<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> (<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.
 
@@ -19,7 +23,7 @@ pnpm dlx @hanzlaa/rcode install
 [![CI](https://github.com/hanzlahabib/rcode/actions/workflows/test.yml/badge.svg)](https://github.com/hanzlahabib/rcode/actions/workflows/test.yml)
 [![License: MIT](https://img.shields.io/badge/license-MIT-blue.svg)](LICENSE)
 
-Status: `@hanzlaa/rcode` v4.3.2 on npm. 497 automated tests (100% green) across 61 files · 45 agents · 116 commands · 129 workflows · 88 skills · **1 runtime dependency**. Every number here is reproducible — run `node benchmarks/facts.cjs`. Actively dogfooded on real projects every week.
+Status: `@hanzlaa/rcode` v4.4.1 on npm. 45 agents · 116 commands · 129 workflows · **1 runtime dependency**. Test status tracked by CI badge above. Actively dogfooded on real projects every week.
 
 ---
 
@@ -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`.
@@ -218,6 +224,23 @@ Named for the Arabic root **رحال** (rahhal) — the traveler. rcode walks al
 
 ---
 
+## 💙 Feedback welcome
+
+rcode is built and maintained by one person. If you hit a rough edge, a confusing command, or anything that doesn't work — please share it. Every report, even a one-liner, makes the tool better for everyone.
+
+**Post an issue in 30 seconds → [github.com/hanzlahabib/rcode/issues/new](https://github.com/hanzlahabib/rcode/issues/new)**
+
+Quick template:
+```
+Title:  what you were trying to do
+Body:   what happened vs what you expected
+        + the command or phrase you used
+```
+
+Thank you for using rcode. 🙏
+
+---
+
 ## Credits
 
 - [Andrej Karpathy's coding observations](https://github.com/forrestchang/andrej-karpathy-skills) (MIT) — wired into the code-review agents as hard constraints.

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/generate-command-skills.cjs

@@ -100,7 +100,7 @@ description: >
   Sidebar entry — invokes the same workflow as /rcode-${cmdName}.
 triggers:
 ${triggers.map((t) => `  - "${t}"`).join('\n')}
-user-invocable: false
+user-invocable: true
 generated: true
 generated-by: rcode-install-v${version}
 source: rcode/commands/${cmdName}.md

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('╰───────────────────────────────────────────────────────────╯'),
@@ -480,7 +480,8 @@ async function resolveIde(opts) {
  */
 async function resolveCommitPlanning(opts) {
   if (opts.commitPlanning !== null) return opts.commitPlanning;
-  if (opts.noPrompt || opts.global) return false; // global install: no planning artifacts
+  if (opts.global) return false; // global install: no planning artifacts
+  if (opts.noPrompt) return true; // non-interactive project install: commit planning by default (#936)
 
   // Issue #685: on re-install, read the existing .rcode/config.yaml and use
   // its commit_planning value as the default. Otherwise the new prompt
@@ -726,7 +727,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 +
@@ -744,10 +745,8 @@ function seedStarterPlanning(target, projectName) {
     `**Milestone: M1 — Initial Delivery** (v1.0)\n` +
     `Started: ${today} · Current\n\n` +
     `---\n\n` +
-    `## Phase 01 — Setup & Scaffolding\n\n` +
-    `**Goal:** Lay the foundation. Replace this with your first phase when ready.\n\n` +
-    `**Status:** Planned\n\n` +
-    `**Acceptance:** Working dev environment; first feature in progress.\n\n` +
+    `> **No phases yet.** Run \`/rcode-new-project\` to design your roadmap,\n` +
+    `> or \`/rcode-add-phase <name>\` to add your first phase manually.\n\n` +
     `---\n\n` +
     `## Backlog\n\n` +
     `Ideas and future phases go here.\n`
@@ -758,7 +757,7 @@ function seedStarterPlanning(target, projectName) {
     `# ${name} — State\n\n` +
     `**Last updated:** ${today}\n` +
     `**Milestone:** M1 — Initial Delivery\n` +
-    `**Current phase:** 01 — Setup & Scaffolding\n` +
+    `**Current phase:** none — run /rcode-new-project or /rcode-add-phase\n` +
     `**Branch:** main\n\n` +
     `---\n\n` +
     `## Decisions\n\n_None yet._\n\n` +
@@ -1753,7 +1752,6 @@ function generateConfigYaml(opts) {
     `mode: "${sanitizeYamlValue(opts.mode)}"`,
     `model_profile: "balanced"`,
     `commit_planning: ${opts.commitPlanning !== false}`,
-    `rcode_source_path: "${sanitizeYamlValue(resolveStableSourcePath())}/"`,
     'workflow:',
     '  research_by_default: false',
     '  plan_checker: true',

package/cli/postinstall.js

@@ -132,9 +132,29 @@ To set up per-project state + planning structure, run inside your project:
 
 More:
   rcode help         # all commands (grouped)
-  rcode dashboard    # view-only Diwan on :7717
+  rcode dashboard    # Diwan project dashboard on :7717
 
-Docs: https://github.com/hanzlahabib/rihal-code
+Docs: https://github.com/hanzlahabib/rcode
+
+━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
+ 💙 Thank you for using rcode!
+━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
+
+ This package is built and maintained by one person.
+ Every piece of feedback — a bug, a rough edge, a
+ confusing command — makes it meaningfully better.
+
+ Hit a problem? Post it in 30 seconds:
+
+   https://github.com/hanzlahabib/rcode/issues/new
+
+ Quick template (just paste and fill):
+   Title:  what you were trying to do
+   Body:   what happened vs what you expected
+           + paste the command or phrase you used
+
+ Even a one-line report helps. Thank you. 🙏
+━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
 `);
 }
 

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.