Skip to content

Run make format #1106

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jul 14, 2025
Merged

Run make format #1106

merged 1 commit into from
Jul 14, 2025

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