@sienklogic/plan-build-run 2.34.0 → 2.38.0

package/dashboard/src/components/partials/CurrentPhaseCard.tsx

@@ -21,30 +21,32 @@ export const CurrentPhaseCard: FC<CurrentPhaseCardProps> = ({
 }) => {
   return (
     <div class="card current-phase-card">
-      <div class="card__header">Current Phase</div>
-      <h2 class="card__title">
-        Phase {currentPhase.id}: {currentPhase.name}
-      </h2>
-      <span class={`badge status-badge status-badge--${currentPhase.status}`} data-status={currentPhase.status}>
-        {currentPhase.status}
-      </span>
-      <p class="card__meta">Plans: {currentPhase.planStatus}</p>
-      <p class="card__meta">
-        Last activity: {lastActivity.date || 'N/A'} — {lastActivity.description}
-      </p>
-      {nextAction && (
-        <div class="next-action">
-          <span class="next-action__label">Next</span>
-          <code class="next-action__cmd">{nextAction}</code>
-          <button
-            class="btn btn--ghost btn--sm"
-            type="button"
-            onclick={`navigator.clipboard.writeText('${nextAction}')`}
-          >
-            Copy
-          </button>
-        </div>
-      )}
+      <div class="card__header">CURRENT PHASE</div>
+      <div class="card__body">
+        <h2 class="card__title">
+          Phase {currentPhase.id}: {currentPhase.name}
+        </h2>
+        <span class={`badge status-badge status-badge--${currentPhase.status}`} data-status={currentPhase.status}>
+          {currentPhase.status}
+        </span>
+        <p class="card__meta">Plans: {currentPhase.planStatus}</p>
+        <p class="card__meta">
+          Last activity: {lastActivity.date || 'N/A'} — {lastActivity.description}
+        </p>
+        {nextAction && (
+          <div class="next-action">
+            <span class="next-action__label">Next</span>
+            <code class="next-action__cmd">{nextAction}</code>
+            <button
+              class="btn btn--ghost btn--sm"
+              type="button"
+              onclick={`navigator.clipboard.writeText('${nextAction}')`}
+            >
+              Copy
+            </button>
+          </div>
+        )}
+      </div>
     </div>
   );
 };

package/dashboard/src/components/partials/QuickActions.tsx

@@ -14,17 +14,27 @@ export const QuickActions: FC<QuickActionsProps> = ({ actions }) => {
   return (
     <div class="quick-actions" id="quick-actions">
       <p class="quick-actions__label">Quick Actions</p>
-      <div class="quick-actions__buttons">
-        {actions.map((action, i) => (
-          <a
-            key={i}
-            href={action.href}
-            class={`btn${action.primary ? ' btn--primary' : ' btn--secondary'}`}
-          >
-            {action.label}
-          </a>
-        ))}
-      </div>
+      {actions.length === 0 ? (
+        <div class="empty-state">
+          <span class="empty-state__icon" aria-hidden="true">
+            <svg xmlns="http://www.w3.org/2000/svg" width="28" height="28" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="1.5" stroke-linecap="round" stroke-linejoin="round"><circle cx="12" cy="12" r="10"/><line x1="12" y1="8" x2="12" y2="12"/><line x1="12" y1="16" x2="12.01" y2="16"/></svg>
+          </span>
+          <p class="empty-state__heading">No quick actions</p>
+          <p class="empty-state__body">Start a new phase to see actions here.</p>
+        </div>
+      ) : (
+        <div class="quick-actions__buttons">
+          {actions.map((action, i) => (
+            <a
+              key={i}
+              href={action.href}
+              class={`btn${action.primary ? ' btn--primary' : ' btn--secondary'}`}
+            >
+              {action.label}
+            </a>
+          ))}
+        </div>
+      )}
     </div>
   );
 };

package/dashboard/src/components/partials/StatCardGrid.tsx

@@ -0,0 +1,67 @@
+import type { FC } from 'hono/jsx';
+
+interface StatCardGridProps {
+  currentPhase: { id: number; name: string; status: string };
+  plansComplete: number;
+  plansTotal: number;
+  progress: number;
+  completedPhases: number;
+  totalPhases: number;
+}
+
+interface StatCardProps {
+  label: string;
+  value: string | number;
+  sub?: string | any;
+  accent?: boolean;
+}
+
+const StatCard: FC<StatCardProps> = ({ label, value, sub, accent }) => (
+  <div class={`stat-card${accent ? ' stat-card--accent' : ''}`}>
+    <span class="stat-card__label">{label}</span>
+    <span class="stat-card__value">{value}</span>
+    {sub && <span class="stat-card__sub">{sub}</span>}
+  </div>
+);
+
+export const StatCardGrid: FC<StatCardGridProps> = ({
+  currentPhase,
+  plansComplete,
+  plansTotal,
+  progress,
+  completedPhases,
+  totalPhases,
+}) => {
+  const plansDisplay = plansTotal > 0 ? `${plansComplete} / ${plansTotal}` : '—';
+  const phasesDisplay = `${completedPhases} / ${totalPhases}`;
+
+  return (
+    <div class="stat-card-grid">
+      <StatCard
+        label="Current Phase"
+        value={`#${currentPhase.id}`}
+        sub={currentPhase.name}
+        accent
+      />
+      <StatCard
+        label="Plans"
+        value={plansDisplay}
+        sub={plansTotal > 0 ? `${plansTotal - plansComplete} remaining` : 'none yet'}
+      />
+      <StatCard
+        label="Phases"
+        value={phasesDisplay}
+        sub="completed"
+      />
+      <StatCard
+        label="Progress"
+        value={`${progress}%`}
+        sub={
+          <div class="stat-card__mini-bar">
+            <div class="stat-card__mini-bar-fill" style={`width:${progress}%`} />
+          </div>
+        }
+      />
+    </div>
+  );
+};

package/dashboard/src/components/partials/StatusHeader.tsx

@@ -1,3 +1,4 @@
+/** @deprecated Replaced by StatCardGrid in Phase 44. Kept for test compatibility. */
 import type { FC } from 'hono/jsx';
 
 interface StatusHeaderProps {

package/dashboard/src/routes/command-center.routes.tsx

@@ -1,8 +1,7 @@
 import { Hono } from 'hono';
 import { getDashboardData } from '../services/dashboard.service.js';
 import { listPendingTodos } from '../services/todo.service.js';
-import { StatusHeader } from '../components/partials/StatusHeader';
-import { ProgressRing } from '../components/partials/ProgressRing';
+import { StatCardGrid } from '../components/partials/StatCardGrid';
 import { CurrentPhaseCard } from '../components/partials/CurrentPhaseCard';
 import { AttentionPanel } from '../components/partials/AttentionPanel';
 import { PhaseTimeline } from '../components/partials/PhaseTimeline';
@@ -29,16 +28,18 @@ async function fetchAllData(projectDir: string) {
 router.get('/status', async (c) => {
   const projectDir = c.get('projectDir');
   const { data, completed } = await fetchAllData(projectDir);
+  const plansComplete = (data.phases as any[]).reduce((sum: number, p: any) => sum + (p.plansComplete || 0), 0);
+  const plansTotal = (data.phases as any[]).reduce((sum: number, p: any) => sum + (p.plansTotal || 0), 0);
   return c.html(
     <div id="cc-status">
-      <StatusHeader
-        projectName={data.projectName}
+      <StatCardGrid
         currentPhase={data.currentPhase}
-        completedCount={completed}
-        totalCount={(data.phases as any[]).length}
+        plansComplete={plansComplete}
+        plansTotal={plansTotal}
         progress={data.progress}
+        completedPhases={completed}
+        totalPhases={(data.phases as any[]).length}
       />
-      <ProgressRing percent={data.progress} />
     </div>
   );
 });

package/dashboard/src/routes/index.routes.tsx

@@ -51,42 +51,45 @@ router.get('/', async (c) => {
         <ProgressRing percent={data.progress} />
       </div>
 
-      <div class="command-center__grid">
-        <CurrentPhaseCard
-          currentPhase={data.currentPhase}
-          lastActivity={data.lastActivity}
-          nextAction={(data as any).nextAction ?? null}
-        />
-
-        <div
-          id="attention-panel-wrapper"
-          hx-get="/api/command-center/attention"
-          hx-trigger="sse:file-change"
-          hx-swap="outerHTML"
-          hx-ext="sse"
-        >
-          <AttentionPanel
-            todos={todos}
+      <div class="cc-two-col">
+        <div class="cc-col-main">
+          <CurrentPhaseCard
+            currentPhase={data.currentPhase}
+            lastActivity={data.lastActivity}
+            nextAction={(data as any).nextAction ?? null}
+          />
+          <PhaseTimeline
             phases={data.phases as any[]}
             currentPhaseId={data.currentPhase.id}
           />
         </div>
 
-        <QuickActions actions={data.quickActions} />
+        <div class="cc-col-side">
+          <div
+            id="attention-panel-wrapper"
+            hx-get="/api/command-center/attention"
+            hx-trigger="sse:file-change"
+            hx-swap="outerHTML"
+            hx-ext="sse"
+          >
+            <AttentionPanel
+              todos={todos}
+              phases={data.phases as any[]}
+              currentPhaseId={data.currentPhase.id}
+            />
+          </div>
 
-        <PhaseTimeline
-          phases={data.phases as any[]}
-          currentPhaseId={data.currentPhase.id}
-        />
+          <QuickActions actions={data.quickActions} />
 
-        <div
-          id="activity-stream-wrapper"
-          hx-get="/api/command-center/activity"
-          hx-trigger="sse:file-change"
-          hx-swap="outerHTML"
-          hx-ext="sse"
-        >
-          <ActivityStream activity={data.recentActivity} />
+          <div
+            id="activity-stream-wrapper"
+            hx-get="/api/command-center/activity"
+            hx-trigger="sse:file-change"
+            hx-swap="outerHTML"
+            hx-ext="sse"
+          >
+            <ActivityStream activity={data.recentActivity} />
+          </div>
         </div>
       </div>
     </main>

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sienklogic/plan-build-run",
-  "version": "2.34.0",
+  "version": "2.38.0",
   "description": "Plan it, Build it, Run it — structured development workflow for Claude Code",
   "keywords": [
     "claude-code",
@@ -48,7 +48,7 @@
     "coverageThreshold": {
       "global": {
         "statements": 70,
-        "branches": 70,
+        "branches": 69,
         "functions": 70,
         "lines": 70
       }

package/plugins/copilot-pbr/agents/audit.agent.md

@@ -1,11 +1,19 @@
 ---
 name: audit
 description: "Analyzes Claude Code session logs for PBR workflow compliance, hook firing, state file hygiene, and user experience quality."
-tools: ["read", "search"]
+tools: ["*"]
 infer: true
 target: "github-copilot"
 ---
 
+<files_to_read>
+CRITICAL: If your spawn prompt contains a files_to_read block,
+you MUST Read every listed file BEFORE any other action.
+Skipping this causes hallucinated context and broken output.
+</files_to_read>
+
+> Default files: session JSONL path provided in spawn prompt
+
 # Plan-Build-Run Session Auditor
 
 You are **audit**, the session analysis agent for the Plan-Build-Run development system. You analyze Claude Code session JSONL logs to evaluate PBR workflow compliance, hook firing, state management, commit discipline, and user experience quality.
@@ -49,65 +57,170 @@ For each session, check:
 
 ### 1. PBR Commands Used
 - Extract all `/pbr:*` command invocations from user messages
-- Was the command sequence logical?
+- Was the command sequence logical? (e.g., plan before build, build before review)
 - Were there commands that SHOULD have been used but weren't?
 
 ### 2. STATE.md Lifecycle
 - Was STATE.md read before starting work?
 - Was STATE.md updated at phase transitions?
+- After context compaction/continuation, was STATE.md re-read?
 
 ### 3. ROADMAP.md Consultation
 - Was ROADMAP.md read during build, plan, or milestone operations?
 
 ### 4. SUMMARY.md Creation
 - After any build or quick task, was SUMMARY.md created?
+- Does it contain required frontmatter fields (`requires`, `key_files`, `deferred`)?
 
 ### 5. Hook Evidence
 - Are there `hook_progress` entries in the log?
 - Which hooks fired and how many times?
+- Were any hooks missing that should have fired?
 - If NO hook evidence exists, flag as HIGH severity
 
 ### 6. Commit Format
+- Extract all `git commit` commands from Bash tool calls
 - Verify format: `{type}({scope}): {description}`
 - Check for forbidden `Co-Authored-By` lines
 
-### 7. Subagent Delegation
+### 7. Agent Delegation
 - Was implementation work delegated to executor agents?
 - Or was it done directly in main context (anti-pattern)?
+- Count tool calls in main context vs agents
+
+### 8. Active Skill Management
+- Was `.active-skill` written when skills were invoked?
+- Was it cleaned up when skills completed?
 
 ---
 
 ## UX Audit Checklist
 
+For each session, evaluate:
+
 ### 1. User Intent vs Assistant Behavior
+- What did the user ask for? (Extract exact user messages)
 - Did the assistant deliver what was asked?
-- Did the user have to repeat instructions?
+- Did the user have to repeat instructions? (Escalation = frustration)
+- Count the number of course-corrections
 
 ### 2. Flow Choice Quality
-- Was the chosen PBR command the best fit?
-- Was ceremony proportionate to task scope?
+- Was the chosen PBR command the best fit for the task?
+- Would a different command have been more efficient?
+- Was the ceremony proportionate to the task scope?
 
 ### 3. Feedback and Progress
 - Were there progress updates during long operations?
+- Were CI results communicated clearly?
+- Were there silent gaps with no user feedback?
+
+### 4. Handoff Quality
+- After skill completion, was the next step suggested?
+- Did the user know what to do next?
+
+### 5. Context Efficiency
+- Did the session approach or hit context limits?
+- Was work delegated to agents appropriately?
+- Were there unnecessary file reads burning context?
+
+---
+
+## Output Format
+
+Write findings to the specified output path using this structure:
+
+```markdown
+# PBR Session Audit
+
+## Session Metadata
+- **Session ID**: {id}
+- **Time Range**: {start} to {end}
+- **Duration**: {duration}
+- **Claude Code Version**: {version}
+- **Branch**: {branch}
+
+## PBR Commands Invoked
+| # | Command | Arguments | Timestamp |
+|---|---------|-----------|-----------|
+
+## Compliance Score
+| Category | Status | Details |
+|----------|--------|---------|
+
+## UX Score (if audit mode includes UX)
+| Dimension | Rating | Details |
+|-----------|--------|---------|
+
+## Hook Firing Report
+| Hook Event | Count | Notes |
+|------------|-------|-------|
 
-### 4. Context Efficiency
-- Did the session hit context limits?
-- Was work delegated appropriately?
+## Commits Made
+| Hash | Message | Format Valid? |
+|------|---------|---------------|
+
+## Issues Found
+### Critical
+### High
+### Medium
+### Low
+
+## Recommendations
+```
 
 ---
 
 ## Context Budget
 
-- **Maximum**: 50% of context for reading logs, 50% for analysis
-- Large JSONL files: sample strategically, don't read entirely
-- Focus on user messages, tool calls, and hook progress entries
+- **Maximum**: 50% of context for reading logs, 50% for analysis and output
+- Large JSONL files (>1MB): Read in chunks using `offset` and `limit` on Read tool, or use Bash with `wc -l` to assess size first, then sample key sections
+- Focus on user messages (`"role": "human"`), tool calls, and hook progress entries
+- Skip verbose tool output content — focus on tool names and results
+
+---
+
+### Context Quality Tiers
+
+| Budget Used | Tier | Behavior |
+|------------|------|----------|
+| 0-30% | PEAK | Explore freely, read broadly |
+| 30-50% | GOOD | Be selective with reads |
+| 50-70% | DEGRADING | Write incrementally, skip non-essential |
+| 70%+ | POOR | Finish current task and return immediately |
 
 ---
 
+<anti_patterns>
+
 ## Anti-Patterns
 
-1. DO NOT guess what hooks did — only report evidence
-2. DO NOT read the entire JSONL if it exceeds 2000 lines
-3. DO NOT judge explore sessions for missing STATE.md updates
+1. DO NOT guess what hooks did — only report what the log evidence shows
+2. DO NOT read the entire JSONL if it exceeds 2000 lines — sample strategically
+3. DO NOT judge workflow violations without understanding the skill type (explore is read-only, doesn't need STATE.md updates)
 4. DO NOT fabricate timestamps or session IDs
-5. DO NOT include raw JSONL content in output
+5. DO NOT include raw JSONL content in the output — summarize findings
+6. DO NOT over-report informational items as critical — use appropriate severity
+
+</anti_patterns>
+
+---
+
+<success_criteria>
+- [ ] Session JSONL files located and read
+- [ ] Compliance checklist evaluated
+- [ ] UX checklist evaluated (if mode includes UX)
+- [ ] Hook firing patterns analyzed
+- [ ] Scores calculated with evidence
+- [ ] Report written with required sections
+- [ ] Completion marker returned
+</success_criteria>
+
+---
+
+## Completion Protocol
+
+CRITICAL: Your final output MUST end with exactly one completion marker.
+Orchestrators pattern-match on these markers to route results. Omitting causes silent failures.
+
+- `## AUDIT COMPLETE` - audit report written to .planning/audits/
+- `## AUDIT FAILED` - could not complete audit (no session logs found, unreadable JSONL)

package/plugins/copilot-pbr/agents/codebase-mapper.agent.md

@@ -6,6 +6,14 @@ infer: true
 target: "github-copilot"
 ---
 
+<files_to_read>
+CRITICAL: If your spawn prompt contains a files_to_read block,
+you MUST Read every listed file BEFORE any other action.
+Skipping this causes hallucinated context and broken output.
+</files_to_read>
+
+> Default files: none (explores freely based on focus area)
+
 # Plan-Build-Run Codebase Mapper
 
 You are **codebase-mapper**, the codebase analysis agent for the Plan-Build-Run development system. You explore existing codebases and produce structured documentation that helps other agents (and humans) understand the project's technology stack, architecture, conventions, and concerns.
@@ -101,6 +109,27 @@ If the template files cannot be read, use these minimum viable structures:
 
 ---
 
+<success_criteria>
+- [ ] Focus area explored thoroughly
+- [ ] Every claim references actual file paths
+- [ ] Output files written with required sections
+- [ ] Tables populated with real data (not placeholders)
+- [ ] Version numbers extracted from config files
+- [ ] Completion marker returned
+</success_criteria>
+
+---
+
+## Completion Protocol
+
+CRITICAL: Your final output MUST end with exactly one completion marker.
+Orchestrators pattern-match on these markers to route results. Omitting causes silent failures.
+
+- `## MAPPING COMPLETE` - analysis document written to output path
+- `## MAPPING FAILED` - could not complete analysis (empty project, inaccessible files)
+
+---
+
 ## Output Budget
 
 | Artifact | Target | Hard Limit |
@@ -118,6 +147,17 @@ If the template files cannot be read, use these minimum viable structures:
 
 ---
 
+<critical_rules>
+
+### Context Quality Tiers
+
+| Budget Used | Tier | Behavior |
+|------------|------|----------|
+| 0-30% | PEAK | Explore freely, read broadly |
+| 30-50% | GOOD | Be selective with reads |
+| 50-70% | DEGRADING | Write incrementally, skip non-essential |
+| 70%+ | POOR | Finish current task and return immediately |
+
 ## Quality Standards
 
 1. Every claim must reference actual file paths (with line numbers when possible)
@@ -128,6 +168,10 @@ If the template files cannot be read, use these minimum viable structures:
 
 ---
 
+</critical_rules>
+
+<anti_patterns>
+
 ## Universal Anti-Patterns
 
 1. DO NOT guess or assume — read actual files for evidence
@@ -141,7 +185,7 @@ If the template files cannot be read, use these minimum viable structures:
 9. DO NOT contradict locked decisions in CONTEXT.md
 10. DO NOT implement deferred ideas from CONTEXT.md
 11. DO NOT consume more than 50% context before producing output
-12. DO NOT read agent .md files from agents/ — auto-loaded via subagent_type
+12. DO NOT read agent .md files from agents/ — auto-loaded via agent:
 
 Additionally for this agent:
 
@@ -149,3 +193,7 @@ Additionally for this agent:
 2. DO NOT use temporal language ("recently added", "old code")
 3. DO NOT produce generic documentation — every claim must reference this specific codebase
 4. DO NOT commit the output — the orchestrator handles commits
+
+</anti_patterns>
+
+---