Recover game journey metrics runtime path

Author: ToolboxAid

docs_build/dev/reports/PR_26177_OWNER_057-game-journey-metrics-regression-recovery_branch-validation.md

@@ -0,0 +1,19 @@
+# PR_26177_OWNER_057-game-journey-metrics-regression-recovery Branch Validation
+
+Status: PASS
+
+## Gate
+
+- PASS: Started on `main`.
+- PASS: Fetched origin.
+- PASS: Pulled `origin/main` with `--ff-only`.
+- PASS: Verified worktree clean before branch creation.
+- PASS: Verified `main...origin/main` was `0 0` before branch creation.
+- PASS: Created `PR_26177_OWNER_057-game-journey-metrics-regression-recovery` from latest `main`.
+
+## Branch Scope
+
+- PASS: Runtime changes are limited to the Game Journey completion metrics store and toolbox accordion Creator-facing wording.
+- PASS: Tests are limited to targeted Game Journey completion metrics regression coverage.
+- PASS: Did not delete, move, overwrite, export, or migrate `tmp/local-api/game-journey-completion-metrics.sqlite`.
+- PASS: Did not start Alfa Tags PRs.

docs_build/dev/reports/PR_26177_OWNER_057-game-journey-metrics-regression-recovery_manual-validation-notes.md

@@ -0,0 +1,13 @@
+# PR_26177_OWNER_057-game-journey-metrics-regression-recovery Manual Validation Notes
+
+Status: PASS
+
+## Notes
+
+- Confirmed the repo-local `tmp/local-api/game-journey-completion-metrics.sqlite` file exists before validation.
+- Confirmed active `createGameJourneyCompletionMetricsStore({ postgresClient })` no longer resolves that retired path by default.
+- Confirmed active metrics load 14 Postgres-backed completion buckets while the retired file remains untouched.
+- Confirmed explicit `legacyDbPath` protection remains covered by the existing migration/regression test file.
+- Confirmed the toolbox page renders neutral Creator-facing outage wording when active metrics are unavailable.
+- Confirmed the toolbox page does not render the forbidden warning string, SQLite wording, `tmp/local-api`, or Postgres internals in the simulated outage lane.
+- Confirmed no Alfa Tags PR work was started.

docs_build/dev/reports/PR_26177_OWNER_057-game-journey-metrics-regression-recovery_report.md

@@ -0,0 +1,45 @@
+# PR_26177_OWNER_057-game-journey-metrics-regression-recovery Report
+
+Status: PASS
+Branch: PR_26177_OWNER_057-game-journey-metrics-regression-recovery
+Date: 2026-06-26
+
+## Scope
+
+Recover the Game Journey completion metrics path so active Alfa and Owner work no longer surfaces the retired legacy SQLite regression. Preserve Postgres-backed Game Journey completion metrics as the active path and prevent Creator-facing UI from rendering the forbidden `Game Journey completion metrics unavailable:` warning.
+
+## Implementation Summary
+
+- Removed active runtime defaulting to `tmp/local-api/game-journey-completion-metrics.sqlite` in `createGameJourneyCompletionMetricsStore`.
+- Kept the explicit `legacyDbPath` guard intact for recovery/migration callers so legacy data is still protected from silent overwrite or deletion.
+- Updated `toolbox/tools-page-accordions.js` to render neutral Creator-safe progress outage wording instead of backend diagnostics.
+- Added a store-level regression test proving a retired default SQLite-shaped file does not block active Postgres-backed metrics.
+- Added a focused Playwright test proving the toolbox page does not render the forbidden warning, SQLite wording, local filesystem path, or Postgres internals when metrics are unavailable.
+
+## Reference Comparison
+
+- Compared the relevant strings in Bravo, Charlie, and Delta reference branches against current main.
+- Those branches contained the same legacy-default metrics store and forbidden toolbox warning strings.
+- Their non-error behavior depended on the retired SQLite file not being present at the default path.
+- This recovery fixes the active behavior directly so the current repo is not sensitive to that retired file.
+
+## Validation
+
+- PASS: `node --check` on modified source and test files.
+- PASS: `node ./scripts/run-node-test-files.mjs tests/dev-runtime/GameJourneyCompletionMetricsStore.test.mjs tests/dev-runtime/GameJourneyCompletionMetricsMigration.test.mjs`.
+- PASS: `npx playwright test tests/playwright/tools/GameJourneyTool.spec.mjs --project=playwright --workers=1 --reporter=line -g "Game Journey Local API persists completion metrics to Postgres|Toolbox renders Creator-safe Game Journey progress outage copy"`.
+- PASS: Direct proof against the actual existing `tmp/local-api/game-journey-completion-metrics.sqlite` file confirmed active Postgres metrics load 14 buckets and do not resolve a legacy path.
+- PASS: Runtime source search found no `Game Journey completion metrics unavailable` string.
+- PASS: Runtime source search found no active metrics-store default reference to `game-journey-completion-metrics.sqlite`, `GAMEFOUNDRY_GAME_JOURNEY_METRICS_DB_PATH`, or `defaultLegacySqlitePath`.
+- PASS: `git diff --check` reported no whitespace errors. Git emitted line-ending warnings only.
+
+## Files
+
+- `src/dev-runtime/persistence/game-journey-completion-metrics-store.mjs`
+- `tests/dev-runtime/GameJourneyCompletionMetricsStore.test.mjs`
+- `tests/playwright/tools/GameJourneyTool.spec.mjs`
+- `toolbox/tools-page-accordions.js`
+
+## Artifact
+
+- `tmp/PR_26177_OWNER_057-game-journey-metrics-regression-recovery_delta.zip`

