Skip to content

Update codex to 0.114.0 - #69

Closed
codex-acp-ci-app[bot] wants to merge 3 commits into
mainfrom
codex-update/0.114.0
Closed

Update codex to 0.114.0#69
codex-acp-ci-app[bot] wants to merge 3 commits into
mainfrom
codex-update/0.114.0

Conversation

@codex-acp-ci-app

Copy link
Copy Markdown

What's new

Finalized and committed using the codex-update-compat workflow.

Commit message: fix: resolve types and tests after Codex update

Previously failing checks and reasons:

  • src/__tests__/CodexACPAgent/CodexAcpClient.test.ts (should authenticate with key) failed because the data/auth-with-key.json snapshot no longer matched updated model/list output after the Codex upgrade (new gpt-5.4 entry, changed upgradeInfo, and updated migration markdown text/order).
  • TypeScript failed in src/CodexEventHandler.ts with TS2366 because createUpdateEvent did not explicitly handle all ServerNotification variants introduced by the regenerated types.

What I changed and why:

  • Updated src/CodexEventHandler.ts to explicitly handle new notification variants with intentional no-op cases:
    • command/exec/outputDelta
    • hook/started
    • hook/completed
      This restores exhaustive event handling for the updated schema and removes the type regression without adding a fallback.
  • Updated src/__tests__/CodexACPAgent/data/auth-with-key.json to the current model/list behavior so the authentication-with-key snapshot reflects the post-upgrade payload.

@ishulgin ishulgin closed this Mar 11, 2026
@ishulgin
ishulgin deleted the codex-update/0.114.0 branch March 11, 2026 13:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant