@codeyam/codeyam-cli 0.1.0-staging.c90f8c9 → 0.1.0-staging.c9dc00c

package/codeyam-cli/templates/codeyam:diagnose.md

@@ -2,100 +2,60 @@
 
 When a user asks to debug a scenario or analysis (e.g., `/codeyam:diagnose [ID]`), use this systematic process.
 
-## Goal
+## Goal & Rules
 
-**Capture quality screenshots** by fixing all errors, then document what was wrong. Every issue can be fixed locally—the question is whether the fix is in the database (Categories 1-3) or in the tmp folder code (Category 4). Either way, you must capture screenshots before concluding.
+**Goal:** Capture quality screenshots by fixing all errors, then document what was wrong.
 
-## Definition of Done
+**Definition of Done** — debugging is NOT complete until one of:
 
-**Debugging is NOT complete until one of these conditions is met:**
+| Outcome                | What's Required                                                                                                                               |
+| ---------------------- | --------------------------------------------------------------------------------------------------------------------------------------------- |
+| **SUCCESS (DB Fix)**   | Issue fixed via database/config changes, screenshots recaptured, all files verified on disk AND visually inspected                            |
+| **SUCCESS (File Fix)** | Issue fixed via tmp folder edits, screenshots recaptured with `--capture-only`, all files verified AND visually inspected, changes documented |
+| **UNFIXABLE**          | Issue cannot be fixed locally (rare), documented in final report with full details                                                            |
 
-1. **SUCCESS (Database Fix)**: Issue fixed via database/config changes, screenshots recaptured, ALL screenshot files verified on disk AND visually inspected
-2. **SUCCESS (File Fix)**: Issue fixed via tmp folder edits, screenshots recaptured with `--capture-only`, ALL screenshot files verified AND visually inspected, changes documented for CodeYam team
-3. **UNFIXABLE**: Issue cannot be fixed locally (rare), documented in final report with full details
+### KEY RULE 1: Never stop before capturing screenshots
 
-**CRITICAL: NEVER stop debugging before capturing quality screenshots.** Even if you identify the root cause as a "Category 4 engine bug", you MUST:
+Category 4 ("engine bug") is a **diagnosis category, not a blocker**. You have full access to edit any file in `/tmp/codeyam/local-dev/{slug}/project/` — the "engine" just wrote those files as plain TypeScript/JavaScript. Every issue can be fixed locally.
 
-1. Fix the issue manually in the tmp folder (this is always possible)
-2. Run `codeyam recapture --capture-only` to capture screenshots with your fixes in place
-3. Verify the screenshots show the component rendering correctly
-4. ONLY THEN document the engine bug for the CodeYam team
+If you find yourself thinking "this requires an engine-level fix" or "this is blocked by a code generation bug" — **STOP**.
 
-Stopping after identifying a bug category without capturing screenshots defeats the purpose of debugging.
+- Fix the files in the tmp folder, then capture screenshots, THEN document the bug
+- Try at least 3 different approaches before asking the user whether to continue
+- Only stop when the user agrees or you've captured quality screenshots
 
-## Critical: Persistence and User Communication
+### KEY RULE 2: Original Repo vs Tmp Folder
 
-**DO NOT give up prematurely.** Every issue can be fixed by editing files in the tmp folder. If you find yourself thinking:
+| Location                                                  | What's There                                                        | Use For                                               |
+| --------------------------------------------------------- | ------------------------------------------------------------------- | ----------------------------------------------------- |
+| **Original Repo** (your working directory)                | Database (`.codeyam/db.sqlite3`), config, universal mocks, captures | **Run all `codeyam` commands here**                   |
+| **Tmp Folder** (`/tmp/codeyam/local-dev/{slug}/project/`) | Copy of project with mock injections, generated scenario files      | **Only edit files here — never run codeyam commands** |
 
-- "This requires an engine-level fix"
-- "This is blocked by a code generation bug"
-- "The engine doesn't properly handle X"
+Running `codeyam recapture` from the tmp folder fails with "Not in a CodeYam project" because the database isn't there.
 
-**STOP.** These statements are incorrect. You have full access to edit any file in `/tmp/codeyam/local-dev/{slug}/project/`. The "engine" just wrote those files—they're plain TypeScript/JavaScript that you can edit directly.
+### KEY RULE 3: Never modify client code
 
-### Before Stopping or Concluding
+Fix the **mock data or mock code** — never the client's components or application code. The client's code is sacred. Our job is to make the simulation environment correct. You CAN modify:
 
-1. **Try at least 3 different approaches** to fix the issue
-2. **Never unilaterally decide to stop.** If you've been debugging and aren't making progress, ASK THE USER:
-   > "I've tried X, Y, and Z but haven't resolved the issue yet. Would you like me to continue investigating other approaches, or should I document what I've found so far?"
-3. **Only stop when the user agrees** or when you've successfully captured quality screenshots
+- Database data (`.codeyam/db.sqlite3`) — in Original Repo
+- Universal mocks (`.codeyam/universal-mocks`) — in Original Repo
+- Generated files in tmp folder (`__codeyamMocks__/`, layouts, routes)
 
-### The "Engine Bug" Misconception
+### KEY RULE 4: `--capture-only` preserves tmp folder edits
 
-When you identify a Category 4 issue (code generation bug), this does NOT mean you're blocked. It means:
+- `codeyam recapture ID` — regenerates files from database, then captures (overwrites tmp edits)
+- `codeyam recapture ID --capture-only` — captures without regenerating (preserves your edits)
+- `codeyam debug ID` — regenerates files only (also overwrites tmp edits)
 
-- The bug is in how CodeYam generates code (useful info for the CodeYam team)
-- You MUST still fix it manually by editing files in the tmp folder
-- You MUST still capture quality screenshots
-- You document what you fixed so the CodeYam team can improve the engine
-
-**"Engine bug" is a diagnosis category, not an excuse to stop.**
+If you edited files in the tmp folder, you **MUST** use `--capture-only` when recapturing.
 
 ---
 
-## Debug Flow Summary
+## Setup
 
-```
-┌─────────────────────────────────────────────────────────────────────┐
-│  Step 0: Query database for scenario status (DO THIS FIRST!)        │
-│          Check ALL scenarios in the analysis for failures           │
-│          Retrieve the analysis id to use in the debug command       │
-│                    │                                                │
-│                    ▼                                                │
-│  Step 1: Copy the logs to a temporary location (see below).         |
-|  Step 2: codeyam debug [ANALYSIS ID]  (generates files to inspect)  │
-│  Step 3: Identify error type from logs/output                       │
-│  Step 4: Diagnose root cause category (1-4)                         │
-│  Step 5: Attempt fix (database edit or universal mock)              │
-│  Step 6: Verify locally (curl returns 200)                          │
-│                   │                                                 │
-│         ┌─────────┴─────────┐                                       │
-│         ▼                   ▼                                       │
-│   Fix worked           Fix didn't work                              │
-│   (DB changes)              │                                       │
-│         │                   ▼                                       │
-│         │         Step 5a: Fix code in tmp folder                   │
-│         │         (edit files directly, track changes)              │
-│         │                   │                                       │
-│         │                   ▼                                       │
-│         │         Verify without regenerating                       │
-│         │                   │                                       │
-│         ▼                   ▼                                       │
-│   Step 7: Recapture    Step 7: Recapture --capture-only             │
-│   (normal)             (preserves file fixes)                       │
-│         │                   │                                       │
-│         └─────────┬─────────┘                                       │
-│                   ▼                                                 │
-│   Step 8: Verify screenshots (check for client-side errors)         │
-│                   │                                                 │
-│                   ▼                                                 │
-│   Verify ALL files exist on disk                                    │
-│                   │                                                 │
-│                   ▼                                                 │
-│   ✅ DONE - Produce Final Debug Report                              │
-│   (document ALL changes for CodeYam team)                           │
-└─────────────────────────────────────────────────────────────────────┘
-```
+### Note Key Locations
+
+**Database:** The codeyam database relevant for this command should always be found in the repo where this command is called in the `.codeyam/db.sqlite3` file.
 
 **Logs:** Before debugging copy the original logs to a temorary location as the debugging will overwrite these logs and they are valuable for debugging issues.
 
@@ -117,15 +77,12 @@ At the **very start** of debugging, create `.codeyam/debug-session.md` in the **
 ```markdown
 # Debug Session Context
 
-**IMPORTANT**: Keep a todo item "📍 Re-read debug session context" that is NEVER marked complete.
-If the todo disappears, add it back immediately.
-
 ## Key Locations
 
-- **Original project**: [FULL PATH - run all codeyam commands here]
-- **Tmp folder**: /tmp/codeyam/local-dev/[SLUG]/project (edit files here, do NOT run codeyam commands)
-- **Database**: [ORIGINAL PROJECT]/.codeyam/db.sqlite3
-- **Logs**: /tmp/codeyam/local-dev/[SLUG]/codeyam/log.txt
+- **Original Repo**: [FULL PATH]
+- **Tmp Folder**: /tmp/codeyam/local-dev/[PROJECT SLUG]/project
+- **Database**: [Original Repo]/.codeyam/db.sqlite3
+- **Logs**: /tmp/codeyam/local-dev/[PROJECT SLUG]/codeyam/log.txt
 
 ## IDs
 
@@ -133,198 +90,109 @@ If the todo disappears, add it back immediately.
 - **Scenario ID**: [ID]
 - **Entity**: [NAME] at [FILE PATH]
 
-## URLs (fill in after running codeyam debug)
+## URLs (fill in after codeyam debug)
 
 - **Scenario URL**: http://localhost:[PORT]/static/codeyam-sample
-- **Dev server port**: [PORT from codeyam debug output]
-
-## Commands Reference
-
-All commands run from: [ORIGINAL PROJECT PATH]
-
-- `codeyam debug [ANALYSIS_ID]` - regenerate files (overwrites tmp folder!)
-- `codeyam recapture [ID]` - regenerate AND capture (overwrites tmp folder!)
-- `codeyam recapture [ID] --capture-only` - capture WITHOUT regenerating (preserves your edits)
-- `codeyam test-startup` - test startup (overwrites tmp folder!)
-```
-
-### Step 2: Create Persistent Todo
-
-Use TodoWrite to create this todo list with the first item being a persistent reminder:
-
-```
-📍 Re-read .codeyam/debug-session.md before any codeyam command (NEVER complete this)
-```
-
-**CRITICAL**: The first todo item must NEVER be marked as completed. It serves as a constant reminder to check your session context before running commands. If you accidentally complete it or it disappears, add it back immediately.
-
-### Step 3: Read Context Before Actions
-
-**Before running ANY of these commands**, re-read `.codeyam/debug-session.md`:
-
-- `codeyam debug`
-- `codeyam recapture`
-- `codeyam test-startup`
-- Any database queries
-
-This prevents running commands from the wrong directory or forgetting key IDs.
-
----
-
-## Critical: Where to Run Commands
-
-**All `codeyam` commands MUST be run from the ORIGINAL PROJECT DIRECTORY**, not from the tmp folder.
-
-| Location                                                               | What's There                                                   | Run Commands Here?                                |
-| ---------------------------------------------------------------------- | -------------------------------------------------------------- | ------------------------------------------------- |
-| **Original project** (e.g., `/Users/.../clients-codeyam/boltwise/app`) | Database (`.codeyam/db.sqlite3`), config, universal mocks      | **YES - run all codeyam commands here**           |
-| **Tmp folder** (`/tmp/codeyam/local-dev/{slug}/project/`)              | Copy of project with mock injections, generated scenario files | **NO - only edit files here, don't run commands** |
-
-**Common mistake:** Running `codeyam recapture` from the tmp folder. This will fail with "Not in a CodeYam project" because the database doesn't exist there.
-
-```bash
-# WRONG - running from tmp folder
-cd /tmp/codeyam/local-dev/boltwise-app/project
-codeyam recapture abc123  # ❌ Fails: "Not in a CodeYam project"
-
-# CORRECT - running from original project directory
-cd /Users/jaredcosulich/workspace/codeyam/clients-codeyam/boltwise/app
-codeyam recapture abc123  # ✅ Works
 ```
 
