@hanzlaa/rcode 4.4.1 → 4.4.2

package/README.md

@@ -23,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.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.
+Status: `@hanzlaa/rcode` v4.4.2 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.
 
 ---
 
@@ -127,18 +127,18 @@ 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. |
-| **Automated tests** | **497**, 100% passing across 61 files | The methodology is *guarded*, not vibes. CI runs them on every push; `node --test` reproduces locally in seconds. |
-| **Tested CLI engine** | **9,005 lines** (`rcode-tools.cjs` + `lib/`) | The deterministic brain — routing, state, planning math — is real code under test, not prompt soup. |
-| **Runtime dependencies** | **1** (`ws`, for the dashboard socket) | The view-only dashboard is pure Node stdlib. Almost nothing to audit, nothing to CVE-scan, nothing to break on `npm install`. |
-| **Specialist agents / commands / workflows / skills** | **45 / 116 / 129 / 88** | An entire engineering org, phrase-activated, that travels with you across Claude Code, Cursor, Codex, VS Code, Antigravity, Windsurf. |
-| **Core-op latency** | **~33 ms**, **0 LLM tokens** (best-of-7) | Intent routing, state reads, and milestone-health are deterministic *local* compute — a few ms over Node's own cold-start floor. The orchestration layer doesn't burn API tokens on bookkeeping the way pure-LLM agent frameworks do. |
-
-**The headline:** an entire software-delivery methodology — 94k lines of it — guarded by 497 tests, riding on a single runtime dependency, with an orchestration brain that costs **zero tokens** to think. Not a viral prompt. A system you can verify line by line.
+| **Portable methodology corpus** | **~105,000 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. |
+| **Automated tests** | **495** across 62 files | The methodology is *guarded*, not vibes. Live pass/fail status is on the CI badge above; `node --test` (or `node benchmarks/facts.cjs`) reproduces locally in seconds. |
+| **Tested CLI engine** | **~9,700 lines** (`rcode-tools.cjs` + `lib/`) | The deterministic brain — routing, state, planning math — is real code under test, not prompt soup. |
+| **Runtime dependencies** | **1** (`ws`, for the dashboard socket) | The dashboard is otherwise pure Node stdlib. Almost nothing to audit, nothing to CVE-scan, nothing to break on `npm install`. |
+| **Specialist agents / commands / workflows / skills** | **45 / 117 / 129 / 96** | An entire engineering org, phrase-activated, that travels with you across Claude Code, Cursor, Codex, VS Code, Antigravity, Windsurf. |
+| **Core-op latency** | **~60 ms**, **0 LLM tokens** (best-of-7) | Intent routing, state reads, and milestone-health are deterministic *local* compute — a few ms over Node's own cold-start floor. The orchestration layer doesn't burn API tokens on bookkeeping the way pure-LLM agent frameworks do. |
+
+> Counts above are produced by `node benchmarks/facts.cjs` and may drift slightly between releases — run it for the exact current figures.
+
+**The headline:** an entire software-delivery methodology — ~105k lines of it — guarded by a real test suite, riding on a single runtime dependency, with an orchestration brain that costs **zero tokens** to think. Not a viral prompt. A system you can verify line by line.
 
 ---
 

package/cli/install.js

@@ -825,15 +825,20 @@ function seedStarterPlanning(target, projectName) {
 
     const state = {
       version: '1',
-      project: resolvedProject,
+      // #940 — match the canonical schema (cli/lib/schemas.cjs stateSchema):
+      // schema_version is required, milestones is a plural array. Without these
+      // a freshly-installed state.json failed `rcode-tools validate` until the
+      // first migrateState() read. Write conformant state up front.
+      schema_version: 2,
+      project: resolvedProject || path.basename(target),
       ...(isStubProject ? { _seeded_stub: true } : {}),
       created: now,
       updated: now,
       current_phase: resolvedPhase,
       current_plan: 0,
       current_sprint: null,
-      milestone: null,
       phases: [],
+      milestones: [],
       executions: [],
       decisions: [],
       blockers: [],

package/dist/rcode.js

@@ -196,7 +196,7 @@ _None._
 ## Next Action
 
 Run \`/rcode-new-project <description>\` to bootstrap, or \`/rcode-sprint-planning\` once a real phase exists.
-`);let c=p.join(t,".rcode","state.json");function l(){let d=p.join(t,".planning","ROADMAP.md");if(!D.existsSync(d))return!0;try{return D.readFileSync(d,"utf8").includes("<!-- INSTALL STUB")}catch{return!0}}if(!D.existsSync(c)){let d=new Date().toISOString(),f=l(),g=null,m=p.join(t,".rcode","config.yaml");if(D.existsSync(m))try{let F=D.readFileSync(m,"utf8").match(/^project_name:\s*"?([^"\n]+)"?/m);F&&(g=F[1].trim())}catch{}let y=null,h=p.join(t,".planning","ROADMAP.md");if(!f&&D.existsSync(h))try{let b=D.readFileSync(h,"utf8"),F=b.match(/^\|\s*(\d+(?:\.\d+)?)\s*\|/m);if(F)y=String(parseInt(F[1],10));else{let T=b.match(/^#{2,4}\s*Phase\s+(\d+(?:\.\d+)?)/im);T&&(y=String(parseInt(T[1],10)))}}catch{}let w={version:"1",project:g,...f?{_seeded_stub:!0}:{},created:d,updated:d,current_phase:y,current_plan:0,current_sprint:null,milestone:null,phases:[],executions:[],decisions:[],blockers:[],council_sessions:[],chains:[],workstreams:[],active_workstream:null,last_session:null,velocity_history:[]};D.mkdirSync(p.dirname(c),{recursive:!0}),Ft(c,JSON.stringify(w,null,2)+`
+`);let c=p.join(t,".rcode","state.json");function l(){let d=p.join(t,".planning","ROADMAP.md");if(!D.existsSync(d))return!0;try{return D.readFileSync(d,"utf8").includes("<!-- INSTALL STUB")}catch{return!0}}if(!D.existsSync(c)){let d=new Date().toISOString(),f=l(),g=null,m=p.join(t,".rcode","config.yaml");if(D.existsSync(m))try{let F=D.readFileSync(m,"utf8").match(/^project_name:\s*"?([^"\n]+)"?/m);F&&(g=F[1].trim())}catch{}let y=null,h=p.join(t,".planning","ROADMAP.md");if(!f&&D.existsSync(h))try{let b=D.readFileSync(h,"utf8"),F=b.match(/^\|\s*(\d+(?:\.\d+)?)\s*\|/m);if(F)y=String(parseInt(F[1],10));else{let T=b.match(/^#{2,4}\s*Phase\s+(\d+(?:\.\d+)?)/im);T&&(y=String(parseInt(T[1],10)))}}catch{}let w={version:"1",schema_version:2,project:g||p.basename(t),...f?{_seeded_stub:!0}:{},created:d,updated:d,current_phase:y,current_plan:0,current_sprint:null,phases:[],milestones:[],executions:[],decisions:[],blockers:[],council_sessions:[],chains:[],workstreams:[],active_workstream:null,last_session:null,velocity_history:[]};D.mkdirSync(p.dirname(c),{recursive:!0}),Ft(c,JSON.stringify(w,null,2)+`
 `)}return!0}function uk(t,e={}){let r=e.commitPlanning!==!1,n="# ===== rcode-managed gitignore block (npx @hanzlaa/rcode install) =====",s="# ===== end rcode-managed gitignore block =====",i=["",n,"# Added automatically on first rcode install. Idempotent \u2014 safe to re-run.","# Edit `commit_planning` in .rcode/config.yaml to flip planning-artifact tracking.","","# Installed methodology files (regenerate with: npx @hanzlaa/rcode install)",".claude/",".rcode/bin/",".rcode/workflows/",".rcode/references/",".rcode/commands/",".rcode/skills/","","# Pulled rcode brain content (refresh with: rcode brain pull)",".rcode/brain/rcode-github/",".rcode/brain/rcode-docs/",".rcode/brain/best-practices/","","# Runtime noise","node_modules/",".rcode/state.json.lock",".planning/debug/",".planning/_backup/"];r||i.push("","# Planning artifacts \u2014 kept local (commit_planning: false)",".planning/"),i.push("","# What you DO commit:","#   .rcode/config.yaml        - project mode/language/profile/commit_planning","#   .rcode/state.json         - decisions, roadmap pointer, blockers","#   .rcode/brain/sources.yaml - brain source manifest",r?"#   .planning/                - PRD, roadmap, sprints, SUMMARY.md files":"#   (planning artifacts are NOT committed \u2014 see commit_planning in config)",s,"");let o=i.join(`
 `),a=p.join(t,".gitignore");try{let l=function(d,f){let g=d.indexOf(n);if(g<0)return null;let m=d.indexOf(s,g);if(m<0){let w=g;return w>0&&d[w-1]===`
 `&&(w-=1),d.slice(0,w)+f}let y=g;y>0&&d[y-1]===`

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hanzlaa/rcode",
-  "version": "4.4.1",
+  "version": "4.4.2",
   "description": "rcode — the AI team that never forgets. Persistent memory, specialist agents, and slash commands for AI IDEs. Works in Claude Code, Cursor, Gemini, VS Code, and Antigravity.",
   "main": "cli/index.js",
   "bin": {

package/rcode/bin/rcode-tools.cjs

@@ -7596,21 +7596,43 @@ async function main() {
         const validateErrors = [];
 
         if (target === 'state' || target === 'all') {
-          const state = readState();
-          if (!state) {
-            validateErrors.push('state: state.json not found or empty');
+          // Read state.json directly — readState() is scoped inside cmdState()
+          // and isn't reachable here (#940). Validate against the REAL schema:
+          // phases[] + milestones[], current_phase nullable on fresh installs.
+          const statePathV = path.join(RCODE_DIR, 'state.json');
+          if (!fs.existsSync(statePathV)) {
+            validateErrors.push('state: state.json not found — run rcode install');
           } else {
-            if (!state.current_phase) validateErrors.push('state: missing current_phase');
-            if (!state.current_milestone) validateErrors.push('state: missing current_milestone');
-            if (state.schema_version !== 2) validateErrors.push(`state: schema_version should be 2, got ${state.schema_version}`);
-            if (!Array.isArray(state.phases) && typeof state.phases !== 'object') validateErrors.push('state: phases must be array or object');
+            let state = null;
+            try {
+              state = JSON.parse(fs.readFileSync(statePathV, 'utf8'));
+            } catch (e) {
+              validateErrors.push(`state: invalid JSON — ${e.message}`);
+            }
+            if (state) {
+              // current_phase is legitimately null on a fresh / stub project,
+              // so do NOT require it. milestones is an array (no current_milestone field).
+              // schema_version 1 is valid on disk — migrateState() upgrades it to 2
+              // transparently on read, so accept either and only flag the unknown.
+              if (state.schema_version != null && ![1, 2].includes(state.schema_version)) {
+                validateErrors.push(`state: unknown schema_version ${state.schema_version} (expected 1 or 2)`);
+              }
+              if (state.phases != null && !Array.isArray(state.phases) && typeof state.phases !== 'object') {
+                validateErrors.push('state: phases must be an array or object');
+              }
+              if (state.milestones != null && !Array.isArray(state.milestones)) {
+                validateErrors.push('state: milestones must be an array');
+              }
+            }
           }
         }
 
         if (target === 'config' || target === 'all') {
+          // config.yaml holds project identity + workflow prefs only — it does
+          // NOT track current_phase (that lives in state.json). Only require
+          // the fields config actually owns (#940).
           const config = readConfig();
           if (!config.project_name) validateErrors.push('config: missing project_name');
-          if (!config.current_phase && !config.phase) validateErrors.push('config: missing current_phase or phase');
         }
 
         if (validateErrors.length) {

package/rcode/commands/lazy.md

@@ -5,6 +5,12 @@ argument-hint: "[challenge or code to simplify] [--intensity=lite|full|ultra]"
 allowed-tools:
   - Read
   - AskUserQuestion
+  - Skill
 ---
 
-@rcode/skills/core/rcode-lazy/SKILL.md
+Invoke the `rcode-lazy` skill (via the Skill tool) and apply it to: $ARGUMENTS
+
+`rcode-lazy` is the always-on "lazy senior dev" lens — it forces the simplest
+solution that actually works (YAGNI, stdlib before custom code, native platform
+features before dependencies, one line before fifty) before any code is written.
+Pass `--intensity=lite|full|ultra` through if the user provided it; default is `full`.