summaryrefslogtreecommitdiff
path: root/Hutch/Views/Tickets/TrackerListViewModel.swift
diff options
context:
space:
mode:
authorChristian Cleberg <[email protected]>2026-05-06 20:41:49 -0500
committerChristian Cleberg <[email protected]>2026-05-06 20:41:49 -0500
commit205cf2687ce48f3d004792ff223422a86cc6bdbc (patch)
tree7edb78cede354e79aa8abd764ed9c5047e0ee168 /Hutch/Views/Tickets/TrackerListViewModel.swift
parent57e4f34b4613c09beb0cb757ac2ba2b43cc04daf (diff)
downloadhutch-205cf2687ce48f3d004792ff223422a86cc6bdbc.tar.gz
hutch-205cf2687ce48f3d004792ff223422a86cc6bdbc.tar.bz2
hutch-205cf2687ce48f3d004792ff223422a86cc6bdbc.zip
feat(cache): persist read-only API responsesv3.3.1
Add a bounded stale-while-revalidate cache at the Sourcehut API boundary with stable keys, centralized TTLs, request coalescing, payload hashing, and LRU disk pruning. Cache high-value read-only repo, build, ticket, project, profile, paste, and Home/Work Queue data while keeping mutations network-only and invalidating related prefixes after successful writes. Add focused cache tests and implementation notes.
Diffstat (limited to 'Hutch/Views/Tickets/TrackerListViewModel.swift')
-rw-r--r--Hutch/Views/Tickets/TrackerListViewModel.swift27
1 files changed, 23 insertions, 4 deletions
diff --git a/Hutch/Views/Tickets/TrackerListViewModel.swift b/Hutch/Views/Tickets/TrackerListViewModel.swift
index f18bb7b..e4db23c 100644
--- a/Hutch/Views/Tickets/TrackerListViewModel.swift
+++ b/Hutch/Views/Tickets/TrackerListViewModel.swift
@@ -118,6 +118,12 @@ final class TrackerListViewModel {
hasMore = true
do {
+ if trackers.isEmpty, let cached = try? await fetchPage(cursor: nil, policy: .cacheOnly) {
+ trackers = cached.results
+ cursor = cached.cursor
+ hasMore = cached.cursor != nil
+ isLoading = false
+ }
let page = try await fetchPage(cursor: nil)
trackers = page.results
cursor = page.cursor
@@ -181,6 +187,7 @@ final class TrackerListViewModel {
responseType: CreateTrackerResponse.self
)
let tracker = result.createTracker
+ await invalidateTrackerCaches()
trackers.insert(tracker, at: 0)
return tracker
} catch {
@@ -224,6 +231,7 @@ final class TrackerListViewModel {
],
responseType: UpdateTrackerResponse.self
)
+ await invalidateTrackerCaches()
applyTrackerUpdate(result.updateTracker)
return result.updateTracker
} catch {
@@ -246,6 +254,7 @@ final class TrackerListViewModel {
variables: ["id": tracker.id],
responseType: DeleteTrackerResponse.self
)
+ await invalidateTrackerCaches()
trackers.removeAll { $0.id == tracker.id }
await loadTrackers()
return true
@@ -269,18 +278,22 @@ final class TrackerListViewModel {
// MARK: - Private
- private func fetchPage(cursor: String?) async throws -> TrackersPage {
+ private func fetchPage(cursor: String?, policy: CachePolicy = .cacheFirstThenRefresh) async throws -> TrackersPage {
var variables: [String: any Sendable] = [:]
if let cursor {
variables["cursor"] = cursor
}
- let result = try await client.execute(
+ let cached = try await client.executeCached(
service: .todo,
query: Self.query,
variables: variables.isEmpty ? nil : variables,
- responseType: TrackersResponse.self
+ responseType: TrackersResponse.self,
+ cacheKey: APICacheKeys.trackers(cursor: cursor),
+ resourceType: .ticketList,
+ ttl: APICacheTTLs.ticketList,
+ policy: policy
)
- return result.trackers
+ return cached.value.trackers
}
private struct CreateTrackerResponse: Decodable, Sendable {
@@ -290,4 +303,10 @@ final class TrackerListViewModel {
private func trackerCreationErrorMessage(for error: Error) -> String {
"Couldn’t create the tracker. \(error.userFacingMessage)"
}
+
+ private func invalidateTrackerCaches() async {
+ await client.invalidateCache(prefix: APICacheKeys.prefix(SRHTService.todo.rawValue, "trackers"))
+ await client.invalidateCache(prefix: APICacheKeys.prefix(SRHTService.todo.rawValue, "tracker"))
+ await client.invalidateCache(prefix: APICacheKeys.prefix("home"))
+ }
}