difit 4.0.7 → 5.0.1

package/dist/cli/index.test.js

@@ -1,8 +1,6 @@
 import { Command } from 'commander';
-import React from 'react';
 import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
 import { DiffMode } from '../types/watch.js';
-import { DEFAULT_DIFF_VIEW_MODE, normalizeDiffViewMode } from '../utils/diffMode.js';
 import pkg from '../../package.json' with { type: 'json' };
 // Mock all external dependencies
 vi.mock('simple-git');
@@ -128,9 +126,7 @@ describe('CLI index.ts', () => {
                 .option('--port <port>', 'port', parseInt)
                 .option('--host <host>', 'host', '')
                 .option('--no-open', 'no-open')
-                .option('--mode <mode>', 'mode', normalizeDiffViewMode, DEFAULT_DIFF_VIEW_MODE)
                 .option('--merge-base', 'merge-base')
-                .option('--tui', 'tui')
                 .option('--pr <url>', 'pr')
                 .action(async (commitish, _compareWith, options) => {
                 // Simulate the logic from index.ts
@@ -162,7 +158,6 @@ describe('CLI index.ts', () => {
                     preferredPort: options.port,
                     host: options.host,
                     openBrowser: options.open,
-                    mode: options.mode,
                 });
             });
             await program.parseAsync([...args], { from: 'user' });
@@ -171,7 +166,6 @@ describe('CLI index.ts', () => {
                 preferredPort: undefined,
                 host: '',
                 openBrowser: true,
-                mode: 'split',
             });
         });
     });
@@ -192,21 +186,6 @@ describe('CLI index.ts', () => {
                 args: ['--no-open'],
                 expectedOptions: { open: false },
             },
-            {
-                name: '--mode option',
-                args: ['--mode', 'unified'],
-                expectedOptions: { mode: 'unified' },
-            },
-            {
-                name: '--mode option (legacy inline)',
-                args: ['--mode', 'inline'],
-                expectedOptions: { mode: 'unified' },
-            },
-            {
-                name: '--mode option (legacy side-by-side)',
-                args: ['--mode', 'side-by-side'],
-                expectedOptions: { mode: 'split' },
-            },
             {
                 name: '--clean option',
                 args: ['--clean'],
@@ -236,8 +215,6 @@ describe('CLI index.ts', () => {
                 .option('--port <port>', 'port', parseInt)
                 .option('--host <host>', 'host', '')
                 .option('--no-open', 'no-open')
-                .option('--mode <mode>', 'mode', normalizeDiffViewMode, DEFAULT_DIFF_VIEW_MODE)
-                .option('--tui', 'tui')
                 .option('--pr <url>', 'pr')
                 .option('--merge-base', 'merge-base')
                 .option('--clean', 'start with a clean slate by clearing all existing comments')
@@ -253,7 +230,6 @@ describe('CLI index.ts', () => {
                     preferredPort: options.port,
                     host: options.host,
                     openBrowser: options.open,
-                    mode: options.mode,
                     clearComments: options.clean,
                     keepAlive: options.keepAlive,
                     contextLines: options.context,
@@ -267,7 +243,6 @@ describe('CLI index.ts', () => {
                 preferredPort: expectedOptions.port,
                 host: expectedOptions.host || '',
                 openBrowser: expectedOptions.open !== false,
-                mode: expectedOptions.mode || 'split',
                 clearComments: expectedOptions.clean,
                 keepAlive: expectedOptions.keepAlive,
                 contextLines: expectedOptions.context,
@@ -330,13 +305,11 @@ describe('CLI index.ts', () => {
                 .argument('[commit-ish]', 'commit-ish', 'HEAD')
                 .argument('[compare-with]', 'compare-with')
                 .option('--context <lines>', 'context', parseInt)
-                .option('--tui', 'tui')
                 .action(async (commitish, _compareWith, options) => {
                 const readFromStdin = shouldReadStdin({
                     commitish,
                     hasPositionalArgs: program.args.length > 0,
                     hasPrOption: false,
-                    hasTuiOption: Boolean(options.tui),
                 });
                 if (readFromStdin && options.context !== undefined) {
                     console.error('Error: --context option cannot be used with stdin diff');
@@ -385,13 +358,11 @@ describe('CLI index.ts', () => {
                 .argument('[commit-ish]', 'commit-ish', 'HEAD')
                 .argument('[compare-with]', 'compare-with')
                 .option('--merge-base', 'merge-base')
-                .option('--tui', 'tui')
                 .action(async (commitish, _compareWith, options) => {
                 const readFromStdin = shouldReadStdin({
                     commitish,
                     hasPositionalArgs: program.args.length > 0,
                     hasPrOption: false,
-                    hasTuiOption: Boolean(options.tui),
                 });
                 if (readFromStdin && options.mergeBase) {
                     console.error('Error: --merge-base option cannot be used with stdin diff');
@@ -487,8 +458,6 @@ describe('CLI index.ts', () => {
                 .option('--port <port>', 'port', parseInt)
                 .option('--host <host>', 'host', '')
                 .option('--no-open', 'no-open')
-                .option('--mode <mode>', 'mode', normalizeDiffViewMode, DEFAULT_DIFF_VIEW_MODE)
-                .option('--tui', 'tui')
                 .option('--pr <url>', 'pr')
                 .action(async (commitish, _compareWith, options) => {
                 if (commitish === 'working' || commitish === '.') {
@@ -501,7 +470,6 @@ describe('CLI index.ts', () => {
                     preferredPort: options.port,
                     host: options.host,
                     openBrowser: options.open,
-                    mode: options.mode,
                 });
             });
             await program.parseAsync(['working'], { from: 'user' });
@@ -517,8 +485,6 @@ describe('CLI index.ts', () => {
                 .option('--port <port>', 'port', parseInt)
                 .option('--host <host>', 'host', '')
                 .option('--no-open', 'no-open')
-                .option('--mode <mode>', 'mode', normalizeDiffViewMode, DEFAULT_DIFF_VIEW_MODE)
-                .option('--tui', 'tui')
                 .option('--pr <url>', 'pr')
                 .action(async (commitish, _compareWith, options) => {
                 if (commitish === 'working' || commitish === '.') {
@@ -530,7 +496,6 @@ describe('CLI index.ts', () => {
                     preferredPort: options.port,
                     host: options.host,
                     openBrowser: options.open,
-                    mode: options.mode,
                 });
             });
             await program.parseAsync(['HEAD'], { from: 'user' });
@@ -547,8 +512,6 @@ describe('CLI index.ts', () => {
                 .option('--port <port>', 'port', parseInt)
                 .option('--host <host>', 'host', '')
                 .option('--no-open', 'no-open')
-                .option('--mode <mode>', 'mode', normalizeDiffViewMode, DEFAULT_DIFF_VIEW_MODE)
-                .option('--tui', 'tui')
                 .option('--pr <url>', 'pr')
                 .option('--include-untracked', 'include untracked')
                 .action(async (commitish, _compareWith, options) => {
@@ -564,7 +527,6 @@ describe('CLI index.ts', () => {
                     preferredPort: options.port,
                     host: options.host,
                     openBrowser: options.open,
-                    mode: options.mode,
                 });
             });
             await program.parseAsync(['.', '--include-untracked'], { from: 'user' });
@@ -582,8 +544,6 @@ describe('CLI index.ts', () => {
                 .option('--port <port>', 'port', parseInt)
                 .option('--host <host>', 'host', '')
                 .option('--no-open', 'no-open')
-                .option('--mode <mode>', 'mode', normalizeDiffViewMode, DEFAULT_DIFF_VIEW_MODE)
-                .option('--tui', 'tui')
                 .option('--pr <url>', 'pr')
                 .option('--include-untracked', 'include untracked')
                 .action(async (commitish, _compareWith, options) => {
@@ -600,7 +560,6 @@ describe('CLI index.ts', () => {
                     preferredPort: options.port,
                     host: options.host,
                     openBrowser: options.open,
-                    mode: options.mode,
                 });
             });
             await program.parseAsync(['.'], { from: 'user' });
@@ -644,8 +603,6 @@ describe('CLI index.ts', () => {
                 .option('--port <port>', 'port', parseInt)
                 .option('--host <host>', 'host', '')
                 .option('--no-open', 'no-open')
-                .option('--mode <mode>', 'mode', normalizeDiffViewMode, DEFAULT_DIFF_VIEW_MODE)
-                .option('--tui', 'tui')
                 .option('--pr <url>', 'pr')
                 .action(async (commitish, _compareWith, options) => {
                 const manualCommentImports = actualParseCommentOptions(options.comment);
@@ -663,7 +620,6 @@ describe('CLI index.ts', () => {
                     preferredPort: options.port,
                     host: options.host,
                     openBrowser: options.open,
-                    mode: options.mode,
                     commentImports,
                 });
             });
@@ -680,7 +636,6 @@ describe('CLI index.ts', () => {
                 preferredPort: undefined,
                 host: '',
                 openBrowser: true,
-                mode: 'split',
                 commentImports: [
                     ...prCommentImports,
                     {
@@ -710,8 +665,6 @@ describe('CLI index.ts', () => {
                 .option('--port <port>', 'port', parseInt)
                 .option('--host <host>', 'host', '')
                 .option('--no-open', 'no-open')
-                .option('--mode <mode>', 'mode', normalizeDiffViewMode, DEFAULT_DIFF_VIEW_MODE)
-                .option('--tui', 'tui')
                 .option('--pr <url>', 'pr')
                 .action(async (commitish, _compareWith, options) => {
                 const manualCommentImports = actualParseCommentOptions(options.comment);
@@ -734,7 +687,6 @@ describe('CLI index.ts', () => {
                     preferredPort: options.port,
                     host: options.host,
                     openBrowser: options.open,
-                    mode: options.mode,
                     ...(commentImports.length > 0 ? { commentImports } : {}),
                 });
             });
@@ -745,7 +697,6 @@ describe('CLI index.ts', () => {
                 preferredPort: undefined,
                 host: '',
                 openBrowser: true,
-                mode: 'split',
             });
         });
         it('rejects PR option with positional arguments', async () => {
@@ -756,8 +707,6 @@ describe('CLI index.ts', () => {
                 .option('--port <port>', 'port', parseInt)
                 .option('--host <host>', 'host', '')
                 .option('--no-open', 'no-open')
-                .option('--mode <mode>', 'mode', normalizeDiffViewMode, DEFAULT_DIFF_VIEW_MODE)
-                .option('--tui', 'tui')
                 .option('--pr <url>', 'pr')
                 .action(async (commitish, _compareWith, options) => {
                 if (options.pr) {
@@ -773,35 +722,6 @@ describe('CLI index.ts', () => {
             expect(console.error).toHaveBeenCalledWith('Error: --pr option cannot be used with positional arguments');
             expect(process.exit).toHaveBeenCalledWith(1);
         });
-        it('rejects PR option with --tui', async () => {
-            const prUrl = 'https://github.com/owner/repo/pull/123';
-            const program = new Command();
-            program
-                .argument('[commit-ish]', 'commit-ish', 'HEAD')
-                .argument('[compare-with]', 'compare-with')
-                .option('--port <port>', 'port', parseInt)
-                .option('--host <host>', 'host', '')
-                .option('--no-open', 'no-open')
-                .option('--mode <mode>', 'mode', normalizeDiffViewMode, DEFAULT_DIFF_VIEW_MODE)
-                .option('--tui', 'tui')
-                .option('--pr <url>', 'pr')
-                .action(async (commitish, _compareWith, options) => {
-                if (options.pr) {
-                    if (commitish !== 'HEAD' || _compareWith) {
-                        console.error('Error: --pr option cannot be used with positional arguments');
-                        process.exit(1);
-                    }
-                    if (options.tui) {
-                        console.error('Error: --pr option cannot be used with --tui');
-                        process.exit(1);
-                    }
-                }
-            });
-            await program.parseAsync(['--pr', prUrl, '--tui'], { from: 'user' });
-            expect(console.error).toHaveBeenCalledWith('Error: --pr option cannot be used with --tui');
-            expect(process.exit).toHaveBeenCalledWith(1);
-            expect(mockStartServer).not.toHaveBeenCalled();
-        });
     });
     describe('--comment option', () => {
         it('passes parsed comment imports to startServer', async () => {
@@ -812,7 +732,6 @@ describe('CLI index.ts', () => {
                 .option('--port <port>', 'port', parseInt)
                 .option('--host <host>', 'host', '')
                 .option('--no-open', 'no-open')
-                .option('--mode <mode>', 'mode', normalizeDiffViewMode, DEFAULT_DIFF_VIEW_MODE)
                 .action(async (commitish, options) => {
                 const commentImports = actualParseCommentOptions(options.comment);
                 await startServer({
@@ -820,7 +739,6 @@ describe('CLI index.ts', () => {
                     preferredPort: options.port,
                     host: options.host,
                     openBrowser: options.open,
-                    mode: options.mode,
                     commentImports,
                 });
             });
@@ -833,7 +751,6 @@ describe('CLI index.ts', () => {
                 preferredPort: undefined,
                 host: '',
                 openBrowser: true,
-                mode: 'split',
                 commentImports: [
                     {
                         type: 'thread',
@@ -849,27 +766,6 @@ describe('CLI index.ts', () => {
                 ],
             });
         });
-        it('rejects --comment with --tui', async () => {
-            const program = new Command();
-            program
-                .argument('[commit-ish]', 'commit-ish', 'HEAD')
-                .option('--comment <json>', 'comment', (value, previous) => [...previous, value], [])
-                .option('--tui', 'tui')
-                .action(async (_commitish, options) => {
-                const commentImports = actualParseCommentOptions(options.comment);
-                if (options.tui && commentImports.length > 0) {
-                    console.error('Error: --comment option cannot be used with --tui');
-                    process.exit(1);
-                }
-            });
-            await program.parseAsync([
-                '--tui',
-                '--comment',
-                '{"type":"thread","filePath":"src/example.ts","position":{"side":"new","line":10},"body":"Imported comment"}',
-            ], { from: 'user' });
-            expect(console.error).toHaveBeenCalledWith('Error: --comment option cannot be used with --tui');
-            expect(process.exit).toHaveBeenCalledWith(1);
-        });
         it('reports invalid comment json before starting the server', async () => {
             const program = new Command();
             program
@@ -905,8 +801,6 @@ describe('CLI index.ts', () => {
                 .option('--port <port>', 'port', parseInt)
                 .option('--host <host>', 'host', '')
                 .option('--no-open', 'no-open')
-                .option('--mode <mode>', 'mode', normalizeDiffViewMode, DEFAULT_DIFF_VIEW_MODE)
-                .option('--tui', 'tui')
                 .option('--pr <url>', 'pr')
                 .option('--clean', 'start with a clean slate by clearing all existing comments')
                 .action(async (commitish, _compareWith, options) => {
@@ -915,7 +809,6 @@ describe('CLI index.ts', () => {
                     preferredPort: options.port,
                     host: options.host,
                     openBrowser: options.open,
-                    mode: options.mode,
                     clearComments: options.clean,
                 });
                 console.log(`\n🚀 difit server started on ${url}`);
@@ -944,8 +837,6 @@ describe('CLI index.ts', () => {
                 .option('--port <port>', 'port', parseInt)
                 .option('--host <host>', 'host', '')
                 .option('--no-open', 'no-open')
-                .option('--mode <mode>', 'mode', normalizeDiffViewMode, DEFAULT_DIFF_VIEW_MODE)
-                .option('--tui', 'tui')
                 .option('--pr <url>', 'pr')
                 .option('--clean', 'start with a clean slate by clearing all existing comments')
                 .action(async (commitish, _compareWith, options) => {
@@ -954,7 +845,6 @@ describe('CLI index.ts', () => {
                     preferredPort: options.port,
                     host: options.host,
                     openBrowser: options.open,
-                    mode: options.mode,
                     clearComments: options.clean,
                 });
                 console.log(`\n🚀 difit server started on ${url}`);
@@ -985,8 +875,6 @@ describe('CLI index.ts', () => {
                 .option('--port <port>', 'port', parseInt)
                 .option('--host <host>', 'host', '')
                 .option('--no-open', 'no-open')
-                .option('--mode <mode>', 'mode', normalizeDiffViewMode, DEFAULT_DIFF_VIEW_MODE)
-                .option('--tui', 'tui')
                 .option('--pr <url>', 'pr')
                 .option('--clean', 'start with a clean slate by clearing all existing comments')
                 .option('--keep-alive', 'keep server running even after browser disconnects')
@@ -996,7 +884,6 @@ describe('CLI index.ts', () => {
                     preferredPort: options.port,
                     host: options.host,
                     openBrowser: options.open,
-                    mode: options.mode,
                     clearComments: options.clean,
                     keepAlive: options.keepAlive,
                 });
@@ -1026,8 +913,6 @@ describe('CLI index.ts', () => {
                 .option('--port <port>', 'port', parseInt)
                 .option('--host <host>', 'host', '')
                 .option('--no-open', 'no-open')
-                .option('--mode <mode>', 'mode', normalizeDiffViewMode, DEFAULT_DIFF_VIEW_MODE)
-                .option('--tui', 'tui')
                 .option('--pr <url>', 'pr')
                 .option('--clean', 'start with a clean slate by clearing all existing comments')
                 .option('--keep-alive', 'keep server running even after browser disconnects')
@@ -1037,7 +922,6 @@ describe('CLI index.ts', () => {
                     preferredPort: options.port,
                     host: options.host,
                     openBrowser: options.open,
-                    mode: options.mode,
                     clearComments: options.clean,
                     keepAlive: options.keepAlive,
                 });
@@ -1069,8 +953,6 @@ describe('CLI index.ts', () => {
                 .option('--port <port>', 'port', parseInt)
                 .option('--host <host>', 'host', '')
                 .option('--no-open', 'no-open')
-                .option('--mode <mode>', 'mode', normalizeDiffViewMode, DEFAULT_DIFF_VIEW_MODE)
-                .option('--tui', 'tui')
                 .option('--pr <url>', 'pr')
                 .action(async (commitish, _compareWith, options) => {
                 const { url, isEmpty } = await startServer({
@@ -1078,7 +960,6 @@ describe('CLI index.ts', () => {
                     preferredPort: options.port,
                     host: options.host,
                     openBrowser: options.open,
-                    mode: options.mode,
                 });
                 console.log(`\n🚀 difit server started on ${url}`);
                 console.log(`📋 Reviewing: ${commitish}`);
@@ -1111,8 +992,6 @@ describe('CLI index.ts', () => {
                 .option('--port <port>', 'port', parseInt)
                 .option('--host <host>', 'host', '')
                 .option('--no-open', 'no-open')
-                .option('--mode <mode>', 'mode', normalizeDiffViewMode, DEFAULT_DIFF_VIEW_MODE)
-                .option('--tui', 'tui')
                 .option('--pr <url>', 'pr')
                 .action(async (commitish, _compareWith, options) => {
                 const { url, isEmpty } = await startServer({
@@ -1120,7 +999,6 @@ describe('CLI index.ts', () => {
                     preferredPort: options.port,
                     host: options.host,
                     openBrowser: options.open,
-                    mode: options.mode,
                 });
                 console.log(`\n🚀 difit server started on ${url}`);
                 console.log(`📋 Reviewing: ${commitish}`);
@@ -1134,249 +1012,6 @@ describe('CLI index.ts', () => {
             expect(console.log).toHaveBeenCalledWith('   Server is running at http://localhost:4966 if you want to check manually.\n');
         });
     });
-    describe('Server mode option handling', () => {
-        it('passes mode option to startServer', async () => {
-            mockFindUntrackedFiles.mockResolvedValue([]);
-            const program = new Command();
-            program
-                .argument('[commit-ish]', 'commit-ish', 'HEAD')
-                .argument('[compare-with]', 'compare-with')
-                .option('--port <port>', 'port', parseInt)
-                .option('--host <host>', 'host', '')
-                .option('--no-open', 'no-open')
-                .option('--mode <mode>', 'mode', normalizeDiffViewMode, DEFAULT_DIFF_VIEW_MODE)
-                .option('--tui', 'tui')
-                .option('--pr <url>', 'pr')
-                .action(async (commitish, _compareWith, options) => {
-                await startServer({
-                    selection: { targetCommitish: commitish, baseCommitish: commitish + '^' },
-                    preferredPort: options.port,
-                    host: options.host,
-                    openBrowser: options.open,
-                    mode: options.mode,
-                });
-            });
-            await program.parseAsync(['--mode', 'unified'], { from: 'user' });
-            expect(mockStartServer).toHaveBeenCalledWith(expect.objectContaining({
-                mode: 'unified',
-            }));
-        });
-        it('uses default mode when not specified', async () => {
-            mockFindUntrackedFiles.mockResolvedValue([]);
-            const program = new Command();
-            program
-                .argument('[commit-ish]', 'commit-ish', 'HEAD')
-                .argument('[compare-with]', 'compare-with')
-                .option('--port <port>', 'port', parseInt)
-                .option('--host <host>', 'host', '')
-                .option('--no-open', 'no-open')
-                .option('--mode <mode>', 'mode', normalizeDiffViewMode, DEFAULT_DIFF_VIEW_MODE)
-                .option('--tui', 'tui')
-                .option('--pr <url>', 'pr')
-                .action(async (commitish, _compareWith, options) => {
-                await startServer({
-                    selection: { targetCommitish: commitish, baseCommitish: commitish + '^' },
-                    preferredPort: options.port,
-                    host: options.host,
-                    openBrowser: options.open,
-                    mode: options.mode,
-                });
-            });
-            await program.parseAsync([], { from: 'user' });
-            expect(mockStartServer).toHaveBeenCalledWith(expect.objectContaining({
-                mode: 'split',
-            }));
-        });
-    });
-    describe('TUI mode', () => {
-        let mockRender;
-        let mockTuiApp;
-        const expectRenderedTuiProps = (props) => {
-            expect(mockRender).toHaveBeenCalledTimes(1);
-            expect(mockRender).toHaveBeenCalledWith({
-                component: mockTuiApp,
-                props,
-            });
-        };
-        beforeEach(() => {
-            mockRender = vi.fn();
-            mockTuiApp = vi.fn();
-            // Mock React.createElement for testing
-            vi.spyOn(React, 'createElement').mockImplementation((component, props) => ({ component, props }));
-            // Mock process.stdin.isTTY
-            Object.defineProperty(process.stdin, 'isTTY', {
-                value: true,
-                configurable: true,
-            });
-        });
-        afterEach(() => {
-            vi.restoreAllMocks();
-        });
-        it('passes arguments to TUI app correctly', async () => {
-            mockFindUntrackedFiles.mockResolvedValue([]);
-            const program = new Command();
-            program
-                .argument('[commit-ish]', 'commit-ish', 'HEAD')
-                .argument('[compare-with]', 'compare-with')
-                .option('--port <port>', 'port', parseInt)
-                .option('--host <host>', 'host', '')
-                .option('--no-open', 'no-open')
-                .option('--mode <mode>', 'mode', normalizeDiffViewMode, DEFAULT_DIFF_VIEW_MODE)
-                .option('--tui', 'tui')
-                .option('--pr <url>', 'pr')
-                .action(async (commitish, _compareWith, options) => {
-                if (options.tui) {
-                    if (!process.stdin.isTTY) {
-                        console.error('Error: TUI mode requires an interactive terminal (TTY).');
-                        process.exit(1);
-                    }
-                    const render = mockRender;
-                    const TuiApp = mockTuiApp;
-                    render(React.createElement(TuiApp, {
-                        selection: { targetCommitish: commitish, baseCommitish: commitish + '^' },
-                        mode: options.mode,
-                    }));
-                }
-            });
-            await program.parseAsync(['main', '--tui'], { from: 'user' });
-            expectRenderedTuiProps({
-                selection: { targetCommitish: 'main', baseCommitish: 'main^' },
-                mode: 'split',
-            });
-        });
-        it('passes context option to TUI app', async () => {
-            mockFindUntrackedFiles.mockResolvedValue([]);
-            const program = new Command();
-            program
-                .argument('[commit-ish]', 'commit-ish', 'HEAD')
-                .argument('[compare-with]', 'compare-with')
-                .option('--port <port>', 'port', parseInt)
-                .option('--host <host>', 'host', '')
-                .option('--no-open', 'no-open')
-                .option('--mode <mode>', 'mode', normalizeDiffViewMode, DEFAULT_DIFF_VIEW_MODE)
-                .option('--context <lines>', 'context', parseInt)
-                .option('--tui', 'tui')
-                .option('--pr <url>', 'pr')
-                .action(async (commitish, _compareWith, options) => {
-                if (options.tui) {
-                    const render = mockRender;
-                    const TuiApp = mockTuiApp;
-                    render(React.createElement(TuiApp, {
-                        selection: { targetCommitish: commitish, baseCommitish: commitish + '^' },
-                        mode: options.mode,
-                        contextLines: options.context,
-                    }));
-                }
-            });
-            await program.parseAsync(['--tui', '--context', '2'], { from: 'user' });
-            expectRenderedTuiProps({
-                selection: { targetCommitish: 'HEAD', baseCommitish: 'HEAD^' },
-                mode: 'split',
-                contextLines: 2,
-            });
-        });
-        it('passes mode option to TUI app', async () => {
-            mockFindUntrackedFiles.mockResolvedValue([]);
-            const program = new Command();
-            program
-                .argument('[commit-ish]', 'commit-ish', 'HEAD')
-                .argument('[compare-with]', 'compare-with')
-                .option('--port <port>', 'port', parseInt)
-                .option('--host <host>', 'host', '')
-                .option('--no-open', 'no-open')
-                .option('--mode <mode>', 'mode', normalizeDiffViewMode, DEFAULT_DIFF_VIEW_MODE)
-                .option('--tui', 'tui')
-                .option('--pr <url>', 'pr')
-                .action(async (commitish, _compareWith, options) => {
-                if (options.tui) {
-                    if (!process.stdin.isTTY) {
-                        console.error('Error: TUI mode requires an interactive terminal (TTY).');
-                        process.exit(1);
-                    }
-                    const render = mockRender;
-                    const TuiApp = mockTuiApp;
-                    render(React.createElement(TuiApp, {
-                        selection: { targetCommitish: commitish, baseCommitish: commitish + '^' },
-                        mode: options.mode,
-                    }));
-                }
-            });
-            await program.parseAsync(['--tui', '--mode', 'unified'], { from: 'user' });
-            expectRenderedTuiProps({
-                selection: { targetCommitish: 'HEAD', baseCommitish: 'HEAD^' },
-                mode: 'unified',
-            });
-        });
-        it('handles special arguments with TUI mode', async () => {
-            mockFindUntrackedFiles.mockResolvedValue([]);
-            const program = new Command();
-            program
-                .argument('[commit-ish]', 'commit-ish', 'HEAD')
-                .argument('[compare-with]', 'compare-with')
-                .option('--port <port>', 'port', parseInt)
-                .option('--host <host>', 'host', '')
-                .option('--no-open', 'no-open')
-                .option('--mode <mode>', 'mode', normalizeDiffViewMode, DEFAULT_DIFF_VIEW_MODE)
-                .option('--tui', 'tui')
-                .option('--pr <url>', 'pr')
-                .action(async (commitish, _compareWith, options) => {
-                if (options.tui) {
-                    const render = mockRender;
-                    const TuiApp = mockTuiApp;
-                    let targetCommitish = commitish;
-                    let baseCommitish;
-                    if (commitish === 'working') {
-                        baseCommitish = 'staged';
-                    }
-                    else if (commitish === 'staged' || commitish === '.') {
-                        baseCommitish = 'HEAD';
-                    }
-                    else {
-                        baseCommitish = commitish + '^';
-                    }
-                    render(React.createElement(TuiApp, {
-                        selection: { targetCommitish, baseCommitish },
-                        mode: options.mode,
-                    }));
-                }
-            });
-            await program.parseAsync(['working', '--tui', '--mode', 'unified'], { from: 'user' });
-            expectRenderedTuiProps({
-                selection: { targetCommitish: 'working', baseCommitish: 'staged' },
-                mode: 'unified',
-            });
-        });
-        it('rejects TUI mode in non-TTY environment', async () => {
-            // Mock non-TTY environment
-            Object.defineProperty(process.stdin, 'isTTY', {
-                value: false,
-                configurable: true,
-            });
-            const program = new Command();
-            program
-                .argument('[commit-ish]', 'commit-ish', 'HEAD')
-                .argument('[compare-with]', 'compare-with')
-                .option('--port <port>', 'port', parseInt)
-                .option('--host <host>', 'host', '')
-                .option('--no-open', 'no-open')
-                .option('--mode <mode>', 'mode', normalizeDiffViewMode, DEFAULT_DIFF_VIEW_MODE)
-                .option('--tui', 'tui')
-                .option('--pr <url>', 'pr')
-                .action(async (_commitish, _compareWith, options) => {
-                if (options.tui) {
-                    if (!process.stdin.isTTY) {
-                        console.error('Error: TUI mode requires an interactive terminal (TTY).');
-                        console.error('Try running the command directly in your terminal without piping.');
-                        process.exit(1);
-                    }
-                }
-            });
-            await program.parseAsync(['--tui'], { from: 'user' });
-            expect(console.error).toHaveBeenCalledWith('Error: TUI mode requires an interactive terminal (TTY).');
-            expect(console.error).toHaveBeenCalledWith('Try running the command directly in your terminal without piping.');
-            expect(process.exit).toHaveBeenCalledWith(1);
-        });
-    });
     describe('Diff mode determination', () => {
         const testCases = [
             {
@@ -1420,8 +1055,6 @@ describe('CLI index.ts', () => {
                     .option('--port <port>', 'port', parseInt)
                     .option('--host <host>', 'host', '')
                     .option('--no-open', 'no-open')
-                    .option('--mode <mode>', 'mode', normalizeDiffViewMode, DEFAULT_DIFF_VIEW_MODE)
-                    .option('--tui', 'tui')
                     .option('--pr <url>', 'pr')
                     .action(async (commitish, compareWith, options) => {
                     // Simulate determineDiffMode function behavior
@@ -1449,7 +1082,6 @@ describe('CLI index.ts', () => {
                         preferredPort: options.port,
                         host: options.host,
                         openBrowser: options.open,
-                        mode: options.mode,
                         diffMode,
                     });
                 });
@@ -1468,8 +1100,6 @@ describe('CLI index.ts', () => {
                 .option('--port <port>', 'port', parseInt)
                 .option('--host <host>', 'host', '')
                 .option('--no-open', 'no-open')
-                .option('--mode <mode>', 'mode', normalizeDiffViewMode, DEFAULT_DIFF_VIEW_MODE)
-                .option('--tui', 'tui')
                 .option('--pr <url>', 'pr')
                 .action(async (commitish, compareWith, options) => {
                 // Simulate determineDiffMode function behavior
@@ -1497,7 +1127,6 @@ describe('CLI index.ts', () => {
                     preferredPort: options.port,
                     host: options.host,
                     openBrowser: options.open,
-                    mode: options.mode,
                     diffMode,
                 });
             });
@@ -1516,8 +1145,6 @@ describe('CLI index.ts', () => {
                 .option('--port <port>', 'port', parseInt)
                 .option('--host <host>', 'host', '')
                 .option('--no-open', 'no-open')
-                .option('--mode <mode>', 'mode', normalizeDiffViewMode, DEFAULT_DIFF_VIEW_MODE)
-                .option('--tui', 'tui')
                 .option('--pr <url>', 'pr')
                 .action(async (commitish, compareWith, options) => {
                 // Simulate determineDiffMode function behavior with the fix
@@ -1545,7 +1172,6 @@ describe('CLI index.ts', () => {
                     preferredPort: options.port,
                     host: options.host,
                     openBrowser: options.open,
-                    mode: options.mode,
                     diffMode,
                 });
             });