Packages/Pachyderm/Sources/Pachyderm/Model/TimelineMarkers.swift

@@ -7,83 +7,34 @@
 
 import Foundation
 
-public struct TimelineMarkers {
-    private init() {}
+public struct TimelineMarkers: Decodable, Sendable {
+    public let home: Marker?
+    public let notifications: Marker?
     
-    public static func request<T: TimelineMarkerType>(timeline: T) -> Request<TimelineMarker<T.Payload>> {
-        Request(method: .get, path: "/api/v1/markers", queryParameters: ["timeline[]" => T.name])
+    public static func request(timelines: [Timeline]) -> Request<TimelineMarkers> {
+        return Request(method: .get, path: "/api/v1/markers", queryParameters: "timeline[]" => timelines.map(\.rawValue))
     }
     
-    public static func update<T: TimelineMarkerType>(timeline: T, lastReadID: String) -> Request<Empty> {
-        Request(method: .post, path: "/api/v1/markers", body: ParametersBody([
-            "\(T.name)[last_read_id]" => lastReadID
+    public static func update(timeline: Timeline, lastReadID: String) -> Request<Empty> {
+        return Request(method: .post, path: "/api/v1/markers", body: ParametersBody([
+            "\(timeline.rawValue)[last_read_id]" => lastReadID,
         ]))
     }
-}
     
-public struct TimelineMarker<Payload: TimelineMarkerTypePayload>: Decodable, Sendable {
-    let payload: Payload
-    
-    public var lastReadID: String {
-        payload.payload.lastReadID
-    }
-    public var version: Int {
-        payload.payload.version
-    }
-    public var updatedAt: Date {
-        payload.payload.updatedAt
+    public enum Timeline: String {
+        case home
+        case notifications
     }
     
-    public init(from decoder: any Decoder) throws {
-        self.payload = try Payload(from: decoder)
+    public struct Marker: Decodable, Sendable {
+        public let lastReadID: String
+        public let version: Int
+        public let updatedAt: Date
+        
+        enum CodingKeys: String, CodingKey {
+            case lastReadID = "last_read_id"
+            case version
+            case updatedAt = "updated_at"
+        }
     }
 }
-
-public protocol TimelineMarkerTypePayload: Decodable, Sendable {
-    var payload: MarkerPayload { get }
-}
-
-public struct HomeMarkerPayload: TimelineMarkerTypePayload {
-    public var home: MarkerPayload
-    public var payload: MarkerPayload { home }
-}
-
-public struct NotificationsMarkerPayload: TimelineMarkerTypePayload {
-    public var notifications: MarkerPayload
-    public var payload: MarkerPayload { notifications }
-}
-
-public struct MarkerPayload: Decodable, Sendable {
-    public let lastReadID: String
-    public let version: Int
-    public let updatedAt: Date
-    
-    enum CodingKeys: String, CodingKey {
-        case lastReadID = "last_read_id"
-        case version
-        case updatedAt = "updated_at"
-    }
-}
-
-public protocol TimelineMarkerType {
-    static var name: String { get }
-    associatedtype Payload: TimelineMarkerTypePayload
-}
-
-extension TimelineMarkerType where Self == HomeMarker {
-    public static var home: Self { .init() }
-}
-
-extension TimelineMarkerType where Self == NotificationsMarker {
-    public static var notifications: Self { .init() }
-}
-
-public struct HomeMarker: TimelineMarkerType {
-    public typealias Payload = HomeMarkerPayload
-    public static var name: String { "home" }
-}
-
-public struct NotificationsMarker: TimelineMarkerType {
-    public typealias Payload = NotificationsMarkerPayload
-    public static var name: String { "notifications" }
-}

Tusker/Screens/Notifications/NotificationsCollectionViewController.swift

@@ -12,9 +12,6 @@ import Combine
 #if canImport(Sentry)
 import Sentry
 #endif
-import OSLog
-
-private let logger = Logger(subsystem: Bundle.main.bundleIdentifier!, category: "NotificationsCVC")
 
 class NotificationsCollectionViewController: UIViewController, TimelineLikeCollectionViewController, CollectionViewController {
     
@@ -35,9 +32,6 @@ class NotificationsCollectionViewController: UIViewController, TimelineLikeColle
     private var newer: RequestRange?
     private var older: RequestRange?
     
-    var updatesNotificationsMarker: Bool = false
-    private var newestDisplayedNotification: Item?
-    
     init(allowedTypes: [Pachyderm.Notification.Kind], mastodonController: MastodonController) {
         self.allowedTypes = allowedTypes
         self.mastodonController = mastodonController
@@ -211,12 +205,6 @@ class NotificationsCollectionViewController: UIViewController, TimelineLikeColle
         }
     }
     
-    override func viewWillDisappear(_ animated: Bool) {
-        super.viewWillDisappear(animated)
-        
-        updateNotificationsMarkerIfNecessary()
-    }
-    
     @objc func refresh() {
         Task { @MainActor in
             if case .notLoadedInitial = controller.state {
@@ -323,23 +311,6 @@ class NotificationsCollectionViewController: UIViewController, TimelineLikeColle
         await apply(snapshot, animatingDifferences: true)
     }
     
-    private func updateNotificationsMarkerIfNecessary() {
-        guard updatesNotificationsMarker,
-              case let .group(group, _, _) = newestDisplayedNotification,
-              let notification = group.notifications.first else {
-            return
-        }
-        logger.debug("Updating notifications marker with \(notification.id)")
-        Task {
-            let req = TimelineMarkers.update(timeline: .notifications, lastReadID: notification.id)
-            do {
-                _ = try await mastodonController.run(req)
-            } catch {
-                logger.error("Failed to update notifications marker: \(String(describing: error))")
-            }
-        }
-    }
-    
 }
 
 extension NotificationsCollectionViewController {
@@ -574,21 +545,6 @@ extension NotificationsCollectionViewController: UICollectionViewDelegate {
         guard case .notifications = dataSource.sectionIdentifier(for: indexPath.section) else {
             return
         }
-        
-        if updatesNotificationsMarker {
-            let shouldUpdateNewestDisplayedNotification: Bool
-            if let newestDisplayedNotification,
-               let currentNewestIndexPath = dataSource.indexPath(for: newestDisplayedNotification) {
-                shouldUpdateNewestDisplayedNotification = indexPath < currentNewestIndexPath
-            } else {
-                shouldUpdateNewestDisplayedNotification = true
-            }
-            if shouldUpdateNewestDisplayedNotification,
-               let item = dataSource.itemIdentifier(for: indexPath) {
-                newestDisplayedNotification = item
-            }
-        }
-        
         let itemsInSection = collectionView.numberOfItems(inSection: indexPath.section)
         if indexPath.row == itemsInSection - 1 {
             Task {
@@ -809,9 +765,3 @@ extension NotificationsCollectionViewController: StatusBarTappableViewController
         return .stop
     }
 }
-
-extension NotificationsCollectionViewController: BackgroundableViewController {
-    func sceneDidEnterBackground() {
-        updateNotificationsMarkerIfNecessary()
-    }
-}

Tusker/Screens/Notifications/NotificationsPageViewController.swift

@@ -24,7 +24,6 @@ class NotificationsPageViewController: SegmentedPageViewController<Notifications
             let vc = NotificationsCollectionViewController(allowedTypes: page.allowedTypes, mastodonController: mastodonController)
             vc.title = page.title
             vc.userActivity = page.userActivity(accountID: mastodonController.accountInfo!.id)
-            vc.updatesNotificationsMarker = page == .all
             return vc
         }
         

Tusker/Screens/Timeline/TimelineViewController.swift

@@ -387,6 +387,16 @@ class TimelineViewController: UIViewController, TimelineLikeCollectionViewContro
                     _ = try await mastodonController.run(req)
                 } catch {
                     stateRestorationLogger.error("TimelineViewController: failed to update timeline marker: \(String(describing: error))")
+                    
+                    #if canImport(Sentry)
+                    if let error = error as? Client.Error,
+                       case .networkError(_) = error.type {
+                        return
+                    }
+                    let event = Event(error: error)
+                    event.message = SentryMessage(formatted: "Failed to update timeline marker: \(String(describing: error))")
+                    SentrySDK.capture(event: event)
+                    #endif
                 }
             }
         }
@@ -563,9 +573,12 @@ class TimelineViewController: UIViewController, TimelineLikeCollectionViewContro
     @MainActor
     private func restoreStatusesFromMarkerPosition() async -> Bool {
         do {
-            let (marker, _) = try await mastodonController.run(TimelineMarkers.request(timeline: .home))
-            async let status = try await mastodonController.run(Client.getStatus(id: marker.lastReadID)).0
-            async let olderStatuses = try await mastodonController.run(Client.getStatuses(timeline: .home, range: .before(id: marker.lastReadID, count: Self.pageSize))).0
+            let (marker, _) = try await mastodonController.run(TimelineMarkers.request(timelines: [.home]))
+            guard let home = marker.home else {
+                return false
+            }
+            async let status = try await mastodonController.run(Client.getStatus(id: home.lastReadID)).0
+            async let olderStatuses = try await mastodonController.run(Client.getStatuses(timeline: .home, range: .before(id: home.lastReadID, count: Self.pageSize))).0
             
             let allStatuses = try await [status] + olderStatuses
             await mastodonController.persistentContainer.addAll(statuses: allStatuses)
@@ -577,11 +590,21 @@ class TimelineViewController: UIViewController, TimelineLikeCollectionViewContro
             await apply(snapshot, animatingDifferences: false)
             collectionView.contentOffset = CGPoint(x: 0, y: -collectionView.adjustedContentInset.top)
             
-            stateRestorationLogger.debug("TimelineViewController: restored from timeline marker with last read ID: \(marker.lastReadID)")
+            stateRestorationLogger.debug("TimelineViewController: restored from timeline marker with last read ID: \(home.lastReadID)")
             
             return true
         } catch {
             stateRestorationLogger.error("TimelineViewController: failed to load from timeline marker: \(String(describing: error))")
+            
+            #if canImport(Sentry)
+            if let error = error as? Client.Error,
+               case .networkError(_) = error.type {
+                return false
+            }
+            let event = Event(error: error)
+            event.message = SentryMessage(formatted: "Failed to load from timeline marker: \(String(describing: error))")
+            SentrySDK.capture(event: event)
+            #endif
             return false
         }
     }