feat(models): add MiniMax-M3 and MiniMax-M2.7 to recommended models - #931
feat(models): add MiniMax-M3 and MiniMax-M2.7 to recommended models#931octo-patch wants to merge 1 commit into
Conversation
Add MiniMax-M3 (1,000,000-token context) and MiniMax-M2.7 (204,800-token context) to RECOMMENDED_MODEL_NAMES and the FRONTIER_MODEL_FAMILIES registry so they are recognized as recommended frontier models. Document both regional endpoints (global api.minimax.io and China api.minimaxi.com) and the correct context windows in a new MiniMax provider guide, the providers overview, and the README.
|
|
||
| ## Notes | ||
|
|
||
| - MiniMax API is fully OpenAI-compatible, so it works via the `openai/` LiteLLM prefix with `LLM_API_BASE` |
There was a problem hiding this comment.
Correct the provider routing description
The openai/ prefix routes this configuration through Strix's native OpenAI provider, not its LiteLLM fallback. Calling it a LiteLLM prefix misdirects users troubleshooting the documented MiniMax setup.
| - MiniMax API is fully OpenAI-compatible, so it works via the `openai/` LiteLLM prefix with `LLM_API_BASE` | |
| - MiniMax API is fully OpenAI-compatible, so it works via the native `openai/` provider prefix with `LLM_API_BASE` |
Knowledge Base Used: Configuration and Telemetry
Prompt To Fix With AI
This is a comment left during a code review.
Path: docs/llm-providers/minimax.mdx
Line: 53
Comment:
**Correct the provider routing description**
The `openai/` prefix routes this configuration through Strix's native OpenAI provider, not its LiteLLM fallback. Calling it a LiteLLM prefix misdirects users troubleshooting the documented MiniMax setup.
```suggestion
- MiniMax API is fully OpenAI-compatible, so it works via the native `openai/` provider prefix with `LLM_API_BASE`
```
**Knowledge Base Used:** [Configuration and Telemetry](https://app.greptile.com/strix-org-3/-/custom-context/knowledge-base/usestrix/strix/-/docs/telemetry-and-config.md)
---
For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!
Greptile SummaryThis PR adds MiniMax-M3 and MiniMax-M2.7 to recommended/frontier model metadata and documents MiniMax configuration.
Confidence Score: 4/5The PR appears safe to merge, with one non-blocking documentation correction needed for the description of MiniMax provider routing. The added model metadata and tested classification paths are internally consistent, while the provider guide incorrectly labels the native Files Needing Attention: docs/llm-providers/minimax.mdx Important Files Changed
Prompt To Fix All With AI### Issue 1
docs/llm-providers/minimax.mdx:53
**Correct the provider routing description**
The `openai/` prefix routes this configuration through Strix's native OpenAI provider, not its LiteLLM fallback. Calling it a LiteLLM prefix misdirects users troubleshooting the documented MiniMax setup.
```suggestion
- MiniMax API is fully OpenAI-compatible, so it works via the native `openai/` provider prefix with `LLM_API_BASE`
```
---
For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.Reviews (1): Last reviewed commit: "feat(models): add MiniMax-M3 and MiniMax..." | Re-trigger Greptile |
Reason: Add MiniMax-M3 and MiniMax-M2.7 to Strix's recommended model metadata and provider documentation with correct token contexts and regional endpoints.
Changes
openai/MiniMax-M3(1,000,000-token context) andopenai/MiniMax-M2.7(204,800-token context) toRECOMMENDED_MODEL_NAMESinstrix/config/models.pyso they are recognized as recommended frontier models.minimaxentry toFRONTIER_MODEL_FAMILIESso bare and prefixed MiniMax model names are matched byis_recommended_or_frontier_model.docs/llm-providers/minimax.mdxprovider guide documenting both models with their correct context windows and both regional endpoints (globalapi.minimax.ioand Chinaapi.minimaxi.com).docs/docs.jsonnavigation and link it from the providers overview and the README recommended-models list.Checks
uv run ruff check strix/config/models.py tests/test_models.py— all checks passeduv run ruff format --check strix/config/models.py tests/test_models.py— already formatteduv run pytest tests/test_models.py -q— 70 passed