@hanzlaa/rcode 3.4.17 → 3.4.18

package/cli/index.js

@@ -91,9 +91,56 @@ Documentation: https://github.com/hanzlahabib/rihal-code
   `.trim());
 }
 
+/**
+ * npm 10+ suppresses postinstall script output during global installs, so users
+ * who run `npm install -g @hanzlaa/rcode` see only "added 1 package" with no
+ * confirmation that 100+ commands and skills were installed. We detect a fresh
+ * install on the first `rcode <anything>` invocation by checking for a marker
+ * file under ~/.rihal/, print a one-time welcome banner, then drop the marker.
+ */
+function maybeShowFirstRunBanner() {
+  const os = require('os');
+  const home = os.homedir();
+  const markerDir = path.join(home, '.rihal');
+  const marker = path.join(markerDir, '.welcome-shown');
+  if (fs.existsSync(marker)) return;
+
+  // Only show banner if global install actually ran — i.e. ~/.claude/commands/
+  // has rihal-*.md files. Otherwise this is a developer running from source.
+  const globalCommands = path.join(home, '.claude', 'commands');
+  let hasGlobalRihal = false;
+  try {
+    hasGlobalRihal = fs.existsSync(globalCommands) &&
+      fs.readdirSync(globalCommands).some(f => f.startsWith('rihal-') && f.endsWith('.md'));
+  } catch { /* unreadable */ }
+  if (!hasGlobalRihal) return;
+
+  console.log(`\n🕌 Rihal Code v${PACKAGE_JSON.version} — first run detected.\n`);
+  console.log(`   ✓ ${countGlobalRihal(globalCommands)} slash commands installed → ~/.claude/commands/`);
+  console.log(`   ✓ All /rihal-* commands available in every Claude Code project.`);
+  console.log(`\n   To set up a project:  cd my-project && rcode install`);
+  console.log(`   Show all commands:    rcode help`);
+  console.log(`   Diagnose issues:      rcode doctor\n`);
+
+  try {
+    fs.mkdirSync(markerDir, { recursive: true });
+    fs.writeFileSync(marker, `installed ${PACKAGE_JSON.version} at ${new Date().toISOString()}\n`);
+  } catch { /* if we can't write the marker, banner shows again next time — annoying but not broken */ }
+}
+
+function countGlobalRihal(dir) {
+  try {
+    return fs.readdirSync(dir).filter(f => f.startsWith('rihal-') && f.endsWith('.md')).length;
+  } catch { return 0; }
+}
+
 async function main() {
   const [, , command = 'help', ...args] = process.argv;
 
+  // Show first-run banner before dispatching — npm hides postinstall output,
+  // so this is the user's first visible confirmation that the install worked.
+  maybeShowFirstRunBanner();
+
   const handler = COMMANDS[command];
   if (!handler) {
     console.error(`Unknown command: ${command}`);

package/cli/install.js

@@ -177,6 +177,13 @@ function parseArgs(argv) {
     opts.target = path.resolve(positional[0]);
     opts.targetProvided = true;
   }
+  // --global without an explicit target means "install to ~/.claude/" — i.e.
+  // home dir. Without this, running `rcode install --global` from inside a
+  // project directory wrote rihal artifacts to that project, not to the user's
+  // home where Claude Code reads global commands from.
+  if (opts.global && !opts.targetProvided) {
+    opts.target = os.homedir();
+  }
   if (!opts.projectName) opts.projectName = path.basename(opts.target);
   return opts;
 }

package/dist/rcode.js

@@ -15063,6 +15063,9 @@ var require_install = __commonJS({
         opts.target = path2.resolve(positional[0]);
         opts.targetProvided = true;
       }
+      if (opts.global && !opts.targetProvided) {
+        opts.target = os.homedir();
+      }
       if (!opts.projectName) opts.projectName = path2.basename(opts.target);
       return opts;
     }
@@ -20945,8 +20948,46 @@ Getting started:
 Documentation: https://github.com/hanzlahabib/rihal-code
   `.trim());
 }
