Skip to content

Fix IOSLiveActivitiesUpdated not firing after start() on iOS 17.2+ - #132

Merged
rlepinski merged 1 commit into
mainfrom
fix/live-activity-start-ios-17-2
Jul 10, 2026
Merged

rlepinski merged 1 commit into
mainfrom
fix/live-activity-start-ios-17-2

Conversation

@crow

@crow crow commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

On iOS 17.2+, start() skipped checkForActivities() assuming pushToStartTokenUpdates would trigger watcher setup. That stream fires at launch and on system token refreshes, but not on direct app-initiated starts — so startWatchingActivityUpdates was never registered for the new activity and no IOSLiveActivitiesUpdated events were emitted.

Fix: after start() on iOS 17.2+, set up the watcher and emit the initial event inline, mirroring what checkForActivities() would have done. Guarded by activityState[result.id] == nil so it's a no-op if a concurrent pushToStartTokenUpdates already handled it.

On iOS 17.2+, start() skipped checkForActivities() assuming pushToStartTokenUpdates
would trigger watcher setup. That stream fires at launch/token refresh but not on
direct app-initiated starts, so no state changes were ever observed for the new activity.

Fix: set up the watcher and emit the initial event inline after start() on iOS 17.2+,
mirroring what checkForActivities() would have done. Guarded by activityState nil check
so it's a no-op if a concurrent pushToStartTokenUpdates already handled it.
@crow
crow requested a review from a team as a code owner June 23, 2026 13:19
@rlepinski
rlepinski merged commit b5abb66 into main Jul 10, 2026
3 checks passed
@rlepinski
rlepinski deleted the fix/live-activity-start-ios-17-2 branch July 10, 2026 16:54
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.

4 participants