Skip to content

Update MiniMax provider defaults for MiniMax-M3#1056

Open
octo-patch wants to merge 1 commit into
rohitg00:mainfrom
octo-patch:feat/minimax-m3-provider-defaults
Open

Update MiniMax provider defaults for MiniMax-M3#1056
octo-patch wants to merge 1 commit into
rohitg00:mainfrom
octo-patch:feat/minimax-m3-provider-defaults

Conversation

@octo-patch

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

Copy link
Copy Markdown

Supersedes #1032 with a clean signed-off commit from current main.

Summary

  • Makes MiniMax-M3 the default while retaining MiniMax-M2.7 as a documented override.
  • Normalizes Anthropic-compatible base URLs so requests use exactly one /v1/messages suffix.
  • Documents global and China endpoints for Anthropic-compatible and OpenAI-compatible configuration.
  • Updates onboarding, fallback resolution, and real request capture coverage.

Testing

  • npm test -- test/minimax-provider.test.ts test/fallback-model-resolution.test.ts test/fetch-timeout.test.ts
  • npm run build
  • npm test -- --exclude test/hook-project.test.ts
  • npm test -- test/hook-project.test.ts with the expected checkout basename

Summary by CodeRabbit

  • New Features

    • Added MiniMax provider support guidance for global and China endpoints using Anthropic- and OpenAI-compatible configurations.
    • Updated the default MiniMax model to MiniMax-M3.
    • Increased the default maximum token limit to 4,096.
    • Improved endpoint handling for custom MiniMax base URLs.
  • Documentation

    • Updated onboarding labels, environment examples, and provider setup instructions with current MiniMax settings.
  • Bug Fixes

    • Ensured MiniMax requests consistently use the correct /v1/messages endpoint.

Signed-off-by: octo-patch <266937838+octo-patch@users.noreply.github.com>
@vercel

vercel Bot commented Jul 13, 2026

Copy link
Copy Markdown

@octo-patch is attempting to deploy a commit to the rohitg00's projects Team on Vercel.

A member of the Team first needs to authorize it.

@coderabbitai

coderabbitai Bot commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: f2a025db-b8b7-4b74-9770-f7840fd63d93

📥 Commits

Reviewing files that changed from the base of the PR and between 93ae9bc and d623f82.

📒 Files selected for processing (8)
  • .env.example
  • README.md
  • src/cli/onboarding.ts
  • src/config.ts
  • src/providers/index.ts
  • src/providers/minimax.ts
  • test/fallback-model-resolution.test.ts
  • test/minimax-provider.test.ts

📝 Walkthrough

Walkthrough

MiniMax configuration now defaults to MiniMax-M3, documents global and China endpoints, normalizes /v1/messages request URLs, and adds behavioral coverage for defaults and URL variants.

Changes

MiniMax provider update

Layer / File(s) Summary
MiniMax defaults and configuration
.env.example, README.md, src/cli/onboarding.ts, src/config.ts, src/providers/index.ts, src/providers/minimax.ts
MiniMax defaults change to MiniMax-M3; configuration documentation adds protocol-specific global and China endpoints, and provider guidance updates the default token limit and base URL.
Message endpoint normalization
src/providers/minimax.ts
Request URL construction trims trailing slashes, removes a terminal /v1, and produces the versioned /v1/messages endpoint.
Provider behavior validation
test/fallback-model-resolution.test.ts, test/minimax-provider.test.ts
Tests expect the M3 fallback model and validate default and custom MiniMax request URLs through mocked fetch calls.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Possibly related PRs

  • rohitg00/agentmemory#791: Both changes update provider default or fallback model resolution in src/providers/index.ts.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly summarizes the main change: updating MiniMax provider defaults to MiniMax-M3.
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.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

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