Repository navigation
feat: register MiniMax M3.1 Flash text model - #117
Open
octo-patch wants to merge 6 commits into
Open
octo-patch wants to merge 6 commits into
octo-patch wants to merge 6 commits into
Conversation
Author
|
Follow-up commit |
added 4 commits
October 21, 2026 18:36
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: Route the MiniMax text alias to M3.1 Flash, preserve unknown billing throughout the console, and serialize compatible MiniMax reasoning controls.
Behavior
minimaxalias targeting Flash through global Chat, with priorities global Chat, China Chat, global Anthropic, China Anthropic. Global defaults and saved/custom routes remain unchanged. Requests must still name a model/alias.reasoning_effort/ Anthropicoutput_config.effort. M3 uses adaptive/disabled thinking and ignores ordinary string effort values, includinghigh, without forwarding effort. Previously advertised adaptive/disabled mode presets remain compatible. Invalid Flash effort/thinking and malformed controls return 400 before inference; ordinary M3 effort is not an error.Validation and current audit state
Independent audit of prior head
da58965097e9ee5400dfd292550ad94ae185ccd9remains FAIL: chart null coercion and M3 ordinary effort rejection. This follow-up fixes those two findings; it does not claim independent acceptance.Before the fix, a focused Go test reproduced M3
loweffort rejection, and Node reproduced the chart expressions coercing null to numeric/display zero. After the fix:env -i HOME="$HOME" PATH="$PATH" GOPROXY=off GOTOOLCHAIN=local CGO_ENABLED=0 go test -vet=off -p 1 ./internal/protocol/minimax -count=1, including ordinary effort ignored via both Responses extensions and native Core output.git diff --checkandgit diff --cached --check.Previous head's Go package and 34-case HTTP/SSE checks remain historical evidence only. New head requires independent review and broader validation when resources permit. No paid inference or real provider credentials were used in tests.
Unresolved blockers
mainwhile CONTRIBUTING.md requiresdev; target reconciliation remains unresolved.