fix(app): handle iOS backgrounding - resync session and reconnect SSE on resume#11671
Open
athal7 wants to merge 1 commit intoanomalyco:devfrom
Open
fix(app): handle iOS backgrounding - resync session and reconnect SSE on resume#11671athal7 wants to merge 1 commit intoanomalyco:devfrom
athal7 wants to merge 1 commit intoanomalyco:devfrom
Conversation
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. |
Contributor
|
The following comment was made by an LLM, it may be inaccurate: No duplicate PRs found |
athal7
force-pushed
the
fix/ios-backgrounding-session-resync
branch
from
9a3bb8c to
de93714
Compare
athal7
force-pushed
the
fix/ios-backgrounding-session-resync
branch
7 times, most recently
from
e059811 to
3ab571d
Compare
… 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 anomalyco#10721
athal7
force-pushed
the
fix/ios-backgrounding-session-resync
branch
from
3ab571d to
3c54233
Compare
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
1 participant
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
visibility.tsutility - Tracks page visibility state withwasRecentlyBackgrounded(),getLastHiddenAt(),isHidden()Testing
trim.8FDE21FE-1ABE-460B-94A2-807440206C3E.MOV
Related
Revives approach from closed PR #10723