@cnrai/pave 0.3.32 → 0.3.34

package/test/tui-attach.test.js

@@ -0,0 +1,1255 @@
+#!/usr/bin/env node
+/**
+ * Tests for TUI attach mode (Issue #202)
+ * Phase 5: pave -a <agent> attached TUI
+ *
+ * Tests cover:
+ * - Agent registry getAgentServerUrl helper
+ * - Entry point routing (source inspection)
+ * - attached.js module structure (source inspection)
+ * - Args parser routing for -a without command
+ *
+ * Designed to run standalone: node test/tui-attach.test.js
+ */
+
+const path = require('path');
+const fs = require('fs');
+const os = require('os');
+
+const registryPath = path.join(__dirname, '..', 'lib', 'agent-registry.js');
+const registry = require(registryPath);
+const { parseArgs } = require(path.join(__dirname, '..', 'lib', 'args-parser.js'));
+
+const testDir = path.join(os.tmpdir(), 'pave-test-attach-' + process.pid);
+
+let passed = 0;
+let failed = 0;
+
+function assert(condition, msg) {
+  if (!condition) throw new Error('Assertion failed: ' + msg);
+}
+
+function runTest(name, fn) {
+  try {
+    fn();
+    console.log('\u2705 ' + name);
+    passed++;
+  } catch (e) {
+    console.log('\u274C ' + name + ': ' + e.message);
+    failed++;
+  }
+}
+
+function resetTestDir() {
+  try { fs.rmSync(testDir, { recursive: true }); } catch (e) {}
+  registry.setAgentsDir(testDir);
+}
+
+function cleanup() {
+  try { fs.rmSync(testDir, { recursive: true }); } catch (e) {}
+  registry.resetAgentsDir();
+}
+
+// ============================================================
+// getAgentServerUrl
+// ============================================================
+
+resetTestDir();
+
+runTest('getAgentServerUrl: returns null for nonexistent agent', () => {
+  resetTestDir();
+  const url = registry.getAgentServerUrl('nonexistent');
+  assert(url === null, 'should return null');
+});
+
+runTest('getAgentServerUrl: returns null when agent has no serverUrl', () => {
+  resetTestDir();
+  registry.writeStatus('test-agent', {
+    state: 'working',
+    pid: process.pid,
+    sessionId: 'sess-1',
+    iteration: 1,
+    startedAt: Date.now(),
+  });
+  const url = registry.getAgentServerUrl('test-agent');
+  assert(url === null, 'should return null when no serverUrl');
+});
+
+runTest('getAgentServerUrl: returns serverUrl when agent is alive', () => {
+  resetTestDir();
+  registry.writeStatus('test-agent', {
+    state: 'working',
+    pid: process.pid, // Current process is alive
+    serverUrl: 'http://localhost:4096',
+    sessionId: 'sess-1',
+    iteration: 1,
+    startedAt: Date.now(),
+  });
+  const url = registry.getAgentServerUrl('test-agent');
+  assert(url === 'http://localhost:4096', 'should return serverUrl: ' + url);
+});
+
+runTest('getAgentServerUrl: returns null when agent is stopped', () => {
+  resetTestDir();
+  registry.writeStatus('test-agent', {
+    state: 'stopped',
+    pid: process.pid,
+    serverUrl: 'http://localhost:4096',
+    sessionId: 'sess-1',
+    iteration: 5,
+    startedAt: Date.now(),
+  });
+  const url = registry.getAgentServerUrl('test-agent');
+  assert(url === null, 'should return null when agent is stopped');
+});
+
+runTest('getAgentServerUrl: returns null when agent PID is dead', () => {
+  resetTestDir();
+  registry.writeStatus('test-agent', {
+    state: 'working',
+    pid: 999999999, // Almost certainly not a real PID
+    serverUrl: 'http://localhost:4096',
+    sessionId: 'sess-1',
+    iteration: 1,
+    startedAt: Date.now(),
+  });
+  const url = registry.getAgentServerUrl('test-agent');
+  assert(url === null, 'should return null when PID is dead');
+});
+
+// ============================================================
+// Status.json includes serverUrl (source inspection)
+// ============================================================
+
+runTest('agent loop writes serverUrl to status.json', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', 'index.js'), 'utf8');
+  // Count occurrences of serverUrl in writeStatus calls (shorthand or explicit)
+  const matches = source.match(/\bserverUrl[,\s]/g) || [];
+  assert(matches.length >= 3, 'should write serverUrl in at least 3 writeStatus calls, found ' + matches.length);
+});
+
+runTest('serverUrl declared in outer scope for cleanup access', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', 'index.js'), 'utf8');
+  // serverUrl should be declared as let in the outer handleAgentCommand scope
+  assert(source.indexOf('let serverUrl = null;') !== -1, 'serverUrl should be declared as let in outer scope');
+});
+
+// ============================================================
+// Entry point routing (source inspection)
+// ============================================================
+
+runTest('pave/index.js routes -a to runTUIAttached', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', 'index.js'), 'utf8');
+  assert(source.indexOf('runTUIAttached') !== -1, 'should reference runTUIAttached');
+  assert(source.indexOf("require('../tui/attached.js')") !== -1 ||
+         source.indexOf("require('../tui/attached')") !== -1,
+  'should require tui/attached.js');
+});
+
+runTest('entry point checks args.agent before routing', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', 'index.js'), 'utf8');
+  // The routing code should check args.agent
+  const routeIdx = source.indexOf('args.agent');
+  const tuiIdx = source.indexOf('runTUIAttached');
+  assert(routeIdx < tuiIdx, 'args.agent check should come before runTUIAttached call');
+});
+
+// ============================================================
+// attached.js module structure
+// ============================================================
+
+runTest('tui/attached.js exists', () => {
+  const attachedPath = path.join(__dirname, '..', '..', 'tui', 'attached.js');
+  assert(fs.existsSync(attachedPath), 'attached.js should exist at: ' + attachedPath);
+});
+
+runTest('tui/attached.js exports runTUIAttached', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  assert(source.indexOf('runTUIAttached') !== -1, 'should define runTUIAttached');
+  assert(source.indexOf('module.exports') !== -1, 'should have module.exports');
+});
+
+runTest('attached.js uses blessed for TUI', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  assert(source.indexOf("require('blessed')") !== -1, 'should require blessed');
+  assert(source.indexOf('blessed.screen') !== -1, 'should create blessed screen');
+  assert(source.indexOf('blessed.box') !== -1, 'should create blessed boxes');
+  assert(source.indexOf('blessed.textarea') !== -1, 'should create blessed textarea for multi-line input');
+});
+
+runTest('attached.js connects to SSE for real-time streaming', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  assert(source.indexOf('/global/event') !== -1 || source.indexOf('event?sessionID') !== -1,
+    'should connect to SSE endpoint');
+  assert(source.indexOf('text/event-stream') !== -1, 'should request text/event-stream');
+});
+
+runTest('attached.js handles opencode-lite SSE events', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  assert(source.indexOf('message.part.delta') !== -1, 'should handle message.part.delta');
+  assert(source.indexOf('message.part.updated') !== -1, 'should handle message.part.updated');
+  assert(source.indexOf('message.updated') !== -1, 'should handle message.updated');
+  assert(source.indexOf('session.status') !== -1, 'should handle session.status');
+  assert(source.indexOf('session.error') !== -1, 'should handle session.error');
+  // Also keeps legacy events for backward compat
+  assert(source.indexOf('text.delta') !== -1, 'should handle legacy text.delta');
+});
+
+runTest('attached.js uses correct server endpoints', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  // History: GET /session/<id>/message
+  assert(source.indexOf("/session/' + currentSessionId + '/message'") !== -1 ||
+         source.indexOf("/session/' + currentSessionId + '/message") !== -1,
+  'should use /session/<id>/message for history');
+  // Abort: POST /session/<id>/abort
+  assert(source.indexOf("/session/' + currentSessionId + '/abort'") !== -1,
+    'should use /session/<id>/abort for interrupt');
+});
+
+runTest('attached.js routes Enter to inbox', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  assert(source.indexOf('writeInboxMessage') !== -1, 'should call writeInboxMessage for normal messages');
+});
+
+runTest('attached.js routes Ctrl+U to interrupt', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  assert(source.indexOf('writeInterrupt') !== -1, 'should call writeInterrupt for Ctrl+U');
+  assert(source.indexOf('C-u') !== -1, 'should bind Ctrl+U key');
+});
+
+runTest('attached.js Ctrl+C detaches without killing agent', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  assert(source.indexOf('C-c') !== -1, 'should bind Ctrl+C');
+  assert(source.indexOf('detach') !== -1, 'should call detach function');
+  // Should NOT contain process.kill or SIGTERM for the agent
+  assert(source.indexOf('process.kill') === -1, 'should NOT kill agent process');
+});
+
+runTest('attached.js polls status.json for updates', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  assert(source.indexOf('STATUS_POLL_INTERVAL') !== -1, 'should define STATUS_POLL_INTERVAL');
+  assert(source.indexOf('setInterval') !== -1, 'should use setInterval for polling');
+  assert(source.indexOf('readStatus') !== -1, 'should read status.json');
+});
+
+runTest('attached.js has SSE reconnection with exponential backoff', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  assert(source.indexOf('SSE_INITIAL_BACKOFF') !== -1, 'should define SSE_INITIAL_BACKOFF');
+  assert(source.indexOf('SSE_MAX_BACKOFF') !== -1, 'should define SSE_MAX_BACKOFF');
+  assert(source.indexOf('sseBackoff * 2') !== -1 || source.indexOf('sseBackoff*2') !== -1,
+    'should double backoff');
+});
+
+runTest('attached.js cleans up sockets on SSE non-200', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  // Non-200 handler should call res.resume(), res.destroy(), req.destroy()
+  assert(source.indexOf('res.resume()') !== -1, 'should call res.resume() on non-200');
+  assert(source.indexOf('res.destroy()') !== -1, 'should call res.destroy() on non-200');
+  assert(source.indexOf('req.destroy()') !== -1, 'should call req.destroy() on non-200');
+});
+
+runTest('attached.js detects session compaction', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  assert(source.indexOf('compacted') !== -1 || source.indexOf('Session compacted') !== -1,
+    'should handle session compaction');
+});
+
+runTest('attached.js detects agent stop', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  assert(source.indexOf('Agent has stopped') !== -1 || source.indexOf('has stopped') !== -1,
+    'should detect and display when agent stops');
+});
+
+runTest('attached.js never POSTs to /session/:id/message', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  // The attached TUI should NEVER post messages to the session directly
+  // It should only use inbox/interrupt (and GET /session/:id/message for history)
+  // Specifically, there must be no httpPost call constructing /session/:id/message
+  const postSessionMessagePattern = /httpPost\s*\([^)]*['"`]\/session\/['"`]\s*\+\s*[^)]*['"`]\/message['"`]/;
+  assert(!postSessionMessagePattern.test(source),
+    'should not POST to /session/:id/message');
+});
+
+// ============================================================
+// Args parser: -a without command
+// ============================================================
+
+runTest('parseArgs: -a without command sets agent', () => {
+  const result = parseArgs(['-a', 'my-agent']);
+  assert(result.agent === 'my-agent', 'agent should be my-agent: ' + result.agent);
+  assert(!result.command || result.command === null, 'command should be null');
+});
+
+runTest('parseArgs: --agent without command sets agent', () => {
+  const result = parseArgs(['--agent', 'test-bot']);
+  assert(result.agent === 'test-bot', 'agent should be test-bot');
+});
+
+// ============================================================
+// Fix: escapeBlessed uses {open}/{close} tokens (Issue #239 Fix 1)
+// ============================================================
+
+runTest('escapeBlessed: uses {open}/{close} tokens, not backslash escaping', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+
+  // Should NOT use backslash escaping (broken in blessed 0.1.81)
+  assert(source.indexOf("'\\\\{'") === -1 && source.indexOf("'\\\\}'") === -1,
+    'escapeBlessed should not use backslash escaping');
+
+  // Should use {open}/{close} approach
+  assert(source.indexOf('{open}') !== -1 && source.indexOf('{close}') !== -1,
+    'escapeBlessed should use {open}/{close} tokens');
+});
+
+// ============================================================
+// Fix: formatUptime called correctly (Issue #239 Fix 2)
+// ============================================================
+
+runTest('formatUptime: receives startedAt timestamp, not pre-computed elapsed', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+
+  // Should NOT have the double-subtraction pattern
+  assert(source.indexOf('Date.now() - st.startedAt') === -1,
+    'formatUptime should not receive Date.now() - st.startedAt (double subtraction)');
+
+  // Should pass st.startedAt directly
+  assert(source.indexOf('registry.formatUptime(st.startedAt)') !== -1,
+    'formatUptime should receive st.startedAt directly');
+});
+
+runTest('formatUptime: returns reasonable value for recent timestamp', () => {
+  // formatUptime expects a raw startedAt timestamp
+  const fiveMinAgo = Date.now() - 5 * 60 * 1000;
+  const result = registry.formatUptime(fiveMinAgo);
+  assert(result === '5m', 'Expected 5m for 5 minutes ago, got: ' + result);
+});
+
+runTest('formatUptime: double-subtraction would produce ~55000 years', () => {
+  // Demonstrate the bug: if you pass (Date.now() - startedAt) instead of startedAt,
+  // formatUptime computes Date.now() - (Date.now() - startedAt) = startedAt ≈ 1.75 trillion ms
+  const fiveMinAgo = Date.now() - 5 * 60 * 1000;
+  const buggyElapsed = Date.now() - fiveMinAgo; // ~300000 ms
+  const buggyResult = registry.formatUptime(buggyElapsed);
+  // This would give Date.now() - 300000 ≈ current time minus 5 min ≈ huge uptime
+  // The correct result for 5 min ago should be "5m", not something with 'd'
+  assert(buggyResult.indexOf('d') !== -1 || buggyResult.indexOf('h') !== -1,
+    'Passing elapsed ms to formatUptime should give wrong large result (demonstrates bug): ' + buggyResult);
+});
+
+// ============================================================
+// Fix: No unused fs import (Issue #239 Fix 3)
+// ============================================================
+
+runTest('attached.js: no unused fs import', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+
+  // Should not import fs
+  const fsImport = /const\s+fs\s*=\s*require\s*\(\s*['"]fs['"]\s*\)/.test(source);
+  assert(!fsImport, 'attached.js should not have unused fs import');
+});
+
+// ============================================================
+// Fix: resolveTerminal extracted to shared utility (Issue #239 Fix 4)
+// ============================================================
+
+runTest('attached.js: imports resolveTerminal from lib/terminal-utils, not index', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+
+  // Should NOT require('./index')
+  assert(source.indexOf("require('./index')") === -1,
+    'attached.js should not require the full TUI index.js');
+
+  // Should require from lib/terminal-utils
+  assert(source.indexOf("require('./lib/terminal-utils')") !== -1,
+    'attached.js should import from lib/terminal-utils');
+});
+
+runTest('lib/terminal-utils.js: exports resolveTerminal function', () => {
+  const termUtils = require(path.join(__dirname, '..', '..', 'tui', 'lib', 'terminal-utils.js'));
+
+  assert(typeof termUtils.resolveTerminal === 'function',
+    'terminal-utils should export resolveTerminal');
+  assert(Array.isArray(termUtils.KNOWN_TERMINALS),
+    'terminal-utils should export KNOWN_TERMINALS');
+  assert(typeof termUtils.REMAP_TERMINALS === 'object',
+    'terminal-utils should export REMAP_TERMINALS');
+});
+
+runTest('lib/terminal-utils.js: resolveTerminal works correctly', () => {
+  const termUtils = require(path.join(__dirname, '..', '..', 'tui', 'lib', 'terminal-utils.js'));
+
+  // Known terminal passes through
+  let result = termUtils.resolveTerminal('xterm-256color');
+  assert(result.effectiveTerm === 'xterm-256color', 'xterm-256color should pass through');
+  assert(!result.wasRemapped, 'should not be remapped');
+  assert(!result.wasUnknown, 'should not be unknown');
+
+  // Ghostty gets remapped
+  result = termUtils.resolveTerminal('xterm-ghostty');
+  assert(result.effectiveTerm === 'xterm-256color', 'ghostty should remap to xterm-256color');
+  assert(result.wasRemapped, 'should be remapped');
+
+  // Unknown falls back
+  result = termUtils.resolveTerminal('totally-unknown-term');
+  assert(result.effectiveTerm === 'xterm-256color', 'unknown should fallback');
+  assert(result.wasUnknown, 'should be flagged as unknown');
+
+  // Null input
+  result = termUtils.resolveTerminal(null);
+  assert(result.effectiveTerm === 'unknown', 'null should give unknown');
+});
+
+runTest('index.js: imports resolveTerminal from lib/terminal-utils', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'index.js'), 'utf8');
+
+  assert(source.indexOf("require('./lib/terminal-utils')") !== -1,
+    'index.js should import from lib/terminal-utils');
+
+  // Should NOT have the inline function definition anymore
+  assert(source.indexOf('function resolveTerminal(term)') === -1,
+    'index.js should not define resolveTerminal inline anymore');
+});
+
+// ============================================================
+// Issue #245 Fix 1: Chat box padding — bottom:3 instead of height
+// ============================================================
+
+runTest('245-1: chatBox uses bottom:3 to avoid overlap with input', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  // Should use bottom:3 for chatBox (robust positioning)
+  assert(source.indexOf('bottom: 3') !== -1, 'chatBox should use bottom: 3');
+  // Should NOT use the old height: "100%-3" pattern for chatBox
+  // (The string '100%-3' in the initial chatBox creation was the cause)
+  const chatBoxSection = source.substring(
+    source.indexOf('// Chat display area'),
+    source.indexOf('// Input box'),
+  );
+  assert(chatBoxSection.indexOf("'100%-3'") === -1 && chatBoxSection.indexOf('"100%-3"') === -1,
+    'chatBox should not use height: 100%-3');
+});
+
+// ============================================================
+// Issue #245 Fix 2: Ctrl+U works when input is focused
+// ============================================================
+
+runTest('245-2: Ctrl+U bound on inputBox (not just screen)', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  // Should have inputBox.key(['C-u']) binding
+  assert(source.indexOf("inputBox.key(['C-u']") !== -1 || source.indexOf('inputBox.key(["C-u"]') !== -1,
+    'Ctrl+U should be bound on inputBox');
+  // Should also have screen.key(['C-u']) binding
+  assert(source.indexOf("screen.key(['C-u']") !== -1,
+    'Ctrl+U should also be bound on screen');
+});
+
+runTest('245-2: Ctrl+U handler calls writeInterrupt', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  // Both bindings should route to the same handler that calls writeInterrupt
+  assert(source.indexOf('handleCtrlU') !== -1, 'Should use shared handleCtrlU handler');
+  assert(source.indexOf('writeInterrupt') !== -1, 'handleCtrlU should call writeInterrupt');
+});
+
+// ============================================================
+// Issue #245 Fix 3: Ctrl+C works when input is focused
+// ============================================================
+
+runTest('245-3: Ctrl+C bound on inputBox (not just screen)', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  // Should have inputBox.key(['C-c']) binding
+  assert(source.indexOf("inputBox.key(['C-c']") !== -1 || source.indexOf('inputBox.key(["C-c"]') !== -1,
+    'Ctrl+C should be bound on inputBox');
+});
+
+runTest('245-3: Ctrl+C calls detach (not process.kill)', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  // The Ctrl+C inputBox handler should call detach()
+  let ctrlCIndex = source.indexOf("inputBox.key(['C-c']");
+  if (ctrlCIndex === -1) {
+    ctrlCIndex = source.indexOf("inputBox.key('C-c'");
+  }
+  assert(ctrlCIndex !== -1, 'Should have Ctrl+C binding on inputBox');
+  const ctrlCSection = source.substring(ctrlCIndex, ctrlCIndex + 200);
+  assert(ctrlCSection.indexOf('detach()') !== -1, 'Ctrl+C on inputBox should call detach()');
+  // Should NOT kill agent
+  assert(source.indexOf('process.kill') === -1, 'Should not kill agent process');
+});
+
+// ============================================================
+// Issue #245 Fix 4: Markdown rendering
+// ============================================================
+
+runTest('245-4: attached.js imports MarkdownRenderer', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  assert(source.indexOf("require('./lib/markdown-renderer')") !== -1,
+    'Should import MarkdownRenderer');
+  assert(source.indexOf('new MarkdownRenderer') !== -1,
+    'Should instantiate MarkdownRenderer');
+});
+
+runTest('245-4: assistant history messages rendered with markdown', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  // History rendering should use mdRenderer.render() for assistant messages
+  assert(source.indexOf('mdRenderer.render(') !== -1,
+    'Should call mdRenderer.render() for assistant messages');
+  // Should have role check for assistant
+  assert(source.indexOf("role === 'assistant'") !== -1,
+    'Should check role === assistant for markdown rendering');
+});
+
+runTest('245-4: streaming text accumulated for markdown finalization', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  // Should accumulate streaming text
+  assert(source.indexOf('streamingRawText') !== -1,
+    'Should track streamingRawText for markdown rendering');
+  assert(source.indexOf('finalizeStreaming') !== -1,
+    'Should have finalizeStreaming function to render markdown');
+});
+
+runTest('245-4: MarkdownRenderer renders basic markdown correctly', () => {
+  const MR = require(path.join(__dirname, '..', '..', 'tui', 'lib', 'markdown-renderer.js'));
+  const md = new MR({ width: 80 });
+
+  // Bold renders
+  let result = md.render('**bold text**');
+  assert(result.indexOf('{bold}') !== -1, 'Should render bold: ' + result);
+
+  // Code block renders (var and 1 get syntax-highlighted, check for structure)
+  result = md.render('```js\nvar x = 1;\n```');
+  assert(result.indexOf('x =') !== -1, 'Should render code block with assignment: ' + result.substring(0, 100));
+  assert(result.indexOf('{gray-fg}') !== -1, 'Should have gray border in code block');
+
+  // Headers render
+  result = md.render('# Hello');
+  assert(result.indexOf('{bold}') !== -1, 'Should render header as bold');
+
+  // Inline code renders
+  result = md.render('Use `foo` here');
+  assert(result.indexOf('{yellow-fg}') !== -1 || result.indexOf('{inverse}') !== -1,
+    'Should render inline code');
+});
+
+// ============================================================
+// Issue #245 Fix 5: Text selection via Ctrl+S mouse toggle
+// ============================================================
+
+runTest('245-5: Ctrl+S bound on both screen and inputBox', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  // screen.key(['C-s'])
+  assert(source.indexOf("screen.key(['C-s']") !== -1,
+    'Ctrl+S should be bound on screen');
+  // inputBox.key(['C-s'])
+  assert(source.indexOf("inputBox.key(['C-s']") !== -1,
+    'Ctrl+S should be bound on inputBox');
+});
+
+runTest('245-5: mouse toggle uses disableMouse/enableMouse', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  assert(source.indexOf('screen.program.disableMouse()') !== -1,
+    'Should call disableMouse for text selection mode');
+  assert(source.indexOf('screen.program.enableMouse()') !== -1,
+    'Should call enableMouse for scroll mode');
+});
+
+runTest('245-5: default mouse mode is selection (disabled)', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  // At the end of initialization, mouse should be disabled by default
+  // Check that the default state sets mouseEnabled = false
+  const defaultSection = source.substring(source.indexOf('// Default to text selection'));
+  assert(defaultSection.indexOf('disableMouse') !== -1,
+    'Should disable mouse capture by default');
+  assert(defaultSection.indexOf('mouseEnabled = false') !== -1,
+    'Should set mouseEnabled = false by default');
+});
+
+// ============================================================
+// Issue #245 Fix 6: Multi-line input (textarea + Alt+Enter)
+// ============================================================
+
+runTest('245-6: inputBox is blessed.textarea (not textbox)', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  assert(source.indexOf('blessed.textarea(') !== -1,
+    'Should use blessed.textarea for multi-line input');
+  // Should NOT use blessed.textbox
+  assert(source.indexOf('blessed.textbox(') === -1,
+    'Should NOT use blessed.textbox');
+});
+
+runTest('245-6: textarea has wrap and wordWrap options', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  // Find the textarea options section
+  const textareaSection = source.substring(
+    source.indexOf('blessed.textarea('),
+    source.indexOf('blessed.textarea(') + 500,
+  );
+  assert(textareaSection.indexOf('wrap: true') !== -1,
+    'textarea should have wrap: true');
+  assert(textareaSection.indexOf('wordWrap: true') !== -1,
+    'textarea should have wordWrap: true');
+});
+
+runTest('245-6: Alt+Enter intercepted at stdin level for newline insertion', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  // Should intercept at raw stdin level (same as main TUI)
+  assert(source.indexOf('origStdinEmit') !== -1,
+    'Should intercept stdin emit for Alt+Enter');
+  assert(source.indexOf('\\x1b\\r') !== -1 || source.indexOf('\\x1b\\n') !== -1,
+    'Should detect Alt+Enter escape sequences');
+  assert(source.indexOf('appendNewline') !== -1,
+    'Should have appendNewline function');
+});
+
+runTest('245-6: adjustInputHeight grows textarea based on content', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  assert(source.indexOf('adjustInputHeight') !== -1,
+    'Should have adjustInputHeight function');
+  // Should have a max height limit
+  assert(source.indexOf('maxHeight') !== -1,
+    'Should limit max height for input');
+});
+
+// ============================================================
+// Issue #245 Fix 7: Escape behavior (unfocus first, detach second)
+// ============================================================
+
+runTest('245-7: Escape unfocuses input first, then detaches', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  // Should track inputFocused state
+  assert(source.indexOf('inputFocused') !== -1,
+    'Should track inputFocused state');
+  // Escape handler should check inputFocused
+  const escIdx = source.indexOf("screen.key(['escape']");
+  assert(escIdx !== -1, 'Escape handler should bind on screen.key');
+  const escSection = source.substring(escIdx);
+  assert(escSection.indexOf('inputFocused') !== -1,
+    'Escape handler should check inputFocused');
+  // Should call inputBox.cancel() or similar to unfocus
+  assert(source.indexOf('inputBox.cancel()') !== -1 || source.indexOf('chatBox.focus()') !== -1,
+    'Should unfocus input on first Escape');
+});
+
+runTest('245-7: Enter re-focuses input when chatBox is focused', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  // chatBox.key(['enter']) should refocus input
+  assert(source.indexOf("chatBox.key(['enter']") !== -1,
+    'Enter on chatBox should re-focus input');
+});
+
+runTest('245-7: inputBox focus event updates inputFocused state', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  assert(source.indexOf("inputBox.on('focus'") !== -1,
+    'Should listen for inputBox focus event');
+});
+
+// ============================================================
+// General: stdin interceptor cleanup on detach
+// ============================================================
+
+runTest('245-general: detach restores original stdin emit', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  // The detach function should restore original stdin emit
+  const detachSection = source.substring(source.indexOf('function detach()'));
+  assert(detachSection.indexOf('origStdinEmit') !== -1,
+    'detach should restore original stdin emit');
+});
+
+runTest('245-general: help text updated with new shortcuts', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  // Help text should mention Ctrl+S and Esc
+  assert(source.indexOf('Ctrl+S') !== -1,
+    'Help text should mention Ctrl+S for mouse toggle');
+  assert(source.indexOf('Esc') !== -1,
+    'Help text should mention Esc');
+});
+
+runTest('245-general: status bar shows mouse mode', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  assert(source.indexOf('mouseMode') !== -1 || source.indexOf('mouse:') !== -1,
+    'Status bar should show mouse mode');
+});
+
+runTest('245-general: resize handler updates markdown renderer width', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  assert(source.indexOf('mdRenderer.width') !== -1,
+    'Resize handler should update mdRenderer.width');
+});
+
+// ============================================================
+// Copilot review follow-up tests
+// ============================================================
+
+runTest('review-1: finalizeStreaming preserves non-stream content (tool/status lines)', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  // finalizeStreaming should locate the escaped raw text and replace only that segment
+  const finalizeSection = source.substring(source.indexOf('function finalizeStreaming'));
+  assert(finalizeSection.indexOf('escapedRawText') !== -1,
+    'Should search for escaped raw text to replace only the streamed segment');
+  assert(finalizeSection.indexOf('after') !== -1,
+    'Should preserve content after the streamed segment');
+  // The new version should include 'after' in the setContent call
+  assert(finalizeSection.indexOf('rendered + after') !== -1,
+    'Should append preserved tail content after rendered markdown');
+});
+
+runTest('review-2: Enter-to-send calls adjustInputHeight after clearValue', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  // Find the Enter key handler (use larger window to capture the full handler)
+  const enterIdx = source.indexOf("inputBox.key('enter'");
+  const enterSection = source.substring(enterIdx, enterIdx + 600);
+  // The send branch (after sendToInbox) should have clearValue followed by adjustInputHeight
+  const sendIdx = enterSection.indexOf('sendToInbox(');
+  const clearAfterSend = enterSection.indexOf('clearValue()', sendIdx);
+  const adjustAfterSend = enterSection.indexOf('adjustInputHeight()', sendIdx);
+  assert(clearAfterSend !== -1, 'Enter send branch should call clearValue');
+  assert(adjustAfterSend !== -1, 'Enter send branch should call adjustInputHeight');
+  assert(adjustAfterSend > clearAfterSend, 'adjustInputHeight should come after clearValue in send branch');
+});
+
+runTest('review-3: adjustInputHeight uses wrap-aware visual line counting', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  const adjustSection = source.substring(source.indexOf('function adjustInputHeight'));
+  // Should calculate visual lines based on innerWidth wrapping
+  assert(adjustSection.indexOf('innerWidth') !== -1,
+    'Should calculate innerWidth for wrap-aware counting');
+  assert(adjustSection.indexOf('Math.ceil') !== -1,
+    'Should use Math.ceil for visual line calculation');
+  assert(adjustSection.indexOf('visualLines') !== -1,
+    'Should count visual lines, not just logical lines');
+});
+
+runTest('review-4: streamingContentStart includes trailing space', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  // Both message.start and message.updated should set header with trailing space
+  const matches = source.match(/streamingContentStart\s*=\s*'[^']*Agent[^']*'/g) || [];
+  for (let i = 0; i < matches.length; i++) {
+    const m = matches[i];
+    // Should end with a space before the closing quote
+    assert(m.indexOf("Agent:{/green-fg} '") !== -1,
+      'streamingContentStart should have trailing space: ' + m);
+  }
+  assert(matches.length >= 2, 'Should have at least 2 streamingContentStart Agent assignments (message.start + message.updated), got: ' + matches.length);
+});
+
+runTest('review-5: inputBox blur handler keeps inputFocused in sync', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  assert(source.indexOf("inputBox.on('blur'") !== -1,
+    'Should have inputBox blur handler');
+  // Blur handler should set inputFocused = false
+  const blurIdx = source.indexOf("inputBox.on('blur'");
+  const blurSection = source.substring(blurIdx, blurIdx + 100);
+  assert(blurSection.indexOf('inputFocused = false') !== -1,
+    'Blur handler should set inputFocused = false');
+});
+
+runTest('review-5: chatBox focus handler keeps inputFocused in sync', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  assert(source.indexOf("chatBox.on('focus'") !== -1,
+    'Should have chatBox focus handler');
+  const focusIdx = source.indexOf("chatBox.on('focus'");
+  const focusSection = source.substring(focusIdx, focusIdx + 100);
+  assert(focusSection.indexOf('inputFocused = false') !== -1,
+    'chatBox focus handler should set inputFocused = false');
+});
+
+// Copilot review round 2 tests
+
+runTest('review2-1: mouseEnabled declared before formatStatusBar is called', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  // mouseEnabled must be declared before the first formatStatusBar call
+  const mouseDecl = source.indexOf('let mouseEnabled');
+  const firstFormatCall = source.indexOf('formatStatusBar(agentName');
+  assert(mouseDecl !== -1, 'mouseEnabled should be declared');
+  assert(firstFormatCall !== -1, 'formatStatusBar should be called');
+  assert(mouseDecl < firstFormatCall,
+    'mouseEnabled (' + mouseDecl + ') must be declared before first formatStatusBar call (' + firstFormatCall + ')');
+});
+
+runTest('review2-2: finalizeStreaming guards on rawIdx to avoid corrupting interleaved output', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  const finalizeSection = source.substring(source.indexOf('function finalizeStreaming'));
+  // Should check if rawIdx was found
+  assert(finalizeSection.indexOf('rawIdx !== -1') !== -1,
+    'Should check if rawIdx was found');
+  // When rawIdx === -1, should skip finalization (not append duplicate)
+  assert(finalizeSection.indexOf('Skip') !== -1 || finalizeSection.indexOf('skip') !== -1,
+    'Should skip finalization when raw text not found');
+});
+
+runTest('review2-3: Enter key handler comment is accurate (no submit reference)', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  // Find the line before the enter key binding
+  const enterIdx = source.indexOf("inputBox.key('enter'");
+  const contextBefore = source.substring(Math.max(0, enterIdx - 100), enterIdx);
+  assert(contextBefore.indexOf('submit') === -1,
+    'Comment before Enter handler should not misleadingly reference submit event');
+});
+
+runTest('review2-4: test section comment does not reference wrong PR number', () => {
+  const testSource = fs.readFileSync(__filename, 'utf8');
+  // Test file should not reference "PR #247" in section headers
+  const sectionHeaders = testSource.match(/^\/\/ .+$/gm) || [];
+  for (let i = 0; i < sectionHeaders.length; i++) {
+    assert(sectionHeaders[i].indexOf('PR #247') === -1,
+      'Test section header should not reference PR #247: ' + sectionHeaders[i]);
+  }
+});
+
+// Copilot review round 3 tests
+
+runTest('review3-1: finalizeStreaming skips on rawIdx=-1 (no dead append code)', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  const finalizeSection = source.substring(source.indexOf('function finalizeStreaming'));
+  // Should NOT have the old pattern: rawIdx = headerEndIdx (which made rawIdx !== -1 always true)
+  assert(finalizeSection.indexOf('rawIdx = headerEndIdx') === -1,
+    'Should not reassign rawIdx to headerEndIdx (makes append fallback dead code)');
+  // The rawIdx !== -1 check should be reachable
+  assert(finalizeSection.indexOf('rawIdx !== -1') !== -1,
+    'Should still check rawIdx !== -1');
+  // Should NOT have append fallback (skip instead to avoid duplicates)
+  assert(finalizeSection.indexOf('needsSpaceAppend') === -1,
+    'Should not have append fallback (skip finalization when raw text not found)');
+});
+
+runTest('review3-2: appendNewline renamed from insertNewlineAtCursor', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  // Should use appendNewline (accurate name)
+  assert(source.indexOf('function appendNewline') !== -1,
+    'Should have appendNewline function');
+  // Should NOT have the old misleading name
+  assert(source.indexOf('insertNewlineAtCursor') === -1,
+    'Should not use old insertNewlineAtCursor name');
+  // Function comment should mention appending (not cursor insertion)
+  const funcIdx = source.indexOf('function appendNewline');
+  const commentBefore = source.substring(Math.max(0, funcIdx - 200), funcIdx);
+  assert(commentBefore.indexOf('Append') !== -1 || commentBefore.indexOf('append') !== -1,
+    'Comment should describe append semantics');
+});
+
+runTest('review3-3: no unused variables in test file', () => {
+  const testSource = fs.readFileSync(__filename, 'utf8');
+  // The oldPattern variable was flagged as unused — search for declaration pattern
+  const unusedVar = 'var old' + 'Pattern';
+  assert(testSource.indexOf(unusedVar) === -1,
+    'Should not have unused oldPattern variable');
+});
+
+runTest('review3-4: status bar refreshed after default mouse mode set', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  // After mouseEnabled = false (default), should call updateStatusBar
+  const defaultSection = source.substring(source.indexOf('// Default to text selection'));
+  const mouseSetIdx = defaultSection.indexOf('mouseEnabled = false');
+  const updateIdx = defaultSection.indexOf('updateStatusBar()');
+  assert(mouseSetIdx !== -1, 'Should set mouseEnabled = false');
+  assert(updateIdx !== -1, 'Should call updateStatusBar after default mouse mode');
+  assert(updateIdx > mouseSetIdx, 'updateStatusBar should come after mouseEnabled = false');
+});
+
+// Copilot review round 4 tests
+
+runTest('review4-1: Enter on empty input clears accumulated newlines', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  const enterIdx = source.indexOf("inputBox.key('enter'");
+  const enterSection = source.substring(enterIdx, enterIdx + 600);
+  // Should handle empty/whitespace-only input
+  assert(enterSection.indexOf('!value.trim()') !== -1,
+    'Enter handler should check for empty/whitespace input');
+  // Should clear and return early for empty input
+  const emptyBranchIdx = enterSection.indexOf('!value.trim()');
+  const clearInBranch = enterSection.indexOf('clearValue()', emptyBranchIdx);
+  assert(clearInBranch !== -1, 'Should call clearValue in empty input branch');
+  const returnInBranch = enterSection.indexOf('return', emptyBranchIdx);
+  assert(returnInBranch !== -1 && returnInBranch < enterSection.indexOf('sendToInbox'),
+    'Should return early for empty input before sendToInbox');
+});
+
+runTest('review4-2: finalizeStreaming uses conditional space (no double-spacing)', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  const finalizeSection = source.substring(source.indexOf('function finalizeStreaming'));
+  // Should check for trailing whitespace before adding space
+  assert(finalizeSection.indexOf('needsSpaceBefore') !== -1,
+    'Should conditionally add space before rendered markdown');
+  assert(finalizeSection.indexOf('/\\s$/') !== -1,
+    'Should use regex to check for trailing whitespace');
+  // Should NOT have unconditional hard-coded space
+  assert(finalizeSection.indexOf("before + ' ' + rendered") === -1,
+    'Should not have unconditional space before rendered content');
+});
+
+// Copilot review round 5 tests
+
+runTest('review5-1: help text includes Ctrl+C detach shortcut', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  assert(source.indexOf('Ctrl+C: detach') !== -1,
+    'Help text should mention Ctrl+C: detach');
+});
+
+// Copilot review round 6 tests
+
+runTest('review6-1: assistant history messages are capped for responsiveness', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  const histSection = source.substring(source.indexOf('function fetchHistory'));
+  // Should cap assistant content length before rendering
+  assert(histSection.indexOf('cappedContent') !== -1,
+    'Should cap assistant content before markdown rendering');
+  assert(histSection.indexOf('truncated') !== -1,
+    'Should add truncation marker for long assistant messages');
+  // Cap should be higher than user messages (500) but still bounded
+  assert(histSection.indexOf('5000') !== -1,
+    'Assistant cap should be 5000 chars (higher than user 500 but still bounded)');
+});
+
+// Copilot review round 7 tests
+
+runTest('review7-1: message.updated gates streaming start on finish===null', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  const updatedSection = source.substring(source.indexOf("eventName === 'message.updated'"));
+  // Should check finish to avoid double Agent: header
+  assert(updatedSection.indexOf('!finish') !== -1,
+    'message.updated handler should gate on finish being null/falsy');
+  assert(updatedSection.indexOf('!streamingActive') !== -1,
+    'message.updated handler should check streamingActive to prevent re-start');
+  // Should finalize on completion (finish set)
+  assert(updatedSection.indexOf('finish && streamingActive') !== -1,
+    'message.updated should finalize when finish is set');
+});
+
+runTest('review7-2: streamingRawText is capped during streaming', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  const deltaSection = source.substring(source.indexOf("eventName === 'message.part.delta'"));
+  // Should cap streaming text accumulation
+  assert(deltaSection.indexOf('STREAM_CAP') !== -1,
+    'Should define STREAM_CAP for streaming text limit');
+  assert(deltaSection.indexOf('streamingRawText.length >= STREAM_CAP') !== -1,
+    'Should check streamingRawText length against cap for early return');
+  // Legacy text.delta should also be capped
+  const legacySection = source.substring(source.indexOf("eventName === 'text.delta'"));
+  assert(legacySection.indexOf('LEGACY_STREAM_CAP') !== -1,
+    'Legacy text.delta should also cap streaming text');
+});
+
+runTest('review7-3: unused source var removed from review2-4 test', () => {
+  const testSource = fs.readFileSync(__filename, 'utf8');
+  // The review2-4 test should not read attached.js (only reads __filename)
+  const review24Idx = testSource.indexOf("review2-4:");
+  const nextRunTestIdx = testSource.indexOf('runTest(', review24Idx + 10);
+  const review24Section = testSource.substring(review24Idx, nextRunTestIdx);
+  assert(review24Section.indexOf("attached.js") === -1,
+    'review2-4 test should not read attached.js (unused var was removed)');
+});
+
+runTest('review7-4: streaming uses part.delta for text extraction', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  // The extractText function should handle part.delta shape
+  assert(source.indexOf('part.delta') !== -1 || source.indexOf("'message.part.delta'") !== -1,
+    'Should handle message.part.delta event for streaming text');
+  // extractText should look at parsed data for text content
+  const extractSection = source.substring(source.indexOf('function extractText'));
+  assert(extractSection.indexOf('parsed') !== -1,
+    'extractText should examine parsed event data');
+});
+
+runTest('review7-5: adjustInputHeight derives width from element or screen', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  const adjustSection = source.substring(source.indexOf('function adjustInputHeight'));
+  // Should prefer inputBox.iwidth (computed inner width from blessed)
+  assert(adjustSection.indexOf('inputBox.iwidth') !== -1,
+    'Should check inputBox.iwidth for computed inner width');
+  // Should still fall back to screen.width when iwidth is not available
+  assert(adjustSection.indexOf('screen.width') !== -1,
+    'Should fall back to screen.width when inputBox.iwidth is not available');
+});
+
+// Copilot review round 8 tests
+
+runTest('review8-1: finalizeStreaming resets state even when no text accumulated', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  const finalizeSection = source.substring(source.indexOf('function finalizeStreaming'),
+    source.indexOf('function finalizeStreaming') + 800);
+  // Should reset streamingActive before the early return for no text
+  const activeResetIdx = finalizeSection.indexOf('streamingActive = false');
+  const hadTextIdx = finalizeSection.indexOf('if (!hadText) return');
+  assert(activeResetIdx !== -1, 'Should reset streamingActive');
+  assert(hadTextIdx !== -1, 'Should have early return for no text');
+  assert(activeResetIdx < hadTextIdx,
+    'streamingActive reset should come before the no-text early return');
+  // Should save rawText/contentStart before clearing
+  assert(finalizeSection.indexOf('const rawText = streamingRawText') !== -1
+    || finalizeSection.indexOf('let rawText = streamingRawText') !== -1
+    || finalizeSection.indexOf('var rawText = streamingRawText') !== -1,
+  'Should save streamingRawText to local var before clearing');
+  assert(finalizeSection.indexOf('const contentStart = streamingContentStart') !== -1
+    || finalizeSection.indexOf('let contentStart = streamingContentStart') !== -1
+    || finalizeSection.indexOf('var contentStart = streamingContentStart') !== -1,
+  'Should save streamingContentStart to local var before clearing');
+});
+
+runTest('review8-2: help line width is dynamic (not fixed)', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  // helpLine should not have a fixed numeric width
+  const helpSection = source.substring(source.indexOf('var helpLine'));
+  const widthMatch = helpSection.match(/width:\s*(\S+)/);
+  assert(widthMatch, 'helpLine should have a width property');
+  // Should be dynamic (shrink, percentage, or computed) not a fixed number
+  const widthVal = widthMatch[1].replace(/[,}]/, '');
+  assert(isNaN(Number(widthVal)), 'helpLine width should not be a fixed number, got: ' + widthVal);
+});
+
+// Copilot review round 9 tests
+
+runTest('review9-1: streaming cap also limits chatBox display (not just rawText)', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  const deltaSection = source.substring(source.indexOf("eventName === 'message.part.delta'"),
+    source.indexOf("eventName === 'message.part.delta'") + 1500);
+  // Should return early when at cap (skip both accumulation AND display)
+  assert(deltaSection.indexOf('streamingRawText.length >= STREAM_CAP') !== -1,
+    'Should check >= cap for early return');
+  // Should cap what gets displayed too
+  assert(deltaSection.indexOf('cappedText') !== -1,
+    'Should use cappedText variable for both accumulation and display');
+  // Should show truncation marker when cap is hit
+  assert(deltaSection.indexOf('streaming truncated') !== -1,
+    'Should show truncation marker when cap is reached');
+});
+
+runTest('review9-2: legacy streaming cap also limits chatBox display', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  const legacySection = source.substring(source.indexOf("eventName === 'text.delta'"),
+    source.indexOf("eventName === 'text.delta'") + 600);
+  // Should return early when at cap
+  assert(legacySection.indexOf('streamingRawText.length >= LEGACY_STREAM_CAP') !== -1,
+    'Legacy path should check >= cap for early return');
+  // Should use capped text for display
+  assert(legacySection.indexOf('legacyCapped') !== -1,
+    'Legacy path should use capped text for display');
+});
+
+runTest('review9-3: bracketed paste handling enabled', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  // Should enable bracketed paste mode
+  assert(source.indexOf('\\x1b[?2004h') !== -1,
+    'Should enable bracketed paste mode with escape sequence');
+  // Should detect paste start/end markers
+  assert(source.indexOf('\\x1b[200~') !== -1,
+    'Should detect paste start marker');
+  assert(source.indexOf('\\x1b[201~') !== -1,
+    'Should detect paste end marker');
+  // Should buffer pasted text
+  assert(source.indexOf('pasteBuffer') !== -1,
+    'Should buffer pasted text');
+  assert(source.indexOf('isPasting') !== -1,
+    'Should track paste state');
+  // Should disable bracketed paste on detach
+  assert(source.indexOf('\\x1b[?2004l') !== -1,
+    'Should disable bracketed paste mode on detach');
+});
+
+// Copilot review round 10 tests
+
+runTest('review10-1: rapid-input fallback uses burst count (not single timing)', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  // Should track burst count, not just single timestamp
+  assert(source.indexOf('rapidBurstCount') !== -1,
+    'Should track rapidBurstCount for multi-char burst detection');
+  assert(source.indexOf('RAPID_BURST_MIN') !== -1,
+    'Should define RAPID_BURST_MIN constant');
+  // Should require multiple rapid events before suppressing Enter
+  assert(source.indexOf('rapidBurstCount >= RAPID_BURST_MIN') !== -1,
+    'Should compare burst count against minimum threshold');
+});
+
+runTest('review10-2: finalizeStreaming skips when raw segment not found (no duplicate)', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  const finalizeSection = source.substring(source.indexOf('function finalizeStreaming'));
+  // Should NOT append rendered content when rawIdx === -1
+  assert(finalizeSection.indexOf('needsSpaceAppend') === -1,
+    'Should not have append-with-space fallback (would duplicate content)');
+  // The else branch should skip finalization
+  assert(finalizeSection.indexOf('Skip') !== -1 || finalizeSection.indexOf('skip') !== -1,
+    'Should skip finalization when raw text not found');
+});
+
+runTest('review10-3: adjustInputHeight called on keypress (throttled)', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  // Should listen for keypress events on inputBox
+  assert(source.indexOf("inputBox.on('keypress'") !== -1,
+    'Should bind keypress handler on inputBox');
+  // Should throttle the calls
+  assert(source.indexOf('adjustThrottleTimer') !== -1,
+    'Should use a throttle timer for keypress-triggered adjustInputHeight');
+  // Should call adjustInputHeight within the throttled handler
+  const keypressSection = source.substring(source.indexOf("inputBox.on('keypress'"),
+    source.indexOf("inputBox.on('keypress'") + 300);
+  assert(keypressSection.indexOf('adjustInputHeight()') !== -1,
+    'Should call adjustInputHeight in throttled keypress handler');
+});
+
+// Copilot review round 11 tests
+
+runTest('review11-1: message.part.updated handled separately from part.delta', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  // Should NOT handle both in the same branch
+  assert(source.indexOf("eventName === 'message.part.delta' || eventName === 'message.part.updated'") === -1,
+    'part.delta and part.updated should NOT be in the same if-branch');
+  // Should have separate handler for part.updated
+  assert(source.indexOf("eventName === 'message.part.updated'") !== -1,
+    'Should have separate handler for message.part.updated');
+  // part.updated should skip when streaming already has content (to avoid duplicate)
+  const updatedSection = source.substring(source.indexOf("eventName === 'message.part.updated'"),
+    source.indexOf("eventName === 'message.part.updated'") + 500);
+  assert(updatedSection.indexOf('streamingRawText.length > 0') !== -1,
+    'part.updated should check if deltas were already received');
+});
+
+runTest('review11-2: extractText handles part.delta field', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  const extractSection = source.substring(source.indexOf('function extractText'),
+    source.indexOf('function extractText') + 300);
+  assert(extractSection.indexOf('part.delta') !== -1,
+    'extractText should check part.delta for delta events');
+});
+
+// Copilot review round 12 tests
+
+runTest('review12-1: sendToInbox preserves original text (trimEnd only)', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  const sendSection = source.substring(source.indexOf('function sendToInbox'),
+    source.indexOf('function sendToInbox') + 400);
+  // Should NOT use trim() to modify the text before sending
+  assert(sendSection.indexOf("text = text.trim()") === -1,
+    'sendToInbox should NOT use full trim() which strips leading indentation');
+  // Should use trimEnd (or trimRight) to preserve leading whitespace
+  assert(sendSection.indexOf('trimEnd()') !== -1,
+    'sendToInbox should use trimEnd to preserve leading whitespace in multi-line input');
+});
+
+runTest('review12-2: rapid-input uses burst count not single-event timing', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  const stdinSection = source.substring(source.indexOf('process.stdin.emit = function'),
+    source.indexOf('process.stdin.emit = function') + 1500);
+  assert(stdinSection.indexOf('rapidBurstCount >= RAPID_BURST_MIN') !== -1,
+    'Should use burst count threshold to suppress Enter');
+  // Should count chars, not just events
+  assert(stdinSection.indexOf('altStr.length') !== -1,
+    'Should count characters in multi-char chunks');
+  // Should reset counter when timing gap exceeds threshold
+  assert(stdinSection.indexOf('rapidBurstCount = 0') !== -1,
+    'Should reset burst counter on slow events');
+});
+
+// Copilot review round 13 tests
+
+runTest('review13-1: process-level cleanup for bracketed paste', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  assert(source.indexOf('emergencyCleanup') !== -1,
+    'Should define emergencyCleanup function');
+  assert(source.indexOf("process.on('exit'") !== -1,
+    'Should register exit handler');
+  assert(source.indexOf("process.on('SIGINT'") !== -1,
+    'Should register SIGINT handler');
+  assert(source.indexOf("process.on('SIGTERM'") !== -1,
+    'Should register SIGTERM handler');
+  assert(source.indexOf("process.on('uncaughtException'") !== -1,
+    'Should register uncaughtException handler');
+});
+
+runTest('review13-2: detach() guards against double-call', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  const detachSection = source.substring(source.indexOf('function detach()'),
+    source.indexOf('function detach()') + 300);
+  // Should check detaching flag and return early
+  assert(detachSection.indexOf('if (detaching) return') !== -1,
+    'detach() should return early if already detaching');
+});
+
+runTest('review13-3: multi-char stdin chunk with newlines handled as paste', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  const stdinSection = source.substring(source.indexOf('process.stdin.emit = function'),
+    source.indexOf('process.stdin.emit = function') + 2000);
+  // Should detect multi-char chunks with embedded newlines
+  assert(stdinSection.indexOf('altStr.length > 1') !== -1,
+    'Should detect multi-char data chunks');
+  // Should replace CR/LF with newlines
+  assert(stdinSection.indexOf("replace(/\\r\\n/g, '\\n')") !== -1,
+    'Should normalize CRLF to LF in pasted text');
+});
+
+// Copilot review round 14 tests
+
+runTest('review14-1: Ctrl+C test guards against indexOf=-1 false positive', () => {
+  const testSource = fs.readFileSync(__filename, 'utf8');
+  const ctrlCTestSection = testSource.substring(testSource.indexOf('245-3: Ctrl+C calls detach'));
+  // Should check ctrlCIndex !== -1 before using it
+  assert(ctrlCTestSection.indexOf('ctrlCIndex !== -1') !== -1,
+    'Test should assert index is not -1 before substring');
+});
+
+runTest('review14-2: rapidBurstCount reset after bracketed paste', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  // After paste start marker
+  const pasteStartIdx = source.indexOf("\\x1b[200~') !== -1");
+  assert(pasteStartIdx !== -1, 'Should find paste start marker detection code');
+  const pasteStartSection = source.substring(pasteStartIdx, pasteStartIdx + 400);
+  assert(pasteStartSection.indexOf('rapidBurstCount = 0') !== -1,
+    'Should reset rapidBurstCount when paste starts');
+  // After paste end — search from isPasting = false with bigger window
+  const pasteEndIdx = source.indexOf("isPasting = false");
+  assert(pasteEndIdx !== -1, 'Should find end-of-paste handler marker "isPasting = false"');
+  const pasteEndSection = source.substring(pasteEndIdx, pasteEndIdx + 500);
+  assert(pasteEndSection.indexOf('rapidBurstCount = 0') !== -1,
+    'Should reset rapidBurstCount when paste ends');
+});
+
+runTest('review14-3: uncaughtException handler destroys screen', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  const uncaughtSection = source.substring(source.indexOf('uncaughtException'),
+    source.indexOf('uncaughtException') + 300);
+  assert(uncaughtSection.indexOf('screen.destroy()') !== -1,
+    'uncaughtException handler should call screen.destroy()');
+});
+
+runTest('review14-4: adjustInputHeight uses inputBox.iwidth when available', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  const adjustSection = source.substring(source.indexOf('function adjustInputHeight'),
+    source.indexOf('function adjustInputHeight') + 500);
+  assert(adjustSection.indexOf('inputBox.iwidth') !== -1,
+    'Should use inputBox.iwidth for inner width calculation');
+});
+
+// Copilot review round 15 tests
+
+runTest('review15-1: Escape bound on both screen and inputBox', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  // Should have a shared handleEscape function
+  assert(source.indexOf('function handleEscape') !== -1,
+    'Should define handleEscape function');
+  // Should bind on screen
+  assert(source.indexOf("screen.key(['escape'], handleEscape)") !== -1,
+    'Escape should be bound on screen');
+  // Should bind on inputBox
+  assert(source.indexOf("inputBox.key(['escape'], handleEscape)") !== -1,
+    'Escape should also be bound on inputBox');
+});
+
+// Copilot review round 17 tests
+
+runTest('review17-1: rapidBurstCount reset after multi-char paste chunk', () => {
+  const source = fs.readFileSync(path.join(__dirname, '..', '..', 'tui', 'attached.js'), 'utf8');
+  // Find the multi-char chunk handling section (non-bracketed paste)
+  const multiCharIdx = source.indexOf('altStr.length > 1');
+  assert(multiCharIdx !== -1, 'Should find multi-char chunk detection code');
+  const multiCharSection = source.substring(multiCharIdx, multiCharIdx + 600);
+  // Should reset rapidBurstCount after handling the paste chunk
+  assert(multiCharSection.indexOf('rapidBurstCount = 0') !== -1,
+    'Should reset rapidBurstCount after multi-char paste chunk');
+  // Should also reset lastStdinDataTime
+  assert(multiCharSection.indexOf('lastStdinDataTime = 0') !== -1,
+    'Should reset lastStdinDataTime after multi-char paste chunk');
+});
+
+// ============================================================
+// Cleanup & Summary
+// ============================================================
+
+cleanup();
+
+console.log('\nTotal: ' + (passed + failed) + ', Passed: ' + passed + ', Failed: ' + failed);
+if (failed > 0) process.exitCode = 1;