Fix types and tests after Codex update compatibility changes

Author: github-actions[bot]

src/CodexEventHandler.ts

@@ -133,6 +133,7 @@ export class CodexEventHandler {
             case "thread/realtime/itemAdded":
             case "thread/realtime/transcriptUpdated":
             case "thread/realtime/outputAudio/delta":
+            case "thread/realtime/sdp":
             case "thread/realtime/error":
             case "thread/realtime/closed":
             case "windowsSandbox/setupCompleted":

src/__tests__/CodexACPAgent/CodexAcpClient.test.ts

@@ -229,6 +229,7 @@ describe('ACP server test', { timeout: 40_000 }, () => {
                 defaultReasoningEffort: "medium",
                 inputModalities: ["text"],
                 supportsPersonality: false,
+                additionalSpeedTiers: [],
                 isDefault: true
             }],
             nextCursor: null
@@ -524,7 +525,15 @@ describe('ACP server test', { timeout: 40_000 }, () => {
 
         vi.spyOn(codexAppServerClient, "awaitTurnCompleted").mockResolvedValue({
             threadId: "session-id",
-            turn: { id: "turn-id", items: [], status: "completed", error: null }
+            turn: {
+                id: "turn-id",
+                items: [],
+                status: "completed",
+                error: null,
+                startedAt: null,
+                completedAt: null,
+                durationMs: null,
+            }
         });
 
         const sessionState: SessionState = createTestSessionState();
@@ -735,6 +744,7 @@ describe('ACP server test', { timeout: 40_000 }, () => {
             ],
             defaultReasoningEffort: 'medium',
             supportsPersonality: false,
+            additionalSpeedTiers: [],
             isDefault: false,
             inputModalities: []
         },
@@ -752,6 +762,7 @@ describe('ACP server test', { timeout: 40_000 }, () => {
             ],
             defaultReasoningEffort: 'low',
             supportsPersonality: false,
+            additionalSpeedTiers: [],
             isDefault: true,
             inputModalities: []
         }
@@ -775,11 +786,27 @@ describe('ACP server test', { timeout: 40_000 }, () => {
         const mockFixture = createCodexMockTestFixture();
         const sessionState = createTestSessionState(sessionOverrides);
         const turnStartSpy = vi.spyOn(mockFixture.getCodexAppServerClient(), "turnStart").mockResolvedValue({
-            turn: { id: "turn-id", items: [], status: "inProgress", error: null }
+            turn: {
+                id: "turn-id",
+                items: [],
+                status: "inProgress",
+                error: null,
+                startedAt: null,
+                completedAt: null,
+                durationMs: null,
+            }
         });
         vi.spyOn(mockFixture.getCodexAppServerClient(), "awaitTurnCompleted").mockResolvedValue({
             threadId: sessionState.sessionId,
-            turn: { id: "turn-id", items: [], status: "completed", error: null }
+            turn: {
+                id: "turn-id",
+                items: [],
+                status: "completed",
+                error: null,
+                startedAt: null,
+                completedAt: null,
+                durationMs: null,
+            }
         });
         vi.spyOn(mockFixture.getCodexAcpAgent(), "getSessionState").mockReturnValue(sessionState);
         return { mockFixture, sessionState, turnStartSpy };

src/__tests__/CodexACPAgent/list-sessions.test.ts

@@ -14,6 +14,7 @@ describe("CodexACPAgent - list sessions", () => {
 
         const threadA: Thread = {
             id: "sess-1",
+            forkedFromId: null,
             preview: "First session",
             ephemeral: false,
             modelProvider: "openai",
@@ -32,6 +33,7 @@ describe("CodexACPAgent - list sessions", () => {
         };
         const threadB: Thread = {
             id: "sess-2",
+            forkedFromId: null,
             preview: "Other session",
             ephemeral: false,
             modelProvider: "openai",

src/__tests__/CodexACPAgent/load-session.test.ts

@@ -33,6 +33,7 @@ describe("CodexACPAgent - loadSession", () => {
             defaultReasoningEffort: "medium",
             inputModalities: ["text", "image"],
             supportsPersonality: false,
+            additionalSpeedTiers: [],
             isDefault: true,
         };
 
@@ -43,6 +44,7 @@ describe("CodexACPAgent - loadSession", () => {
 
         const thread: Thread = {
             id: "session-1",
+            forkedFromId: null,
             preview: "Hi",
             ephemeral: false,
             modelProvider: "openai",
@@ -62,6 +64,9 @@ describe("CodexACPAgent - loadSession", () => {
                     id: "turn-1",
                     status: "completed",
                     error: null,
+                    startedAt: null,
+                    completedAt: null,
+                    durationMs: null,
                     items: [
                         {
                             type: "userMessage",
@@ -192,6 +197,7 @@ describe("CodexACPAgent - loadSession", () => {
             defaultReasoningEffort: "medium",
             inputModalities: ["text"],
             supportsPersonality: false,
+            additionalSpeedTiers: [],
             isDefault: true,
         };
 
@@ -202,6 +208,7 @@ describe("CodexACPAgent - loadSession", () => {
         codexAppServerClient.threadResume = vi.fn().mockResolvedValue({
             thread: {
                 id: "session-1",
+                forkedFromId: null,
                 preview: "",
                 ephemeral: false,
                 modelProvider: "openai",
@@ -263,6 +270,7 @@ describe("CodexACPAgent - loadSession", () => {
             defaultReasoningEffort: "medium",
             inputModalities: ["text"],
             supportsPersonality: false,
+            additionalSpeedTiers: [],
             isDefault: true,
         };
 
@@ -273,6 +281,7 @@ describe("CodexACPAgent - loadSession", () => {
         codexAppServerClient.threadResume = vi.fn().mockResolvedValue({
             thread: {
                 id: "session-1",
+                forkedFromId: null,
                 preview: "",
                 ephemeral: false,
                 modelProvider: "openai",

src/__tests__/CodexACPAgent/mcp-session.test.ts

@@ -1,30 +1,31 @@
 // noinspection ES6RedundantAwait
 
 import {describe, expect, it, vi, beforeEach} from 'vitest';
-import {createTestFixture, type TestFixture} from "../acp-test-utils";
-import type {McpServerStdio} from "@agentclientprotocol/sdk";
+import {createCodexMockTestFixture, createTestSessionState, type CodexMockTestFixture} from "../acp-test-utils";
 
 describe('MCP session configuration', { timeout: 40_000 }, () => {
 
-    let fixture: TestFixture;
+    let fixture: CodexMockTestFixture;
     beforeEach(() => {
-        fixture = createTestFixture();
+        fixture = createCodexMockTestFixture();
         vi.clearAllMocks();
     });
 
 
     it('should return configured mcp', async () => {
         const codexAcpAgent = fixture.getCodexAcpAgent();
-        await codexAcpAgent.initialize({protocolVersion: 1});
-
-        fixture.getCodexAcpClient().authRequired = vi.fn().mockResolvedValue(false);
-        const mcpServer: McpServerStdio = {
-            name: "test-mcp", command: "./node_modules/.bin/mcp-hello-world", args: ["example"], env: [{name:"example", value: "example"}]
-        };
-
-        const newSessionResponse = await codexAcpAgent.newSession({cwd: "", mcpServers: [mcpServer]});
-        fixture.clearAcpConnectionDump();
-        await codexAcpAgent.prompt({sessionId: newSessionResponse.sessionId, prompt: [{type: "text", text: "/mcp"}]});
+        const sessionState = createTestSessionState({
+            sessionId: "session-id",
+            sessionMcpServers: ["test-mcp"],
+        });
+        vi.spyOn(codexAcpAgent, "getSessionState").mockReturnValue(sessionState);
+
+        vi.spyOn(fixture.getCodexAcpClient(), "listMcpServers").mockResolvedValue({
+            data: [],
+            nextCursor: null,
+        });
+
+        await codexAcpAgent.prompt({sessionId: "session-id", prompt: [{type: "text", text: "/mcp"}]});
         const transportDump = fixture.getAcpConnectionDump([]);
         expect(transportDump).contain("Configured MCP servers:");
         expect(transportDump).contain("- test-mcp");

src/__tests__/CodexACPAgent/model-filtering.test.ts

@@ -25,6 +25,7 @@ describe("Model filtering", () => {
                 supportedReasoningEfforts: efforts,
                 defaultReasoningEffort: "medium",
                 supportsPersonality: false,
+                additionalSpeedTiers: [],
                 isDefault: false,
                 inputModalities: []
             },
@@ -40,6 +41,7 @@ describe("Model filtering", () => {
                 supportedReasoningEfforts: [defaultEffort],
                 defaultReasoningEffort: "medium",
                 supportsPersonality: false,
+                additionalSpeedTiers: [],
                 isDefault: false,
                 inputModalities: []
             },
@@ -55,6 +57,7 @@ describe("Model filtering", () => {
                 supportedReasoningEfforts: [defaultEffort],
                 defaultReasoningEffort: "medium",
                 supportsPersonality: false,
+                additionalSpeedTiers: [],
                 isDefault: false,
                 inputModalities: []
             },
@@ -70,6 +73,7 @@ describe("Model filtering", () => {
                 supportedReasoningEfforts: [defaultEffort],
                 defaultReasoningEffort: "medium",
                 supportsPersonality: false,
+                additionalSpeedTiers: [],
                 isDefault: false,
                 inputModalities: []
             },