Skip to content

feat(models): upgrade MiniMax default model to M3 - #6473

Open
octo-patch wants to merge 2 commits into
NousResearch:mainfrom
octo-patch:feature/add-minimax-m2.7-highspeed
Open

feat(models): upgrade MiniMax default model to M3#6473
octo-patch wants to merge 2 commits into
NousResearch:mainfrom
octo-patch:feature/add-minimax-m2.7-highspeed

Conversation

@octo-patch

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

Copy link
Copy Markdown
Contributor

Summary

Upgrade the MiniMax model catalogs to use MiniMax-M3 as the default while keeping MiniMax-M2.7 and MiniMax-M2.7-highspeed as alternatives. Older models (M1 family and M2.5) are removed from the catalogs and metadata.

Changes

  • Add MiniMax-M3 to the catalog for both minimax and minimax-cn providers and place it first (default position)
  • Keep MiniMax-M2.7 and MiniMax-M2.7-highspeed as alternatives
  • Remove older models from the catalogs and setup.py defaults: MiniMax-M1, MiniMax-M1-40k, MiniMax-M1-80k, MiniMax-M1-128k, MiniMax-M1-256k, MiniMax-M2.5
  • Update default auxiliary model for minimax and minimax-cn to MiniMax-M3 in agent/auxiliary_client.py
  • Update agent/model_metadata.py DEFAULT_CONTEXT_LENGTHS: add minimax-m3 (524288 / 512K), drop M1 and M2.5 entries, set generic minimax prefix fallback to 524288
  • Update hindsight default model from MiniMax-M2.7 to MiniMax-M3
  • Update example commands in website/docs/integrations/providers.md to use MiniMax-M3
  • Update related unit tests in tests/agent/test_minimax_provider.py and tests/hermes_cli/test_setup_model_selection.py

Why

MiniMax-M3 is the latest model, with a 512K context window, up to 128K output, and image input support (OpenAI- and Anthropic-compatible). M2.7 and M2.7-highspeed remain available for callers that still prefer them.

Testing

  • pytest tests/agent/test_minimax_provider.py — 12 passed
  • pytest tests/hermes_cli/test_setup_model_selection.py — 9 passed
  • pytest tests/hermes_cli/test_setup_model_provider.py tests/hermes_cli/test_model_normalize.py tests/hermes_cli/test_api_key_providers.py tests/agent/test_auxiliary_client.py — 256 passed
  • pytest tests/run_agent/test_fallback_model.py tests/agent/test_minimax_auxiliary_url.py — 36 passed

Add MiniMax-M2.7-highspeed to the minimax and minimax-cn provider model
lists. MiniMax-M2.7-highspeed delivers the same performance as M2.7
but with faster inference and lower latency.

API reference: https://platform.minimax.io/docs/api-reference/text-openai-api
@alt-glitch alt-glitch added type/feature New feature or request P3 Low — cosmetic, nice to have provider/minimax MiniMax (Anthropic transport) labels Apr 30, 2026
@alt-glitch

Copy link
Copy Markdown
Collaborator

Addresses #11367 (request to add MiniMax highspeed model variants).

- Add MiniMax-M3 to model list and set as default for minimax and minimax-cn
- Keep MiniMax-M2.7 and MiniMax-M2.7-highspeed as alternatives
- Remove older models (M1 variants, M2.5) from catalogs and metadata
- Update auxiliary model defaults to MiniMax-M3
- Update hindsight default model and docs examples
- Update related unit tests
@octo-patch octo-patch changed the title feat(models): add MiniMax-M2.7-highspeed to provider model catalog feat(models): upgrade MiniMax default model to M3 Jun 3, 2026
@teknium1

Copy link
Copy Markdown
Contributor

Thanks for the MiniMax catalog update. Main has since landed most of the M3 runtime work: e3b3d4d75 added M3 first in the native catalogs and its 1M context handling, while 3d1d0a49f moved native MiniMax auxiliary defaults to provider profiles.

Problems

  • The proposed agent/model_metadata.py value of 524,288 would regress current main's M3 handling at agent/model_metadata.py:281-287; current regression coverage requires a 1M-class result in tests/agent/test_minimax_provider.py:24-32.
  • The auxiliary edit targets a legacy fallback mapping removed by 3d1d0a49f; the active source is now plugins/model-providers/minimax/__init__.py:62-80.
  • The catalog-list assertions are change-detector tests for mutable provider data, contrary to AGENTS.md:80-83.

