Skip to content

fix(minimax): align api_key params with official API#27

Closed
hendr15k wants to merge 1 commit into
mnfst:mainfrom
hendr15k:minimax-params-v2
Closed

fix(minimax): align api_key params with official API#27
hendr15k wants to merge 1 commit into
mnfst:mainfrom
hendr15k:minimax-params-v2

Conversation

@hendr15k
Copy link
Copy Markdown

Summary

  • Replace max_completion_tokens with max_tokens (official MiniMax API uses max_tokens)
  • Replace reasoning_split (boolean) with thinking enum (disabled/enabled) to match API behavior
  • Applies to all MiniMax api_key models: m2, m2.1, m2.5, m2.7 (+ highspeed variants)
  • subscription models already correct (max_tokens, temperature, top_p)

Changes

  • 7 YAML files modified across all MiniMax api_key variants
  • Parameters now match the official MiniMax Anthropic-compatible API docs

Testing

  • npm run validate passes ✓ (113 models validated)

… API

- Replace max_completion_tokens with max_tokens (official API param)
- Replace reasoning_split boolean with thinking enum (disabled/enabled)
- Applies to all MiniMax api_key models: m2, m2.1, m2.5, m2.7 (+ highspeed variants)
- subscription models already correct (max_tokens, temperature, top_p)
- Validation passes (113 models)
@vercel
Copy link
Copy Markdown

vercel Bot commented May 24, 2026

Someone is attempting to deploy a commit to the Manifest Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions github-actions Bot added the model Add a model that's missing label May 24, 2026
@guillaumegay13
Copy link
Copy Markdown
Collaborator

thanks for your contribution!

Closing as superseded/incorrect.

#31 covers the MiniMax/Z.ai subscription gap and preserves provider-native MiniMax casing. This PR changes MiniMax API-key rows in a way that does not match the current MiniMax OpenAI-compatible docs (max_completion_tokens, reasoning_split) and fails the parameter removal guard by removing existing params.

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

Labels

model Add a model that's missing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants