From f1d6b48573c3f76b9d770daf086b16a5a5e4ff62 Mon Sep 17 00:00:00 2001 From: "Mr. Q" <97194984+ToolboxAid@users.noreply.github.com> Date: Sat, 20 Jun 2026 23:26:03 -0400 Subject: [PATCH] Plan active SQLite runtime removal --- ...1-sqlite-active-runtime-affected-files.txt | 52 + ...te-active-runtime-follow-up-pr-sequence.md | 78 + ...l-plan-instruction-compliance-checklist.md | 14 + ...me-removal-plan-manual-validation-notes.md | 24 + ..._021-sqlite-active-runtime-removal-plan.md | 128 + .../dev/reports/codex_changed_files.txt | 83 +- docs_build/dev/reports/codex_review.diff | 2996 ++--------------- 7 files changed, 586 insertions(+), 2789 deletions(-) create mode 100644 docs_build/dev/reports/PR_26171_GAMMA_021-sqlite-active-runtime-affected-files.txt create mode 100644 docs_build/dev/reports/PR_26171_GAMMA_021-sqlite-active-runtime-follow-up-pr-sequence.md create mode 100644 docs_build/dev/reports/PR_26171_GAMMA_021-sqlite-active-runtime-removal-plan-instruction-compliance-checklist.md create mode 100644 docs_build/dev/reports/PR_26171_GAMMA_021-sqlite-active-runtime-removal-plan-manual-validation-notes.md create mode 100644 docs_build/dev/reports/PR_26171_GAMMA_021-sqlite-active-runtime-removal-plan.md diff --git a/docs_build/dev/reports/PR_26171_GAMMA_021-sqlite-active-runtime-affected-files.txt b/docs_build/dev/reports/PR_26171_GAMMA_021-sqlite-active-runtime-affected-files.txt new file mode 100644 index 000000000..e9f074b15 --- /dev/null +++ b/docs_build/dev/reports/PR_26171_GAMMA_021-sqlite-active-runtime-affected-files.txt @@ -0,0 +1,52 @@ +PR_26171_GAMMA_021-sqlite-active-runtime-removal-plan affected files + +Fresh-main baseline: +1b27b0a9a3d67821fa586e34d5331567ec6f49b7 + +Active runtime removal targets: +src/dev-runtime/messages/messages-sqlite-service.mjs +src/dev-runtime/persistence/game-journey-completion-metrics-store.mjs +src/dev-runtime/persistence/mock-db-store.js + +Local API removal target: +src/dev-runtime/server/local-api-router.mjs + +Direct SQLite test references: +tests/dev-runtime/DbSeedIntegrity.test.mjs +tests/playwright/tools/AdminDbViewer.spec.mjs +tests/playwright/tools/AdminHealthOperationsPage.spec.mjs +tests/playwright/tools/BrowserApiUrlConfig.spec.mjs +tests/playwright/tools/GameJourneyTool.spec.mjs +tests/playwright/tools/LoginSessionMode.spec.mjs +tests/playwright/tools/MessagesTool.spec.mjs + +Shared mock-db-store dependency tests: +tests/dev-runtime/AiCreditFoundation.test.mjs +tests/dev-runtime/LegalFoundation.test.mjs +tests/dev-runtime/MarketplaceCategories.test.mjs +tests/dev-runtime/MembershipDataModel.test.mjs +tests/dev-runtime/SupabaseDevAuthTestUserCleanup.test.mjs +tests/dev-runtime/TeamsFoundation.test.mjs +tests/playwright/account/AchievementsPage.spec.mjs +tests/playwright/tools/AdminNotesLocalViewer.spec.mjs +tests/playwright/tools/AdminPlatformToolsWireframes.spec.mjs +tests/playwright/tools/AssetToolMockRepository.spec.mjs +tests/playwright/tools/BuildPathProgressSimplification.spec.mjs +tests/playwright/tools/GameHubMockRepository.spec.mjs +tests/playwright/tools/IdeaBoardTableNotes.spec.mjs +tests/playwright/tools/InputMappingV2Tool.spec.mjs +tests/playwright/tools/PaletteToolMockRepository.spec.mjs +tests/playwright/tools/RootToolsFutureState.spec.mjs +tests/playwright/tools/TagsTool.spec.mjs +tests/playwright/tools/ToolboxAdminMetadataSsot.spec.mjs +tests/playwright/tools/ToolboxRoutePages.spec.mjs + +Active docs/data references: +docs_build/dev/PROJECT_INSTRUCTIONS.md +docs_build/database/dml/DML_INDEX.md +docs_build/database/dml/messages.sql +docs_build/database/seed/messages.json + +Explicitly excluded archive/reference paths: +docs_build/dev/reports/** +docs_build/pr/** diff --git a/docs_build/dev/reports/PR_26171_GAMMA_021-sqlite-active-runtime-follow-up-pr-sequence.md b/docs_build/dev/reports/PR_26171_GAMMA_021-sqlite-active-runtime-follow-up-pr-sequence.md new file mode 100644 index 000000000..ea6d725d9 --- /dev/null +++ b/docs_build/dev/reports/PR_26171_GAMMA_021-sqlite-active-runtime-follow-up-pr-sequence.md @@ -0,0 +1,78 @@ +# PR_26171_GAMMA_021 Follow-Up PR Sequence + +## 1. Messages Database Cutover + +Owner: Team Beta + +Purpose: +- Replace `src/dev-runtime/messages/messages-sqlite-service.mjs`. +- Keep the Messages Local API contract stable. +- Move message categories, messages, emotion profiles, and TTS profile persistence to Postgres. +- Update `docs_build/database/dml/messages.sql`, `docs_build/database/seed/messages.json`, and Messages tests. + +Suggested validation: +- `git diff --check` +- targeted Messages Local API tests +- `tests/playwright/tools/MessagesTool.spec.mjs` +- no samples unless Messages API behavior is consumed by samples + +## 2. Game Journey Metrics Database Cutover + +Owner: Team Alpha + +Purpose: +- Replace `src/dev-runtime/persistence/game-journey-completion-metrics-store.mjs`. +- Preserve current Game Journey completion metrics response shape. +- Use Postgres through `GAMEFOUNDRY_DATABASE_URL`. +- Update Game Journey tests that currently assert SQLite engine metadata. + +Suggested validation: +- `git diff --check` +- targeted Game Journey Local API tests +- `tests/playwright/tools/GameJourneyTool.spec.mjs` +- no samples unless Game Journey sample behavior is directly affected + +## 3. Shared Dev Database Adapter Cleanup + +Owner: Master Control assignment required + +Purpose: +- Split shared schema/key constants from `src/dev-runtime/persistence/mock-db-store.js`. +- Remove active Local DB/SQLite adapter wording from session metadata. +- Preserve compatibility for current mock repository imports until owner-assigned tool migrations land. + +Suggested validation: +- `git diff --check` +- targeted dev-runtime schema tests +- only affected tool Playwright specs whose fixtures import the changed contract + +## 4. Local API SQLite Metadata Removal + +Owner: Team Gamma coordination, with Alpha/Beta route owners for implementation details + +Purpose: +- Remove SQLite imports and constructor wiring from `src/dev-runtime/server/local-api-router.mjs`. +- Remove SQLite database engine metadata after replacement services are live. +- Keep Admin System Health diagnostics Postgres-only. + +Suggested validation: +- `git diff --check` +- targeted Local API startup tests +- targeted Admin System Health validation +- targeted Messages and Game Journey validation + +## 5. Final Guard Cleanup And Inventory + +Owner: Team Gamma + +Purpose: +- Update `scripts/validate-browser-env-agnostic.mjs` technical-debt notes after active SQLite removal. +- Run a fresh inventory. +- Confirm remaining SQLite references are only governance docs or archive/reference reports. + +Suggested validation: +- `git diff --check` +- targeted static/script validation +- final SQLite inventory report +- no Playwright unless runtime JavaScript changes +- no samples diff --git a/docs_build/dev/reports/PR_26171_GAMMA_021-sqlite-active-runtime-removal-plan-instruction-compliance-checklist.md b/docs_build/dev/reports/PR_26171_GAMMA_021-sqlite-active-runtime-removal-plan-instruction-compliance-checklist.md new file mode 100644 index 000000000..26a5fd7a4 --- /dev/null +++ b/docs_build/dev/reports/PR_26171_GAMMA_021-sqlite-active-runtime-removal-plan-instruction-compliance-checklist.md @@ -0,0 +1,14 @@ +# PR_26171_GAMMA_021 Instruction Compliance Checklist + +- PASS: Read `docs_build/dev/PROJECT_INSTRUCTIONS.md`. +- PASS: Read `docs_build/dev/PROJECT_MULTI_PC.txt`. +- PASS: PR name includes required TEAM token: `GAMMA`. +- PASS: TEAM ownership matches Gamma governance/diagnostics planning scope. +- PASS: Started from clean `main`. +- PASS: Fast-forwarded `team/GAMMA/admin` to fresh `main` before edits. +- PASS: Scope is report-only planning; no runtime code removed. +- PASS: Archive/reference report history is excluded. +- PASS: Validation scope is docs/static only. +- PASS: Playwright skipped with documented reason. +- PASS: Samples skipped with documented reason. +- PASS: ZIP required under `tmp/`. diff --git a/docs_build/dev/reports/PR_26171_GAMMA_021-sqlite-active-runtime-removal-plan-manual-validation-notes.md b/docs_build/dev/reports/PR_26171_GAMMA_021-sqlite-active-runtime-removal-plan-manual-validation-notes.md new file mode 100644 index 000000000..326c85bae --- /dev/null +++ b/docs_build/dev/reports/PR_26171_GAMMA_021-sqlite-active-runtime-removal-plan-manual-validation-notes.md @@ -0,0 +1,24 @@ +# PR_26171_GAMMA_021 Manual Validation Notes + +## Checks + +- PASS: Fresh-main SQLite inventory was reviewed from commit `1b27b0a9a3d67821fa586e34d5331567ec6f49b7`. +- PASS: Active runtime references were classified. +- PASS: Local API references were classified. +- PASS: Direct SQLite test references were identified. +- PASS: Shared `mock-db-store.js` dependency tests were identified separately from literal SQLite hits. +- PASS: Active docs/data references were identified. +- PASS: Archive/reference report history under `docs_build/dev/reports/**` and `docs_build/pr/**` was explicitly excluded from removal. +- PASS: No SQLite code was removed. +- PASS: No archive reports were touched. +- PASS: Playwright and samples were skipped because this is a docs/report-only planning PR. + +## Validation Commands + +- `git diff --check` +- Targeted text verification for active SQLite classification +- Targeted text verification for archive/reference exclusion + +## Notes + +The inventory distinguishes the two true `node:sqlite` active runtime services from `mock-db-store.js`, which is shared dev-runtime metadata but still carries active SQLite adapter wording. diff --git a/docs_build/dev/reports/PR_26171_GAMMA_021-sqlite-active-runtime-removal-plan.md b/docs_build/dev/reports/PR_26171_GAMMA_021-sqlite-active-runtime-removal-plan.md new file mode 100644 index 000000000..a2d43e3d1 --- /dev/null +++ b/docs_build/dev/reports/PR_26171_GAMMA_021-sqlite-active-runtime-removal-plan.md @@ -0,0 +1,128 @@ +# PR_26171_GAMMA_021-sqlite-active-runtime-removal-plan + +## Summary + +This report creates a focused removal plan for active SQLite references found on fresh `main` after PR #36 merged. + +Fresh-main baseline: +- Branch: `team/GAMMA/admin` +- Fresh main commit used for inventory: `1b27b0a9a3d67821fa586e34d5331567ec6f49b7` +- Inventory command: `rg -i "sqlite|better-sqlite|sqlite3|\\.sqlite|sql\\.js"` +- Raw inventory excluding generated diff noise: 130 files, 358 matching lines + +This PR does not remove code, change runtime behavior, touch archive reports, run Playwright, or run samples. + +## Active Classification + +### Active Runtime + +- `src/dev-runtime/messages/messages-sqlite-service.mjs` + - Classification: active runtime + - Current state: uses `node:sqlite`, `DatabaseSync`, `GAMEFOUNDRY_MESSAGES_SQLITE_PATH`, and `tmp/messages/messages.sqlite`. + - Replacement path: replace with a Postgres-backed Messages database service using `GAMEFOUNDRY_DATABASE_URL` and the existing Local API message contracts. Prefer a vendor-neutral active service name such as `messages-database-service.mjs`; if a compatibility wrapper is needed, keep it temporary and log the deprecated name. + +- `src/dev-runtime/persistence/game-journey-completion-metrics-store.mjs` + - Classification: active runtime + - Current state: uses `node:sqlite`, `DatabaseSync`, `GAMEFOUNDRY_GAME_JOURNEY_METRICS_DB_PATH`, and `tmp/local-api/game-journey-completion-metrics.sqlite`. + - Replacement path: move completion metrics reads/writes to the authoritative Postgres connection path using `GAMEFOUNDRY_DATABASE_URL`. Keep the current Local API response shape stable while replacing the storage implementation. + +- `src/dev-runtime/persistence/mock-db-store.js` + - Classification: active runtime / allowed technical debt + - Current state: shared dev-runtime schema and seed metadata; it does not import `node:sqlite`, but its active session adapter metadata still says `LocalDbAdapter backed by server SQLite storage`. + - Replacement path: split shared schema/seed constants from persistence-provider labeling, then replace Local DB/SQLite-facing adapter copy with a Postgres-backed or neutral dev database adapter description. Preserve existing `MOCK_DB_KEYS` consumers until owner-assigned follow-up PRs migrate each tool surface. + +### Local API + +- `src/dev-runtime/server/local-api-router.mjs` + - Classification: local API + - Current state: imports `createMessagesSqliteService`, constructs the Messages SQLite service, and returns SQLite database engine metadata for remaining local routes. + - Replacement path: inject the Postgres-backed Messages service and Game Journey metrics service through existing safe Local API contracts. Remove SQLite engine metadata only after the replacement services are available and targeted tests prove the same API response shapes. + +### Tests + +Direct SQLite tests: +- `tests/playwright/tools/MessagesTool.spec.mjs` +- `tests/playwright/tools/GameJourneyTool.spec.mjs` +- `tests/dev-runtime/DbSeedIntegrity.test.mjs` +- `tests/playwright/tools/LoginSessionMode.spec.mjs` +- `tests/playwright/tools/BrowserApiUrlConfig.spec.mjs` +- `tests/playwright/tools/AdminDbViewer.spec.mjs` +- `tests/playwright/tools/AdminHealthOperationsPage.spec.mjs` + +Shared `mock-db-store.js` dependency tests that may need updates when the adapter label or constants split changes: +- `tests/dev-runtime/AiCreditFoundation.test.mjs` +- `tests/dev-runtime/LegalFoundation.test.mjs` +- `tests/dev-runtime/MarketplaceCategories.test.mjs` +- `tests/dev-runtime/MembershipDataModel.test.mjs` +- `tests/dev-runtime/SupabaseDevAuthTestUserCleanup.test.mjs` +- `tests/dev-runtime/TeamsFoundation.test.mjs` +- `tests/playwright/account/AchievementsPage.spec.mjs` +- `tests/playwright/tools/AdminNotesLocalViewer.spec.mjs` +- `tests/playwright/tools/AdminPlatformToolsWireframes.spec.mjs` +- `tests/playwright/tools/AssetToolMockRepository.spec.mjs` +- `tests/playwright/tools/BuildPathProgressSimplification.spec.mjs` +- `tests/playwright/tools/GameHubMockRepository.spec.mjs` +- `tests/playwright/tools/IdeaBoardTableNotes.spec.mjs` +- `tests/playwright/tools/InputMappingV2Tool.spec.mjs` +- `tests/playwright/tools/PaletteToolMockRepository.spec.mjs` +- `tests/playwright/tools/RootToolsFutureState.spec.mjs` +- `tests/playwright/tools/TagsTool.spec.mjs` +- `tests/playwright/tools/ToolboxAdminMetadataSsot.spec.mjs` +- `tests/playwright/tools/ToolboxRoutePages.spec.mjs` + +### Docs + +- `docs_build/dev/PROJECT_INSTRUCTIONS.md` + - Keep as authoritative deprecation governance. +- `docs_build/database/dml/DML_INDEX.md` +- `docs_build/database/dml/messages.sql` +- `docs_build/database/seed/messages.json` + - Update in the Messages Postgres cutover PR so seed/setup wording no longer references the server-side Messages SQLite service. + +### Archive/Reference + +Historical report and PR history under these paths is explicitly excluded from removal: +- `docs_build/dev/reports/**` +- `docs_build/pr/**` + +Archive/reference history may continue to mention SQLite for traceability. Do not rewrite old reports unless a future governance PR explicitly scopes archival cleanup. + +## Required Follow-Up PR Sequence + +1. Messages database cutover + - Owner: Team Beta + - Scope: replace `messages-sqlite-service.mjs` with a Postgres-backed Messages service, keep current Local API endpoints stable, migrate message seed/setup docs, and update `MessagesTool.spec.mjs`. + - Validation: targeted Messages Local API and Messages Playwright coverage; no samples unless the changed API contract affects samples. + +2. Game Journey completion metrics database cutover + - Owner: Team Alpha + - Scope: replace `game-journey-completion-metrics-store.mjs` with a Postgres-backed metrics store and update Game Journey completion tests. + - Validation: targeted Game Journey Local API and Game Journey Playwright coverage; no full suite unless shared runtime behavior expands. + +3. Shared dev database adapter label and schema split + - Owner: Master Control assignment required because `mock-db-store.js` is shared across Alpha/Beta/Gamma surfaces. + - Scope: split schema/key constants from persistence adapter labeling, remove Local DB/SQLite wording from active adapter metadata, and update affected tests by ownership lane. + - Validation: targeted dev-runtime schema tests plus only the tool Playwright lanes whose fixtures import the changed contract. + +4. Local API SQLite metadata removal + - Owner: Team Gamma for diagnostics/governance coordination, with Alpha/Beta implementation owners for affected routes. + - Scope: remove SQLite imports, constructor wiring, and database engine metadata from `local-api-router.mjs` after replacement services have landed. + - Validation: targeted Local API startup, Admin System Health diagnostics, Messages, and Game Journey lanes. + +5. Guard cleanup and final inventory + - Owner: Team Gamma + - Scope: update `scripts/validate-browser-env-agnostic.mjs` technical-debt notes, rerun the active SQLite inventory, and confirm only governance docs and archive/reference reports remain. + - Validation: `git diff --check`, targeted script/static validation, final inventory report. + +## Manual Validation Notes + +- Confirmed the plan targets only active runtime, Local API, test, and active docs/data references. +- Confirmed archive/reference report history is excluded. +- Confirmed no code was removed. +- Confirmed no runtime behavior was changed. +- Confirmed Postgres remains the replacement direction. + +## Skipped Validation + +- Playwright skipped: docs/report-only planning PR. +- Samples skipped: no sample files or sample runtime behavior changed. diff --git a/docs_build/dev/reports/codex_changed_files.txt b/docs_build/dev/reports/codex_changed_files.txt index 1a893c6ef..4b6cbbb27 100644 --- a/docs_build/dev/reports/codex_changed_files.txt +++ b/docs_build/dev/reports/codex_changed_files.txt @@ -1,69 +1,26 @@ -# PR_26171_GAMMA_019-admin-workstream-mergeability-recovery changed files +# PR_26171_GAMMA_021-sqlite-active-runtime-removal-plan changed files -Base: origin/main 35b04c02ea54da8b13c10354126f1ee8ddd14a89 -Head before recovery merge commit: 1806adbf5df787f7072c6579d23b99bb4257466b +Base main commit: 1b27b0a9a3d67821fa586e34d5331567ec6f49b7 +Head before PR021 commit: 1b27b0a9a3d67821fa586e34d5331567ec6f49b7 -## Changed files against origin/main -M admin/system-health.html -M assets/theme-v2/js/admin-system-health.js -A docs_build/dev/reports/PR_26171_GAMMA_011-admin-system-health-foundation-instruction-compliance-checklist.md -A docs_build/dev/reports/PR_26171_GAMMA_011-admin-system-health-foundation-manual-validation-notes.md -A docs_build/dev/reports/PR_26171_GAMMA_011-admin-system-health-foundation.md -A docs_build/dev/reports/PR_26171_GAMMA_012-admin-system-health-status-reason-cleanup-instruction-compliance-checklist.md -A docs_build/dev/reports/PR_26171_GAMMA_012-admin-system-health-status-reason-cleanup-manual-validation-notes.md -A docs_build/dev/reports/PR_26171_GAMMA_012-admin-system-health-status-reason-cleanup.md -A docs_build/dev/reports/PR_26171_GAMMA_013-admin-system-health-diagnostics-plan-instruction-compliance-checklist.md -A docs_build/dev/reports/PR_26171_GAMMA_013-admin-system-health-diagnostics-plan-manual-validation-notes.md -A docs_build/dev/reports/PR_26171_GAMMA_013-admin-system-health-diagnostics-plan.md -A docs_build/dev/reports/PR_26171_GAMMA_014-admin-postgres-diagnostics-runtime-instruction-compliance-checklist.md -A docs_build/dev/reports/PR_26171_GAMMA_014-admin-postgres-diagnostics-runtime-manual-validation-notes.md -A docs_build/dev/reports/PR_26171_GAMMA_014-admin-postgres-diagnostics-runtime.md -A docs_build/dev/reports/PR_26171_GAMMA_015-admin-r2-diagnostics-runtime-instruction-compliance-checklist.md -A docs_build/dev/reports/PR_26171_GAMMA_015-admin-r2-diagnostics-runtime-manual-validation-notes.md -A docs_build/dev/reports/PR_26171_GAMMA_015-admin-r2-diagnostics-runtime.md -A docs_build/dev/reports/PR_26171_GAMMA_016-admin-runtime-environment-runtime-instruction-compliance-checklist.md -A docs_build/dev/reports/PR_26171_GAMMA_016-admin-runtime-environment-runtime-manual-validation-notes.md -A docs_build/dev/reports/PR_26171_GAMMA_016-admin-runtime-environment-runtime.md -A docs_build/dev/reports/PR_26171_GAMMA_019-admin-workstream-mergeability-recovery.md -M docs_build/dev/reports/codex_changed_files.txt -M docs_build/dev/reports/codex_review.diff -M docs_build/dev/reports/coverage_changed_js_guardrail.txt -M docs_build/dev/reports/playwright_v8_coverage_report.txt -M src/dev-runtime/server/local-api-router.mjs -M tests/playwright/tools/AdminHealthOperationsPage.spec.mjs +## Changed files +A docs_build/dev/reports/PR_26171_GAMMA_021-sqlite-active-runtime-affected-files.txt +A docs_build/dev/reports/PR_26171_GAMMA_021-sqlite-active-runtime-follow-up-pr-sequence.md +A docs_build/dev/reports/PR_26171_GAMMA_021-sqlite-active-runtime-removal-plan-instruction-compliance-checklist.md +A docs_build/dev/reports/PR_26171_GAMMA_021-sqlite-active-runtime-removal-plan-manual-validation-notes.md +A docs_build/dev/reports/PR_26171_GAMMA_021-sqlite-active-runtime-removal-plan.md -## Diff stat against origin/main - admin/system-health.html | 366 +-- - assets/theme-v2/js/admin-system-health.js | 505 ++-- - ...-foundation-instruction-compliance-checklist.md | 32 + - ...em-health-foundation-manual-validation-notes.md | 34 + - ...171_GAMMA_011-admin-system-health-foundation.md | 65 + - ...son-cleanup-instruction-compliance-checklist.md | 57 + - ...tatus-reason-cleanup-manual-validation-notes.md | 26 + - ...12-admin-system-health-status-reason-cleanup.md | 70 + - ...ostics-plan-instruction-compliance-checklist.md | 64 + - ...lth-diagnostics-plan-manual-validation-notes.md | 27 + - ...MMA_013-admin-system-health-diagnostics-plan.md | 82 + - ...ics-runtime-instruction-compliance-checklist.md | 64 + - ...-diagnostics-runtime-manual-validation-notes.md | 27 + - ...GAMMA_014-admin-postgres-diagnostics-runtime.md | 93 + - ...ics-runtime-instruction-compliance-checklist.md | 62 + - ...-diagnostics-runtime-manual-validation-notes.md | 26 + - ...26171_GAMMA_015-admin-r2-diagnostics-runtime.md | 95 + - ...ent-runtime-instruction-compliance-checklist.md | 66 + - ...-environment-runtime-manual-validation-notes.md | 28 + - ..._GAMMA_016-admin-runtime-environment-runtime.md | 102 + - ...A_019-admin-workstream-mergeability-recovery.md | 50 + - docs_build/dev/reports/codex_changed_files.txt | 79 +- - docs_build/dev/reports/codex_review.diff | 2696 +++++++++++++++++--- - .../dev/reports/coverage_changed_js_guardrail.txt | 28 + - .../dev/reports/playwright_v8_coverage_report.txt | 94 +- - src/dev-runtime/server/local-api-router.mjs | 46 + - .../tools/AdminHealthOperationsPage.spec.mjs | 151 +- - 27 files changed, 4034 insertions(+), 1001 deletions(-) +## Diff stat + ...MA_021-sqlite-active-runtime-affected-files.txt | 52 +++++++++ + ...-sqlite-active-runtime-follow-up-pr-sequence.md | 78 +++++++++++++ + ...emoval-plan-instruction-compliance-checklist.md | 14 +++ + ...runtime-removal-plan-manual-validation-notes.md | 24 ++++ + ...GAMMA_021-sqlite-active-runtime-removal-plan.md | 128 +++++++++++++++++++++ + 5 files changed, 296 insertions(+) ## Validation evidence - git diff --check: PASS -- git diff --cached --check: PASS -- npx playwright test tests/playwright/tools/AdminHealthOperationsPage.spec.mjs --config=codex_playwright_system_chrome.config.cjs --project=playwright: PASS (3 passed) -- samples smoke: SKIPPED by request +- active SQLite references classified: PASS +- archive/reference report history explicitly excluded: PASS +- Playwright: SKIPPED, docs/report-only planning PR +- samples: SKIPPED, no sample files or runtime behavior changed diff --git a/docs_build/dev/reports/codex_review.diff b/docs_build/dev/reports/codex_review.diff index 1c29de68d..12624a1df 100644 --- a/docs_build/dev/reports/codex_review.diff +++ b/docs_build/dev/reports/codex_review.diff @@ -1,2782 +1,326 @@ -diff --git a/admin/system-health.html b/admin/system-health.html -index c5ef74f44..f5330bcdd 100644 ---- a/admin/system-health.html -+++ b/admin/system-health.html -@@ -6,7 +6,7 @@ - - - System Health - Game Foundry Studio LLC -- -+ - - - -@@ -18,7 +18,7 @@ -
-
Admin
-

System Health

--

Review safe runtime readiness across account, Local DB, R2 storage, migrations, package planning, promotion safety, and storage connectivity.

-+

Review the foundation view for environment, Postgres, Cloudflare R2, runtime configuration, capacity, and diagnostics.

-
- -
-@@ -36,21 +36,15 @@ - - -
-- Status Sources -+ Health Sections -
--

Account/session readiness

--

Product Data / Local DB

--

Project Asset Storage / R2

--

Project package readiness

--
--
--
-- Storage Connectivity --
-- -- -- -- -+

Environment Summary

-+

Database Health

-+

Storage Health

-+

Runtime Environment

-+

Limits & Capacity

-+

Diagnostics Plan

-+

Diagnostics Log

-
-
- -@@ -58,329 +52,163 @@ -
-
-
--
Read-only Status
--

System Health Overview

-+
Foundation
-+

System Health Tables

-
-
-- -- -+
System Health Dashboard Summary
-+ - - -+ -+ -+ - -- -- -- -- -- -- -- -- -- -- --
Environment Summary
EnvironmentPurposePromotion GateStatusScorePASSWARNFAILLast Refresh
Loading0000Loading refresh timestamp.
--
--
-- -- -- -- -- -- -- -- -- -- -- -- --
System Health Overview
AreaStatusSummary
System HealthLoadingLoading safe status.
--
--
-- -- -- -- -- -- -- -- -- -- -- -- -- --
Operational Health Summary
AreaStatusCountDiagnostic
OperationsLoading0Loading operational health.
--
--
-- -- -- -- -- -- - - - -- -- -- -- -+ -+ -+ -+ - -
Membership Support State
FilterValue
Plan -- --
DEVLocal implementation and diagnosticsDeveloper validationPASS
ISTIntegrated system testingCross-feature validationPASS
UATUser acceptance testingOwner reviewPASS
PRDProduction runtimeEOD approvalPASS
-
-
-- -- -+
Membership Assignments
-+ - - -- -- -- -- -- -- -- -- -- --
Database Health - Postgres Only
UserPlanStatusSource
LoadingLoadingLoadingLoading membership state.
--
--
-- -- -- -- -- -+ - -- -- -- -- -- -- -- -- --
Invite Support
FilterFieldValue
Status -- --
--
--
-- -- -- -- -- - -- -- -- -- -- -- -- --
Invite Records
EmailStatusCodeExpires
LoadingLoadingLoadingLoading invite state.
--
--
-- -- -- -- -- -- - - - -- -- -- -- -+ -+ -+ -+ -+ -+ - -
AI Usage Monitoring
FilterValue
Action -- --
ProviderPostgresPASS
HostConfigured host placeholderPENDING
Port5432PASS
DatabaseConfigured database placeholderPENDING
Migration VersionPending migration readerPENDING
StatusConnection check pendingPENDING
-
-
-- -- -+
AI Usage Rows
-+ - - -- -- -- -- -- -- -- -- -- -- --
Storage Health - Cloudflare R2
UserActionSourceDeltaBalance After
LoadingLoadingLoading0Loading AI usage.
--
--
-- -- -- -- -- -+ -+ - -- -- -- - - -- -- -- --
Operational Configuration Issues
AreaCheckTargetStatusTableIssueNext Step
OperationsLoadingLoadingLoading configuration issues.Loading next step.
--
--
-- -- -- -- -- -- -- -- -- -- -- -- --
Configured Connections
ConnectionStatusMode
AccountLoadingLoading configured connection summary.
--
--
-- -- -- -- -- -- -- -- -- -- -- -- --
Database Status
FieldStatusValue
Connection ConfiguredLoadingLoading database status.
--
--
-- -- -- -- -- -- -- -- -- -- -- -+ -+ -+ -+ -+ -+ - -
Project Asset Storage Status
FieldStatusValue
Storage ConfiguredLoadingLoading storage status.
BucketConfigured bucket placeholderPENDING
ListObjects prefixPENDING
ReadHealth objectPENDING
WriteHealth objectPENDING
DeleteHealth objectPENDING
-
-
-- -- -+
R2 Operational Readiness
-+ - - -- -- -+ -+ - -- -- - - -- -- -+ -+ -+ -+ -+ -+ - -
Runtime Environment - Alphabetical Variables
AreaSignalVariableDisplayStatusValueNext Step
Project Asset Storage / R2StatusLoadingLoadingLoading R2 readiness.
CLOUDFLARE_R2_ACCESS_KEY_ID********PASS
CLOUDFLARE_R2_BUCKETConfigured bucket placeholderPENDING
CLOUDFLARE_R2_SECRET_ACCESS_KEY********PASS
DATABASE_URL********PASS
GAMEFOUNDRY_ENVIRONMENTDEV / IST / UAT / PRDPASS
-
-
-- -- -+
Storage Connectivity Results
-+ - - -- -+ -+ -+ - -- -- - - -- -- -+ -+ -+ -+ -+ -+ - -
Limits & Capacity
ActionLimitCurrentCapacity SignalStatusExecutedMessage
startupSKIPnoStorage connectivity startup has not run yet.
DB SizePending metricThreshold not connectedPENDING
ConnectionsPending metricPool usage not connectedPENDING
StoragePending metricBucket usage not connectedPENDING
Class A OpsPending metricR2 write/list budget not connectedPENDING
Class B OpsPending metricR2 read budget not connectedPENDING
-
-
-- -- -+
System Health Details
-+ - - -- -- -+ -+ -+ - -- -- -- -- -- -- --
Diagnostics Plan
AreaFieldDiagnostic CheckFuture SignalPlanned SourceStatusValue
System HealthStatusLoadingLoading safe details.
--
--
-- -- -- -- -- -- -- -- -- - - -- -- -+ -+ -+ -+ -+ -+ -+ -+ -+ -+ - -
Environment Limits from current .env; values may differ by DEV/IST/UAT/PRD
VariableLimitUsagePressureNext Step
System HealthLoadingLoadingLoadingLoading limit diagnostics.
Postgres ConnectionConnection success, latency, and actionable failure detailServer-owned Postgres health readerPENDING
Postgres Migration ReaderCurrent migration version and pending migration signalServer-owned Postgres migration readerPENDING
R2 Bucket ConfiguredConfigured bucket name presence without exposing secretsServer-owned Cloudflare R2 configuration readerPENDING
R2 ListList operation availability and scoped prefix resultServer-owned Cloudflare R2 storage diagnosticPENDING
R2 ReadHealth object read availabilityServer-owned Cloudflare R2 storage diagnosticPENDING
R2 WriteHealth object write availabilityServer-owned Cloudflare R2 storage diagnosticPENDING
R2 DeleteHealth object cleanup availabilityServer-owned Cloudflare R2 storage diagnosticPENDING
Runtime Environment MaskingMasked display verification for secret-bearing variablesServer-owned runtime environment diagnosticPENDING
Limits/Capacity MetricsDB size, connections, storage usage, and R2 Class A/Class B operation signalsServer-owned metrics diagnosticPENDING
-
-
-