@sienklogic/plan-build-run 2.11.0 → 2.12.1

package/dashboard/src/routes/pages.routes.js

@@ -1,308 +0,0 @@
-import { Router } from 'express';
-import { getPhaseDetail, getPhaseDocument } from '../services/phase.service.js';
-import { getRoadmapData } from '../services/roadmap.service.js';
-import { parseStateFile, derivePhaseStatuses } from '../services/dashboard.service.js';
-import { listPendingTodos, getTodoDetail, createTodo, completeTodo } from '../services/todo.service.js';
-import { getAllMilestones, getMilestoneDetail } from '../services/milestone.service.js';
-
-const router = Router();
-
-router.get('/phases', async (req, res) => {
-  const projectDir = req.app.locals.projectDir;
-  const [roadmapData, stateData] = await Promise.all([
-    getRoadmapData(projectDir),
-    parseStateFile(projectDir)
-  ]);
-
-  const templateData = {
-    title: 'Phases',
-    activePage: 'phases',
-    currentPath: '/phases',
-    phases: derivePhaseStatuses(roadmapData.phases, stateData.currentPhase),
-    milestones: roadmapData.milestones
-  };
-
-  res.setHeader('Vary', 'HX-Request');
-
-  if (req.get('HX-Request') === 'true') {
-    res.render('partials/phases-content', templateData);
-  } else {
-    res.render('phases', templateData);
-  }
-});
-
-router.get('/phases/:phaseId', async (req, res) => {
-  const { phaseId } = req.params;
-
-  // Validate phaseId: two digits, optionally followed by decimal (e.g., 01, 05, 3.1)
-  if (!/^\d{1,2}(\.\d+)?$/.test(phaseId)) {
-    const err = new Error('Phase ID must be a number (e.g., 01, 05, 3.1)');
-    err.status = 404;
-    throw err;
-  }
-
-  const projectDir = req.app.locals.projectDir;
-  const phaseData = await getPhaseDetail(projectDir, phaseId);
-
-  const templateData = {
-    title: `Phase ${phaseId}: ${phaseData.phaseName}`,
-    activePage: 'phases',
-    currentPath: '/phases/' + phaseId,
-    ...phaseData
-  };
-
-  res.setHeader('Vary', 'HX-Request');
-
-  if (req.get('HX-Request') === 'true') {
-    res.render('partials/phase-content', templateData);
-  } else {
-    res.render('phase-detail', templateData);
-  }
-});
-
-router.get('/phases/:phaseId/:planId/:docType', async (req, res) => {
-  const { phaseId, planId, docType } = req.params;
-
-  // Validate phaseId
-  if (!/^\d{1,2}(\.\d+)?$/.test(phaseId)) {
-    const err = new Error('Phase ID must be a number (e.g., 01, 05, 3.1)');
-    err.status = 404;
-    throw err;
-  }
-
-  // Validate planId: NN-NN format
-  if (!/^\d{2}-\d{2}$/.test(planId)) {
-    const err = new Error('Plan ID must be in NN-NN format (e.g., 04-01)');
-    err.status = 404;
-    throw err;
-  }
-
-  // Validate docType
-  if (docType !== 'plan' && docType !== 'summary') {
-    const err = new Error('Document type must be "plan" or "summary"');
-    err.status = 404;
-    throw err;
-  }
-
-  const projectDir = req.app.locals.projectDir;
-  const doc = await getPhaseDocument(projectDir, phaseId, planId, docType);
-
-  if (!doc) {
-    const err = new Error(`${docType === 'plan' ? 'Plan' : 'Summary'} ${planId} not found for phase ${phaseId}`);
-    err.status = 404;
-    throw err;
-  }
-
-  const docLabel = docType === 'plan' ? 'Plan' : 'Summary';
-  const templateData = {
-    title: `${docLabel} ${planId} — Phase ${phaseId}: ${doc.phaseName}`,
-    activePage: 'phases',
-    currentPath: `/phases/${phaseId}/${planId}/${docType}`,
-    ...doc
-  };
-
-  res.setHeader('Vary', 'HX-Request');
-
-  if (req.get('HX-Request') === 'true') {
-    res.render('partials/phase-doc-content', templateData);
-  } else {
-    res.render('phase-doc', templateData);
-  }
-});
-
-router.get('/todos', async (req, res) => {
-  const projectDir = req.app.locals.projectDir;
-  const todos = await listPendingTodos(projectDir);
-
-  const templateData = {
-    title: 'Todos',
-    activePage: 'todos',
-    currentPath: '/todos',
-    todos
-  };
-
-  res.setHeader('Vary', 'HX-Request');
-
-  if (req.get('HX-Request') === 'true') {
-    res.render('partials/todos-content', templateData);
-  } else {
-    res.render('todos', templateData);
-  }
-});
-
-router.get('/todos/new', (req, res) => {
-  const templateData = {
-    title: 'Create Todo',
-    activePage: 'todos',
-    currentPath: '/todos/new'
-  };
-
-  res.setHeader('Vary', 'HX-Request');
-
-  if (req.get('HX-Request') === 'true') {
-    res.render('partials/todo-create-content', templateData);
-  } else {
-    res.render('todo-create', templateData);
-  }
-});
-
-router.get('/todos/:id', async (req, res) => {
-  const { id } = req.params;
-
-  // Validate ID format: must be exactly three digits
-  if (!/^\d{3}$/.test(id)) {
-    const err = new Error('Todo ID must be a three-digit number (e.g., 001, 005, 042)');
-    err.status = 404;
-    throw err;
-  }
-
-  const projectDir = req.app.locals.projectDir;
-  const todo = await getTodoDetail(projectDir, id);
-
-  const templateData = {
-    title: `Todo ${todo.id}: ${todo.title}`,
-    activePage: 'todos',
-    currentPath: '/todos/' + id,
-    ...todo
-  };
-
-  res.setHeader('Vary', 'HX-Request');
-
-  if (req.get('HX-Request') === 'true') {
-    res.render('partials/todo-detail-content', templateData);
-  } else {
-    res.render('todo-detail', templateData);
-  }
-});
-
-router.post('/todos', async (req, res) => {
-  const { title, priority, phase, description } = req.body;
-  const projectDir = req.app.locals.projectDir;
-
-  const todoId = await createTodo(projectDir, {
-    title,
-    priority,
-    phase: phase || '',
-    description
-  });
-
-  if (req.get('HX-Request') === 'true') {
-    // For HTMX: fetch the new todo and render its detail as a fragment
-    const todo = await getTodoDetail(projectDir, todoId);
-    res.render('partials/todo-detail-content', {
-      title: `Todo ${todo.id}: ${todo.title}`,
-      activePage: 'todos',
-      currentPath: '/todos/' + todoId,
-      ...todo
-    });
-  } else {
-    res.redirect(`/todos/${todoId}`);
-  }
-});
-
-router.post('/todos/:id/done', async (req, res) => {
-  const { id } = req.params;
-
-  if (!/^\d{3}$/.test(id)) {
-    const err = new Error('Todo ID must be a three-digit number');
-    err.status = 404;
-    throw err;
-  }
-
-  const projectDir = req.app.locals.projectDir;
-  await completeTodo(projectDir, id);
-
-  if (req.get('HX-Request') === 'true') {
-    // For HTMX: re-render the full todo list as a fragment
-    const todos = await listPendingTodos(projectDir);
-    res.render('partials/todos-content', {
-      title: 'Todos',
-      activePage: 'todos',
-      currentPath: '/todos',
-      todos
-    });
-  } else {
-    res.redirect('/todos');
-  }
-});
-
-router.get('/milestones', async (req, res) => {
-  const projectDir = req.app.locals.projectDir;
-  const milestoneData = await getAllMilestones(projectDir);
-
-  const templateData = {
-    title: 'Milestones',
-    activePage: 'milestones',
-    currentPath: '/milestones',
-    ...milestoneData
-  };
-
-  res.setHeader('Vary', 'HX-Request');
-
-  if (req.get('HX-Request') === 'true') {
-    res.render('partials/milestones-content', templateData);
-  } else {
-    res.render('milestones', templateData);
-  }
-});
-
-router.get('/milestones/:version', async (req, res) => {
-  const { version } = req.params;
-
-  // Validate version: alphanumeric with dots and dashes
-  if (!/^[\w.-]+$/.test(version)) {
-    const err = new Error('Invalid milestone version format');
-    err.status = 404;
-    throw err;
-  }
-
-  const projectDir = req.app.locals.projectDir;
-  const detail = await getMilestoneDetail(projectDir, version);
-
-  if (detail.sections.length === 0) {
-    const err = new Error(`No archived files found for milestone v${version}`);
-    err.status = 404;
-    throw err;
-  }
-
-  const templateData = {
-    title: `Milestone v${version}`,
-    activePage: 'milestones',
-    currentPath: '/milestones/' + version,
-    ...detail
-  };
-
-  res.setHeader('Vary', 'HX-Request');
-
-  if (req.get('HX-Request') === 'true') {
-    res.render('partials/milestone-detail-content', templateData);
-  } else {
-    res.render('milestone-detail', templateData);
-  }
-});
-
-router.get('/roadmap', async (req, res) => {
-  const projectDir = req.app.locals.projectDir;
-  const [roadmapData, stateData] = await Promise.all([
-    getRoadmapData(projectDir),
-    parseStateFile(projectDir)
-  ]);
-
-  const templateData = {
-    title: 'Roadmap',
-    activePage: 'roadmap',
-    currentPath: '/roadmap',
-    phases: derivePhaseStatuses(roadmapData.phases, stateData.currentPhase),
-    milestones: roadmapData.milestones
-  };
-
-  res.setHeader('Vary', 'HX-Request');
-
-  if (req.get('HX-Request') === 'true') {
-    res.render('partials/roadmap-content', templateData);
-  } else {
-    res.render('roadmap', templateData);
-  }
-});
-
-export default router;

