Skip to content

fix(perf): preserve Unicode line separators in SSE payloads - #1642

Merged
Yunnglin merged 1 commit into
mainfrom
fix/perf-sse-unicode-lines
Aug 27, 2026
Merged

fix(perf): preserve Unicode line separators in SSE payloads#1642
Yunnglin merged 1 commit into
mainfrom
fix/perf-sse-unicode-lines

Conversation

@Yunnglin

@Yunnglin Yunnglin commented Aug 27, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • parse SSE fields only on LF instead of str.splitlines()
  • preserve U+0085, U+2028, and U+2029 inside JSON payloads
  • add fragmented-stream regression coverage for both SSE parsers

Testing

  • targeted streaming handler tests (7 passed)
  • pre-commit checks on changed files

Fixes #1638

Copilot AI lite review requested due to automatic review settings August 27, 2026 03:05
Copilot AI reviewed Aug 27, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@Yunnglin
Yunnglin merged commit 19333d0 into main Aug 27, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot
Copilot review effort, defaults to Lite
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[Bug][perf] JSONDecodeError: Unterminated string when parsing streamed SSE chunks under high concurrency

2 participants

@Yunnglin