docs_build/dev/reports/PR_26177_OWNER_057-game-journey-metrics-regression-recovery_requirement-checklist.md

@@ -0,0 +1,24 @@
+# PR_26177_OWNER_057-game-journey-metrics-regression-recovery Requirement Checklist
+
+Status: PASS
+
+- PASS: Hard stop gate verified current branch was `main` before branch creation.
+- PASS: Fetched origin.
+- PASS: Pulled `origin/main` with `--ff-only`.
+- PASS: Verified worktree clean and `main...origin/main` was `0 0`.
+- PASS: Created `PR_26177_OWNER_057-game-journey-metrics-regression-recovery`.
+- PASS: Compared Alfa/Owner behavior against Bravo/Charlie/Delta reference states.
+- PASS: Fixed only the Game Journey completion metrics regression.
+- PASS: Did not delete, move, overwrite, export, or migrate `tmp/local-api/game-journey-completion-metrics.sqlite`.
+- PASS: Stopped active runtime from defaulting to `tmp/local-api/game-journey-completion-metrics.sqlite`.
+- PASS: Preserved Postgres-backed Game Journey completion metrics as the active path.
+- PASS: Ensured `toolbox/tools-page-accordions.js` cannot render `Game Journey completion metrics unavailable:`.
+- PASS: Creator-facing UI does not expose SQLite, local filesystem paths, migration/export language, or Postgres internals.
+- PASS: Did not introduce silent fallback behavior; metrics outage remains visible with neutral wording.
+- PASS: Added targeted regression tests.
+- PASS: Proved the existing legacy SQLite file does not block active metrics.
+- PASS: Proved the forbidden warning string is not rendered.
+- PASS: Proved Game Journey metrics still load through the active DB/API path.
+- PASS: Used targeted validation only.
+- PASS: Required reports were produced.
+- PASS: Repo-structured ZIP was produced under `tmp/`.

docs_build/dev/reports/PR_26177_OWNER_057-game-journey-metrics-regression-recovery_validation-lane.md

