Skip to content

fix: keep SSE clients connected for live updates - #17

Merged
wjc2821296948 merged 1 commit into
Low-Rate:mainfrom
lijiashu1:fix/sse-client-lifecycle
Oct 20, 2026
Merged

wjc2821296948 merged 1 commit into
Low-Rate:mainfrom
lijiashu1:fix/sse-client-lifecycle

Conversation

@lijiashu1

Copy link
Copy Markdown
Member

Fix the event-stream lifecycle so connected players continue receiving room and game-state broadcasts. Add a regression test for room-list events.

wjc2821296948
wjc2821296948 previously approved these changes Oct 19, 2026
auto-merge was automatically disabled October 20, 2026 09:50

Head branch was pushed to by a user without write access

@lijiashu1
lijiashu1 force-pushed the fix/sse-client-lifecycle branch from aa387d2 to 4eb5f09 Compare October 20, 2026 09:50

Copy link
Copy Markdown
Member Author

I rebased this PR onto the current main and kept the regression test that verifies SSE clients receive subsequent room-list events. The branch is now conflict-free. The previous approval was dismissed by GitHub after the head changed, so a fresh review is needed; the repository also requires a test status check, but no workflow runs are currently available.

auto-merge was automatically disabled October 20, 2026 09:52

Head branch was pushed to by a user without write access

@lijiashu1
lijiashu1 force-pushed the fix/sse-client-lifecycle branch from 4eb5f09 to 86e19f6 Compare October 20, 2026 09:52
@wjc2821296948
wjc2821296948 merged commit 6c1c356 into Low-Rate:main Oct 20, 2026
1 check 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.

3 participants