Thanks to visit codestin.com
Credit goes to github.com

Skip to content

🤖 fix: enable GPT-6 Astra Pro mode through Coder - #4124

Merged
ThomasK33 merged 7 commits into
mainfrom
pro-mode-9ytp
Sep 7, 2026
Merged

🤖 fix: enable GPT-6 Astra Pro mode through Coder#4124
ThomasK33 merged 7 commits into
mainfrom
pro-mode-9ytp

Conversation

@ThomasK33

@ThomasK33 ThomasK33 commented Sep 7, 2026

Copy link
Copy Markdown
Member

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

  • Allow Coder's actual OpenAI Responses instances, including custom-named instances, using their provider type rather than the instance name.
  • Retain only instance name/type metadata in the existing disabled policy-hidden Coder shell, so allowed fallbacks work without exposing endpoints, models, or credentials.
  • Preserve explicit gateway precedence and fallback behavior, the direct-only Fast mode gate, and restrictions for incompatible gateways, Chat Completions, and Codex OAuth.
  • Use the policy-aware effective route in the picker and palette. Keep scoped Treat-as mappings capability-only, and preserve actual Coder instance identity for nested advisors.
  • Share one creation-time model result across nested tools and continuous compaction, carrying the effective route, scoped identity, actual wire format, and provider-config snapshot into request construction.
  • Send the existing native OpenAI reasoning mode option without changing the selected reasoning effort.

Validation

  • 1,063 targeted provider-options, agent-settings, thinking-policy, routing, palette, AI-service, advisor, model-factory, continuous-compaction, Intuition, provider-service, and Settings UI tests passed, including capture of the real SDK's outgoing reasoning.mode: "pro" field.
  • All 13 isolated CI-prefix suites passed (440 tests), including the corrected plugin-compaction fixture; an API/MutexMap coverage rerun passed 31 tests.
  • Both full-app Storybook interactions passed with --failOnConsole after rebasing onto current main.
  • Manually verified pointer and keyboard toggles in Storybook manager at 375×667, 390×844, and 1280×800; High remains selected, Pro persists, and the menu stays within bounds.
  • make static-check passed. Final local self-review found no remaining actionable issues after addressing all independent-review findings.
  • Verification was offline; no billable model requests were sent.
Dogfood screenshots and recording

Desktop: Astra with Pro mode checked

Phone: Astra with Pro mode checked

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

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 -->
@ThomasK33

Copy link
Copy Markdown
Member Author

@codex review

@ThomasK33

Copy link
Copy Markdown
Member Author

@codex security review

@chatgpt-codex-connector

This comment has been minimized.

@chatgpt-codex-connector

This comment has been minimized.

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

Copy link
Copy Markdown

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: 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".

Comment thread src/common/utils/ai/proMode.ts Outdated
Comment thread src/common/utils/ai/proMode.ts Outdated
Comment thread src/browser/stories/App.astraPro.stories.tsx Outdated
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 -->
@ThomasK33

Copy link
Copy Markdown
Member Author

@codex review

@ThomasK33

Copy link
Copy Markdown
Member Author

@codex security review

@chatgpt-codex-connector

This comment has been minimized.

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

Copy link
Copy Markdown

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: 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".

Comment thread src/common/utils/ai/openaiProviderOptionsAvailability.ts
Comment thread src/common/utils/ai/proMode.ts Outdated
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 -->
@ThomasK33

Copy link
Copy Markdown
Member Author

@codex review

@ThomasK33

Copy link
Copy Markdown
Member Author

@codex security review

@chatgpt-codex-connector

This comment has been minimized.

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

Copy link
Copy Markdown

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: 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".

Comment thread src/node/services/turnRequestBuilder.ts Outdated
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 -->
@ThomasK33

Copy link
Copy Markdown
Member Author

@codex review

@ThomasK33

Copy link
Copy Markdown
Member Author

@codex security review

@chatgpt-codex-connector

This comment has been minimized.

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

Copy link
Copy Markdown

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: 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".

Comment thread src/common/utils/ai/providerOptions.ts
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 -->
@ThomasK33

Copy link
Copy Markdown
Member Author

@codex review

@ThomasK33

Copy link
Copy Markdown
Member Author

@codex security review

@chatgpt-codex-connector

This comment has been minimized.

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

Copy link
Copy Markdown

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: 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".

Comment thread src/common/utils/ai/proMode.ts
Comment thread src/common/utils/ai/proMode.ts
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 -->
@ThomasK33

Copy link
Copy Markdown
Member Author

@codex review

@ThomasK33

Copy link
Copy Markdown
Member Author

@codex security review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Breezy!

Reviewed commit: 8c5204f594

ℹ️ 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".

@chatgpt-codex-connector

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 -->
@ThomasK33

Copy link
Copy Markdown
Member Author

@codex review

@ThomasK33

Copy link
Copy Markdown
Member Author

@codex security review

@chatgpt-codex-connector

This comment has been minimized.

@ThomasK33

Copy link
Copy Markdown
Member Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Already looking forward to the next diff.

Reviewed commit: eb1cc446d2

ℹ️ 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".

@chatgpt-codex-connector

This comment has been minimized.

@ThomasK33
ThomasK33 added this pull request to the merge queue Sep 7, 2026
Merged via the queue into main with commit 5f81578 Sep 7, 2026
19 of 20 checks passed
@ThomasK33
ThomasK33 deleted the pro-mode-9ytp branch September 7, 2026 21:48
yermakoffivan pushed a commit to yermakoffivan/mux that referenced this pull request Sep 9, 2026
## 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 -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant