Skip to content

fix: preserve reasoning capabilities for routed model aliases - #21

Open
xhwxt wants to merge 1 commit into
BigStrongSun:mainfrom
xhwxt:codex/reasoning-model-catalog-fix
Open

fix: preserve reasoning capabilities for routed model aliases#21
xhwxt wants to merge 1 commit into
BigStrongSun:mainfrom
xhwxt:codex/reasoning-model-catalog-fix

Conversation

@xhwxt

@xhwxt xhwxt commented Aug 17, 2026

Copy link
Copy Markdown

Summary

  • Read reasoning capabilities from inline model_providers.*.models[] definitions in the Codex config.
  • Resolve routed aliases through model, id, slug, and upstreamModel identifiers.
  • Preserve supported efforts, defaults, upstream parameter mappings, and provider-config provenance in the generated model catalog.
  • Add coverage for an alias whose reasoning metadata exists only on the upstream model definition.

Validation

  • git diff --check
  • Python reasoning-guard syntax and smoke tests
  • Rust tests/build not run locally because the Rust toolchain is unavailable in this environment.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant