@sienklogic/plan-build-run 2.27.0 → 2.27.1

package/dashboard/src/views/partials/todo-create-content.ejs

@@ -1,54 +1,59 @@
 <%- include('breadcrumbs', { breadcrumbs: typeof breadcrumbs !== 'undefined' ? breadcrumbs : [] }) %>
 <h1>Create Todo</h1>
 
-<p><a href="/todos">&larr; Back to Todos</a></p>
+<p><a href="/todos" class="btn btn-outline-secondary">&larr; Back to Todos</a></p>
 
-<article>
-  <form method="POST" action="/todos"
-        hx-post="/todos"
-        hx-target="#main-content">
-    <label>
-      Title
-      <input
-        type="text"
-        name="title"
-        placeholder="Todo title"
-        required
-        maxlength="200"
-      />
-    </label>
+<div class="card">
+  <div class="card-body">
+    <form method="POST" action="/todos"
+          hx-post="/todos"
+          hx-target="#main-content">
+      <div class="mb-3">
+        <label class="form-label">Title</label>
+        <input
+          type="text"
+          name="title"
+          placeholder="Todo title"
+          required
+          maxlength="200"
+          class="form-control"
+        />
+      </div>
 
-    <label>
-      Priority
-      <select name="priority" required>
-        <option value="">Select priority...</option>
-        <option value="P0">P0 - Critical</option>
-        <option value="P1">P1 - High</option>
-        <option value="P2">P2 - Medium</option>
-        <option value="PX">PX - Low</option>
-      </select>
-    </label>
+      <div class="mb-3">
+        <label class="form-label">Priority</label>
+        <select name="priority" required class="form-select">
+          <option value="">Select priority...</option>
+          <option value="P0">P0 - Critical</option>
+          <option value="P1">P1 - High</option>
+          <option value="P2">P2 - Medium</option>
+          <option value="PX">PX - Low</option>
+        </select>
+      </div>
 
-    <label>
-      Phase (optional)
-      <input
-        type="text"
-        name="phase"
-        placeholder="e.g., 09-todo-write-operations"
-      />
-      <small>Format: 01-phase-name</small>
-    </label>
+      <div class="mb-3">
+        <label class="form-label">Phase (optional)</label>
+        <input
+          type="text"
+          name="phase"
+          placeholder="e.g., 09-todo-write-operations"
+          class="form-control"
+        />
+        <small class="form-text text-muted">Format: 01-phase-name</small>
+      </div>
 
-    <label>
-      Description
-      <textarea
-        name="description"
-        rows="10"
-        required
-        placeholder="Markdown content..."
-      ></textarea>
-    </label>
+      <div class="mb-3">
+        <label class="form-label">Description</label>
+        <textarea
+          name="description"
+          rows="10"
+          required
+          placeholder="Markdown content..."
+          class="form-control"
+        ></textarea>
+      </div>
 
-    <button type="submit">Create Todo</button>
-  </form>
-</article>
+      <button type="submit" class="btn btn-primary">Create Todo</button>
+    </form>
+  </div>
+</div>

package/dashboard/src/views/partials/todo-detail-content.ejs

@@ -3,8 +3,8 @@
 
 <p><a href="/todos">&larr; Back to Todos</a></p>
 
-<article>
-  <header>
+<div class="card mb-3">
+  <div class="card-header">
     <strong>Todo <%= id %></strong>
     &nbsp;
     <span class="status-badge" data-priority="<%= priority %>">
@@ -14,29 +14,30 @@
     <span class="status-badge" data-status="<%= status %>">
       <%= status %>
     </span>
-  </header>
+  </div>
+  <div class="card-body">
+    <% if (phase) { %>
+    <p><strong>Phase:</strong> <%= phase %></p>
+    <% } %>
+    <% if (created) { %>
+    <p><strong>Created:</strong> <%= created %></p>
+    <% } %>
 
-  <% if (phase) { %>
-  <p><strong>Phase:</strong> <%= phase %></p>
-  <% } %>
-  <% if (created) { %>
-  <p><strong>Created:</strong> <%= created %></p>
-  <% } %>
+    <hr>
 
