feat(calendar): cache occurrences and notes within fetched range

[domhhv]

Skip redundant API calls when navigating from a wider range (month) to a narrower one (week/day) by tracking the last fetched range and checking containment before fetching. occurrences are re-indexed in memory on cache hit; notes skip the fetch entirely. mutations invalidate the cache.

Summary by CodeRabbit

• Performance Improvements

  • Added smart caching for notes and occurrences data, reducing unnecessary server requests when reviewing previously loaded date ranges.

• Bug Fixes

  • Improved timezone handling for date operations to ensure consistent accuracy across different time zones.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

📝 Walkthrough

Walkthrough

The PR introduces range-based caching for notes and occurrences. It updates the notes service to use ZonedDateTime with absolute string formatting, and modifies both notes and occurrences stores to track fetched date ranges and bypass API calls when requested ranges fall within cached data.

Changes

Cohort / File(s)	Summary
Notes Service Date Types src/services/notes.service.ts	Updated listNotes to accept ZonedDateTime parameters instead of CalendarDateTime and changed date formatting from toString() to toAbsoluteString() for database queries.
Notes Store Caching src/stores/notes.store.ts	Added notesFetchedRange field to cache loaded note ranges; fetchNotes now checks cache before fetching and skips API calls for cached ranges; implemented cache invalidation on mutations (addNote, clearNotes, deleteNote, updateNote); updated date handling to use toZoned with getLocalTimeZone().
Occurrences Store Caching src/stores/occurrences.store.ts	Added occurrencesFetchedRange field to track cached date ranges; enhanced fetchOccurrences with cache logic that rebuilds occurrencesByDate locally from existing data when requested range is fully covered by cache, avoiding API calls; implemented cache invalidation on mutations (addOccurrence, clearOccurrences, removeOccurrence, updateOccurrence).

Sequence Diagram(s)

sequenceDiagram
    participant Component
    participant Store as Notes/Occurrences Store
    participant Cache as Cached Range
    participant Service as Notes/Occurrences Service
    participant API as Backend API

    rect rgba(173, 216, 230, 0.5)
    note over Component,API: Cached Path
    Component->>Store: fetchNotes/fetchOccurrences(range)
    Store->>Cache: Check if range covered?
    Cache-->>Store: Yes, cached
    Store->>Store: Rebuild from existing state
    Store-->>Component: Return cached data
    end

    rect rgba(255, 182, 193, 0.5)
    note over Component,API: Cache Miss Path
    Component->>Store: fetchNotes/fetchOccurrences(range)
    Store->>Cache: Check if range covered?
    Cache-->>Store: No, not cached
    Store->>Service: listNotes/listOccurrences(zoned range)
    Service->>API: Query with absolute date strings
    API-->>Service: Return results
    Service-->>Store: Results
    Store->>Cache: Update notesFetchedRange/occurrencesFetchedRange
    Store-->>Component: Return fresh data
    end

Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

• refactor(occurrence): groups zustand store by date #327: Modifies fetchOccurrences in src/stores/occurrences.store.ts with cache range logic and cache-shortcircuit mechanism for occurrences.
• refactor(zustand): manage notes separately from occurrences #330: Updates src/services/notes.service.ts listNotes function parameter types and date handling for notes listing.

Poem

🐰 Hop, hop, time's in zones!
Cache those ranges, save API bones,
Zoned dates now flow, absolute strings glow,
Fetch once, store it, reuse the data—
Smart hopping through time's data! 🕐

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title directly summarizes the main change: implementing client-side caching for calendar data (occurrences and notes) based on fetched date ranges.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch feat/cache-calendar-range-fetches

Tip

🧪 Unit Test Generation v2 is now available!

We have significantly improved our unit test generation capabilities.

To enable: Add this to your .coderabbit.yaml configuration:

reviews:
  finishing_touches:
    unit_tests:
      enabled: true

Try it out by using the @coderabbitai generate unit tests command on your code files or under ✨ Finishing Touches on the walkthrough!

Have feedback? Share your thoughts on our Discord thread!

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[relativeci]

#196 Bundle Size — 1.84MiB (+0.04%).

47ea7b5(current) vs 1dbb671 main#195(baseline)

Warning

Bundle contains 2 duplicate packages – View duplicate packages

Bundle metrics   2 changes 1 regression

	Current #196	Baseline #195
Initial JS	1.09MiB(+0.07%)	1.09MiB
Initial CSS	0B	0B
Cache Invalidation	87.13%	87.12%
Chunks	8	8
Assets	9	9
Modules	6676	6676
Duplicate Modules	0	0
Duplicate Code	0%	0%
Packages	223	223
Duplicate Packages	2	2

Bundle size by type   1 change 1 regression

	Current #196	Baseline #195
JS	1.61MiB (+0.05%)	1.6MiB
CSS	242.77KiB	242.77KiB

Bundle analysis report Branch feat/cache-calendar-range-fetche... Project dashboard

---

^{Generated by RelativeCI Documentation Report issue}