+function maybeShowFirstRunBanner() {
+  const os = require("os");
+  const home = os.homedir();
+  const markerDir = path.join(home, ".rihal");
+  const marker = path.join(markerDir, ".welcome-shown");
+  if (fs.existsSync(marker)) return;
+  const globalCommands = path.join(home, ".claude", "commands");
+  let hasGlobalRihal = false;
+  try {
+    hasGlobalRihal = fs.existsSync(globalCommands) && fs.readdirSync(globalCommands).some((f) => f.startsWith("rihal-") && f.endsWith(".md"));
+  } catch {
+  }
+  if (!hasGlobalRihal) return;
+  console.log(`
+\u{1F54C} Rihal Code v${PACKAGE_JSON.version} \u2014 first run detected.
+`);
+  console.log(`   \u2713 ${countGlobalRihal(globalCommands)} slash commands installed \u2192 ~/.claude/commands/`);
+  console.log(`   \u2713 All /rihal-* commands available in every Claude Code project.`);
+  console.log(`
+   To set up a project:  cd my-project && rcode install`);
+  console.log(`   Show all commands:    rcode help`);
+  console.log(`   Diagnose issues:      rcode doctor
+`);
+  try {
+    fs.mkdirSync(markerDir, { recursive: true });
+    fs.writeFileSync(marker, `installed ${PACKAGE_JSON.version} at ${(/* @__PURE__ */ new Date()).toISOString()}
+`);
+  } catch {
+  }
+}
+function countGlobalRihal(dir) {
+  try {
+    return fs.readdirSync(dir).filter((f) => f.startsWith("rihal-") && f.endsWith(".md")).length;
+  } catch {
+    return 0;
+  }
+}
 async function main() {
   const [, , command = "help", ...args] = process.argv;
+  maybeShowFirstRunBanner();
   const handler = COMMANDS[command];
   if (!handler) {
     console.error(`Unknown command: ${command}`);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hanzlaa/rcode",
-  "version": "3.4.17",
+  "version": "3.4.18",
   "description": "rcode — the memory bank for AI-driven SaaS teams. Persistent project context, distinctive engineering personas, and phase-based workflows. Built by Rihal. Works in Claude Code, Cursor, Gemini, VS Code, and Antigravity.",
   "main": "cli/index.js",
   "bin": {

package/rihal/bin/rihal-tools.cjs

@@ -2390,6 +2390,11 @@ function cmdState(subArgs) {
     //   Format B — heading style:   ## Phase 01 — Name  /  ### Phase 01: Name
     // Milestone heading is also matched in any of: "## Milestone M1", "## Milestone v1.0 — Name",
     // "**Milestone: v1.0 — Name**".
+
+    // Issue #651 — must be declared in outer scope. The prune step at end of
+    // sync references seenNums even when roadmap_exists is false (no-op prune
+    // path), causing 'seenNums is not defined' crash.
+    const seenNums = new Set();
     if (parsed.roadmap_exists) {
       const roadmap = fs.readFileSync(roadmapPath, 'utf8');
       const milestoneMatches = [
@@ -2425,8 +2430,6 @@ function cmdState(subArgs) {
       parsed.phases_normalized = beforeClean - cleaned.length;
       state.phases = cleaned;
 
-      const seenNums = new Set();
-
       const upsertPhase = (phaseNum, phaseName, phaseGoal) => {
         if (!/^\d/.test(phaseNum)) return;
         if (phaseName.toLowerCase() === 'phase') return;

package/rihal/workflows/execute.md

@@ -573,7 +573,8 @@ If `CODE_REVIEW_ENABLED` is `"false"`: display "Code review skipped (workflow.co
 REVIEWER_MODEL=$(node ".rihal/bin/rihal-tools.cjs" resolve-model code-reviewer 2>/dev/null | node -e "let d='';process.stdin.on('data',c=>d+=c).on('end',()=>{try{console.log(JSON.parse(d).model)}catch{console.log('')}})" || echo "sonnet")
 REVIEWER_MODEL=${REVIEWER_MODEL:-sonnet}
 REVIEWER_SKILLS=$(node ".rihal/bin/rihal-tools.cjs" agent-skills rihal-code-reviewer 2>/dev/null || echo "")
-PADDED=$(printf "%02d" "${PHASE_NUMBER}")
+# Issue #652 — no leading zeros. Variable name kept for backward compat in this workflow.
+PADDED="${PHASE_NUMBER}"
 REVIEW_FILE="${PHASE_DIR}/${PADDED}-REVIEW.md"
 ```
 

package/rihal/workflows/plan.md

@@ -209,8 +209,9 @@ Exit workflow.
 
 ```bash
 EXISTING_PLAN_COUNT=$(ls "${PHASE_DIR}"/*-SPRINT.md 2>/dev/null | wc -l | tr -d ' ')
-NEXT_PLAN_NUMBER=$(printf "%02d" $((EXISTING_PLAN_COUNT + 1)))
-PADDED_PHASE=$(printf "%02d" "${PHASE}")
+# Issue #652 — no leading zeros in planning artifacts. Phase 8 not 08, plan 2 not 02.
+NEXT_PLAN_NUMBER=$((EXISTING_PLAN_COUNT + 1))
+PADDED_PHASE="${PHASE}"
 GAP_PLAN_FILENAME="${PADDED_PHASE}-${NEXT_PLAN_NUMBER}-SPRINT.md"
 GAP_PLAN_PATH="${PHASE_DIR}/${GAP_PLAN_FILENAME}"
 ```