Repository navigation
feat(minimax): prefer M3.1 Flash and preserve reasoning history - #2171
Draft
octo-patch wants to merge 1 commit into
Draft
octo-patch wants to merge 1 commit into
octo-patch wants to merge 1 commit into
Conversation
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.
Problem
ORG2's direct MiniMax fallback still selects M3 for Git AI and M2 models for auxiliary work. Dynamic discovery exposes obsolete text models as ordinary choices. The native Anthropic transport also drops unsigned thinking before a tool call, which loses MiniMax reasoning on the next turn.
Solution
MiniMax-M3.1-Flash, followed byMiniMax-M3, in direct discovery and account projections. Keep enabled legacy IDs, explicit aliases, unrelated providers and non-text models; do not mutate stored selections. Gateway discovery preserves its actual IDs and only prioritizes discovered:free/-freeFlash routes.-nonevariant uses max instead of sending unsupported disabled/none. M3 and legacy requests keep their existing defaults. Direct M3/Flash context metadata is 1,000,000; gateway limits are not inferred.Protocol references: OpenAI compatibility, Anthropic compatibility.
Potential risks
Draft: the new provider HTTP fixtures have not executed. The required shared build guard stopped the pinned Rust run at 99.24 GiB free disk, below its 100 GiB floor. Compilation and integration failures remain possible. No paid inference, live gateway entitlement or desktop UI validation was performed. Shared Anthropic history handling needs the existing regression suite run before marking this ready.
Existing pricing was deliberately preserved; no new unverified price is introduced. No dependencies or lockfiles changed. Performance scope: bounded catalog projection at existing discovery boundaries and request-local parameter/history handling, with no new poller, IPC fan-out or background scan. AI-assisted implementation and review.
Verification
rustc --edition=2021 --test src-tauri/crates/key-vault/src/model_catalog.rs -o <evidence>/catalog-tests && <evidence>/catalog-tests: 3 passed, covering catalog order, saved/custom/non-text preservation and gateway IDs.cargo metadata --locked --offline --format-version 1 --no-deps: passed.rustfmt --edition 2021on touched Rust files andgit diff --check: passed.cargo test --locked --offline -p agent_core minimax --no-default-features --no-run: blocked by missing cachedrust_xlsxwriter.cargo test --locked -p agent_core minimax --no-default-features -- --test-threads=2: not executed after lock acquisition, resource guard exit 75 (disk_available_kb=104059680). GTK 3 / WebKitGTK 4.1 prerequisites are installed.Submit checklist
.github/PR_RULES.md.