Skip to content

feat: add MiniMax tool call parsing support#231

Merged
Thump604 merged 2 commits intowaybarrios:mainfrom
sjswerdloff:feat/minimax-tool-call-parser
Apr 11, 2026
Merged

feat: add MiniMax tool call parsing support#231
Thump604 merged 2 commits intowaybarrios:mainfrom
sjswerdloff:feat/minimax-tool-call-parser

Conversation

@sjswerdloff
Copy link
Copy Markdown
Contributor

Parse MiniMax-M2.5's XML tool call format:
minimax:tool_call

value

</minimax:tool_call>

Handles single/multiple tool calls, JSON parameter values, no-parameter calls, and preserves blocks.

9 unit tests included.

Parse MiniMax-M2.5's XML tool call format:
<minimax:tool_call>
  <invoke name="function">
    <parameter name="arg">value</parameter>
  </invoke>
</minimax:tool_call>

Handles single/multiple tool calls, JSON parameter values,
no-parameter calls, and preserves <think> blocks.

9 unit tests included.
@sjswerdloff
Copy link
Copy Markdown
Contributor Author

See issue #239 (black vs ruff).

Copy link
Copy Markdown
Collaborator

@Thump604 Thump604 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

MiniMax parser path looks good on the refreshed head. I pushed the missing Black formatting fix, reran the dedicated MiniMax parser test locally (
no tests ran in 0.00s), and the full PR CI is now green. Narrow change, good regression coverage, ready to merge.

@Thump604 Thump604 merged commit cb9a9bd into waybarrios:main Apr 11, 2026
7 checks passed
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.

2 participants