Skip to content

Conversation

@brandon93s
Copy link
Contributor

Copilot AI review requested due to automatic review settings January 14, 2026 14:33
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds the missing input limit field to three OpenAI GPT-5.1 Codex model configurations and corrects the documentation example in the README.

Changes:

  • Added input = 272_000 to the [limit] section of three Codex model variants
  • Fixed the README documentation example to correctly show input field instead of duplicate context field

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
providers/openai/models/gpt-5.1-codex.toml Added input limit field (272,000 tokens) to the GPT-5.1 Codex model configuration
providers/openai/models/gpt-5.1-codex-mini.toml Added input limit field (272,000 tokens) to the GPT-5.1 Codex Mini model configuration
providers/openai/models/gpt-5.1-codex-max.toml Added input limit field (272,000 tokens) to the GPT-5.1 Codex Max model configuration
README.md Fixed documentation example to correctly show the input field name in the [limit] section

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@rekram1-node
Copy link
Contributor

wow so simple and i missed this, facepalm

@rekram1-node rekram1-node merged commit 58e2fd8 into anomalyco:dev Jan 14, 2026
6 of 7 checks passed
@brandon93s
Copy link
Contributor Author

@rekram1-node not quite, opencode isn't actually respecting the input limits. anomalyco/opencode#8465

@opencode-agent
Copy link
Contributor

User brandon93s does not have write permissions

github run

@rekram1-node
Copy link
Contributor

Yeah yeah Ik I just forgot about this field, Frank added it a while ago for this and I just forgot

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