-  <hr>
+    <%- html %>
 
-  <%- html %>
-
-  <% if (status === 'pending') { %>
-  <hr>
-  <form method="POST" action="/todos/<%= id %>/done"
-        hx-post="/todos/<%= id %>/done"
-        hx-target="closest article"
-        hx-swap="outerHTML"
-        hx-indicator="#done-spinner"
-        style="display:inline;">
-    <button type="submit" class="secondary">Mark as Done</button>
-    <span id="done-spinner" class="htmx-indicator" aria-busy="true"></span>
-  </form>
-  <% } %>
-</article>
+    <% if (status === 'pending') { %>
+    <hr>
+    <form method="POST" action="/todos/<%= id %>/done"
+          hx-post="/todos/<%= id %>/done"
+          hx-target="closest .card"
+          hx-swap="outerHTML"
+          hx-indicator="#done-spinner"
+          style="display:inline;">
+      <button type="submit" class="btn btn-outline-secondary">Mark as Done</button>
+      <span id="done-spinner" class="spinner-border spinner-border-sm htmx-indicator" role="status"></span>
+    </form>
+    <% } %>
+  </div>
+</div>

package/dashboard/src/views/partials/todos-content.ejs

@@ -2,7 +2,7 @@
 <h1>Todos</h1>
 
 <p>
-  <a href="/todos/new" role="button"
+  <a href="/todos/new" class="btn btn-primary"
       hx-get="/todos/new"
       hx-target="#main-content"
       hx-push-url="true">Create Todo</a>
@@ -14,12 +14,12 @@
 </p>
 
 <% const f = typeof filters !== 'undefined' ? filters : { priority: '', status: '', q: '' }; %>
-<article>
-  <form hx-get="/todos" hx-target="#main-content" hx-push-url="true">
-    <div class="grid">
-      <label>
-        Priority
-        <select name="priority" onchange="this.form.requestSubmit()">
+<form hx-get="/todos" hx-target="#main-content" hx-push-url="true" class="card mb-3">
+  <div class="card-body">
+    <div class="row g-2">
+      <div class="col-auto">
+        <label class="form-label">Priority</label>
+        <select name="priority" class="form-select" onchange="this.form.requestSubmit()">
           <option value="">All</option>
           <option value="high"<%= f.priority === 'high' ? ' selected' : '' %>>High</option>
           <option value="medium"<%= f.priority === 'medium' ? ' selected' : '' %>>Medium</option>
@@ -29,27 +29,28 @@
           <option value="P2"<%= f.priority === 'P2' ? ' selected' : '' %>>P2</option>
           <option value="PX"<%= f.priority === 'PX' ? ' selected' : '' %>>PX</option>
         </select>
-      </label>
-      <label>
-        Status
-        <select name="status" onchange="this.form.requestSubmit()">
+      </div>
+      <div class="col-auto">
+        <label class="form-label">Status</label>
+        <select name="status" class="form-select" onchange="this.form.requestSubmit()">
           <option value="">All</option>
           <option value="pending"<%= f.status === 'pending' ? ' selected' : '' %>>pending</option>
         </select>
-      </label>
-      <label>
-        Search
+      </div>
+      <div class="col-auto">
+        <label class="form-label">Search</label>
         <input type="search" name="q" value="<%= f.q %>" placeholder="Search titles..."
+               class="form-control"
                hx-get="/todos" hx-target="#main-content" hx-push-url="true"
                hx-trigger="keyup changed delay:300ms, search" hx-include="closest form">
-      </label>
+      </div>
     </div>
