Skip to content

Realtime: send session.update event at connection/handoffs #1112

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

Conversation

rm-openai
Copy link
Collaborator

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

This is what tells the model about tools, tool choice, model etc etc.

@rm-openai rm-openai changed the base branch from main to rm/pr1111 July 14, 2025 20:59
@rm-openai rm-openai requested a review from seratch July 14, 2025 20:59
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
@rm-openai rm-openai changed the title Realtime: send session.update event at connection Realtime: send session.update event at connection/handoffs Jul 14, 2025
rm-openai added a commit that referenced this pull request Jul 14, 2025
cleanup





---
[//]: # (BEGIN SAPLING FOOTER)
* #1112
* #1111
* #1107
* __->__ #1106
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
rm-openai added a commit that referenced this pull request Jul 14, 2025
Base automatically changed from rm/pr1111 to main July 14, 2025 21:55
@rm-openai rm-openai enabled auto-merge (squash) July 14, 2025 21:56
@rm-openai rm-openai merged commit 5888cf2 into main Jul 14, 2025
5 checks passed
@rm-openai rm-openai deleted the rm/pr1112 branch July 14, 2025 21:57
@seratch seratch added enhancement New feature or request 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
enhancement New feature or request feature:realtime
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants