test(workbench): share historical database setup

[mldangelo-oai]

Summary

Share historical SQLite setup in migration tests while keeping each legacy schema and upgrade assertion explicit.

Changes

Extract the repeated setup for six historical database scenarios. Use the schema module directly for migration definitions and SQL splitting, and remove the unused workbench SQL-splitter re-export. Preserve the existing fixture helpers and all parameterized migration cases, including the combined profile and public-warning histories. Builds on #1382.

Testing

• Full migration module on the updated stack: 71 passed.
• All 43 test functions, parameter lists, decorators, and assertions retained in the source comparison.
• Required portable source checks passed: Ruff lint and format, TypeScript build:ci, plugin source compatibility, and all nine compatibility checker tests.
• Independent diff review and git diff --check: passed.

Risk and rollout

Test refactor plus removal of a test-only internal re-export. Historical migration order, transaction behavior, rejection cases, and repeated-upgrade checks are retained. No new database migration or public API change; no rollout step required.

Public disclosure review

• No customer, partner, prospect, or user identities, data, or identifying details are included.
• No credentials, personal data, private source, scan findings, or nonpublic links or tickets are included.
• I reviewed the branch name, title, description, commits, changes, comments, logs, screenshots, attachments, and links for public disclosure.

[chatgpt-codex-connector]

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review	Status	Commit	Review trigger
📝 Code Review	✅ Completed 2026-10-07T07:22:31.487519Z	6451e6d	New commits
🔒 Security Review	✅ Completed 2026-10-07T07:23:51.893254Z	6451e6d	New commits

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

[alandelong-oai]

Reviewed the current diff; no actionable issues found.

[kmbroai]

Reviewed both changed files against the reviewed parent, historical migration setup, SQL order/constraints/transactions, and callers of the removed internal re-export. All 68 migration tests passed locally. Current CI and discussions inspected. No actionable findings at this commit.

[zcrab-oai]

Reviewed the diff and relevant surrounding code at ac9d172. No obvious correctness regressions or lost test coverage found. 68 migration tests passed.

[alandelong-oai]

Reviewed the current diff; no actionable issues found.

[alandelong-oai]

Reviewed the current diff; no actionable issues found.