Skip to content

fix(docs): handle errors and early closes in WebSocket examples - #3056

Open
markstuart-oai wants to merge 1 commit into
mainfrom
markstuart/websocket-audit-examples-20261026
Open

markstuart-oai wants to merge 1 commit into
mainfrom
markstuart/websocket-audit-examples-20261026

Conversation

@markstuart-oai

Copy link
Copy Markdown
Contributor

Summary

The Realtime and Responses examples exit with an error after a failed generation, a protocol error, or a close before completion. A 60-second deadline bounds waits for the server.

Verification

Ran 13 local WebSocket cases with the actual SDK source, including two 60-second waits with silent peers. The repository-pinned oxfmt 0.72.0 formatter passed with ultracite 7.11.1 and the repository configuration. The formatting change leaves the compiled JavaScript unchanged.

@github-actions

Copy link
Copy Markdown
Contributor

Castiron custom code

Evaluated main: 381709f60ae0a460d53d795f23c7eba7f576798e.

Trusted main policy: 381709f60ae0a460d53d795f23c7eba7f576798e.

✅ No new custom-code files detected.

0 mixed files remain; 0 existing customizations changed.

Compared 381709f60ae0 → 74e0a679fb8d. Generated baselines verified.

A changed generated baseline means this report cannot reliably identify which handwritten lines changed.

Inspect the custom-code diff

Download the exact patch produced by this run (requires repository access):

gh run download 40428185526 --repo openai/openai-node \
  --name castiron-custom-code-40428185526-1 --dir /tmp/castiron-custom-code-40428185526-1
git apply --stat /tmp/castiron-custom-code-40428185526-1/custom-code.patch
cat /tmp/castiron-custom-code-40428185526-1/custom-code.patch

Or reproduce it from an SDK checkout containing the vendored reporter:

git fetch --no-tags origin 381709f60ae0a460d53d795f23c7eba7f576798e 74e0a679fb8d2e4f437228ecc11cbaa5a558b20a
python3 scripts/castiron/custom_code_report.py report \
  --base 381709f60ae0a460d53d795f23c7eba7f576798e \
  --head 74e0a679fb8d2e4f437228ecc11cbaa5a558b20a --target-branch main --fetch --require-head-hash --public \
  --out /tmp/castiron-custom-code-74e0a679fb8d
cat /tmp/castiron-custom-code-74e0a679fb8d/custom-code.patch

This is the current full custom patch for mixed files, not an attribution of only the handwritten lines changed by this PR.

Full report and patch

@markstuart-oai
markstuart-oai marked this pull request as ready for review October 27, 2026 17:51
@markstuart-oai
markstuart-oai requested a review from a team as a code owner October 27, 2026 17:51
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 27, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-27T17:53:04.938303Z 74e0a67 Draft marked ready
🔒 Security Review ✅ Completed 2026-10-27T17:53:01.517184Z 74e0a67 Draft marked ready
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

This branch has not been deployed

No deployments
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.

1 participant