Merge github/master: reconcile with parallel widget work

Another session pushed 27 commits in parallel covering quick-add, event
detail popups, recurrence display, overdue badges, a manual refresh button,
and its own fix for the same overdue-tasks bug (via a separate
GetOverdueNativeTasks fetch folded into BuildTimeline, rather than widening
GetNativeTasksByDateRange's bound directly). Reconciled rather than blindly
taking one side:

- Reverted GetNativeTasksByDateRange to its original bounded query and kept
  upstream's GetOverdueNativeTasks + BuildTimeline fold-in as the sole
  overdue mechanism for native tasks, to avoid double-counting overdue
  items (my widened query + their separate fetch would have both returned
  them). Re-pointed the regression test at the now-correct contract and
  added a store-level test for GetOverdueNativeTasks directly.
- Kept my GetGoogleTasksByDateRange fix as-is (single unbounded query) --
  upstream never touched Google Tasks overdue handling, so there's no
  duplication risk there.
- Rewove WidgetRoot's LazyColumn structure (added for scrolling) around
  upstream's new header buttons, pinned all-day event rows, and the
  enhanced TomorrowSection, none of which were written LazyColumn-aware
  since that work landed on this side only.
- Combined both sides' additions to TaskDetailActivity/TaskDetailSheet
  (description-edit detail popup + due-date reschedule label) and
  WidgetRepository/Actions (optimistic local removal + refresh button
  wiring) -- these were independent, non-overlapping features that both
  needed to survive.
- Renumbered the migration collision: both sides independently added a
  migration numbered 022. Card-description was already applied to the live
  production DB under that filename earlier this session (migrations are
  tracked by filename), so it keeps 022; the recurring-event-id migration,
  never deployed under any name here, moves to 023.

Verified: go build clean, full test suite passes (only the two
pre-existing agent-handler failures and the pre-existing models package
build error remain, both confirmed unrelated via git stash before this
session began), and a dry run against a copy of the live production
database applies both migrations cleanly with no re-run conflicts.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01EZ7ikw2ukGJFTHE3bJS7zL

1 files changed, 19 insertions, 6 deletions

diff --git a/internal/handlers/handlers.go b/internal/handlers/handlers.go
index aaf1d0d..408006d 100644
--- a/internal/handlers/handlers.go
+++ b/internal/handlers/handlers.go
@@ -314,9 +314,20 @@ func (h *Handler) fetchCalendarEvents(ctx context.Context, forceRefresh bool) ([
 	}
 
 	if len(enabledIDs) == 0 {
-		// No source_configs synced yet — fall back to the configured calendar ID
+		// No source_configs synced yet — fall back to the configured calendar
+		// ID(s). GoogleCalendarID is a single env var that itself holds a
+		// comma-separated list (see GOOGLE_CALENDAR_ID in .env) -- it must be
+		// split before use. Passing the raw joined string straight through
+		// as a single calendar ID (the previous behavior) sends Google's API
+		// a calendarId that matches nothing, failing every fetch with a 404 —
+		// this is a real production incident, not a hypothetical: it silently
+		// broke all calendar events (web and widget both) until fixed.
 		if len(configs) == 0 && h.config.GoogleCalendarID != "" {
-			enabledIDs = []string{h.config.GoogleCalendarID}
+			for _, id := range strings.Split(h.config.GoogleCalendarID, ",") {
+				if trimmed := strings.TrimSpace(id); trimmed != "" {
+					enabledIDs = append(enabledIDs, trimmed)
+				}
+			}
 		} else {
 			// Configs exist but all disabled — respect that
 			return nil, nil
@@ -326,10 +337,12 @@ func (h *Handler) fetchCalendarEvents(ctx context.Context, forceRefresh bool) ([
 	h.googleCalendarClient.SetCalendarIDs(enabledIDs)
 
 	fetcher := &CacheFetcher[models.CalendarEvent]{
-		Store:        h.store,
-		CacheKey:     store.CacheKeyGoogleCalendar,
-		TTLMinutes:   h.config.CacheTTLMinutes,
-		Fetch:        func(ctx context.Context) ([]models.CalendarEvent, error) { return h.googleCalendarClient.GetUpcomingEvents(ctx, 50) },
+		Store:      h.store,
+		CacheKey:   store.CacheKeyGoogleCalendar,
+		TTLMinutes: h.config.CacheTTLMinutes,
+		Fetch: func(ctx context.Context) ([]models.CalendarEvent, error) {
+			return h.googleCalendarClient.GetUpcomingEvents(ctx, 50)
+		},
 		GetFromCache: h.store.GetCalendarEvents,
 		SaveToCache:  h.store.SaveCalendarEvents,
 	}