@hanzlaa/rcode 3.4.4 → 3.4.5

package/rihal/workflows/verify-work.md

@@ -24,7 +24,7 @@ If all pass: `RIHAL ► SPRINT {NN.S} VERIFIED ✓` + Next Up: `/rihal-next`.
 <available_agent_types>
 Valid Rihal subagent types (use exact names — do not fall back to 'general-purpose'):
 - rihal-planner — Creates detailed plans from phase scope
-- rihal-plan-checker — Reviews plan quality before execution
+- rihal-sprint-checker — Reviews plan quality before execution
 </available_agent_types>
 
 <philosophy>
@@ -47,7 +47,7 @@ No Pass/Fail buttons. No severity questions. Just: "Here's what should happen. D
 If $ARGUMENTS contains a phase number, load context:
 
 ```bash
-INIT=$(node ".rihal/bin/rihal-tools.cjs" init verify-work "${PHASE_ARG}")
+INIT=$(node ".rihal/bin/rihal-tools.cjs" init verify-work "${PHASE_ARG}" 2>/dev/null)
 if [[ "$INIT" == @file:* ]]; then INIT=$(cat "${INIT#@file:}"); fi
 AGENT_SKILLS_PLANNER=$(node ".rihal/bin/rihal-tools.cjs" agent-skills rihal-planner 2>/dev/null)
 AGENT_SKILLS_CHECKER=$(node ".rihal/bin/rihal-tools.cjs" agent-skills rihal-checker 2>/dev/null)
@@ -250,7 +250,7 @@ Proceed to `present_test`.
 Render the checkpoint from the structured UAT file instead of composing it freehand:
 
 ```bash
-CHECKPOINT=$(node ".rihal/bin/rihal-tools.cjs" uat render-checkpoint --file "$uat_path" --raw)
+CHECKPOINT=$(node ".rihal/bin/rihal-tools.cjs" uat render-checkpoint --file "$uat_path" --raw 2>/dev/null || echo "")
 if [[ "$CHECKPOINT" == @file:* ]]; then CHECKPOINT=$(cat "${CHECKPOINT#@file:}"); fi
 ```
 
@@ -443,7 +443,7 @@ All tests passed. Ready to continue.
 
 - `/rihal-secure-phase {phase}` — security review (required before advancing)
 - `/rihal-plan {next}` — Plan next phase
-- `/rihal-execute-phase {next}` — Execute next phase
+- `/rihal-execute {next}` — Execute next phase
 - `/rihal-ui-review {phase}` — visual quality audit (if frontend files were modified)
 ```
 
@@ -458,7 +458,7 @@ If `SECURITY_CFG` is `false` OR (`SECURITY_FILE` exists AND `threats_open` is `0
 All tests passed. Ready to continue.
 
 - `/rihal-plan {next}` — Plan next phase
-- `/rihal-execute-phase {next}` — Execute next phase
+- `/rihal-execute {next}` — Execute next phase
 - `/rihal-secure-phase {phase}` — security review
 - `/rihal-ui-review {phase}` — visual quality audit (if frontend files were modified)
 ```
@@ -518,11 +518,12 @@ ${AGENT_SKILLS_PLANNER}
 </planning_context>
 
 <downstream_consumer>
-Output consumed by /rihal-execute-phase
+Output consumed by /rihal-execute
 Plans must be executable prompts.
 </downstream_consumer>
 """,
   subagent_type="rihal-planner",
+  model="sonnet",
   model="{planner_model}",
   description="Plan gap fixes for Phase {phase}"
 )
@@ -547,7 +548,7 @@ Display:
 
 Initialize: `iteration_count = 1`
 
-Spawn rihal-plan-checker:
+Spawn rihal-sprint-checker:
 
 ```
 Task(
@@ -571,7 +572,8 @@ Return one of:
 - ## ISSUES FOUND — structured issue list
 </expected_output>
 """,
-  subagent_type="rihal-plan-checker",
+  subagent_type="rihal-sprint-checker",
+  model="sonnet",
   model="{checker_model}",
   description="Verify Phase {phase} fix plans"
 )
