Skip to content

Conversation

@ben-vargas
Copy link
Contributor

Summary

This PR adds full support for the new gpt-5.1-codex-max model family and the xhigh reasoning effort level.

Changes

  • Added gpt-5.1-codex-max to model normalization and mapping.
  • Added support for xhigh reasoning effort (supported by the max model).
  • Updated config/full-opencode.json to include 4 new verified model variants:
    • gpt-5.1-codex-max-low
    • gpt-5.1-codex-max-medium
    • gpt-5.1-codex-max-high
    • gpt-5.1-codex-max-xhigh
  • Updated documentation (README, config docs) to reflect these additions.
  • Added comprehensive tests for the new model and reasoning level.

verification

  • Added new unit tests in test/request-transformer.test.ts.
  • Verified npm test passes (173 tests passed).

Adds full support for the new `gpt-5.1-codex-max` model family, including:
- 4 new model variants in `full-opencode.json` (low, medium, high, xhigh)
- Support for `xhigh` reasoning effort in request transformer
- Model normalization and mapping for max variants
- Comprehensive tests and documentation updates
@ben-vargas
Copy link
Contributor Author

Oh, I didn't notice that a PR #44 already existed for this add... I'd simply added it to my fork because I saw it didn't exist yet... fine if you want to move forward with PR #44 over this one, or this one if it looks preferred.

Copy link
Owner

@numman-ali numman-ali left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Whoops - ignore comment @ben-vargas was meant to pop on other PR

Let's go with the other one, it covers everything you've added plus CLI tests and full documentation updates

Thanks as always !

@ben-vargas ben-vargas closed this Nov 25, 2025
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