Merge PR_26177_OWNER_057-game-journey-metrics-regression-recovery

# Conflicts:
#	docs_build/dev/reports/codex_changed_files.txt
#	docs_build/dev/reports/codex_review.diff

Author: ToolboxAid

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

@@ -0,0 +1,20 @@
+# 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.
+- PASS: Final audit removed active runtime JS/MJS SQLite and `tmp/local-api` references outside the migration-only utility.

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

@@ -0,0 +1,14 @@
+# 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 })` exposes no `legacyDbPath`.
+- Confirmed active metrics snapshots expose no `legacySqlitePath`.
+- Confirmed active metrics load 14 DB-backed completion buckets while the retired file remains untouched.
+- Confirmed active runtime JS/MJS has no SQLite or `tmp/local-api` metrics references outside the migration-only utility.
+- 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,50 @@
+# 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`.
+- Removed active runtime `legacyDbPath` guard plumbing from the Game Journey metrics store, repository, Local API router, and Playwright test server helper.
+- 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 or get touched by active DB-backed metrics.
+- Added a targeted guardrail test proving active runtime JS/MJS under `src`, `assets`, and `toolbox` has no SQLite or `tmp/local-api` metrics references, excluding the migration-only utility.
+- 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 DB metrics load 14 buckets, expose no legacy path fields, and do not touch the retired file.
+- PASS: Active runtime JS/MJS search found no SQLite, `.sqlite`, `better-sqlite`, `game-journey-completion-metrics.sqlite`, or `tmp/local-api` references outside the migration-only utility.
+- PASS: Runtime source search found no `Game Journey completion metrics unavailable` Creator-facing string.
+- 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`
+- `src/dev-runtime/persistence/tool-repositories/game-journey-mock-repository.js`
+- `src/dev-runtime/server/local-api-router.mjs`
+- `tests/dev-runtime/GameJourneyCompletionMetricsStore.test.mjs`
+- `tests/helpers/playwrightRepoServer.mjs`
+- `tests/playwright/tools/GameJourneyTool.spec.mjs`
+- `tests/playwright/tools/IdeaBoardTableNotes.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,26 @@
+# 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: Removed active runtime `legacyDbPath` SQLite guard plumbing.
+- 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 active runtime JS/MJS has no SQLite or `tmp/local-api` metrics references outside the migration-only utility.
+- 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,49 @@
+# 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 src/dev-runtime/persistence/tool-repositories/game-journey-mock-repository.js
+node --check src/dev-runtime/server/local-api-router.mjs
+node --check toolbox/tools-page-accordions.js
+node --check tests/dev-runtime/GameJourneyCompletionMetricsStore.test.mjs
+node --check tests/helpers/playwrightRepoServer.mjs
+node --check tests/playwright/tools/GameJourneyTool.spec.mjs
+node --check tests/playwright/tools/IdeaBoardTableNotes.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. Includes active runtime JS/MJS SQLite reference guardrail.
+
+```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 retired local file for regression proof'); const before=fs.statSync(legacy).mtimeMs; const store=createGameJourneyCompletionMetricsStore({postgresClient:createGameJourneyCompletionMetricsPostgresClientStub()}); const metrics=await store.listMetrics(); const snapshot=await store.snapshot(); const after=fs.statSync(legacy).mtimeMs; if(Object.hasOwn(store, 'legacyDbPath')) throw new Error('Store exposes legacyDbPath'); if(Object.hasOwn(snapshot, 'legacySqlitePath')) throw new Error('Snapshot exposes legacySqlitePath'); if(metrics.length!==14) throw new Error('Expected 14 active metrics'); if(before!==after) throw new Error('Retired local file was touched'); console.log('PASS active DB metrics ignore and do not inspect retired local file');})"
+```
+
+Result: PASS
+
+```powershell
+rg -n -i "sqlite|better-sqlite|game-journey-completion-metrics\.sqlite|tmp/local-api" src assets toolbox -g "*.js" -g "*.mjs" --glob "!src/dev-runtime/persistence/game-journey-completion-metrics-migration.mjs"
+rg -n "Game Journey completion metrics unavailable" src assets toolbox --glob "!**/*.map"
+```
+
+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,14 @@
-docs_build/dev/reports/codex_changed_files.txt
+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
-docs_build/dev/reports/PR_26177_CHARLIE_008-environment-management-complete.md
-docs_build/dev/reports/PR_26177_CHARLIE_008-environment-management-complete_branch-validation.md
-docs_build/dev/reports/PR_26177_CHARLIE_008-environment-management-complete_manual-validation-notes.md
-docs_build/dev/reports/PR_26177_CHARLIE_008-environment-management-complete_requirements-checklist.md
-docs_build/dev/reports/PR_26177_CHARLIE_008-environment-management-complete_validation-lane.md
+src/dev-runtime/persistence/game-journey-completion-metrics-store.mjs
+src/dev-runtime/persistence/tool-repositories/game-journey-mock-repository.js
 src/dev-runtime/server/local-api-router.mjs
-tests/dev-runtime/AdminHealthOperations.test.mjs
-tests/dev-runtime/PublicEnvironmentConfig.test.mjs
-tests/playwright/tools/AdminHealthOperationsPage.spec.mjs
-tests/playwright/tools/EnvironmentBannerCoverage.spec.mjs
+tests/dev-runtime/GameJourneyCompletionMetricsStore.test.mjs
+tests/helpers/playwrightRepoServer.mjs
+tests/playwright/tools/GameJourneyTool.spec.mjs
+tests/playwright/tools/IdeaBoardTableNotes.spec.mjs