Skip to content

Add MiniMax-M3 and MiniMax-M2.7 to model registry#1217

Open
octo-patch wants to merge 2 commits into
f:mainfrom
octo-patch:octo/20260708-add-minimax-m3-recvoLa9Ftbrui
Open

Add MiniMax-M3 and MiniMax-M2.7 to model registry#1217
octo-patch wants to merge 2 commits into
f:mainfrom
octo-patch:octo/20260708-add-minimax-m3-recvoLa9Ftbrui

Conversation

@octo-patch

@octo-patch octo-patch commented Jul 9, 2026

Copy link
Copy Markdown

Reason: add target provider/model to existing provider registry.

Summary

  • Register MiniMax-M3 and MiniMax-M2.7 under the MiniMax provider in the Works Best With model registry.
  • Add exact lookup coverage for both registered models.

Test plan

  • npm test -- src/__tests__/lib/works-best-with.test.ts
  • npm run lint

Added MiniMax-M3 to the Works Best With model registry (under the MiniMax provider) and extended the model-grouping/getModelInfo test expectations to include MiniMax models—verifying slugs like minimax-m3 (and minimax-m2-7) return { name, provider: "MiniMax" } and that getModelsByProvider groups results under the MiniMax provider key.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Jul 9, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

Added the minimax-m3 model with display name MiniMax-M3 and provider MiniMax, and expanded tests for MiniMax model information and provider grouping.

Changes

MiniMax model addition

Layer / File(s) Summary
Add MiniMax model and test coverage
src/lib/works-best-with.ts, src/__tests__/lib/works-best-with.test.ts
Adds minimax-m3 to AI_MODELS, verifies MiniMax model information for supported slugs, and asserts the grouped result includes a MiniMax provider key.

Estimated code review effort: 1 (Trivial) | ~2 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly describes the main change: adding MiniMax model entries to the registry.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@octo-patch octo-patch changed the title Add MiniMax-M3 to model registry Add MiniMax-M3 and MiniMax-M2.7 to model registry Jul 12, 2026
@octo-patch

Copy link
Copy Markdown
Author

Updated the registry to include both MiniMax-M3 and MiniMax-M2.7, with exact lookup coverage for each model. I ran npm test -- src/__tests__/lib/works-best-with.test.ts (31 tests passed) and npm run lint (0 errors).

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.

1 participant