From 80d72a4c6fe61067cdcaf484a58fbf4899340290 Mon Sep 17 00:00:00 2001 From: luckyPipewrench Date: Sat, 1 Aug 2026 12:34:24 -0400 Subject: [PATCH] fix(server): recover stale Codex approval callbacks --- .../orchestration/Layers/ProviderCommandReactor.test.ts | 6 +++--- .../src/orchestration/Layers/ProviderCommandReactor.ts | 8 +++++--- 2 files changed, 8 insertions(+), 6 deletions(-) diff --git a/apps/server/src/orchestration/Layers/ProviderCommandReactor.test.ts b/apps/server/src/orchestration/Layers/ProviderCommandReactor.test.ts index e4661061b23..a3feb39aa41 100644 --- a/apps/server/src/orchestration/Layers/ProviderCommandReactor.test.ts +++ b/apps/server/src/orchestration/Layers/ProviderCommandReactor.test.ts @@ -2566,15 +2566,15 @@ describe("ProviderCommandReactor", () => { }); }); - it("surfaces stale provider approval request failures without faking approval resolution", async () => { + it("normalizes stale Codex approval callbacks without faking approval resolution", async () => { const harness = await createHarness(); const now = "2026-01-01T00:00:00.000Z"; harness.respondToRequest.mockImplementation(() => Effect.fail( new ProviderAdapterRequestError({ provider: ProviderDriverKind.make("codex"), - method: "session/request_permission", - detail: "Unknown pending permission request: approval-request-1", + method: "item/requestApproval/decision", + detail: "Unknown pending Codex approval request: approval-request-1", }), ), ); diff --git a/apps/server/src/orchestration/Layers/ProviderCommandReactor.ts b/apps/server/src/orchestration/Layers/ProviderCommandReactor.ts index 1135fd579e7..f88278d3df1 100644 --- a/apps/server/src/orchestration/Layers/ProviderCommandReactor.ts +++ b/apps/server/src/orchestration/Layers/ProviderCommandReactor.ts @@ -187,13 +187,15 @@ function isUnknownPendingApprovalRequestError(cause: Cause.Cause