Skip to content

Fix reasoning not working for MiniMax Coding Plan#4062

Merged
chrarnoldus merged 1 commit into
mainfrom
christiaan/fix-m3-not-thinking
Jun 17, 2026
Merged

Fix reasoning not working for MiniMax Coding Plan#4062
chrarnoldus merged 1 commit into
mainfrom
christiaan/fix-m3-not-thinking

Conversation

@chrarnoldus

@chrarnoldus chrarnoldus commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

Seems to be a Vercel bug: reasoning does not show when using chat completions

Partial fix for: Kilo-Org/kilocode#11334

Seems to be a Vercel bug: reasoning does not show when using chat completions
@kilo-code-bot

kilo-code-bot Bot commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Executive Summary

Adds targeted MiniMax reasoning fixes — disables default thinking in the Messages API to match Anthropic SDK behavior, and routes MiniMax through the Anthropic provider on Vercel AI Gateway so reasoning surfaces correctly. Both changes are minimal, follow existing patterns, and are well-documented with inline references.

Files Reviewed (2 files)
  • apps/web/src/lib/ai-gateway/providers/apply-provider-specific-logic.ts
  • apps/web/src/lib/ai-gateway/providers/model-settings.ts

Reviewed by deepseek-v4-pro-20260423 · 262,390 tokens

Review guidance: REVIEW.md from base branch main

@chrarnoldus chrarnoldus merged commit 45a34f8 into main Jun 17, 2026
16 checks passed
@chrarnoldus chrarnoldus deleted the christiaan/fix-m3-not-thinking branch June 17, 2026 09:53
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