-**Workflow:**
+Re-read this file before running any `codeyam` command to avoid running commands from the wrong directory.
 
-1. Edit files in `/tmp/codeyam/local-dev/{slug}/project/` to fix issues
-2. Test by curling the dev server running from the tmp folder
-3. **Switch back to the original project directory** to run `codeyam recapture`
+### Copy Original Logs
 
----
+Copy logs before debugging overwrites them:
 
-## What You Can and Cannot Modify
-
-**NEVER modify the client's original source code in the workspace.** Only modify files in the tmp folder.
-
-**You CAN modify:**
-
-- Data in the local SQLite database (`.codeyam/db.sqlite3`) - in the **original project directory**
-- Universal mocks in `.codeyam/universal-mocks` - in the **original project directory**
-- **Files in `/tmp/codeyam/local-dev/{slug}/project/`** - These are temporary copies used for simulation
-  - Generated mock files in `__codeyamMocks__/` directories
-  - Layout/route files that wrap scenarios
-  - Any generated code that has errors
-- Mock data files for testing
-
-**IMPORTANT**: When you edit files in the tmp folder, you MUST use `codeyam recapture --capture-only` when recapturing, otherwise your changes will be overwritten! Be sure to address all issues so that the simulation runs correctly. Then, if one of the issues required editing code in the tmp directory you MUST use `codeyam recapture --capture-only`, but if only data was changed then as long as that data is updated in the local database as well you can run `codeyam recapture` and have the system overwrite the tmp project code and use the valid data from the database.
+- **From:** `/tmp/codeyam/local-dev/{slug}/codeyam/logs.txt`
+- **To:** `/tmp/codeyam/local-dev/{slug}/debug/original-logs.txt`
 
 ---
 
-## Step 0: Check for Other Failing Scenarios (MANDATORY FIRST STEP)
+## Step 1: Check Database (MANDATORY FIRST STEP)
 
-**DO NOT run `codeyam debug` yet!** First, query the database to check if other scenarios in the same analysis are also failing.
+**DO NOT run `codeyam debug` yet!** First check if other scenarios in the same analysis are also failing.
 
 ```bash
-# First, get the analysis ID (if you have a scenario ID)
+# Get analysis ID (if you have a scenario ID)
 sqlite3 .codeyam/db.sqlite3 "SELECT analysis_id FROM scenarios WHERE id = 'SCENARIO_ID'"
 
-# Then check ALL scenarios for that analysis
+# Check ALL scenarios for that analysis
 sqlite3 .codeyam/db.sqlite3 "
-SELECT
-  s.id,
-  s.name,
-  json_extract(s.metadata, '$.screenshotPaths') as screenshots
-FROM scenarios s
-WHERE s.analysis_id = 'ANALYSIS_ID'
-"
+SELECT s.id, s.name, json_extract(s.metadata, '$.screenshotPaths') as screenshots
+FROM scenarios s WHERE s.analysis_id = 'ANALYSIS_ID'"
 
 # Check for errors in scenario status
 sqlite3 .codeyam/db.sqlite3 "
-SELECT json_extract(status, '$.scenarios') FROM analyses WHERE id = 'ANALYSIS_ID'
-"
+SELECT json_extract(status, '$.scenarios') FROM analyses WHERE id = 'ANALYSIS_ID'"
 ```
 
-**If multiple scenarios are failing, ASK THE USER** whether to fix just the requested scenario or all failing scenarios.
+**If multiple scenarios failing, ASK THE USER** whether to fix just the requested one or all.
 