package/dashboard/src/server.js

@@ -1,42 +0,0 @@
-import { createApp } from './app.js';
-import { createWatcher } from './services/watcher.service.js';
-import { broadcast } from './services/sse.service.js';
-
-export function startServer(config) {
-  const app = createApp(config);
-  const { port, projectDir } = config;
-
-  // Start file watcher for live updates
-  const watcher = createWatcher(projectDir, (event) => {
-    broadcast('file-change', event);
-  });
-
-  const server = app.listen(port, '127.0.0.1', () => {
-    console.log(`PBR Dashboard running at http://127.0.0.1:${port}`);
-    console.log(`Project directory: ${projectDir}`);
-    console.log('File watcher active on .planning/**/*.md');
-  });
-
-  const shutdown = async (signal) => {
-    console.log(`${signal} received. Shutting down PBR Dashboard...`);
-
-    // Close watcher first (stops generating events)
-    try {
-      await watcher.close();
-      console.log('File watcher closed.');
-    } catch (err) {
-      console.error('Error closing watcher:', err.message);
-    }
-
-    // Then close the HTTP server
-    server.close(() => {
-      console.log('Server closed.');
-      process.exit(0);
-    });
-  };
-
-  process.on('SIGTERM', () => shutdown('SIGTERM'));
-  process.on('SIGINT', () => shutdown('SIGINT'));
-
-  return server;
-}

package/dashboard/src/services/dashboard.service.js

@@ -1,309 +0,0 @@
-import { readFile } from 'node:fs/promises';
-import { join } from 'node:path';
-
-/**
- * Strip UTF-8 BOM from file content.
- * Duplicated from planning.repository.js intentionally --
- * this service reads raw text, not via the repository layer.
- *
- * @param {string} content - Raw file content
- * @returns {string} Content without BOM
- */
-function stripBOM(content) {
-  return content.replace(/^\uFEFF/, '');
-}
-
-/**
- * Parse STATE.md to extract project status information.
- * Uses regex on raw markdown body text (not YAML frontmatter).
- *
- * @param {string} projectDir - Absolute path to the project root
- * @returns {Promise<{projectName: string, currentPhase: object, lastActivity: object, progress: number}>}
- */
-export async function parseStateFile(projectDir) {
-  try {
-    const path = join(projectDir, '.planning', 'STATE.md');
-    const raw = await readFile(path, 'utf-8');
-    const content = stripBOM(raw).replace(/\r\n/g, '\n');
-
-    // Parse YAML frontmatter if present (STATE.md v2 format)
-    let frontmatter = {};
-    const fmMatch = content.match(/^---\n([\s\S]*?)\n---/);
-    if (fmMatch) {
-      for (const line of fmMatch[1].split('\n')) {
-        const kv = line.match(/^(\w+):\s*(.+)/);
-        if (kv) {
-          const val = kv[2].trim().replace(/^"(.*)"$/, '$1');
-          frontmatter[kv[1]] = val;
-        }
-      }
-    }
-
-    // Extract project name from **Current focus:** line
-    let projectName = 'Unknown Project';
-    const focusMatch = content.match(/\*\*Current focus:\*\*\s*(.+)/);
-    if (focusMatch) {
-      projectName = focusMatch[1].trim();
-    } else {
-      // Fallback: try H1 heading
-      const h1Match = content.match(/^# (.+)/m);
-      if (h1Match) {
-        projectName = h1Match[1].trim();
-      }
-    }
-
-    // Extract current phase: "Phase: 3 of 12 (UI Shell)"
-    let currentPhaseId = 0;
-    let totalPhases = 0;
-    let phaseName = 'Not Started';
-    const phaseMatch = content.match(/Phase:\s*(\d+)\s*of\s*(\d+)\s*\(([^)]+)\)/);
-    if (phaseMatch) {
-      currentPhaseId = parseInt(phaseMatch[1], 10);
-      totalPhases = parseInt(phaseMatch[2], 10);
-      phaseName = phaseMatch[3].trim();
-    }
-
-    // Extract plan status: "Plan: 2 of 2 complete"
-    let planStatus = 'N/A';
-    const planMatch = content.match(/Plan:\s*(.+)/);
-    if (planMatch) {
-      planStatus = planMatch[1].trim();
-    }
-
-    // Extract last activity: "Last activity: 2026-02-08 -- Phase 3 built (...)"
-    let activityDate = '';
-    let activityDescription = 'No activity recorded';
-    const activityMatch = content.match(/Last activity:\s*([\d-]+)\s*--\s*(.+)/);
-    if (activityMatch) {
-      activityDate = activityMatch[1].trim();
-      activityDescription = activityMatch[2].trim();
-    }
-
-    // Extract progress percentage: "Progress: [...] 25%"
-    let progress = 0;
-    const progressMatch = content.match(/Progress:.*?(\d+)%/);
-    if (progressMatch) {
-      progress = parseInt(progressMatch[1], 10);
-    } else if (totalPhases > 0) {
-      // Calculate from phase numbers if no explicit progress
-      progress = Math.ceil((currentPhaseId / totalPhases) * 100);
-    }
-
-    // Determine phase status from frontmatter or body text
-    // "built", "verified", "complete" all mean the current phase is done
-    const fmStatus = (frontmatter.status || '').toLowerCase();
-    const phaseStatus = ['built', 'verified', 'complete', 'reviewed'].includes(fmStatus)
-      ? 'complete'
-      : ['building', 'planning', 'planned', 'in-progress'].includes(fmStatus)
-        ? 'in-progress'
-        : fmStatus || 'unknown';
-
-    return {
-      projectName,
-      currentPhase: {
-        id: currentPhaseId,
-        total: totalPhases,
-        name: phaseName,
-        planStatus,
-        status: phaseStatus
-      },
-      lastActivity: {
-        date: activityDate,
-        description: activityDescription
-      },
-      progress
-    };
-  } catch (error) {
-    if (error.code === 'ENOENT') {
-      return {
-        projectName: 'Unknown Project',
-        currentPhase: { id: 0, total: 0, name: 'Not Started', planStatus: 'N/A', status: 'unknown' },
-        lastActivity: { date: '', description: 'No activity recorded' },
-        progress: 0
-      };
-    }
-    throw error;
-  }
-}
-
-/**
- * Parse ROADMAP.md to extract phase list from Progress table and checkbox list.
- * Uses the Progress table as primary source (contains ALL phases including those
- * added under milestone headings). Falls back to checkbox parsing for older
- * roadmaps that don't have a Progress table.
- *
- * @param {string} projectDir - Absolute path to the project root
- * @returns {Promise<{phases: Array<{id: number, name: string, description: string, status: string}>, progress: number}>}
- */
-export async function parseRoadmapFile(projectDir) {
-  try {
-    const filePath = join(projectDir, '.planning', 'ROADMAP.md');
-    const raw = await readFile(filePath, 'utf-8');
-    const content = stripBOM(raw).replace(/\r\n/g, '\n');
-
-    // 1. Parse descriptions from checkbox list: "- [x] Phase 01: Name -- Description"
-    const checkboxMap = new Map();
-    const checkboxRegex = /^- \[([ xX])\] Phase (\d+):\s*([^-]+?)\s*--\s*(.+)$/gm;
-    for (const match of content.matchAll(checkboxRegex)) {
-      const id = parseInt(match[2], 10);
-      checkboxMap.set(id, {
-        name: match[3].trim(),
-        description: match[4].trim(),
-        status: (match[1] === 'x' || match[1] === 'X') ? 'complete' : 'not-started'
-      });
-    }
-
-    // 2. Parse descriptions from Phase Details: "### Phase NN: Name\n**Goal**: ..."
-    const goalMap = new Map();
-    const goalRegex = /### Phase (\d+):\s*(.+)\n\*\*Goal:?\*\*:?\s*(.+)/g;
-    for (const match of content.matchAll(goalRegex)) {
-      goalMap.set(parseInt(match[1], 10), match[3].trim());
-    }
-
-    // 3. Parse Progress table: "| 01. Name | 2/2 | Complete | 2026-02-08 |"
-    const progressRegex = /^\|\s*(\d+)\.\s+(.+?)\s*\|\s*(\d+)\/([\d?]+)\s*\|\s*(.+?)\s*\|\s*(.+?)\s*\|$/gm;
-    const progressPhases = [];
-    for (const match of content.matchAll(progressRegex)) {
-      const id = parseInt(match[1], 10);
-      const name = match[2].trim();
-      const statusText = match[5].trim().toLowerCase();
-
-      // Get description: prefer checkbox description, fall back to Phase Details Goal
-      const cbInfo = checkboxMap.get(id);
-      const goalDesc = goalMap.get(id);
-      const description = cbInfo?.description || goalDesc || '';
-
-      let status;
-      if (statusText === 'complete') status = 'complete';
-      else if (statusText === 'in progress') status = 'in-progress';
-      else status = 'not-started';
-
-      progressPhases.push({ id, name, description, status });
-    }
-
-    // 4. Parse H3 Phase Details: "### Phase N: Name\n**Goal:** ..."
-    const h3Phases = [];
-    const h3Regex = /### Phase (\d+):\s*(.+)\n\*\*Goal\*?\*?:?\*?\*?\s*(.+)/g;
-    for (const match of content.matchAll(h3Regex)) {
-      const id = parseInt(match[1], 10);
-      const name = match[2].trim();
-      const description = match[3].trim();
-      // Check checkbox map for status, default to not-started
-      const cbInfo = checkboxMap.get(id);
-      h3Phases.push({
-        id,
-        name,
-        description,
-        status: cbInfo?.status || 'not-started'
-      });
-    }
-
-    // 5. Parse simple two-column table: "| 1. Name | Completed |"
-    //    Used in milestone-archived roadmaps
-    const simpleTableRegex = /^\|\s*(\d+)\.\s+(.+?)\s*\|\s*(.+?)\s*\|$/gm;
-    const simplePhases = [];
-    for (const match of content.matchAll(simpleTableRegex)) {
-      const id = parseInt(match[1], 10);
-      const name = match[2].trim();
-      const statusText = match[3].trim().toLowerCase();
-
-      let status;
-      if (statusText === 'completed' || statusText === 'complete') status = 'complete';
-      else if (statusText === 'in progress' || statusText === 'in-progress') status = 'in-progress';
-      else status = 'not-started';
-
-      const cbInfo = checkboxMap.get(id);
-      const goalDesc = goalMap.get(id);
-      const description = cbInfo?.description || goalDesc || '';
-
-      simplePhases.push({ id, name, description, status });
-    }
-
-    // Use Progress table if available, then simple table, then checkbox list, then H3 headings
-    let phases;
-    if (progressPhases.length > 0) {
-      phases = progressPhases;
-    } else if (simplePhases.length > 0) {
-      phases = simplePhases;
-    } else if (checkboxMap.size > 0) {
-      phases = [...checkboxMap.entries()]
-        .map(([id, info]) => ({
-          id,
-          name: info.name,
-          description: info.description,
-          status: info.status
-        }))
-        .sort((a, b) => a.id - b.id);
-    } else {
-      phases = h3Phases.sort((a, b) => a.id - b.id);
-    }
-
-    const completed = phases.filter(p => p.status === 'complete').length;
-    const total = phases.length;
-    const progress = total > 0 ? Math.ceil((completed / total) * 100) : 0;
-
-    return { phases, progress };
-  } catch (error) {
-    if (error.code === 'ENOENT') {
-      return { phases: [], progress: 0 };
-    }
-    throw error;
-  }
-}
-
-/**
- * Get combined dashboard data by parsing both STATE.md and ROADMAP.md.
- * Orchestrates both parsers in parallel and derives in-progress status.
- *
- * @param {string} projectDir - Absolute path to the project root
- * @returns {Promise<{projectName: string, currentPhase: object, lastActivity: object, progress: number, phases: Array}>}
- */
-/**
- * Derive phase statuses by combining roadmap phases with STATE.md context.
- * Phases before the current phase are marked complete.
- * The current phase gets its status from STATE.md.
- * Phases after the current phase keep their roadmap status (typically not-started).
- *
- * @param {Array} phases - Raw phases from parseRoadmapFile
- * @param {{id: number, status: string}} currentPhase - Current phase from parseStateFile
- * @returns {Array} Phases with derived statuses
- */
-export function derivePhaseStatuses(phases, currentPhase) {
-  const currentId = currentPhase.id;
-  const currentStatus = currentPhase.status || 'unknown';
-  return phases.map(phase => {
-    // If the roadmap already has explicit status (from progress table/checkboxes), keep it
-    if (phase.status === 'complete') return phase;
-
-    if (phase.id < currentId) {
-      return { ...phase, status: 'complete' };
-    }
-    if (phase.id === currentId) {
-      return { ...phase, status: currentStatus === 'complete' ? 'complete' : 'in-progress' };
-    }
-    return phase;
-  });
-}
-
-export async function getDashboardData(projectDir) {
-  const [stateData, roadmapData] = await Promise.all([
-    parseStateFile(projectDir),
-    parseRoadmapFile(projectDir)
-  ]);
-
-  const phases = derivePhaseStatuses(roadmapData.phases, stateData.currentPhase);
-
-  // Recalculate progress from derived phase statuses
-  const completedPhases = phases.filter(p => p.status === 'complete').length;
-  const progress = phases.length > 0
-    ? Math.ceil((completedPhases / phases.length) * 100)
-    : stateData.progress;
-
-  return {
-    projectName: stateData.projectName,
-    currentPhase: stateData.currentPhase,
-    lastActivity: stateData.lastActivity,
-    progress,
-    phases
-  };
-}