Retire reminders subsystem #288

Merged

barrettruth merged 1 commit from shutdown/reminders into main

2026-05-11 16:21:31 +00:00

barrettruth commented

2026-05-11 16:07:36 +00:00

Owner

Problem

Delta is narrowing scope before launch. The reminders subsystem added a second product surface (endpoint management, delivery transports, worker scheduling, proof-of-consent pages, and task-level reminder editing) that no longer fits the current self-hosted tracker direction. This supersedes the earlier reminder-completion plan in closed #231 and replaces the follow-up work in #189 and #243 with removal.

Solution

Remove reminder API routes, delivery/endpoints settings UI, task-panel reminder controls, worker startup, runtime modules, validation helpers, and proof-of-consent pages.
Remove reminder fields from task save/copy/recurrence flows so tasks remain focused on tracker data.
Keep the existing reminder tables inert for now so this PR avoids a risky data migration while the runtime surface disappears.
Update focused tests around task saving, settings primitives, and the integrations settings page.

Verification

pnpm vitest run tests/lib/task-panel-save.test.ts tests/core/task-detail-save.test.ts tests/lib/integrations-section.test.ts tests/lib/settings-primitives.test.ts tests/lib/calendar-settings-section.test.ts
pnpm tsc --noEmit
just ci

just ci still reports the existing Biome specificity warning in src/components/calendar/fc-styles.css; it does not fail the check.

Closes #286
Closes #189
Closes #243
Supersedes closed #231

## Problem Delta is narrowing scope before launch. The reminders subsystem added a second product surface (endpoint management, delivery transports, worker scheduling, proof-of-consent pages, and task-level reminder editing) that no longer fits the current self-hosted tracker direction. This supersedes the earlier reminder-completion plan in closed #231 and replaces the follow-up work in #189 and #243 with removal. ## Solution - Remove reminder API routes, delivery/endpoints settings UI, task-panel reminder controls, worker startup, runtime modules, validation helpers, and proof-of-consent pages. - Remove reminder fields from task save/copy/recurrence flows so tasks remain focused on tracker data. - Keep the existing reminder tables inert for now so this PR avoids a risky data migration while the runtime surface disappears. - Update focused tests around task saving, settings primitives, and the integrations settings page. ## Verification - `pnpm vitest run tests/lib/task-panel-save.test.ts tests/core/task-detail-save.test.ts tests/lib/integrations-section.test.ts tests/lib/settings-primitives.test.ts tests/lib/calendar-settings-section.test.ts` - `pnpm tsc --noEmit` - `just ci` `just ci` still reports the existing Biome specificity warning in `src/components/calendar/fc-styles.css`; it does not fail the check. Closes #286 Closes #189 Closes #243 Supersedes closed #231

barrettruth added 1 commit

2026-05-11 16:07:36 +00:00

Retire reminders subsystem

quality / Test (pull_request) Successful in 16s

Details

quality / Lint (pull_request) Successful in 21s

Details

quality / Build (pull_request) Successful in 47s

Details

e344979ef3