Skip to content

fix(minimax): mm@/mmax@ baseUrl minimax.io → minimaxi.com#122

Open
dev-seahouse wants to merge 1 commit into
MadAppGang:mainfrom
dev-seahouse:fix/minimax-endpoint
Open

fix(minimax): mm@/mmax@ baseUrl minimax.io → minimaxi.com#122
dev-seahouse wants to merge 1 commit into
MadAppGang:mainfrom
dev-seahouse:fix/minimax-endpoint

Conversation

@dev-seahouse
Copy link
Copy Markdown

Closes #121

Summary

  • Corrects the default baseUrl for the minimax provider (mm@, mmax@) from https://api.minimax.io to https://api.minimaxi.com
  • minimax.io is the endpoint for mmc@ (MiniMax Coding Plan) only — mm@/mmax@ should always route to minimaxi.com
  • Updates all references: provider-definitions.ts, config-command.ts, README.md, docs/settings-reference.md, and affected tests

Root cause

BUILTIN_PROVIDERS in provider-definitions.ts had the wrong baseUrl for the minimax entry. The minimax-coding entry correctly used minimax.io, but minimax (the general provider) was pointing there too instead of minimaxi.com. This caused mm@ and mmax@ requests to hit the coding endpoint and get 401s.

Diff

 // provider-definitions.ts
 name: "minimax",
 transport: "anthropic",
-baseUrl: "https://api.minimax.io",
+baseUrl: "https://api.minimaxi.com",
 // model-catalog.test.ts — test now uses the correct base per key type
-const MINIMAX_API_BASE = "https://api.minimax.io/anthropic/v1/messages";
+const MINIMAX_API_BASE = MINIMAX_CODING_KEY
+  ? "https://api.minimax.io/anthropic/v1/messages"
+  : "https://api.minimaxi.com/anthropic/v1/messages";

Behavior change

Provider prefix Before After
mm@, mmax@ https://api.minimax.io (wrong) https://api.minimaxi.com (correct)
mmc@ https://api.minimax.io https://api.minimax.io (unchanged)

Test plan

  • bun test packages/cli/src/providers/provider-definitions.test.ts — new tests assert minimaxminimaxi.com and minimax-codingminimax.io
  • bun test packages/cli/src/providers/transport/anthropic-compat.test.ts — updated fixture baseUrl passes
  • Full repo bun test — exits 0 (no regressions)
  • Manual: claudish --model mm@minimax-m2.7-highspeed --probe confirms transport uses minimaxi.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.

[BUG] wrong endpoint for minimax models mm@ and max@

2 participants