Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 0 additions & 4 deletions docs/llm-providers/models.mdx
Original file line number Diff line number Diff line change
Expand Up @@ -50,10 +50,6 @@ strix --target ./your-app
| GPT-5.2 | `strix/gpt-5.2` |
| GPT-5.1 | `strix/gpt-5.1` |
| GPT-5 | `strix/gpt-5` |
Comment on lines 50 to 52
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Codex models removed from this table but still defined in strix/llm/utils.py lines 40-43 in STRIX_MODEL_MAP. Users can still access these models via the code even though they're not documented.

Prompt To Fix With AI
This is a comment left during a code review.
Path: docs/llm-providers/models.mdx
Line: 50-52

Comment:
Codex models removed from this table but still defined in `strix/llm/utils.py` lines 40-43 in `STRIX_MODEL_MAP`. Users can still access these models via the code even though they're not documented.

How can I resolve this? If you propose a fix, please make it concise.

Comment on lines 50 to 52
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

docs/llm-providers/overview.mdx:52 still mentions "GPT-5 and Codex models" - should update to just "GPT-5 models"

Prompt To Fix With AI
This is a comment left during a code review.
Path: docs/llm-providers/models.mdx
Line: 50-52

Comment:
`docs/llm-providers/overview.mdx:52` still mentions "GPT-5 and Codex models" - should update to just "GPT-5 models"

How can I resolve this? If you propose a fix, please make it concise.

Comment on lines 50 to 52
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

PR description mentions "removing from docs and config.yaml" but no config.yaml file was modified in this commit

Prompt To Fix With AI
This is a comment left during a code review.
Path: docs/llm-providers/models.mdx
Line: 50-52

Comment:
PR description mentions "removing from docs and config.yaml" but no `config.yaml` file was modified in this commit

How can I resolve this? If you propose a fix, please make it concise.

| GPT-5.2 Codex | `strix/gpt-5.2-codex` |
| GPT-5.1 Codex Max | `strix/gpt-5.1-codex-max` |
| GPT-5.1 Codex | `strix/gpt-5.1-codex` |
| GPT-5 Codex | `strix/gpt-5-codex` |

### Google

Expand Down
2 changes: 1 addition & 1 deletion docs/llm-providers/overview.mdx
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ See the [Local Models guide](/llm-providers/local) for setup instructions and re
Recommended models router with high rate limits.
</Card>
<Card title="OpenAI" href="/llm-providers/openai">
GPT-5 and Codex models.
GPT-5 models.
</Card>
<Card title="Anthropic" href="/llm-providers/anthropic">
Claude Opus, Sonnet, and Haiku.
Expand Down
4 changes: 0 additions & 4 deletions strix/llm/utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -37,10 +37,6 @@ def normalize_tool_format(content: str) -> str:
"gpt-5.2": "openai/gpt-5.2",
"gpt-5.1": "openai/gpt-5.1",
"gpt-5": "openai/gpt-5",
"gpt-5.2-codex": "openai/gpt-5.2-codex",
"gpt-5.1-codex-max": "openai/gpt-5.1-codex-max",
"gpt-5.1-codex": "openai/gpt-5.1-codex",
"gpt-5-codex": "openai/gpt-5-codex",
"gemini-3-pro-preview": "gemini/gemini-3-pro-preview",
"gemini-3-flash-preview": "gemini/gemini-3-flash-preview",
"glm-5": "openrouter/z-ai/glm-5",
Expand Down