nubos-pilot 0.7.1 → 0.7.2

package/bin/np-tools/_commands.cjs

@@ -63,7 +63,7 @@ const COMMANDS = [
   { name: 'worktree-remove',   category: 'Execution', description: 'Remove a slice worktree + delete its branch (--force / --keep-branch)' },
   { name: 'worktree-list',     category: 'Execution', description: 'List all nubos-pilot-managed slice worktrees (np/<mid>-<sid> only) as JSON' },
   { name: 'worktree-ff-merge', category: 'Execution', description: 'Fast-forward merge a slice branch back to its base (fails hard on non-FF)' },
-  { name: 'dashboard',         category: 'Utility',   description: 'One-shot console dashboard of milestones/slices/handoffs/worktrees (--json, --no-color, --watch [seconds])' },
+  { name: 'dashboard',         category: 'Utility',   description: 'One-shot console dashboard of milestones, slices, and tasks. Read-only; flags: --json, --no-color' },
   { name: 'thread-resume',     category: 'Utility',  description: 'Bump a thread markdown on resume (status OPEN→IN_PROGRESS, refresh last_resumed) via atomic write' },
   { name: 'state-incr',        category: 'Capture',  description: 'Increment a whitelisted STATE.md counter (e.g. pending_todos) under withFileLock' },
 

package/bin/np-tools/dashboard.cjs

@@ -1,81 +1,29 @@
 'use strict';
 
-const { NubosPilotError } = require('../../lib/core.cjs');
-const { collectSnapshot, renderSnapshot, ANSI } = require('../../lib/dashboard.cjs');
-
-const MIN_WATCH_SECONDS = 1;
-const MAX_WATCH_SECONDS = 3600;
+const { collectSnapshot, renderSnapshot } = require('../../lib/dashboard.cjs');
 
 function _parseArgs(args) {
-  const out = { json: false, noColor: false, watch: null };
-  for (let i = 0; i < args.length; i++) {
-    const a = args[i];
-    if (a === '--json')     { out.json = true; continue; }
-    if (a === '--no-color') { out.noColor = true; continue; }
-    if (a === '--watch') {
-      const raw = args[i + 1];
-      if (raw && !raw.startsWith('-')) { out.watch = Number(raw); i += 1; }
-      else out.watch = 3;
-      continue;
-    }
+  const out = { json: false, noColor: false };
+  for (const a of args) {
+    if (a === '--json')     out.json = true;
+    else if (a === '--no-color') out.noColor = true;
   }
   return out;
 }
 
-function _renderOnce(cwd, stdout, parsed) {
-  const snap = collectSnapshot(cwd);
-  if (parsed.json) {
-    stdout.write(JSON.stringify(snap, null, 2) + '\n');
-    return;
-  }
-  const useColor = !parsed.noColor && Boolean(stdout.isTTY);
-  stdout.write(renderSnapshot(snap, { color: useColor }) + '\n');
-}
-
 function run(args, opts) {
   const o = opts || {};
   const cwd = o.cwd || process.cwd();
   const stdout = o.stdout || process.stdout;
   const parsed = _parseArgs(Array.isArray(args) ? args : []);
 
-  if (parsed.watch != null) {
-    if (!Number.isFinite(parsed.watch) || parsed.watch < MIN_WATCH_SECONDS || parsed.watch > MAX_WATCH_SECONDS) {
-      throw new NubosPilotError(
-        'dashboard-watch-out-of-range',
-        '--watch seconds must be between ' + MIN_WATCH_SECONDS + ' and ' + MAX_WATCH_SECONDS,
-        { got: parsed.watch },
-      );
-    }
-    if (parsed.json) {
-      throw new NubosPilotError(
-        'dashboard-watch-incompatible-json',
-        '--watch cannot be combined with --json (use a shell loop if you need JSON polling)',
-        {},
-      );
-    }
-    const tty = Boolean(stdout.isTTY);
-    const clear = tty ? ANSI.clearScreen : '';
-    const render = () => {
-      try {
-        stdout.write(clear);
-        _renderOnce(cwd, stdout, parsed);
-      } catch (err) {
-        process.stderr.write('[nubos-pilot dashboard] render failed: ' + ((err && err.message) || err) + '\n');
-      }
-    };
-    render();
-    const handle = setInterval(render, parsed.watch * 1000);
-    const stop = () => {
-      clearInterval(handle);
-      try { if (tty) stdout.write('\x1b[?25h'); } catch {}
-      process.exit(0);
-    };
-    process.on('SIGINT', stop);
-    process.on('SIGTERM', stop);
-    return new Promise(() => { });
+  const snap = collectSnapshot(cwd);
+  if (parsed.json) {
+    stdout.write(JSON.stringify(snap, null, 2) + '\n');
+    return 0;
   }
-
-  _renderOnce(cwd, stdout, parsed);
+  const useColor = !parsed.noColor && Boolean(stdout.isTTY);
+  stdout.write(renderSnapshot(snap, { color: useColor }) + '\n');
   return 0;
 }
 

package/lib/dashboard.cjs

@@ -4,11 +4,7 @@ const fs = require('node:fs');
 const path = require('node:path');
 
 const { extractFrontmatter } = require('./frontmatter.cjs');
-const { projectStateDir } = require('./core.cjs');
-const { listMilestones, listSlices, listTasks, mId, sId } = require('./layout.cjs');
-const { listHandoffs } = require('./handoff.cjs');
-const { listSliceWorktrees, worktreeIsolationEnabled } = require('./worktree.cjs');
-const { workspaceGitInfo } = require('./git.cjs');
+const { listMilestones, listSlices, listTasks, mId } = require('./layout.cjs');
 
 const ANSI = Object.freeze({
   reset:  '\x1b[0m',
@@ -20,7 +16,6 @@ const ANSI = Object.freeze({
   blue:   '\x1b[34m',
   cyan:   '\x1b[36m',
   gray:   '\x1b[90m',
-  clearScreen: '\x1b[2J\x1b[H',
 });
 
 const STATUS_GLYPHS = Object.freeze({
@@ -41,6 +36,15 @@ function _safeReadFile(p) {
   catch { return null; }
 }
 
+function _taskStatus(planPath) {
+  const raw = _safeReadFile(planPath);
+  if (!raw) return 'pending';
+  try {
+    const { frontmatter } = extractFrontmatter(raw);
+    return typeof frontmatter.status === 'string' ? frontmatter.status : 'pending';
+  } catch { return 'pending'; }
+}
+
 function _collectMilestones(projectRoot) {
   const out = [];
   for (const m of listMilestones(projectRoot)) {
@@ -49,93 +53,34 @@ function _collectMilestones(projectRoot) {
     for (const s of listSlices(m.number, projectRoot)) {
       const tasks = listTasks(m.number, s.number, projectRoot);
       const counts = { total: 0, pending: 0, 'in-progress': 0, done: 0, skipped: 0, parked: 0 };
+      const statuses = [];
       for (const t of tasks) {
-        const raw = _safeReadFile(t.plan_path);
-        if (!raw) continue;
-        let fm;
-        try { ({ frontmatter: fm } = extractFrontmatter(raw)); } catch { fm = {}; }
-        const status = typeof fm.status === 'string' ? fm.status : 'pending';
+        const status = _taskStatus(t.plan_path);
+        statuses.push(status);
         counts.total += 1;
         if (Object.prototype.hasOwnProperty.call(counts, status)) counts[status] += 1;
       }
       slices.push({
         id: s.id,
         full_id: s.full_id,
-        path: s.path,
         counts,
-        tasks_statuses: tasks.map((t) => {
-          const raw = _safeReadFile(t.plan_path);
-          if (!raw) return 'pending';
-          try { return extractFrontmatter(raw).frontmatter.status || 'pending'; }
-          catch { return 'pending'; }
-        }),
+        task_statuses: statuses,
       });
     }
     out.push({
       id: m.id,
       number: m.number,
-      name: (meta && typeof meta.name === 'string') ? meta.name : null,
-      status: (meta && typeof meta.status === 'string') ? meta.status : null,
+      name: typeof meta.name === 'string' ? meta.name : null,
+      status: typeof meta.status === 'string' ? meta.status : null,
       slices,
     });
   }
   return out;
 }
 
-function _readState(projectRoot) {
-  const p = path.join(projectStateDir(projectRoot), 'STATE.md');
-  const raw = _safeReadFile(p);
-  if (!raw) return { current_milestone: null, current_task: null };
-  try {
-    const { frontmatter } = extractFrontmatter(raw);
-    return {
-      current_milestone: frontmatter.current_milestone || null,
-      current_task: frontmatter.current_task || null,
-    };
-  } catch {
-    return { current_milestone: null, current_task: null };
-  }
-}
-
 function collectSnapshot(projectRoot) {
   const cwd = projectRoot || process.cwd();
-  const git = (() => { try { return workspaceGitInfo(cwd); } catch { return { is_repo: false }; } })();
-  const state = _readState(cwd);
-  const milestones = _collectMilestones(cwd);
-  const worktrees = (() => { try { return listSliceWorktrees(cwd); } catch { return []; } })();
-  const handoffs = (() => {
-    try { return listHandoffs({}, cwd); } catch { return []; }
-  })();
-  const openHandoffs = handoffs.filter((h) => h.status === 'open');
-  return {
-    generated_at: new Date().toISOString(),
-    project_root: cwd,
-    git,
-    state,
-    milestones,
-    worktrees,
-    handoffs: {
-      total: handoffs.length,
-      open: openHandoffs.length,
-      recent: handoffs.slice(-5).reverse(),
-    },
-    worktree_isolation: worktreeIsolationEnabled(cwd),
-  };
-}
-
-function _pad(s, n) {
-  const str = String(s);
-  if (str.length >= n) return str;
-  return str + ' '.repeat(n - str.length);
-}
-
-function _sliceBar(statuses, useColor) {
-  const parts = [];
-  for (const s of statuses) {
-    const g = STATUS_GLYPHS[s] || STATUS_GLYPHS.pending;
-    parts.push(useColor ? g.color + g.glyph + ANSI.reset : g.glyph);
-  }
-  return parts.join(' ');
+  return { milestones: _collectMilestones(cwd) };
 }
 
 function _summarizeCounts(c, useColor) {
@@ -149,73 +94,46 @@ function _summarizeCounts(c, useColor) {
   return bits.join(' · ') || paint(ANSI.dim, 'no tasks');
 }
 
+function _checkboxRow(statuses, useColor) {
+  const parts = [];
+  for (const s of statuses) {
+    const g = STATUS_GLYPHS[s] || STATUS_GLYPHS.pending;
+    parts.push(useColor ? g.color + g.glyph + ANSI.reset : g.glyph);
+  }
+  return parts.join(' ');
+}
+
 function renderSnapshot(snap, opts) {
   const o = opts || {};
   const useColor = o.color !== false;
-  const lines = [];
   const c = (code, text) => useColor ? code + text + ANSI.reset : text;
+  const lines = [];
 
-  const title = 'nubos-pilot';
-  const branch = (snap.git && snap.git.current_branch) ? snap.git.current_branch : '(no git)';
-  lines.push(c(ANSI.bold + ANSI.blue, title) + '  ' + c(ANSI.dim, snap.project_root));
-  lines.push(c(ANSI.dim, 'Branch: ' + branch + '   ·   Generated: ' + snap.generated_at));
-  if (snap.worktree_isolation) {
-    lines.push(c(ANSI.cyan, 'Worktree isolation: on') + c(ANSI.dim, '   (' + snap.worktrees.length + ' active)'));
-  } else {
-    lines.push(c(ANSI.dim, 'Worktree isolation: off'));
-  }
+  lines.push(c(ANSI.bold + ANSI.blue, 'nubos-pilot'));
   lines.push('');
 
-  if (snap.milestones.length === 0) {
+  if (!snap.milestones || snap.milestones.length === 0) {
     lines.push(c(ANSI.dim, 'No milestones yet. Run /np:new-project or /np:new-milestone.'));
-  } else {
-    lines.push(c(ANSI.bold, 'Milestones'));
-    for (const m of snap.milestones) {
-      const name = m.name ? ' — ' + m.name : '';
-      const status = m.status ? '  ' + c(ANSI.dim, '[' + m.status + ']') : '';
-      const marker = (snap.state.current_milestone === m.id) ? c(ANSI.cyan, '▶ ') : '  ';
-      lines.push(marker + c(ANSI.bold, m.id) + name + status);
-      if (m.slices.length === 0) {
-        lines.push('    ' + c(ANSI.dim, 'no slices planned'));
-      }
-      for (const s of m.slices) {
-        lines.push('    ' + c(ANSI.bold, s.full_id) + '  ' + _summarizeCounts(s.counts, useColor));
-        if (s.tasks_statuses.length > 0) {
-          lines.push('    ' + _sliceBar(s.tasks_statuses, useColor));
-        }
-      }
-      lines.push('');
-    }
-  }
-
-  if (snap.worktrees.length > 0) {
-    lines.push(c(ANSI.bold, 'Active worktrees'));
-    for (const w of snap.worktrees) {
-      lines.push('  ' + c(ANSI.cyan, w.slice_full_id) + '  ' + c(ANSI.dim, w.path));
-    }
     lines.push('');
+    return lines.join('\n');
   }
 
-  const totalH = snap.handoffs.total;
-  const openH = snap.handoffs.open;
-  lines.push(c(ANSI.bold, 'Handoffs') + c(ANSI.dim, '  (' + openH + ' open / ' + totalH + ' total)'));
-  if (snap.handoffs.recent.length === 0) {
-    lines.push('  ' + c(ANSI.dim, 'none'));
-  } else {
-    for (const h of snap.handoffs.recent) {
-      const statusColor = h.status === 'open' ? ANSI.yellow
-                        : h.status === 'acted' ? ANSI.green
-                        : h.status === 'archived' ? ANSI.dim
-                        : ANSI.reset;
-      const status = c(statusColor, _pad(h.status, 9));
-      const route = c(ANSI.cyan, _pad(h.from_agent + ' → ' + h.to_agent, 32));
-      const ms = h.milestone ? c(ANSI.dim, ' [' + h.milestone + ']') : '';
-      lines.push('  ' + status + ' ' + route + ' ' + h.topic + ms);
+  for (const m of snap.milestones) {
+    const name = m.name ? ' — ' + m.name : '';
+    const status = m.status ? '  ' + c(ANSI.dim, '[' + m.status + ']') : '';
+    lines.push(c(ANSI.bold, m.id) + name + status);
+    if (m.slices.length === 0) {
+      lines.push('  ' + c(ANSI.dim, 'no slices planned'));
+    }
+    for (const s of m.slices) {
+      lines.push('  ' + c(ANSI.bold, s.full_id) + '  ' + _summarizeCounts(s.counts, useColor));
+      if (s.task_statuses.length > 0) {
+        lines.push('  ' + _checkboxRow(s.task_statuses, useColor));
+      }
     }
+    lines.push('');
   }
-  lines.push('');
 
-  lines.push(c(ANSI.dim, 'Refresh: re-run `np-tools.cjs dashboard` (or --watch <seconds>).'));
   return lines.join('\n');
 }
 

package/lib/dashboard.test.cjs

@@ -50,14 +50,11 @@ function _writeMeta(root, mNum, meta) {
   fs.writeFileSync(path.join(dir, mIdStr + '-META.json'), JSON.stringify(meta), 'utf-8');
 }
 
-test('DB-1: collectSnapshot returns a shape with all top-level keys', () => {
+test('DB-1: collectSnapshot returns only { milestones } shape', () => {
   const root = _sandbox();
   try {
     const snap = dashboard.collectSnapshot(root);
-    for (const key of ['generated_at', 'project_root', 'git', 'state', 'milestones', 'worktrees', 'handoffs', 'worktree_isolation']) {
-      assert.ok(key in snap, 'missing key: ' + key);
-    }
-    assert.equal(snap.project_root, root);
+    assert.deepEqual(Object.keys(snap), ['milestones']);
     assert.equal(Array.isArray(snap.milestones), true);
   } finally {
     fs.rmSync(root, { recursive: true, force: true });
@@ -82,12 +79,13 @@ test('DB-2: collectSnapshot counts task statuses per slice', () => {
     assert.deepEqual(m.slices[0].counts, {
       total: 5, pending: 1, 'in-progress': 1, done: 2, skipped: 1, parked: 0,
     });
+    assert.deepEqual(m.slices[0].task_statuses, ['done', 'done', 'in-progress', 'pending', 'skipped']);
   } finally {
     fs.rmSync(root, { recursive: true, force: true });
   }
 });
 
-test('DB-3: renderSnapshot produces a non-empty string with key headings', () => {
+test('DB-3: renderSnapshot prints milestone, slice, checkbox row', () => {
   const root = _sandbox();
   try {
     _writeMeta(root, 1, { name: 'Auth', status: 'active' });
@@ -95,12 +93,13 @@ test('DB-3: renderSnapshot produces a non-empty string with key headings', () =>
     _writeTask(root, 1, 1, 2, 'pending', 'Logout');
     const snap = dashboard.collectSnapshot(root);
     const out = dashboard.renderSnapshot(snap, { color: false });
-    assert.match(out, /nubos-pilot/);
-    assert.match(out, /Milestones/);
-    assert.match(out, /M001/);
-    assert.match(out, /Auth/);
-    assert.match(out, /Handoffs/);
-    assert.match(out, /Refresh/);
+    assert.match(out, /^nubos-pilot/);
+    assert.match(out, /M001 — Auth/);
+    assert.match(out, /\[active\]/);
+    assert.match(out, /M001-S001/);
+    assert.match(out, /1 done/);
+    assert.match(out, /1 pending/);
+    assert.match(out, /\[x\] \[ \]/);
   } finally {
     fs.rmSync(root, { recursive: true, force: true });
   }
@@ -117,15 +116,14 @@ test('DB-4: renderSnapshot shows "No milestones yet" when none exist', () => {
   }
 });
 
-test('DB-5: renderSnapshot with color=false emits no ANSI escape sequences', () => {
+test('DB-5: renderSnapshot with color=false emits no ANSI codes', () => {
   const root = _sandbox();
   try {
     _writeMeta(root, 1, { name: 'Auth' });
     _writeTask(root, 1, 1, 1, 'done', 'X');
     const snap = dashboard.collectSnapshot(root);
     const out = dashboard.renderSnapshot(snap, { color: false });
-    const seen = /\x1b\[/.test(out);
-    assert.equal(seen, false, 'renderSnapshot with color=false must not emit ANSI codes');
+    assert.equal(/\x1b\[/.test(out), false, 'render with color=false must not emit ANSI');
   } finally {
     fs.rmSync(root, { recursive: true, force: true });
   }
@@ -150,32 +148,31 @@ test('DB-7: STATUS_GLYPHS covers all task-status enum values', () => {
   }
 });
 
-test('DB-8: collectSnapshot falls back to empty handoffs list on unreadable directory', () => {
+test('DB-8: empty slice (no tasks) renders "no tasks" indicator', () => {
   const root = _sandbox();
   try {
+    _writeMeta(root, 1, { name: 'Empty' });
+    fs.mkdirSync(path.join(root, '.nubos-pilot', 'milestones', 'M001', 'slices', 'S001', 'tasks'), { recursive: true });
     const snap = dashboard.collectSnapshot(root);
-    assert.equal(snap.handoffs.total, 0);
-    assert.equal(snap.handoffs.open, 0);
-    assert.deepEqual(snap.handoffs.recent, []);
+    const out = dashboard.renderSnapshot(snap, { color: false });
+    assert.match(out, /M001-S001/);
+    assert.match(out, /no tasks/);
   } finally {
     fs.rmSync(root, { recursive: true, force: true });
   }
 });
 
-test('DB-9: renderSnapshot shows worktree-isolation flag correctly', () => {
+test('DB-9: multiple milestones render in numeric order', () => {
   const root = _sandbox();
   try {
-    const snap1 = dashboard.collectSnapshot(root);
-    const out1 = dashboard.renderSnapshot(snap1, { color: false });
-    assert.match(out1, /Worktree isolation: off/);
-    fs.writeFileSync(
-      path.join(root, '.nubos-pilot', 'config.json'),
-      JSON.stringify({ workflow: { worktree_isolation: true } }),
-      'utf-8',
-    );
-    const snap2 = dashboard.collectSnapshot(root);
-    const out2 = dashboard.renderSnapshot(snap2, { color: false });
-    assert.match(out2, /Worktree isolation: on/);
+    _writeMeta(root, 2, { name: 'Second' });
+    _writeMeta(root, 1, { name: 'First' });
+    _writeTask(root, 1, 1, 1, 'done', 'X');
+    _writeTask(root, 2, 1, 1, 'pending', 'Y');
+    const snap = dashboard.collectSnapshot(root);
+    assert.equal(snap.milestones.length, 2);
+    assert.equal(snap.milestones[0].id, 'M001');
+    assert.equal(snap.milestones[1].id, 'M002');
   } finally {
     fs.rmSync(root, { recursive: true, force: true });
   }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nubos-pilot",
-  "version": "0.7.1",
+  "version": "0.7.2",
   "description": "AI-driven planning and execution tool for code projects",
   "homepage": "https://github.com/Nubos-AI/nubos-pilot",
   "repository": {

package/workflows/dashboard.md

@@ -0,0 +1,49 @@
+---
+command: np:dashboard
+description: One-shot console dashboard of milestones, slices, and tasks. Read-only — no files written, no state mutated, no git commit.
+---
+
+# np:dashboard
+
+Read-only snapshot of milestones, slices, and task statuses for the project.
+
+```bash
+node .nubos-pilot/bin/np-tools.cjs dashboard
+```
+
+That is the entire workflow. The CLI prints the snapshot to stdout — milestone-by-milestone, with one row per slice showing per-status counts plus a checkbox row of all tasks in the slice (`[ ]` pending, `[~]` in-progress, `[x]` done, `[-]` skipped, `[!]` parked).
+
+## No Commit
+
+Read-only. No files are written, no state is mutated, no git commit is made.
+
+## Scope Guardrail
+
+<scope_guardrail>
+**Do:**
+- Run the CLI with no arguments for the formatted view, `--json` for the raw snapshot, or `--no-color` for plain text.
+- Treat the output as a render — re-run the workflow when you want a fresh view.
+
+**Don't:**
+- Add a long-running watch loop here — single-shot only, by design (ADR-0001).
+- Mutate any state from this workflow — strictly read-only.
+- Add additional sections beyond milestones / slices / tasks. Drill-down into handoffs / checkpoints / worktrees uses their dedicated commands.
+</scope_guardrail>
+
+## Output
+
+Stdout only — formatted milestone overview with checkbox rows per slice. No files created. No state mutated. No git commit.
+
+## Success Criteria
+
+- [ ] Empty project renders the "No milestones yet" placeholder line.
+- [ ] Every milestone in `roadmap.yaml` appears with its name and status.
+- [ ] Every slice's checkbox row reflects current task frontmatter `status` values.
+- [ ] `--json` emits the snapshot shape `{ milestones: [{ id, number, name, status, slices: [{ id, full_id, counts, task_statuses }] }] }`.
+- [ ] `--no-color` emits no ANSI escape sequences.
+- [ ] Zero file writes, zero state mutations, zero commits.
+
+## Related Workflows
+
+- **`/np:stats`** — phases-table + metrics aggregation (commit-history view).
+- **`/np:state`** — current STATE.md frontmatter snapshot.