Skip to content

feat: Stream output text and tool calls#183

Open
marcelklehr wants to merge 3 commits into
mainfrom
feat/streaming
Open

feat: Stream output text and tool calls#183
marcelklehr wants to merge 3 commits into
mainfrom
feat/streaming

Conversation

@marcelklehr

@marcelklehr marcelklehr commented Jun 10, 2026

Copy link
Copy Markdown
Member
  • Stream text content
  • Stream tool calls
  • CI test

🤖 AI (if applicable)

  • The content of this PR was partly or fully generated using AI

Assisted-by: Copilot:gpt-5.4
Signed-off-by: Marcel Klehr <mklehr@gmx.net>

@julien-nc julien-nc left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

In my tests, an empty string output is reported multiple times before actually getting some text. This makes the message animated placeholder disappear.

Maybe something could be changed in the assistant to prevent that. I think it would be good if context_agent would not report empty output strings as well. Wdyt?

Otherwise, works like a charm!

@marcelklehr

Copy link
Copy Markdown
Member Author

an empty string output is reported multiple times before actually getting some text.

Is it maybe calling tools?

Signed-off-by: Marcel Klehr <mklehr@gmx.net>
Signed-off-by: Marcel Klehr <mklehr@gmx.net>
@marcelklehr

Copy link
Copy Markdown
Member Author

Is it maybe calling tools?

That was it

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