Skip to content

feat: add MiniMax M3 prompt driver#2231

Open
octo-patch wants to merge 5 commits into
griptape-ai:mainfrom
octo-patch:octo/minimax-m3
Open

feat: add MiniMax M3 prompt driver#2231
octo-patch wants to merge 5 commits into
griptape-ai:mainfrom
octo-patch:octo/minimax-m3

Conversation

@octo-patch

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

Copy link
Copy Markdown

Summary

Add a dedicated MiniMax prompt driver and tokenizer with documented regional OpenAI- and Anthropic-compatible configuration.

Changes

  • Add MinimaxPromptDriver with the global API endpoint as the default and support for overriding base_url with the China API endpoint.
  • Add model-specific local token counting and input/output token limits for MiniMax-M3 and MiniMax-M2.7.
  • Register the driver and tokenizer in the public package exports.
  • Add unit coverage for initialization, serialization, endpoint selection, token counting, and model-specific input and output token limits.
  • Document global and China configuration for the OpenAI- and Anthropic-compatible APIs.

Testing

  • Targeted unit tests: 10 passed
  • SDK request capture for all four regional and protocol endpoint combinations
  • Full unit suite: 3,630 passed, 1 skipped
  • make check
  • make docs

@codecov

codecov Bot commented Jul 6, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@octo-patch

Copy link
Copy Markdown
Author

Updated the tokenizer to use the documented context window for each supported MiniMax model and added coverage for both regional API endpoints. The full unit suite, format, lint, type, spelling, coverage, Windows, Linux, and docs checks pass.

@octo-patch

Copy link
Copy Markdown
Author

Updated the PR to document both global and China endpoints for the OpenAI- and Anthropic-compatible configurations and clarified the tokenizer's scope. I verified all four final SDK request paths and ran the 10 targeted tests, make check, and make docs.

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