🤖 fix: enable GPT-6 Astra Pro mode through Coder - #4124
Conversation
Allow Pro mode on Coder's OpenAI Responses instances without broadening the Fast mode or incompatible gateway gates. Keep raw instance metadata authoritative and verify the picker, persistence, and native SDK request serialization. --- _Generated with [`mux`](https://github.com/coder/mux) • Model: `coder:openai/gpt-6-astra` • Thinking: `high` • Cost: `$20.61`_ <!-- mux-attribution: model=coder:openai/gpt-6-astra thinking=high costs=20.61 -->
|
@codex review |
|
@codex security review |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: c088b822ce
ℹ️ 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".
Resolve mapped Pro capabilities, preserve type-derived identities on direct fallback, and honor catalog exclusions/removals before restoring Coder routes. Pin the phone story's container to Pixel's width and update advisor coverage for the newly supported Coder route. --- _Generated with [`mux`](https://github.com/coder/mux) • Model: `coder:openai/gpt-6-astra` • Thinking: `high` • Cost: `$36.93`_ <!-- mux-attribution: model=coder:openai/gpt-6-astra thinking=high costs=36.93 -->
|
@codex review |
|
@codex security review |
This comment has been minimized.
This comment has been minimized.
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: c7c18c408d
ℹ️ 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".
Honor policy-aware explicit gateway fallback in the picker and palette without letting Treat-as metadata affect routing. Preserve scoped Pro capabilities and actual Coder instance identity through nested advisor option construction. Validated with 440 targeted tests, make static-check, both Astra Pro Storybook plays, and recorded phone/desktop browser checks. --- _Generated with [`mux`](https://github.com/coder/mux) • Model: `coder:openai/gpt-6-astra` • Thinking: `high` • Cost: `$6.17`_ <!-- mux-attribution: model=coder:openai/gpt-6-astra thinking=high costs=6.17 -->
|
@codex review |
|
@codex security review |
This comment has been minimized.
This comment has been minimized.
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: b4e31ab879
ℹ️ 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".
Project nested provider options from the same raw snapshot used for model creation so catalog refreshes cannot change the instance wire or scoped aliases. Cover retyped and removed metadata, update routing-hook test doubles, and restore routing/minimum hooks so the Settings suites do not leak those mocks. Validation: 587 combined UI/provider tests and make static-check on Bun 1.3.5. --- _Generated with [`mux`](https://github.com/coder/mux) • Model: `coder:openai/gpt-6-astra` • Thinking: `high` • Cost: `$42.99`_ <!-- mux-attribution: model=coder:openai/gpt-6-astra thinking=high costs=42.99 -->
|
@codex review |
|
@codex security review |
This comment has been minimized.
This comment has been minimized.
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 83cc9698ab
ℹ️ 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".
Create one request receipt with effective route, wire, scoped metadata and provider options. Continuous summaries and nested tools consume that receipt instead of rebuilding routing from a raw prefix or live configuration. Preserve unset-thinking variant behavior and leave metadata-only callers alone. Cover Coder aliases, unavailable/catalog fallbacks, protocol overrides, configuration/route races and nested advisor behavior. Validated with 891 combined tests and make static-check on Bun 1.3.5. --- _Generated with [`mux`](https://github.com/coder/mux) • Model: `coder:openai/gpt-6-astra` • Thinking: `high` • Cost: `$52.52`_ <!-- mux-attribution: model=coder:openai/gpt-6-astra thinking=high costs=52.52 -->
|
@codex review |
|
@codex security review |
This comment has been minimized.
This comment has been minimized.
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 9e997980a6
ℹ️ 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".
Extend the existing disabled Coder shell with parsed instance names/types so persisted selections can resolve allowed upstream fallback capabilities. Keep forbidden Coder unconfigured, unselectable and absent from provider lists; expose no model catalog, endpoint settings or authentication data. Preserve credential-presence revocation and custom-provider shadow semantics. Validated with 895 combined tests and make static-check on Bun 1.3.5. --- _Generated with [`mux`](https://github.com/coder/mux) • Model: `coder:openai/gpt-6-astra` • Thinking: `high` • Cost: `$73.91`_ <!-- mux-attribution: model=coder:openai/gpt-6-astra thinking=high costs=73.91 -->
|
@codex review |
|
@codex security review |
|
Codex Review: Didn't find any major issues. Breezy! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
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". |
This comment has been minimized.
This comment has been minimized.
Spy on the summary model's migrated creation method and return its full request receipt. Preserve plugin policy, cancellation, cleanup and usage assertions; leave metadata-only callers and production code unchanged. Validated build-main, all 13 isolated CI suites (440 tests), and static checks. --- _Generated with `xum` • Model: `coder:openai/gpt-6-astra` • Thinking: `high` • Cost: `$92.28`_ <!-- mux-attribution: model=coder:openai/gpt-6-astra thinking=high costs=92.28 -->
|
@codex review |
|
@codex security review |
This comment has been minimized.
This comment has been minimized.
|
@codex review |
|
Codex Review: Didn't find any major issues. Already looking forward to the next diff. Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
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". |
This comment has been minimized.
This comment has been minimized.
## Summary Version bump for the v0.28.5 patch release. Headline changes since v0.28.4: remote server connections in the desktop app (coder#4101), self-updating `xum server` under a restart supervisor (coder#4083, coder#4127), first-class GPT-6 Astra and Astra Pro support including Codex OAuth routing (coder#4064, coder#4094, coder#4106, coder#4124), token-budget context window rollovers (coder#4097), the workspace remembering model and mode on send (coder#3968), in-place plugin updates (coder#4164), the optional flat sidebar chat list (coder#3994), and copying selected chat text as Markdown (coder#4170). It also carries a long run of streaming, compaction, and task-lifecycle fixes (reconnect streaming coder#4123, message edits during active streams coder#4153, Codex OAuth prompt-cache routing coder#4159, compaction/history fencing coder#4133 through coder#4148, task lock ordering coder#4161) plus the Effect Wave 4 runtime refactors and deslop passes 1 through 3. ## Implementation Bumped with `node ./scripts/set-package-version.js 0.28.5` so the root `package.json` and the legacy `packages/mux-compat` forwarding package stay version-locked. `src/common/compat/productIdentity.test.ts` passes locally (8/8). After this PR merges, the `v0.28.5` tag will be applied to the squash commit and the GitHub Release published to trigger the desktop/npm/docker pipelines. --- _Generated with `xum` • Model: `anthropic:claude-fable-5-1` • Thinking: `xhigh` • Cost: `$1.64`_ <!-- mux-attribution: model=anthropic:claude-fable-5-1 thinking=xhigh costs=1.64 -->
Summary
Enable Pro mode for GPT-6 Astra through Coder's OpenAI Responses route. Astra was already Pro-capable, but the shared direct-only gate hid the toggle and the request builder dropped the selection for every gateway.
Implementation
Validation
reasoning.mode: "pro"field.--failOnConsoleafter rebasing onto current main.make static-checkpassed. Final local self-review found no remaining actionable issues after addressing all independent-review findings.Dogfood screenshots and recording
routed-phone.webm
Validation limitation
The full local shared-coverage run was inconclusive: one unchanged reconnect assertion failed (then passed narrowly), and the batch produced no terminal Bun summary. Final-head CI subsequently passed the complete shared-unit run, all required checks, and visual-test execution.
Risks
The change affects Pro eligibility and request-option assembly for Coder Responses, nested tools, and continuous compaction. Existing metadata-only status/dream/refine callers retain their behavior. Instance-type, unsupported-model, custom-provider-shadowing, gateway, and wire-format regression cases guard against exposing a toggle that cannot affect requests.
Generated with
mux• Model:coder:openai/gpt-6-astra• Thinking:high• Cost:$177.56