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/handlers/atoms.go | 5 ----- 1 file changed, 5 deletions(-) (limited to 'internal/handlers') diff --git a/internal/handlers/atoms.go b/internal/handlers/atoms.go index fe092a3..9474150 100644 --- a/internal/handlers/atoms.go +++ b/internal/handlers/atoms.go @@ -91,13 +91,8 @@ func SortAtomsByUrgency(atoms []models.Atom) { } // PartitionAtomsByTime separates atoms into current and future lists -// Recurring tasks that are future are excluded entirely func PartitionAtomsByTime(atoms []models.Atom) (current, future []models.Atom) { for _, a := range atoms { - // Don't show recurring tasks until the day they're due - if a.IsRecurring && a.IsFuture { - continue - } if a.IsFuture { future = append(future, a) } else { -- cgit v1.2.3