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, 38 insertions, 11 deletions

diff --git a/internal/handlers/timeline_logic.go b/internal/handlers/timeline_logic.go
index 67ba09d..6fddd1d 100644
--- a/internal/handlers/timeline_logic.go
+++ b/internal/handlers/timeline_logic.go
@@ -81,16 +81,17 @@ func BuildTimeline(ctx context.Context, s *store.Store, start, end time.Time) ([
 		for _, event := range events {
 			endTime := event.End
 			item := models.TimelineItem{
-				ID:           event.ID,
-				Type:         models.TimelineItemTypeEvent,
-				Title:        event.Summary,
-				Time:         event.Start,
-				EndTime:      &endTime,
-				Description:  event.Description,
-				URL:          event.HTMLLink,
-				OriginalItem: event,
-				IsCompleted:  false,
-				Source:       "calendar",
+				ID:               event.ID,
+				Type:             models.TimelineItemTypeEvent,
+				Title:            event.Summary,
+				Time:             event.Start,
+				EndTime:          &endTime,
+				Description:      event.Description,
+				URL:              event.HTMLLink,
+				OriginalItem:     event,
+				IsCompleted:      false,
+				Source:           "calendar",
+				RecurringEventID: event.RecurringEventID,
 			}
 			item.ComputeDaySection(now)
 			items = append(items, item)
@@ -125,7 +126,10 @@ func BuildTimeline(ctx context.Context, s *store.Store, start, end time.Time) ([
 		}
 	}
 
-	// 6. Fetch native (doot-owned) tasks
+	// 6. Fetch native (doot-owned) tasks due within the range, plus any
+	// overdue tasks (due before the range's start) so they still surface --
+	// GetNativeTasksByDateRange's lower bound would otherwise drop them
+	// before ComputeDaySection ever gets a chance to mark them IsOverdue.
 	nativeDated, err := s.GetNativeTasksByDateRange(start, end)
 	if err != nil {
 		log.Printf("Warning: failed to read native tasks: %v", err)
@@ -149,6 +153,29 @@ func BuildTimeline(ctx context.Context, s *store.Store, start, end time.Time) ([
 		}
 	}
 
+	nativeOverdue, err := s.GetOverdueNativeTasks(start)
+	if err != nil {
+		log.Printf("Warning: failed to read overdue native tasks: %v", err)
+	} else {
+		for _, task := range nativeOverdue {
+			if task.DueDate == nil {
+				continue
+			}
+			item := models.TimelineItem{
+				ID:           task.ID,
+				Type:         models.TimelineItemTypeTask,
+				Title:        task.Content,
+				Time:         *task.DueDate,
+				Description:  task.Description,
+				OriginalItem: task,
+				IsCompleted:  task.Completed,
+				Source:       "doot",
+			}
+			item.ComputeDaySection(now)
+			items = append(items, item)
+		}
+	}
+
 	nativeUndated, err := s.GetUndatedNativeTasks()
 	if err != nil {
 		log.Printf("Warning: failed to read undated native tasks: %v", err)