Skip to content

Conversation

@rm-openai
Copy link
Collaborator

@rm-openai rm-openai commented Jul 14, 2025

@rm-openai rm-openai changed the base branch from main to rm/pr1104 July 14, 2025 16:59
@rm-openai rm-openai requested a review from seratch July 14, 2025 17:32
rm-openai added a commit that referenced this pull request Jul 14, 2025
Guardrails were blocking the event loop. Now, they run as a separate
task.




---
[//]: # (BEGIN SAPLING FOOTER)
* #1112
* #1111
* #1107
* #1106
* __->__ #1104
Base automatically changed from rm/pr1104 to main July 14, 2025 21:28
@rm-openai rm-openai merged commit bd988dc into main Jul 14, 2025
10 checks passed
@rm-openai rm-openai deleted the rm/pr1106 branch July 14, 2025 21:35
rm-openai added a commit that referenced this pull request Jul 14, 2025
The model listens to websocket events in a separate task, and the
session receives those events in on_event. So if there's e.g. a
websocket or json exception, it's silently dropped. Similar to the
streaming text agents, we should:
- capture exceptions separately
- raise them in `async for event in session` loop.

Added tests to verify.






---
[//]: # (BEGIN SAPLING FOOTER)
* #1112
* #1111
* __->__ #1107
* #1106
@seratch seratch added documentation Improvements or additions to documentation feature:core feature:realtime labels Jul 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation feature:core feature:realtime

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants