Skip to content

fix: preserve upstream stream-read errors - #13

Merged
voodoohop merged 1 commit into
mainfrom
codex/preserve-stream-read-errors
Sep 13, 2026
Merged

voodoohop merged 1 commit into
mainfrom
codex/preserve-stream-read-errors

Conversation

@voodoohop

@voodoohop voodoohop commented Sep 13, 2026

Copy link
Copy Markdown
Member
  • Emit the original stream-read error as a complete SSE error event instead of silently closing the response.
  • Keep error writing in one helper; existing stream loops only call it from their catch blocks. No retries or timeout changes.
  • Preserve partial output and healthy streams; covers standard and Bedrock readers.
  • Verified: 9 stream tests, 14 parameter tests and build pass. Full type-check output matches clean main exactly.
  • CI exceptions match merged fix: preserve explicit chat provider parameters #12: broken documentation links and formatting in the same six untouched files. Changed-file formatting passes.
  • Companion Pollinations update records this error and retains bounded terminal diagnostics.

@voodoohop
voodoohop merged commit b80c98e into main Sep 13, 2026
2 of 4 checks passed
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