Skip to content

fix(models): remove unstable flag from MiniMax M2.1 - #1538

Merged
rcogal merged 1 commit into
mainfrom
fix/minimax-m2.1-stability
Jan 29, 2026
Merged

rcogal merged 1 commit into
mainfrom
fix/minimax-m2.1-stability

Conversation

@rcogal

@rcogal rcogal commented Jan 29, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

  • Remove stability: "unstable" flag from MiniMax M2.1 provider
  • The timeout issues that originally caused this flag have been resolved

Test plan

  • Ran E2E tests with TEST_MODELS="minimax/minimax-m2.1" - all 14 test files passed

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Bug Fixes
    • Removed the stability property from the minimax M2.1 provider configuration.

✏️ Tip: You can customize this high-level summary in your review settings.

The MiniMax M2.1 model was marked as unstable due to timeout
issues during E2E tests. Retested and all tests now pass.

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

coderabbitai Bot commented Jan 29, 2026 •

Copy link
Copy Markdown
Contributor

Walkthrough

The change removes the stability property from the minimax M2.1 provider entry in the minimax models file. This is a data-level modification affecting a single provider configuration with no impact to type signatures or runtime control flow.

Changes

Cohort / File(s) Summary
Minimax Model Configuration
packages/models/src/models/minimax.ts
Removed the stability property from the minimax M2.1 provider entry.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: removing the unstable flag from MiniMax M2.1, which aligns with the file changes and PR objectives.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings

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 and usage tips.

@rcogal
rcogal added this pull request to the merge queue Jan 29, 2026
Merged via the queue into main with commit d285646 Jan 29, 2026
13 of 14 checks passed
@rcogal
rcogal deleted the fix/minimax-m2.1-stability branch January 29, 2026 18:28
steebchen pushed a commit that referenced this pull request Jan 29, 2026
## Summary
- Remove `stability: "unstable"` flag from MiniMax M2.1 provider
- The timeout issues that originally caused this flag have been resolved

## Test plan
- [x] Ran E2E tests with `TEST_MODELS="minimax/minimax-m2.1"` - all 14
test files passed

🤖 Generated with [Claude Code](https://claude.com/claude-code)

<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->

## Summary by CodeRabbit

* **Bug Fixes**
* Removed the stability property from the minimax M2.1 provider
configuration.

<sub>✏️ Tip: You can customize this high-level summary in your review
settings.</sub>

<!-- end of auto-generated comment: release notes by coderabbit.ai -->

Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>
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