Repository navigation
fix(bedrock): improve authentication and Blue/Red support - #1188
Merged
Merged
Conversation
mldangelo-oai
marked this pull request as ready for review
October 2, 2026 06:09
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
soyeon-oai
approved these changes
Oct 2, 2026
Merged
3 tasks done
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Native Amazon Bedrock scans could receive an unrelated ChatGPT account-access advisory, and AWS failures could lose diagnostic details or suggest the wrong authentication recovery. Bedrock scans now preserve provider diagnostics, give relevant authentication guidance, and support cost estimates and existing budget checks for both Daybreak Blue and Red.
Changes
openai.gpt-daybreak-blue-5.6-solandopenai.gpt-5.6-cyberat AWS Standard commercial in-region rates. Preserve Red's fractional-nanodollar cache-write price, including odd token counts. Blue retains its context-price range; Red has no invented long-context upper estimate.Testing
12345and random seed3649136333each passed with 3,416 tests passed, 51 platform skips, zero failures, and 68,160 assertions across 158 files.Risk and rollout
Inference continues to use native Codex authentication and transport. Enrollment and AWS provisioning for the chosen model remain required. Provisioned Bedrock inference and local results need no additional OpenAI CLI login. Hosted publication retains separate authentication requirements and was not tested.
The model-metadata correction must ship in a Codex release and then be included in the bundled dependency. This change does not claim the current bundled release contains that correction. Live QA with a patched native development executable is recorded separately from testing the bundled executable.
Pricing follows the AWS model cards for Blue and Red. Estimates and budget limits are not guaranteed AWS billing totals.
Public disclosure review