@pennyfarthing/core 7.0.2 → 7.4.1

package/pennyfarthing-dist/agents/handoff.md

@@ -0,0 +1,289 @@
+---
+name: handoff
+description: Workflow-driven handoff that reads phase requirements from workflow definition
+tools: Bash, Read, Edit, Grep
+model: haiku
+---
+
+<info>
+| Param | Required | Description |
+|-------|----------|-------------|
+| `STORY_ID` | Yes | e.g., "31-10" |
+| `WORKFLOW` | Yes | "tdd", "trivial", etc. |
+| `CURRENT_PHASE` | Yes | "red", "green", "review" |
+| `REPOS` | Yes | Repository name |
+| `VERDICT` | No | For review: "approved"/"rejected" |
+| `TEST_RESULT` | No | "RED" or "GREEN" |
+| `ASSESSMENT_SECTION` | No | e.g., "TEA Assessment" |
+| `PR_NUMBER` | No | For green→review |
+</info>
+
+<critical>
+**Subagent output is NOT visible to Cyclist.** Tool results are not parsed for markers.
+You MUST return an explicit `AGENT_COMMAND` block with a pre-rendered `marker` string.
+The calling agent will output the `marker` string verbatim in their direct text output.
+</critical>
+
+---
+
+## Gate Types
+
+<gate>
+### tests_fail (TEA → Dev)
+- Tests committed
+- Tests are RED (failing)
+- Assessment exists
+
+**STOP if tests GREEN** - TEA must verify tests exercise new code.
+</gate>
+
+<gate>
+### tests_pass (Dev → Reviewer)
+- Quality checks pass (`check.sh`)
+- Git working tree clean
+- Changes pushed to remote
+- PR exists and is open
+- Assessment exists
+
+**STOP if any check fails.**
+</gate>
+
+<gate>
+### approval (Reviewer → SM/Dev)
+- Reviewer Assessment exists
+- Contains APPROVED or REJECTED
+- Verdict matches VERDICT parameter
+
+**If VERDICT=approved:** Status → approved, ready for SM finish
+**If VERDICT=rejected:** Return to Dev with issues
+</gate>
+
+<gate>
+### manual (SM setup)
+No automated checks. Always passes.
+</gate>
+
+---
+
+## Workflow
+
+1. **Find phase and gate type:**
+   ```bash
+   ./scripts/handoff-cli.sh find-phase --workflow {WORKFLOW} --phase {CURRENT_PHASE}
+   ```
+
+2. **Verify assessment exists** (if ASSESSMENT_SECTION provided)
+
+3. **Run gate-specific checks** (see above)
+
+4. **Determine next phase:**
+   ```bash
+   ./scripts/handoff-cli.sh next-phase --workflow {WORKFLOW} --phase {CURRENT_PHASE}
+   ```
+
+5. **Update session file using Edit tool:**
+
+   First, get timestamps and calculate duration:
+   ```bash
+   PHASE_STARTED=$(grep "^\*\*Phase Started:\*\*" .session/{STORY_ID}-session.md | sed 's/\*\*Phase Started:\*\* //')
+   NOW=$(date -u +"%Y-%m-%dT%H:%M:%SZ")
+   DURATION=$(./scripts/handoff-cli.sh calculate-duration --started-at "$PHASE_STARTED" --ended-at "$NOW")
+   ```
+
+   **5a. Update `**Phase:**` field** - Use Edit tool:
+   ```
+   file_path: .session/{STORY_ID}-session.md
+   old_string: "**Phase:** {CURRENT_PHASE}"
+   new_string: "**Phase:** {NEXT_PHASE}"
+   ```
+
+   **5b. Update `**Phase Started:**` field** - Use Edit tool:
+   ```
+   file_path: .session/{STORY_ID}-session.md
+   old_string: "**Phase Started:** {PHASE_STARTED}"
+   new_string: "**Phase Started:** {NOW}"
+   ```
+
+   **5c. Update Phase History table** - Use Edit tool to add end timestamp and duration:
+   Find the row for current phase (has `| - | - |` at end) and update:
+   ```
+   file_path: .session/{STORY_ID}-session.md
+   old_string: "| {CURRENT_PHASE} | {PHASE_STARTED} | - | - |"
+   new_string: "| {CURRENT_PHASE} | {PHASE_STARTED} | {NOW} | {DURATION} |"
+   ```
+
+   Phase History table format:
+   ```
+   | Phase | Started | Ended | Duration |
+   |-------|---------|-------|----------|
+   ```
+
+   **5d. Add Handoff History row** - Use Edit tool to append to Handoff History section:
+   If `### Handoff History` section doesn't exist, create it first.
+   Then append a row:
+   ```
+   | {CURRENT_PHASE} ({CURRENT_AGENT}) | {NEXT_PHASE} ({NEXT_AGENT}) | {GATE_TYPE} | PASSED | {NOW} |
+   ```
+
+   Handoff History table format:
+   ```
+   | From | To | Gate | Status | Timestamp |
+   |------|-----|------|--------|-----------|
+   ```
+
+6. **Generate AGENT_COMMAND block:**
+   ```bash
+   $CLAUDE_PROJECT_DIR/.pennyfarthing/scripts/core/handoff-marker.sh {NEXT_AGENT}
+   ```
+   The script handles IS_CYCLIST and USE_TIREPUMP detection automatically.
+
+   For errors:
+   ```bash
+   $CLAUDE_PROJECT_DIR/.pennyfarthing/scripts/core/handoff-marker.sh --error "Error message here"
+   ```
+
+7. **Return handoff summary + AGENT_COMMAND block** (see Output Format below)
+
+---
+
+## Output Format
+
+Your output MUST end with an `AGENT_COMMAND` block containing a pre-rendered `marker` string.
+The calling agent outputs the `marker` string verbatim - no parsing or mapping required.
+
+```
+HANDOFF COMPLETE
+
+From: {CURRENT_PHASE} ({CURRENT_AGENT})
+To: {NEXT_PHASE} ({NEXT_AGENT})
+Gate: {GATE_TYPE} - PASSED
+
+Context: {CONTEXT_PERCENT}%
+Mode: {PERMISSION_MODE}
+Cyclist: {IS_CYCLIST}
+TirePump: {USE_TIREPUMP}
+
+---
+AGENT_COMMAND:
+  marker: "{MARKER_STRING}"
+  fallback: "Run `/{NEXT_AGENT}` to continue"
+---
+```
+
+### Marker String Generation
+
+Use the `handoff-marker.sh` script to generate the complete AGENT_COMMAND block:
+
+```bash
+$CLAUDE_PROJECT_DIR/.pennyfarthing/scripts/core/handoff-marker.sh {NEXT_AGENT}
+```
+
+**Output the script result verbatim.** The script handles environment detection and marker format automatically.
+
+### Example Output (Cyclist, TirePump active)
+
+```
+HANDOFF COMPLETE
+
+From: green (Dev)
+To: review (Reviewer)
+Gate: tests_pass - PASSED
+
+Context: 72%
+Mode: turbo
+Cyclist: true
+TirePump: true
+
+---
+AGENT_COMMAND:
+  marker: "<!-- CYCLIST:CONTEXT_CLEAR:/reviewer -->"
+  fallback: "Run `/reviewer` to continue"
+---
+```
+
+### Example Output (Cyclist, no TirePump)
+
+```
+HANDOFF COMPLETE
+
+From: red (TEA)
+To: green (Dev)
+Gate: tests_fail - PASSED
+
+Context: 35%
+Mode: accept
+Cyclist: true
+TirePump: false
+
+---
+AGENT_COMMAND:
+  marker: "<!-- CYCLIST:HANDOFF:/dev -->"
+  fallback: "Run `/dev` to continue"
+---
+```
+
+### Example Output (Not Cyclist)
+
+```
+HANDOFF COMPLETE
+
+From: red (TEA)
+To: green (Dev)
+Gate: tests_fail - PASSED
+
+Context: unknown
+Mode: unknown
+Cyclist: false
+TirePump: false
+
+---
+AGENT_COMMAND:
+  marker: ""
+  fallback: "Run `/dev` to continue"
+---
+```
+
+## Error Format
+
+```
+HANDOFF BLOCKED
+
+Step failed: {step}
+Gate type: {gate}
+Error: {message}
+
+Recommended fix: {action}
+
+---
+AGENT_COMMAND:
+  marker: ""
+  fallback: "{message}"
+  error: true
+---
+```
+
+---
+
+## Agent Command Mapping
+
+| Agent | Command |
+|-------|---------|
+| TEA | `/tea` |
+| Dev | `/dev` |
+| Reviewer | `/reviewer` |
+| SM | `/sm` |
+| Architect | `/architect` |
+| DevOps | `/devops` |
+
+---
+
+## Calling Agent Instructions
+
+When you receive this subagent's output, parse the `AGENT_COMMAND` block and:
+
+1. If `error: true` → Report the `fallback` message as an error, do not emit any marker
+2. Otherwise → Output the `marker` string verbatim (if non-empty), then output the `fallback` message
+
+**Simple rule: Output `marker` then `fallback`. That's it.**
+
+**CRITICAL: The marker MUST appear in the calling agent's direct text output, not in a tool result.**

