Skip to content

Fix pre-connect Realtime manager event transforms#3432

Open
fengjikui wants to merge 1 commit into
openai:mainfrom
fengjikui:codex/realtime-omit-send
Open

Fix pre-connect Realtime manager event transforms#3432
fengjikui wants to merge 1 commit into
openai:mainfrom
fengjikui:codex/realtime-omit-send

Conversation

@fengjikui

Copy link
Copy Markdown

Fixes #3402.

Summary

  • route queued Realtime manager events through the same transform path used by connected sync sends
  • strip Omit values from dict events before serializing them into the pre-connect send queue
  • add sync and async manager regression coverage for response.cancel with event_id=omit

Validation

  • env HTTP_PROXY= HTTPS_PROXY= ALL_PROXY= NO_PROXY='*' http_proxy= https_proxy= all_proxy= no_proxy='*' uv run --with pytest --with pytest-asyncio --with pytest-xdist --with dirty-equals --with respx --with time-machine --with 'httpx[socks]' --with aiohttp --with httpx-aiohttp pytest tests/api_resources/test_realtime.py -q
  • uv run --with ruff ruff format --check src/openai/resources/realtime/realtime.py tests/api_resources/test_realtime.py
  • uv run --with ruff ruff check src/openai/resources/realtime/realtime.py tests/api_resources/test_realtime.py
  • git diff --check

@fengjikui fengjikui requested a review from a team as a code owner June 23, 2026 21:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Pre-connect Realtime manager.send crashes on Omit in dict events

1 participant