wiggum-cli 0.16.0 → 0.17.1

package/dist/tui/screens/MainShell.js

@@ -7,9 +7,10 @@ import { jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
  * Wrapped in AppShell for consistent layout.
  */
 import { useState, useCallback, useEffect, useMemo } from 'react';
-import { Box, Text, useInput, useApp } from 'ink';
+import { Box, Text, useInput, useApp, useStdout } from 'ink';
 import { MessageList } from '../components/MessageList.js';
 import { ChatInput } from '../components/ChatInput.js';
+import { IssuePicker } from '../components/IssuePicker.js';
 import { ActionOutput } from '../components/ActionOutput.js';
 import { AppShell } from '../components/AppShell.js';
 import { colors, theme, phase } from '../theme.js';
@@ -18,7 +19,18 @@ import { logger } from '../../utils/logger.js';
 import { parseInput, resolveCommandAlias, formatHelpText, } from '../../repl/command-parser.js';
 import { readLoopStatus } from '../utils/loop-status.js';
 import { useSync } from '../hooks/useSync.js';
+import { isGhInstalled, detectGitHubRemote, listRepoIssues, } from '../../utils/github.js';
+import { clearScreen } from '../utils/clear-screen.js';
 import path from 'node:path';
+function slugifyIssueTitle(title, maxWords = 4) {
+    return title
+        .toLowerCase()
+        .replace(/[^a-z0-9\s-]/g, '')
+        .trim()
+        .split(/\s+/)
+        .slice(0, maxWords)
+        .join('-') || 'untitled';
+}
 /**
  * Generate a unique ID for messages
  */
@@ -33,6 +45,7 @@ function generateId() {
  */
 export function MainShell({ header, sessionState, onNavigate, backgroundRuns, initialMessage, initialFiles, }) {
     const { exit } = useApp();
+    const { stdout } = useStdout();
     const [messages, setMessages] = useState(() => {
         const initial = [];
         if (initialMessage) {
@@ -48,6 +61,12 @@ export function MainShell({ header, sessionState, onNavigate, backgroundRuns, in
     const [contextAge, setContextAge] = useState(null);
     // Sync hook
     const { status: syncStatus, error: syncError, sync } = useSync();
+    // Issue picker state
+    const [issuePickerVisible, setIssuePickerVisible] = useState(false);
+    const [issuePickerIssues, setIssuePickerIssues] = useState([]);
+    const [issuePickerLoading, setIssuePickerLoading] = useState(false);
+    const [issuePickerError, setIssuePickerError] = useState();
+    const [issuePickerRepo, setIssuePickerRepo] = useState(null);
     const addSystemMessage = useCallback((content) => {
         const message = {
             id: generateId(),
@@ -171,6 +190,92 @@ export function MainShell({ header, sessionState, onNavigate, backgroundRuns, in
         }
         addSystemMessage(`No running loop found for "${featureName}".`);
     }, [addSystemMessage, backgroundRuns, onNavigate]);
+    const handleAgent = useCallback((args) => {
+        if (!sessionState.initialized) {
+            addSystemMessage('Project not initialized. Run /init first.');
+            return;
+        }
+        // Parse optional flags
+        let dryRun = false;
+        let maxItems;
+        let reviewMode;
+        for (let i = 0; i < args.length; i++) {
+            if (args[i] === '--dry-run') {
+                dryRun = true;
+            }
+            else if (args[i] === '--max-items' && i + 1 < args.length) {
+                maxItems = parseInt(args[i + 1], 10);
+                if (Number.isNaN(maxItems)) {
+                    addSystemMessage(`Invalid --max-items value '${args[i + 1]}'. Must be a number.`);
+                    return;
+                }
+                i++;
+            }
+            else if (args[i] === '--review-mode' && i + 1 < args.length) {
+                reviewMode = args[i + 1];
+                i++;
+            }
+        }
+        if (reviewMode !== undefined && !['manual', 'auto', 'merge'].includes(reviewMode)) {
+            addSystemMessage(`Invalid --review-mode value '${reviewMode}'. Use 'manual', 'auto', or 'merge'.`);
+            return;
+        }
+        onNavigate('agent', { dryRun, maxItems, reviewMode });
+    }, [sessionState.initialized, addSystemMessage, onNavigate]);
+    const handleIssueCommand = useCallback(async (searchQuery) => {
+        if (!sessionState.initialized) {
+            addSystemMessage('Project not initialized. Run /init first.');
+            return;
+        }
+        setIssuePickerVisible(true);
+        setIssuePickerLoading(true);
+        setIssuePickerError(undefined);
+        try {
+            const ghAvailable = await isGhInstalled();
+            if (!ghAvailable) {
+                setIssuePickerError('Install GitHub CLI (gh) for issue browsing');
+                setIssuePickerLoading(false);
+                return;
+            }
+            let repo = issuePickerRepo;
+            if (!repo) {
+                repo = await detectGitHubRemote(sessionState.projectRoot);
+                if (!repo) {
+                    setIssuePickerError('No GitHub remote detected in this project');
+                    setIssuePickerLoading(false);
+                    return;
+                }
+                setIssuePickerRepo(repo);
+            }
+            const result = await listRepoIssues(repo.owner, repo.repo, searchQuery);
+            if (result.error) {
+                setIssuePickerError(result.error);
+            }
+            setIssuePickerIssues(result.issues);
+        }
+        catch (err) {
+            setIssuePickerError(err instanceof Error ? err.message : String(err));
+        }
+        finally {
+            setIssuePickerLoading(false);
+        }
+    }, [sessionState.initialized, sessionState.projectRoot, issuePickerRepo, addSystemMessage]);
+    const handleIssueSelect = useCallback((issue) => {
+        clearScreen(stdout);
+        setIssuePickerVisible(false);
+        setIssuePickerIssues([]);
+        const featureName = slugifyIssueTitle(issue.title);
+        onNavigate('interview', {
+            featureName,
+            initialReferences: [`issue:${issue.number}`],
+        });
+    }, [stdout, onNavigate]);
+    const handleIssueCancel = useCallback(() => {
+        clearScreen(stdout);
+        setIssuePickerVisible(false);
+        setIssuePickerIssues([]);
+        setIssuePickerError(undefined);
+    }, [stdout]);
     const handleConfig = useCallback((args) => {
         if (args.length === 0) {
             addSystemMessage('Config management - not yet implemented in TUI mode. Use CLI: wiggum config');
@@ -220,6 +325,12 @@ export function MainShell({ header, sessionState, onNavigate, backgroundRuns, in
             case 'monitor':
                 handleMonitor(args);
                 break;
+            case 'issue':
+                handleIssueCommand(args.join(' ') || undefined);
+                break;
+            case 'agent':
+                handleAgent(args);
+                break;
             case 'config':
                 handleConfig(args);
                 break;
@@ -229,7 +340,7 @@ export function MainShell({ header, sessionState, onNavigate, backgroundRuns, in
             default:
                 addSystemMessage(`Unknown command: ${commandName}`);
         }
-    }, [handleHelp, handleInit, handleSync, handleNew, handleRun, handleMonitor, handleConfig, handleExit, addSystemMessage]);
+    }, [handleHelp, handleInit, handleSync, handleNew, handleRun, handleMonitor, handleIssueCommand, handleAgent, handleConfig, handleExit, addSystemMessage]);
     const handleNaturalLanguage = useCallback((_text) => {
         addSystemMessage('Tip: Use /help to see available commands, or /new <feature> to create a spec.');
     }, [addSystemMessage]);
@@ -264,10 +375,10 @@ export function MainShell({ header, sessionState, onNavigate, backgroundRuns, in
     });
     // Build tips text
     const tips = sessionState.initialized
-        ? 'Tip: /new <feature> to create spec, /help for commands'
+        ? 'Tip: /new <feature> or /issue to browse issues, /help for commands'
         : 'Tip: /init to set up, /help for commands';
     const specSuggestions = useMemo(() => sessionState.specNames?.map((name) => ({ name, description: '' })), [sessionState.specNames]);
-    const inputElement = (_jsx(ChatInput, { onSubmit: handleSubmit, disabled: false, placeholder: "Enter command or type /help...", onCommand: (cmd) => handleSubmit(`/${cmd}`), specSuggestions: specSuggestions }));
+    const inputElement = (_jsxs(Box, { flexDirection: "column", children: [_jsx(ChatInput, { onSubmit: handleSubmit, disabled: issuePickerVisible, placeholder: "Enter command or type /help...", onCommand: (cmd) => handleSubmit(`/${cmd}`), specSuggestions: specSuggestions }), issuePickerVisible && (_jsx(IssuePicker, { issues: issuePickerIssues, repoSlug: issuePickerRepo ? `${issuePickerRepo.owner}/${issuePickerRepo.repo}` : '...', onSelect: handleIssueSelect, onCancel: handleIssueCancel, isLoading: issuePickerLoading, error: issuePickerError }))] }));
     return (_jsxs(AppShell, { header: header, tips: tips, isWorking: syncStatus === 'running', workingStatus: syncStatus === 'running' ? 'Syncing project context\u2026' : undefined, input: inputElement, footerStatus: {
             action: projectLabel || 'Main Shell',
             phase: sessionState.provider ? `${sessionState.provider}/${sessionState.model}` : 'No provider',

package/dist/tui/screens/RunScreen.js

@@ -25,13 +25,14 @@ import { ActivityFeed } from '../components/ActivityFeed.js';
 import { colors, theme } from '../theme.js';
 import { readLoopStatus, parseImplementationPlan, getGitBranch, formatNumber, getLoopLogPath, parseLoopLog, parsePhaseChanges, } from '../utils/loop-status.js';
 import { buildEnhancedRunSummary } from '../utils/build-run-summary.js';
-import { getCurrentCommitHash } from '../utils/git-summary.js';
+import { getCurrentCommitHash, getRecentCommits } from '../utils/git-summary.js';
 import { writeRunSummaryFile } from '../../utils/summary-file.js';
 import { loadConfigWithDefaults } from '../../utils/config.js';
 import { logger } from '../../utils/logger.js';
 import { readActionRequest, writeActionReply, cleanupActionFiles } from '../utils/action-inbox.js';
 const POLL_INTERVAL_MS = 2500;
 const ERROR_TAIL_LINES = 12;
+const RUN_REVIEW_MODES = ['manual', 'auto', 'merge'];
 function findFeatureLoopScript(projectRoot, scriptsDir) {
     const localScript = join(projectRoot, scriptsDir, 'feature-loop.sh');
     if (existsSync(localScript)) {
@@ -101,6 +102,7 @@ export function RunScreen({ header, featureName, projectRoot, sessionState, moni
         tasksPending: 0,
         e2eDone: 0,
         e2ePending: 0,
+        planExists: false,
     });
     const [branch, setBranch] = useState('-');
     const [error, setError] = useState(null);
@@ -111,6 +113,9 @@ export function RunScreen({ header, featureName, projectRoot, sessionState, moni
     const [activityEvents, setActivityEvents] = useState([]);
     const [latestCommit, setLatestCommit] = useState(null);
     const [baselineCommit, setBaselineCommit] = useState(null);
+    const [recentCommits, setRecentCommits] = useState([]);
+    const [reviewMode, setReviewMode] = useState(reviewModeProp ?? 'manual');
+    const [loopModel, setLoopModel] = useState(null);
     const childRef = useRef(null);
     const stopRequestedRef = useRef(false);
     const isMountedRef = useRef(true);
@@ -124,6 +129,8 @@ export function RunScreen({ header, featureName, projectRoot, sessionState, moni
     const handledActionIdRef = useRef(null);
     const lastLogLineCountRef = useRef(0);
     const lastKnownPhasesRef = useRef(undefined);
+    const lastActivityTimeRef = useRef(Date.now());
+    const lastCommitForEventRef = useRef(null);
     // Read baseline commit once on mount
     useEffect(() => {
         const baselinePath = `/tmp/ralph-loop-${featureName}.baseline`;
@@ -167,6 +174,14 @@ export function RunScreen({ header, featureName, projectRoot, sessionState, moni
             else {
                 onCancel();
             }
+            return;
+        }
+        // Shift+R cycles review mode (manual → auto → merge)
+        if (input === 'R') {
+            setReviewMode((prev) => {
+                const idx = RUN_REVIEW_MODES.indexOf(prev);
+                return RUN_REVIEW_MODES[(idx + 1) % RUN_REVIEW_MODES.length];
+            });
         }
     });
     const refreshStatus = useCallback(async () => {
@@ -181,10 +196,12 @@ export function RunScreen({ header, featureName, projectRoot, sessionState, moni
         if (!isMountedRef.current)
             return;
         setBranch(getGitBranch(projectRoot));
-        // Update latest commit hash
+        // Update latest commit hash and recent commits
         const head = getCurrentCommitHash(projectRoot);
-        if (head && isMountedRef.current)
+        if (head && isMountedRef.current) {
             setLatestCommit(head);
+            setRecentCommits(getRecentCommits(projectRoot, 3));
+        }
         // Collect new activity events from log and phase changes
         const logPath = getLoopLogPath(featureName);
         const allLogEvents = parseLoopLog(logPath);
@@ -198,11 +215,33 @@ export function RunScreen({ header, featureName, projectRoot, sessionState, moni
         if (currentPhases) {
             lastKnownPhasesRef.current = currentPhases;
         }
+        // Emit a commit activity event when HEAD changes
+        if (head && head !== lastCommitForEventRef.current && lastCommitForEventRef.current !== null) {
+            newLogEvents.push({
+                timestamp: Date.now(),
+                message: `New commit: ${head.slice(0, 7)}`,
+                status: 'success',
+            });
+        }
+        lastCommitForEventRef.current = head ?? null;
         const MAX_STORED_EVENTS = 100;
         const newEvents = [...newLogEvents, ...phaseEvents];
         if (newEvents.length > 0 && isMountedRef.current) {
+            lastActivityTimeRef.current = Date.now();
             setActivityEvents((prev) => [...prev, ...newEvents].slice(-MAX_STORED_EVENTS));
         }
+        else if (nextStatus.running &&
+            nextStatus.phase !== 'Idle' &&
+            Date.now() - lastActivityTimeRef.current > 30_000 &&
+            isMountedRef.current) {
+            // Inject a synthetic "session in progress" event when stale
+            // Update lastActivityTimeRef so this doesn't fire every poll cycle
+            lastActivityTimeRef.current = Date.now();
+            setActivityEvents((prev) => [
+                ...prev,
+                { timestamp: Date.now(), message: `${nextStatus.phase} session in progress...`, status: 'in-progress' },
+            ].slice(-MAX_STORED_EVENTS));
+        }
         // Check for pending action request (loop waiting for user input)
         const request = readActionRequest(featureName);
         if (!isMountedRef.current)
@@ -230,6 +269,9 @@ export function RunScreen({ header, featureName, projectRoot, sessionState, moni
             const tasksDone = nextTasks.tasksDone + nextTasks.e2eDone;
             const tasksTotal = tasksDone + nextTasks.tasksPending + nextTasks.e2ePending;
             const errorTail = exitCode !== 0 ? readLogTail(logPath, ERROR_TAIL_LINES) || undefined : undefined;
+            // Use feat/<feature> as the branch name for summary. getGitBranch() returns
+            // "main" after squash-merge + worktree cleanup, which breaks PR/issue detection.
+            const summaryBranch = `feat/${featureName}`;
             const basicSummary = {
                 feature: featureName,
                 iterations: nextStatus.iteration,
@@ -242,13 +284,13 @@ export function RunScreen({ header, featureName, projectRoot, sessionState, moni
                 cacheRead: nextStatus.cacheRead,
                 exitCode,
                 exitCodeInferred: true,
-                branch: getGitBranch(projectRoot),
+                branch: summaryBranch,
                 logPath,
                 errorTail,
             };
             let enhancedSummary;
             try {
-                enhancedSummary = buildEnhancedRunSummary(basicSummary, projectRoot, featureName);
+                enhancedSummary = buildEnhancedRunSummary(basicSummary, projectRoot, featureName, baselineCommit);
             }
             catch (err) {
                 logger.error(`Failed to build enhanced summary for ${featureName}: ${err instanceof Error ? err.message : String(err)}`);
@@ -260,7 +302,7 @@ export function RunScreen({ header, featureName, projectRoot, sessionState, moni
                 logger.error(`Failed to persist summary file for ${featureName}: ${err instanceof Error ? err.message : String(err)}`);
             });
         }
-    }, [featureName, projectRoot, monitorOnly]);
+    }, [featureName, projectRoot, monitorOnly, baselineCommit]);
     // Keep a stable ref to the latest refreshStatus so the spawn effect
     // can schedule polls without re-running when refreshStatus changes.
     const refreshStatusRef = useRef(refreshStatus);
@@ -308,6 +350,10 @@ export function RunScreen({ header, featureName, projectRoot, sessionState, moni
                 try {
                     const config = sessionState.config ?? await loadConfigWithDefaults(projectRoot);
                     specsDirRef.current = config.paths.specs;
+                    if (!cancelled) {
+                        setLoopModel(config.loop.defaultModel);
+                        setReviewMode((prev) => reviewModeProp ?? config.loop.reviewMode ?? prev);
+                    }
                 }
                 catch (err) {
                     const reason = err instanceof Error ? err.message : String(err);
@@ -350,6 +396,7 @@ export function RunScreen({ header, featureName, projectRoot, sessionState, moni
                 configRootRef.current = config.paths.root;
                 maxIterationsRef.current = config.loop.maxIterations;
                 maxE2eAttemptsRef.current = config.loop.maxE2eAttempts;
+                setLoopModel(config.loop.defaultModel);
                 const specFile = findSpecFile(projectRoot, featureName, config.paths.specs);
                 if (!specFile) {
                     setError(`Spec file not found for "${featureName}". Run /new ${featureName} first.`);
@@ -362,9 +409,10 @@ export function RunScreen({ header, featureName, projectRoot, sessionState, moni
                     setIsStarting(false);
                     return;
                 }
-                const reviewMode = reviewModeProp ?? config.loop.reviewMode ?? 'manual';
-                if (reviewMode !== 'manual' && reviewMode !== 'auto' && reviewMode !== 'merge') {
-                    setError(`Invalid reviewMode '${reviewMode}'. Allowed values are 'manual', 'auto', or 'merge'.`);
+                const effectiveReviewMode = reviewModeProp ?? config.loop.reviewMode ?? 'manual';
+                setReviewMode(effectiveReviewMode);
+                if (effectiveReviewMode !== 'manual' && effectiveReviewMode !== 'auto' && effectiveReviewMode !== 'merge') {
+                    setError(`Invalid reviewMode '${effectiveReviewMode}'. Allowed values are 'manual', 'auto', or 'merge'.`);
                     setIsStarting(false);
                     return;
                 }
@@ -381,7 +429,7 @@ export function RunScreen({ header, featureName, projectRoot, sessionState, moni
                         String(config.loop.maxIterations),
                         String(config.loop.maxE2eAttempts),
                         '--review-mode',
-                        reviewMode,
+                        effectiveReviewMode,
                     ];
                     const child = spawn('bash', [scriptPath, ...args], {
                         cwd: dirname(scriptPath),
@@ -425,6 +473,10 @@ export function RunScreen({ header, featureName, projectRoot, sessionState, moni
                             closeSync(logFd);
                             logFdClosed = true;
                         }
+                        if (!isMountedRef.current)
+                            return;
+                        // Wait for bash to flush state files (.phases, .tokens, .final)
+                        await new Promise((r) => setTimeout(r, 200));
                         if (!isMountedRef.current)
                             return;
                         let latestStatus;
@@ -436,7 +488,7 @@ export function RunScreen({ header, featureName, projectRoot, sessionState, moni
                         catch (err) {
                             logger.error(`Failed to read final run status for ${featureName}: ${err instanceof Error ? err.message : String(err)}`);
                             latestStatus = { running: false, iteration: 0, maxIterations: config.loop.maxIterations, phase: 'unknown', tokensInput: 0, tokensOutput: 0, cacheCreate: 0, cacheRead: 0 };
-                            latestTasks = { tasksDone: 0, tasksPending: 0, e2eDone: 0, e2ePending: 0 };
+                            latestTasks = { tasksDone: 0, tasksPending: 0, e2eDone: 0, e2ePending: 0, planExists: false };
                         }
                         const tasksDone = latestTasks.tasksDone + latestTasks.e2eDone;
                         const tasksTotal = tasksDone + latestTasks.tasksPending + latestTasks.e2ePending;
@@ -453,14 +505,14 @@ export function RunScreen({ header, featureName, projectRoot, sessionState, moni
                             cacheCreate: latestStatus.cacheCreate,
                             cacheRead: latestStatus.cacheRead,
                             exitCode,
-                            branch: getGitBranch(projectRoot),
+                            branch: `feat/${featureName}`,
                             logPath,
                             errorTail,
                         };
                         // Build enhanced summary with phases, git stats, PR/issue metadata
                         let enhancedSummary;
                         try {
-                            enhancedSummary = buildEnhancedRunSummary(basicSummary, projectRoot, featureName);
+                            enhancedSummary = buildEnhancedRunSummary(basicSummary, projectRoot, featureName, baselineCommit);
                         }
                         catch (err) {
                             logger.error(`Failed to build enhanced summary for ${featureName}: ${err instanceof Error ? err.message : String(err)}`);
@@ -516,8 +568,11 @@ export function RunScreen({ header, featureName, projectRoot, sessionState, moni
         : actionRequest
             ? 'Select an option, Esc for default'
             : monitorOnly
-                ? 'Ctrl+C stop, Esc back'
-                : 'Ctrl+C stop, Esc background';
+                ? 'Ctrl+C stop, Esc back, Shift+R review mode'
+                : 'Ctrl+C stop, Esc background, Shift+R review mode';
+    // Progress bar label padding — align all labels to the longest one
+    const PROGRESS_LABEL_WIDTH = 17; // "Implementation: " padded
+    const padLabel = (label) => label.padEnd(PROGRESS_LABEL_WIDTH);
     // Action select handler — awaits write before clearing prompt
     const handleActionSelect = useCallback(async (choiceId) => {
         if (!actionRequest)
@@ -558,7 +613,8 @@ export function RunScreen({ header, featureName, projectRoot, sessionState, moni
             action: 'Run Loop',
             phase: phaseLine,
             path: featureName,
-        }, children: completionSummary ? (_jsx(RunCompletionSummary, { summary: completionSummary })) : (!error && (_jsxs(_Fragment, { children: [_jsxs(Box, { marginTop: 1, flexDirection: "row", children: [_jsx(Text, { children: "Phase: " }), _jsx(Text, { color: colors.yellow, children: phaseLine }), _jsx(Text, { dimColor: true, children: theme.statusLine.separator }), _jsx(Text, { children: "Iter: " }), _jsx(Text, { color: colors.green, children: status.iteration }), _jsxs(Text, { dimColor: true, children: ["/", status.maxIterations || maxIterationsRef.current || '-'] }), _jsx(Text, { dimColor: true, children: theme.statusLine.separator }), _jsx(Text, { children: "Branch: " }), _jsx(Text, { color: colors.blue, children: branch })] }), _jsxs(Box, { marginTop: 1, flexDirection: "row", children: [_jsx(Text, { children: "Tokens: " }), _jsx(Text, { color: colors.pink, children: formatNumber(totalTokens) }), _jsxs(Text, { dimColor: true, children: [" (in:", formatNumber(status.tokensInput), " out:", formatNumber(status.tokensOutput), " cache:", formatNumber(status.cacheRead), ")"] }), _jsx(Text, { dimColor: true, children: theme.statusLine.separator }), _jsxs(Text, { dimColor: true, children: ["Elapsed: ", formatDuration(startTimeRef.current)] })] }), _jsxs(Box, { marginTop: 1, flexDirection: "column", children: [_jsxs(Box, { flexDirection: "row", alignItems: "center", gap: 1, children: [_jsx(Text, { bold: true, children: "Implementation:" }), _jsx(ProgressBar, { percent: percentTasks }), _jsxs(Text, { children: [percentTasks, "%"] }), _jsxs(Text, { color: colors.green, children: ['\u2713', " ", tasks.tasksDone] }), _jsxs(Text, { color: colors.yellow, children: ['\u25cb', " ", tasks.tasksPending] })] }), totalE2e > 0 && (_jsxs(Box, { flexDirection: "row", alignItems: "center", gap: 1, children: [_jsx(Text, { bold: true, children: "E2E Tests:" }), _jsx(ProgressBar, { percent: percentE2e }), _jsxs(Text, { children: [percentE2e, "%"] }), _jsxs(Text, { color: colors.green, children: ['\u2713', " ", tasks.e2eDone] }), _jsxs(Text, { color: colors.yellow, children: ['\u25cb', " ", tasks.e2ePending] })] })), _jsxs(Box, { flexDirection: "row", alignItems: "center", gap: 1, marginTop: 1, children: [_jsx(Text, { bold: true, children: "Overall:" }), _jsx(ProgressBar, { percent: percentAll }), _jsxs(Text, { children: [percentAll, "%"] }), _jsxs(Text, { color: colors.green, children: ['\u2713', " ", doneAll] }), _jsxs(Text, { color: colors.yellow, children: ['\u25cb', " ", totalAll - doneAll] })] })] }), _jsxs(Box, { marginTop: 1, flexDirection: "column", children: [_jsx(Text, { bold: true, children: "Activity" }), _jsx(ActivityFeed, { events: activityEvents, latestCommit: baselineCommit && latestCommit && baselineCommit !== latestCommit
+            extra: `review: ${reviewMode}`,
+        }, children: completionSummary ? (_jsx(RunCompletionSummary, { summary: completionSummary })) : (!error && (_jsxs(_Fragment, { children: [_jsxs(Box, { marginTop: 1, flexDirection: "row", children: [_jsx(Text, { children: "Phase: " }), _jsx(Text, { color: colors.yellow, children: phaseLine }), _jsx(Text, { dimColor: true, children: theme.statusLine.separator }), _jsx(Text, { children: "Iter: " }), _jsx(Text, { color: colors.green, children: status.iteration }), _jsxs(Text, { dimColor: true, children: ["/", status.maxIterations || maxIterationsRef.current || '-'] }), _jsx(Text, { dimColor: true, children: theme.statusLine.separator }), _jsx(Text, { children: "Branch: " }), _jsx(Text, { color: colors.blue, children: branch }), loopModel && (_jsxs(_Fragment, { children: [_jsx(Text, { dimColor: true, children: theme.statusLine.separator }), _jsx(Text, { children: "Model: " }), _jsx(Text, { color: colors.blue, children: loopModel })] }))] }), _jsxs(Box, { marginTop: 1, flexDirection: "row", children: [_jsx(Text, { children: "Tokens: " }), _jsx(Text, { color: colors.pink, children: formatNumber(totalTokens) }), _jsxs(Text, { dimColor: true, children: [" (in:", formatNumber(status.tokensInput), " out:", formatNumber(status.tokensOutput), " cache:", formatNumber(status.cacheRead), ")"] }), _jsx(Text, { dimColor: true, children: theme.statusLine.separator }), _jsxs(Text, { dimColor: true, children: ["Elapsed: ", formatDuration(startTimeRef.current)] })] }), _jsxs(Box, { marginTop: 1, flexDirection: "column", children: [_jsxs(Box, { flexDirection: "row", alignItems: "center", gap: 1, children: [_jsx(Text, { bold: true, children: padLabel('Implementation:') }), _jsx(ProgressBar, { percent: percentTasks }), _jsxs(Text, { children: [String(percentTasks).padStart(3), "%"] }), _jsxs(Text, { color: colors.green, children: ['\u2713', " ", tasks.tasksDone] }), _jsxs(Text, { color: colors.yellow, children: ['\u25cb', " ", tasks.tasksPending] })] }), totalE2e > 0 && (_jsxs(Box, { flexDirection: "row", alignItems: "center", gap: 1, children: [_jsx(Text, { bold: true, children: padLabel('E2E Tests:') }), _jsx(ProgressBar, { percent: percentE2e }), _jsxs(Text, { children: [String(percentE2e).padStart(3), "%"] }), _jsxs(Text, { color: colors.green, children: ['\u2713', " ", tasks.e2eDone] }), _jsxs(Text, { color: colors.yellow, children: ['\u25cb', " ", tasks.e2ePending] })] })), _jsxs(Box, { flexDirection: "row", alignItems: "center", gap: 1, marginTop: 1, children: [_jsx(Text, { bold: true, children: padLabel('Overall:') }), _jsx(ProgressBar, { percent: percentAll }), _jsxs(Text, { children: [String(percentAll).padStart(3), "%"] }), _jsxs(Text, { color: colors.green, children: ['\u2713', " ", doneAll] }), _jsxs(Text, { color: colors.yellow, children: ['\u25cb', " ", totalAll - doneAll] })] })] }), recentCommits.length > 0 && (_jsxs(Box, { marginTop: 1, flexDirection: "column", children: [_jsx(Text, { bold: true, children: "Recent Commits" }), recentCommits.map((c) => (_jsx(Box, { flexDirection: "row", children: _jsxs(Text, { dimColor: true, children: ["  ", c.hash, " ", c.title] }) }, c.hash)))] })), _jsxs(Box, { marginTop: 1, flexDirection: "column", children: [_jsx(Text, { bold: true, children: "Activity" }), _jsx(ActivityFeed, { events: activityEvents, latestCommit: baselineCommit && latestCommit && baselineCommit !== latestCommit
                                 ? `${baselineCommit} \u2192 ${latestCommit}`
                                 : latestCommit || undefined })] })] }))) }));
 }

package/dist/tui/utils/build-run-summary.d.ts

@@ -21,4 +21,4 @@ import type { RunSummary } from '../screens/RunScreen.js';
  * which blocks the event loop. Callers should wrap in try-catch to handle
  * failures gracefully.
  */
-export declare function buildEnhancedRunSummary(basicSummary: RunSummary, projectRoot: string, feature: string): RunSummary;
+export declare function buildEnhancedRunSummary(basicSummary: RunSummary, projectRoot: string, feature: string, baselineOverride?: string | null): RunSummary;

package/dist/tui/utils/build-run-summary.js

@@ -3,86 +3,10 @@
  */
 import { existsSync, readFileSync } from 'node:fs';
 import { logger } from '../../utils/logger.js';
+import { execFileSync } from 'node:child_process';
 import { getCurrentCommitHash, getDiffStats, getCommitList } from './git-summary.js';
 import { getPrForBranch, getLinkedIssue } from './pr-summary.js';
-/**
- * Phase ID to human-readable label mapping
- */
-const PHASE_LABELS = {
-    planning: 'Planning',
-    implementation: 'Implementation',
-    e2e_testing: 'E2E Testing',
-    verification: 'Verification',
-    pr_review: 'PR & Review',
-};
-const VALID_PHASE_STATUSES = new Set(['success', 'skipped', 'failed']);
-/**
- * Parse phase information from the phases file written by feature-loop.sh.
- *
- * Format: phase_id|status|start_timestamp|end_timestamp
- * The parser handles duplicate phase entries defensively (last status wins,
- * durations aggregate), though feature-loop.sh normally writes one final
- * line per phase.
- *
- * @param phasesFilePath - Path to the phases file
- * @returns Array of phase info objects
- */
-function parsePhases(phasesFilePath) {
-    if (!existsSync(phasesFilePath)) {
-        logger.debug(`Phases file not found: ${phasesFilePath}`);
-        return [];
-    }
-    try {
-        const content = readFileSync(phasesFilePath, 'utf-8').trim();
-        if (!content) {
-            return [];
-        }
-        const lines = content.split('\n');
-        const phaseMap = new Map();
-        for (const line of lines) {
-            const parts = line.split('|');
-            if (parts.length < 4) {
-                logger.warn(`Skipping malformed phase line: ${line}`);
-                continue;
-            }
-            const [id, status, startStr, endStr] = parts;
-            // Validate status
-            if (!VALID_PHASE_STATUSES.has(status)) {
-                logger.warn(`Unknown phase status "${status}" for phase "${id}", treating as failed`);
-            }
-            const validatedStatus = VALID_PHASE_STATUSES.has(status)
-                ? status
-                : 'failed';
-            // Parse timestamps
-            const startTime = parseInt(startStr, 10) || 0;
-            const endTime = parseInt(endStr, 10) || 0;
-            // Calculate duration (end - start) in milliseconds
-            const durationMs = endTime > 0 && startTime > 0 ? (endTime - startTime) * 1000 : undefined;
-            // Get or create phase entry
-            let phase = phaseMap.get(id);
-            if (!phase) {
-                phase = {
-                    id,
-                    label: PHASE_LABELS[id] || id,
-                    status: validatedStatus,
-                    durationMs: 0,
-                };
-                phaseMap.set(id, phase);
-            }
-            // Update status (last status wins)
-            phase.status = validatedStatus;
-            // Aggregate duration
-            if (durationMs !== undefined) {
-                phase.durationMs = (phase.durationMs || 0) + durationMs;
-            }
-        }
-        return Array.from(phaseMap.values());
-    }
-    catch (err) {
-        logger.warn(`Failed to parse phases file: ${err instanceof Error ? err.message : String(err)}`);
-        return [];
-    }
-}
+import { parsePhases } from './loop-status.js';
 /**
  * Read baseline commit hash from the baseline file.
  *
@@ -127,7 +51,7 @@ function readBaselineCommit(baselineFilePath) {
  * which blocks the event loop. Callers should wrap in try-catch to handle
  * failures gracefully.
  */
-export function buildEnhancedRunSummary(basicSummary, projectRoot, feature) {
+export function buildEnhancedRunSummary(basicSummary, projectRoot, feature, baselineOverride) {
     const phasesFilePath = `/tmp/ralph-loop-${feature}.phases`;
     const baselineFilePath = `/tmp/ralph-loop-${feature}.baseline`;
     // Parse phases and set implementation iterations from actual loop count
@@ -153,8 +77,10 @@ export function buildEnhancedRunSummary(basicSummary, projectRoot, feature) {
         completed: basicSummary.tasksDone,
         total: basicSummary.tasksTotal,
     };
-    // Git changes and commits
-    const baselineCommit = readBaselineCommit(baselineFilePath);
+    // Git changes and commits — use override if provided (avoids re-reading a cleaned-up file)
+    const baselineCommit = baselineOverride !== undefined
+        ? (baselineOverride ? baselineOverride.substring(0, 7) : null)
+        : readBaselineCommit(baselineFilePath);
     const currentCommit = getCurrentCommitHash(projectRoot);
     let changes = { available: false };
     let commits = { available: false };
@@ -205,13 +131,19 @@ export function buildEnhancedRunSummary(basicSummary, projectRoot, feature) {
                     available: true,
                     created: true,
                 };
-                // Try to get linked issue, passing prInfo to avoid redundant gh call
-                const issueInfo = getLinkedIssue(projectRoot, basicSummary.branch, prInfo);
+                // Try to get linked issue, passing prInfo and feature name for fallback detection
+                const issueInfo = getLinkedIssue(projectRoot, basicSummary.branch, prInfo, feature);
                 if (issueInfo) {
+                    // When PR is merged with "Closes #N", GitHub auto-closes the issue.
+                    // The summary may be built before GitHub processes the webhook, so
+                    // infer closure from PR state to avoid showing stale "OPEN" status.
+                    const inferredState = prInfo.state === 'MERGED' && issueInfo.state === 'OPEN'
+                        ? 'CLOSED'
+                        : issueInfo.state;
                     issue = {
                         number: issueInfo.number,
                         url: issueInfo.url,
-                        status: issueInfo.state,
+                        status: inferredState,
                         available: true,
                         linked: true,
                     };
@@ -219,6 +151,30 @@ export function buildEnhancedRunSummary(basicSummary, projectRoot, feature) {
                 else {
                     issue = { available: true, linked: false };
                 }
+                // Enrich commits from PR when squash-merge detected (1 local commit + merged PR)
+                if (prInfo.state === 'MERGED' &&
+                    commits.available &&
+                    commits.commitList &&
+                    commits.commitList.length <= 1) {
+                    try {
+                        const prCommitsOutput = execFileSync('gh', ['pr', 'view', String(prInfo.number), '--json', 'commits'], { cwd: projectRoot, encoding: 'utf-8', timeout: 10_000 }).trim();
+                        const prCommitsData = JSON.parse(prCommitsOutput);
+                        const prCommits = prCommitsData.commits;
+                        if (Array.isArray(prCommits) && prCommits.length > 1) {
+                            commits = {
+                                ...commits,
+                                commitList: prCommits.map((c) => ({
+                                    hash: c.oid?.substring(0, 7) ?? '',
+                                    title: c.messageHeadline ?? '',
+                                })),
+                                mergeType: 'squash',
+                            };
+                        }
+                    }
+                    catch (err) {
+                        logger.debug(`Failed to fetch PR commits: ${err instanceof Error ? err.message : String(err)}`);
+                    }
+                }
             }
             else {
                 pr = { available: true, created: false };

package/dist/tui/utils/clear-screen.d.ts

@@ -0,0 +1,14 @@
+/**
+ * Clear the terminal screen to prevent stale PTY scroll buffer artifacts.
+ *
+ * When Ink re-renders significantly shorter output (e.g. after dismissing a
+ * 25-row IssuePicker), the old content remains in the PTY scroll buffer,
+ * causing visual duplication of the banner and other content. This sends
+ * ANSI escape sequences to clear the entire screen and reset the cursor.
+ *
+ * This is an intentional mix of imperative stdout writes with Ink's
+ * declarative rendering — Ink has no API to flush the PTY scroll buffer.
+ * Call this *before* state updates that shrink the rendered output so
+ * Ink's next render paints onto a clean screen.
+ */
+export declare function clearScreen(stdout: NodeJS.WriteStream): void;

package/dist/tui/utils/clear-screen.js

@@ -0,0 +1,16 @@
+/**
+ * Clear the terminal screen to prevent stale PTY scroll buffer artifacts.
+ *
+ * When Ink re-renders significantly shorter output (e.g. after dismissing a
+ * 25-row IssuePicker), the old content remains in the PTY scroll buffer,
+ * causing visual duplication of the banner and other content. This sends
+ * ANSI escape sequences to clear the entire screen and reset the cursor.
+ *
+ * This is an intentional mix of imperative stdout writes with Ink's
+ * declarative rendering — Ink has no API to flush the PTY scroll buffer.
+ * Call this *before* state updates that shrink the rendered output so
+ * Ink's next render paints onto a clean screen.
+ */
+export function clearScreen(stdout) {
+    stdout.write('\x1b[2J\x1b[H');
+}

package/dist/tui/utils/git-summary.d.ts

@@ -34,4 +34,5 @@ export declare function getDiffStats(projectRoot: string, fromHash: string, toHa
  * @param toHash - Ending commit hash (inclusive)
  * @returns Array of commit entries, or null if not available
  */
+export declare function getRecentCommits(projectRoot: string, count?: number): CommitLogEntry[];
 export declare function getCommitList(projectRoot: string, fromHash: string, toHash: string): CommitLogEntry[] | null;