@cluesmith/codev 2.0.0-rc.7 → 2.0.0-rc.9

package/dist/commands/porch/index.js

@@ -1,1213 +1,521 @@
 /**
  * Porch - Protocol Orchestrator
  *
- * Generic loop orchestrator that reads protocol definitions from JSON
- * and executes them with Claude. Implements the Ralph pattern: fresh
- * context per iteration with state persisted to files.
+ * Claude calls porch as a tool; porch returns prescriptive instructions.
+ * All commands produce clear, actionable output.
  */
 import * as fs from 'node:fs';
 import * as path from 'node:path';
-import * as readline from 'node:readline';
-import { spawn } from 'node:child_process';
 import chalk from 'chalk';
-import { findProjectRoot, getSkeletonDir } from '../../lib/skeleton.js';
-import { getProjectDir, readState, writeState, createInitialState, updateState, approveGate, requestGateApproval, updatePhaseStatus, setPlanPhases, findProjects, findExecutions, findStatusFile, findPendingGates, getConsultationAttempts, incrementConsultationAttempts, resetConsultationAttempts, } from './state.js';
-import { extractPhasesFromPlanFile, findPlanFile, } from './plan-parser.js';
-import { extractSignal } from './signal-parser.js';
-import { runPhaseChecks, formatCheckResults } from './checks.js';
-import { runConsultationLoop, formatConsultationResults, hasConsultation, } from './consultation.js';
-import { loadProtocol as loadProtocolFromLoader, listProtocols as listProtocolsFromLoader, } from './protocol-loader.js';
-import { createNotifier, } from './notifications.js';
+import { readState, writeState, createInitialState, findStatusPath, getStatusPath, } from './state.js';
+import { loadProtocol, getPhaseConfig, getNextPhase, getPhaseChecks, getPhaseGate, isPhased, getPhaseCompletionChecks, } from './protocol.js';
+import { findPlanFile, extractPhasesFromFile, getCurrentPlanPhase, getPhaseContent, advancePlanPhase, allPlanPhasesComplete, } from './plan.js';
+import { runPhaseChecks, formatCheckResults, allChecksPassed, } from './checks.js';
 // ============================================================================
-// Protocol Loading (delegates to protocol-loader.ts)
+// Output Helpers
 // ============================================================================
-/**
- * List available protocols
- * Delegates to protocol-loader.ts for proper conversion
- */
-export function listProtocols(projectRoot) {
-    const root = projectRoot || findProjectRoot();
-    return listProtocolsFromLoader(root);
+function header(text) {
+    const line = '═'.repeat(50);
+    return `${line}\n  ${text}\n${line}`;
 }
-/**
- * Load a protocol definition
- * Delegates to protocol-loader.ts which properly converts steps→substates
- */
-export function loadProtocol(name, projectRoot) {
-    const root = projectRoot || findProjectRoot();
-    const protocol = loadProtocolFromLoader(root, name);
-    if (!protocol) {
-        throw new Error(`Protocol not found: ${name}\nAvailable protocols: ${listProtocols(root).join(', ')}`);
-    }
-    return protocol;
-}
-/**
- * Load a prompt file for a phase
- */
-function loadPrompt(protocol, phaseId, projectRoot) {
-    const phase = protocol.phases.find(p => p.id === phaseId);
-    if (!phase?.prompt) {
-        return null;
-    }
-    // New structure: protocols/<protocol>/prompts/<prompt>.md
-    const promptPaths = [
-        path.join(projectRoot, 'codev', 'protocols', protocol.name, 'prompts', phase.prompt),
-        path.join(getSkeletonDir(), 'protocols', protocol.name, 'prompts', phase.prompt),
-        // Legacy paths
-        path.join(projectRoot, 'codev', 'porch', 'prompts', phase.prompt),
-        path.join(getSkeletonDir(), 'porch', 'prompts', phase.prompt),
-    ];
-    for (const promptPath of promptPaths) {
-        if (fs.existsSync(promptPath)) {
-            return fs.readFileSync(promptPath, 'utf-8');
-        }
-        // Try with .md extension
-        if (fs.existsSync(`${promptPath}.md`)) {
-            return fs.readFileSync(`${promptPath}.md`, 'utf-8');
-        }
-    }
-    return null;
+function section(title, content) {
+    return `\n${chalk.bold(title)}:\n${content}`;
 }
 // ============================================================================
-// Protocol Helpers
-// ============================================================================
-/**
- * Check if a phase is terminal
- */
-function isTerminalPhase(protocol, phaseId) {
-    const phase = protocol.phases.find(p => p.id === phaseId);
-    return phase?.terminal === true;
-}
-/**
- * Find the phase that has a gate blocking after the given state
- */
-function getGateForState(protocol, state) {
-    const [phaseId, substate] = state.split(':');
-    const phase = protocol.phases.find(p => p.id === phaseId);
-    if (phase?.gate && phase.gate.after === substate) {
-        const gateId = `${phaseId}_approval`;
-        return { gateId, phase };
-    }
-    return null;
-}
-/**
- * Get next state after gate passes
- */
-function getGateNextState(protocol, phaseId) {
-    const phase = protocol.phases.find(p => p.id === phaseId);
-    return phase?.gate?.next || null;
-}
-/**
- * Get signal-based next state
- */
-function getSignalNextState(protocol, phaseId, signal) {
-    const phase = protocol.phases.find(p => p.id === phaseId);
-    return phase?.signals?.[signal] || null;
-}
-/**
- * Get the default next state for a phase (first substate or next phase)
- */
-function getDefaultNextState(protocol, state) {
-    const [phaseId, substate] = state.split(':');
-    const phase = protocol.phases.find(p => p.id === phaseId);
-    if (!phase)
-        return null;
-    // If phase has substates, move to next substate
-    if (phase.substates && substate) {
-        const currentIdx = phase.substates.indexOf(substate);
-        if (currentIdx >= 0 && currentIdx < phase.substates.length - 1) {
-            return `${phaseId}:${phase.substates[currentIdx + 1]}`;
-        }
-    }
-    // Move to next phase
-    const phaseIdx = protocol.phases.findIndex(p => p.id === phaseId);
-    if (phaseIdx >= 0 && phaseIdx < protocol.phases.length - 1) {
-        const nextPhase = protocol.phases[phaseIdx + 1];
-        if (nextPhase.substates && nextPhase.substates.length > 0) {
-            return `${nextPhase.id}:${nextPhase.substates[0]}`;
-        }
-        return nextPhase.id;
-    }
-    return null;
-}
-// ============================================================================
-// Claude Invocation
+// Commands
 // ============================================================================
-// Note: extractSignal is now imported from signal-parser.js
 /**
- * Rolling buffer for last N lines of output
+ * porch status <id>
+ * Shows current state and prescriptive next steps.
  */
-class OutputBuffer {
-    lines = [];
-    currentLine = '';
-    maxLines;
-    constructor(maxLines = 20) {
-        this.maxLines = maxLines;
-    }
-    append(data) {
-        const text = data.toString();
-        for (const char of text) {
-            if (char === '\n') {
-                this.lines.push(this.currentLine);
-                if (this.lines.length > this.maxLines) {
-                    this.lines.shift();
-                }
-                this.currentLine = '';
-            }
-            else {
-                this.currentLine += char;
-            }
-        }
-    }
-    getContext() {
-        return this.currentLine
-            ? [...this.lines, this.currentLine]
-            : [...this.lines];
-    }
-}
-/**
- * Invoke Claude for a phase with escape-to-cancel and output buffering
- */
-async function invokeClaude(protocol, phaseId, state, statusFilePath, projectRoot, options) {
-    const promptContent = loadPrompt(protocol, phaseId, projectRoot);
-    if (!promptContent) {
-        console.log(chalk.yellow(`[porch] No prompt file for phase: ${phaseId}`));
-        return '';
-    }
-    if (options.dryRun) {
-        console.log(chalk.yellow(`[porch] [DRY RUN] Would invoke Claude for phase: ${phaseId}`));
-        return '';
-    }
-    if (options.noClaude) {
-        console.log(chalk.blue(`[porch] [NO_CLAUDE] Simulating phase: ${phaseId}`));
-        await new Promise(r => setTimeout(r, 1000));
-        console.log(chalk.green(`[porch] Simulated completion of phase: ${phaseId}`));
-        return '';
-    }
-    console.log(chalk.cyan(`[phase] Invoking Claude for phase: ${phaseId}`));
-    console.log(chalk.gray(`[porch] Press ESC to cancel`));
+export async function status(projectRoot, projectId) {
+    const statusPath = findStatusPath(projectRoot, projectId);
+    if (!statusPath) {
+        throw new Error(`Project ${projectId} not found.\nRun 'porch init' to create a new project.`);
+    }
+    const state = readState(statusPath);
+    const protocol = loadProtocol(projectRoot, state.protocol);
+    const phaseConfig = getPhaseConfig(protocol, state.phase);
+    // Header
     console.log('');
-    const timeout = protocol.config?.claude_timeout || 600000; // 10 minutes default
-    const startTime = Date.now();
-    const fullPrompt = `## Protocol: ${protocol.name}
-## Phase: ${phaseId}
-## Project ID: ${state.id}
-
-## Current Status
-\`\`\`yaml
-state: "${state.current_state}"
-iteration: ${state.iteration}
-started_at: "${state.started_at}"
-\`\`\`
-
-## Task
-Execute the ${phaseId} phase for project ${state.id} - ${state.title}
-
-## Phase Instructions
-${promptContent}
-
-## Important
-- Project ID: ${state.id}
-- Protocol: ${protocol.name}
-- Follow the instructions above precisely
-- Output <signal>...</signal> tags when you reach completion points
-`;
-    return new Promise((resolve, reject) => {
-        const args = ['--print', '-p', fullPrompt, '--dangerously-skip-permissions'];
-        const proc = spawn('claude', args, {
-            stdio: ['ignore', 'pipe', 'pipe'],
-            timeout,
-        });
-        let output = '';
-        let stderr = '';
-        let cancelled = false;
-        const outputBuffer = new OutputBuffer(20);
-        // Set up escape key handler
-        const wasRawMode = process.stdin.isRaw;
-        if (process.stdin.isTTY) {
-            process.stdin.setRawMode(true);
-            process.stdin.resume();
-        }
-        const escapeHandler = (key) => {
-            // ESC key is ASCII 27, Ctrl+C is 3
-            if (key[0] === 27 || key[0] === 3) {
-                cancelled = true;
-                console.log('');
-                console.log(chalk.yellow('━'.repeat(50)));
-                console.log(chalk.yellow('[porch] Cancelling Claude execution...'));
-                console.log(chalk.yellow('━'.repeat(50)));
-                // Show last 20 lines of context
-                const context = outputBuffer.getContext();
-                if (context.length > 0) {
-                    console.log(chalk.gray('\nLast output:'));
-                    console.log(chalk.gray('─'.repeat(40)));
-                    for (const line of context.slice(-10)) {
-                        console.log(chalk.gray('  ' + line.slice(0, 70)));
-                    }
-                    console.log(chalk.gray('─'.repeat(40)));
-                }
-                proc.kill('SIGTERM');
-                // Give it a moment then force kill
-                setTimeout(() => {
-                    if (!proc.killed) {
-                        proc.kill('SIGKILL');
-                    }
-                }, 2000);
-            }
-        };
-        if (process.stdin.isTTY) {
-            process.stdin.on('data', escapeHandler);
-        }
-        // Periodic status update timer
-        let lineCount = 0;
-        const statusInterval = setInterval(() => {
-            const elapsed = Math.floor((Date.now() - startTime) / 1000);
-            const mins = Math.floor(elapsed / 60);
-            const secs = elapsed % 60;
-            const timeStr = mins > 0 ? `${mins}m ${secs}s` : `${secs}s`;
-            // Use stderr so it doesn't interfere with output parsing
-            process.stderr.write(chalk.gray(`\n[porch] Running... ${lineCount} lines | ${timeStr} elapsed | ESC to cancel\n`));
-        }, 30000); // Every 30 seconds
-        const cleanup = () => {
-            clearInterval(statusInterval);
-            if (process.stdin.isTTY) {
-                process.stdin.removeListener('data', escapeHandler);
-                process.stdin.setRawMode(wasRawMode ?? false);
-                process.stdin.pause();
+    console.log(header(`PROJECT: ${state.id} - ${state.title}`));
+    console.log(`  PROTOCOL: ${state.protocol}`);
+    console.log(`  PHASE: ${state.phase} (${phaseConfig?.name || 'unknown'})`);
+    // For phased protocols, show plan phase status
+    if (isPhased(protocol, state.phase) && state.plan_phases.length > 0) {
+        console.log('');
+        console.log(chalk.bold('PLAN PHASES:'));
+        console.log('');
+        // Status icons
+        const icon = (status) => {
+            switch (status) {
+                case 'complete': return chalk.green('✓');
+                case 'in_progress': return chalk.yellow('►');
+                default: return chalk.gray('○');
             }
         };
-        proc.stdout.on('data', (data) => {
-            const text = data.toString();
-            output += text;
-            outputBuffer.append(text);
-            // Count newlines for status
-            lineCount += (text.match(/\n/g) || []).length;
-            process.stdout.write(data);
-        });
-        proc.stderr.on('data', (data) => {
-            stderr += data.toString();
-            process.stderr.write(data);
-        });
-        proc.on('close', (code) => {
-            cleanup();
-            // Show completion summary
-            const elapsed = Math.floor((Date.now() - startTime) / 1000);
-            const mins = Math.floor(elapsed / 60);
-            const secs = elapsed % 60;
-            const timeStr = mins > 0 ? `${mins}m ${secs}s` : `${secs}s`;
-            if (cancelled) {
-                console.log(chalk.yellow(`[porch] Cancelled after ${timeStr}`));
-                resolve(''); // Return empty on cancel, don't reject
+        // Show phases
+        for (const phase of state.plan_phases) {
+            const isCurrent = phase.status === 'in_progress';
+            const prefix = isCurrent ? chalk.cyan('→ ') : '  ';
+            const title = isCurrent ? chalk.bold(phase.title) : phase.title;
+            console.log(`${prefix}${icon(phase.status)} ${phase.id}: ${title}`);
+        }
+        const currentPlanPhase = getCurrentPlanPhase(state.plan_phases);
+        if (currentPlanPhase) {
+            console.log('');
+            console.log(chalk.bold(`CURRENT: ${currentPlanPhase.id} - ${currentPlanPhase.title}`));
+            // Show phase content from plan
+            const planPath = findPlanFile(projectRoot, state.id, state.title);
+            if (planPath) {
+                const content = fs.readFileSync(planPath, 'utf-8');
+                const phaseContent = getPhaseContent(content, currentPlanPhase.id);
+                if (phaseContent) {
+                    console.log(section('FROM THE PLAN', phaseContent.slice(0, 500)));
+                }
             }
-            else if (code !== 0) {
-                console.log(chalk.red(`[porch] Failed after ${timeStr} (exit code ${code})`));
-                reject(new Error(`Claude exited with code ${code}\n${stderr}`));
+            // Find the next phase name for the warning
+            const currentIdx = state.plan_phases.findIndex(p => p.id === currentPlanPhase.id);
+            const nextPlanPhase = state.plan_phases[currentIdx + 1];
+            console.log('');
+            console.log(chalk.red.bold('╔══════════════════════════════════════════════════════════════╗'));
+            console.log(chalk.red.bold('║  🛑 CRITICAL RULES                                           ║'));
+            if (nextPlanPhase) {
+                console.log(chalk.red.bold(`║  1. DO NOT start ${nextPlanPhase.id} until you run porch again!`.padEnd(63) + '║'));
             }
             else {
-                console.log('');
-                console.log(chalk.green(`[porch] Phase complete: ${lineCount} lines in ${timeStr}`));
-                resolve(output);
+                console.log(chalk.red.bold('║  1. DO NOT start the next phase until you run porch again!   ║'));
             }
-        });
-        proc.on('error', (err) => {
-            cleanup();
-            reject(err);
-        });
-    });
-}
-// ============================================================================
-// Commands
-// ============================================================================
-/**
- * Initialize a new project with a protocol
- */
-export async function init(protocolName, projectId, projectName, options = {}) {
-    const projectRoot = findProjectRoot();
-    const protocol = loadProtocol(protocolName, projectRoot);
-    // Create project directory
-    const projectDir = getProjectDir(projectRoot, projectId, projectName);
-    fs.mkdirSync(projectDir, { recursive: true });
-    // Create initial state
-    const state = createInitialState(protocol, projectId, projectName, options.worktree);
-    const statusPath = path.join(projectDir, 'status.yaml');
-    await writeState(statusPath, state);
-    console.log(chalk.green(`[porch] Initialized project ${projectId} with protocol ${protocolName}`));
-    console.log(chalk.blue(`[porch] Project directory: ${projectDir}`));
-    console.log(chalk.blue(`[porch] Initial state: ${state.current_state}`));
-}
-/**
- * Check if a protocol phase is a "phased" phase (runs per plan-phase)
- */
-function isPhasedPhase(protocol, phaseId) {
-    const phase = protocol.phases.find(p => p.id === phaseId);
-    return phase?.phased === true;
-}
-/**
- * Get the IDE phases (implement, defend, evaluate) that run per plan-phase
- */
-function getIDEPhases(protocol) {
-    return protocol.phases
-        .filter(p => p.phased === true)
-        .map(p => p.id);
-}
-/**
- * Parse the current plan-phase from state like "implement:phase_1"
- */
-function parsePlanPhaseFromState(state) {
-    const parts = state.split(':');
-    const phaseId = parts[0];
-    // Check if second part is a plan phase (phase_N) or a substate
-    if (parts.length > 1) {
-        if (parts[1].startsWith('phase_')) {
-            return { phaseId, planPhaseId: parts[1], substate: parts[2] || null };
+            console.log(chalk.red.bold('║  2. Run /compact before starting each new phase              ║'));
+            console.log(chalk.red.bold('║  3. After completing this phase, run: porch done ' + state.id.padEnd(12) + '║'));
+            console.log(chalk.red.bold('╚══════════════════════════════════════════════════════════════╝'));
         }
-        return { phaseId, planPhaseId: null, substate: parts[1] };
     }
-    return { phaseId, planPhaseId: null, substate: null };
-}
-/**
- * Get the next IDE state for a phased phase
- * implement:phase_1 → defend:phase_1 → evaluate:phase_1 → implement:phase_2 → ...
- */
-function getNextIDEState(protocol, currentState, planPhases, signal) {
-    const { phaseId, planPhaseId } = parsePlanPhaseFromState(currentState);
-    if (!planPhaseId)
-        return null;
-    const idePhases = getIDEPhases(protocol);
-    const currentIdeIndex = idePhases.indexOf(phaseId);
-    if (currentIdeIndex < 0)
-        return null;
-    // If not at the end of IDE phases, move to next IDE phase for same plan-phase
-    if (currentIdeIndex < idePhases.length - 1) {
-        return `${idePhases[currentIdeIndex + 1]}:${planPhaseId}`;
+    // Show checks status
+    const checks = getPhaseChecks(protocol, state.phase);
+    if (Object.keys(checks).length > 0) {
+        const checkLines = Object.keys(checks).map(name => `  ○ ${name} (not yet run)`);
+        console.log(section('CRITERIA', checkLines.join('\n')));
     }
-    // At end of IDE phases (evaluate), move to next plan-phase
-    const currentPlanIndex = planPhases.findIndex(p => p.id === planPhaseId);
-    if (currentPlanIndex < 0)
-        return null;
-    // Check if there's a next plan phase
-    if (currentPlanIndex < planPhases.length - 1) {
-        const nextPlanPhase = planPhases[currentPlanIndex + 1];
-        return `${idePhases[0]}:${nextPlanPhase.id}`; // Start implement for next phase
+    // Instructions
+    const gate = getPhaseGate(protocol, state.phase);
+    if (gate && state.gates[gate]?.status === 'pending' && state.gates[gate]?.requested_at) {
+        console.log(section('STATUS', chalk.yellow('WAITING FOR HUMAN APPROVAL')));
+        console.log(`\n  Gate: ${gate}`);
+        console.log('  Do not proceed until gate is approved.');
+        console.log(`\n  To approve: porch approve ${state.id} ${gate}`);
     }
-    // All plan phases complete, move to review
-    const reviewPhase = protocol.phases.find(p => p.id === 'review');
-    if (reviewPhase) {
-        return 'review';
+    else {
+        console.log(section('INSTRUCTIONS', getInstructions(state, protocol)));
     }
-    return 'complete';
-}
-// ============================================================================
-// Interactive REPL
-// ============================================================================
-/**
- * Create a readline interface for interactive input
- */
-function createRepl() {
-    return readline.createInterface({
-        input: process.stdin,
-        output: process.stdout,
-        terminal: true,
-    });
-}
-/**
- * Prompt user for input with a given message
- */
-async function prompt(rl, message) {
-    return new Promise((resolve) => {
-        rl.question(message, (answer) => {
-            resolve(answer.trim().toLowerCase());
-        });
-    });
-}
-/**
- * Show REPL help
- */
-function showReplHelp() {
-    console.log('');
-    console.log(chalk.blue('Porch REPL Commands:'));
-    console.log('  ' + chalk.green('approve') + ' [gate]  - Approve pending gate (or current if omitted)');
-    console.log('  ' + chalk.green('view') + '            - View the full artifact for current gate');
-    console.log('  ' + chalk.green('status') + '          - Show current project status');
-    console.log('  ' + chalk.green('continue') + '        - Continue to next iteration');
-    console.log('  ' + chalk.green('skip') + '            - Skip current phase (use with caution)');
-    console.log('  ' + chalk.green('help') + '            - Show this help');
-    console.log('  ' + chalk.green('quit') + '            - Exit porch');
+    console.log(section('NEXT ACTION', getNextAction(state, protocol)));
     console.log('');
 }
 /**
- * Display current status summary
+ * porch check <id>
+ * Runs the phase checks and reports results.
  */
-function displayStatus(state, protocol) {
-    console.log('');
-    console.log(chalk.blue('─'.repeat(50)));
-    console.log(chalk.blue(`Project: ${state.id} - ${state.title}`));
-    console.log(chalk.blue(`Protocol: ${state.protocol}`));
-    console.log(chalk.blue(`State: ${state.current_state}`));
-    console.log(chalk.blue(`Iteration: ${state.iteration}`));
-    // Show pending gates
-    const pendingGates = Object.entries(state.gates)
-        .filter(([, g]) => g.status === 'pending' && g.requested_at)
-        .map(([id]) => id);
-    if (pendingGates.length > 0) {
-        console.log(chalk.yellow(`Pending gates: ${pendingGates.join(', ')}`));
+export async function check(projectRoot, projectId) {
+    const statusPath = findStatusPath(projectRoot, projectId);
+    if (!statusPath) {
+        throw new Error(`Project ${projectId} not found.`);
+    }
+    const state = readState(statusPath);
+    const protocol = loadProtocol(projectRoot, state.protocol);
+    const checks = getPhaseChecks(protocol, state.phase);
+    if (Object.keys(checks).length === 0) {
+        console.log(chalk.dim('No checks defined for this phase.'));
+        return;
     }
-    console.log(chalk.blue('─'.repeat(50)));
+    const checkEnv = { PROJECT_ID: state.id, PROJECT_TITLE: state.title };
     console.log('');
-}
-/**
- * Get the artifact file path for a gate
- */
-function getGateArtifact(gateId, state, projectRoot) {
-    const projectDir = `${state.id}-${state.title}`;
-    if (gateId === 'specify_approval') {
-        // Look for spec file
-        const specPath = path.join(projectRoot, 'codev', 'specs', `${projectDir}.md`);
-        if (fs.existsSync(specPath))
-            return specPath;
-        // Try alternate naming
-        const altPath = path.join(projectRoot, 'codev', 'specs', `${state.id}-${state.title}.md`);
-        if (fs.existsSync(altPath))
-            return altPath;
-    }
-    else if (gateId === 'plan_approval') {
-        // Look for plan file
-        const planPath = path.join(projectRoot, 'codev', 'plans', `${projectDir}.md`);
-        if (fs.existsSync(planPath))
-            return planPath;
-        const altPath = path.join(projectRoot, 'codev', 'plans', `${state.id}-${state.title}.md`);
-        if (fs.existsSync(altPath))
-            return altPath;
-    }
-    else if (gateId === 'review_approval') {
-        // Look for review file
-        const reviewPath = path.join(projectRoot, 'codev', 'reviews', `${projectDir}.md`);
-        if (fs.existsSync(reviewPath))
-            return reviewPath;
-    }
-    return null;
-}
-/**
- * Display gate context - show what artifact was created and key information
- */
-function displayGateContext(gateId, state, projectRoot) {
-    const artifactPath = getGateArtifact(gateId, state, projectRoot);
+    console.log(chalk.bold('RUNNING CHECKS...'));
     console.log('');
-    console.log(chalk.cyan('─'.repeat(50)));
-    console.log(chalk.cyan('  CONTEXT FOR REVIEW'));
-    console.log(chalk.cyan('─'.repeat(50)));
-    if (artifactPath) {
-        console.log(chalk.white(`  Artifact: ${artifactPath}`));
-        console.log('');
-        // Read and show first ~30 lines of the artifact
-        try {
-            const content = fs.readFileSync(artifactPath, 'utf-8');
-            const lines = content.split('\n');
-            const preview = lines.slice(0, 30).join('\n');
-            console.log(chalk.gray('  ┌' + '─'.repeat(46) + '┐'));
-            for (const line of preview.split('\n')) {
-                console.log(chalk.gray('  │ ') + line.slice(0, 44));
-            }
-            if (lines.length > 30) {
-                console.log(chalk.gray(`  │ ... (${lines.length - 30} more lines)`));
-            }
-            console.log(chalk.gray('  └' + '─'.repeat(46) + '┘'));
-            console.log('');
-            console.log(chalk.white(`  To view full document: cat ${artifactPath}`));
-        }
-        catch (e) {
-            console.log(chalk.yellow(`  Could not read artifact: ${e}`));
-        }
+    const results = await runPhaseChecks(checks, projectRoot, checkEnv);
+    console.log(formatCheckResults(results));
+    console.log('');
+    if (allChecksPassed(results)) {
+        console.log(chalk.green('RESULT: ALL CHECKS PASSED'));
+        console.log(`\n  Run: porch done ${state.id} (to advance)`);
     }
     else {
-        console.log(chalk.yellow('  No artifact file found for this gate.'));
-        console.log(chalk.yellow('  This may indicate Claude did not produce the expected output.'));
-        // Show what we expected
-        if (gateId === 'specify_approval') {
-            console.log(chalk.gray(`  Expected: codev/specs/${state.id}-${state.title}.md`));
-        }
-        else if (gateId === 'plan_approval') {
-            console.log(chalk.gray(`  Expected: codev/plans/${state.id}-${state.title}.md`));
-        }
+        console.log(chalk.red('RESULT: CHECKS FAILED'));
+        console.log(`\n  Fix the failures and run: porch check ${state.id}`);
     }
     console.log('');
 }
 /**
- * Run the protocol loop for a project (Interactive REPL mode)
+ * porch done <id>
+ * Advances to next phase if checks pass. Refuses if checks fail.
  */
-export async function run(projectId, options = {}) {
-    const projectRoot = findProjectRoot();
-    // Find status file
-    const statusFilePath = findStatusFile(projectRoot, projectId);
-    if (!statusFilePath) {
-        throw new Error(`Status file not found for project: ${projectId}\n` +
-            `Run: porch init <protocol> ${projectId} <project-name>`);
-    }
-    // Read state and load protocol
-    const state = readState(statusFilePath);
-    if (!state) {
-        throw new Error(`Could not read state from: ${statusFilePath}`);
-    }
-    const protocol = loadProtocol(state.protocol, projectRoot);
-    const maxIterations = protocol.config?.max_iterations || 100;
-    // Create notifier for this project (desktop notifications for important events)
-    const notifier = createNotifier(projectId, { desktop: true });
-    // Create REPL interface
-    const rl = createRepl();
-    console.log('');
-    console.log(chalk.green('═'.repeat(50)));
-    console.log(chalk.green(`  PORCH - Protocol Orchestrator`));
-    console.log(chalk.green('═'.repeat(50)));
-    console.log(chalk.blue(`  Project: ${state.id} - ${state.title}`));
-    console.log(chalk.blue(`  Protocol: ${state.protocol}`));
-    console.log(chalk.blue(`  State: ${state.current_state}`));
-    console.log(chalk.green('═'.repeat(50)));
-    console.log('');
-    console.log(chalk.gray('Type "help" for commands, "quit" to exit'));
-    console.log('');
-    let currentState = state;
-    // Extract plan phases if not already done and we're past planning
-    if (!currentState.plan_phases || currentState.plan_phases.length === 0) {
-        const planFile = findPlanFile(projectRoot, projectId, currentState.title);
-        if (planFile) {
-            try {
-                const planPhases = extractPhasesFromPlanFile(planFile);
-                currentState = setPlanPhases(currentState, planPhases);
-                await writeState(statusFilePath, currentState);
-                console.log(chalk.blue(`[porch] Extracted ${planPhases.length} phases from plan`));
-                for (const phase of planPhases) {
-                    console.log(chalk.blue(`  - ${phase.id}: ${phase.title}`));
-                }
-            }
-            catch (e) {
-                console.log(chalk.yellow(`[porch] Could not extract plan phases: ${e}`));
-            }
+export async function done(projectRoot, projectId) {
+    const statusPath = findStatusPath(projectRoot, projectId);
+    if (!statusPath) {
+        throw new Error(`Project ${projectId} not found.`);
+    }
+    let state = readState(statusPath);
+    const protocol = loadProtocol(projectRoot, state.protocol);
+    const checks = getPhaseChecks(protocol, state.phase);
+    // Run checks first
+    if (Object.keys(checks).length > 0) {
+        const checkEnv = { PROJECT_ID: state.id, PROJECT_TITLE: state.title };
+        console.log('');
+        console.log(chalk.bold('RUNNING CHECKS...'));
+        const results = await runPhaseChecks(checks, projectRoot, checkEnv);
+        console.log(formatCheckResults(results));
+        if (!allChecksPassed(results)) {
+            console.log('');
+            console.log(chalk.red('CHECKS FAILED. Cannot advance.'));
+            console.log(`\n  Fix the failures and try again.`);
+            process.exit(1);
         }
     }
-    for (let iteration = currentState.iteration; iteration <= maxIterations; iteration++) {
-        console.log(chalk.blue('━'.repeat(40)));
-        console.log(chalk.blue(`[porch] Iteration ${iteration}`));
-        console.log(chalk.blue('━'.repeat(40)));
-        // Fresh read of state each iteration (Ralph pattern)
-        currentState = readState(statusFilePath) || currentState;
-        console.log(chalk.blue(`[porch] Current state: ${currentState.current_state}`));
-        // Parse state into phase and substate
-        const { phaseId, planPhaseId, substate } = parsePlanPhaseFromState(currentState.current_state);
-        // Re-attempt plan phase extraction if entering a phased phase without plan phases
-        // This handles the case where porch started during Specify phase (no plan file yet)
-        if (isPhasedPhase(protocol, phaseId) && (!currentState.plan_phases || currentState.plan_phases.length === 0)) {
-            const planFile = findPlanFile(projectRoot, projectId, currentState.title);
-            if (planFile) {
-                try {
-                    const planPhases = extractPhasesFromPlanFile(planFile);
-                    currentState = setPlanPhases(currentState, planPhases);
-                    await writeState(statusFilePath, currentState);
-                    console.log(chalk.blue(`[porch] Late discovery: Extracted ${planPhases.length} phases from plan`));
-                    for (const phase of planPhases) {
-                        console.log(chalk.blue(`  - ${phase.id}: ${phase.title}`));
-                    }
-                }
-                catch (e) {
-                    console.log(chalk.yellow(`[porch] Could not extract plan phases: ${e}`));
-                }
-            }
-            else {
-                console.log(chalk.yellow(`[porch] Warning: Entering phased phase '${phaseId}' but no plan file found`));
-            }
-        }
-        // Check if terminal phase
-        if (isTerminalPhase(protocol, phaseId)) {
-            console.log(chalk.green('━'.repeat(40)));
-            console.log(chalk.green(`[porch] ${state.protocol} loop COMPLETE`));
-            console.log(chalk.green(`[porch] Project ${projectId} finished all phases`));
-            console.log(chalk.green('━'.repeat(40)));
-            rl.close();
-            return;
-        }
-        // Check if there's a pending gate from a previous iteration (step already executed)
-        const pendingGateInfo = getGateForState(protocol, currentState.current_state);
-        if (pendingGateInfo) {
-            const { gateId } = pendingGateInfo;
-            // Check if gate is already approved
-            if (currentState.gates[gateId]?.status === 'passed') {
-                // Gate approved - proceed to next state
-                const nextState = getGateNextState(protocol, phaseId);
-                if (nextState) {
-                    console.log(chalk.green(`[porch] Gate ${gateId} passed! Proceeding to ${nextState}`));
-                    await notifier.gateApproved(gateId);
-                    // Reset any consultation attempts for the gated state
-                    currentState = resetConsultationAttempts(currentState, currentState.current_state);
-                    // If entering a phased phase, start with first plan phase
-                    if (isPhasedPhase(protocol, nextState.split(':')[0]) && currentState.plan_phases?.length) {
-                        const firstPlanPhase = currentState.plan_phases[0];
-                        currentState = updateState(currentState, `${nextState.split(':')[0]}:${firstPlanPhase.id}`);
-                        currentState = updatePhaseStatus(currentState, firstPlanPhase.id, 'in_progress');
-                    }
-                    else {
-                        currentState = updateState(currentState, nextState);
-                    }
-                    await writeState(statusFilePath, currentState);
-                    continue; // Start next iteration with new state
-                }
-            }
-            else if (currentState.gates[gateId]?.requested_at) {
-                // Gate requested but not approved - prompt user interactively
+    // Check for gate
+    const gate = getPhaseGate(protocol, state.phase);
+    if (gate && state.gates[gate]?.status !== 'approved') {
+        console.log('');
+        console.log(chalk.yellow(`GATE REQUIRED: ${gate}`));
+        console.log(`\n  Run: porch gate ${state.id}`);
+        console.log('  Wait for human approval before advancing.');
+        return;
+    }
+    // Handle phased protocols (plan phases with checks at completion)
+    if (isPhased(protocol, state.phase) && state.plan_phases.length > 0) {
+        const currentPlanPhase = getCurrentPlanPhase(state.plan_phases);
+        if (currentPlanPhase && !allPlanPhasesComplete(state.plan_phases)) {
+            // Run phase completion checks (implement + defend + evaluate all at once)
+            const completionChecks = getPhaseCompletionChecks(protocol);
+            if (Object.keys(completionChecks).length > 0) {
+                const checkEnv = { PROJECT_ID: state.id, PROJECT_TITLE: state.title };
                 console.log('');
-                console.log(chalk.yellow('═'.repeat(50)));
-                console.log(chalk.yellow(`  GATE PENDING: ${gateId}`));
-                console.log(chalk.yellow('═'.repeat(50)));
-                console.log(chalk.cyan(`  Phase: ${phaseId}`));
-                console.log(chalk.cyan(`  State: ${currentState.current_state}`));
-                // Show context for the gate - what artifact was created
-                displayGateContext(gateId, currentState, projectRoot);
-                // Interactive gate approval loop
-                let gateHandled = false;
-                while (!gateHandled) {
-                    const answer = await prompt(rl, chalk.yellow(`Approve ${gateId}? [y/n/help]: `));
-                    switch (answer) {
-                        case 'y':
-                        case 'yes':
-                        case 'approve':
-                            currentState = approveGate(currentState, gateId);
-                            await writeState(statusFilePath, currentState);
-                            console.log(chalk.green(`✓ Gate ${gateId} approved`));
-                            gateHandled = true;
-                            break;
-                        case 'n':
-                        case 'no':
-                            console.log(chalk.yellow('Gate not approved. Staying in current state.'));
-                            console.log(chalk.gray('Type "quit" to exit or wait for changes.'));
-                            break;
-                        case 'status':
-                            displayStatus(currentState, protocol);
-                            break;
-                        case 'view':
-                        case 'cat':
-                        case 'show':
-                            // Show full artifact
-                            const artifactPath = getGateArtifact(gateId, currentState, projectRoot);
-                            if (artifactPath) {
-                                console.log(chalk.cyan(`\n─── ${artifactPath} ───\n`));
-                                try {
-                                    const content = fs.readFileSync(artifactPath, 'utf-8');
-                                    console.log(content);
-                                    console.log(chalk.cyan(`\n─── End of ${artifactPath} ───\n`));
-                                }
-                                catch (e) {
-                                    console.log(chalk.red(`Error reading file: ${e}`));
-                                }
-                            }
-                            else {
-                                console.log(chalk.yellow('No artifact file found for this gate.'));
-                            }
-                            break;
-                        case 'help':
-                        case '?':
-                            showReplHelp();
-                            break;
-                        case 'quit':
-                        case 'exit':
-                            console.log(chalk.blue('Exiting porch...'));
-                            rl.close();
-                            return;
-                        default:
-                            console.log(chalk.gray('Type "y" to approve, "n" to decline, or "help" for commands'));
-                    }
+                console.log(chalk.bold(`RUNNING PHASE COMPLETION CHECKS (${currentPlanPhase.id})...`));
+                const results = await runPhaseChecks(completionChecks, projectRoot, checkEnv);
+                console.log(formatCheckResults(results));
+                if (!allChecksPassed(results)) {
+                    console.log('');
+                    console.log(chalk.red('PHASE COMPLETION CHECKS FAILED. Cannot advance.'));
+                    console.log(`\n  Ensure your commit includes:`);
+                    console.log(`    - Implementation code`);
+                    console.log(`    - Tests`);
+                    console.log(`    - 3-way review results in commit message`);
+                    console.log(`\n  Then try again.`);
+                    process.exit(1);
                 }
-                continue;
             }
-            // If gate not yet requested, fall through to execute phase first
-        }
-        // Get the current phase definition
-        const phase = protocol.phases.find(p => p.id === phaseId);
-        // Show plan phase context if in a phased phase
-        if (planPhaseId && currentState.plan_phases) {
-            const planPhase = currentState.plan_phases.find(p => p.id === planPhaseId);
-            if (planPhase) {
-                console.log(chalk.cyan(`[phase] IDE Phase: ${phaseId} | Plan Phase: ${planPhase.title}`));
+            // Advance to next plan phase
+            const { phases: updatedPhases, moveToReview } = advancePlanPhase(state.plan_phases, currentPlanPhase.id);
+            state.plan_phases = updatedPhases;
+            console.log('');
+            console.log(chalk.green(`PLAN PHASE COMPLETE: ${currentPlanPhase.id} - ${currentPlanPhase.title}`));
+            // Check if moving to review (all plan phases done)
+            if (moveToReview) {
+                state.phase = 'review';
+                state.current_plan_phase = null;
+                writeState(statusPath, state);
+                console.log(chalk.cyan('All plan phases complete. Moving to REVIEW phase.'));
+                console.log(`\n  Run: porch status ${state.id}`);
+                return;
             }
-            else {
-                console.log(chalk.cyan(`[phase] Phase: ${phaseId}`));
-            }
-        }
-        else {
-            console.log(chalk.cyan(`[phase] Phase: ${phaseId}`));
-        }
-        // Notify phase start
-        await notifier.phaseStart(phaseId);
-        // Execute phase
-        const output = await invokeClaude(protocol, phaseId, currentState, statusFilePath, projectRoot, options);
-        const signal = extractSignal(output);
-        // Run phase checks (build/test) if defined
-        if (phase?.checks && !options.dryRun) {
-            console.log(chalk.blue(`[porch] Running checks for phase ${phaseId}...`));
-            const checkResult = await runPhaseChecks(phase, {
-                cwd: projectRoot,
-                dryRun: options.dryRun,
-            });
-            console.log(formatCheckResults(checkResult));
-            if (!checkResult.success) {
-                // Notify about check failure
-                const failedCheck = checkResult.checks.find(c => !c.success);
-                await notifier.checkFailed(phaseId, failedCheck?.name || 'build/test', failedCheck?.error || 'Check failed');
-                // If checks fail, handle based on check configuration
-                if (checkResult.returnTo) {
-                    console.log(chalk.yellow(`[porch] Checks failed, returning to ${checkResult.returnTo}`));
-                    if (planPhaseId) {
-                        currentState = updateState(currentState, `${checkResult.returnTo}:${planPhaseId}`);
-                    }
-                    else {
-                        currentState = updateState(currentState, checkResult.returnTo);
-                    }
-                    await writeState(statusFilePath, currentState);
-                    continue;
-                }
-                // No returnTo means we should retry (already handled in checks)
-            }
-        }
-        // Run consultation if configured for this phase/substate
-        if (phase?.consultation && hasConsultation(phase) && !options.dryRun && !options.noClaude) {
-            const consultConfig = phase.consultation;
-            const currentSubstate = substate || parsePlanPhaseFromState(currentState.current_state).substate;
-            // Check if consultation is triggered by current substate
-            if (consultConfig.on === currentSubstate || consultConfig.on === phaseId) {
-                const maxRounds = consultConfig.max_rounds || 3;
-                const stateKey = currentState.current_state;
-                // Get attempt count from state (persisted across porch iterations)
-                const attemptCount = getConsultationAttempts(currentState, stateKey) + 1;
-                console.log(chalk.blue(`[porch] Consultation triggered for phase ${phaseId} (attempt ${attemptCount}/${maxRounds})`));
-                await notifier.consultationStart(phaseId, consultConfig.models || ['gemini', 'codex', 'claude']);
-                const consultResult = await runConsultationLoop(consultConfig, {
-                    subcommand: consultConfig.type.includes('pr') ? 'pr' : consultConfig.type.includes('spec') ? 'spec' : 'plan',
-                    identifier: projectId,
-                    cwd: projectRoot,
-                    timeout: protocol.config?.consultation_timeout,
-                    dryRun: options.dryRun,
-                });
-                console.log(formatConsultationResults(consultResult));
-                await notifier.consultationComplete(phaseId, consultResult.feedback, consultResult.allApproved);
-                // If not all approved, track attempt and check for escalation
-                if (!consultResult.allApproved) {
-                    // Increment attempt count in state (persists across iterations)
-                    currentState = incrementConsultationAttempts(currentState, stateKey);
-                    await writeState(statusFilePath, currentState);
-                    // Check if we've reached max attempts
-                    if (attemptCount >= maxRounds) {
-                        // Create escalation gate - requires human intervention
-                        const escalationGateId = `${phaseId}_consultation_escalation`;
-                        // Check if escalation gate was already approved (human override)
-                        if (currentState.gates[escalationGateId]?.status === 'passed') {
-                            console.log(chalk.green(`[porch] Consultation escalation gate already approved, continuing`));
-                            // Reset attempts and fall through to next state handling
-                            currentState = resetConsultationAttempts(currentState, stateKey);
-                            await writeState(statusFilePath, currentState);
-                        }
-                        else {
-                            console.log(chalk.red(`[porch] Consultation failed after ${attemptCount} attempts - escalating to human`));
-                            console.log(chalk.yellow(`[porch] To override and continue: porch approve ${projectId} ${escalationGateId}`));
-                            // Request human gate if not already requested
-                            if (!currentState.gates[escalationGateId]?.requested_at) {
-                                currentState = requestGateApproval(currentState, escalationGateId);
-                                await writeState(statusFilePath, currentState);
-                                await notifier.gatePending(phaseId, escalationGateId);
-                            }
-                            // Prompt user to approve escalation gate
-                            console.log('');
-                            console.log(chalk.red('═'.repeat(50)));
-                            console.log(chalk.red(`  ESCALATION: ${escalationGateId}`));
-                            console.log(chalk.red('═'.repeat(50)));
-                            console.log(chalk.yellow(`  Consultation failed after ${attemptCount} attempts`));
-                            console.log('');
-                            let escalationHandled = false;
-                            while (!escalationHandled) {
-                                const answer = await prompt(rl, chalk.red(`Override and continue? [y/n/help]: `));
-                                switch (answer) {
-                                    case 'y':
-                                    case 'yes':
-                                    case 'approve':
-                                    case 'override':
-                                        currentState = approveGate(currentState, escalationGateId);
-                                        currentState = resetConsultationAttempts(currentState, stateKey);
-                                        await writeState(statusFilePath, currentState);
-                                        console.log(chalk.green(`✓ Escalation gate ${escalationGateId} approved`));
-                                        escalationHandled = true;
-                                        break;
-                                    case 'n':
-                                    case 'no':
-                                        console.log(chalk.yellow('Escalation not approved. Consultation loop will continue.'));
-                                        break;
-                                    case 'status':
-                                        displayStatus(currentState, protocol);
-                                        break;
-                                    case 'help':
-                                    case '?':
-                                        showReplHelp();
-                                        break;
-                                    case 'quit':
-                                    case 'exit':
-                                        console.log(chalk.blue('Exiting porch...'));
-                                        rl.close();
-                                        return;
-                                    default:
-                                        console.log(chalk.gray('Type "y" to override and continue, "n" to decline, or "help" for commands'));
-                                }
-                            }
-                            continue;
-                        }
-                    }
-                    else {
-                        console.log(chalk.yellow(`[porch] Consultation requested changes (attempt ${attemptCount}/${maxRounds}), continuing for revision`));
-                        // Stay in same state for Claude to revise on next iteration
-                        continue;
-                    }
-                }
-                else {
-                    // All approved - reset attempt counter
-                    currentState = resetConsultationAttempts(currentState, stateKey);
-                    await writeState(statusFilePath, currentState);
-                }
-                // All approved (or escalation gate passed) - use consultation's next state if defined
-                if (consultConfig.next) {
-                    if (planPhaseId) {
-                        currentState = updateState(currentState, `${consultConfig.next}:${planPhaseId}`);
-                    }
-                    else {
-                        currentState = updateState(currentState, consultConfig.next);
-                    }
-                    await writeState(statusFilePath, currentState);
-                    continue;
-                }
+            // Update current plan phase tracker
+            const newCurrentPhase = getCurrentPlanPhase(state.plan_phases);
+            state.current_plan_phase = newCurrentPhase?.id || null;
+            writeState(statusPath, state);
+            if (newCurrentPhase) {
+                console.log(chalk.cyan(`NEXT: ${newCurrentPhase.id} - ${newCurrentPhase.title}`));
             }
+            console.log(`\n  Run: porch status ${state.id}`);
+            return;
         }
-        // Check if current state has a gate that should trigger AFTER this step
-        // This is the gate trigger point - step has executed, now block before transition
-        const gateInfo = getGateForState(protocol, currentState.current_state);
-        if (gateInfo) {
-            const { gateId } = gateInfo;
-            // Request gate approval if not already requested
-            if (!currentState.gates[gateId]?.requested_at) {
-                currentState = requestGateApproval(currentState, gateId);
-                await writeState(statusFilePath, currentState);
-                console.log(chalk.yellow(`[porch] Step complete. Gate approval requested: ${gateId}`));
-                await notifier.gatePending(phaseId, gateId);
-            }
-            // Gate not yet approved - prompt user interactively
-            if (currentState.gates[gateId]?.status !== 'passed') {
-                console.log('');
-                console.log(chalk.yellow('═'.repeat(50)));
-                console.log(chalk.yellow(`  GATE PENDING: ${gateId}`));
-                console.log(chalk.yellow('═'.repeat(50)));
-                let gateHandled = false;
-                while (!gateHandled) {
-                    const answer = await prompt(rl, chalk.yellow(`Approve ${gateId}? [y/n/help]: `));
-                    switch (answer) {
-                        case 'y':
-                        case 'yes':
-                        case 'approve':
-                            currentState = approveGate(currentState, gateId);
-                            await writeState(statusFilePath, currentState);
-                            console.log(chalk.green(`✓ Gate ${gateId} approved`));
-                            gateHandled = true;
-                            break;
-                        case 'n':
-                        case 'no':
-                            console.log(chalk.yellow('Gate not approved. Staying in current state.'));
-                            break;
-                        case 'status':
-                            displayStatus(currentState, protocol);
-                            break;
-                        case 'help':
-                        case '?':
-                            showReplHelp();
-                            break;
-                        case 'quit':
-                        case 'exit':
-                            console.log(chalk.blue('Exiting porch...'));
-                            rl.close();
-                            return;
-                        default:
-                            console.log(chalk.gray('Type "y" to approve, "n" to decline, or "help" for commands'));
-                    }
-                }
-                continue;
+    }
+    // Advance to next protocol phase
+    advanceProtocolPhase(state, protocol, statusPath);
+}
+function advanceProtocolPhase(state, protocol, statusPath) {
+    const nextPhase = getNextPhase(protocol, state.phase);
+    if (!nextPhase) {
+        console.log('');
+        console.log(chalk.green.bold('🎉 PROTOCOL COMPLETE'));
+        console.log(`\n  Project ${state.id} has completed the ${state.protocol} protocol.`);
+        return;
+    }
+    state.phase = nextPhase.id;
+    // If entering a phased phase (implement), extract plan phases
+    if (isPhased(protocol, nextPhase.id)) {
+        const planPath = findPlanFile(process.cwd(), state.id, state.title);
+        if (planPath) {
+            state.plan_phases = extractPhasesFromFile(planPath);
+            // extractPhasesFromFile already marks first phase as in_progress
+            if (state.plan_phases.length > 0) {
+                state.current_plan_phase = state.plan_phases[0].id;
             }
         }
-        // Determine next state
-        let nextState = null;
-        if (signal) {
-            console.log(chalk.green(`[porch] Signal received: ${signal}`));
-            nextState = getSignalNextState(protocol, phaseId, signal);
-        }
-        if (!nextState) {
-            // Check if this is a phased phase (IDE loop)
-            if (isPhasedPhase(protocol, phaseId) && currentState.plan_phases?.length) {
-                nextState = getNextIDEState(protocol, currentState.current_state, currentState.plan_phases, signal || undefined);
-                // Mark current plan phase as complete if moving to next
-                if (nextState && planPhaseId) {
-                    const { planPhaseId: nextPlanPhaseId } = parsePlanPhaseFromState(nextState);
-                    if (nextPlanPhaseId !== planPhaseId) {
-                        currentState = updatePhaseStatus(currentState, planPhaseId, 'complete');
-                        if (nextPlanPhaseId) {
-                            currentState = updatePhaseStatus(currentState, nextPlanPhaseId, 'in_progress');
-                        }
-                    }
-                }
-            }
-            else {
-                // Use default transition
-                nextState = getDefaultNextState(protocol, currentState.current_state);
+    }
+    writeState(statusPath, state);
+    console.log('');
+    console.log(chalk.green(`ADVANCING TO: ${nextPhase.id} - ${nextPhase.name}`));
+    // If we just entered implement phase, show phase 1 info and the critical warning
+    if (isPhased(protocol, nextPhase.id) && state.plan_phases.length > 0) {
+        const firstPhase = state.plan_phases[0];
+        const nextPlanPhase = state.plan_phases[1];
+        console.log('');
+        console.log(chalk.bold(`YOUR TASK: ${firstPhase.id} - "${firstPhase.title}"`));
+        // Show phase content from plan
+        const planPath = findPlanFile(process.cwd(), state.id, state.title);
+        if (planPath) {
+            const content = fs.readFileSync(planPath, 'utf-8');
+            const phaseContent = getPhaseContent(content, firstPhase.id);
+            if (phaseContent) {
+                console.log(section('FROM THE PLAN', phaseContent.slice(0, 800)));
             }
         }
-        if (nextState) {
-            currentState = updateState(currentState, nextState, signal ? { signal } : undefined);
-            await writeState(statusFilePath, currentState);
+        console.log('');
+        console.log(chalk.red.bold('╔══════════════════════════════════════════════════════════════╗'));
+        console.log(chalk.red.bold('║  🛑 CRITICAL RULES                                           ║'));
+        if (nextPlanPhase) {
+            console.log(chalk.red.bold(`║  1. DO NOT start ${nextPlanPhase.id} until you run porch again!`.padEnd(63) + '║'));
         }
         else {
-            console.log(chalk.yellow(`[porch] No transition defined, staying in current state`));
+            console.log(chalk.red.bold('║  1. DO NOT start the next phase until you run porch again!   ║'));
         }
+        console.log(chalk.red.bold('║  2. Run /compact before starting each new phase              ║'));
+        console.log(chalk.red.bold('║  3. When phase complete, run: porch done ' + state.id.padEnd(20) + '║'));
+        console.log(chalk.red.bold('╚══════════════════════════════════════════════════════════════╝'));
     }
-    rl.close();
-    throw new Error(`Max iterations (${maxIterations}) reached!`);
+    console.log(`\n  Run: porch status ${state.id}`);
 }
 /**
- * Approve a gate
+ * porch gate <id>
+ * Requests human approval for current gate.
  */
-export async function approve(projectId, gateId) {
-    const projectRoot = findProjectRoot();
-    const statusFilePath = findStatusFile(projectRoot, projectId);
-    if (!statusFilePath) {
-        throw new Error(`Status file not found for project: ${projectId}`);
+export async function gate(projectRoot, projectId) {
+    const statusPath = findStatusPath(projectRoot, projectId);
+    if (!statusPath) {
+        throw new Error(`Project ${projectId} not found.`);
+    }
+    const state = readState(statusPath);
+    const protocol = loadProtocol(projectRoot, state.protocol);
+    const gateName = getPhaseGate(protocol, state.phase);
+    if (!gateName) {
+        console.log(chalk.dim('No gate required for this phase.'));
+        console.log(`\n  Run: porch done ${state.id}`);
+        return;
     }
-    const state = readState(statusFilePath);
-    if (!state) {
-        throw new Error(`Could not read state from: ${statusFilePath}`);
+    // Mark gate as requested
+    if (!state.gates[gateName]) {
+        state.gates[gateName] = { status: 'pending' };
     }
-    const updatedState = approveGate(state, gateId);
-    await writeState(statusFilePath, updatedState);
-    console.log(chalk.green(`[porch] Approved: ${gateId}`));
-}
-/**
- * Show project status
- */
-export async function status(projectId) {
-    const projectRoot = findProjectRoot();
-    if (projectId) {
-        // Show specific project
-        const statusFilePath = findStatusFile(projectRoot, projectId);
-        if (!statusFilePath) {
-            throw new Error(`Status file not found for project: ${projectId}`);
-        }
-        const state = readState(statusFilePath);
-        if (!state) {
-            throw new Error(`Could not read state from: ${statusFilePath}`);
-        }
-        console.log(chalk.blue(`[porch] Status for project ${projectId}:`));
-        console.log('');
-        console.log(`  ID:        ${state.id}`);
-        console.log(`  Title:     ${state.title}`);
-        console.log(`  Protocol:  ${state.protocol}`);
-        console.log(`  State:     ${state.current_state}`);
-        console.log(`  Iteration: ${state.iteration}`);
-        console.log(`  Started:   ${state.started_at}`);
-        console.log(`  Updated:   ${state.last_updated}`);
-        console.log('');
-        if (Object.keys(state.gates).length > 0) {
-            console.log('  Gates:');
-            for (const [gateId, gateStatus] of Object.entries(state.gates)) {
-                const icon = gateStatus.status === 'passed' ? '✓' : gateStatus.status === 'failed' ? '✗' : '⏳';
-                console.log(`    ${icon} ${gateId}: ${gateStatus.status}`);
-            }
-            console.log('');
-        }
-        if (state.plan_phases && state.plan_phases.length > 0) {
-            console.log('  Plan Phases:');
-            for (const phase of state.plan_phases) {
-                const phaseStatus = state.phases[phase.id]?.status || 'pending';
-                const icon = phaseStatus === 'complete' ? '✓' : phaseStatus === 'in_progress' ? '🔄' : '○';
-                console.log(`    ${icon} ${phase.id}: ${phase.title}`);
-            }
-        }
+    if (!state.gates[gateName].requested_at) {
+        state.gates[gateName].requested_at = new Date().toISOString();
+        writeState(statusPath, state);
     }
-    else {
-        // Show all projects
-        const projects = findProjects(projectRoot);
-        const executions = findExecutions(projectRoot);
-        if (projects.length === 0 && executions.length === 0) {
-            console.log(chalk.yellow('[porch] No projects found'));
-            return;
-        }
-        console.log(chalk.blue('[porch] Projects:'));
-        for (const { id, path: statusPath } of projects) {
-            const state = readState(statusPath);
-            if (state) {
-                const pendingGates = Object.entries(state.gates)
-                    .filter(([, g]) => g.status === 'pending' && g.requested_at)
-                    .map(([id]) => id);
-                const gateStr = pendingGates.length > 0 ? chalk.yellow(` [${pendingGates.join(', ')}]`) : '';
-                console.log(`  ${id} ${state.title} - ${state.current_state}${gateStr}`);
-            }
-        }
-        if (executions.length > 0) {
+    console.log('');
+    console.log(chalk.bold(`GATE: ${gateName}`));
+    console.log('');
+    // Show relevant artifact and open it for review
+    const artifact = getArtifactForPhase(projectRoot, state);
+    if (artifact) {
+        const fullPath = path.join(projectRoot, artifact);
+        if (fs.existsSync(fullPath)) {
+            console.log(`  Artifact: ${artifact}`);
             console.log('');
-            console.log(chalk.blue('[porch] Executions:'));
-            for (const { protocol, id, path: statusPath } of executions) {
-                const state = readState(statusPath);
-                if (state) {
-                    console.log(`  ${protocol}/${id} - ${state.current_state}`);
-                }
-            }
+            console.log(chalk.cyan('  Opening artifact for human review...'));
+            // Use af open to display in annotation viewer
+            const { spawn } = await import('node:child_process');
+            spawn('af', ['open', fullPath], {
+                stdio: 'inherit',
+                detached: true
+            }).unref();
         }
     }
+    console.log('');
+    console.log(chalk.yellow('  Human approval required. STOP and wait.'));
+    console.log('  Do not proceed until gate is approved.');
+    console.log('');
+    console.log(chalk.bold('STATUS: WAITING FOR HUMAN APPROVAL'));
+    console.log('');
+    console.log(chalk.dim(`  To approve: porch approve ${state.id} ${gateName}`));
+    console.log('');
 }
 /**
- * List available protocols
+ * porch approve <id> <gate> --a-human-explicitly-approved-this
+ * Human approves a gate. Requires explicit flag to prevent automated approvals.
  */
-export async function list() {
-    const projectRoot = findProjectRoot();
-    const protocols = listProtocols(projectRoot);
-    if (protocols.length === 0) {
-        console.log(chalk.yellow('[porch] No protocols found'));
+export async function approve(projectRoot, projectId, gateName, hasHumanFlag) {
+    const statusPath = findStatusPath(projectRoot, projectId);
+    if (!statusPath) {
+        throw new Error(`Project ${projectId} not found.`);
+    }
+    const state = readState(statusPath);
+    if (!state.gates[gateName]) {
+        const knownGates = Object.keys(state.gates).join(', ');
+        throw new Error(`Unknown gate: ${gateName}\nKnown gates: ${knownGates || 'none'}`);
+    }
+    if (state.gates[gateName].status === 'approved') {
+        console.log(chalk.yellow(`Gate ${gateName} is already approved.`));
         return;
     }
-    console.log(chalk.blue('[porch] Available protocols:'));
-    for (const name of protocols) {
-        try {
-            const protocol = loadProtocol(name, projectRoot);
-            console.log(`  - ${name}: ${protocol.description}`);
-        }
-        catch {
-            console.log(`  - ${name}: (error loading)`);
+    // Require explicit human flag
+    if (!hasHumanFlag) {
+        console.log('');
+        console.log(chalk.red('ERROR: Human approval required.'));
+        console.log('');
+        console.log('  To approve, please run:');
+        console.log('');
+        console.log(chalk.cyan(`    porch approve ${projectId} ${gateName} --a-human-explicitly-approved-this`));
+        console.log('');
+        process.exit(1);
+    }
+    // Run phase checks before approving
+    const protocol = loadProtocol(projectRoot, state.protocol);
+    const checks = getPhaseChecks(protocol, state.phase);
+    if (Object.keys(checks).length > 0) {
+        const checkEnv = { PROJECT_ID: state.id, PROJECT_TITLE: state.title };
+        console.log('');
+        console.log(chalk.bold('RUNNING CHECKS...'));
+        const results = await runPhaseChecks(checks, projectRoot, checkEnv);
+        console.log(formatCheckResults(results));
+        if (!allChecksPassed(results)) {
+            console.log('');
+            console.log(chalk.red('CHECKS FAILED. Cannot approve gate.'));
+            console.log(`\n  Fix the failures and try again.`);
+            process.exit(1);
         }
     }
-}
-/**
- * Show protocol definition
- */
-export async function show(protocolName) {
-    const projectRoot = findProjectRoot();
-    const protocol = loadProtocol(protocolName, projectRoot);
-    console.log(chalk.blue(`[porch] Protocol: ${protocolName}`));
+    state.gates[gateName].status = 'approved';
+    state.gates[gateName].approved_at = new Date().toISOString();
+    writeState(statusPath, state);
+    console.log('');
+    console.log(chalk.green(`Gate ${gateName} approved.`));
+    console.log(`\n  Run: porch done ${state.id} (to advance)`);
     console.log('');
-    console.log(JSON.stringify(protocol, null, 2));
 }
 /**
- * Show pending gates across all projects
+ * porch init <protocol> <id> <name>
+ * Initialize a new project.
  */
-export async function pending() {
-    const projectRoot = findProjectRoot();
-    const gates = findPendingGates(projectRoot);
-    if (gates.length === 0) {
-        console.log(chalk.green('[porch] No pending gates'));
-        return;
-    }
-    console.log(chalk.yellow('[porch] Pending gates:'));
-    for (const gate of gates) {
-        const requestedAt = gate.requestedAt ? ` (requested ${gate.requestedAt})` : '';
-        console.log(`  ${gate.projectId}: ${gate.gateId}${requestedAt}`);
-        console.log(`    → porch approve ${gate.projectId} ${gate.gateId}`);
-    }
+export async function init(projectRoot, protocolName, projectId, projectName) {
+    const protocol = loadProtocol(projectRoot, protocolName);
+    const statusPath = getStatusPath(projectRoot, projectId, projectName);
+    // Check if already exists
+    if (fs.existsSync(statusPath)) {
+        throw new Error(`Project ${projectId}-${projectName} already exists.`);
+    }
+    const state = createInitialState(protocol, projectId, projectName);
+    writeState(statusPath, state);
+    console.log('');
+    console.log(chalk.green(`Project initialized: ${projectId}-${projectName}`));
+    console.log(`  Protocol: ${protocolName}`);
+    console.log(`  Initial phase: ${state.phase}`);
+    console.log(`\n  Run: porch status ${projectId}`);
+    console.log('');
 }
 // ============================================================================
-// Auto-Detection
+// Helpers
 // ============================================================================
-/**
- * Auto-detect project ID from current directory
- *
- * Detection methods:
- * 1. Check if cwd is a worktree matching pattern: .builders/<id> or worktrees/<protocol>_<id>_*
- * 2. Check for a single project in codev/projects/
- * 3. Check for .porch-project marker file
- */
-function autoDetectProject() {
-    const cwd = process.cwd();
-    // Method 1: Check path pattern for builder worktree
-    // Pattern: .builders/<id> or .builders/<id>-<name>
-    const buildersMatch = cwd.match(/[/\\]\.builders[/\\](\d+)(?:-[^/\\]*)?(?:[/\\]|$)/);
-    if (buildersMatch) {
-        return buildersMatch[1];
+function getInstructions(state, protocol) {
+    const phase = state.phase;
+    if (isPhased(protocol, phase) && state.plan_phases.length > 0) {
+        const current = getCurrentPlanPhase(state.plan_phases);
+        if (current) {
+            return `  You are implementing ${current.id}: "${current.title}".\n\n  Complete the work, then run: porch check ${state.id}`;
+        }
     }
-    // Pattern: worktrees/<protocol>_<id>_<name>
-    const worktreeMatch = cwd.match(/[/\\]worktrees[/\\]\w+_(\d+)_[^/\\]*(?:[/\\]|$)/);
-    if (worktreeMatch) {
-        return worktreeMatch[1];
+    const phaseConfig = getPhaseConfig(protocol, phase);
+    return `  You are in the ${phaseConfig?.name || phase} phase.\n\n  When complete, run: porch done ${state.id}`;
+}
+function getNextAction(state, protocol) {
+    const checks = getPhaseChecks(protocol, state.phase);
+    const gate = getPhaseGate(protocol, state.phase);
+    if (gate && state.gates[gate]?.status === 'pending' && state.gates[gate]?.requested_at) {
+        return chalk.yellow('Wait for human to approve the gate.');
     }
-    // Method 2: Check for .porch-project marker file
-    const markerPath = path.join(cwd, '.porch-project');
-    if (fs.existsSync(markerPath)) {
-        const content = fs.readFileSync(markerPath, 'utf-8').trim();
-        if (content) {
-            return content;
+    if (isPhased(protocol, state.phase)) {
+        const current = getCurrentPlanPhase(state.plan_phases);
+        if (current) {
+            return `Implement ${current.title} as specified in the plan.`;
         }
     }
-    // Method 3: Check if there's exactly one project in codev/projects/
-    try {
-        const projectRoot = findProjectRoot();
-        const projects = findProjects(projectRoot);
-        if (projects.length === 1) {
-            return projects[0].id;
-        }
+    if (Object.keys(checks).length > 0) {
+        return `Complete the phase work, then run: porch check ${state.id}`;
     }
-    catch {
-        // Not in a codev project
+    return `Complete the phase work, then run: porch done ${state.id}`;
+}
+function getArtifactForPhase(projectRoot, state) {
+    switch (state.phase) {
+        case 'specify':
+            return `codev/specs/${state.id}-${state.title}.md`;
+        case 'plan':
+            return `codev/plans/${state.id}-${state.title}.md`;
+        case 'review':
+            return `codev/reviews/${state.id}-${state.title}.md`;
+        default:
+            return null;
     }
-    return null;
 }
-export async function porch(options) {
-    const { subcommand, args, dryRun, noClaude, pollInterval, description, worktree } = options;
-    switch (subcommand.toLowerCase()) {
-        case 'run': {
-            let projectId = args[0];
-            // Auto-detect project if not provided
-            if (!projectId) {
-                const detected = autoDetectProject();
-                if (!detected) {
-                    throw new Error('Usage: porch run <project-id>\n' +
-                        'Or run from a project worktree to auto-detect.');
+// ============================================================================
+// CLI
+// ============================================================================
+export async function cli(args) {
+    const [command, ...rest] = args;
+    const projectRoot = process.cwd();
+    try {
+        switch (command) {
+            case 'run':
+                if (!rest[0])
+                    throw new Error('Usage: porch run <id>');
+                const { run } = await import('./run.js');
+                await run(projectRoot, rest[0]);
+                break;
+            case 'status':
+                if (!rest[0])
+                    throw new Error('Usage: porch status <id>');
+                await status(projectRoot, rest[0]);
+                break;
+            case 'check':
+                if (!rest[0])
+                    throw new Error('Usage: porch check <id>');
+                await check(projectRoot, rest[0]);
+                break;
+            case 'done':
+                if (!rest[0])
+                    throw new Error('Usage: porch done <id>');
+                await done(projectRoot, rest[0]);
+                break;
+            case 'gate':
+                if (!rest[0])
+                    throw new Error('Usage: porch gate <id>');
+                await gate(projectRoot, rest[0]);
+                break;
+            case 'approve':
+                if (!rest[0] || !rest[1])
+                    throw new Error('Usage: porch approve <id> <gate> --a-human-explicitly-approved-this');
+                const hasHumanFlag = rest.includes('--a-human-explicitly-approved-this');
+                await approve(projectRoot, rest[0], rest[1], hasHumanFlag);
+                break;
+            case 'init':
+                if (!rest[0] || !rest[1] || !rest[2]) {
+                    throw new Error('Usage: porch init <protocol> <id> <name>');
                 }
-                projectId = detected;
-                console.log(chalk.blue(`[porch] Auto-detected project: ${projectId}`));
-            }
-            await run(projectId, { dryRun, noClaude, pollInterval });
-            break;
-        }
-        case 'init': {
-            if (args.length < 3) {
-                throw new Error('Usage: porch init <protocol> <project-id> <project-name>');
-            }
-            await init(args[0], args[1], args[2], { description, worktree });
-            break;
-        }
-        case 'approve': {
-            if (args.length < 2) {
-                throw new Error('Usage: porch approve <project-id> <gate-id>');
-            }
-            await approve(args[0], args[1]);
-            break;
-        }
-        case 'status': {
-            await status(args[0]);
-            break;
-        }
-        case 'pending': {
-            await pending();
-            break;
-        }
-        case 'list':
-        case 'list-protocols': {
-            await list();
-            break;
-        }
-        case 'show':
-        case 'show-protocol': {
-            if (args.length < 1) {
-                throw new Error('Usage: porch show <protocol>');
-            }
-            await show(args[0]);
-            break;
+                await init(projectRoot, rest[0], rest[1], rest[2]);
+                break;
+            default:
+                console.log('porch - Protocol Orchestrator');
+                console.log('');
+                console.log('Commands:');
+                console.log('  run <id>                 Run the protocol (porch as outer loop)');
+                console.log('  status <id>              Show current state and instructions');
+                console.log('  check <id>               Run checks for current phase');
+                console.log('  done <id>                Advance to next phase (if checks pass)');
+                console.log('  gate <id>                Request human approval');
+                console.log('  approve <id> <gate> --a-human-explicitly-approved-this');
+                console.log('  init <protocol> <id> <name>  Initialize a new project');
+                console.log('');
+                process.exit(command ? 1 : 0);
         }
-        default:
-            throw new Error(`Unknown subcommand: ${subcommand}\n` +
-                'Valid subcommands: run, init, approve, status, pending, list, show');
+    }
+    catch (err) {
+        console.error(chalk.red(`Error: ${err.message}`));
+        process.exit(1);
     }
 }
 //# sourceMappingURL=index.js.map