-### Check for Client-Side Errors in Screenshots
+### Check Existing Screenshots for Client-Side Errors
 
-**IMPORTANT:** Even if the database shows no errors and screenshots exist, the scenario may still have a client-side error. Use the Read tool to view the screenshot file and check for:
+Even if screenshots exist, view them with the Read tool and check for:
 
-- Red error boundaries or error messages in the rendered UI
+- Red error boundaries or React error overlays
 - "Something went wrong" or similar error text
 - Blank/white screens that should have content
-- React error overlays
 
-```bash
-# View the screenshot to check for client-side errors
-# Use the Read tool on the screenshot path from the database
-```
-
-If the screenshot shows a client-side error, proceed to Step 1 to debug it.
+If the screenshot shows errors, proceed to Step 2.
 
 ---
 
-## Step 1: Run Debug Command
+## Step 2: Run Debug Command
 
 ```bash
 codeyam debug ANALYSIS_ID
 ```
 
-Output includes:
-
-- **Project path**: `/tmp/codeyam/local-dev/{slug}/project`
-- **Start command**: How to run the dev server
-- **URL**: Where to access the scenario
-- **Log path**: Where to watch for errors
+Output includes: **project path**, **start command**, **URL**, **log path**. Update your session context file with these.
 
 ---
 
-## Step 2: Identify the Error Type
+## Step 3: Identify Error Type
 
-Read the log file or error output to categorize the issue:
+Read the log file or error output to categorize:
 
-| Error Type                         | Symptoms                            | Likely Category                          |
-| ---------------------------------- | ----------------------------------- | ---------------------------------------- |
-| **Syntax error in generated file** | `Unexpected token`, `Parse error`   | Category 4 (mock code writing)           |
-| **Runtime error**                  | `TypeError`, `ReferenceError`       | Category 1-3 (data issue) or 4           |
-| **Missing mock**                   | `Cannot read property of undefined` | Category 1-2 (missing in data structure) |
-| **Wrong mock type**                | `X is not a function`               | Category 3-4 (wrong data or mock code)   |
+| Error Type                         | Symptoms                            | Likely Category   |
+| ---------------------------------- | ----------------------------------- | ----------------- |
+| **Syntax error in generated file** | `Unexpected token`, `Parse error`   | Category 4        |
+| **Runtime error**                  | `TypeError`, `ReferenceError`       | Category 1-3 or 4 |
+| **Missing mock**                   | `Cannot read property of undefined` | Category 1-2      |
+| **Wrong mock type**                | `X is not a function`               | Category 3-4      |
 
 ---
 
-## Step 3: Diagnose Root Cause Category
+## Step 4: Diagnose Root Cause Category
 
-This is the most important step. Determine WHERE in the pipeline the bug originated.
+Determine WHERE in the pipeline the bug originated.
 
 ### The 4 Error Categories
 
-| Category | What's Wrong                                         | Where to Check            | Fixable Locally?              |
-| -------- | ---------------------------------------------------- | ------------------------- | ----------------------------- |
-| **1**    | Missing attribute in `isolatedDataStructure`         | `entities.metadata`       | Sometimes (re-analyze)        |
-| **2**    | Attribute not merged into `mergedDataStructure`      | `analyses.metadata`       | Sometimes (re-analyze)        |
-| **3**    | Data structure complete, but scenario data wrong     | `scenarios.metadata`      | Yes (edit mockData)           |
-| **4**    | Data complete, but mock code/mock data written wrong | Generated files in `/tmp` | **Yes (edit files directly)** |
+| Category | What's Wrong                                     | Where to Check       | Fix                 |
+| -------- | ------------------------------------------------ | -------------------- | ------------------- |
+| **1**    | Missing attribute in `isolatedDataStructure`     | `entities.metadata`  | Re-analyze          |
+| **2**    | Attribute not merged into `mergedDataStructure`  | `analyses.metadata`  | Re-analyze          |
+| **3**    | Data structure complete, but scenario data wrong | `scenarios.metadata` | Edit mockData in DB |
+| **4**    | Data complete, but generated code wrong          | Files in `/tmp`      | Edit files directly |
 
 ### Diagnostic Queries
 
 ```bash
 # Category 1: Check isolatedDataStructure
-# Is the attribute traced at all in the entity?
 sqlite3 .codeyam/db.sqlite3 \
   "SELECT json_extract(metadata, '$.isolatedDataStructure') FROM entities WHERE sha = 'ENTITY_SHA'" \
   | python3 -m json.tool
 
 # Category 2: Check mergedDataStructure
-# Was the attribute properly merged from dependencies?
 sqlite3 .codeyam/db.sqlite3 \
   "SELECT json_extract(metadata, '$.mergedDataStructure') FROM analyses WHERE id = 'ANALYSIS_ID'" \
   | python3 -m json.tool
 
-# Category 2 (also check): scenariosDataStructure
+# Category 2 (also): scenariosDataStructure
 sqlite3 .codeyam/db.sqlite3 \
   "SELECT json_extract(metadata, '$.scenariosDataStructure') FROM analyses WHERE id = 'ANALYSIS_ID'" \
   | python3 -m json.tool
 
 # Category 3: Check scenario mockData
-# Did the LLM generate correct mock data?
 sqlite3 .codeyam/db.sqlite3 \
   "SELECT json_extract(metadata, '$.data.mockData') FROM scenarios WHERE id = 'SCENARIO_ID'" \
   | python3 -m json.tool
@@ -332,49 +200,41 @@ sqlite3 .codeyam/db.sqlite3 \
 
 ### Diagnosis Flow
 
-1. **Start with the error** - What attribute/value is missing or wrong at runtime?
-2. **Check `isolatedDataStructure`** - Is the attribute traced in the entity? If missing → Category 1
-3. **Check `mergedDataStructure`** - Is the attribute present after merging? If missing → Category 2
-4. **Check `scenariosDataStructure`** - Is the data structure for mocks correct? If wrong → Category 2
-5. **Check scenario `mockData`** - Does the scenario have correct mock values? If wrong → Category 3
-6. **Check generated mock code** - Is the mock code syntactically/semantically correct? If wrong → **Category 4 (engine bug)**
-7. **Check generated imports** - Are the imports correct? Are entities containing environment variables mocked out? If wrong → **Category 4 (engine bug)**
-
-### Category 4 Indicators (Code Generation Issue)
+Work through these checks in order — stop at the first failure:
 
-The issue is Category 4 if:
+| Step | Check                                                                           | If wrong → |
+| ---- | ------------------------------------------------------------------------------- | ---------- |
+| 1    | **Start with the error** — what attribute/value is missing or wrong at runtime? | —          |
+| 2    | Is the attribute in `isolatedDataStructure`?                                    | Category 1 |
+| 3    | Is the attribute in `mergedDataStructure`?                                      | Category 2 |
+| 4    | Is `scenariosDataStructure` correct?                                            | Category 2 |
+| 5    | Does scenario `mockData` have correct values?                                   | Category 3 |
+| 6    | Is the generated mock code syntactically/semantically correct?                  | Category 4 |
+| 7    | Are imports correct? Environment variable entities mocked out?                  | Category 4 |
 
-- All data structures in the database look correct
-- The scenario's mockData has the right keys and values
-- BUT the generated mock code has syntax errors, wrong structure, or doesn't match the data
-- Database edits don't fix the generated code (it regenerates wrong)
+If all database data looks correct but the generated code has syntax errors, wrong structure, or doesn't match the data → **Category 4**.
 
-**Category 4 does NOT mean you are blocked.** You have full access to fix these issues:
-
-1. **Edit the files directly** in `/tmp/codeyam/local-dev/{slug}/project/`
-2. **Fix any syntax errors, wrong imports, missing mocks** - these are just TypeScript files
-3. **Restart the dev server** and verify the fix works
-4. **Capture quality screenshots** with `codeyam recapture --capture-only`
-5. **Document what you fixed** for the CodeYam team to improve the engine
+---
 
-The term "engine bug" means the CodeYam code generation has a bug—it does NOT mean you cannot fix the issue locally. You always can.
+## Step 5: Fix the Issue
 
----
+Jump to the subsection matching your category:
 
-## Step 4: Attempt Fix
+- **Categories 1-2** → Re-analyze
+- **Category 3** → Edit mockData in database
+- **Category 4** → Edit generated files in tmp folder
 
-### For Categories 1-2 (Data Structure Issues)
+### Categories 1-2 (Data Structure Issues)
 
-These usually require re-running analysis. You can try:
+Re-run analysis:
 
 ```bash
-# Delete the analysis and re-analyze
 codeyam analyze --entity EntityName path/to/file.tsx
 ```
 
-### For Category 3 (Scenario Data Issues)
+### Category 3 (Scenario Data Issues)
 
-Fix the mockData directly in the database:
+Fix mockData directly in the database:
 
 ```bash
 # Export metadata
@@ -393,14 +253,11 @@ with open('/tmp/metadata_fixed.json', 'w') as f:
     f.write(json.dumps(data))
 EOF
 
-# Update database (use Python to avoid escaping issues)
+# Update database
 python3 << 'EOF'
-import sqlite3
-import json
-
+import sqlite3, json
 with open('/tmp/metadata_fixed.json') as f:
     metadata = f.read()
-
 conn = sqlite3.connect('.codeyam/db.sqlite3')
 conn.execute("UPDATE scenarios SET metadata = ? WHERE id = 'SCENARIO_ID'", (metadata,))
 conn.commit()
