Read and write the widget snapshot through CalendarrCore
The snapshot format was an internal detail shared between the app and its own widget extension. A second Mac app needs the same data, and the way to give it that is a documented contract rather than a format it reverse-engineers and then drifts from. Both targets now link CalendarrCore and go through SnapshotStore. Shared/WidgetData.swift becomes a thin facade. The typealiases and the flat colour accessors exist so the ~70 existing call sites across the app and the widget views compile unchanged; they are a migration convenience, not a design. The snapshot now carries what a reader outside this app actually needs: coverageStart / coverageEnd, because the published window is ~7 days back and ~42 ahead. Outside it the snapshot holds no information, which is not the same as holding no events — and only the writer knows where that edge is. Without it a consumer renders a convincingly empty March and is simply wrong. isLoggedIn plus a session record, so a reader can say "sign in to Calendarr" rather than "open Calendarr once". The events are deleted on sign-out, so the absence of a cache alone cannot tell those two apart. writerVersion, purely so a mismatch between the two apps is diagnosable. The coverage constants move to SnapshotCoverage, so the writer and the code that reconstructs the window for older files can no longer disagree about it. CalendarStore needs an explicit `import CalendarrCore` because the app target builds with SWIFT_UPCOMING_FEATURE_MEMBER_IMPORT_VISIBILITY, which requires the defining module to be imported directly rather than picked up transitively. Verified: builds for iOS and Mac Catalyst; CalendarrKit's 11 tests pass. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This commit is contained in:
@@ -41,16 +41,17 @@ struct CalendarrTimelineProvider: AppIntentTimelineProvider {
|
||||
let ids = (config.selectedCalendars ?? []).map { $0.id }
|
||||
guard !ids.isEmpty else { return snapshot } // nothing selected = show all
|
||||
let keep = Set(ids)
|
||||
// Filtering narrows the events but not the window: the snapshot still
|
||||
// speaks for the same range, it just has fewer calendars in it.
|
||||
return WidgetSnapshot(
|
||||
writtenAt: snapshot.writtenAt,
|
||||
events: snapshot.events.filter { keep.contains($0.calendarKey) },
|
||||
todayColorHex: snapshot.todayColorHex,
|
||||
textColorHex: snapshot.textColorHex,
|
||||
backgroundColorHex: snapshot.backgroundColorHex,
|
||||
lineColorHex: snapshot.lineColorHex,
|
||||
primaryColorHex: snapshot.primaryColorHex,
|
||||
accentColorHex: snapshot.accentColorHex,
|
||||
language: snapshot.language
|
||||
writtenAt: snapshot.writtenAt,
|
||||
coverageStart: snapshot.coverageStart,
|
||||
coverageEnd: snapshot.coverageEnd,
|
||||
isLoggedIn: snapshot.isLoggedIn,
|
||||
writerVersion: snapshot.writerVersion,
|
||||
events: snapshot.events.filter { keep.contains($0.calendarKey) },
|
||||
theme: snapshot.theme,
|
||||
language: snapshot.language
|
||||
)
|
||||
}
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user