Ignore leading whitespace when streaming text, fixing run_stream + Ollama + Qwen3 #2294
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.
Ollama + Qwen3 will emit
<think>\n</think>\n\nahead of tool calls, which we don't want to end up treating as a final result when usingrun_stream(as that would stop the run before even handling the tool calls).(See example code below for context on this output)
Before, the
TextPart(content='\n\n')is treated as the final result:After, only the actual text after the tool call is treated as the final result: (If it was a
final_resultcall, that call would be treated as the final result)Code