@@ -408,242 +265,141 @@ conn.close()
 EOF
 ```
 
-### For Category 4 (Code Generation Issues)
-
-**MUST be fixed locally by editing files in the tmp folder.** This is a REQUIRED step, not optional.
-
-1. Go to Step 5a and fix all code issues in the tmp folder
-2. Verify the scenario loads correctly (curl returns 200)
-3. Run `codeyam recapture --capture-only` to capture screenshots (this preserves your fixes)
-4. Verify the screenshots look correct
-5. THEN document the engine bug in your final report
-
-**DO NOT stop after identifying a Category 4 issue.** The goal is to capture quality screenshots, then document what you fixed.
-
-### Universal Mocks (for infrastructure dependencies)
-
-You can use universal mocks to mock out functions that are not detected by the analysis (are not part of the dependency tree). This often includes things like middleware or functions that run on startup and require environment variables.
-
-## Quick Reference: Mock Path Decision Tree
-
-```
-Where is the error coming from?
-│
-├── node_modules package (e.g., @prisma/client, @supabase/supabase-js)
-│   └── Mock path: .codeyam/universal-mocks/node_modules/{package}.ts
-│       Example: .codeyam/universal-mocks/node_modules/@prisma/client.ts
-│
-└── Project file (e.g., lib/db.ts, packages/prisma/index.ts)
-    └── Mock path: .codeyam/universal-mocks/{same-path-as-original}
-        Example: .codeyam/universal-mocks/lib/db.ts
-        Example: .codeyam/universal-mocks/packages/prisma/index.ts
-```
-
-## Quick Reference: Mock Writing Rules
-
-**BEFORE writing any mock:**
-
-1. Read the original file first
-2. Note all its export names exactly
-
-**WHEN writing the mock:**
-
-1. Export names MUST match exactly (case-sensitive)
-2. ALL code MUST be inside exports (no helper variables outside)
-3. Keep it minimal - empty methods are fine
-
-**AFTER writing the mock:**
-
-```bash
-codeyam validate-mock .codeyam/universal-mocks/{path-to-your-mock}
-```
-
----
-
-## Step 5: Verify Database Fixes Locally
-
-After making **database changes** (Categories 1-3):
-
-1. **Re-run `codeyam debug ID`** to regenerate scenario files with updated data
-2. **Start the dev server** and curl the URL to check for errors:
-   ```bash
-   curl -s http://localhost:3112/static/... | head -50
-   ```
-3. **Check for 200 response** (no `statusCode:500`)
-
-**If verification fails and you've already tried database fixes:** Proceed to Step 5a to fix code directly.
-
----
-
-## Step 5a: Fix Code Issues in Tmp Folder (Category 4)
-
-If the error is in bad imports, generated code or mock files, you can fix them directly in the tmp folder:
-
-### Locate the Problematic File
+### Category 4 (Code Generation Issues) — Edit Files Directly
 
-Files are in `/tmp/codeyam/local-dev/{slug}/project/`. Common locations:
+Fix generated files in `/tmp/codeyam/local-dev/{slug}/project/`. Common locations:
 
 - **Generated mocks**: `__codeyamMocks__/MockData_*.tsx` or `__codeyamMocks__/MockCode_*.tsx`
 - **Scenario layouts**: `app/static/{projectSlug}/{analysisId}/{entitySlug}/{scenarioSlug}/`
 - **Route files**: `app/routes/` or `pages/` depending on framework
 
-### Make the Fix
+Common fixes:
 
-Edit the problematic file directly using the Edit tool. Common fixes include:
+- Syntax errors (missing brackets, commas, quotes)
+- Wrong import paths
+- Missing named or default exports
+- TypeScript type issues
+- Wrong mock data shape
 
-- **Syntax errors**: Fix missing brackets, commas, or quotes in generated code
-- **Wrong import paths**: Correct import statements
-- **Missing exports**: Add missing named or default exports
-- **Type errors**: Fix TypeScript type issues
-- **Wrong mock structure**: Adjust the shape of generated mock data
+**Only fix GENERATED files** (mocks, layouts, routes) — never the client's application code. If a client component crashes, the fix is in the mock data or mock code.
 
-### Track Your Changes (MANDATORY)
-
-For every file you edit, record:
-
-- The full file path
-- What you changed (brief description)
-- The before/after code (for the final report)
-
-### Verify Without Regenerating
+### Universal Mocks (for infrastructure dependencies)
 
-After editing files, **do NOT run `codeyam debug` again** (it would overwrite your changes).
+Use universal mocks to mock functions not in the dependency tree (middleware, startup functions needing env vars).
 
-Instead, manually start the dev server:
+**Mock path decision tree:**
 
-```bash
-cd /tmp/codeyam/local-dev/{slug}/project
-npm run dev  # or pnpm dev / yarn dev
 ```
-
-Then curl the scenario URL to verify the fix works:
-
-```bash
-curl -s http://localhost:3112/static/codeyam-sample | head -50
+Error source?
+├── node_modules package → .codeyam/universal-mocks/node_modules/{package}.ts
+│   Example: .codeyam/universal-mocks/node_modules/@prisma/client.ts
+└── Project file → .codeyam/universal-mocks/{same-path-as-original}
+    Example: .codeyam/universal-mocks/lib/db.ts
 ```
 
----
+**Mock writing rules:**
 
-## Change Tracking (MANDATORY)
+1. Read the original file first, note all export names exactly
+2. Export names MUST match exactly (case-sensitive)
+3. ALL code MUST be inside exports (no helper variables outside)
+4. Keep it minimal — empty methods are fine
+5. Validate: `codeyam validate-mock .codeyam/universal-mocks/{path}`
 
-**You MUST track all changes for the final report.** This is essential for the CodeYam team to fix engine bugs.
+### Track All Changes (MANDATORY)
 
-### Database Changes
+For every change, record for the final report:
 
-For each database modification, record:
+**Database changes:**
 
 | Table       | Record ID | Field Path                   | Before    | After       |
 | ----------- | --------- | ---------------------------- | --------- | ----------- |
 | `scenarios` | `abc123`  | `metadata.data.mockData.key` | `"wrong"` | `"correct"` |
 
-### File Changes
+**File changes:**
 
-For each file modification in the tmp folder, record:
+| File Path                                            | Change Description  |
+| ---------------------------------------------------- | ------------------- |
+| `/tmp/.../project/__codeyamMocks__/MockData_xyz.tsx` | Fixed missing comma |
 
-| File Path                                            | Change Description                    |
-| ---------------------------------------------------- | ------------------------------------- |
-| `/tmp/.../project/__codeyamMocks__/MockData_xyz.tsx` | Fixed missing comma in object literal |
+Include actual diffs for file changes.
 
-Include the actual diff:
+---
 
-```diff
---- before
-+++ after
-@@ -10,3 +10,3 @@
--  key: "value"
-+  key: "value",
-   anotherKey: "value2"
-```
+## Step 6: Verify Fix
 
----
+### After Database Changes (Categories 1-3)
+
+1. Re-run `codeyam debug ANALYSIS_ID` to regenerate files with updated data
+2. Curl the URL to check for errors:
+   ```bash
+   curl -s http://localhost:PORT/static/codeyam-sample | head -50
+   ```
+3. Check for 200 response (no `statusCode:500`)
+
+If verification fails after database fixes, proceed to fix files directly (Category 4 approach).
+
+### After File Changes (Category 4)
+
+**Do NOT run `codeyam debug` again** — it overwrites your changes.
+
+1. Start the dev server manually from the tmp folder
+2. Curl the URL to check for errors
+3. For visual/rendering issues, ask the user to verify: _"Can you visit http://localhost:PORT/static/codeyam-sample and confirm the component renders correctly?"_
 
-## Step 6: Recapture Screenshots
+---
 
-**Only proceed here if Step 5 or 5a verification passed.**
+## Step 7: Recapture & Verify Screenshots
 
-### Choose the Right Recapture Mode
+### Choose Recapture Mode
 
-**If you only made DATABASE changes (Categories 1-3):**
+**Database-only changes:**
 
 ```bash
 codeyam recapture SCENARIO_ID
 ```
 
-This will regenerate the tmp folder files using the updated database data, then capture screenshots.
-
-**If you made FILE changes in the tmp folder (Category 4):**
+**File changes (or mixed):**
 
 ```bash
 codeyam recapture SCENARIO_ID --capture-only
 ```
 
-**CRITICAL**: The `--capture-only` flag preserves your manual file fixes. Without this flag, your changes would be overwritten!
-
-### Verify Files After Capture
+### Verify Files Exist
 
 ```bash
