fix: Streaming reasoning parser buffer phase eating content#8
Merged
raullenchai merged 1 commit intomainfrom Feb 25, 2026
Merged
fix: Streaming reasoning parser buffer phase eating content#8raullenchai merged 1 commit intomainfrom
raullenchai merged 1 commit intomainfrom
Conversation
During the first 80 chars of output, the MiniMax reasoning parser emitted ALL text as reasoning_content before deciding if it actually was reasoning. With openclaw's reasoning:true config, this hidden the model's actual response (e.g., "I am OpenClaw") in the reasoning block, making it appear the model "forgot who it is." Fix: return None during buffering (truly silent), then flush the entire accumulated buffer as the correct type (content or reasoning) once the decision is made. Also handle finalize_streaming for responses shorter than the buffer threshold. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
4 tasks
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
1 participant
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.
Summary
reasoning_contentduring the first 80-char buffer phasereasoning: trueconfig, this caused the model to appear to "forget who it is" — the identity response was hidden in reasoningFix
Noneduring buffering (truly silent) instead ofDeltaMessage(reasoning=delta_text)finalize_streamingfor responses shorter than buffer thresholdTest plan
🤖 Generated with Claude Code