@hanzlaa/rcode 4.12.1 → 4.13.0

package/AGENTS.md

@@ -24,7 +24,7 @@ If a user says "just keep going" or "don't stop until done", that authorization
 
 - Follow [Conventional Commits](https://www.conventionalcommits.org/) format: `type(scope): subject`
 - Types allowed: `feat`, `fix`, `docs`, `style`, `refactor`, `test`, `chore`, `perf`, `revert`
-- Scopes allowed: `agents`, `skills`, `workflows`, `templates`, `dashboard`, `docs`, `config`, `github`, `github-sync`, `commands`, `memory`, `brand`, `cli`, `ci`, `release`, `meta`, `tasks`, `migrations`, `refs`, `state`, `hooks`, `init`, `install`, `parity`, `triggers`, `dogfood`, `namespace`, `planning`, `insights`, `help`, `roadmap`, `session`, `audits`, `execute`, `executor`, `plan`, `planner`, `readme`, `rcode`, `review`, `sync`, `sprint`, `agent-exp`, `extensibility`, `lens-audit`, `tiers`, `build`, `council`, `doctor`, `postinstall`, `progress`, `security`, `tools`, `uninstall`, `update`, `test`, `changelog`, `scopes`, `phases`, `references`, `kanban`, `orchestrator`, `orchpanel`, `status`, `bin`, `brain`, `dogfeed`, `new-project`, `package`, `rcode-tools`, `rihal-tools`, `team`, `usp`, `v4`, `observability`, `audit`, `agent-rules`, `cursor`, `i18n`, `phase`, `scaffold`, `campaign`, `ship`, `getting-started`, `do-router`, `milestone-health`, `modules`, `project-types`, `roadmapper`, `token`, `benchmarks`, `eval`, `scan`, `verifier`, `discuss-phase`, `ui-phase`, `state-sync`, `autonomous`, plus numeric phase/sprint scopes (e.g. `docs(15)`, `feat(8.3)`)
+- Scopes allowed: `agents`, `skills`, `workflows`, `templates`, `dashboard`, `docs`, `config`, `github`, `github-sync`, `commands`, `memory`, `brand`, `cli`, `ci`, `release`, `meta`, `tasks`, `migrations`, `refs`, `state`, `hooks`, `init`, `install`, `parity`, `triggers`, `dogfood`, `namespace`, `planning`, `insights`, `help`, `roadmap`, `session`, `audits`, `execute`, `executor`, `plan`, `planner`, `readme`, `rcode`, `review`, `sync`, `sprint`, `agent-exp`, `extensibility`, `lens-audit`, `tiers`, `build`, `council`, `doctor`, `postinstall`, `progress`, `security`, `tools`, `uninstall`, `update`, `test`, `changelog`, `scopes`, `phases`, `references`, `kanban`, `orchestrator`, `orchpanel`, `status`, `bin`, `brain`, `dogfeed`, `new-project`, `package`, `rcode-tools`, `rihal-tools`, `team`, `usp`, `v4`, `observability`, `audit`, `agent-rules`, `cursor`, `i18n`, `phase`, `scaffold`, `campaign`, `ship`, `getting-started`, `do-router`, `milestone-health`, `modules`, `project-types`, `roadmapper`, `token`, `benchmarks`, `eval`, `scan`, `map-codebase`, `verifier`, `discuss-phase`, `ui-phase`, `state-sync`, `autonomous`, plus numeric phase/sprint scopes (e.g. `docs(15)`, `feat(8.3)`)
 - Subject: lowercase first letter, imperative mood, no trailing period, under 72 chars
 - **NEVER add Claude/AI attribution to commit messages.** No "Generated with Claude Code", no "Co-Authored-By: Claude", no "🤖 Generated". The user does not want this.
 - **NEVER use `--no-verify`** to bypass hooks. If hooks fail, fix the underlying issue.

package/CLAUDE.md

@@ -24,7 +24,7 @@ If a user says "just keep going" or "don't stop until done", that authorization
 
 - Follow [Conventional Commits](https://www.conventionalcommits.org/) format: `type(scope): subject`
 - Types allowed: `feat`, `fix`, `docs`, `style`, `refactor`, `test`, `chore`, `perf`, `revert`
-- Scopes allowed: `agents`, `skills`, `workflows`, `templates`, `dashboard`, `docs`, `config`, `github`, `github-sync`, `commands`, `memory`, `brand`, `cli`, `ci`, `release`, `meta`, `tasks`, `migrations`, `refs`, `state`, `hooks`, `install`, `parity`, `triggers`, `dogfood`, `namespace`, `planning`, `insights`, `help`, `roadmap`, `session`, `audits`, `execute`, `executor`, `plan`, `planner`, `readme`, `rcode`, `review`, `sync`, `sprint`, `agent-exp`, `extensibility`, `lens-audit`, `tiers`, `build`, `council`, `doctor`, `postinstall`, `progress`, `security`, `tools`, `uninstall`, `update`, `test`, `changelog`, `scopes`, `phases`, `references`, `kanban`, `orchestrator`, `orchpanel`, `status`, `bin`, `brain`, `dogfeed`, `new-project`, `package`, `rcode-tools`, `rihal-tools`, `team`, `usp`, `v4`, `observability`, `audit`, `init`, `agent-rules`, `cursor`, `i18n`, `phase`, `scaffold`, `campaign`, `ship`, `getting-started`, `do-router`, `milestone-health`, `modules`, `project-types`, `roadmapper`, `token`, `verifier`, `discuss-phase`, `ui-phase`, `state-sync`, `autonomous`, plus numeric phase/sprint scopes (e.g. `docs(15)`, `feat(8.3)`)
+- Scopes allowed: `agents`, `skills`, `workflows`, `templates`, `dashboard`, `docs`, `config`, `github`, `github-sync`, `commands`, `memory`, `brand`, `cli`, `ci`, `release`, `meta`, `tasks`, `migrations`, `refs`, `state`, `hooks`, `install`, `parity`, `triggers`, `dogfood`, `namespace`, `planning`, `insights`, `help`, `roadmap`, `session`, `audits`, `execute`, `executor`, `plan`, `planner`, `readme`, `rcode`, `review`, `sync`, `sprint`, `agent-exp`, `extensibility`, `lens-audit`, `tiers`, `build`, `council`, `doctor`, `postinstall`, `progress`, `security`, `tools`, `uninstall`, `update`, `test`, `changelog`, `scopes`, `phases`, `references`, `kanban`, `orchestrator`, `orchpanel`, `status`, `bin`, `brain`, `dogfeed`, `new-project`, `package`, `rcode-tools`, `rihal-tools`, `team`, `usp`, `v4`, `observability`, `audit`, `init`, `agent-rules`, `cursor`, `i18n`, `phase`, `scaffold`, `campaign`, `ship`, `getting-started`, `do-router`, `milestone-health`, `modules`, `project-types`, `roadmapper`, `token`, `map-codebase`, `verifier`, `discuss-phase`, `ui-phase`, `state-sync`, `autonomous`, plus numeric phase/sprint scopes (e.g. `docs(15)`, `feat(8.3)`)
 - Subject: lowercase first letter, imperative mood, no trailing period, under 72 chars
 - **NEVER add Claude/AI attribution to commit messages.** No "Generated with Claude Code", no "Co-Authored-By: Claude", no "🤖 Generated". The user does not want this.
 - **NEVER use `--no-verify`** to bypass hooks. If hooks fail, fix the underlying issue.

package/CONTRIBUTING.md

@@ -366,6 +366,7 @@ We use [Conventional Commits](https://www.conventionalcommits.org/) format. The
 - `benchmarks` — `benchmarks/` scripts and benchmark reports
 - `eval` — `test/eval/` behavioral drift baselines and eval harness
 - `scan` — `/rcode-scan` workflow and existing-codebase analysis
+- `map-codebase` — `/rcode-map-codebase` workflow and `rcode-codebase-mapper` agent
 - `verifier` — `rcode-verifier` agent, verification playbook/rules, VERIFICATION.md flow
 - `discuss-phase` — `/rcode-discuss-phase` workflow and its gray-area/standing checks
 - `ui-phase` — `/rcode-ui-phase` workflow, UI-SPEC.md/WIREFRAMES.md generation, design-library

package/cli/doctor.js

@@ -127,6 +127,36 @@ function findStuckExecutingPhases(state) {
   });
 }
 
+/**
+ * The inverse of findStuckExecutingPhases: phases whose directory holds a
+ * VERIFICATION.md but whose state entry is not complete. Completion write-back
+ * failed (or never ran), so the project's real progress is higher than
+ * state.json reports. Under-reporting is as much a state defect as
+ * over-reporting — it is just the one nobody instruments.
+ */
+function findUnrecordedVerifiedPhases(state, planningDir) {
+  const phasesDir = path.join(planningDir, 'phases');
+  if (!fs.existsSync(phasesDir)) return [];
+  const norm = (k) => String(k ?? '').trim().replace(/^0+(\d)/, '$1');
+  const verifiedOnDisk = new Set();
+  for (const entry of fs.readdirSync(phasesDir)) {
+    const full = path.join(phasesDir, entry);
+    let files;
+    try {
+      if (!fs.statSync(full).isDirectory()) continue;
+      files = fs.readdirSync(full);
+    } catch { continue; }
+    const m = entry.match(/^(\d+(?:\.\d+)?)/);
+    if (!m) continue;
+    if (files.some((f) => /VERIFICATION\.md$/i.test(f))) verifiedOnDisk.add(norm(m[1]));
+  }
+  const done = new Set(['complete', 'completed', 'done', 'verified']);
+  return (Array.isArray(state?.phases) ? state.phases : [])
+    .filter((p) => p && !done.has(String(p.status ?? '').toLowerCase()) && !p.completed)
+    .filter((p) => verifiedOnDisk.has(norm(p.number ?? p.id)))
+    .map((p) => String(p.number ?? p.id));
+}
+
 // ---------- Preflight checks ----------
 
 /**
@@ -333,13 +363,18 @@ function runPreflight(cwd, packageRoot) {
       try {
         const state = JSON.parse(fs.readFileSync(statePath, 'utf8'));
         const stuck = findStuckExecutingPhases(state);
+        const unrecorded = findUnrecordedVerifiedPhases(state, path.join(cwd, '.planning'));
+        const problems = [];
+        if (stuck.length > 0) {
+          problems.push(`${stuck.length} phase(s) stuck 'executing' while a later phase is complete: ${stuck.map((p) => p.number).join(', ')}`);
+        }
+        if (unrecorded.length > 0) {
+          problems.push(`${unrecorded.length} phase(s) verified on disk but not complete in state.json: ${unrecorded.join(', ')} — completion write-back never ran. Fix: node .rcode/bin/rcode-tools.cjs state sync --from-disk`);
+        }
         checks.push({
           label: 'Phase state',
-          status: stuck.length > 0 ? 'warn' : 'ok',
-          message:
-            stuck.length > 0
-              ? `${stuck.length} phase(s) stuck 'executing' while a later phase is complete: ${stuck.map((p) => p.number).join(', ')}`
-              : 'no stuck phases',
+          status: problems.length > 0 ? 'warn' : 'ok',
+          message: problems.length > 0 ? problems.join('; ') : 'no stuck or unrecorded phases',
         });
       } catch (e) {
         checks.push({

package/cli/install.js

@@ -1232,7 +1232,12 @@ function ensureRcodePreCommitHook(target, options = {}) {
     '# Added by rcode install — safe to re-run (idempotent).',
     'if git diff --cached --name-only | grep -qE "^\\.planning/|^\\.rcode/brain/sources\\.yaml$"; then',
     '  if [ -x .rcode/bin/rcode-tools.cjs ]; then',
-    '    node .rcode/bin/rcode-tools.cjs state sync --from-disk > /dev/null 2>&1 || true',
+    '    # Never silence this. A swallowed sync is how a project reaches 35',
+    '    # executed sprints with executions:0 and nobody is told.',
+    '    if ! _rcode_sync=$(node .rcode/bin/rcode-tools.cjs state sync --from-disk 2>&1); then',
+    '      echo "rcode: state sync --from-disk failed (commit continues):" >&2',
+    '      echo "$_rcode_sync" | tail -3 >&2',
+    '    fi',
     '    git add .rcode/state.json 2>/dev/null || true',
     '  fi',
     'fi',