From aa5d0e4bf849625ab6876129950c71992ea322b8 Mon Sep 17 00:00:00 2001 From: Peter Stone Date: Wed, 15 Jul 2026 02:15:00 +0000 Subject: chore: remove dead IsRecurring field and its always-false branch Task.IsRecurring/Atom.IsRecurring were never assigned anywhere in the codebase (confirmed via full-repo search), so PartitionAtomsByTime's "hide future recurring tasks until due" branch was permanently dead. RecurrenceSeriesID is the real recurring indicator now; wiring up "hide future recurring successors" as an actual feature is a separate design decision, not part of this cleanup pass. Co-Authored-By: Claude Sonnet 5 Claude-Session: https://claude.ai/code/session_01VTUSAEKfsPc6WGDq45yPHD --- internal/models/types.go | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) (limited to 'internal/models/types.go') diff --git a/internal/models/types.go b/internal/models/types.go index f74d264..fdde455 100644 --- a/internal/models/types.go +++ b/internal/models/types.go @@ -19,11 +19,9 @@ type Task struct { Labels []string `json:"labels"` URL string `json:"url"` CreatedAt time.Time `json:"created_at"` - IsRecurring bool `json:"is_recurring"` // Recurrence (doot-native tasks only). RecurrenceSeriesID != "" is the - // real "is this task recurring" indicator -- IsRecurring above predates - // this feature and is never set by it. + // indicator that a task is recurring. RecurrenceFreq string `json:"recurrence_freq,omitempty"` RecurrenceInterval int `json:"recurrence_interval,omitempty"` RecurrenceWeekdays []int `json:"recurrence_weekdays,omitempty"` -- cgit v1.2.3