Suggested changes

  • Retain current profile routing and 1M M3 metadata, then narrow a salvage to the still-stale setup fallback (hermes_cli/setup.py:106-107), Hindsight default (plugins/memory/hindsight/__init__.py:72), and direct-provider examples (website/docs/integrations/providers.md:233-239) after current upstream availability is confirmed.

Automated hermes-sweeper review.

@teknium1 teknium1 added sweeper:risk-compatibility Sweeper risk: may break existing users, config, migrations, defaults, or upgrades sweeper:blast-moderate Sweeper blast radius: moderate — a subsystem or single platform labels Jul 12, 2026

@GottZ GottZ left a comment

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.

This was generated by AI during triage.

Summary

Six PRs address successive MiniMax catalog upgrades: #1882, #1900, #1932, and #4197 moved native and OpenRouter references from M2.5 to M2.7, while #6473 and #36191 move remaining defaults and catalogs to M3. The M2.7 work is already represented on main through #1900 and the #4197 salvage in #4208; for M3, #36191 covers OpenRouter/Nous lists, while #6473 contains several remaining updates but also conflicts with current runtime architecture and metadata.

Related pull requests

  • #1882 [closed] related — (+21/-13) — superseded by #1900: upgraded MiniMax defaults, provider catalogs, metadata, fallback tests, RL examples, and documentation from M2.5 to M2.7; the contributor confirmed these changes were manually carried into merged PR #1900 after refactor conflicts.
  • #1900 [merged] related — (+25/-14) — merged reference implementation: landed the substantive #1882 M2.7 upgrade across native defaults, setup, metadata, tests, RL tooling, and docs, with additional OpenRouter catalog entries.
  • #1932 [closed] related — (+9/-3) — already satisfied on main: repeated the M2.7 catalog, metadata, and setup-selection changes after #1900 had landed; the contributor confirmed that M2.7 and its highspeed variant were already present and ordered first.
  • #4197 [closed] related — (+195/-4) — partially salvaged via #4208: overlapped #1932 and existing main for most M2.7 registrations, but identified remaining gaps in the CLI provider flow and opencode-go catalog that the contributor reports were cherry-picked into #4208.
  • #6473 related — (+48/-56) — keep open for a narrowed revision, not merge as-is: it adds M3/M2.7-highspeed to setup and updates the Hindsight default and direct-provider examples, but its 524,288 context value would regress main's 1M-class M3 handling, its auxiliary edit targets a removed legacy mapping, and its mutable catalog assertions conflict with repository testing policy. This follows the visible keep_open review on #6473: retain current profile routing and metadata, then salvage only the still-stale setup, Hindsight, and documentation changes.
  • #36191 [merged] related — (+2/-2) — merged complementary implementation: updated the OpenRouter and Nous curated entries from minimax-m2.7 to minimax-m3; it does not cover #6473's remaining native setup fallback, Hindsight default, or direct-provider documentation.

Duplicates

#1882 and #1900 implement substantially the same M2.7 upgrade, with #1900 being the merged superseding version. #1932 and most of #4197 duplicate M2.7 work already on main, while #4197's remaining gaps were salvaged through #4208. #36191 is complementary to, not a duplicate of, the narrow salvageable portion of #6473.

Suggested consolidation

Revise and then merge #6473 as the consolidation target for the remaining M3 work: keep only the current-main-compatible setup fallback, Hindsight default, and direct-provider documentation updates, while dropping the 524,288 metadata regression, legacy auxiliary mapping edit, already-landed catalogs, and mutable catalog-list tests. Keep #1882, #1932, and #4197 closed as superseded/already-salvaged duplicates; #1900 and #36191 remain the merged reference implementations for the M2.7 baseline and OpenRouter/Nous M3 catalog update, respectively.

Cross-PR triage: Reviewed 6 pull requests and 0 issues in this complex. Each diff was read against this issue; Assessment working set: 45 kB of PR diffs, 7 kB of issue/PR text, 3 kB of discussion (6 comments), 0 verify verdicts. verdicts reflect diff content, not PR titles. Part of an automated triage batch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

P3 Low — cosmetic, nice to have provider/minimax MiniMax (Anthropic transport) sweeper:blast-moderate Sweeper blast radius: moderate — a subsystem or single platform sweeper:risk-compatibility Sweeper risk: may break existing users, config, migrations, defaults, or upgrades type/feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants