Skip to content

Conversation

@felixarntz
Copy link
Member

As the title says, it's crucial to make a distinction (mostly for model generated content), whether it's regular output or thinking / reasoning. This PR adds that distinction to message parts within a single message.

This is in line with how API providers typically handle it (a single message may contain both kinds of parts/content).

@felixarntz felixarntz added this to the Finish the foundation milestone Aug 14, 2025
@felixarntz felixarntz added the [Type] Enhancement A suggestion for improvement. label Aug 14, 2025
Copy link
Member

@JasonTheAdams JasonTheAdams left a comment

Choose a reason for hiding this comment

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

Interesting! I was actually just wondering how thinking is identified. There we go!

One really minor suggestion that I'm not sure of your intent on, so I'll approve with handling it at your discretion.

@felixarntz felixarntz merged commit 2687daa into trunk Aug 15, 2025
4 checks passed
@JasonTheAdams JasonTheAdams deleted the add/message-part-channel branch September 10, 2025 16:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[Type] Enhancement A suggestion for improvement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants