Skip to content

feat: map interchange's neutral effort to codex reasoning.effort - #35

Merged
TheGreatAxios merged 1 commit into
mainfrom
feat/map-inference-effort
Oct 29, 2026
Merged

TheGreatAxios merged 1 commit into
mainfrom
feat/map-inference-effort

Conversation

@TheGreatAxios

Copy link
Copy Markdown
Contributor

Closes #33

Stacked on #32 (base feat/account-id-quirk).

What changed

Verification

  • bun run check (typecheck, lint, format, 21 unit tests) passes.
  • New tests pin the per-model max, the unlisted-model fallback, and option precedence (non-empty wins, none suppresses, empty falls through).
  • Independent review findings addressed (empty-option precedence, comment accuracy, return type, tests, docs).

max sends the highest level each model lists in the Codex CLI catalog (openai/codex@b1e72963c3, supported_reasoning_levels), else high. An explicit codexReasoningEffort option still wins.
@corbits-triage

corbits-triage Bot commented Oct 29, 2026 •

Copy link
Copy Markdown

Triage: In progress

Triage is waiting for the decision model's answer before it decides. This comment shows the verdict once it has.

@TheGreatAxios
TheGreatAxios merged commit 2dd3008 into main Oct 29, 2026
4 checks passed
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.

Map Interchange's neutral effort to Codex reasoning.effort

1 participant