Skip to content

Realtime: one guardrail trip event per response #1458

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
Aug 14, 2025
Merged

Realtime: one guardrail trip event per response #1458

merged 1 commit into from
Aug 14, 2025

Conversation

rm-openai
Copy link
Collaborator

@rm-openai rm-openai commented Aug 13, 2025

There was a problem with the current implementation, where for a single repsonse, we might have many different guardrails fire. We should have at most one per response.

@rm-openai rm-openai requested review from seratch, jhills20, dkundel-openai and rzhao-openai and removed request for seratch August 13, 2025 17:44
@seratch seratch added bug Something isn't working feature:realtime labels Aug 14, 2025
@rm-openai rm-openai merged commit c436389 into main Aug 14, 2025
7 checks passed
@rm-openai rm-openai deleted the rm/pr1458 branch August 14, 2025 16:09
vcshih pushed a commit to veris-ai/openai-agents-python that referenced this pull request Aug 15, 2025
There was a problem with the current implementation, where for a single
repsonse, we might have many different guardrails fire. We should have
at most one per response.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working feature:realtime
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants