factory-factory 0.3.2 → 0.3.4

package/dist/src/backend/domains/session/acp/codex-app-server-adapter/codex-app-server-acp-adapter.test.js

@@ -0,0 +1,1407 @@
+import { describe, expect, it, vi } from 'vitest';
+import { CodexAppServerAcpAdapter } from './codex-app-server-acp-adapter.js';
+import { CodexRequestError } from './codex-rpc-client.js';
+function createMockConnection() {
+    let resolveClosed = null;
+    const closed = new Promise((resolve) => {
+        resolveClosed = resolve;
+    });
+    return {
+        connection: {
+            closed,
+            sessionUpdate: vi.fn(async () => undefined),
+            requestPermission: vi.fn(() => {
+                return Promise.resolve({
+                    outcome: { outcome: 'selected', optionId: 'allow_once' },
+                });
+            }),
+        },
+        resolveClosed: () => {
+            resolveClosed?.();
+        },
+    };
+}
+function createMockCodexClient() {
+    const mocks = {
+        start: vi.fn(),
+        stop: vi.fn(async () => undefined),
+        request: vi.fn(),
+        notify: vi.fn(),
+        respondSuccess: vi.fn(),
+        respondError: vi.fn(),
+    };
+    return {
+        client: mocks,
+        mocks,
+    };
+}
+function getCodexRequestCalls(codex, method) {
+    return codex.request.mock.calls.filter((call) => call[0] === method);
+}
+const DEFAULT_APPROVAL_POLICY = 'on-failure';
+const DEFAULT_ALLOWED_APPROVAL_POLICIES = [DEFAULT_APPROVAL_POLICY, 'on-request', 'never'];
+const DEFAULT_ALLOWED_SANDBOX_MODES = ['read-only', 'workspace-write', 'danger-full-access'];
+const DEFAULT_COLLABORATION_MODES = [
+    { name: 'Default', mode: 'default' },
+    { name: 'Plan', mode: 'plan' },
+];
+async function initializeAdapterWithDefaultModel(adapter, codex) {
+    codex.request.mockResolvedValueOnce({});
+    codex.request.mockResolvedValueOnce({
+        requirements: {
+            allowedApprovalPolicies: DEFAULT_ALLOWED_APPROVAL_POLICIES,
+            allowedSandboxModes: DEFAULT_ALLOWED_SANDBOX_MODES,
+        },
+    });
+    codex.request.mockResolvedValueOnce({
+        data: DEFAULT_COLLABORATION_MODES,
+        nextCursor: null,
+    });
+    codex.request.mockResolvedValueOnce({
+        data: [
+            {
+                id: 'gpt-5',
+                displayName: 'GPT-5',
+                description: 'Default model',
+                defaultReasoningEffort: 'medium',
+                supportedReasoningEfforts: [],
+                inputModalities: ['text'],
+                isDefault: true,
+            },
+        ],
+        nextCursor: null,
+    });
+    await adapter.initialize({
+        protocolVersion: 1,
+        clientCapabilities: {},
+        clientInfo: { name: 'test-client', version: '0.0.1' },
+    });
+}
+describe('CodexAppServerAcpAdapter', () => {
+    it('paginates model/list during initialize', async () => {
+        const { connection } = createMockConnection();
+        const { client: codexClient, mocks: codex } = createMockCodexClient();
+        const adapter = new CodexAppServerAcpAdapter(connection, codexClient);
+        codex.request.mockResolvedValueOnce({});
+        codex.request.mockResolvedValueOnce({
+            requirements: {
+                allowedApprovalPolicies: DEFAULT_ALLOWED_APPROVAL_POLICIES,
+                allowedSandboxModes: DEFAULT_ALLOWED_SANDBOX_MODES,
+            },
+        });
+        codex.request.mockResolvedValueOnce({
+            data: DEFAULT_COLLABORATION_MODES,
+            nextCursor: null,
+        });
+        codex.request.mockResolvedValueOnce({
+            data: [
+                {
+                    id: 'gpt-5-mini',
+                    displayName: 'GPT-5 Mini',
+                    description: 'Fast model',
+                    defaultReasoningEffort: 'low',
+                    supportedReasoningEfforts: [],
+                    inputModalities: ['text'],
+                    isDefault: false,
+                },
+            ],
+            nextCursor: 'cursor-2',
+        });
+        codex.request.mockResolvedValueOnce({
+            data: [
+                {
+                    id: 'gpt-5',
+                    displayName: 'GPT-5',
+                    description: 'Default model',
+                    defaultReasoningEffort: 'medium',
+                    supportedReasoningEfforts: [],
+                    inputModalities: ['text'],
+                    isDefault: true,
+                },
+            ],
+            nextCursor: null,
+        });
+        await adapter.initialize({
+            protocolVersion: 1,
+            clientCapabilities: {},
+            clientInfo: { name: 'test-client', version: '0.0.1' },
+        });
+        expect(codex.start).toHaveBeenCalledTimes(1);
+        expect(codex.request).toHaveBeenNthCalledWith(2, 'configRequirements/read', undefined);
+        expect(codex.request).toHaveBeenNthCalledWith(3, 'collaborationMode/list', {});
+        expect(codex.request).toHaveBeenNthCalledWith(4, 'model/list', expect.objectContaining({}));
+        expect(codex.request).toHaveBeenNthCalledWith(5, 'model/list', { cursor: 'cursor-2' });
+    });
+    it('assigns session ids as sess_<threadId> on newSession', async () => {
+        const { connection } = createMockConnection();
+        const { client: codexClient, mocks: codex } = createMockCodexClient();
+        const adapter = new CodexAppServerAcpAdapter(connection, codexClient);
+        await initializeAdapterWithDefaultModel(adapter, codex);
+        codex.request.mockResolvedValueOnce({
+            thread: { id: 'thread_123', cwd: '/tmp/workspace' },
+            approvalPolicy: DEFAULT_APPROVAL_POLICY,
+            reasoningEffort: 'medium',
+        });
+        const response = await adapter.newSession({
+            cwd: '/tmp/workspace',
+            mcpServers: [],
+        });
+        expect(response.sessionId).toBe('sess_thread_123');
+    });
+    it('writes provided MCP servers to Codex config and reloads on newSession', async () => {
+        const { connection } = createMockConnection();
+        const { client: codexClient, mocks: codex } = createMockCodexClient();
+        const adapter = new CodexAppServerAcpAdapter(connection, codexClient);
+        await initializeAdapterWithDefaultModel(adapter, codex);
+        codex.request.mockResolvedValueOnce({
+            thread: { id: 'thread_with_mcp', cwd: '/tmp/workspace' },
+            approvalPolicy: DEFAULT_APPROVAL_POLICY,
+            reasoningEffort: 'medium',
+        });
+        codex.request.mockResolvedValueOnce({});
+        codex.request.mockResolvedValueOnce({});
+        const mcpServers = [
+            {
+                name: 'local-tools',
+                command: 'npx',
+                args: ['-y', '@modelcontextprotocol/server-filesystem', '/tmp/workspace'],
+                env: [{ name: 'NODE_ENV', value: 'test' }],
+            },
+        ];
+        await adapter.newSession({
+            cwd: '/tmp/workspace',
+            mcpServers,
+        });
+        expect(getCodexRequestCalls(codex, 'config/value/write')).toEqual([
+            [
+                'config/value/write',
+                {
+                    keyPath: 'mcp_servers',
+                    mergeStrategy: 'replace',
+                    value: {
+                        'local-tools': {
+                            enabled: true,
+                            command: 'npx',
+                            args: ['-y', '@modelcontextprotocol/server-filesystem', '/tmp/workspace'],
+                            env: { NODE_ENV: 'test' },
+                        },
+                    },
+                },
+            ],
+        ]);
+        expect(getCodexRequestCalls(codex, 'config/mcpServer/reload')).toEqual([
+            ['config/mcpServer/reload', {}],
+        ]);
+    });
+    it('deduplicates MCP server names when writing Codex config', async () => {
+        const { connection } = createMockConnection();
+        const { client: codexClient, mocks: codex } = createMockCodexClient();
+        const adapter = new CodexAppServerAcpAdapter(connection, codexClient);
+        await initializeAdapterWithDefaultModel(adapter, codex);
+        codex.request.mockResolvedValueOnce({
+            thread: { id: 'thread_mcp_names', cwd: '/tmp/workspace' },
+            approvalPolicy: DEFAULT_APPROVAL_POLICY,
+            reasoningEffort: 'medium',
+        });
+        codex.request.mockResolvedValueOnce({});
+        codex.request.mockResolvedValueOnce({});
+        const mcpServers = [
+            { name: '', command: 'server-a', args: [], env: [] },
+            { name: 'dup', command: 'server-b', args: [], env: [] },
+            { name: 'dup', command: 'server-c', args: [], env: [] },
+        ];
+        await adapter.newSession({
+            cwd: '/tmp/workspace',
+            mcpServers,
+        });
+        const configWriteCalls = getCodexRequestCalls(codex, 'config/value/write');
+        expect(configWriteCalls).toHaveLength(1);
+        const firstWriteCall = configWriteCalls[0];
+        if (!firstWriteCall) {
+            throw new Error('expected config/value/write call');
+        }
+        expect(firstWriteCall[1]).toEqual({
+            keyPath: 'mcp_servers',
+            mergeStrategy: 'replace',
+            value: {
+                mcp_server_1: { enabled: true, command: 'server-a', args: [] },
+                dup: { enabled: true, command: 'server-b', args: [] },
+                dup_2: { enabled: true, command: 'server-c', args: [] },
+            },
+        });
+    });
+    it('does not keep a session when MCP config write fails during newSession', async () => {
+        const { connection } = createMockConnection();
+        const { client: codexClient, mocks: codex } = createMockCodexClient();
+        const adapter = new CodexAppServerAcpAdapter(connection, codexClient);
+        await initializeAdapterWithDefaultModel(adapter, codex);
+        codex.request.mockResolvedValueOnce({
+            thread: { id: 'thread_mcp_error', cwd: '/tmp/workspace' },
+            approvalPolicy: DEFAULT_APPROVAL_POLICY,
+            reasoningEffort: 'medium',
+        });
+        codex.request.mockRejectedValueOnce(new Error('failed to write codex config'));
+        await expect(adapter.newSession({
+            cwd: '/tmp/workspace',
+            mcpServers: [{ name: 'broken', command: 'bad-server', args: [], env: [] }],
+        })).rejects.toThrow('failed to write codex config');
+        await expect(adapter.setSessionConfigOption({
+            sessionId: 'sess_thread_mcp_error',
+            configId: 'mode',
+            value: 'code',
+        })).rejects.toBeInstanceOf(Error);
+    });
+    it('keeps existing MCP server config when a subsequent session has no MCP servers', async () => {
+        const { connection } = createMockConnection();
+        const { client: codexClient, mocks: codex } = createMockCodexClient();
+        const adapter = new CodexAppServerAcpAdapter(connection, codexClient);
+        await initializeAdapterWithDefaultModel(adapter, codex);
+        codex.request.mockResolvedValueOnce({
+            thread: { id: 'thread_with_mcp_then_clear', cwd: '/tmp/workspace' },
+            approvalPolicy: DEFAULT_APPROVAL_POLICY,
+            reasoningEffort: 'medium',
+        });
+        codex.request.mockResolvedValueOnce({});
+        codex.request.mockResolvedValueOnce({});
+        codex.request.mockResolvedValueOnce({
+            thread: { id: 'thread_without_mcp', cwd: '/tmp/workspace' },
+            approvalPolicy: DEFAULT_APPROVAL_POLICY,
+            reasoningEffort: 'medium',
+        });
+        codex.request.mockResolvedValueOnce({});
+        codex.request.mockResolvedValueOnce({});
+        await adapter.newSession({
+            cwd: '/tmp/workspace',
+            mcpServers: [{ name: 'tools', command: 'server-a', args: [], env: [] }],
+        });
+        await adapter.newSession({
+            cwd: '/tmp/workspace',
+            mcpServers: [],
+        });
+        const configWriteCalls = getCodexRequestCalls(codex, 'config/value/write');
+        expect(configWriteCalls).toHaveLength(1);
+        expect(configWriteCalls[0]).toEqual([
+            'config/value/write',
+            {
+                keyPath: 'mcp_servers',
+                mergeStrategy: 'replace',
+                value: {
+                    tools: {
+                        enabled: true,
+                        command: 'server-a',
+                        args: [],
+                    },
+                },
+            },
+        ]);
+    });
+    it('loadSession replays thread history from thread/read', async () => {
+        const { connection } = createMockConnection();
+        const { client: codexClient, mocks: codex } = createMockCodexClient();
+        const adapter = new CodexAppServerAcpAdapter(connection, codexClient);
+        await initializeAdapterWithDefaultModel(adapter, codex);
+        codex.request.mockResolvedValueOnce({
+            thread: { id: 'thread_abc', cwd: '/tmp/workspace' },
+            approvalPolicy: DEFAULT_APPROVAL_POLICY,
+            reasoningEffort: 'medium',
+        });
+        codex.request.mockResolvedValueOnce({
+            thread: {
+                id: 'thread_abc',
+                turns: [
+                    {
+                        id: 'turn_1',
+                        items: [
+                            {
+                                type: 'userMessage',
+                                id: 'item_u1',
+                                content: [{ type: 'text', text: 'hello' }],
+                            },
+                            {
+                                type: 'agentMessage',
+                                id: 'item_a1',
+                                text: 'hi',
+                            },
+                            {
+                                type: 'commandExecution',
+                                id: 'item_c1',
+                                command: 'pwd',
+                            },
+                        ],
+                    },
+                ],
+            },
+        });
+        await adapter.loadSession({
+            sessionId: 'sess_thread_abc',
+            cwd: '/tmp/workspace',
+            mcpServers: [],
+        });
+        const updates = connection.sessionUpdate.mock.calls.map((call) => call[0].update);
+        expect(updates.some((update) => update.sessionUpdate === 'user_message_chunk')).toBe(true);
+        expect(updates.some((update) => update.sessionUpdate === 'agent_message_chunk')).toBe(true);
+        expect(updates.some((update) => update.sessionUpdate === 'tool_call' &&
+            update.status === 'completed' &&
+            update.toolCallId === 'codex:thread_abc:turn_1:item_c1')).toBe(true);
+    });
+    it('writes provided MCP servers to Codex config and reloads on loadSession', async () => {
+        const { connection } = createMockConnection();
+        const { client: codexClient, mocks: codex } = createMockCodexClient();
+        const adapter = new CodexAppServerAcpAdapter(connection, codexClient);
+        await initializeAdapterWithDefaultModel(adapter, codex);
+        codex.request.mockResolvedValueOnce({
+            thread: { id: 'thread_resume', cwd: '/tmp/workspace' },
+            approvalPolicy: DEFAULT_APPROVAL_POLICY,
+            reasoningEffort: 'medium',
+        });
+        codex.request.mockResolvedValueOnce({});
+        codex.request.mockResolvedValueOnce({});
+        codex.request.mockResolvedValueOnce({
+            thread: { id: 'thread_resume', turns: [] },
+        });
+        const mcpServers = [
+            {
+                type: 'http',
+                name: 'remote-tools',
+                url: 'https://mcp.example.com',
+                headers: [{ name: 'Authorization', value: 'Bearer token' }],
+            },
+        ];
+        await adapter.loadSession({
+            sessionId: 'sess_thread_resume',
+            cwd: '/tmp/workspace',
+            mcpServers,
+        });
+        expect(getCodexRequestCalls(codex, 'config/value/write')).toEqual([
+            [
+                'config/value/write',
+                {
+                    keyPath: 'mcp_servers',
+                    mergeStrategy: 'replace',
+                    value: {
+                        'remote-tools': {
+                            enabled: true,
+                            url: 'https://mcp.example.com',
+                            http_headers: {
+                                Authorization: 'Bearer token',
+                            },
+                        },
+                    },
+                },
+            ],
+        ]);
+        expect(getCodexRequestCalls(codex, 'config/mcpServer/reload')).toEqual([
+            ['config/mcpServer/reload', {}],
+        ]);
+    });
+    it('rejects invalid thought level values', async () => {
+        const { connection } = createMockConnection();
+        const { client: codexClient, mocks: codex } = createMockCodexClient();
+        const adapter = new CodexAppServerAcpAdapter(connection, codexClient);
+        await initializeAdapterWithDefaultModel(adapter, codex);
+        codex.request.mockResolvedValueOnce({
+            thread: { id: 'thread_123', cwd: '/tmp/workspace' },
+            approvalPolicy: DEFAULT_APPROVAL_POLICY,
+            reasoningEffort: 'medium',
+        });
+        const session = await adapter.newSession({
+            cwd: '/tmp/workspace',
+            mcpServers: [],
+        });
+        await expect(adapter.setSessionConfigOption({
+            sessionId: session.sessionId,
+            configId: 'reasoning_effort',
+            value: 'turbo',
+        })).rejects.toBeInstanceOf(Error);
+    });
+    it('falls back to default execution presets when config requirements are missing', async () => {
+        const { connection } = createMockConnection();
+        const { client: codexClient, mocks: codex } = createMockCodexClient();
+        const adapter = new CodexAppServerAcpAdapter(connection, codexClient);
+        codex.request.mockResolvedValueOnce({});
+        codex.request.mockResolvedValueOnce({
+            requirements: null,
+        });
+        codex.request.mockResolvedValueOnce({
+            data: DEFAULT_COLLABORATION_MODES,
+            nextCursor: null,
+        });
+        codex.request.mockResolvedValueOnce({
+            data: [
+                {
+                    id: 'gpt-5',
+                    displayName: 'GPT-5',
+                    description: 'Default model',
+                    defaultReasoningEffort: 'medium',
+                    supportedReasoningEfforts: [],
+                    inputModalities: ['text'],
+                    isDefault: true,
+                },
+            ],
+            nextCursor: null,
+        });
+        await adapter.initialize({
+            protocolVersion: 1,
+            clientCapabilities: {},
+            clientInfo: { name: 'test-client', version: '0.0.1' },
+        });
+        codex.request.mockResolvedValueOnce({
+            thread: { id: 'thread_defaults', cwd: '/tmp/workspace' },
+            approvalPolicy: 'on-request',
+            sandbox: {
+                type: 'workspaceWrite',
+                writableRoots: ['/tmp/workspace'],
+                readOnlyAccess: { type: 'fullAccess' },
+                networkAccess: true,
+                excludeTmpdirEnvVar: false,
+                excludeSlashTmp: false,
+            },
+            reasoningEffort: 'medium',
+        });
+        const session = await adapter.newSession({
+            cwd: '/tmp/workspace',
+            mcpServers: [],
+        });
+        const executionModeOption = (session.configOptions ?? []).find((option) => option.id === 'execution_mode');
+        expect(executionModeOption?.type).toBe('select');
+        if (executionModeOption?.type !== 'select') {
+            throw new Error('expected execution_mode select option');
+        }
+        const presetValues = executionModeOption.options.flatMap((option) => 'value' in option ? [option.value] : []);
+        const presetEntries = executionModeOption.options.flatMap((option) => 'value' in option ? [option] : []);
+        expect(presetValues.length).toBeGreaterThan(1);
+        expect(presetValues).toContain(JSON.stringify(['never', 'danger-full-access']));
+        expect(presetValues).toContain(JSON.stringify(['on-failure', 'workspace-write']));
+        expect(presetEntries).toContainEqual(expect.objectContaining({
+            value: JSON.stringify(['never', 'danger-full-access']),
+            name: 'YOLO (Full Access)',
+        }));
+        expect(presetEntries).toContainEqual(expect.objectContaining({
+            value: JSON.stringify(['on-request', 'workspace-write']),
+            name: 'On Request (Workspace Write)',
+        }));
+    });
+    it('applies discovered execution mode and plan collaboration mode to turn/start', async () => {
+        const { connection } = createMockConnection();
+        const { client: codexClient, mocks: codex } = createMockCodexClient();
+        const adapter = new CodexAppServerAcpAdapter(connection, codexClient);
+        await initializeAdapterWithDefaultModel(adapter, codex);
+        codex.request.mockResolvedValueOnce({
+            thread: { id: 'thread_modes', cwd: '/tmp/workspace' },
+            approvalPolicy: 'on-request',
+            sandbox: {
+                type: 'workspaceWrite',
+                writableRoots: ['/tmp/workspace'],
+                readOnlyAccess: { type: 'fullAccess' },
+                networkAccess: true,
+                excludeTmpdirEnvVar: false,
+                excludeSlashTmp: false,
+            },
+            reasoningEffort: 'medium',
+        });
+        const session = await adapter.newSession({
+            cwd: '/tmp/workspace',
+            mcpServers: [],
+        });
+        const executionModeOption = (session.configOptions ?? []).find((option) => option.id === 'execution_mode');
+        const yoloLikeOption = executionModeOption?.type === 'select'
+            ? executionModeOption.options.find((option) => 'value' in option &&
+                option.value.includes('never') &&
+                option.value.includes('danger-full-access'))
+            : null;
+        expect(yoloLikeOption).toBeDefined();
+        if (!yoloLikeOption) {
+            throw new Error('expected discovered execution mode option');
+        }
+        await adapter.setSessionMode({ sessionId: session.sessionId, modeId: 'plan' });
+        await adapter.setSessionConfigOption({
+            sessionId: session.sessionId,
+            configId: 'execution_mode',
+            value: yoloLikeOption.value,
+        });
+        codex.request.mockResolvedValueOnce({
+            turn: { id: 'turn_modes', status: 'completed' },
+        });
+        const promptResponse = await adapter.prompt({
+            sessionId: session.sessionId,
+            prompt: [{ type: 'text', text: 'hello' }],
+        });
+        expect(promptResponse.stopReason).toBe('end_turn');
+        expect(codex.request).toHaveBeenCalledWith('turn/start', expect.objectContaining({
+            threadId: 'thread_modes',
+            approvalPolicy: 'never',
+            sandboxPolicy: expect.objectContaining({ type: 'dangerFullAccess' }),
+            collaborationMode: expect.objectContaining({
+                mode: 'plan',
+            }),
+        }));
+    });
+    it('rejects empty prompt content and does not start a turn', async () => {
+        const { connection } = createMockConnection();
+        const { client: codexClient, mocks: codex } = createMockCodexClient();
+        const adapter = new CodexAppServerAcpAdapter(connection, codexClient);
+        await initializeAdapterWithDefaultModel(adapter, codex);
+        codex.request.mockResolvedValueOnce({
+            thread: { id: 'thread_empty_prompt', cwd: '/tmp/workspace' },
+            approvalPolicy: DEFAULT_APPROVAL_POLICY,
+            reasoningEffort: 'medium',
+        });
+        const session = await adapter.newSession({
+            cwd: '/tmp/workspace',
+            mcpServers: [],
+        });
+        await expect(adapter.prompt({
+            sessionId: session.sessionId,
+            prompt: [{ type: 'text', text: '   ' }],
+        })).rejects.toBeInstanceOf(Error);
+        expect(getCodexRequestCalls(codex, 'turn/start')).toHaveLength(0);
+        codex.request.mockResolvedValueOnce({
+            turn: { id: 'turn_after_empty_prompt', status: 'completed' },
+        });
+        await expect(adapter.prompt({
+            sessionId: session.sessionId,
+            prompt: [{ type: 'text', text: 'run now' }],
+        })).resolves.toEqual({ stopReason: 'end_turn' });
+        expect(getCodexRequestCalls(codex, 'turn/start')).toHaveLength(1);
+    });
+    it('requests ExitPlanMode approval after completed plan item in plan mode', async () => {
+        const { connection } = createMockConnection();
+        connection.requestPermission.mockResolvedValue({
+            outcome: { outcome: 'selected', optionId: 'allow_once' },
+        });
+        const { client: codexClient, mocks: codex } = createMockCodexClient();
+        const adapter = new CodexAppServerAcpAdapter(connection, codexClient);
+        await initializeAdapterWithDefaultModel(adapter, codex);
+        codex.request.mockResolvedValueOnce({
+            thread: { id: 'thread_plan_approval', cwd: '/tmp/workspace' },
+            approvalPolicy: DEFAULT_APPROVAL_POLICY,
+            reasoningEffort: 'medium',
+        });
+        const session = await adapter.newSession({
+            cwd: '/tmp/workspace',
+            mcpServers: [],
+        });
+        await adapter.setSessionMode({ sessionId: session.sessionId, modeId: 'plan' });
+        await adapter.handleCodexNotification('item/started', {
+            threadId: 'thread_plan_approval',
+            turnId: 'turn_plan_approval',
+            item: {
+                type: 'plan',
+                id: 'item_plan_approval',
+                status: 'inProgress',
+            },
+        });
+        await adapter.handleCodexNotification('item/plan/delta', {
+            threadId: 'thread_plan_approval',
+            turnId: 'turn_plan_approval',
+            itemId: 'item_plan_approval',
+            delta: '## Proposed Plan\n1. Add adapter plan approval bridge',
+        });
+        await adapter.handleCodexNotification('item/completed', {
+            threadId: 'thread_plan_approval',
+            turnId: 'turn_plan_approval',
+            item: {
+                type: 'plan',
+                id: 'item_plan_approval',
+                status: 'completed',
+            },
+        });
+        expect(connection.requestPermission).toHaveBeenCalledWith(expect.objectContaining({
+            toolCall: expect.objectContaining({
+                title: 'ExitPlanMode',
+                kind: 'switch_mode',
+            }),
+            options: [
+                { optionId: 'allow_once', name: 'Approve plan', kind: 'allow_once' },
+                { optionId: 'reject_once', name: 'Request changes', kind: 'reject_once' },
+            ],
+        }));
+        expect(connection.sessionUpdate.mock.calls).toEqual(expect.arrayContaining([
+            [
+                expect.objectContaining({
+                    update: expect.objectContaining({
+                        sessionUpdate: 'tool_call',
+                        title: 'ExitPlanMode',
+                        kind: 'switch_mode',
+                        status: 'pending',
+                        rawInput: expect.objectContaining({
+                            type: 'ExitPlanMode',
+                            plan: expect.objectContaining({
+                                type: 'text',
+                                text: expect.stringContaining('## Proposed Plan'),
+                            }),
+                        }),
+                    }),
+                }),
+            ],
+            [
+                expect.objectContaining({
+                    update: expect.objectContaining({
+                        sessionUpdate: 'config_option_update',
+                        configOptions: expect.arrayContaining([
+                            expect.objectContaining({
+                                id: 'mode',
+                                currentValue: 'default',
+                            }),
+                        ]),
+                    }),
+                }),
+            ],
+            [
+                expect.objectContaining({
+                    update: expect.objectContaining({
+                        sessionUpdate: 'tool_call_update',
+                        title: 'ExitPlanMode',
+                        kind: 'switch_mode',
+                        status: 'completed',
+                    }),
+                }),
+            ],
+        ]));
+    });
+    it('holds prompt completion until plan approval resolves and exits plan mode', async () => {
+        let resolvePlanApproval = () => {
+            throw new Error('expected plan approval request callback');
+        };
+        const { connection } = createMockConnection();
+        connection.requestPermission.mockImplementation(() => new Promise((resolve) => {
+            resolvePlanApproval = resolve;
+        }));
+        const { client: codexClient, mocks: codex } = createMockCodexClient();
+        const adapter = new CodexAppServerAcpAdapter(connection, codexClient);
+        await initializeAdapterWithDefaultModel(adapter, codex);
+        codex.request.mockResolvedValueOnce({
+            thread: { id: 'thread_plan_hold', cwd: '/tmp/workspace' },
+            approvalPolicy: DEFAULT_APPROVAL_POLICY,
+            reasoningEffort: 'medium',
+        });
+        const session = await adapter.newSession({
+            cwd: '/tmp/workspace',
+            mcpServers: [],
+        });
+        await adapter.setSessionMode({ sessionId: session.sessionId, modeId: 'plan' });
+        codex.request.mockResolvedValueOnce({
+            turn: { id: 'turn_plan_hold', status: 'inProgress' },
+        });
+        const promptPromise = adapter.prompt({
+            sessionId: session.sessionId,
+            prompt: [{ type: 'text', text: 'continue' }],
+        });
+        await vi.waitFor(() => {
+            expect(codex.request).toHaveBeenCalledWith('turn/start', expect.objectContaining({
+                threadId: 'thread_plan_hold',
+            }));
+        });
+        await adapter.handleCodexNotification('item/started', {
+            threadId: 'thread_plan_hold',
+            turnId: 'turn_plan_hold',
+            item: {
+                type: 'plan',
+                id: 'item_plan_hold',
+                status: 'inProgress',
+            },
+        });
+        await adapter.handleCodexNotification('item/plan/delta', {
+            threadId: 'thread_plan_hold',
+            turnId: 'turn_plan_hold',
+            itemId: 'item_plan_hold',
+            delta: '# Plan\n- implement it',
+        });
+        const itemCompletedPromise = adapter.handleCodexNotification('item/completed', {
+            threadId: 'thread_plan_hold',
+            turnId: 'turn_plan_hold',
+            item: {
+                type: 'plan',
+                id: 'item_plan_hold',
+                status: 'completed',
+            },
+        });
+        await adapter.handleCodexNotification('turn/completed', {
+            threadId: 'thread_plan_hold',
+            turn: { id: 'turn_plan_hold', status: 'completed', error: null, items: [] },
+        });
+        let promptSettled = false;
+        void promptPromise.then(() => {
+            promptSettled = true;
+        });
+        await new Promise((resolve) => setTimeout(resolve, 0));
+        expect(promptSettled).toBe(false);
+        resolvePlanApproval({
+            outcome: { outcome: 'selected', optionId: 'allow_once' },
+        });
+        await itemCompletedPromise;
+        await expect(promptPromise).resolves.toEqual({ stopReason: 'end_turn' });
+        codex.request.mockResolvedValueOnce({
+            turn: { id: 'turn_after_plan_hold', status: 'completed' },
+        });
+        const secondPrompt = await adapter.prompt({
+            sessionId: session.sessionId,
+            prompt: [{ type: 'text', text: 'implement now' }],
+        });
+        expect(secondPrompt.stopReason).toBe('end_turn');
+        const turnStartCalls = getCodexRequestCalls(codex, 'turn/start');
+        const secondTurnStart = turnStartCalls[1]?.[1];
+        expect(secondTurnStart).toBeDefined();
+        expect(secondTurnStart).toEqual(expect.objectContaining({
+            collaborationMode: expect.objectContaining({
+                mode: 'default',
+            }),
+        }));
+    });
+    it('marks synthetic ExitPlanMode approval as failed when user rejects', async () => {
+        const { connection } = createMockConnection();
+        connection.requestPermission.mockResolvedValue({
+            outcome: { outcome: 'selected', optionId: 'reject_once' },
+        });
+        const { client: codexClient, mocks: codex } = createMockCodexClient();
+        const adapter = new CodexAppServerAcpAdapter(connection, codexClient);
+        await initializeAdapterWithDefaultModel(adapter, codex);
+        codex.request.mockResolvedValueOnce({
+            thread: { id: 'thread_plan_reject', cwd: '/tmp/workspace' },
+            approvalPolicy: DEFAULT_APPROVAL_POLICY,
+            reasoningEffort: 'medium',
+        });
+        const session = await adapter.newSession({
+            cwd: '/tmp/workspace',
+            mcpServers: [],
+        });
+        await adapter.setSessionMode({ sessionId: session.sessionId, modeId: 'plan' });
+        await adapter.handleCodexNotification('item/started', {
+            threadId: 'thread_plan_reject',
+            turnId: 'turn_plan_reject',
+            item: {
+                type: 'plan',
+                id: 'item_plan_reject',
+                status: 'inProgress',
+            },
+        });
+        await adapter.handleCodexNotification('item/plan/delta', {
+            threadId: 'thread_plan_reject',
+            turnId: 'turn_plan_reject',
+            itemId: 'item_plan_reject',
+            delta: '# Plan\n- gather context',
+        });
+        await adapter.handleCodexNotification('item/completed', {
+            threadId: 'thread_plan_reject',
+            turnId: 'turn_plan_reject',
+            item: {
+                type: 'plan',
+                id: 'item_plan_reject',
+                status: 'completed',
+            },
+        });
+        expect(connection.sessionUpdate.mock.calls).toEqual(expect.arrayContaining([
+            [
+                expect.objectContaining({
+                    update: expect.objectContaining({
+                        sessionUpdate: 'tool_call_update',
+                        title: 'ExitPlanMode',
+                        kind: 'switch_mode',
+                        status: 'failed',
+                        rawOutput: 'Plan approval rejected',
+                    }),
+                }),
+            ],
+        ]));
+    });
+    it('maps tool request_user_input selections into answer payloads', async () => {
+        const { connection } = createMockConnection();
+        connection.requestPermission.mockResolvedValue({
+            outcome: { outcome: 'selected', optionId: 'answer_1' },
+        });
+        const { client: codexClient, mocks: codex } = createMockCodexClient();
+        const adapter = new CodexAppServerAcpAdapter(connection, codexClient);
+        await initializeAdapterWithDefaultModel(adapter, codex);
+        codex.request.mockResolvedValueOnce({
+            thread: { id: 'thread_perm', cwd: '/tmp/workspace' },
+            approvalPolicy: DEFAULT_APPROVAL_POLICY,
+            reasoningEffort: 'medium',
+        });
+        await adapter.newSession({
+            cwd: '/tmp/workspace',
+            mcpServers: [],
+        });
+        await adapter.handleCodexServerRequest({
+            id: 17,
+            method: 'item/tool/requestUserInput',
+            params: {
+                threadId: 'thread_perm',
+                turnId: 'turn_perm',
+                itemId: 'item_perm',
+                questions: [
+                    {
+                        id: 'choice',
+                        header: 'Pick one',
+                        question: 'Select an option',
+                        isOther: false,
+                        isSecret: false,
+                        options: [
+                            { label: 'First', description: 'One' },
+                            { label: 'Second', description: 'Two' },
+                        ],
+                    },
+                ],
+            },
+        });
+        expect(codex.respondSuccess).toHaveBeenCalledWith(17, {
+            answers: {
+                choice: { answers: ['Second'] },
+            },
+        });
+        expect(connection.requestPermission).toHaveBeenCalledWith(expect.objectContaining({
+            toolCall: expect.objectContaining({
+                title: 'item/tool/requestUserInput',
+            }),
+        }));
+        expect(connection.sessionUpdate.mock.calls).toEqual(expect.arrayContaining([
+            [
+                expect.objectContaining({
+                    update: expect.objectContaining({
+                        sessionUpdate: 'tool_call_update',
+                        status: 'completed',
+                    }),
+                }),
+            ],
+        ]));
+    });
+    it('maps multi-question request_user_input answers from ACP _meta payload', async () => {
+        const { connection } = createMockConnection();
+        connection.requestPermission.mockResolvedValue({
+            _meta: {
+                factoryFactory: {
+                    toolUserInputAnswers: {
+                        color: ['Blue'],
+                        shell: ['zsh', 'fish'],
+                        unknown: ['ignored'],
+                    },
+                },
+            },
+            outcome: { outcome: 'selected', optionId: 'allow_once' },
+        });
+        const { client: codexClient, mocks: codex } = createMockCodexClient();
+        const adapter = new CodexAppServerAcpAdapter(connection, codexClient);
+        await initializeAdapterWithDefaultModel(adapter, codex);
+        codex.request.mockResolvedValueOnce({
+            thread: { id: 'thread_multi_question', cwd: '/tmp/workspace' },
+            approvalPolicy: DEFAULT_APPROVAL_POLICY,
+            reasoningEffort: 'medium',
+        });
+        await adapter.newSession({
+            cwd: '/tmp/workspace',
+            mcpServers: [],
+        });
+        await adapter.handleCodexServerRequest({
+            id: 19,
+            method: 'item/tool/requestUserInput',
+            params: {
+                threadId: 'thread_multi_question',
+                turnId: 'turn_multi_question',
+                itemId: 'item_multi_question',
+                questions: [
+                    {
+                        id: 'color',
+                        header: 'Pick color',
+                        question: 'Favorite color?',
+                        isOther: false,
+                        isSecret: false,
+                        options: [{ label: 'Blue', description: 'cool' }],
+                    },
+                    {
+                        id: 'shell',
+                        header: 'Pick shells',
+                        question: 'Preferred shells?',
+                        isOther: true,
+                        isSecret: false,
+                        options: [
+                            { label: 'zsh', description: 'default' },
+                            { label: 'fish', description: 'friendly' },
+                        ],
+                    },
+                ],
+            },
+        });
+        expect(codex.respondSuccess).toHaveBeenCalledWith(19, {
+            answers: {
+                color: { answers: ['Blue'] },
+                shell: { answers: ['zsh', 'fish'] },
+            },
+        });
+        expect(connection.requestPermission).toHaveBeenCalledWith(expect.objectContaining({
+            options: [
+                { optionId: 'allow_once', name: 'Submit', kind: 'allow_once' },
+                { optionId: 'reject_once', name: 'Cancel', kind: 'reject_once' },
+            ],
+        }));
+    });
+    it('fails multi-question request_user_input when structured answers are missing', async () => {
+        const { connection } = createMockConnection();
+        connection.requestPermission.mockResolvedValue({
+            outcome: { outcome: 'selected', optionId: 'answer_1' },
+        });
+        const { client: codexClient, mocks: codex } = createMockCodexClient();
+        const adapter = new CodexAppServerAcpAdapter(connection, codexClient);
+        await initializeAdapterWithDefaultModel(adapter, codex);
+        codex.request.mockResolvedValueOnce({
+            thread: { id: 'thread_multi_question_no_meta', cwd: '/tmp/workspace' },
+            approvalPolicy: DEFAULT_APPROVAL_POLICY,
+            reasoningEffort: 'medium',
+        });
+        await adapter.newSession({
+            cwd: '/tmp/workspace',
+            mcpServers: [],
+        });
+        await adapter.handleCodexServerRequest({
+            id: 20,
+            method: 'item/tool/requestUserInput',
+            params: {
+                threadId: 'thread_multi_question_no_meta',
+                turnId: 'turn_multi_question_no_meta',
+                itemId: 'item_multi_question_no_meta',
+                questions: [
+                    {
+                        id: 'color',
+                        header: 'Pick color',
+                        question: 'Favorite color?',
+                        isOther: false,
+                        isSecret: false,
+                        options: [{ label: 'Blue', description: 'cool' }],
+                    },
+                    {
+                        id: 'shell',
+                        header: 'Pick shells',
+                        question: 'Preferred shells?',
+                        isOther: true,
+                        isSecret: false,
+                        options: [
+                            { label: 'zsh', description: 'default' },
+                            { label: 'fish', description: 'friendly' },
+                        ],
+                    },
+                ],
+            },
+        });
+        expect(codex.respondError).toHaveBeenCalledWith(20, expect.objectContaining({
+            message: 'Failed to map requestUserInput answers',
+        }));
+        expect(connection.sessionUpdate.mock.calls).toEqual(expect.arrayContaining([
+            [
+                expect.objectContaining({
+                    update: expect.objectContaining({
+                        sessionUpdate: 'tool_call_update',
+                        status: 'failed',
+                        rawOutput: expect.objectContaining({
+                            error: 'Missing structured answers for multi-question requestUserInput',
+                        }),
+                    }),
+                }),
+            ],
+        ]));
+    });
+    it('treats cancelled request_user_input permission outcomes as rejected', async () => {
+        const { connection } = createMockConnection();
+        connection.requestPermission.mockResolvedValue({
+            outcome: { outcome: 'cancelled' },
+        });
+        const { client: codexClient, mocks: codex } = createMockCodexClient();
+        const adapter = new CodexAppServerAcpAdapter(connection, codexClient);
+        await initializeAdapterWithDefaultModel(adapter, codex);
+        codex.request.mockResolvedValueOnce({
+            thread: { id: 'thread_cancelled_input', cwd: '/tmp/workspace' },
+            approvalPolicy: DEFAULT_APPROVAL_POLICY,
+            reasoningEffort: 'medium',
+        });
+        await adapter.newSession({
+            cwd: '/tmp/workspace',
+            mcpServers: [],
+        });
+        await adapter.handleCodexServerRequest({
+            id: 21,
+            method: 'item/tool/requestUserInput',
+            params: {
+                threadId: 'thread_cancelled_input',
+                turnId: 'turn_cancelled_input',
+                itemId: 'item_cancelled_input',
+                questions: [
+                    {
+                        id: 'choice',
+                        header: 'Pick one',
+                        question: 'Select an option',
+                        isOther: false,
+                        isSecret: false,
+                        options: [
+                            { label: 'First', description: 'One' },
+                            { label: 'Second', description: 'Two' },
+                        ],
+                    },
+                ],
+            },
+        });
+        expect(codex.respondSuccess).toHaveBeenCalledWith(21, {
+            answers: {},
+        });
+        expect(connection.sessionUpdate.mock.calls).toEqual(expect.arrayContaining([
+            [
+                expect.objectContaining({
+                    update: expect.objectContaining({
+                        sessionUpdate: 'tool_call_update',
+                        status: 'failed',
+                        rawOutput: { outcome: 'rejected' },
+                    }),
+                }),
+            ],
+        ]));
+    });
+    it('stops codex subprocess when ACP connection closes', async () => {
+        const { connection, resolveClosed } = createMockConnection();
+        const { client: codexClient, mocks: codex } = createMockCodexClient();
+        new CodexAppServerAcpAdapter(connection, codexClient);
+        resolveClosed();
+        await Promise.resolve();
+        await Promise.resolve();
+        expect(codex.stop).toHaveBeenCalledTimes(1);
+    });
+    it('does not access connection.closed synchronously during constructor', async () => {
+        let allowClosedRead = false;
+        const closedPromise = Promise.resolve();
+        const connection = {
+            get closed() {
+                if (!allowClosedRead) {
+                    throw new Error('closed accessed too early');
+                }
+                return closedPromise;
+            },
+            sessionUpdate: vi.fn(async () => undefined),
+            requestPermission: vi.fn(() => Promise.resolve({
+                outcome: { outcome: 'selected', optionId: 'allow_once' },
+            })),
+        };
+        const { client: codexClient, mocks: codex } = createMockCodexClient();
+        expect(() => {
+            new CodexAppServerAcpAdapter(connection, codexClient);
+        }).not.toThrow();
+        allowClosedRead = true;
+        await Promise.resolve();
+        await Promise.resolve();
+        await new Promise((resolve) => setTimeout(resolve, 0));
+        expect(codex.stop).toHaveBeenCalledTimes(1);
+    });
+    it('retries closed watcher registration when closed getter is not ready', async () => {
+        let closedReadCount = 0;
+        let resolveClosed;
+        const closed = new Promise((resolve) => {
+            resolveClosed = () => resolve();
+        });
+        const connection = {
+            get closed() {
+                closedReadCount += 1;
+                if (closedReadCount === 1) {
+                    throw new Error('closed not ready yet');
+                }
+                return closed;
+            },
+            sessionUpdate: vi.fn(async () => undefined),
+            requestPermission: vi.fn(() => Promise.resolve({
+                outcome: { outcome: 'selected', optionId: 'allow_once' },
+            })),
+        };
+        const { client: codexClient, mocks: codex } = createMockCodexClient();
+        new CodexAppServerAcpAdapter(connection, codexClient);
+        await Promise.resolve();
+        await new Promise((resolve) => setTimeout(resolve, 0));
+        resolveClosed?.();
+        await Promise.resolve();
+        await Promise.resolve();
+        expect(closedReadCount).toBeGreaterThanOrEqual(2);
+        expect(codex.stop).toHaveBeenCalledTimes(1);
+    });
+    it('stops retrying close watcher registration after retry limit', async () => {
+        vi.useFakeTimers();
+        const stderrSpy = vi.spyOn(process.stderr, 'write').mockReturnValue(true);
+        try {
+            const connection = {
+                get closed() {
+                    throw new Error('closed never ready');
+                },
+                sessionUpdate: vi.fn(async () => undefined),
+                requestPermission: vi.fn(() => Promise.resolve({
+                    outcome: { outcome: 'selected', optionId: 'allow_once' },
+                })),
+            };
+            const { client: codexClient, mocks: codex } = createMockCodexClient();
+            new CodexAppServerAcpAdapter(connection, codexClient);
+            await vi.runAllTimersAsync();
+            expect(stderrSpy).toHaveBeenCalledWith(expect.stringContaining('failed to attach close watcher after 50 attempts'));
+            expect(codex.stop).toHaveBeenCalledTimes(1);
+        }
+        finally {
+            stderrSpy.mockRestore();
+            vi.useRealTimers();
+        }
+    });
+    it('returns end_turn on overloaded turn/start and emits fallback message', async () => {
+        const { connection } = createMockConnection();
+        const { client: codexClient, mocks: codex } = createMockCodexClient();
+        const adapter = new CodexAppServerAcpAdapter(connection, codexClient);
+        await initializeAdapterWithDefaultModel(adapter, codex);
+        codex.request.mockResolvedValueOnce({
+            thread: { id: 'thread_overload', cwd: '/tmp/workspace' },
+            approvalPolicy: DEFAULT_APPROVAL_POLICY,
+            reasoningEffort: 'medium',
+        });
+        const session = await adapter.newSession({
+            cwd: '/tmp/workspace',
+            mcpServers: [],
+        });
+        codex.request.mockRejectedValueOnce(new CodexRequestError({
+            code: -32_001,
+            message: 'Server overloaded; retry later.',
+        }));
+        const response = await adapter.prompt({
+            sessionId: session.sessionId,
+            prompt: [{ type: 'text', text: 'hello' }],
+        });
+        expect(response.stopReason).toBe('end_turn');
+        expect(connection.sessionUpdate.mock.calls).toEqual(expect.arrayContaining([
+            [
+                expect.objectContaining({
+                    update: expect.objectContaining({
+                        sessionUpdate: 'agent_message_chunk',
+                    }),
+                }),
+            ],
+        ]));
+    });
+    it('queues early cancel until turn id is known, then interrupts the turn', async () => {
+        const { connection } = createMockConnection();
+        const { client: codexClient, mocks: codex } = createMockCodexClient();
+        const adapter = new CodexAppServerAcpAdapter(connection, codexClient);
+        await initializeAdapterWithDefaultModel(adapter, codex);
+        codex.request.mockResolvedValueOnce({
+            thread: { id: 'thread_early_cancel', cwd: '/tmp/workspace' },
+            approvalPolicy: DEFAULT_APPROVAL_POLICY,
+            reasoningEffort: 'medium',
+        });
+        const session = await adapter.newSession({
+            cwd: '/tmp/workspace',
+            mcpServers: [],
+        });
+        let resolveTurnStart;
+        codex.request.mockImplementationOnce(async (method) => {
+            if (method !== 'turn/start') {
+                return { turn: { id: 'unexpected', status: 'inProgress' } };
+            }
+            return await new Promise((resolve) => {
+                resolveTurnStart = resolve;
+            });
+        });
+        const promptPromise = adapter.prompt({
+            sessionId: session.sessionId,
+            prompt: [{ type: 'text', text: 'hello' }],
+        });
+        await vi.waitFor(() => {
+            const activeTurn = adapter.sessions.get(session.sessionId)?.activeTurn;
+            expect(activeTurn?.turnId).toBe('__pending_turn__');
+        });
+        await adapter.cancel({ sessionId: session.sessionId });
+        expect(codex.request).not.toHaveBeenCalledWith('turn/interrupt', expect.objectContaining({
+            threadId: 'thread_early_cancel',
+        }));
+        resolveTurnStart?.({ turn: { id: 'turn_early_cancel', status: 'inProgress' } });
+        await vi.waitFor(() => {
+            expect(codex.request).toHaveBeenCalledWith('turn/interrupt', {
+                threadId: 'thread_early_cancel',
+                turnId: 'turn_early_cancel',
+            });
+        });
+        await adapter.handleCodexNotification('turn/completed', {
+            threadId: 'thread_early_cancel',
+            turn: { id: 'turn_early_cancel', status: 'interrupted', error: null, items: [] },
+        });
+        await expect(promptPromise).resolves.toEqual({ stopReason: 'cancelled' });
+    });
+    it('cancel interrupts active turn and resolves prompt with cancelled', async () => {
+        const { connection } = createMockConnection();
+        const { client: codexClient, mocks: codex } = createMockCodexClient();
+        const adapter = new CodexAppServerAcpAdapter(connection, codexClient);
+        await initializeAdapterWithDefaultModel(adapter, codex);
+        codex.request.mockResolvedValueOnce({
+            thread: { id: 'thread_cancel', cwd: '/tmp/workspace' },
+            approvalPolicy: DEFAULT_APPROVAL_POLICY,
+            reasoningEffort: 'medium',
+        });
+        const session = await adapter.newSession({
+            cwd: '/tmp/workspace',
+            mcpServers: [],
+        });
+        codex.request.mockResolvedValueOnce({
+            turn: { id: 'turn_cancel', status: 'inProgress' },
+        });
+        codex.request.mockResolvedValueOnce({});
+        const promptPromise = adapter.prompt({
+            sessionId: session.sessionId,
+            prompt: [{ type: 'text', text: 'hello' }],
+        });
+        await vi.waitFor(() => {
+            expect(codex.request).toHaveBeenCalledWith('turn/start', expect.objectContaining({ threadId: 'thread_cancel' }));
+        });
+        await vi.waitFor(() => {
+            const activeTurn = adapter.sessions.get(session.sessionId)?.activeTurn;
+            expect(activeTurn?.turnId).toBe('turn_cancel');
+        });
+        await adapter.cancel({ sessionId: session.sessionId });
+        expect(codex.request).toHaveBeenCalledWith('turn/interrupt', {
+            threadId: 'thread_cancel',
+            turnId: 'turn_cancel',
+        });
+        await adapter.handleCodexNotification('turn/completed', {
+            threadId: 'thread_cancel',
+            turn: { id: 'turn_cancel', status: 'interrupted', error: null, items: [] },
+        });
+        await expect(promptPromise).resolves.toEqual({ stopReason: 'cancelled' });
+    });
+    it('declines command approvals when permission is rejected', async () => {
+        const { connection } = createMockConnection();
+        connection.requestPermission.mockResolvedValue({
+            outcome: { outcome: 'selected', optionId: 'reject_once' },
+        });
+        const { client: codexClient, mocks: codex } = createMockCodexClient();
+        const adapter = new CodexAppServerAcpAdapter(connection, codexClient);
+        await initializeAdapterWithDefaultModel(adapter, codex);
+        codex.request.mockResolvedValueOnce({
+            thread: { id: 'thread_decline', cwd: '/tmp/workspace' },
+            approvalPolicy: DEFAULT_APPROVAL_POLICY,
+            reasoningEffort: 'medium',
+        });
+        await adapter.newSession({
+            cwd: '/tmp/workspace',
+            mcpServers: [],
+        });
+        await adapter.handleCodexServerRequest({
+            id: 18,
+            method: 'item/commandExecution/requestApproval',
+            params: {
+                threadId: 'thread_decline',
+                turnId: 'turn_decline',
+                itemId: 'item_decline',
+                command: 'rm -rf tmp',
+            },
+        });
+        expect(codex.respondSuccess).toHaveBeenCalledWith(18, { decision: 'decline' });
+        expect(connection.sessionUpdate.mock.calls).toEqual(expect.arrayContaining([
+            [
+                expect.objectContaining({
+                    update: expect.objectContaining({
+                        sessionUpdate: 'tool_call_update',
+                        status: 'failed',
+                    }),
+                }),
+            ],
+        ]));
+    });
+    it('reconciles turn/completed notifications that arrive before prompt attaches active turn', async () => {
+        const { connection } = createMockConnection();
+        const { client: codexClient, mocks: codex } = createMockCodexClient();
+        const adapter = new CodexAppServerAcpAdapter(connection, codexClient);
+        await initializeAdapterWithDefaultModel(adapter, codex);
+        codex.request.mockResolvedValueOnce({
+            thread: { id: 'thread_race', cwd: '/tmp/workspace' },
+            approvalPolicy: DEFAULT_APPROVAL_POLICY,
+            reasoningEffort: 'medium',
+        });
+        const session = await adapter.newSession({
+            cwd: '/tmp/workspace',
+            mcpServers: [],
+        });
+        codex.request.mockImplementationOnce(async (method) => {
+            if (method !== 'turn/start') {
+                return {};
+            }
+            await adapter.handleCodexNotification('turn/completed', {
+                threadId: 'thread_race',
+                turn: { id: 'turn_race', status: 'completed', error: null, items: [] },
+            });
+            return {
+                turn: { id: 'turn_race', status: 'inProgress' },
+            };
+        });
+        await expect(adapter.prompt({
+            sessionId: session.sessionId,
+            prompt: [{ type: 'text', text: 'hello' }],
+        })).resolves.toEqual({ stopReason: 'end_turn' });
+    });
+    it('clears stale tool call state when a turn ends without item/completed events', async () => {
+        const { connection } = createMockConnection();
+        const { client: codexClient, mocks: codex } = createMockCodexClient();
+        const adapter = new CodexAppServerAcpAdapter(connection, codexClient);
+        await initializeAdapterWithDefaultModel(adapter, codex);
+        codex.request.mockResolvedValueOnce({
+            thread: { id: 'thread_stale_tools', cwd: '/tmp/workspace' },
+            approvalPolicy: DEFAULT_APPROVAL_POLICY,
+            reasoningEffort: 'medium',
+        });
+        const session = await adapter.newSession({
+            cwd: '/tmp/workspace',
+            mcpServers: [],
+        });
+        codex.request.mockResolvedValueOnce({
+            turn: { id: 'turn_stale_tools', status: 'inProgress' },
+        });
+        const promptPromise = adapter.prompt({
+            sessionId: session.sessionId,
+            prompt: [{ type: 'text', text: 'hello' }],
+        });
+        await vi.waitFor(() => {
+            expect(codex.request).toHaveBeenCalledWith('turn/start', expect.objectContaining({ threadId: 'thread_stale_tools' }));
+        });
+        await adapter.handleCodexNotification('item/started', {
+            threadId: 'thread_stale_tools',
+            turnId: 'turn_stale_tools',
+            item: {
+                type: 'commandExecution',
+                id: 'item_stale_tool',
+                status: 'inProgress',
+                command: 'echo hello',
+            },
+        });
+        const adapterSession = adapter.sessions.get(session.sessionId);
+        expect(adapterSession?.toolCallsByItemId.size).toBe(1);
+        await adapter.handleCodexNotification('turn/completed', {
+            threadId: 'thread_stale_tools',
+            turn: { id: 'turn_stale_tools', status: 'completed', error: null, items: [] },
+        });
+        await expect(promptPromise).resolves.toEqual({ stopReason: 'end_turn' });
+        expect(adapterSession?.activeTurn).toBeNull();
+        expect(adapterSession?.toolCallsByItemId.size).toBe(0);
+    });
+    it('responds with codex error payload when approval bridge throws', async () => {
+        const { connection } = createMockConnection();
+        connection.requestPermission.mockRejectedValueOnce(new Error('permission bridge unavailable'));
+        const { client: codexClient, mocks: codex } = createMockCodexClient();
+        const adapter = new CodexAppServerAcpAdapter(connection, codexClient);
+        await initializeAdapterWithDefaultModel(adapter, codex);
+        codex.request.mockResolvedValueOnce({
+            thread: { id: 'thread_approval_error', cwd: '/tmp/workspace' },
+            approvalPolicy: DEFAULT_APPROVAL_POLICY,
+            reasoningEffort: 'medium',
+        });
+        await adapter.newSession({
+            cwd: '/tmp/workspace',
+            mcpServers: [],
+        });
+        await expect(adapter.handleCodexServerRequest({
+            id: 32,
+            method: 'item/commandExecution/requestApproval',
+            params: {
+                threadId: 'thread_approval_error',
+                turnId: 'turn_approval_error',
+                itemId: 'item_approval_error',
+                command: 'echo test',
+            },
+        })).resolves.toBeUndefined();
+        expect(codex.respondError).toHaveBeenCalledWith(32, expect.objectContaining({
+            code: -32_600,
+            message: 'Failed to process codex approval request',
+        }));
+    });
+});
+//# sourceMappingURL=codex-app-server-acp-adapter.test.js.map