@@ -0,0 +1,45 @@
+# PR_26177_OWNER_057-game-journey-metrics-regression-recovery Validation Lane
+
+Status: PASS
+
+## Commands
+
+```powershell
+node --check src/dev-runtime/persistence/game-journey-completion-metrics-store.mjs
+node --check toolbox/tools-page-accordions.js
+node --check tests/dev-runtime/GameJourneyCompletionMetricsStore.test.mjs
+node --check tests/playwright/tools/GameJourneyTool.spec.mjs
+```
+
+Result: PASS
+
+```powershell
+node ./scripts/run-node-test-files.mjs tests/dev-runtime/GameJourneyCompletionMetricsStore.test.mjs tests/dev-runtime/GameJourneyCompletionMetricsMigration.test.mjs
+```
+
+Result: PASS, 2 targeted node test files passed
+
+```powershell
+npx playwright test tests/playwright/tools/GameJourneyTool.spec.mjs --project=playwright --workers=1 --reporter=line -g "Game Journey Local API persists completion metrics to Postgres|Toolbox renders Creator-safe Game Journey progress outage copy"
+```
+
+Result: PASS, 2 passed
+
+```powershell
+node -e "import('node:fs').then(async fs=>{const [{createGameJourneyCompletionMetricsStore}, {createGameJourneyCompletionMetricsPostgresClientStub}] = await Promise.all([import('./src/dev-runtime/persistence/game-journey-completion-metrics-store.mjs'), import('./tests/helpers/gameJourneyCompletionMetricsPostgresClientStub.mjs')]); const legacy='tmp/local-api/game-journey-completion-metrics.sqlite'; if(!fs.existsSync(legacy)) throw new Error('Expected existing legacy SQLite file for regression proof'); const store=createGameJourneyCompletionMetricsStore({postgresClient:createGameJourneyCompletionMetricsPostgresClientStub()}); const metrics=await store.listMetrics(); if(store.legacyDbPath) throw new Error('Active store resolved a legacy path'); if(metrics.length!==14) throw new Error('Expected 14 active metrics'); console.log('PASS active Postgres metrics ignore existing retired legacy SQLite file');})"
+```
+
+Result: PASS
+
+```powershell
+rg -n "Game Journey completion metrics unavailable" src assets toolbox --glob "!**/*.map"
+rg -n "game-journey-completion-metrics\.sqlite|GAMEFOUNDRY_GAME_JOURNEY_METRICS_DB_PATH|defaultLegacySqlitePath" src/dev-runtime/persistence/game-journey-completion-metrics-store.mjs toolbox/tools-page-accordions.js assets/toolbox/game-journey/js/index.js
+```
+
+Result: PASS, no matches
+
+```powershell
+git diff --check
+```
+
+Result: PASS, line-ending warnings only

docs_build/dev/reports/codex_changed_files.txt

@@ -1,12 +1,11 @@
-docs_build/dev/BUILD_PR.md
-docs_build/dev/PLAN_PR.md
-docs_build/dev/reports/PR_26177_006-shared-time-foundation.md
-docs_build/dev/reports/PR_26177_006-shared-time-foundation_branch-validation.md
-docs_build/dev/reports/PR_26177_006-shared-time-foundation_closeout.md
-docs_build/dev/reports/PR_26177_006-shared-time-foundation_manual-validation-notes.md
-docs_build/dev/reports/PR_26177_006-shared-time-foundation_requirement-checklist.md
-docs_build/dev/reports/PR_26177_006-shared-time-foundation_validation-lane.md
+docs_build/dev/reports/PR_26177_OWNER_057-game-journey-metrics-regression-recovery_branch-validation.md
+docs_build/dev/reports/PR_26177_OWNER_057-game-journey-metrics-regression-recovery_manual-validation-notes.md
+docs_build/dev/reports/PR_26177_OWNER_057-game-journey-metrics-regression-recovery_report.md
+docs_build/dev/reports/PR_26177_OWNER_057-game-journey-metrics-regression-recovery_requirement-checklist.md
+docs_build/dev/reports/PR_26177_OWNER_057-game-journey-metrics-regression-recovery_validation-lane.md
 docs_build/dev/reports/codex_changed_files.txt
 docs_build/dev/reports/codex_review.diff
-src/shared/time/time.js
-tests/shared/TimeFoundation.test.mjs
+src/dev-runtime/persistence/game-journey-completion-metrics-store.mjs
+tests/dev-runtime/GameJourneyCompletionMetricsStore.test.mjs
+tests/playwright/tools/GameJourneyTool.spec.mjs
+toolbox/tools-page-accordions.js