Skip to content

Fix streaming bugs sync#155

Merged
danielmillerp merged 1 commit into
mainfrom
dm/streaming-bug-fix
Oct 17, 2025
Merged

Fix streaming bugs sync#155
danielmillerp merged 1 commit into
mainfrom
dm/streaming-bug-fix

Conversation

@danielmillerp
Copy link
Copy Markdown
Contributor

@danielmillerp danielmillerp commented Oct 16, 2025

@MichaelSun48 pointed out the following 3 errors:

  1. Streaming seems to be broken after the first turn because of a pydantic validation error
  2. Tons of issues with json serializability. I can't pass in tools and model_settings are broken
  3. mcp server needs to be populated, or else it'll error out.

Was not able to reproduce 2. But was able to fix 1 and 3 and make changes backwards compatible. Note this is just for sync agents!

@danielmillerp danielmillerp changed the title Fix streaming bugs Fix streaming bugs sync Oct 16, 2025
Comment thread src/agentex/lib/adk/providers/_modules/openai.py
Copy link
Copy Markdown
Contributor

@RoxyFarhad RoxyFarhad left a comment

Choose a reason for hiding this comment

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

small nit thing but otherwise LGTM 🚀

@danielmillerp danielmillerp merged commit 6958ffd into main Oct 17, 2025
9 of 10 checks passed
@danielmillerp danielmillerp deleted the dm/streaming-bug-fix branch October 17, 2025 14:42
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.

3 participants