Skip to content

fix: enable Codex Astra Max and Ultra reasoning - #406

Merged
lodystage[bot] merged 2 commits into
mainfrom
feat/codex-provider-gpt-56-max-ultra-gpt-6-astra
Sep 5, 2026
Merged

fix: enable Codex Astra Max and Ultra reasoning#406
lodystage[bot] merged 2 commits into
mainfrom
feat/codex-provider-gpt-56-max-ultra-gpt-6-astra

Conversation

@lodystage

@lodystage lodystage Bot commented Sep 5, 2026

Copy link
Copy Markdown
Contributor

Problem / pressure

The built-in Codex UI removed Max and Ultra from gpt-6-astra because its reasoning normalization recognized only three GPT-5.6 model IDs. It also offered Ultra for Luna, although the current ACP model catalog advertises only Max for that model.

Summary

Use an exact per-model extended-effort map: Astra, Sol, and Terra support Max/Ultra; Luna supports Max. Preserve upstream labels and descriptions, fill missing supported options in cached selectors, and remove unsupported efforts when the selected model changes. Older and unknown models keep their existing restrictions.

Test plan

  • 53 tests passed across acp-selector-options, acp-session-config-selection, and session-config-selection-oscillation.
  • Full workspace typechecking (pnpm typecheck) and pnpm --filter @lody/components typecheck passed.
  • Full workspace and scoped type-aware lint completed with zero errors; changed-file Prettier checks passed.
  • Public repository boundary check passed.
  • Full pnpm check was stopped after repeated failures in untouched UI test suites reporting act is not a function. Reproduced independently with pnpm --filter @lody/components test tests/agent-config-dialog.test.tsx --bail=1: it fails at the test renderer's act() call before rendering. The full suite and later checks therefore did not complete.
  • Confirmed the model/effort matrix through the current machine's ACP capabilities exposed by Lody's session-options interface; committed fixtures are synthetic.
  • CI follow-up: both original jobs stopped at dependency installation with ERR_PNPM_OUTDATED_LOCKFILE. Align the root lockfile specifier with the Codex submodule's ^0.153.4 requirement; the resolved version stays 0.153.4. pnpm install --frozen-lockfile, pnpm format:check, and all 53 relevant tests passed locally after this one-line fix.

Align extended reasoning with the ACP model catalog, including Max-only support for Luna.

Model: gpt-6-astra
Match the submodule's ^0.153.4 requirement so frozen installs can run in both CI jobs. The resolved package remains 0.153.4.

Model: gpt-6-astra
@lodystage
lodystage Bot merged commit 673a884 into main Sep 5, 2026
5 checks passed
@lodystage
lodystage Bot deleted the feat/codex-provider-gpt-56-max-ultra-gpt-6-astra branch September 5, 2026 03:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant