test(codex): type resume approval request mock

LaPhilosophie · LaPhilosophie · commit 8e1f9b389092 · 2026-05-20T15:12:18.000+08:00
diff --git a/extensions/codex/src/node-cli-sessions.test.ts b/extensions/codex/src/node-cli-sessions.test.ts
@@ -189,7 +189,10 @@ describe("codex cli node sessions", () => {
     }
     const invokeNode = vi.fn(async () => ({ ok: true as const, payload: { ok: true } }));
     const approvals = {
-      request: vi.fn(async () => ({ id: "approval-1", decision: "deny" as const })),
+      request: vi.fn(async (_request: Record<string, unknown>) => ({
+        id: "approval-1",
+        decision: "deny" as const,
+      })),
     };
 
     const result = await policy.handle({
@@ -220,8 +223,8 @@ describe("codex cli node sessions", () => {
         allowedDecisions: ["allow-once", "deny"],
       }),
     );
-    const [approvalRequest] = approvals.request.mock.calls[0] ?? [];
-    expect((approvalRequest as { timeoutMs?: unknown } | undefined)?.timeoutMs).toBeUndefined();
+    const approvalRequest = approvals.request.mock.calls[0]?.[0];
+    expect(approvalRequest?.timeoutMs).toBeUndefined();
   });
 
   it("does not treat allow-always as a valid Codex CLI resume approval", async () => {