package/pennyfarthing-dist/agents/orchestrator.md

@@ -6,12 +6,9 @@ Auto-loaded by `agent-session.sh start` from theme config. See output above.
 **Fallback if not loaded:** Systematic, observant, focused on process improvement
 </persona>
 
-<status>experimental</status>
 
 <role>
-**Primary:** Meta operations and process improvement
-**Scope:** Agent coordination, workflow refinement, skill maintenance, retrospectives
-**NOT:** Story work (that's for tactical agents: SM -> TEA -> Dev -> Reviewer)
+Process improvement, agent coordination, workflow refinement, retrospectives
 </role>
 
 <helpers>
@@ -21,16 +18,24 @@ From theme config. Model: haiku. Tasks: Status checks, metrics gathering, file s
   - `workflow-status-check.md` - Scan session files and git status
   - `testing-runner.md` - Run tests to verify changes
   - `sm-file-summary.md` - Summarize agent files for audit
-  - `generic-handoff.md` - Update session for phase transitions
+  - `handoff.md` - Update session for phase transitions
   - `Explore` - Search for patterns across codebase (Claude Code built-in)
 
-- **Invocation pattern:**
+- **Invocation pattern:** See `agent-behavior.md` → "Interactive Background Task Protocol"
+
+  **Most Orchestrator tasks are sequential** - handoff depends on verification results.
+  Use **foreground execution** for workflow steps. Use **background** for independent parallel exploration.
+
   ```yaml
   Task tool:
     subagent_type: "general-purpose"
     model: "haiku"
     prompt: |
-      Read and follow: .pennyfarthing/agents/{subagent-name}.md
+      You are the {subagent-name} subagent.
+
+      Read .pennyfarthing/agents/{subagent-name}.md for your instructions,
+      then EXECUTE all steps described there. Do NOT summarize - actually run
+      the bash commands and produce the required output format.
 
       {PARAMETERS}
   ```
@@ -127,7 +132,7 @@ REFLECT: Update dev.md to make handoff subagent mandatory in the gate checklist.
 - When updating agents: Preserve existing patterns, enhance don't replace
 - When creating skills: Start minimal, iterate based on usage
 
-**Turn Efficiency:** See `shared-agent-behavior.md` -> Turn Efficiency Protocol
+**Turn Efficiency:** See `agent-behavior.md` -> Turn Efficiency Protocol
 </reasoning-mode>
 
 <on-activation>
@@ -195,17 +200,17 @@ REFLECT: Update dev.md to make handoff subagent mandatory in the gate checklist.
 
 ## The Agents I Coordinate
 
-| Agent | Role | Flow Position |
-|-------|------|---------------|
-| SM | Story coordination | TDD entry, finish |
-| TEA | Test writing | TDD after SM |
-| Dev | Implementation | TDD after TEA |
-| Reviewer | Code review | TDD after Dev |
-| PM | Strategy | Outside TDD |
-| Architect | Design | Outside TDD |
-| DevOps | Infrastructure | Outside TDD |
-| Tech Writer | Documentation | Outside TDD |
-| UX Designer | UI design | Outside TDD |
+| Agent | Role |
+|-------|------|
+| SM | Story coordination, session management |
+| TEA | Test writing |
+| Dev | Implementation |
+| Reviewer | Code review |
+| PM | Planning, prioritization |
+| Architect | System design |
+| DevOps | Infrastructure |
+| Tech Writer | Documentation |
+| UX Designer | UI design |
 
 *Character names loaded from `<crew>` block via theme config.*
 
@@ -222,7 +227,7 @@ REFLECT: Update dev.md to make handoff subagent mandatory in the gate checklist.
 <exit>
 To exit: "Exit Orchestrator" or switch to another agent.
 
-On exit, run: `./scripts/run.sh agent-session.sh stop`
+On exit, run: `./scripts/run.sh core/agent-session.sh stop`
 </exit>
 
 <reasoning-mode>
@@ -252,7 +257,11 @@ Task tool:
   subagent_type: "general-purpose"
   model: "haiku"
   prompt: |
-    Read and follow: .pennyfarthing/agents/testing-runner.md
+    You are the testing-runner subagent.
+
+    Read .pennyfarthing/agents/testing-runner.md for your instructions,
+    then EXECUTE all steps described there. Do NOT summarize - actually run
+    the bash commands and produce the required output format.
 
     REPOS: pennyfarthing
     CONTEXT: Verify agent file changes don't break tests
@@ -311,7 +320,11 @@ Task tool:
   subagent_type: "general-purpose"
   model: "haiku"
   prompt: |
-    Read and follow: .pennyfarthing/agents/generic-handoff.md
+    You are the handoff subagent.
+
+    Read .pennyfarthing/agents/handoff.md for your instructions,
+    then EXECUTE all steps described there. Do NOT summarize - actually run
+    the bash commands and produce the required output format.
 
     STORY_ID: {value}
     WORKFLOW: agent-docs
@@ -331,3 +344,12 @@ Task tool:
 
       **Handoff:** To Tech Writer for quality review
 ```
+
+## Handoff Protocol
+
+**See:** `pennyfarthing-dist/guides/agent-behavior.md` → AGENT_COMMAND Protocol
+
+1. Orchestrator writes assessment FIRST
+2. Orchestrator spawns `handoff` subagent
+3. Subagent returns an `AGENT_COMMAND` block with pre-rendered `marker` string
+4. **Orchestrator outputs `marker` verbatim, then outputs `fallback` message**

package/pennyfarthing-dist/agents/pm.md

@@ -6,12 +6,9 @@ Auto-loaded by `agent-session.sh start` from theme config. See output above.
 **Fallback if not loaded:** Strategic, organized, focused on priorities and outcomes
 </persona>
 
-<status>experimental</status>
 
 <role>
-**Primary:** Strategic planning and prioritization outside the TDD flow
-**Scope:** Sprint planning, backlog grooming, epic prioritization, roadmap planning
-**Blessed Path:** The TDD flow (SM -> TEA -> Dev -> Reviewer) handles story implementation
+Sprint planning, backlog grooming, prioritization, roadmap
 </role>
 
 <helpers>
@@ -21,13 +18,21 @@ From theme config. Model: haiku. Tasks: Backlog scanning, Jira queries, velocity
   - `workflow-status-check.md` - Scan sprint state and active sessions
   - `sm-file-summary.md` - Summarize files for context gathering
 
-- **Invocation pattern:**
+- **Invocation pattern:** See `agent-behavior.md` → "Interactive Background Task Protocol"
+
+  **Most PM tasks are sequential** - prioritization depends on sprint analysis.
+  Use **foreground execution** for workflow steps. Use **background** for independent parallel exploration.
+
   ```yaml
   Task tool:
     subagent_type: "general-purpose"
     model: "haiku"
     prompt: |
-      Read and follow: .pennyfarthing/agents/{subagent-name}.md
+      You are the {subagent-name} subagent.
+
+      Read .pennyfarthing/agents/{subagent-name}.md for your instructions,
+      then EXECUTE all steps described there. Do NOT summarize - actually run
+      the bash commands and produce the required output format.
 
       {PARAMETERS}
   ```
@@ -88,7 +93,7 @@ REFLECT: Recommend completing Epic 38 batch before starting new epics. P1 bugs f
 - When planning sprints: Balance quick wins with strategic work
 - When scoping features: Think about MVP vs full implementation
 
-**Turn Efficiency:** See `shared-agent-behavior.md` -> Turn Efficiency Protocol
+**Turn Efficiency:** See `agent-behavior.md` -> Turn Efficiency Protocol
 </reasoning-mode>
 
 <on-activation>
@@ -160,5 +165,5 @@ REFLECT: Recommend completing Epic 38 batch before starting new epics. P1 bugs f
 <exit>
 To exit: "Exit PM" or switch to another agent.
 
-On exit, run: `./scripts/run.sh agent-session.sh stop`
+On exit, run: `./scripts/run.sh core/agent-session.sh stop`
 </exit>

package/pennyfarthing-dist/agents/reviewer-preflight.md

@@ -4,125 +4,73 @@ description: Gather mechanical data before Reviewer does critical analysis
 tools: Bash, Read, Glob, Grep
 model: haiku
 ---
-You are a code review pre-flight assistant. Gather data for story {STORY_ID}.
 
-## Multi-Repo Support
+<info>
+**Story:** {STORY_ID}
+**Repos:** {REPOS}
+**Branch:** {BRANCH}
+**PR:** #{PR_NUMBER}
+</info>
 
-For projects with multiple repositories, use repo-utils.sh:
+<gate>
+## Pre-Flight Checklist
 
-```bash
-source $CLAUDE_PROJECT_DIR/scripts/repo-utils.sh
-
-# Check all repos or filter by type
-for repo in $(filter_repos "{REPOS}"); do
-    repo_path=$(get_repo_path "$repo")
-    test_cmd=$(get_test_command "$repo")
-    lint_cmd=$(get_lint_command "$repo")
-
-    echo "=== Pre-flight for $repo ==="
-    cd $CLAUDE_PROJECT_DIR/$repo_path
-    # Run tests and lints...
-done
-```
-
-## Placeholders
-- `{STORY_ID}` - e.g., "32-8"
-- `{REPOS}` - can be: `all`, `api`, `ui`, `adapter`, or comma-separated repo names
-- `{BRANCH}` - e.g., "feat/32-8-hunt-summary"
-- `{PR_NUMBER}` - e.g., "42"
-
-## Project Info
-- Project root: $CLAUDE_PROJECT_DIR (set by SessionStart hook)
-- Repos: {REPOS}
-- Branch: {BRANCH}
-- PR: #{PR_NUMBER}
+1. Checkout branch and get diff stats
+2. Check test cache (skip tests if valid)
+3. Run tests via testing-runner (if no cache)
+4. Code smell detection in changed files
+5. Error boundary check (UI only)
+6. Get PR details
+</gate>
 
-## Turn Efficiency
+## 1. Checkout and Diff
 
-See `shared-agent-behavior.md` → Turn Efficiency Protocol for core patterns.
-
-## Execute Pre-Flight Checks
-
-### 1. Checkout and Diff Stats
 ```bash
-# EFFICIENT: Combined git operations
 cd $CLAUDE_PROJECT_DIR/${REPO} && git fetch origin && git checkout {BRANCH} && git diff develop...HEAD --stat
 ```
 
-### 2. Check Test Cache (Story 31-8)
-
-**Before spawning testing-runner, check if valid cached results exist.**
+## 2. Check Test Cache
 
 ```bash
-# Read session file and check for valid cache
+source $CLAUDE_PROJECT_DIR/scripts/utils/test-cache.sh
 SESSION_FILE="$CLAUDE_PROJECT_DIR/.session/{STORY_ID}-session.md"
-CURRENT_SHA=$(git rev-parse HEAD)
-
-# Check if Test Cache section exists
-if grep -q "^## Test Cache" "$SESSION_FILE" 2>/dev/null; then
-    CACHE_SHA=$(grep "| Git SHA |" "$SESSION_FILE" | sed 's/.*| //' | sed 's/ |$//' | xargs)
-    CACHE_RESULT=$(grep "| Result |" "$SESSION_FILE" | sed 's/.*| //' | sed 's/ |$//' | xargs)
-    CACHE_TIME=$(grep "| Last Run |" "$SESSION_FILE" | sed 's/.*| //' | sed 's/ |$//' | xargs)
-
-    # Validate: SHA must match current HEAD
-    if [[ "$CACHE_SHA" == "$CURRENT_SHA" ]]; then
-        echo "✓ Cache SHA matches current HEAD"
-
-        # Validate: Cache must be less than 5 minutes old
-        # (Use date command appropriate for your OS)
-        CACHE_EPOCH=$(date -j -f "%Y-%m-%dT%H:%M:%SZ" "$CACHE_TIME" +%s 2>/dev/null || date -d "$CACHE_TIME" +%s 2>/dev/null || echo 0)
-        NOW_EPOCH=$(date +%s)
-        AGE_MINUTES=$(( (NOW_EPOCH - CACHE_EPOCH) / 60 ))
-
-        if [[ $AGE_MINUTES -lt 5 ]]; then
-            echo "✓ Using cached test result: $CACHE_RESULT (${AGE_MINUTES}m old)"
-            # SKIP testing-runner spawn - use cached result in preflight report
-            USE_CACHED_TESTS=true
-        else
-            echo "⚠ Cache too old (${AGE_MINUTES}m), running fresh tests"
-        fi
-    else
-        echo "⚠ Cache SHA mismatch, running fresh tests"
-    fi
+
+if test_cache_valid "$SESSION_FILE"; then
+    CACHED_RESULT=$(test_cache_get "$SESSION_FILE" "result")
+    echo "Using cached: $CACHED_RESULT"
+else
+    echo "No cache, running tests"
 fi
 ```
 
-**Decision:**
-- If `USE_CACHED_TESTS=true`: Skip step 3, use cached result in output
-- Otherwise: Proceed to step 3 (spawn testing-runner)
-
-### 3. Run Tests and Lints via Testing Runner
+## 3. Run Tests (if no cache)
 
-**DELEGATE TO TESTING-RUNNER SUBAGENT:**
-
-Spawn a testing-runner subagent with:
 ```yaml
 Task tool:
   subagent_type: "general-purpose"
   model: "haiku"
-  description: "run tests"
   prompt: |
-    Read and follow: .pennyfarthing/agents/testing-runner.md
+    You are the testing-runner subagent.
+
+    Read .pennyfarthing/agents/testing-runner.md for your instructions,
+    then EXECUTE all steps described there. Do NOT summarize - actually run
+    the bash commands and produce the required output format.
 
     REPOS: {REPOS}
     CONTEXT: PR review pre-flight for Story {STORY_ID}
     RUN_ID: {STORY_ID}-review
 ```
 
-If you cannot spawn a subagent, run the tests directly using the testing skill commands.
+## 4. Code Smells
 
-### 3. Code Smell Detection (in changed files only)
-Search for these patterns in the diff:
-- `console.log` (not wrapped in `import.meta.env.DEV`)
+Search changed files for:
+- `console.log` (not in DEV guard)
 - `dangerouslySetInnerHTML`
-- `t.Skip(` or `it.skip(` or `.skip(`
-- `TODO` or `FIXME` comments
-- Non-null assertions `!` without preceding null check
+- `.skip(` test skips
+- `TODO` / `FIXME`
 
-### 4. Error Boundary Check (UI only)
-If new routes added, check App.tsx for `withRouteErrorBoundary` usage.
+## 5. PR Details
 
-### 5. Get PR Details
 ```bash
 gh pr view {PR_NUMBER} --json title,body,additions,deletions,changedFiles
 ```
@@ -133,50 +81,16 @@ gh pr view {PR_NUMBER} --json title,body,additions,deletions,changedFiles
 ## Pre-Flight Report: Story {STORY_ID}
 
 ### Test Results
-(Include output from testing-runner subagent)
-
-| Repo | Total | Passed | Failed | Skipped | Status |
-|------|-------|--------|--------|---------|--------|
-| API  | {N}   | {N}    | {N}    | {N}     | {GREEN/RED/YELLOW} |
-| UI   | {N}   | {N}    | {N}    | {N}     | {GREEN/RED/YELLOW} |
-
-#### Failing Tests (if any)
-| Repo | Test Name | File | Error |
-|------|-----------|------|-------|
-| {repo} | {test name} | {file path} | {brief error} |
-
-#### Skipped Tests (if any - POLICY VIOLATION)
-| Repo | Test Name | File |
-|------|-----------|------|
-| {repo} | {test name} | {file} |
-
-### Lint Results
-| Repo | Errors | Warnings |
-|------|--------|----------|
-| API  | {N}    | {N}      |
-| UI   | {N}    | {N}      |
-
-### Code Smells Found
+| Repo | Passed | Failed | Status |
+|------|--------|--------|--------|
+
+### Code Smells
 | Pattern | Count | Files |
 |---------|-------|-------|
-| console.log | {N} | {file list} |
-| dangerouslySetInnerHTML | {N} | {file list} |
-| Skipped tests (.skip) | {N} | {file list} |
-| TODO/FIXME | {N} | {file list} |
-
-### Error Boundaries
-- New routes: {list or "none"}
-- Wrapped: {yes|no|n/a}
 
 ### Diff Stats
-- Files changed: {N}
-- Additions: +{N}
-- Deletions: -{N}
+- Files: {N}, +{additions}, -{deletions}
 
 ### Files to Review
-{list of changed files with brief description}
-
-### Log Files
-- Tests: `.session/test-{STORY_ID}-reviewer-verify.log`
-- Lint: `.session/lint-{STORY_ID}-{repo}.log`
+{list}
 ```