Skip to content

PR_26177_GOLF_036-game-journey-metrics-sqlite-to-postgres-migration#191

Merged
ToolboxAid merged 1 commit into
mainfrom
pr/26177-GOLF-036-game-journey-metrics-sqlite-to-postgres-migration
Jun 25, 2026
Merged

PR_26177_GOLF_036-game-journey-metrics-sqlite-to-postgres-migration#191
ToolboxAid merged 1 commit into
mainfrom
pr/26177-GOLF-036-game-journey-metrics-sqlite-to-postgres-migration

Conversation

@ToolboxAid

Copy link
Copy Markdown
Owner

Summary

  • Add a one-time server-side migration utility for legacy Game Journey completion metrics SQLite data.
  • Inspect/export the legacy SQLite table, detect duplicates/conflicts, write through the existing Postgres client contract, and archive the SQLite file only after a successful migration.
  • Ran the migration: 14 legacy rows were already present in Postgres except timestamps, so the migration patched 14 createdAt/updatedAt values and moved the legacy file to tmp/local-api/legacy-migrated/game-journey-completion-metrics-20260625T195902Z.sqlite.

Validation

  • PASS: node --check src/dev-runtime/persistence/game-journey-completion-metrics-migration.mjs
  • PASS: node --check scripts/migrate-game-journey-completion-metrics-sqlite-to-postgres.mjs
  • PASS: node --check src/dev-runtime/persistence/game-journey-completion-metrics-store.mjs
  • PASS: node --test tests/dev-runtime/GameJourneyCompletionMetricsMigration.test.mjs
  • PASS: node --use-system-ca scripts/migrate-game-journey-completion-metrics-sqlite-to-postgres.mjs --inspect-only
  • PASS: node --use-system-ca scripts/migrate-game-journey-completion-metrics-sqlite-to-postgres.mjs
  • PASS: npx playwright test tests/playwright/tools/GameJourneyTool.spec.mjs --grep "completion metrics" --workers=1 --reporter=line
  • PASS: git diff --check

Artifact

  • tmp/PR_26177_GOLF_036-game-journey-metrics-sqlite-to-postgres-migration_delta.zip

Do not merge without Owner approval.

@ToolboxAid ToolboxAid marked this pull request as ready for review June 25, 2026 20:22
@ToolboxAid ToolboxAid merged commit c66099d into main Jun 25, 2026
1 check 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