-# Check database for screenshot paths
 sqlite3 .codeyam/db.sqlite3 "SELECT name, json_extract(metadata, '$.screenshotPaths') FROM scenarios WHERE analysis_id = 'ANALYSIS_ID'"
-
-# Verify files actually exist
 find .codeyam/captures -name "*.png" -path "*ANALYSIS_ID*" -mmin -5
 ```
 
----
-
-## Step 7: Verify Screenshots for Client-Side Errors
-
-**IMPORTANT**: Even if capture completes successfully, the screenshots may show client-side errors!
-
-### View the Screenshots
+### Visually Inspect Screenshots
 
-Use the Read tool to view each captured screenshot file:
+Use the Read tool to view each screenshot. Check for:
 
-```bash
-# Get the screenshot path from the database
-sqlite3 .codeyam/db.sqlite3 "SELECT json_extract(metadata, '$.screenshotPaths[0]') FROM scenarios WHERE id = 'SCENARIO_ID'"
-```
-
-Then read the screenshot file to visually inspect it.
-
-### Check for These Issues
-
-- **Red error boundaries** or React error overlays
-- **"Something went wrong"** or similar error text
-- **Blank/white screens** that should have content
-- **Missing components** or broken layouts
-- **Console errors** shown in error overlays
-
-If the screenshot shows a dedicated 500 page with no clear error then you may want to find and comment out any error boundary logic that is catching errors and preventing you from easily reading them.
+- Red error boundaries or React error overlays
+- "Something went wrong" or similar error text
+- Blank/white screens that should have content
+- Missing components or broken layouts
 
-### If Screenshot Shows Errors
+If the screenshot shows a dedicated 500 page with no clear error, consider commenting out error boundary logic to expose the real error.
 
-1. The fix didn't fully work - go back to Step 5 or 5a
-2. Check the log file for additional error details:
-   ```bash
-   tail -100 /tmp/codeyam/local-dev/{slug}/codeyam/log.txt
-   ```
-3. Fix the remaining issues
-4. Run recapture again (with `--capture-only` if you edited files)
+**If screenshots show errors:** Go back to Step 5, check the log file (`tail -100 /tmp/codeyam/local-dev/{slug}/codeyam/log.txt`), fix remaining issues, and recapture again.
 
 ---
 
-## Final Debug Report (MANDATORY)
+## Reference
 
-**Always produce this report**, whether the issue was fixed or identified as an engine bug.
+### Debug Report Template
 
----
+**Always produce this report** after debugging. Save to `.codeyam/debug-report.md`.
 
+```markdown
 ## Debug Session Report
 
 ### Issue Summary
 
-**Error:** `[Exact error message, e.g., "Transform failed: Unexpected token at line 66"]`
-
+**Error:** `[Exact error message]`
 **Entity:** `[Entity name]` in `[file/path.tsx]`
