Skip to content

fix: recover codex streams with missing final output - #32897

Closed
sgtworkman wants to merge 1 commit into
NousResearch:mainfrom
sgtworkman:fix/codex-stream-none-output-recovery
Closed

fix: recover codex streams with missing final output#32897
sgtworkman wants to merge 1 commit into
NousResearch:mainfrom
sgtworkman:fix/codex-stream-none-output-recovery

Conversation

@sgtworkman

Copy link
Copy Markdown
Contributor

Summary

  • Recover Codex Responses streams when the OpenAI SDK crashes parsing the final response after valid streamed output items/text.
  • Apply the same recovery to the auxiliary Codex adapter used by title generation and other side tasks.
  • Add regressions for main Codex stream and auxiliary title-generation stream recovery paths.

Validation

  • scripts/run_tests.sh tests/agent/test_auxiliary_client.py tests/run_agent/test_run_agent_codex_responses.py — 245 passed
  • python -m pytest tests/agent/test_auxiliary_client.py -q — 178 passed, 1 warning
  • hermes -z "you good" --provider openai-codex -m gpt-5.5 — passed and returned a normal response
  • git diff --check — passed

Safety

  • Source-only change.
  • No merge.
  • No deploy.
  • No DNS/indexing/production mutation.

@teknium1

Copy link
Copy Markdown
Contributor

Closing as duplicate — the Codex null-output fix has been merged via #32963 (cherry-picked from @carltonawong's PR #32890). Thanks for the help during the outage. Closes #11179.

@teknium1 teknium1 closed this May 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

codex comp/agent Core agent runtime: loop, agent_init, prompt builder, context-compression, responses endpoint P3 Low — cosmetic, nice to have provider/openai OpenAI / Codex Responses API type/bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants