@cnrai/pave 0.3.35 → 0.3.51

package/test/tool-output-formatter.test.js

@@ -1,486 +0,0 @@
-#!/usr/bin/env node
-/**
- * Tool Output Formatter Test Suite
- * Tests for formatToolOutput, extractToolCommand, and shell box formatting
- */
-
-const assert = require('assert');
-const { formatToolOutput, extractToolCommand, TOOL_COLORS, BOX } = require('../lib/tool-output-formatter');
-
-class TestRunner {
-  constructor() {
-    this.tests = [];
-    this.passed = 0;
-    this.failed = 0;
-  }
-
-  test(name, fn) {
-    this.tests.push({ name, fn });
-  }
-
-  async run() {
-    console.log('Testing tool-output-formatter.js\n');
-
-    for (const { name, fn } of this.tests) {
-      try {
-        await fn();
-        console.log(`  [PASS] ${name}`);
-        this.passed++;
-      } catch (error) {
-        console.log(`  [FAIL] ${name}`);
-        console.log(`         ${error.message}`);
-        this.failed++;
-      }
-    }
-
-    console.log(`\nResults: ${this.passed} passed, ${this.failed} failed`);
-    return this.failed === 0;
-  }
-}
-
-const runner = new TestRunner();
-
-// ========================================
-// TOOL_COLORS tests
-// ========================================
-
-runner.test('TOOL_COLORS: has all required color codes', () => {
-  assert.ok(TOOL_COLORS.reset, 'reset color should exist');
-  assert.ok(TOOL_COLORS.bold, 'bold color should exist');
-  assert.ok(TOOL_COLORS.dim, 'dim color should exist');
-  assert.ok(TOOL_COLORS.red, 'red color should exist');
-  assert.ok(TOOL_COLORS.green, 'green color should exist');
-  assert.ok(TOOL_COLORS.yellow, 'yellow color should exist');
-  assert.ok(TOOL_COLORS.cyan, 'cyan color should exist');
-  assert.ok(TOOL_COLORS.gray, 'gray color should exist');
-});
-
-runner.test('TOOL_COLORS: all values are ANSI escape codes', () => {
-  for (const [name, value] of Object.entries(TOOL_COLORS)) {
-    assert.ok(value.startsWith('\x1b['), `${name} should be an ANSI escape code`);
-  }
-});
-
-// ========================================
-// BOX characters tests
-// ========================================
-
-runner.test('BOX: has all required box drawing characters', () => {
-  assert.ok(BOX.topLeft, 'topLeft should exist');
-  assert.ok(BOX.topRight, 'topRight should exist');
-  assert.ok(BOX.bottomLeft, 'bottomLeft should exist');
-  assert.ok(BOX.bottomRight, 'bottomRight should exist');
-  assert.ok(BOX.horizontal, 'horizontal should exist');
-  assert.ok(BOX.vertical, 'vertical should exist');
-});
-
-runner.test('BOX: uses correct Unicode box drawing characters', () => {
-  assert.strictEqual(BOX.topLeft, '╭', 'topLeft should be ╭');
-  assert.strictEqual(BOX.topRight, '╮', 'topRight should be ╮');
-  assert.strictEqual(BOX.bottomLeft, '╰', 'bottomLeft should be ╰');
-  assert.strictEqual(BOX.bottomRight, '╯', 'bottomRight should be ╯');
-  assert.strictEqual(BOX.horizontal, '─', 'horizontal should be ─');
-  assert.strictEqual(BOX.vertical, '│', 'vertical should be │');
-});
-
-// ========================================
-// extractToolCommand tests
-// ========================================
-
-runner.test('extractToolCommand: returns tool name when no input', () => {
-  assert.strictEqual(extractToolCommand('bash', null), 'bash');
-  assert.strictEqual(extractToolCommand('read', undefined), 'read');
-});
-
-runner.test('extractToolCommand: extracts bash command from object', () => {
-  const input = { command: 'ls -la' };
-  assert.strictEqual(extractToolCommand('bash', input), 'ls -la');
-});
-
-runner.test('extractToolCommand: extracts bash command from JSON string', () => {
-  const input = JSON.stringify({ command: 'git status' });
-  assert.strictEqual(extractToolCommand('bash', input), 'git status');
-});
-
-runner.test('extractToolCommand: formats read tool', () => {
-  const input = { filePath: '/path/to/file.js' };
-  assert.strictEqual(extractToolCommand('read', input), 'read /path/to/file.js');
-});
-
-runner.test('extractToolCommand: formats grep tool with pattern and path', () => {
-  const input = { pattern: 'TODO', path: 'src/' };
-  assert.strictEqual(extractToolCommand('grep', input), 'grep "TODO" src/');
-});
-
-runner.test('extractToolCommand: formats grep tool with pattern and include', () => {
-  const input = { pattern: 'function', include: '*.js' };
-  assert.strictEqual(extractToolCommand('grep', input), 'grep "function" *.js');
-});
-
-runner.test('extractToolCommand: formats glob tool', () => {
-  const input = { pattern: '**/*.ts' };
-  assert.strictEqual(extractToolCommand('glob', input), 'glob "**/*.ts"');
-});
-
-runner.test('extractToolCommand: formats edit tool', () => {
-  const input = { filePath: '/path/to/edit.js' };
-  assert.strictEqual(extractToolCommand('edit', input), 'edit /path/to/edit.js');
-});
-
-runner.test('extractToolCommand: formats write tool', () => {
-  const input = { filePath: '/path/to/new-file.js' };
-  assert.strictEqual(extractToolCommand('write', input), 'write /path/to/new-file.js');
-});
-
-runner.test('extractToolCommand: formats task tool', () => {
-  const input = { description: 'Find all React components' };
-  assert.strictEqual(extractToolCommand('task', input), 'task "Find all React components"');
-});
-
-runner.test('extractToolCommand: formats webfetch tool', () => {
-  const input = { url: 'https://example.com/api' };
-  assert.strictEqual(extractToolCommand('webfetch', input), 'fetch https://example.com/api');
-});
-
-runner.test('extractToolCommand: returns tool name for invalid JSON', () => {
-  assert.strictEqual(extractToolCommand('bash', 'not valid json'), 'bash');
-});
-
-runner.test('extractToolCommand: returns tool name for unknown tool', () => {
-  const input = { someField: 'someValue' };
-  assert.strictEqual(extractToolCommand('unknown', input), 'unknown');
-});
-
-// ========================================
-// formatToolOutput basic tests
-// ========================================
-
-runner.test('formatToolOutput: returns string for basic invocation', () => {
-  const result = formatToolOutput('bash', 'completed', null, 'Hello', 100);
-  assert.strictEqual(typeof result, 'string', 'should return a string');
-  assert.ok(result.length > 0, 'should not be empty');
-});
-
-runner.test('formatToolOutput: includes tool name in header', () => {
-  const result = formatToolOutput('read', 'completed', null, 'file content', 50);
-  assert.ok(result.includes('read'), 'should include tool name');
-});
-
-runner.test('formatToolOutput: includes duration when provided', () => {
-  const result = formatToolOutput('bash', 'completed', null, 'output', 123);
-  assert.ok(result.includes('123ms'), 'should include duration');
-});
-
-runner.test('formatToolOutput: handles missing duration', () => {
-  const result = formatToolOutput('bash', 'completed', null, 'output');
-  assert.strictEqual(typeof result, 'string', 'should return a string');
-  assert.ok(!result.includes('undefinedms'), 'should not include undefined duration');
-  assert.ok(!result.includes('ms)'), 'should not include ms when no duration');
-});
-
-// ========================================
-// Shell box structure tests
-// ========================================
-
-runner.test('formatToolOutput: uses box drawing characters', () => {
-  const result = formatToolOutput('bash', 'completed', null, 'output', 100);
-  assert.ok(result.includes('╭'), 'should include top-left corner');
-  assert.ok(result.includes('╮'), 'should include top-right corner');
-  assert.ok(result.includes('╰'), 'should include bottom-left corner');
-  assert.ok(result.includes('╯'), 'should include bottom-right corner');
-  // Note: left border vertical lines removed per issue #47
-  assert.ok(result.includes('─'), 'should include horizontal line');
-});
-
-runner.test('formatToolOutput: command line directly follows top border (no empty line gap)', () => {
-  const input = { command: 'echo hello' };
-  const result = formatToolOutput('bash', 'completed', input, 'output', 100);
-  const lines = result.split('\n');
-  // Find the top border line (starts with colored ╭)
-  const topBorderIndex = lines.findIndex((l) => l.includes('╭') && l.includes('╮'));
-  assert.ok(topBorderIndex >= 0, 'should have top border line');
-  // The line immediately following the top border should be the command line, not empty
-  // Strip ANSI codes for a clean assertion
-  const lineAfterTopBorder = lines[topBorderIndex + 1].replace(/\x1b\[[0-9;]*m/g, '').trim();
-  assert.ok(lineAfterTopBorder.startsWith('$'), 'line after top border should start with $ prompt, got: "' + lineAfterTopBorder + '"');
-});
-
-runner.test('formatToolOutput: trims trailing empty lines from output', () => {
-  const input = { command: 'echo hello' };
-  // Output ending with multiple newlines should not produce blank lines before bottom border
-  const result = formatToolOutput('bash', 'completed', input, 'hello\n\n\n', 100);
-  const lines = result.split('\n');
-  // Find the bottom border line
-  const bottomBorderIndex = lines.findIndex((l) => l.includes('╰') && l.includes('╯'));
-  assert.ok(bottomBorderIndex >= 0, 'should have bottom border line');
-  // The line immediately before the bottom border should NOT be empty
-  const lineBeforeBottom = lines[bottomBorderIndex - 1].replace(/\x1b\[[0-9;]*m/g, '').trim();
-  assert.ok(lineBeforeBottom !== '', 'line before bottom border should not be empty (trailing lines trimmed)');
-});
-
-runner.test('formatToolOutput: trims trailing empty lines in error output', () => {
-  const input = { command: 'fail' };
-  // Error output with trailing newlines should also trim (ANSI color codes should not prevent trimming)
-  const result = formatToolOutput('bash', 'error', input, 'error occurred\n\n', 100);
-  const lines = result.split('\n');
-  const bottomBorderIndex = lines.findIndex((l) => l.includes('╰') && l.includes('╯'));
-  assert.ok(bottomBorderIndex >= 0, 'should have bottom border line');
-  const lineBeforeBottom = lines[bottomBorderIndex - 1].replace(/\x1b\[[0-9;]*m/g, '').trim();
-  assert.ok(lineBeforeBottom !== '', 'line before bottom border in error output should not be empty');
-});
-
-runner.test('formatToolOutput: shows $ prompt for command', () => {
-  const input = { command: 'ls -la' };
-  const result = formatToolOutput('bash', 'completed', input, 'output', 100);
-  // $ is wrapped in color codes, so check for $ and command separately
-  assert.ok(result.includes('$'), 'should include $ prompt');
-  assert.ok(result.includes('ls -la'), 'should include command');
-});
-
-runner.test('formatToolOutput: shows extracted command for non-bash tools', () => {
-  const input = { filePath: '/path/to/file.js' };
-  const result = formatToolOutput('read', 'completed', input, 'file contents', 50);
-  // $ is wrapped in color codes, so check for $ and command separately
-  assert.ok(result.includes('$'), 'should include $ prompt');
-  assert.ok(result.includes('read /path/to/file.js'), 'should include formatted read command');
-});
-
-runner.test('formatToolOutput: includes output lines without vertical bar prefix', () => {
-  const result = formatToolOutput('bash', 'completed', null, 'line1\nline2', 100);
-  // Lines should NOT be prefixed with │ (per issue #47 - remove left border)
-  const lines = result.split('\n');
-  const outputLines = lines.filter((l) => l.includes('line1') || l.includes('line2'));
-  assert.strictEqual(outputLines.length, 2, 'should have 2 output lines');
-  // Verify that each output line doesn't start with vertical bar prefix
-  outputLines.forEach((line) => {
-    assert.ok(!line.startsWith('│'), `output line "${line}" should not have vertical bar prefix`);
-  });
-});
-
-// ========================================
-// Status icon tests
-// ========================================
-
-runner.test('formatToolOutput: shows checkmark for completed status', () => {
-  const result = formatToolOutput('bash', 'completed', null, 'output', 100);
-  assert.ok(result.includes('✓'), 'should include checkmark');
-});
-
-runner.test('formatToolOutput: shows X for error status', () => {
-  const result = formatToolOutput('bash', 'error', null, 'error output', 100);
-  assert.ok(result.includes('✗'), 'should include X mark');
-});
-
-runner.test('formatToolOutput: shows spinner for running status', () => {
-  const result = formatToolOutput('bash', 'running', null, '', 0);
-  assert.ok(result.includes('⟳'), 'should include spinner');
-});
-
-runner.test('formatToolOutput: shows cursor for running status', () => {
-  const result = formatToolOutput('bash', 'running', { command: 'ls' }, '', 0);
-  assert.ok(result.includes('█'), 'should include block cursor for running state');
-});
-
-runner.test('formatToolOutput: no cursor for completed status', () => {
-  const result = formatToolOutput('bash', 'completed', { command: 'ls' }, 'output', 100);
-  assert.ok(!result.includes('█'), 'should not include block cursor for completed state');
-});
-
-// ========================================
-// Output handling tests (NO TRUNCATION)
-// ========================================
-
-runner.test('formatToolOutput: does not truncate long output', () => {
-  const longOutput = 'x'.repeat(10000);
-  const result = formatToolOutput('bash', 'completed', null, longOutput, 100);
-  assert.ok(result.includes(longOutput), 'should include full long output without truncation');
-});
-
-runner.test('formatToolOutput: includes full output for very long strings', () => {
-  const veryLongOutput = 'line\n'.repeat(500);
-  const result = formatToolOutput('bash', 'completed', null, veryLongOutput, 100);
-  const matches = result.match(/line/g) || [];
-  assert.strictEqual(matches.length, 500, 'should include all 500 lines');
-});
-
-// ========================================
-// Error handling tests
-// ========================================
-
-runner.test('formatToolOutput: error output is colored red', () => {
-  const result = formatToolOutput('bash', 'error', null, 'Something went wrong', 100);
-  assert.ok(result.includes(TOOL_COLORS.red), 'should include red color for error output');
-});
-
-runner.test('formatToolOutput: error status uses red header color', () => {
-  const result = formatToolOutput('bash', 'error', null, 'error', 100);
-  // The X mark should be red
-  assert.ok(result.includes(TOOL_COLORS.red + '✗'), 'should have red X mark');
-});
-
-// ========================================
-// noColor option tests
-// ========================================
-
-runner.test('formatToolOutput: includes colors by default', () => {
-  const result = formatToolOutput('bash', 'completed', null, 'output', 100);
-  assert.ok(result.includes('\x1b['), 'should include ANSI escape codes by default');
-});
-
-runner.test('formatToolOutput: removes colors when noColor is true', () => {
-  const result = formatToolOutput('bash', 'completed', null, 'output', 100, { noColor: true });
-  assert.ok(!result.includes('\x1b['), 'should not include ANSI escape codes when noColor is true');
-});
-
-runner.test('formatToolOutput: still shows checkmark without colors', () => {
-  const result = formatToolOutput('bash', 'completed', null, 'output', 100, { noColor: true });
-  assert.ok(result.includes('✓'), 'should include checkmark even without colors');
-});
-
-runner.test('formatToolOutput: still uses box characters without colors', () => {
-  const result = formatToolOutput('bash', 'completed', null, 'output', 100, { noColor: true });
-  assert.ok(result.includes('╭'), 'should include box characters even without colors');
-  assert.ok(result.includes('╯'), 'should include box characters even without colors');
-});
-
-// ========================================
-// Width option tests
-// ========================================
-
-runner.test('formatToolOutput: respects custom width option', () => {
-  const result = formatToolOutput('bash', 'completed', null, 'output', 100, { width: 40 });
-  // The bottom border should be shorter
-  const bottomBorderMatch = result.match(/╰─+╯/);
-  assert.ok(bottomBorderMatch, 'should have bottom border');
-  // Width of 40 means 39 dashes in bottom border
-  assert.ok(bottomBorderMatch[0].length < 50, 'should have shorter border with smaller width');
-});
-
-// ========================================
-// Edge cases
-// ========================================
-
-runner.test('formatToolOutput: handles empty output', () => {
-  const result = formatToolOutput('bash', 'completed', null, '', 100);
-  assert.strictEqual(typeof result, 'string', 'should return a string');
-  // Should still have box structure
-  assert.ok(result.includes('╭'), 'should include top border');
-  assert.ok(result.includes('╰'), 'should include bottom border');
-});
-
-runner.test('formatToolOutput: handles null output', () => {
-  const result = formatToolOutput('bash', 'completed', null, null, 100);
-  assert.strictEqual(typeof result, 'string', 'should return a string');
-});
-
-runner.test('formatToolOutput: handles undefined output', () => {
-  const result = formatToolOutput('bash', 'completed', null, undefined, 100);
-  assert.strictEqual(typeof result, 'string', 'should return a string');
-});
-
-runner.test('formatToolOutput: handles multiline output', () => {
-  const output = 'line1\nline2\nline3';
-  const result = formatToolOutput('bash', 'completed', null, output, 100);
-  assert.ok(result.includes('line1'), 'should include line1');
-  assert.ok(result.includes('line2'), 'should include line2');
-  assert.ok(result.includes('line3'), 'should include line3');
-});
-
-// ========================================
-// Color scheme tests (matching TUI)
-// ========================================
-
-runner.test('formatToolOutput: uses green color for $ prompt', () => {
-  const input = { command: 'ls' };
-  const result = formatToolOutput('bash', 'completed', input, 'output', 100);
-  // $ prompt should be green
-  assert.ok(result.includes(TOOL_COLORS.green + '$'), 'should use green color for $ prompt');
-});
-
-runner.test('formatToolOutput: uses gray color for box borders', () => {
-  const result = formatToolOutput('bash', 'completed', null, 'output', 100);
-  // Box borders should be gray
-  assert.ok(result.includes(TOOL_COLORS.gray + '╭'), 'should use gray for top border');
-  assert.ok(result.includes(TOOL_COLORS.gray + '╰'), 'should use gray for bottom border');
-});
-
-runner.test('formatToolOutput: uses cyan header color for completed status', () => {
-  const result = formatToolOutput('bash', 'completed', null, 'output', 100);
-  // Tool name should have cyan color for completed
-  assert.ok(result.includes(TOOL_COLORS.cyan), 'should use cyan for completed status header');
-});
-
-runner.test('formatToolOutput: uses red header color for error status', () => {
-  const result = formatToolOutput('bash', 'error', null, 'error', 100);
-  // Tool name should have red color for error (header area)
-  const headerPart = result.split('\n')[1]; // Second line is header
-  assert.ok(headerPart.includes(TOOL_COLORS.red), 'should use red color in error status header');
-});
-
-runner.test('formatToolOutput: uses yellow header color for running status', () => {
-  const result = formatToolOutput('bash', 'running', { command: 'ls' }, '', 0);
-  // Tool name should have yellow color for running
-  assert.ok(result.includes(TOOL_COLORS.yellow), 'should use yellow for running status header');
-});
-
-// ========================================
-// Default width tests (matching TUI)
-// ========================================
-
-runner.test('formatToolOutput: uses default width of 60 (matching TUI)', () => {
-  const result = formatToolOutput('bash', 'completed', null, 'output', 100);
-  // Bottom border should be 60 chars wide (╰ + 59 dashes + ╯)
-  const bottomBorderMatch = result.match(/╰─+╯/);
-  assert.ok(bottomBorderMatch, 'should have bottom border');
-  // Width should be approximately 60 (╰ + ─s + ╯ = 60)
-  assert.strictEqual(bottomBorderMatch[0].length, 60, 'default width should be 60');
-});
-
-// ========================================
-// JSON output handling
-// ========================================
-
-runner.test('formatToolOutput: stringifies object output', () => {
-  const output = { key: 'value', nested: { a: 1 } };
-  const result = formatToolOutput('bash', 'completed', null, output, 100);
-  assert.ok(result.includes('key'), 'should include JSON key');
-  assert.ok(result.includes('value'), 'should include JSON value');
-});
-
-// ========================================
-// Consistency with TUI format
-// ========================================
-
-runner.test('formatToolOutput: header format matches TUI (tool + status + duration)', () => {
-  const result = formatToolOutput('bash', 'completed', null, 'output', 100);
-  // Header should contain: ╭─ bash ✓ (100ms) ─...─╮
-  assert.ok(result.includes('bash'), 'should include tool name');
-  assert.ok(result.includes('✓'), 'should include status icon');
-  assert.ok(result.includes('100ms'), 'should include duration');
-});
-
-runner.test('formatToolOutput: command line format ($ command without left border)', () => {
-  const input = { command: 'npm test' };
-  const result = formatToolOutput('bash', 'completed', input, 'passed', 50);
-  // Should have $ prompt and command (no vertical bar per issue #47)
-  assert.ok(result.includes('$'), 'should have $ prompt');
-  assert.ok(result.includes('npm test'), 'should have command');
-});
-
-runner.test('formatToolOutput: output lines have no vertical bar prefix', () => {
-  const result = formatToolOutput('bash', 'completed', null, 'test output', 100);
-  const lines = result.split('\n');
-  const outputLine = lines.find((l) => l.includes('test output'));
-  assert.ok(outputLine, 'should have output line');
-  // Per issue #47: output lines should NOT have vertical bar prefix
-  assert.ok(!outputLine.startsWith('│'), 'output line should not have vertical bar prefix');
-});
-
-// Run tests if executed directly
-if (require.main === module) {
-  runner.run().then((success) => {
-    process.exit(success ? 0 : 1);
-  });
-}
-
-module.exports = { runner };

package/test/tool-output-gating.test.js

@@ -1,143 +0,0 @@
-/**
- * Tests for tool output gating based on showTools flag
- * Tool output is shown by default (showTools: true)
- * Use --no-tools to hide tool output (showTools: false)
- */
-
-describe('Tool Output Gating', () => {
-  let mockStdout;
-
-  beforeEach(() => {
-    mockStdout = jest.spyOn(process.stdout, 'write').mockImplementation();
-  });
-
-  afterEach(() => {
-    mockStdout.mockRestore();
-  });
-
-  test('should not show tool output when showTools is false (--no-tools)', () => {
-    const _toolPart = {
-      type: 'tool',
-      id: 'tool-123',
-      tool: 'bash',
-      state: {
-        status: 'completed',
-        input: '{"command": "ls"}',
-        output: 'file1.txt\\nfile2.txt',
-      },
-    };
-
-    // Simulate processing tool part without showTools
-    const showTools = false;
-    const json = false;
-    const _noStream = false;
-
-    // Tool output should NOT be displayed
-    if (showTools && !json) {
-      process.stdout.write('TOOL OUTPUT');
-    }
-
-    expect(mockStdout).not.toHaveBeenCalledWith(expect.stringContaining('TOOL OUTPUT'));
-  });
-
-  test('should show tool output when showTools is true (default)', () => {
-    const _toolPart = {
-      type: 'tool',
-      id: 'tool-123',
-      tool: 'bash',
-      state: {
-        status: 'completed',
-        input: '{"command": "ls"}',
-        output: 'file1.txt\\nfile2.txt',
-      },
-    };
-
-    // Simulate processing tool part with showTools
-    const showTools = true;
-    const json = false;
-    const _noStream = false;
-
-    if (showTools && !json) {
-      process.stdout.write('TOOL OUTPUT SHOWN');
-    }
-
-    expect(mockStdout).toHaveBeenCalledWith('TOOL OUTPUT SHOWN');
-  });
-
-  test('should buffer tool output in no-stream mode', () => {
-    let bufferedToolOutput = '';
-    const toolOutput = 'Buffered tool output\\n';
-
-    // Simulate --tools + --no-stream
-    const showTools = true;
-    const json = false;
-    const noStream = true;
-
-    if (showTools && !json) {
-      if (noStream) {
-        bufferedToolOutput += toolOutput;
-      } else {
-        process.stdout.write(toolOutput);
-      }
-    }
-
-    // Output should be buffered, not written immediately
-    expect(mockStdout).not.toHaveBeenCalledWith(toolOutput);
-    expect(bufferedToolOutput).toBe(toolOutput);
-  });
-});
-
-describe('JSON Schema Consistency', () => {
-  test('should always include tools array when showTools is true (default)', () => {
-    const showTools = true;
-    const toolOutputs = [];
-
-    const output = {
-      sessionId: 'test-session',
-      messageId: 'test-message',
-      response: 'Test response',
-    };
-
-    if (showTools) {
-      output.tools = Array.isArray(toolOutputs) ? toolOutputs : [];
-    }
-
-    expect(output).toHaveProperty('tools');
-    expect(Array.isArray(output.tools)).toBe(true);
-  });
-
-  test('should not include tools array when showTools is false (--no-tools)', () => {
-    const showTools = false;
-    const toolOutputs = [{ id: 'tool-1', name: 'bash' }];
-
-    const output = {
-      sessionId: 'test-session',
-      messageId: 'test-message',
-      response: 'Test response',
-    };
-
-    if (showTools) {
-      output.tools = Array.isArray(toolOutputs) ? toolOutputs : [];
-    }
-
-    expect(output).not.toHaveProperty('tools');
-  });
-
-  test('should include tools even when empty with showTools true (default)', () => {
-    const showTools = true;
-    const toolOutputs = []; // Empty but --tools was used
-
-    const output = {
-      sessionId: 'test-session',
-      messageId: 'test-message',
-      response: 'Test response',
-    };
-
-    if (showTools) {
-      output.tools = Array.isArray(toolOutputs) ? toolOutputs : [];
-    }
-
-    expect(output.tools).toEqual([]);
-    expect(Array.isArray(output.tools)).toBe(true);
-  });
-});