From 465093343ddd398ce5f6377fc9c472d8251c618b Mon Sep 17 00:00:00 2001 From: Peter Stone Date: Fri, 23 Jan 2026 21:37:18 -1000 Subject: Refactor: reduce code duplication with shared abstractions - Add BaseClient HTTP abstraction (internal/api/http.go) to eliminate duplicated HTTP boilerplate across Todoist, Trello, and PlanToEat clients - Add response helpers (internal/handlers/response.go) for JSON/HTML responses - Add generic cache wrapper (internal/handlers/cache.go) using Go generics - Consolidate HandleCompleteAtom/HandleUncompleteAtom into handleAtomToggle - Merge TabsHandler into Handler, delete tabs.go - Extract sortTasksByUrgency and filterAndSortTrelloTasks helpers - Update tests to work with new BaseClient structure Co-Authored-By: Claude Opus 4.5 --- cmd/dashboard/main.go | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) (limited to 'cmd/dashboard') diff --git a/cmd/dashboard/main.go b/cmd/dashboard/main.go index 050c8d0..8716db9 100644 --- a/cmd/dashboard/main.go +++ b/cmd/dashboard/main.go @@ -100,7 +100,6 @@ func main() { // Initialize handlers h := handlers.New(db, todoistClient, trelloClient, planToEatClient, googleCalendarClient, cfg) - tabsHandler := handlers.NewTabsHandler(db, googleCalendarClient, cfg.TemplateDir) // Set up router r := chi.NewRouter() @@ -133,9 +132,9 @@ func main() { r.Get("/api/boards", h.HandleGetBoards) // Tab routes for HTMX - r.Get("/tabs/tasks", tabsHandler.HandleTasks) - r.Get("/tabs/planning", tabsHandler.HandlePlanning) - r.Get("/tabs/meals", tabsHandler.HandleMeals) + r.Get("/tabs/tasks", h.HandleTabTasks) + r.Get("/tabs/planning", h.HandleTabPlanning) + r.Get("/tabs/meals", h.HandleTabMeals) r.Post("/tabs/refresh", h.HandleRefreshTab) // Trello card operations -- cgit v1.2.3