Skip to content

BYE-340 route GPT-5.6 labels through Codex runtime - #1371

Open
b-yelverton wants to merge 3 commits into
cyrusagents:mainfrom
b-yelverton:bye-340-gpt56-runtime-routing
Open

BYE-340 route GPT-5.6 labels through Codex runtime#1371
b-yelverton wants to merge 3 commits into
cyrusagents:mainfrom
b-yelverton:bye-340-gpt56-runtime-routing

Conversation

@b-yelverton

@b-yelverton b-yelverton commented Jul 12, 2026

Copy link
Copy Markdown

What changed

  • Map the terra, luna, sol, and bare gpt-5.6 labels to the requested GPT-5.6 model IDs and select the Codex runner.
  • Carry bare or effort:-prefixed low/medium/high labels through runner config, defaulting to medium.
  • Serialize the selected effort into the Codex app-server config so it reaches OpenAI as reasoning.effort.
  • Fail loudly for recognized GPT-5.6 labels that conflict with a Claude-family selector or an existing non-Codex session.
  • Log unknown model-family-shaped labels before applying the normal fallback.

Root cause

RunnerSelectionService only recognized gpt-* model labels. terra/luna/sol therefore selected the default Claude runner, and the existing Codex modelReasoningEffort field was never serialized by AppServerCodexBackend.

Validation

  • pnpm install --frozen-lockfile
  • pnpm build
  • pnpm typecheck
  • pnpm test:packages:run — all package tests green; EdgeWorker 66 files / 744 tests and Codex 12 files / 71 tests
  • pnpm --filter './apps/*' test:run — CLI 5 files / 94 tests; F1 no-test pass
  • targeted Biome check and git diff --check

Linear: BYE-340

@b-yelverton
b-yelverton marked this pull request as ready for review July 21, 2026 17:34

@Connoropolous Connoropolous 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.

this looks great, just ask that the change in default reasoning effort, and documentation on how to use reasoning overrides via labels make it into the changelog entry

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.

2 participants