Skip to content

fix: Bedrock GPT-5.4 reasoning levels#19461

Merged
celia-oai merged 4 commits intomainfrom
codex/fix-bedrock-54-reasoning-levels
Apr 25, 2026
Merged

fix: Bedrock GPT-5.4 reasoning levels#19461
celia-oai merged 4 commits intomainfrom
codex/fix-bedrock-54-reasoning-levels

Conversation

@celia-oai
Copy link
Copy Markdown
Collaborator

@celia-oai celia-oai commented Apr 24, 2026

Why

When using the Amazon Bedrock provider with openai.gpt-5.4-cmb, the model picker allowed xhigh because the CMB catalog entry was derived from the bundled gpt-5.4 reasoning metadata. Bedrock rejects that effort level, causing the request to fail before the turn can run:

{"error":{"code":"validation_error","message":"Failed to deserialize the JSON body into the target type: Invalid 'reasoning': Invalid 'effort': unknown variant `xhigh`, expected one of `high`, `low`, `medium`, `minimal` at line 1 column 77239","param":null,"type":"invalid_request_error"}}

What Changed

  • Replace the runtime lookup of bundled gpt-5.4 metadata for openai.gpt-5.4-cmb with an explicit Bedrock CMB ModelInfo entry.
  • Advertise only the Bedrock-supported CMB reasoning levels: minimal, low, medium, and high.
  • Keep the existing GPT OSS Bedrock model metadata and reasoning levels unchanged.
  • Add catalog coverage for the hardcoded CMB metadata and Bedrock-compatible reasoning level list.

@celia-oai celia-oai changed the title Fix Bedrock GPT-5.4 reasoning levels fix: Bedrock GPT-5.4 reasoning levels Apr 24, 2026
@celia-oai celia-oai requested a review from pakrym-oai April 24, 2026 23:41
@celia-oai celia-oai enabled auto-merge (squash) April 24, 2026 23:59
@celia-oai celia-oai merged commit d19de6d into main Apr 25, 2026
25 checks passed
@celia-oai celia-oai deleted the codex/fix-bedrock-54-reasoning-levels branch April 25, 2026 00:05
@github-actions github-actions Bot locked and limited conversation to collaborators Apr 25, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants