Skip to content

Conversation

@vrtnis
Copy link
Contributor

@vrtnis vrtnis commented Jul 14, 2025

This is a follow-up to pr #1043 The original changes were reverted due to missing updates in RealtimeSession, which caused runtime test failures.

This PR:

  • Reapplies the tool_name and tool_call_id additions to ToolContext.
  • Updates RealtimeSession._handle_tool_call to instantiate ToolContext with tool_name=event.name and tool_call_id=event.call_id.
  • Adjusts tests as needed so that all 533 tests (including old-version Python 3.9) pass cleanly.

Closes #1030

Copy link
Member

@seratch seratch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Apart from the changes under ./docs, looks good to me; @rm-openai can you take a look?

Copy link
Member

@seratch seratch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@rm-openai rm-openai merged commit 72d0d75 into openai:main Jul 15, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

3 participants