Conversation
- switch model runner to stream_events parsing for anthropic stream compatibility - only pass parallel_tool_calls=False for anthropic/vertexaianthropic - prioritize stream errors before tool follow-up decisions - add regression coverage for stream event compatibility and error handling
|
can you help fix the lint? |
|
Sure, I have fixed the lint errors. Let me know if there's anything else |
|
Hi @cUp1d, Thanks. I just got back from vacation and don’t have a suitable API to test right now. I’m leaning toward moving the provider compatibility logic to Republic. Would love to hear your thoughts since you’re the expert on this now. |
|
Welcome back! I have some Anthropic API keys available for testing. Happy to share it with you and bub project collaborators. Agreed that moving the compatibility logic to republic is the cleaner approach. one thing i noticed — there's an open PR (#22) to move republic into the bub monorepo. might make sense to land that first so the fix can be done in one place? or if that's not happening soon i can open a separate PR on republic directly. let me know what you think. |
Related Issue
Fixes #46
Modification
ModelRunnerfromrun_tools_asynctostream_events_asyncfor Anthropic stream compatibility.parallel_tool_calls=Falseforanthropic/vertexaianthropicmodels to avoid breaking non-Anthropic providers.finalevents are not treated as successful tool continuation.parallel_tool_calls[OK]/[ERROR]) intelegram_send.pyfor PowerShell-friendly output.Tests
uv run pytestpassedCode Quality Checks
uv run ruff check .passeduv run mypypassed