-
 **Root Cause Category:** [1, 2, 3, or 4]
-
 **Outcome:** [Fixed via database changes / Fixed via file edits (engine bug) / Unfixable]
 
 ### Affected Scenarios
@@ -655,115 +411,58 @@ If the screenshot shows a dedicated 500 page with no clear error then you may wa
 ### Root Cause Category Determination
 
 **Category identified:** [1 / 2 / 3 / 4]
-
 **Evidence:**
 
-- [ ] **Category 1** (isolatedDataStructure): [What was missing/wrong in entity metadata?]
-- [ ] **Category 2** (mergedDataStructure): [What was missing/wrong after merge?]
-- [ ] **Category 3** (scenario mockData): [What was wrong in the LLM-generated data?]
-- [ ] **Category 4** (mock code writing): [What was wrong in the generated code?]
+- [ ] **Category 1** (isolatedDataStructure): [What was missing/wrong?]
+- [ ] **Category 2** (mergedDataStructure): [What was missing/wrong?]
+- [ ] **Category 3** (scenario mockData): [What was wrong in LLM-generated data?]
+- [ ] **Category 4** (mock code writing): [What was wrong in generated code?]
 
-**How determined:**
-[Explain what you checked and what you found. Include relevant JSON snippets.]
+**How determined:** [Explain what you checked. Include relevant JSON snippets.]
 
 ### Database Changes Made (if any)
 
-| Table     | Record ID | Field Path | Before        | After         |
-| --------- | --------- | ---------- | ------------- | ------------- |
-| `[table]` | `[id]`    | `[path]`   | `[old value]` | `[new value]` |
+| Table     | Record ID | Field Path | Before  | After   |
+| --------- | --------- | ---------- | ------- | ------- |
+| `[table]` | `[id]`    | `[path]`   | `[old]` | `[new]` |
 
 **SQL/Python commands used:**
-
-```sql
-[Include the exact commands]
-```
+[Include exact commands]
 
 ### File Changes Made (if any)
 
-If you edited files in the tmp folder to fix Category 4 issues:
-
-| File Path                                                | Change Description         |
-| -------------------------------------------------------- | -------------------------- |
-| `/tmp/codeyam/local-dev/{slug}/project/path/to/file.tsx` | [Brief description of fix] |
+| File Path                                                | Change Description  |
+| -------------------------------------------------------- | ------------------- |
+| `/tmp/codeyam/local-dev/{slug}/project/path/to/file.tsx` | [Brief description] |
 
 **Diffs:**
-
-```diff
---- before
-+++ after
-@@ -line,count +line,count @@
--[old code]
-+[new code]
-```
+[Include actual diffs]
 
 **Recapture command used:**
+`codeyam recapture SCENARIO_ID [--capture-only]`
 
-```bash
-codeyam recapture SCENARIO_ID --capture-only
-```
-
-### For Category 4 (Code Generation Bug) - Required Details
-
-If this is an engine bug, include:
-
-**What the data looks like (correct):**
-
-```json
-[Show the relevant data structure from the database that looks correct]
-```
-
-**What the generated code looks like (wrong):**
+### For Category 4 — Required Details
 
-```javascript
-[Show the problematic generated mock code]
-```
-
-**What the generated code should look like:**
-
-```javascript
-[Show what correct code would look like]
-```
-
-**Pattern/Edge case:**
-[Describe the pattern that causes this bug, e.g., "Function call signatures like '()' in array item schemas get converted to empty computed property keys"]
+**What the data looks like (correct):** [database data structure]
+**What the generated code looks like (wrong):** [problematic code]
+**What the generated code should look like:** [correct code]
+**Pattern/Edge case:** [Describe the pattern that causes this bug]
 
 ### Verification
 
-**All of these must be checked before the debug session is complete:**
-
 - [ ] Scenario loads without errors (curl returns 200)
-- [ ] Screenshots recaptured successfully (this is REQUIRED, not optional)
+- [ ] Screenshots recaptured successfully
 - [ ] Used `--capture-only` flag (if file changes were made)
 - [ ] Screenshot files verified to exist on disk
-- [ ] Screenshots visually inspected - no client-side errors visible
-- [ ] All changes documented above for CodeYam team
-
----
-
-## Saving and Uploading the Report (MANDATORY)
-
-After completing the debug session, **you MUST save the report**:
-
-1. **Write the report to file:**
-
-   ```bash
-   # Write the full "Debug Session Report" section above to this file
-   # Use the Write tool to save to:
-   .codeyam/debug-report.md
-   ```
-
-2. **Ask user if they want to upload:**
-   After saving, ask the user: "Would you like me to upload this debug report to CodeYam for the team to review?"
-
-- If the user response affirmatively then run `codeyam report --upload` to upload the report.
-
-The report will be:
+- [ ] Screenshots visually inspected — no client-side errors visible
+- [ ] All changes documented for CodeYam team
+```
 
-- Included in the delta tarball at the root level for easy access
-- Stored in the database metadata for searchability
-- Available to CodeYam engineers when they download the report
+### Saving & Uploading Report
 
----
+1. Save report to `.codeyam/debug-report.md`
+2. Ask user: "Would you like me to upload this debug report to CodeYam for the team to review?"
+3. If yes: `codeyam report --upload`
 
 ### Helper Queries