mirror of
https://github.com/openai/codex.git
synced 2026-05-02 02:17:22 +00:00
8a97f3cf03
## Summary Codex is repurposing `session` to mean a thread group, so the realtime provider session id should no longer use `session_id` / `sessionId` in Codex-facing protocol payloads. This PR renames that provider-specific field to `realtime_session_id` / `realtimeSessionId` and intentionally breaks clients that still send the old field names. ## What Changed - Renamed realtime provider session fields in `ConversationStartParams`, `RealtimeConversationStartedEvent`, and `RealtimeEvent::SessionUpdated`. - Renamed app-server v2 realtime request and notification fields to `realtimeSessionId`. - Removed legacy serde aliases for `session_id` / `sessionId`; clients must send the new names. - Propagated the rename through core realtime startup, app-server adapters, codex-api websocket handling, and TUI realtime state. - Regenerated app-server protocol schema/TypeScript outputs and updated app-server README examples. - Kept upstream Realtime API concepts unchanged: provider `session.id` parsing and `x-session-id` headers still use the upstream wire names. ## Testing - CI is running on the latest pushed commit. - Earlier local verification on this PR: - `cargo test -p codex-protocol` - `CODEX_SKIP_VENDORED_BWRAP=1 cargo test -p codex-core realtime_conversation` - `cargo test -p codex-app-server-protocol` - `CODEX_SKIP_VENDORED_BWRAP=1 cargo test -p codex-app-server realtime_conversation` - attempted `CODEX_SKIP_VENDORED_BWRAP=1 cargo test -p codex-tui` (local linker bus error while linking the test binary) --------- Co-authored-by: Codex <noreply@openai.com>
33 lines
647 B
JSON
Generated
33 lines
647 B
JSON
Generated
{
|
|
"$schema": "http://json-schema.org/draft-07/schema#",
|
|
"definitions": {
|
|
"RealtimeConversationVersion": {
|
|
"enum": [
|
|
"v1",
|
|
"v2"
|
|
],
|
|
"type": "string"
|
|
}
|
|
},
|
|
"description": "EXPERIMENTAL - emitted when thread realtime startup is accepted.",
|
|
"properties": {
|
|
"realtimeSessionId": {
|
|
"type": [
|
|
"string",
|
|
"null"
|
|
]
|
|
},
|
|
"threadId": {
|
|
"type": "string"
|
|
},
|
|
"version": {
|
|
"$ref": "#/definitions/RealtimeConversationVersion"
|
|
}
|
|
},
|
|
"required": [
|
|
"threadId",
|
|
"version"
|
|
],
|
|
"title": "ThreadRealtimeStartedNotification",
|
|
"type": "object"
|
|
} |