Skip to content

feat: change amazon Bedrock GPT-5.6 display names#31636

Merged
celia-oai merged 2 commits into
mainfrom
dev/cc/change-name
Jul 8, 2026
Merged

feat: change amazon Bedrock GPT-5.6 display names#31636
celia-oai merged 2 commits into
mainfrom
dev/cc/change-name

Conversation

@celia-oai

@celia-oai celia-oai commented Jul 8, 2026

Copy link
Copy Markdown
Collaborator

Why

Amazon Bedrock's GPT-5.6 variants currently appear as only Sol, Terra, and Luna. Those labels omit the model family and version, making them ambiguous in model lists and inconsistent with the naming of other GPT models.

What changed

  • Rename the three Bedrock model display names to GPT-5.6 Sol, GPT-5.6 Terra, and GPT-5.6 Luna.
  • Strengthen the Bedrock model-manager test to verify both model IDs and their propagated display names.

Model IDs, ordering, priorities, reasoning support, and default selection are unchanged.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 9c3014f367

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread codex-rs/tui/src/chatwidget/tests/popups_and_settings.rs Outdated
@celia-oai celia-oai force-pushed the dev/cc/change-name branch from 9c3014f to 2dfef46 Compare July 8, 2026 20:30
@celia-oai celia-oai changed the title changes Qualify Amazon Bedrock GPT-5.6 display names Jul 8, 2026
@celia-oai celia-oai changed the title Qualify Amazon Bedrock GPT-5.6 display names feat: change amazon Bedrock GPT-5.6 display names Jul 8, 2026
@celia-oai celia-oai requested a review from owenlin0 July 8, 2026 20:33
@celia-oai celia-oai enabled auto-merge (squash) July 8, 2026 20:35
@celia-oai celia-oai disabled auto-merge July 8, 2026 20:35
@celia-oai celia-oai enabled auto-merge (squash) July 8, 2026 20:38
AMAZON_BEDROCK_GPT_5_6_SOL_MODEL_ID,
"Sol",
"GPT-5.6 Sol",
/*priority*/ 2,

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should the 5.6 models be the highest priority? In the Codex service it seems we put the 5.6 models at the top of the list (in Sol, Terra, Luna order)

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oh good call!

@celia-oai celia-oai disabled auto-merge July 8, 2026 20:44
@celia-oai celia-oai merged commit bff9c49 into main Jul 8, 2026
35 checks passed
@celia-oai celia-oai deleted the dev/cc/change-name branch July 8, 2026 20:53
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 8, 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.

3 participants