From cc564717b2781e4b31664580da0db84422819854 Mon Sep 17 00:00:00 2001 From: Hariharan Mudaliar Date: Mon, 24 Nov 2025 09:28:37 +0530 Subject: [PATCH] Refactor: Remove redundant reminder EventKit calls --- .../ReminderLiveActivityManager.swift | 27 +------------------ 1 file changed, 1 insertion(+), 26 deletions(-) diff --git a/DynamicIsland/managers/ReminderLiveActivityManager.swift b/DynamicIsland/managers/ReminderLiveActivityManager.swift index 7b74353..ad03ca9 100644 --- a/DynamicIsland/managers/ReminderLiveActivityManager.swift +++ b/DynamicIsland/managers/ReminderLiveActivityManager.swift @@ -43,7 +43,6 @@ final class ReminderLiveActivityManager: ObservableObject { case eventStoreChange case calendarListChange case calendarEventsUpdate - case fallback case evaluation case hover case ticker @@ -54,7 +53,6 @@ final class ReminderLiveActivityManager: ObservableObject { private var cancellables = Set() private var tickerTask: Task? { didSet { oldValue?.cancel() } } private var evaluationTask: Task? - private var fallbackRefreshTask: Task? private var refreshTask: Task? private var pendingRefreshTask: Task? private var pendingRefreshForce = false @@ -152,8 +150,6 @@ final class ReminderLiveActivityManager: ObservableObject { tickerTask = nil evaluationTask?.cancel() evaluationTask = nil - fallbackRefreshTask?.cancel() - fallbackRefreshTask = nil pendingRefreshTask?.cancel() pendingRefreshTask = nil pendingRefreshForce = false @@ -230,22 +226,6 @@ final class ReminderLiveActivityManager: ObservableObject { } } - private func scheduleFallbackRefresh() { - fallbackRefreshTask?.cancel() - let delay: TimeInterval = 15 * 60 - logger.debug("[Reminder] Scheduling fallback refresh in \(delay, format: .fixed(precision: 0))s") - fallbackRefreshTask = Task { [weak self] in - guard let self else { return } - do { - try await Task.sleep(nanoseconds: UInt64(delay * 1_000_000_000)) - } catch { - self.logger.debug("[Reminder] Cancelled pending fallback refresh before execution") - return - } - await self.scheduleRefresh(force: true, reason: .fallback) - } - } - private func scheduleRefresh(force: Bool, reason: RefreshReason = .other) { let wasPending = pendingRefreshTask != nil pendingRefreshForce = pendingRefreshForce || force @@ -348,7 +328,6 @@ final class ReminderLiveActivityManager: ObservableObject { } private func handleEntrySelection(_ entry: ReminderEntry?, referenceDate: Date) { - fallbackRefreshTask?.cancel() nextReminder = entry hasShownCriticalSneakPeek = false Task { await self.evaluateCurrentState(at: referenceDate) } @@ -368,8 +347,7 @@ final class ReminderLiveActivityManager: ObservableObject { guard let first = upcoming.first else { deactivateReminder() - logger.debug("[Reminder] No upcoming reminders found; scheduling fallback refresh") - scheduleFallbackRefresh() + logger.debug("[Reminder] No upcoming reminders found; awaiting next calendar update") return } @@ -468,9 +446,6 @@ final class ReminderLiveActivityManager: ObservableObject { } stopTicker() hasShownCriticalSneakPeek = false - if fallbackRefreshTask == nil { - scheduleFallbackRefresh() - } return }