-  </form>
-</article>
+  </div>
+</form>
 
 <% if (todos.length > 0) { %>
 <p>
-  <button class="secondary"
+  <button class="btn btn-outline-secondary"
           hx-post="/todos/bulk-complete?priority=<%= encodeURIComponent(f.priority) %>&status=<%= encodeURIComponent(f.status) %>&q=<%= encodeURIComponent(f.q) %>"
           hx-target="#main-content"
           hx-swap="outerHTML"
@@ -57,51 +58,53 @@
           hx-confirm="Complete all <%= todos.length %> visible todo(s)?">
     Complete All Visible (<%= todos.length %>)
   </button>
-  <span id="bulk-spinner" class="htmx-indicator" aria-busy="true"></span>
+  <span id="bulk-spinner" class="spinner-border spinner-border-sm htmx-indicator" role="status"></span>
 </p>
-<article>
-  <div class="table-wrap">
-  <table>
-    <thead>
-      <tr>
-        <th scope="col">ID</th>
-        <th scope="col">Title</th>
-        <th scope="col">Priority</th>
-        <th scope="col">Phase</th>
-        <th scope="col">Status</th>
-        <th scope="col">Created</th>
-      </tr>
-    </thead>
-    <tbody>
-      <% todos.forEach(function(todo) { %>
-      <tr>
-        <td><%= todo.id %></td>
-        <td>
-          <a href="/todos/<%= todo.id %>"
-             hx-get="/todos/<%= todo.id %>"
-             hx-target="#main-content"
-             hx-push-url="true">
-            <%= todo.title %>
-          </a>
-        </td>
-        <td>
-          <span class="status-badge" data-priority="<%= todo.priority %>">
-            <%= todo.priority %>
-          </span>
-        </td>
-        <td><%= todo.phase || '—' %></td>
-        <td>
-          <span class="status-badge" data-status="<%= todo.status %>">
-            <%= todo.status %>
-          </span>
-        </td>
-        <td><%= todo.created ? new Date(todo.created).toISOString().slice(0, 10) : '—' %></td>
-      </tr>
-      <% }); %>
-    </tbody>
-  </table>
+<div class="card">
+  <div class="card-body p-0">
+    <div class="table-responsive">
+    <table class="table table-vcenter table-hover card-table">
+      <thead>
+        <tr>
+          <th scope="col">ID</th>
+          <th scope="col">Title</th>
+          <th scope="col">Priority</th>
+          <th scope="col">Phase</th>
+          <th scope="col">Status</th>
+          <th scope="col">Created</th>
+        </tr>
+      </thead>
+      <tbody>
+        <% todos.forEach(function(todo) { %>
+        <tr>
+          <td><%= todo.id %></td>
+          <td>
+            <a href="/todos/<%= todo.id %>"
+               hx-get="/todos/<%= todo.id %>"
+               hx-target="#main-content"
+               hx-push-url="true">
+              <%= todo.title %>
+            </a>
+          </td>
+          <td>
+            <span class="status-badge" data-priority="<%= todo.priority %>">
+              <%= todo.priority %>
+            </span>
+          </td>
+          <td><%= todo.phase || '—' %></td>
+          <td>
+            <span class="status-badge" data-status="<%= todo.status %>">
+              <%= todo.status %>
+            </span>
+          </td>
+          <td><%= todo.created ? new Date(todo.created).toISOString().slice(0, 10) : '—' %></td>
+        </tr>
+        <% }); %>
+      </tbody>
+    </table>
+    </div>
   </div>
-</article>
+</div>
 <% } else { %>
-<%- include('empty-state', { icon: '📋', title: 'No todos found', action: '<p><a href="/todos/new" role="button" hx-get="/todos/new" hx-target="#main-content" hx-push-url="true">Create Todo</a></p>' }) %>
+<%- include('empty-state', { icon: '📋', title: 'No todos found', action: '<p><a href="/todos/new" class="btn btn-primary" hx-get="/todos/new" hx-target="#main-content" hx-push-url="true">Create Todo</a></p>' }) %>
 <% } %>

package/dashboard/src/views/partials/todos-done-content.ejs

@@ -7,38 +7,40 @@
       hx-push-url="true">Back to Pending Todos</a></p>
 
 <% if (todos.length > 0) { %>
-<article>
-  <div class="table-wrap">
-  <table>
-    <thead>
-      <tr>
-        <th scope="col">ID</th>
-        <th scope="col">Title</th>
-        <th scope="col">Priority</th>
-        <th scope="col">Phase</th>
-        <th scope="col">Completed</th>
-      </tr>
-    </thead>
-    <tbody>
-      <% todos.forEach(function(todo) { %>
-      <tr>
-        <td><%= todo.id %></td>
-        <td><%= todo.title %></td>
-        <td>
-          <% if (todo.priority) { %>
-          <span class="status-badge" data-priority="<%= todo.priority %>">
-            <%= todo.priority %>
-          </span>
-          <% } %>
-        </td>
-        <td><%= todo.phase %></td>
-        <td><%= todo.completedAt || '' %></td>
-      </tr>
-      <% }); %>
-    </tbody>
-  </table>
+<div class="card">
+  <div class="card-body p-0">
+    <div class="table-responsive">
+    <table class="table table-vcenter card-table">
+      <thead>
+        <tr>
+          <th scope="col">ID</th>
+          <th scope="col">Title</th>
+          <th scope="col">Priority</th>
+          <th scope="col">Phase</th>
+          <th scope="col">Completed</th>
+        </tr>
+      </thead>
+      <tbody>
+        <% todos.forEach(function(todo) { %>
+        <tr>
+          <td><%= todo.id %></td>
+          <td><%= todo.title %></td>
+          <td>
+            <% if (todo.priority) { %>
+            <span class="status-badge" data-priority="<%= todo.priority %>">
+              <%= todo.priority %>
+            </span>
+            <% } %>
+          </td>
+          <td><%= todo.phase %></td>
+          <td><%= todo.completedAt || '' %></td>
+        </tr>
+        <% }); %>
+      </tbody>
+    </table>
+    </div>
   </div>
-</article>
+</div>
 <% } else { %>
 <%- include('empty-state', { icon: '✓', title: 'No completed todos', action: '<p><a href="/todos" hx-get="/todos" hx-target="#main-content" hx-push-url="true">View pending todos</a></p>' }) %>
 <% } %>

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sienklogic/plan-build-run",
-  "version": "2.27.0",
+  "version": "2.27.1",
   "description": "Plan it, Build it, Run it — structured development workflow for Claude Code",
   "keywords": [
     "claude-code",

package/plugins/copilot-pbr/plugin.json

@@ -1,7 +1,7 @@
 {
   "name": "pbr",
   "displayName": "Plan-Build-Run",
-  "version": "2.27.0",
+  "version": "2.27.1",
   "description": "Plan-Build-Run — Structured development workflow for GitHub Copilot CLI. Solves context rot through disciplined agent delegation, structured planning, atomic execution, and goal-backward verification.",
   "author": {
     "name": "SienkLogic",

package/plugins/cursor-pbr/.cursor-plugin/plugin.json

@@ -1,7 +1,7 @@
 {
   "name": "pbr",
   "displayName": "Plan-Build-Run",
-  "version": "2.27.0",
+  "version": "2.27.1",
   "description": "Plan-Build-Run — Structured development workflow for Cursor. Solves context rot through disciplined subagent delegation, structured planning, atomic execution, and goal-backward verification.",
   "author": {
     "name": "SienkLogic",

package/plugins/pbr/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "pbr",
-  "version": "2.27.0",
+  "version": "2.27.1",
   "description": "Plan-Build-Run — Structured development workflow for Claude Code. Solves context rot through disciplined subagent delegation, structured planning, atomic execution, and goal-backward verification.",
   "author": {
     "name": "SienkLogic",

package/plugins/pbr/scripts/local-llm/metrics.js

@@ -29,6 +29,15 @@ function logMetric(planningDir, entry) {
     const logFile = path.join(logsDir, 'local-llm-metrics.jsonl');
 
     fs.mkdirSync(logsDir, { recursive: true });
+
+    // Update lifetime totals BEFORE appending to JSONL so seeding
+    // (which reads the JSONL) doesn't double-count this entry
+    try {
+      updateLifetimeTotals(logsDir, entry);
+    } catch (_) {
+      // Totals update failure is non-fatal
+    }
+
     fs.appendFileSync(logFile, JSON.stringify(entry) + '\n', 'utf8');
 
     // Rotate if over MAX_ENTRIES
@@ -47,6 +56,63 @@ function logMetric(planningDir, entry) {
   }
 }
 
+/**
+ * Atomically increments the lifetime-totals.json running counters.
+ * This file persists across JSONL log rotations so lifetime metrics never plateau.
+ *
+ * @param {string} logsDir - path to the .planning/logs directory
+ * @param {object} entry - the metric entry being logged
+ */
+function updateLifetimeTotals(logsDir, entry) {
+  const totalsFile = path.join(logsDir, 'lifetime-totals.json');
+  let totals = null;
+
+  try {
+    totals = JSON.parse(fs.readFileSync(totalsFile, 'utf8'));
+  } catch (_) {
+    // File doesn't exist yet or is corrupt — seed from existing JSONL
+    totals = seedTotalsFromJsonl(logsDir);
+  }
+
+  totals.total_calls = (totals.total_calls || 0) + 1;
+  totals.fallback_count = (totals.fallback_count || 0) + (entry.fallback_used ? 1 : 0);
+  totals.tokens_saved = (totals.tokens_saved || 0) + (entry.tokens_saved_frontier || 0);
+  totals.total_latency_ms = (totals.total_latency_ms || 0) + (entry.latency_ms || 0);
+
+  fs.writeFileSync(totalsFile, JSON.stringify(totals) + '\n', 'utf8');
+}
+
+/**
+ * Seeds lifetime totals by scanning the existing JSONL log.
+ * Called once when lifetime-totals.json doesn't exist yet (migration from pre-totals installs).
+ * Returns the accumulated totals from whatever entries remain in the JSONL.
+ *
+ * @param {string} logsDir - path to the .planning/logs directory
+ * @returns {{ total_calls: number, fallback_count: number, tokens_saved: number, total_latency_ms: number }}
+ */
+function seedTotalsFromJsonl(logsDir) {
+  const seed = { total_calls: 0, fallback_count: 0, tokens_saved: 0, total_latency_ms: 0 };
+  try {
+    const logFile = path.join(logsDir, 'local-llm-metrics.jsonl');
+    const contents = fs.readFileSync(logFile, 'utf8');
+    const lines = contents.split(/\r?\n/).filter((l) => l.trim() !== '');
+    for (const line of lines) {
+      try {
+        const e = JSON.parse(line);
+        seed.total_calls += 1;
+        seed.fallback_count += e.fallback_used ? 1 : 0;
+        seed.tokens_saved += e.tokens_saved_frontier || 0;
+        seed.total_latency_ms += e.latency_ms || 0;
+      } catch (_) {
+        // Skip malformed lines
+      }
+    }
+  } catch (_) {
+    // No JSONL file — start at zero
+  }
+  return seed;
+}
+
 /**
  * Reads metric entries from the JSONL log that occurred at or after sessionStartTime.
  *
@@ -131,47 +197,69 @@ function computeLifetimeMetrics(planningDir, frontierTokenRate) {
   };
 
   try {
-    const logFile = path.join(planningDir, 'logs', 'local-llm-metrics.jsonl');
-    let contents;
+    const rate = frontierTokenRate != null ? frontierTokenRate : 3.0;
+    const logsDir = path.join(planningDir, 'logs');
+    const totalsFile = path.join(logsDir, 'lifetime-totals.json');
+
+    // Primary path: read from lifetime-totals.json (survives log rotation)
+    let totals = null;
     try {
-      contents = fs.readFileSync(logFile, 'utf8');
+      totals = JSON.parse(fs.readFileSync(totalsFile, 'utf8'));
     } catch (_) {
-      return zero;
+      // No totals file — fall back to JSONL scan (migration path for existing installs)
     }
 
-    const entries = contents
-      .split(/\r?\n/)
-      .filter((l) => l.trim() !== '')
-      .map((l) => {
-        try {
-          return JSON.parse(l);
-        } catch (_) {
-          return null;
-        }
-      })
-      .filter((e) => e !== null);
-
-    if (entries.length === 0) return zero;
+    // Build by_operation from the JSONL (only covers recent entries, but still useful)
+    const by_operation = {};
+    const logFile = path.join(logsDir, 'local-llm-metrics.jsonl');
+    try {
+      const contents = fs.readFileSync(logFile, 'utf8');
+      const entries = contents
+        .split(/\r?\n/)
+        .filter((l) => l.trim() !== '')
+        .map((l) => {
+          try {
+            return JSON.parse(l);
+          } catch (_) {
+            return null;
+          }
+        })
+        .filter((e) => e !== null);
 
-    const rate = frontierTokenRate != null ? frontierTokenRate : 3.0;
-    const total_calls = entries.length;
-    const fallback_count = entries.filter((e) => e.fallback_used).length;
-    const totalLatency = entries.reduce((sum, e) => sum + (e.latency_ms || 0), 0);
-    const avg_latency_ms = total_calls > 0 ? totalLatency / total_calls : 0;
-    const tokens_saved = entries.reduce((sum, e) => sum + (e.tokens_saved_frontier || 0), 0);
-    const cost_saved_usd = tokens_saved * (rate / 1_000_000);
+      for (const e of entries) {
+        const op = e.operation || 'unknown';
+        if (!by_operation[op]) {
+          by_operation[op] = { calls: 0, fallbacks: 0, tokens_saved: 0 };
+        }
+        by_operation[op].calls += 1;
+        if (e.fallback_used) by_operation[op].fallbacks += 1;
+        by_operation[op].tokens_saved += e.tokens_saved_frontier || 0;
+      }
 
-    const by_operation = {};
-    for (const e of entries) {
-      const op = e.operation || 'unknown';
-      if (!by_operation[op]) {
-        by_operation[op] = { calls: 0, fallbacks: 0, tokens_saved: 0 };
+      // If no totals file, compute from JSONL (legacy/migration path)
+      if (!totals) {
+        if (entries.length === 0) return zero;
+        const total_calls = entries.length;
+        const fallback_count = entries.filter((e) => e.fallback_used).length;
+        const totalLatency = entries.reduce((sum, e) => sum + (e.latency_ms || 0), 0);
+        const avg_latency_ms = total_calls > 0 ? totalLatency / total_calls : 0;
+        const tokens_saved = entries.reduce((sum, e) => sum + (e.tokens_saved_frontier || 0), 0);
+        const cost_saved_usd = tokens_saved * (rate / 1_000_000);
+        return { total_calls, fallback_count, avg_latency_ms, tokens_saved, cost_saved_usd, by_operation };
       }
-      by_operation[op].calls += 1;
-      if (e.fallback_used) by_operation[op].fallbacks += 1;
-      by_operation[op].tokens_saved += e.tokens_saved_frontier || 0;
+    } catch (_) {
+      // No JSONL file — if we have totals, continue; otherwise return zero
+      if (!totals) return zero;
     }
 
+    // Use lifetime totals for the headline numbers
+    const total_calls = totals.total_calls || 0;
+    const fallback_count = totals.fallback_count || 0;
+    const tokens_saved = totals.tokens_saved || 0;
+    const total_latency_ms = totals.total_latency_ms || 0;
+    const avg_latency_ms = total_calls > 0 ? total_latency_ms / total_calls : 0;
+    const cost_saved_usd = tokens_saved * (rate / 1_000_000);
+
     return { total_calls, fallback_count, avg_latency_ms, tokens_saved, cost_saved_usd, by_operation };
   } catch (_) {
     return zero;
@@ -249,4 +337,4 @@ function logAgreement(planningDir, entry) {
   }
 }
 
-module.exports = { logMetric, readSessionMetrics, summarizeMetrics, computeLifetimeMetrics, formatSessionSummary, logAgreement };
+module.exports = { logMetric, readSessionMetrics, summarizeMetrics, computeLifetimeMetrics, formatSessionSummary, logAgreement, updateLifetimeTotals, seedTotalsFromJsonl };