cleanup: finish retired-schema forward cleanup coverage #359

New issue

Closed

opened 2026-05-11 21:02:05 +00:00 by barrettruth · 0 comments

barrettruth commented

2026-05-11 21:02:05 +00:00

Owner

Parent: #355

Problem

The compact migration baseline tracks retired schema names in tests, but the deployed-schema cleanup path only creates/drops a subset of retired tables. If any retired table exists on a deployed database, the forward cleanup should remove it deliberately.

Solution

Add a new forward migration if the existing deployed cleanup may already be applied:

do not rewrite already-merged migration history
drop any remaining retired tables from the canonical retired list
extend deployed-schema migration tests to include every retired table and assert all are gone

Acceptance criteria

The deployed cleanup path removes all retired login/reminder/share/system-config/keymap tables listed by schema tests.
Fresh baseline remains unchanged except for new journal metadata as needed.
Schema tests cover the forward path.

Parent: #355 ## Problem The compact migration baseline tracks retired schema names in tests, but the deployed-schema cleanup path only creates/drops a subset of retired tables. If any retired table exists on a deployed database, the forward cleanup should remove it deliberately. ## Solution Add a new forward migration if the existing deployed cleanup may already be applied: - do not rewrite already-merged migration history - drop any remaining retired tables from the canonical retired list - extend deployed-schema migration tests to include every retired table and assert all are gone ## Acceptance criteria - The deployed cleanup path removes all retired login/reminder/share/system-config/keymap tables listed by schema tests. - Fresh baseline remains unchanged except for new journal metadata as needed. - Schema tests cover the forward path.

barrettruth added the

track:core

type:cleanup

labels

2026-05-11 21:02:05 +00:00

barrettruth referenced this issue

2026-05-11 21:02:45 +00:00

tracker: foundation hardening before Google sync #355

barrettruth referenced this issue from a pull request that will close it,

2026-05-11 21:10:40 +00:00

cleanup: finish retired-schema forward cleanup coverage #374

barrettruth closed this issue