Skip to content

Replace real-derived regression fixtures with deterministic synthetic data - #81

Merged
tuoro merged 3 commits into
mainfrom
codex/p0-privacy-minimal
Aug 11, 2026
Merged

Replace real-derived regression fixtures with deterministic synthetic data#81
tuoro merged 3 commits into
mainfrom
codex/p0-privacy-minimal

Conversation

@tuoro

@tuoro tuoro commented Aug 11, 2026

Copy link
Copy Markdown
Owner

Why

The previous regression samples were derived from real invoices and tickets. This change removes those samples from the current tree and closes the production capture/export path that could recreate or expose them.

What changed

  • Replace the old fixture set with one fixed, deterministic synthetic generator covering five regression scenarios.
  • Remove regression fixture export, import, sync, force, and redaction flows, including their production backend and frontend endpoints.
  • Tighten the legacy-data migration to require all three identifying columns and fail closed at startup when cleanup cannot be completed.
  • Keep regression fixture data out of the final container image.
  • Enforce LF line endings for generated fixture files on Windows.

Impact

  • The application no longer exposes UI or API flows for capturing, managing, importing, or exporting regression samples.
  • The migration targets only rows matching the complete legacy regression schema; normal invoice, payment, OCR, and email records are not selected by that cleanup condition.

Validation

  • Canonical fixture validation and the complete Go test suite
  • go vet, go mod verify, and a temporary Go build
  • Frontend lint, 10 test files / 58 tests, and production build
  • Base and production Docker Compose configuration rendering
  • Clean-checkout verification with Windows core.autocrlf=true
  • Independent release review: no P0, P1, or P2 findings

CI follow-up

The PR remains a draft while CI verifies the exact head commit, including CGO-enabled SQLite migration behavior and the actual final container image.

Scope boundary

This PR changes the current repository tree only. Rewriting Git history or removing historical tags, GitHub Releases, published container images, and Actions caches requires a separate, explicitly approved cleanup.

Reviewed head: 8344de9cc29706620bff168063876d5c0741d44a

@tuoro
tuoro marked this pull request as ready for review August 11, 2026 19:19
@tuoro
tuoro merged commit 318d379 into main Aug 11, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant