Repository navigation
feat: refresh MiniMax Flash model discovery and compatibility - #1040
Open
octo-patch wants to merge 4 commits into
Open
octo-patch wants to merge 4 commits into
octo-patch wants to merge 4 commits into
Conversation
Contributor
Author
|
I corrected the MiniMax China text route to use the documented api.minimaxi.com host and pushed a new commit. I ran the focused registry and MiniMax transport/history tests (156 passed) plus the scoped pre-commit checks. |
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Reason: The shipped registry omits MiniMax-M3.1-Flash, and ordinary listings still offer older MiniMax entries. MiniMax tool continuations also need reasoning fields preserved through node results and agent history.
Add verified Flash metadata and pricing. Ordinary MiniMax listings show Flash first, then M3; explicit legacy lookup,
include_legacy=True, saved model IDs and custom registrations remain supported. All 26 existing metadata entries retain their original values apart from three listing-only legacy flags.On official MiniMax endpoints, forward Anthropic-compatible thinking/effort controls per request without registering global model aliases or mutating shared clients. Ignore M3 effort before SDK validation so
xhighneither rejects the call nor enables thinking. Other endpoints, legacy/custom models and registry-sync opt-out retain their behavior.Preserve OpenAI
reasoning_contentand Anthropic signed thinking blocks through shared response handling, message serialization and agent tool history. For official MiniMax Anthropic streams, keep thought text once and concatenate signature fragments within the same thinking block. New thinking deltas and redacted blocks preserve separate block boundaries, including adjacent blocks with identical text. The implementation keeps LiteLLM's existing HTTP, message, response and streaming handling; the small request wrapper delegates to its pooled client.Validation (Python 3.13.14, frozen lockfile, LiteLLM 1.103.0):
tests/unit/nodes/prompts/test_manager.py::test_file_tools_persist_across_resets, is explicitly deselected because it makes an unmocked placeholder-key API call. Its timeout was previously reproduced on unchanged baseb62e66b93e17139f2bc16f9052b3ac6faba48302.The offline fixtures test protocol edge cases; they do not establish their frequency in production. No paid inference or local Docker build was performed. Remote CI is tracked separately for the latest commit; earlier-head results are not claimed for the follow-up.
Sources: model capabilities, pricing, OpenAI-compatible requests, Anthropic-compatible requests.