doot.git/internal/handlers/meals.go, branch masterdoot — personal productivity web app
https://git.terst.org/doot.git/atom?h=master2026-03-18T10:04:57+00:00refactor: RF-03/06 extract groupMeals helper, eliminate convertSyncItemToTask wrapper2026-03-18T10:04:57+00:00Claudeagent@claude.ai2026-03-18T10:04:57+00:00urn:sha1:e85b42d373de55781af9d699b246c0d6a492aec1
RF-03: Extract shared groupMeals helper into internal/handlers/meals.go.
Both HandleTabMeals and BuildTimeline now call groupMeals instead of
duplicating the date+mealType grouping algorithm inline. CombinedMeal
gains ID and Meals fields to carry the first-meal ID and original records
needed by BuildTimeline when constructing TimelineItems.
RF-06: Add api.ConvertSyncItemToTask for single-item conversion.
ConvertSyncItemsToTasks now delegates to it, eliminating duplication.
The Handler.convertSyncItemToTask wrapper (which allocated a one-element
slice just to unwrap it) is deleted; its caller uses api.ConvertSyncItemToTask
directly. Covered by TestConvertSyncItemToTask in internal/api/todoist_test.go.
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>