fix(app): handle iOS backgrounding - resync session and reconnect SSE on resume - #11671
Closed
athal7 wants to merge 1 commit into
Closed
fix(app): handle iOS backgrounding - resync session and reconnect SSE on resume#11671athal7 wants to merge 1 commit into
athal7 wants to merge 1 commit into
Conversation
Copy link
Copy Markdown
Contributor
Sorry, something went wrong.
Copy link
Copy Markdown
Contributor
|
Thanks for your contribution! This PR doesn't have a linked issue. All PRs must reference an existing issue. Please:
See CONTRIBUTING.md for details. |
Sorry, something went wrong.
Copy link
Copy Markdown
Contributor
|
The following comment was made by an LLM, it may be inaccurate: No duplicate PRs found |
Sorry, something went wrong.
athal7
force-pushed
the
fix/ios-backgrounding-session-resync
branch
from
9a3bb8c to
de93714
Compare
February 1, 2026 21:33
athal7
marked this pull request as ready for review
February 1, 2026 21:34
athal7
force-pushed
the
fix/ios-backgrounding-session-resync
branch
7 times, most recently
from
e059811 to
3ab571d
Compare
February 3, 2026 11:15
… on resume - Add visibilitychange listener in session.tsx to resync when tab becomes visible - Refactor global-sdk.tsx SSE consumption with auto-reconnect on clean disconnect - Use exponential backoff (1s-30s) and pause reconnects while hidden - Add visibility tracking utility to detect recent backgrounding - Gate error toasts in prompt-input.tsx when failure likely due to backgrounding - Check if message landed before removing optimistic UI on resume Fixes #10721
athal7
force-pushed
the
fix/ios-backgrounding-session-resync
branch
from
3ab571d to
3c54233
Compare
February 6, 2026 17:21
Copy link
Copy Markdown
Contributor
Author
|
Closing — mobile UI and PWA issues have been fixed upstream. |
Sorry, something went wrong.
Copy link
Copy Markdown
Contributor
Author
|
iOS backgrounding and SSE reconnection have been fixed upstream. Closing this PR as no longer needed. |
Sorry, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Fixes iOS Safari backgrounding issues where the browser kills network connections when the app is backgrounded. When users return, requests would fail and confusing error toasts would appear.
Closes #10721
Changes
Testing
- 10 unit tests for visibility tracking utility
- Manually verified on iOS Safari with side-by-side comparison video:
- Before: Error toast appears after backgrounding
- After: No error toast, graceful recovery
trim.8FDE21FE-1ABE-460B-94A2-807440206C3E.MOVRelated
Revives approach from closed PR #10723