@sienklogic/plan-build-run 2.0.0 → 2.0.2

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

@@ -1,195 +1,245 @@
-import { Router } from 'express';
-import { getPhaseDetail } from '../services/phase.service.js';
-import { getRoadmapData } from '../services/roadmap.service.js';
-import { listPendingTodos, getTodoDetail, createTodo, completeTodo } from '../services/todo.service.js';
-
-const router = Router();
-
-router.get('/phases', async (req, res) => {
-  const projectDir = req.app.locals.projectDir;
-  const roadmapData = await getRoadmapData(projectDir);
-
-  const templateData = {
-    title: 'Phases',
-    activePage: 'phases',
-    currentPath: '/phases',
-    phases: roadmapData.phases,
-    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('/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('/roadmap', async (req, res) => {
-  const projectDir = req.app.locals.projectDir;
-  const roadmapData = await getRoadmapData(projectDir);
-
-  const templateData = {
-    title: 'Roadmap',
-    activePage: 'roadmap',
-    currentPath: '/roadmap',
-    phases: roadmapData.phases,
-    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;
+import { Router } from 'express';
+import { getPhaseDetail, getPhaseDocument } from '../services/phase.service.js';
+import { getRoadmapData } from '../services/roadmap.service.js';
+import { listPendingTodos, getTodoDetail, createTodo, completeTodo } from '../services/todo.service.js';
+
+const router = Router();
+
+router.get('/phases', async (req, res) => {
+  const projectDir = req.app.locals.projectDir;
+  const roadmapData = await getRoadmapData(projectDir);
+
+  const templateData = {
+    title: 'Phases',
+    activePage: 'phases',
+    currentPath: '/phases',
+    phases: roadmapData.phases,
+    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('/roadmap', async (req, res) => {
+  const projectDir = req.app.locals.projectDir;
+  const roadmapData = await getRoadmapData(projectDir);
+
+  const templateData = {
+    title: 'Roadmap',
+    activePage: 'roadmap',
+    currentPath: '/roadmap',
+    phases: roadmapData.phases,
+    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 +1,42 @@
-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;
-}
+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;
+}