feat: add openrouter/minimax-m2 to SEND_REASONING_CONTENT_PATTERNS #1152
+1
−0
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.
Overview
Add support for MiniMax-M2 interleaved thinking when used via OpenRouter by adding
openrouter/minimax-m2toSEND_REASONING_CONTENT_PATTERNS.Problem
MiniMax-M2 uses "interleaved thinking" where reasoning is mixed with responses across multi-turn conversations. When using MiniMax-M2 through OpenRouter, the reasoning content must be preserved and sent back in subsequent requests to maintain the model's chain of thought. Without this preservation, users reported significant performance degradation (up to 40% accuracy drop on complex tasks).
Solution
How It Works
reasoningfieldreasoning→reasoning_contentMessage.from_llm_chat_message()automatically capturesreasoning_contentSEND_REASONING_CONTENT_PATTERNS, the SDK will sendreasoning_contentin subsequent requestsWhy
openrouter/minimax-m2Specifically?The pattern matching uses substring matching, so:
"openrouter/minimax-m2"matches onlyopenrouter/minimax-m2"minimax-m2"would matchminimax-m2,openrouter/minimax-m2,groq/minimax-m2, etc.Using the specific pattern avoids affecting direct API users who may use:
thinking_blocks)<think>tagsChanges
"openrouter/minimax-m2"toSEND_REASONING_CONTENT_PATTERNSinmodel_features.pyTesting
Verified with pre-commit hooks:
Manual Testing Scenarios
Users should test:
model="openrouter/minimax-m2"Impact
openrouter/minimax-m2model identifierReferences
Related Work
This follows the same pattern as Kimi K2-Thinking support, which was the first model added to
SEND_REASONING_CONTENT_PATTERNS. Both models benefit from reasoning content preservation, though they achieve it through different API mechanisms.@neubig can click here to continue refining the PR
Agent Server images for this PR
• GHCR package: https://github.com/OpenHands/agent-sdk/pkgs/container/agent-server
Variants & Base Images
eclipse-temurin:17-jdknikolaik/python-nodejs:python3.12-nodejs22golang:1.21-bookwormPull (multi-arch manifest)
# Each variant is a multi-arch manifest supporting both amd64 and arm64 docker pull ghcr.io/openhands/agent-server:706d74a-pythonRun
All tags pushed for this build
About Multi-Architecture Support
706d74a-python) is a multi-arch manifest supporting both amd64 and arm64706d74a-python-amd64) are also available if needed