@@ -616,6 +618,7 @@ Do NOT replan from scratch unless issues are fundamental.
 </instructions>
 """,
   subagent_type="rihal-planner",
+  model="sonnet",
   model="{planner_model}",
   description="Revise Phase {phase} plans"
 )
@@ -659,7 +662,7 @@ Plans verified and ready for execution.
 
 **Execute fixes** — run fix plans
 
-`/clear` then `/rihal-execute-phase {phase} --gaps-only`
+`/clear` then `/rihal-execute {phase} --gaps-only`
 
 ───────────────────────────────────────────────────────────────
 ```
@@ -711,7 +714,18 @@ Default to **major** if unclear. User can correct if needed.
 - [ ] Committed on completion
 - [ ] If issues: parallel debug agents diagnose root causes
 - [ ] If issues: rihal-planner creates fix plans (gap_closure mode)
-- [ ] If issues: rihal-plan-checker verifies fix plans
+- [ ] If issues: rihal-sprint-checker verifies fix plans
 - [ ] If issues: revision loop until plans pass (max 3 iterations)
-- [ ] Ready for `/rihal-execute-phase --gaps-only` when complete
+- [ ] Ready for `/rihal-execute --gaps-only` when complete
 </success_criteria>
+
+## On Completion
+
+When gaps found and fix plans generated:
+  /rihal-execute {phase} --gaps-only — re-execute just the gap plans
+  /rihal-progress — see updated phase status
+
+When all tests pass:
+  /rihal-add-tests {phase} — generate unit + E2E tests for this phase
+  /rihal-next — advance to next phase
+  /rihal-plan {next} — plan the next phase

package/rihal/workflows/workstream.md

@@ -55,12 +55,12 @@ If validation fails, print error and STOP.
 
 **Execute:**
 ```bash
-node .rihal/bin/rihal-tools.cjs state workstream-create --name "$NAME"
+node .rihal/bin/rihal-tools.cjs state workstream-create --name "$NAME" 2>/dev/null || echo '{"ok":false,"error":"workstream-create failed"}'
 ```
 
 **Output:**
 ```
-✅ Workstream created: {name}
+✓ Workstream created: {name}
    ID: {id}
    Active: false
    Phases: 0
@@ -74,12 +74,12 @@ Set it as active unless user specifies `--no-activate` (not currently supported
 
 **Execute:**
 ```bash
-node .rihal/bin/rihal-tools.cjs state workstream-switch --name "$NAME"
+node .rihal/bin/rihal-tools.cjs state workstream-switch --name "$NAME" 2>/dev/null || echo '{"ok":false,"error":"workstream-switch failed"}'
 ```
 
 **Output:**
 ```
-✅ Switched to workstream: {name}
+✓ Switched to workstream: {name}
    Previous: {old_name}
    Phases in this workstream: {count}
 ```
@@ -88,7 +88,7 @@ node .rihal/bin/rihal-tools.cjs state workstream-switch --name "$NAME"
 
 **Execute:**
 ```bash
-node .rihal/bin/rihal-tools.cjs state workstream-list
+node .rihal/bin/rihal-tools.cjs state workstream-list 2>/dev/null || echo '[]'
 ```
 
 **Output (formatted table):**
@@ -104,7 +104,7 @@ API Refactor             ✗       ✗          1       2026-04-09
 
 **Execute:**
 ```bash
-node .rihal/bin/rihal-tools.cjs state workstream-status
+node .rihal/bin/rihal-tools.cjs state workstream-status 2>/dev/null || echo '{"ok":false,"error":"no active workstream"}'
 ```
 
 **Output:**
@@ -128,12 +128,12 @@ Recent phases:
 
 **Execute:**
 ```bash
-node .rihal/bin/rihal-tools.cjs state workstream-complete --name "$NAME"
+node .rihal/bin/rihal-tools.cjs state workstream-complete --name "$NAME" 2>/dev/null || echo '{"ok":false,"error":"workstream-complete failed"}'
 ```
 
 **Output:**
 ```
-✅ Marked workstream complete: {name}
+✓ Marked workstream complete: {name}
    Completed at: {ISO date}
    All phases: {count}
    Commits in this workstream: {count}
@@ -195,3 +195,15 @@ If arguments are invalid, missing files, or subagent fails:
 - Check that required files exist
 - Retry with clearer arguments or report the specific error to the user
 
+
+## On Completion
+
+/rihal-plan {next_phase} — plan next phase in this workstream
+/rihal-next — get suggested next action
+/rihal-progress — see all workstreams and phases
+
+## ▶ Next Up
+
+- **Workstream created:** `/rihal-execute {workstream-phase}` — begin execution
+- **View all workstreams:** `/rihal-progress` — see parallel workstream status
+- **Stuck:** `/rihal-forensics` — diagnose any blocked workstream

package/server/lib/html/client.js

@@ -161,7 +161,7 @@ function phaseCard(p) {
   const sps = p.sprints || [];
   const stories = sps.flatMap(s => s.stories || []);
   const done = stories.filter(t => t.status === 'done' || t.status === 'completed').length;
-  const isCur = p.id === S.currentPhase;
+  const isCur = String(p.id) === String(S.currentPhase);
   return '<div class="item item-clickable" onclick="navTo(\\'phases/' + p.id + '\\')"' +
     (isCur ? ' style="border-left-color:var(--accent-amber)"' : '') + '>' +
     '<div class="item-title">Phase ' + esc(p.id) + ' — ' + esc(p.name) +
@@ -180,6 +180,7 @@ function sprintCard(s) {
   const stories = s.stories || [];
   const done = stories.filter(t => t.status === 'done' || t.status === 'completed').length;
   const isCur = s.id === S.currentSprint;
+  const phaseId = s.phaseId || s.id || '';
   return '<div class="item item-clickable' + (isCur ? ' sprint-current' : '') + '" onclick="navTo(\\'sprints/' + s.id + '\\')"' +
     (isCur ? ' style="border-left-color:var(--accent-amber);background:rgba(245,158,11,0.04)"' : '') + '>' +
     '<div class="item-title">Sprint ' + esc(s.id) + ' — ' + esc(s.goal || 'No goal') +
@@ -190,6 +191,7 @@ function sprintCard(s) {
     (s.velocity_target != null ? tag('Target: ' + s.velocity_target + 'pts') : '') +
     (s.velocity_actual != null ? tag('Actual: ' + s.velocity_actual + 'pts') : '') + '</div>' +
     '<div style="margin-top:6px;">' + progressBar(done, stories.length) + '</div>' +
+    (stories.length === 0 ? '<div class="empty-action" style="margin-top:var(--space-2);font-size:var(--text-xs);">No tasks — run <code>/rihal-plan ' + esc(phaseId) + '</code> to populate</div>' : '') +
     (s.started_at ? '<div style="color:var(--text-muted);font-size:var(--text-xs);margin-top:4px;">' +
       humanDate(s.started_at) + (s.completed_at ? ' → ' + humanDate(s.completed_at) : ' → ongoing') + '</div>' : '') +
     '</div>';
@@ -451,7 +453,7 @@ function renderMilestones(subId) {
 function renderPhases(subId) {
   const el = document.getElementById('view-phases');
   if (subId) {
-    const p = _phases.find(ph => ph.id === subId || ph.number === subId);
+    const p = _phases.find(ph => String(ph.id) === String(subId) || String(ph.number) === String(subId));
     // Fix #319: guard against missing sprints key
     if (!p) { el.innerHTML = breadcrumb('Phases','phases') + '<div class="empty">Phase not found.</div>'; return; }
     const sps = Array.isArray(p.sprints) ? p.sprints : [];
@@ -501,7 +503,7 @@ function renderSprints(subId) {
   const el = document.getElementById('view-sprints');
   const sprints = allSprints();
   if (subId) {
-    const s = sprints.find(sp => sp.id === subId);
+    const s = sprints.find(sp => String(sp.id) === String(subId));
     if (!s) { el.innerHTML = breadcrumb('All Sprints','sprints') + '<div class="empty">Sprint not found.</div>'; return; }
     const rawStories = Array.isArray(s.stories) ? s.stories : [];
     const stories = rawStories.map(function(t) { return Object.assign({}, t, {sprintId: s.id, sprintGoal: s.goal || '', phaseId: s.phaseId, phaseName: s.phaseName}); });
@@ -578,7 +580,11 @@ function renderTasks() {
 }
 
 function renderTasksGrouped(tasks) {
-  if (!tasks.length) return '<div class="empty">No tasks yet.<div class="empty-action">Run /rihal-create-story to add tasks</div></div>';
+  if (!tasks.length) {
+    var phaseHint = S.currentPhase ? ' ' + S.currentPhase : '';
+    return '<div class="empty">No tasks yet.' +
+      '<div class="empty-action">Run <code>/rihal-plan' + phaseHint + '</code> to generate tasks for this project.</div></div>';
+  }
   const groups = {};
   for (const t of tasks) {
     const key = t.sprintId || 'unassigned';
@@ -813,61 +819,13 @@ function filterItems(input, listId) {
   });
 }
 
-// ---- File tree (sidebar) ----
-(async function() {
-  let groups = [];
-  try { const r = await fetch('/api/files'); groups = await r.json(); } catch { return; }
-  const tree = document.getElementById('sidebar-file-tree');
-  if (!tree) return;
-  // #301: search input
-  let h = '<div style="padding:0 var(--space-2) var(--space-2);"><input class="filter-input" style="width:100%;max-width:none;" type="text" placeholder="Search files…" id="file-tree-search" oninput="filterFileTree(this.value)"></div>';
-  h += '<div class="file-tree" id="file-tree-items">';
-  h += groups.map(function(g) {
-    // Support sub-groups (e.g. Phases with per-phase sub-groups)
-    if (g.subGroups) {
-      return '<details class="file-tree-group" open><summary>' + g.group + '</summary>' +
-        g.subGroups.map(function(sg) {
-          return '<details class="file-tree-subgroup" open style="margin-left:var(--space-2);margin-bottom:var(--space-1);">' +
-            '<summary style="font-size:var(--text-xs);font-weight:500;color:var(--text-secondary);cursor:pointer;padding:var(--space-1) 0;user-select:none;">' + sg.subGroup + ' <span style="color:var(--text-muted);font-weight:400;">(' + sg.files.length + ')</span></summary>' +
-            sg.files.map(function(f) {
-              return '<span class="file-tree-item" data-path="' + esc(f.path) + '" data-filter-text="' + esc(f.label + ' ' + f.path + ' ' + sg.subGroup).toLowerCase() + '" style="padding-left:var(--space-4);">' + esc(f.label) + '</span>';
-            }).join('') + '</details>';
-        }).join('') + '</details>';
-    }
-    return '<details class="file-tree-group" open><summary>' + g.group + '</summary>' +
-      g.files.map(function(f) {
-        return '<span class="file-tree-item" data-path="' + esc(f.path) + '" data-filter-text="' + esc(f.label + ' ' + f.path).toLowerCase() + '">' + esc(f.label) + '</span>';
-      }).join('') + '</details>';
-  }).join('');
-  h += '</div>';
-  tree.innerHTML = h;
-  tree.addEventListener('click', async function(e) {
-    var item = e.target.closest('.file-tree-item');
-    if (!item) return;
-    tree.querySelectorAll('.file-tree-item').forEach(function(el) { el.classList.remove('selected'); });
-    item.classList.add('selected');
-    navTo('files');
-    var fv = document.getElementById('file-view');
-    // #315: loading skeleton
-    fv.innerHTML = '<div class="skeleton"></div><div class="skeleton" style="height:200px;"></div>';
-    // Scroll file content into view
-    fv.scrollIntoView({ behavior: 'smooth', block: 'start' });
-    try {
-      var resp = await fetch('/api/file?path=' + encodeURIComponent(item.dataset.path));
-      if (!resp.ok) { fv.innerHTML = '<div style="color:var(--accent-red);padding:16px;">Failed to load file.</div>'; return; }
-      var text = await resp.text();
-      // #298: show file path + #299: copy path button
-      fv.innerHTML = '<div class="file-path-header"><span>' + esc(item.dataset.path) + '</span>' +
-        '<button class="copy-btn" onclick="navigator.clipboard.writeText(\\'' + item.dataset.path.replace(/'/g, "\\\\'") + '\\');showToast(\\'Path copied!\\')">📋 Copy</button></div>' +
-        '<div class="md-render">' + renderMd(text) + '</div>';
-    } catch { fv.innerHTML = '<div style="color:var(--accent-red);padding:16px;">Network error.</div>'; }
-  });
-})();
+// ---- Shared file-list fetch (single request for all consumers) ----
+const _filesPromise = fetch('/api/files').then(function(r) { return r.json(); }).catch(function() { return []; });
 
 // Inline file list inside Files view
 (async function() {
   let groups = [];
-  try { const r = await fetch('/api/files'); groups = await r.json(); } catch { return; }
+  try { groups = await _filesPromise; } catch { return; }
   const el = document.getElementById('file-list-inline');
   if (!el) return;
   let h = '<div class="filter-bar"><input class="filter-input" type="text" placeholder="Search files…" oninput="filterInlineFiles(this.value)"></div>';
@@ -929,14 +887,6 @@ async function loadInlineFile(el) {
   } catch { fv.innerHTML = '<div style="color:var(--accent-red);padding:16px;">Network error.</div>'; }
 }
 
-// #301: filter file tree
-function filterFileTree(q) {
-  q = q.toLowerCase().trim();
-  document.querySelectorAll('#file-tree-items .file-tree-item').forEach(function(item) {
-    item.style.display = !q || (item.dataset.filterText || '').includes(q) ? '' : 'none';
-  });
-}
-
 // ---- Markdown + frontmatter ----
 function stripFrontmatter(md) {
   if (!md.startsWith('---')) return md;

package/server/lib/html/shell.js

@@ -83,7 +83,6 @@ ${renderCss()}
       <button class="nav-link" data-view="decisions">⚖ Decisions</button>
       <button class="nav-link" data-view="memory">🧠 Memory Bank</button>
     </nav>
-    <div id="sidebar-file-tree" style="margin-top:var(--space-4);padding:0 var(--space-2);"></div>
   </aside>
   <div id="sidebar-backdrop" onclick="closeSidebar()"></div>
   <div class="content-area" id="main-content">

package/server/lib/scanner.js

@@ -78,11 +78,42 @@ function scanState(rihalDir) {
     const phasesDir = path.join(projectDir, '.planning', 'phases');
     state.phases = state.raw.phases.map(p => {
       const sprints    = Array.isArray(p.sprints) ? p.sprints : [];
-      const allStories = sprints.flatMap(s => Array.isArray(s.stories) ? s.stories : []);
+      let   allStories = sprints.flatMap(s => Array.isArray(s.stories) ? s.stories : []);
+
+      // #590 fallback: when state.json has no stories but a SPRINT.md exists, parse task lines
+      // so the Tasks view is not empty. Synthetic entries carry _source:'sprint-md-fallback'.
+      if (allStories.length === 0) {
+        const phasesDir2 = path.join(projectDir, '.planning', 'phases');
+        try {
+          const intIdFb  = String(p.id || p.number || '').split('.')[0];
+          const paddedFb = intIdFb.padStart(2, '0');
+          const dirsFb   = fs.readdirSync(phasesDir2, { withFileTypes: true });
+          const matchFb  = dirsFb.find(d => d.isDirectory() && d.name.startsWith(paddedFb + '-'));
+          if (matchFb) {
+            const allMdFb    = fs.readdirSync(path.join(phasesDir2, matchFb.name)).filter(f => f.endsWith('.md'));
+            const numberedFb = allMdFb.filter(f => /^\d{2}-\d{2}-/.test(f)).sort().reverse();
+            const chosenFb   = numberedFb.length ? numberedFb[0] : allMdFb.sort().reverse()[0];
+            if (chosenFb) {
+              const mdText = safeReadText(path.join(phasesDir2, matchFb.name, chosenFb));
+              if (mdText) {
+                const taskLines = mdText.split('\n').filter(l => /^[-*]\s+\[[ xX]\]/.test(l.trim()));
+                allStories = taskLines.map((l, i) => ({
+                  id:      (p.id || p.number || 'p') + '-task-' + (i + 1),
+                  title:   l.replace(/^[-*]\s+\[[ xX]\]\s*/, '').trim() || ('Task ' + (i + 1)),
+                  status:  /\[[xX]\]/.test(l) ? 'done' : 'todo',
+                  _source: 'sprint-md-fallback',
+                }));
+              }
+            }
+          }
+        } catch { /* phasesDir missing or unreadable */ }
+      }
+
       const done  = allStories.filter(s => s.status === 'done' || s.status === 'completed').length;
       const total = allStories.length;
 
-      const padded = String(p.id || p.number || '').padStart(2, '0');
+      const intId  = String(p.id || p.number || '').split('.')[0];
+      const padded = intId.padStart(2, '0');
       let phaseDir = null, sprintFile = null;
       try {
         const dirs = fs.readdirSync(phasesDir, { withFileTypes: true });