cleanup: finish retired-schema forward cleanup coverage #374

Merged

barrettruth merged 1 commit from issue-359-retired-schema-cleanup into main

2026-05-11 21:23:26 +00:00

barrettruth commented

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

Owner

Problem

The deployed-schema migration test only exercised a subset of the retired tables tracked by the canonical retired schema list, leaving the forward cleanup path under-covered.

Solution

add a new forward migration that drops every retired table in the canonical list with IF EXISTS guards
extend the deployed-schema fixture to create every retired table from that list
assert the deployed migration removes the full retired table list while preserving the fresh baseline

Verification

nix develop --command pnpm exec vitest run tests/core/schema.test.ts --reporter=verbose
nix develop --command just ci

Closes #359

## Problem The deployed-schema migration test only exercised a subset of the retired tables tracked by the canonical retired schema list, leaving the forward cleanup path under-covered. ## Solution - add a new forward migration that drops every retired table in the canonical list with `IF EXISTS` guards - extend the deployed-schema fixture to create every retired table from that list - assert the deployed migration removes the full retired table list while preserving the fresh baseline ## Verification - `nix develop --command pnpm exec vitest run tests/core/schema.test.ts --reporter=verbose` - `nix develop --command just ci` Closes #359

barrettruth added 1 commit

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

cleanup: finish retired schema cleanup coverage

quality / Test (pull_request) Successful in 17s

Details

quality / Lint (pull_request) Successful in 22s

Details

quality / Build (pull_request) Successful in 45s

Details

352f866e76