Skip to content

feat(http): stabilize ServerSentEventParseStream#7050

Open
tomas-zijdemans wants to merge 3 commits intodenoland:mainfrom
tomas-zijdemans:stabilize-sse
Open

feat(http): stabilize ServerSentEventParseStream#7050
tomas-zijdemans wants to merge 3 commits intodenoland:mainfrom
tomas-zijdemans:stabilize-sse

Conversation

@tomas-zijdemans
Copy link
Contributor

I made the mistake of naming the file unstable_server_sent_event_stream.ts here. That should have been unstable_server_sent_event_parse_stream.ts . I was probably thinking that both the parser and serializer should be in the same file (but that's not following the encoder/decoder and stringify/parse convention in std).

Ideally server_sent_event_stream.ts would be renamed server_sent_event_stringify_stream.ts, but it's been stabilized. So I guess we are stuck with that name.

@tomas-zijdemans tomas-zijdemans requested a review from kt3k as a code owner March 15, 2026 19:24
@github-actions github-actions bot added the http label Mar 15, 2026
@codecov
Copy link

codecov bot commented Mar 15, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.36%. Comparing base (6df7f21) to head (b5077a1).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #7050      +/-   ##
==========================================
+ Coverage   94.01%   94.36%   +0.34%     
==========================================
  Files         627      627              
  Lines       50029    50034       +5     
  Branches     8807     8809       +2     
==========================================
+ Hits        47037    47216     +179     
+ Misses       2413     2251     -162     
+ Partials      579      567      -12     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant