Skip to content

PR_26171_BETA_022-messages-postgres-service-cutover#44

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
ToolboxAid merged 1 commit into from
Jun 21, 2026
Merged

PR_26171_BETA_022-messages-postgres-service-cutover #44

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion docs_build/database/dml/DML_INDEX.md
View file
Open in desktop
Original file line numberDiff line numberDiff line change
Expand Up@@ -14,7 +14,7 @@ Direct SQL setup is intentionally narrow. Account DEV users now require server-s
| Game Design | `game-design.sql` | Server-seed-owned | Server-side seed API |
| Game Journey | `game-journey.sql` | Server-seed-owned | Server-side seed API |
| Game Hub | `game-workspace.sql` | Server-seed-owned | Server-side seed API |
| Messages | `messages.sql` | Server-seed-owned | Messages Local API/server-side SQLite service |
| Messages | `messages.sql` | Server-seed-owned | Messages Local API/server-side Postgres service |
| Objects | `objects.sql` | Server-seed-owned | Server-side seed API |
| Palette | `palette.sql` | Server-seed-owned | Server-side seed API |
| Support Tickets | `support-tickets.sql` | Server-seed-owned | Admin Site Setup/server-side seed API |
Expand Down
2 changes: 1 addition & 1 deletion docs_build/database/dml/messages.sql
View file
Open in desktop
Original file line numberDiff line numberDiff line change
Expand Up@@ -6,7 +6,7 @@
-- Owned tables: messages_categories, messages_emotion_profiles, messages_tts_profiles, messages_records, messages_segments

-- DML status: Server-seed-owned.
-- Setup is performed through the Messages Local API / server-side SQLite service for this PR.
-- Setup is performed through the Messages Local API / server-side Postgres service for this PR.
-- Browser pages must not seed authoritative records.
-- The server/API layer generates all keys and audit fields.
-- This SQL file intentionally has no direct INSERT statements because direct SQL would bypass key/audit ownership.
Expand Down
2 changes: 1 addition & 1 deletion docs_build/database/seed/messages.json
View file
Open in desktop
Original file line numberDiff line numberDiff line change
Expand Up@@ -31,5 +31,5 @@
"messages_records": [],
"messages_segments": []
},
"note": "Seed names are applied by the server-side Messages SQLite service. Browser pages must not seed authoritative records."
"note": "Seed names are applied by the server-side Messages Postgres service. Browser pages must not seed authoritative records."
}
16 changes: 16 additions & 0 deletions ..._BETA_022-messages-postgres-service-cutover-instruction-compliance-checklist.md
View file
Open in desktop
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,16 @@
# PR_26171_BETA_022 Instruction Compliance Checklist

TEAM ownership: BETA.

- PASS: Read `docs_build/dev/PROJECT_INSTRUCTIONS.md`.
- PASS: Read `docs_build/dev/PROJECT_MULTI_PC.txt`.
- PASS: Verified Messages belongs to Team Beta ownership.
- PASS: Started from latest `main` before creating `team/BETA/messages`.
- PASS: Scope stayed within Messages Local API/service/tests and directly affected active database docs.
- PASS: Removed active Messages SQLite runtime dependency without touching unrelated SQLite work.
- PASS: Used targeted validation only.
- PASS: Did not run samples.
- PASS: Did not merge.
- PASS: Required shared reports are generated under `docs_build/dev/reports/`.
- PASS: Manual validation notes are present.
- PASS: Repo-structured ZIP is required under `tmp/`.
21 changes: 21 additions & 0 deletions .../PR_26171_BETA_022-messages-postgres-service-cutover-manual-validation-notes.md
View file
Open in desktop
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,21 @@
# PR_26171_BETA_022 Manual Validation Notes

TEAM ownership: BETA.

Manual validation performed:
- Confirmed the Messages page loads through the repo Playwright server with the injected Postgres client stub.
- Confirmed seeded Messages categories, emotion profiles, and TTS profiles are available from `/api/messages/*`.
- Confirmed creating a message preserves `message`, `persistence`, `categoryName`, and `emotionProfileName` response shape fields.
- Confirmed creating a message segment preserves `segment`, `messageName`, and `emotionProfileName` response shape fields.
- Confirmed playback behavior remains covered by the Messages Playwright spec.
- Confirmed active Messages SQLite runtime dependency strings are absent from scoped active paths.

Expected outcome:
- Messages Local API persists through the Postgres service contract.
- Messages tool behavior remains unchanged from the browser user's perspective.
- No `messages.sqlite` file is created or required by the Messages tool tests.

Out of scope:
- Full samples smoke.
- Broad runtime regression suite.
- Production database connectivity against a live Postgres instance.
39 changes: 39 additions & 0 deletions docs_build/dev/reports/PR_26171_BETA_022-messages-postgres-service-cutover.md
View file
Open in desktop
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,39 @@
# PR_26171_BETA_022-messages-postgres-service-cutover

## Summary

TEAM ownership: BETA.

Branch: `team/BETA/messages`.

Scope completed:
- Replaced the active Messages Local API persistence service with a Postgres-backed implementation.
- Routed `/api/messages/*` through `createMessagesPostgresService`.
- Preserved the existing Local API response envelopes for Messages, categories, emotion profiles, TTS profiles, and segments.
- Removed the active Messages `node:sqlite`, `DatabaseSync`, `messages.sqlite`, and `GAMEFOUNDRY_MESSAGES_SQLITE_PATH` runtime path.
- Updated targeted Messages Playwright coverage to use an injected Postgres client stub.
- Updated active Messages DML/seed documentation from SQLite service wording to Postgres service wording.

Compatibility wrapper:
- No wrapper was kept. Active imports were migrated to `messages-postgres-service.mjs`, and targeted text checks found no remaining active reference to `messages-sqlite-service`.

Postgres authority:
- The runtime service uses the existing `createPostgresConnectionClient` path, which reads `GAMEFOUNDRY_DATABASE_URL` for the server-side database connection.
- Tests inject `createMessagesPostgresClientStub()` to preserve deterministic Local API and Playwright behavior without creating a local SQLite file.

## Validation

Passed:
- `node --test --test-name-pattern "Messages Local API" tests/dev-runtime/DbSeedIntegrity.test.mjs`
- `npx playwright test tests/playwright/tools/MessagesTool.spec.mjs --config=codex_playwright_system_chrome.config.cjs --project=playwright`
- `git diff --check`
- Targeted stale SQLite dependency check across Messages service, Local API router, affected tests, helper stubs, and active Messages docs.
- Targeted Postgres wiring check for `GAMEFOUNDRY_DATABASE_URL`, `createPostgresConnectionClient`, and Postgres service references.

Skipped:
- Full samples smoke: out of scope for this Messages Local API cutover.
- Broad runtime test suite: out of scope; validation was limited to the affected Messages and Local API surfaces.

## Notes

Playwright V8 coverage reports were refreshed by the targeted Messages Playwright run. Server-side runtime files are listed as advisory warnings because browser V8 coverage does not collect Node-side Local API modules.
101 changes: 34 additions & 67 deletions docs_build/dev/reports/codex_changed_files.txt
View file
Open in desktop
Original file line numberDiff line numberDiff line change
@@ -1,69 +1,36 @@
# PR_26171_GAMMA_019-admin-workstream-mergeability-recovery changed files
## git status --short
M docs_build/database/dml/DML_INDEX.md
M docs_build/database/dml/messages.sql
M docs_build/database/seed/messages.json
A docs_build/dev/reports/PR_26171_BETA_022-messages-postgres-service-cutover-instruction-compliance-checklist.md
A docs_build/dev/reports/PR_26171_BETA_022-messages-postgres-service-cutover-manual-validation-notes.md
A docs_build/dev/reports/PR_26171_BETA_022-messages-postgres-service-cutover.md
M docs_build/dev/reports/coverage_changed_js_guardrail.txt
M docs_build/dev/reports/playwright_v8_coverage_report.txt
A src/dev-runtime/messages/messages-postgres-service.mjs
D src/dev-runtime/messages/messages-sqlite-service.mjs
M src/dev-runtime/server/local-api-router.mjs
M tests/dev-runtime/DbSeedIntegrity.test.mjs
A tests/helpers/messagesPostgresClientStub.mjs
M tests/helpers/playwrightRepoServer.mjs
M tests/playwright/tools/MessagesTool.spec.mjs

Base: origin/main 35b04c02ea54da8b13c10354126f1ee8ddd14a89
Head before recovery merge commit: 1806adbf5df787f7072c6579d23b99bb4257466b
## git diff --cached --stat
docs_build/database/dml/DML_INDEX.md | 2 +-
docs_build/database/dml/messages.sql | 2 +-
docs_build/database/seed/messages.json | 2 +-
...ice-cutover-instruction-compliance-checklist.md | 16 +
...gres-service-cutover-manual-validation-notes.md | 21 +
...1_BETA_022-messages-postgres-service-cutover.md | 39 +
.../dev/reports/coverage_changed_js_guardrail.txt | 32 +-
.../dev/reports/playwright_v8_coverage_report.txt | 91 +-
.../messages/messages-postgres-service.mjs | 1117 +++++++++++++++++++
.../messages/messages-sqlite-service.mjs | 1118 --------------------
src/dev-runtime/server/local-api-router.mjs | 16 +-
tests/dev-runtime/DbSeedIntegrity.test.mjs | 61 +-
tests/helpers/messagesPostgresClientStub.mjs | 83 ++
tests/helpers/playwrightRepoServer.mjs | 6 +-
tests/playwright/tools/MessagesTool.spec.mjs | 39 +-
15 files changed, 1377 insertions(+), 1268 deletions(-)

## 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

## 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(-)

## 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
## git diff --stat
Binary file modifieddocs_build/dev/reports/codex_review.diff
View file
Open in desktop
Binary file not shown.
32 changes: 2 additions & 30 deletions docs_build/dev/reports/coverage_changed_js_guardrail.txt
View file
Open in desktop
Original file line numberDiff line numberDiff line change
Expand Up@@ -6,37 +6,9 @@ Missing changed runtime JS files are WARN, not FAIL.
Source: Playwright/Chromium built-in V8 coverage from the active Playwright run.

Changed runtime JS files considered:
(0%) assets/theme-v2/js/account-achievements.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) src/dev-runtime/messages/messages-sqlite-service.mjs - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) src/dev-runtime/persistence/mock-db-store.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) src/dev-runtime/persistence/tool-repositories/game-journey-mock-repository.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) src/dev-runtime/seed/server-seed-loader.mjs - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) src/dev-runtime/messages/messages-postgres-service.mjs - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) src/dev-runtime/server/local-api-router.mjs - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) src/shared/toolbox/tool-metadata-inventory.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) toolbox/controls/controls.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) toolbox/game-hub/game-hub-api-client.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) toolbox/game-hub/game-hub.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) toolbox/game-journey/game-journey.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) toolbox/idea-board/index.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) toolbox/messages/messages.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) toolbox/text-to-speech/text2speech.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) toolbox/tools-page-accordions.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(74%) assets/theme-v2/js/gamefoundry-partials.js - executed lines 977/977; executed functions 67/90
(90%) assets/theme-v2/js/admin-system-health.js - executed lines 227/227; executed functions 28/31

Guardrail warnings:
(0%) assets/theme-v2/js/account-achievements.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) src/dev-runtime/messages/messages-sqlite-service.mjs - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) src/dev-runtime/persistence/mock-db-store.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) src/dev-runtime/persistence/tool-repositories/game-journey-mock-repository.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) src/dev-runtime/seed/server-seed-loader.mjs - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) src/dev-runtime/messages/messages-postgres-service.mjs - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) src/dev-runtime/server/local-api-router.mjs - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) src/shared/toolbox/tool-metadata-inventory.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) toolbox/controls/controls.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) toolbox/game-hub/game-hub-api-client.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) toolbox/game-hub/game-hub.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) toolbox/game-journey/game-journey.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) toolbox/idea-board/index.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) toolbox/messages/messages.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) toolbox/text-to-speech/text2speech.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) toolbox/tools-page-accordions.js - WARNING: changed runtime JS file missing from coverage; advisory only
Loading
Loading
, 'i'); if (__m === '*' || __re.test(location.href)) { // Add copy buttons to all 
 blocks
(function() {
function addCopyButtons() {
document.querySelectorAll('pre code').forEach(function(codeBlock) {
if (codeBlock.parentElement.hasAttribute('data-copy-added')) return;
codeBlock.parentElement.setAttribute('data-copy-added', 'true');
var btn = document.createElement('button');
btn.textContent = 'Copy';
btn.style.cssText = 'position:absolute;top:4px;right:4px;padding:2px 8px;font-size:11px;background:#4ecdc4;border:none;border-radius:4px;color:#1a1a2e;cursor:pointer;opacity:0.7;transition:opacity 0.2s;';
btn.onmouseover = function() { this.style.opacity = '1'; };
btn.onmouseout = function() { this.style.opacity = '0.7'; };
btn.onclick = function() {
navigator.clipboard.writeText(codeBlock.textContent).then(function() {
btn.textContent = 'Copied!';
setTimeout(function() { btn.textContent = 'Copy'; }, 1500);
});
};
codeBlock.parentElement.style.position = 'relative';
codeBlock.parentElement.appendChild(btn);
});
}
addCopyButtons();
// Re-run on dynamic content
var observer = new MutationObserver(addCopyButtons);
observer.observe(document.body, { childList: true, subtree: true });
})();
}
} catch(__e) { console.warn('[Userscript:Add Copy Buttons to Code Blocks]', __e); }
})();
(function(){
try {
var __m = "github.com";
var __re = new RegExp('^' + "github\\.com" + '
PR_26171_BETA_022-messages-postgres-service-cutover by ToolboxAid · Pull Request #44 · ToolboxAid/HTML-JavaScript-Gaming · GitHub
Skip to content
PR_26171_BETA_022-messages-postgres-service-cutover#44

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

 Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and
privacy statement. We’ll occasionally send you account related emails.

Already on GitHub?
Sign in
to your account

 Merged

ToolboxAid
merged 1 commit into

from

Jun 21, 2026
 Merged


PR_26171_BETA_022-messages-postgres-service-cutover
#44
Show file tree
Hide file tree
 Changes from all commits
Commits

Show all changes


1 commit


Select commit


File filter

Filter by extension
Filter by extension
Conversations

Failed to load comments. 
Loading
Jump to

Jump to file


Failed to load files. 
Loading
 Diff view

 Diff view


2 changes: 1 addition & 1 deletion
docs_build/database/dml/DML_INDEX.md

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -14,7 +14,7 @@ Direct SQL setup is intentionally narrow. Account DEV users now require server-s
| Game Design | `game-design.sql` | Server-seed-owned | Server-side seed API |
| Game Journey | `game-journey.sql` | Server-seed-owned | Server-side seed API |
| Game Hub | `game-workspace.sql` | Server-seed-owned | Server-side seed API |
| Messages | `messages.sql` | Server-seed-owned | Messages Local API/server-side SQLite service |
| Messages | `messages.sql` | Server-seed-owned | Messages Local API/server-side Postgres service |
| Objects | `objects.sql` | Server-seed-owned | Server-side seed API |
| Palette | `palette.sql` | Server-seed-owned | Server-side seed API |
| Support Tickets | `support-tickets.sql` | Server-seed-owned | Admin Site Setup/server-side seed API |
Expand Down

2 changes: 1 addition & 1 deletion
docs_build/database/dml/messages.sql

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -6,7 +6,7 @@
-- Owned tables: messages_categories, messages_emotion_profiles, messages_tts_profiles, messages_records, messages_segments

-- DML status: Server-seed-owned.
-- Setup is performed through the Messages Local API / server-side SQLite service for this PR.
-- Setup is performed through the Messages Local API / server-side Postgres service for this PR.
-- Browser pages must not seed authoritative records.
-- The server/API layer generates all keys and audit fields.
-- This SQL file intentionally has no direct INSERT statements because direct SQL would bypass key/audit ownership.
Expand Down

2 changes: 1 addition & 1 deletion
docs_build/database/seed/messages.json

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -31,5 +31,5 @@
 "messages_records": [],
 "messages_segments": []
 },
 "note": "Seed names are applied by the server-side Messages SQLite service. Browser pages must not seed authoritative records."
 "note": "Seed names are applied by the server-side Messages Postgres service. Browser pages must not seed authoritative records."
}

16 changes: 16 additions & 0 deletions
..._BETA_022-messages-postgres-service-cutover-instruction-compliance-checklist.md

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -0,0 +1,16 @@
# PR_26171_BETA_022 Instruction Compliance Checklist

TEAM ownership: BETA.

- PASS: Read `docs_build/dev/PROJECT_INSTRUCTIONS.md`.
- PASS: Read `docs_build/dev/PROJECT_MULTI_PC.txt`.
- PASS: Verified Messages belongs to Team Beta ownership.
- PASS: Started from latest `main` before creating `team/BETA/messages`.
- PASS: Scope stayed within Messages Local API/service/tests and directly affected active database docs.
- PASS: Removed active Messages SQLite runtime dependency without touching unrelated SQLite work.
- PASS: Used targeted validation only.
- PASS: Did not run samples.
- PASS: Did not merge.
- PASS: Required shared reports are generated under `docs_build/dev/reports/`.
- PASS: Manual validation notes are present.
- PASS: Repo-structured ZIP is required under `tmp/`.

21 changes: 21 additions & 0 deletions
.../PR_26171_BETA_022-messages-postgres-service-cutover-manual-validation-notes.md

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -0,0 +1,21 @@
# PR_26171_BETA_022 Manual Validation Notes

TEAM ownership: BETA.

Manual validation performed:
- Confirmed the Messages page loads through the repo Playwright server with the injected Postgres client stub.
- Confirmed seeded Messages categories, emotion profiles, and TTS profiles are available from `/api/messages/*`.
- Confirmed creating a message preserves `message`, `persistence`, `categoryName`, and `emotionProfileName` response shape fields.
- Confirmed creating a message segment preserves `segment`, `messageName`, and `emotionProfileName` response shape fields.
- Confirmed playback behavior remains covered by the Messages Playwright spec.
- Confirmed active Messages SQLite runtime dependency strings are absent from scoped active paths.

Expected outcome:
- Messages Local API persists through the Postgres service contract.
- Messages tool behavior remains unchanged from the browser user's perspective.
- No `messages.sqlite` file is created or required by the Messages tool tests.

Out of scope:
- Full samples smoke.
- Broad runtime regression suite.
- Production database connectivity against a live Postgres instance.

39 changes: 39 additions & 0 deletions
docs_build/dev/reports/PR_26171_BETA_022-messages-postgres-service-cutover.md

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -0,0 +1,39 @@
# PR_26171_BETA_022-messages-postgres-service-cutover

## Summary

TEAM ownership: BETA.

Branch: `team/BETA/messages`.

Scope completed:
- Replaced the active Messages Local API persistence service with a Postgres-backed implementation.
- Routed `/api/messages/*` through `createMessagesPostgresService`.
- Preserved the existing Local API response envelopes for Messages, categories, emotion profiles, TTS profiles, and segments.
- Removed the active Messages `node:sqlite`, `DatabaseSync`, `messages.sqlite`, and `GAMEFOUNDRY_MESSAGES_SQLITE_PATH` runtime path.
- Updated targeted Messages Playwright coverage to use an injected Postgres client stub.
- Updated active Messages DML/seed documentation from SQLite service wording to Postgres service wording.

Compatibility wrapper:
- No wrapper was kept. Active imports were migrated to `messages-postgres-service.mjs`, and targeted text checks found no remaining active reference to `messages-sqlite-service`.

Postgres authority:
- The runtime service uses the existing `createPostgresConnectionClient` path, which reads `GAMEFOUNDRY_DATABASE_URL` for the server-side database connection.
- Tests inject `createMessagesPostgresClientStub()` to preserve deterministic Local API and Playwright behavior without creating a local SQLite file.

## Validation

Passed:
- `node --test --test-name-pattern "Messages Local API" tests/dev-runtime/DbSeedIntegrity.test.mjs`
- `npx playwright test tests/playwright/tools/MessagesTool.spec.mjs --config=codex_playwright_system_chrome.config.cjs --project=playwright`
- `git diff --check`
- Targeted stale SQLite dependency check across Messages service, Local API router, affected tests, helper stubs, and active Messages docs.
- Targeted Postgres wiring check for `GAMEFOUNDRY_DATABASE_URL`, `createPostgresConnectionClient`, and Postgres service references.

Skipped:
- Full samples smoke: out of scope for this Messages Local API cutover.
- Broad runtime test suite: out of scope; validation was limited to the affected Messages and Local API surfaces.

## Notes

Playwright V8 coverage reports were refreshed by the targeted Messages Playwright run. Server-side runtime files are listed as advisory warnings because browser V8 coverage does not collect Node-side Local API modules.

101 changes: 34 additions & 67 deletions
docs_build/dev/reports/codex_changed_files.txt

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -1,69 +1,36 @@
# PR_26171_GAMMA_019-admin-workstream-mergeability-recovery changed files
## git status --short
M docs_build/database/dml/DML_INDEX.md
M docs_build/database/dml/messages.sql
M docs_build/database/seed/messages.json
A docs_build/dev/reports/PR_26171_BETA_022-messages-postgres-service-cutover-instruction-compliance-checklist.md
A docs_build/dev/reports/PR_26171_BETA_022-messages-postgres-service-cutover-manual-validation-notes.md
A docs_build/dev/reports/PR_26171_BETA_022-messages-postgres-service-cutover.md
M docs_build/dev/reports/coverage_changed_js_guardrail.txt
M docs_build/dev/reports/playwright_v8_coverage_report.txt
A src/dev-runtime/messages/messages-postgres-service.mjs
D src/dev-runtime/messages/messages-sqlite-service.mjs
M src/dev-runtime/server/local-api-router.mjs
M tests/dev-runtime/DbSeedIntegrity.test.mjs
A tests/helpers/messagesPostgresClientStub.mjs
M tests/helpers/playwrightRepoServer.mjs
M tests/playwright/tools/MessagesTool.spec.mjs

Base: origin/main 35b04c02ea54da8b13c10354126f1ee8ddd14a89
Head before recovery merge commit: 1806adbf5df787f7072c6579d23b99bb4257466b
## git diff --cached --stat
 docs_build/database/dml/DML_INDEX.md | 2 +-
 docs_build/database/dml/messages.sql | 2 +-
 docs_build/database/seed/messages.json | 2 +-
 ...ice-cutover-instruction-compliance-checklist.md | 16 +
 ...gres-service-cutover-manual-validation-notes.md | 21 +
 ...1_BETA_022-messages-postgres-service-cutover.md | 39 +
 .../dev/reports/coverage_changed_js_guardrail.txt | 32 +-
 .../dev/reports/playwright_v8_coverage_report.txt | 91 +-
 .../messages/messages-postgres-service.mjs | 1117 +++++++++++++++++++
 .../messages/messages-sqlite-service.mjs | 1118 --------------------
 src/dev-runtime/server/local-api-router.mjs | 16 +-
 tests/dev-runtime/DbSeedIntegrity.test.mjs | 61 +-
 tests/helpers/messagesPostgresClientStub.mjs | 83 ++
 tests/helpers/playwrightRepoServer.mjs | 6 +-
 tests/playwright/tools/MessagesTool.spec.mjs | 39 +-
 15 files changed, 1377 insertions(+), 1268 deletions(-)

## 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

## 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(-)

## 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
## git diff --stat
Binary file modifieddocs_build/dev/reports/codex_review.diff

View file


Open in desktop


Binary file not shown.


32 changes: 2 additions & 30 deletions
docs_build/dev/reports/coverage_changed_js_guardrail.txt

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -6,37 +6,9 @@ Missing changed runtime JS files are WARN, not FAIL.
Source: Playwright/Chromium built-in V8 coverage from the active Playwright run.

Changed runtime JS files considered:
(0%) assets/theme-v2/js/account-achievements.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) src/dev-runtime/messages/messages-sqlite-service.mjs - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) src/dev-runtime/persistence/mock-db-store.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) src/dev-runtime/persistence/tool-repositories/game-journey-mock-repository.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) src/dev-runtime/seed/server-seed-loader.mjs - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) src/dev-runtime/messages/messages-postgres-service.mjs - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) src/dev-runtime/server/local-api-router.mjs - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) src/shared/toolbox/tool-metadata-inventory.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) toolbox/controls/controls.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) toolbox/game-hub/game-hub-api-client.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) toolbox/game-hub/game-hub.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) toolbox/game-journey/game-journey.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) toolbox/idea-board/index.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) toolbox/messages/messages.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) toolbox/text-to-speech/text2speech.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) toolbox/tools-page-accordions.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(74%) assets/theme-v2/js/gamefoundry-partials.js - executed lines 977/977; executed functions 67/90
(90%) assets/theme-v2/js/admin-system-health.js - executed lines 227/227; executed functions 28/31

Guardrail warnings:
(0%) assets/theme-v2/js/account-achievements.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) src/dev-runtime/messages/messages-sqlite-service.mjs - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) src/dev-runtime/persistence/mock-db-store.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) src/dev-runtime/persistence/tool-repositories/game-journey-mock-repository.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) src/dev-runtime/seed/server-seed-loader.mjs - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) src/dev-runtime/messages/messages-postgres-service.mjs - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) src/dev-runtime/server/local-api-router.mjs - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) src/shared/toolbox/tool-metadata-inventory.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) toolbox/controls/controls.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) toolbox/game-hub/game-hub-api-client.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) toolbox/game-hub/game-hub.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) toolbox/game-journey/game-journey.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) toolbox/idea-board/index.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) toolbox/messages/messages.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) toolbox/text-to-speech/text2speech.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) toolbox/tools-page-accordions.js - WARNING: changed runtime JS file missing from coverage; advisory only
Loading
Loading

, 'i');
if (__m === '*' || __re.test(location.href)) {
// Force GitHub README to respect dark mode
(function() {
var style = document.createElement('style');
style.textContent = '
.markdown-body {
color-scheme: dark light;
}
.markdown-body pre { background: #161b22 !important; }
.markdown-body code { background: rgba(110, 118, 129, 0.4) !important; }
.markdown-body table th, .markdown-body table td { border-color: #30363d !important; }
.markdown-body img { background: #0d1117; }
.markdown-body blockquote { border-left-color: #8b949e; }
.markdown-body hr { border-color: #30363d; }
';
document.head.appendChild(style);
})();
}
} catch(__e) { console.warn('[Userscript:GitHub Dark Mode README Fix]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '
PR_26171_BETA_022-messages-postgres-service-cutover by ToolboxAid · Pull Request #44 · ToolboxAid/HTML-JavaScript-Gaming · GitHub
Skip to content
PR_26171_BETA_022-messages-postgres-service-cutover#44

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

 Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and
privacy statement. We’ll occasionally send you account related emails.

Already on GitHub?
Sign in
to your account

 Merged

ToolboxAid
merged 1 commit into

from

Jun 21, 2026
 Merged


PR_26171_BETA_022-messages-postgres-service-cutover
#44
Show file tree
Hide file tree
 Changes from all commits
Commits

Show all changes


1 commit


Select commit


File filter

Filter by extension
Filter by extension
Conversations

Failed to load comments. 
Loading
Jump to

Jump to file


Failed to load files. 
Loading
 Diff view

 Diff view


2 changes: 1 addition & 1 deletion
docs_build/database/dml/DML_INDEX.md

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -14,7 +14,7 @@ Direct SQL setup is intentionally narrow. Account DEV users now require server-s
| Game Design | `game-design.sql` | Server-seed-owned | Server-side seed API |
| Game Journey | `game-journey.sql` | Server-seed-owned | Server-side seed API |
| Game Hub | `game-workspace.sql` | Server-seed-owned | Server-side seed API |
| Messages | `messages.sql` | Server-seed-owned | Messages Local API/server-side SQLite service |
| Messages | `messages.sql` | Server-seed-owned | Messages Local API/server-side Postgres service |
| Objects | `objects.sql` | Server-seed-owned | Server-side seed API |
| Palette | `palette.sql` | Server-seed-owned | Server-side seed API |
| Support Tickets | `support-tickets.sql` | Server-seed-owned | Admin Site Setup/server-side seed API |
Expand Down

2 changes: 1 addition & 1 deletion
docs_build/database/dml/messages.sql

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -6,7 +6,7 @@
-- Owned tables: messages_categories, messages_emotion_profiles, messages_tts_profiles, messages_records, messages_segments

-- DML status: Server-seed-owned.
-- Setup is performed through the Messages Local API / server-side SQLite service for this PR.
-- Setup is performed through the Messages Local API / server-side Postgres service for this PR.
-- Browser pages must not seed authoritative records.
-- The server/API layer generates all keys and audit fields.
-- This SQL file intentionally has no direct INSERT statements because direct SQL would bypass key/audit ownership.
Expand Down

2 changes: 1 addition & 1 deletion
docs_build/database/seed/messages.json

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -31,5 +31,5 @@
 "messages_records": [],
 "messages_segments": []
 },
 "note": "Seed names are applied by the server-side Messages SQLite service. Browser pages must not seed authoritative records."
 "note": "Seed names are applied by the server-side Messages Postgres service. Browser pages must not seed authoritative records."
}

16 changes: 16 additions & 0 deletions
..._BETA_022-messages-postgres-service-cutover-instruction-compliance-checklist.md

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -0,0 +1,16 @@
# PR_26171_BETA_022 Instruction Compliance Checklist

TEAM ownership: BETA.

- PASS: Read `docs_build/dev/PROJECT_INSTRUCTIONS.md`.
- PASS: Read `docs_build/dev/PROJECT_MULTI_PC.txt`.
- PASS: Verified Messages belongs to Team Beta ownership.
- PASS: Started from latest `main` before creating `team/BETA/messages`.
- PASS: Scope stayed within Messages Local API/service/tests and directly affected active database docs.
- PASS: Removed active Messages SQLite runtime dependency without touching unrelated SQLite work.
- PASS: Used targeted validation only.
- PASS: Did not run samples.
- PASS: Did not merge.
- PASS: Required shared reports are generated under `docs_build/dev/reports/`.
- PASS: Manual validation notes are present.
- PASS: Repo-structured ZIP is required under `tmp/`.

21 changes: 21 additions & 0 deletions
.../PR_26171_BETA_022-messages-postgres-service-cutover-manual-validation-notes.md

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -0,0 +1,21 @@
# PR_26171_BETA_022 Manual Validation Notes

TEAM ownership: BETA.

Manual validation performed:
- Confirmed the Messages page loads through the repo Playwright server with the injected Postgres client stub.
- Confirmed seeded Messages categories, emotion profiles, and TTS profiles are available from `/api/messages/*`.
- Confirmed creating a message preserves `message`, `persistence`, `categoryName`, and `emotionProfileName` response shape fields.
- Confirmed creating a message segment preserves `segment`, `messageName`, and `emotionProfileName` response shape fields.
- Confirmed playback behavior remains covered by the Messages Playwright spec.
- Confirmed active Messages SQLite runtime dependency strings are absent from scoped active paths.

Expected outcome:
- Messages Local API persists through the Postgres service contract.
- Messages tool behavior remains unchanged from the browser user's perspective.
- No `messages.sqlite` file is created or required by the Messages tool tests.

Out of scope:
- Full samples smoke.
- Broad runtime regression suite.
- Production database connectivity against a live Postgres instance.

39 changes: 39 additions & 0 deletions
docs_build/dev/reports/PR_26171_BETA_022-messages-postgres-service-cutover.md

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -0,0 +1,39 @@
# PR_26171_BETA_022-messages-postgres-service-cutover

## Summary

TEAM ownership: BETA.

Branch: `team/BETA/messages`.

Scope completed:
- Replaced the active Messages Local API persistence service with a Postgres-backed implementation.
- Routed `/api/messages/*` through `createMessagesPostgresService`.
- Preserved the existing Local API response envelopes for Messages, categories, emotion profiles, TTS profiles, and segments.
- Removed the active Messages `node:sqlite`, `DatabaseSync`, `messages.sqlite`, and `GAMEFOUNDRY_MESSAGES_SQLITE_PATH` runtime path.
- Updated targeted Messages Playwright coverage to use an injected Postgres client stub.
- Updated active Messages DML/seed documentation from SQLite service wording to Postgres service wording.

Compatibility wrapper:
- No wrapper was kept. Active imports were migrated to `messages-postgres-service.mjs`, and targeted text checks found no remaining active reference to `messages-sqlite-service`.

Postgres authority:
- The runtime service uses the existing `createPostgresConnectionClient` path, which reads `GAMEFOUNDRY_DATABASE_URL` for the server-side database connection.
- Tests inject `createMessagesPostgresClientStub()` to preserve deterministic Local API and Playwright behavior without creating a local SQLite file.

## Validation

Passed:
- `node --test --test-name-pattern "Messages Local API" tests/dev-runtime/DbSeedIntegrity.test.mjs`
- `npx playwright test tests/playwright/tools/MessagesTool.spec.mjs --config=codex_playwright_system_chrome.config.cjs --project=playwright`
- `git diff --check`
- Targeted stale SQLite dependency check across Messages service, Local API router, affected tests, helper stubs, and active Messages docs.
- Targeted Postgres wiring check for `GAMEFOUNDRY_DATABASE_URL`, `createPostgresConnectionClient`, and Postgres service references.

Skipped:
- Full samples smoke: out of scope for this Messages Local API cutover.
- Broad runtime test suite: out of scope; validation was limited to the affected Messages and Local API surfaces.

## Notes

Playwright V8 coverage reports were refreshed by the targeted Messages Playwright run. Server-side runtime files are listed as advisory warnings because browser V8 coverage does not collect Node-side Local API modules.

101 changes: 34 additions & 67 deletions
docs_build/dev/reports/codex_changed_files.txt

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -1,69 +1,36 @@
# PR_26171_GAMMA_019-admin-workstream-mergeability-recovery changed files
## git status --short
M docs_build/database/dml/DML_INDEX.md
M docs_build/database/dml/messages.sql
M docs_build/database/seed/messages.json
A docs_build/dev/reports/PR_26171_BETA_022-messages-postgres-service-cutover-instruction-compliance-checklist.md
A docs_build/dev/reports/PR_26171_BETA_022-messages-postgres-service-cutover-manual-validation-notes.md
A docs_build/dev/reports/PR_26171_BETA_022-messages-postgres-service-cutover.md
M docs_build/dev/reports/coverage_changed_js_guardrail.txt
M docs_build/dev/reports/playwright_v8_coverage_report.txt
A src/dev-runtime/messages/messages-postgres-service.mjs
D src/dev-runtime/messages/messages-sqlite-service.mjs
M src/dev-runtime/server/local-api-router.mjs
M tests/dev-runtime/DbSeedIntegrity.test.mjs
A tests/helpers/messagesPostgresClientStub.mjs
M tests/helpers/playwrightRepoServer.mjs
M tests/playwright/tools/MessagesTool.spec.mjs

Base: origin/main 35b04c02ea54da8b13c10354126f1ee8ddd14a89
Head before recovery merge commit: 1806adbf5df787f7072c6579d23b99bb4257466b
## git diff --cached --stat
 docs_build/database/dml/DML_INDEX.md | 2 +-
 docs_build/database/dml/messages.sql | 2 +-
 docs_build/database/seed/messages.json | 2 +-
 ...ice-cutover-instruction-compliance-checklist.md | 16 +
 ...gres-service-cutover-manual-validation-notes.md | 21 +
 ...1_BETA_022-messages-postgres-service-cutover.md | 39 +
 .../dev/reports/coverage_changed_js_guardrail.txt | 32 +-
 .../dev/reports/playwright_v8_coverage_report.txt | 91 +-
 .../messages/messages-postgres-service.mjs | 1117 +++++++++++++++++++
 .../messages/messages-sqlite-service.mjs | 1118 --------------------
 src/dev-runtime/server/local-api-router.mjs | 16 +-
 tests/dev-runtime/DbSeedIntegrity.test.mjs | 61 +-
 tests/helpers/messagesPostgresClientStub.mjs | 83 ++
 tests/helpers/playwrightRepoServer.mjs | 6 +-
 tests/playwright/tools/MessagesTool.spec.mjs | 39 +-
 15 files changed, 1377 insertions(+), 1268 deletions(-)

## 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

## 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(-)

## 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
## git diff --stat
Binary file modifieddocs_build/dev/reports/codex_review.diff

View file


Open in desktop


Binary file not shown.


32 changes: 2 additions & 30 deletions
docs_build/dev/reports/coverage_changed_js_guardrail.txt

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -6,37 +6,9 @@ Missing changed runtime JS files are WARN, not FAIL.
Source: Playwright/Chromium built-in V8 coverage from the active Playwright run.

Changed runtime JS files considered:
(0%) assets/theme-v2/js/account-achievements.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) src/dev-runtime/messages/messages-sqlite-service.mjs - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) src/dev-runtime/persistence/mock-db-store.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) src/dev-runtime/persistence/tool-repositories/game-journey-mock-repository.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) src/dev-runtime/seed/server-seed-loader.mjs - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) src/dev-runtime/messages/messages-postgres-service.mjs - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) src/dev-runtime/server/local-api-router.mjs - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) src/shared/toolbox/tool-metadata-inventory.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) toolbox/controls/controls.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) toolbox/game-hub/game-hub-api-client.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) toolbox/game-hub/game-hub.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) toolbox/game-journey/game-journey.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) toolbox/idea-board/index.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) toolbox/messages/messages.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) toolbox/text-to-speech/text2speech.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) toolbox/tools-page-accordions.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(74%) assets/theme-v2/js/gamefoundry-partials.js - executed lines 977/977; executed functions 67/90
(90%) assets/theme-v2/js/admin-system-health.js - executed lines 227/227; executed functions 28/31

Guardrail warnings:
(0%) assets/theme-v2/js/account-achievements.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) src/dev-runtime/messages/messages-sqlite-service.mjs - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) src/dev-runtime/persistence/mock-db-store.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) src/dev-runtime/persistence/tool-repositories/game-journey-mock-repository.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) src/dev-runtime/seed/server-seed-loader.mjs - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) src/dev-runtime/messages/messages-postgres-service.mjs - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) src/dev-runtime/server/local-api-router.mjs - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) src/shared/toolbox/tool-metadata-inventory.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) toolbox/controls/controls.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) toolbox/game-hub/game-hub-api-client.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) toolbox/game-hub/game-hub.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) toolbox/game-journey/game-journey.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) toolbox/idea-board/index.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) toolbox/messages/messages.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) toolbox/text-to-speech/text2speech.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) toolbox/tools-page-accordions.js - WARNING: changed runtime JS file missing from coverage; advisory only
Loading
Loading

, 'i');
if (__m === '*' || __re.test(location.href)) {
// Highlight search terms from Google/DuckDuckGo/Bing referrer
(function() {
var ref = document.referrer;
var terms = [];
if (ref.includes('google.com') || ref.includes('duckduckgo.com') || ref.includes('bing.com')) {
var url = new URL(ref);
var q = url.searchParams.get('q') || url.searchParams.get('p');
if (q) {
terms = q.split(/\s+/).filter(function(t) { return t.length > 2; });
}
}
if (terms.length === 0) return;
var style = document.createElement('style');
style.textContent = '.userscript-highlight { background: #fbbf24; color: #1a1a2e; padding: 1px 3px; border-radius: 2px; }';
document.head.appendChild(style);
function highlight(node) {
if (node.nodeType === 3) { // text node
var text = node.textContent;
var found = false;
terms.forEach(function(term) {
var regex = new RegExp('(' + term.replace(/[.*+?^${}()|[\]\\]/g, '\\') + ')', 'gi');
if (regex.test(text)) {
found = true;
var frag = document.createDocumentFragment();
var parts = text.split(regex);
parts.forEach(function(part, i) {
if (i % 2 === 0) {
frag.appendChild(document.createTextNode(part));
} else {
var span = document.createElement('span');
span.className = 'userscript-highlight';
span.textContent = part;
frag.appendChild(span);
}
});
node.parentNode.replaceChild(frag, node);
}
});
} else if (node.nodeType === 1 && node.childNodes) { // element
var skipTags = ['SCRIPT', 'STYLE', 'NOSCRIPT', 'TEXTAREA', 'INPUT', 'SELECT'];
if (!skipTags.includes(node.tagName)) {
Array.from(node.childNodes).forEach(highlight);
}
}
}
highlight(document.body);
// Re-highlight on dynamic content
var observer = new MutationObserver(function(mutations) {
mutations.forEach(function(m) {
m.addedNodes.forEach(function(node) {
if (node.nodeType === 1 || node.nodeType === 3) highlight(node);
});
});
});
observer.observe(document.body, { childList: true, subtree: true });
})();
}
} catch(__e) { console.warn('[Userscript:Highlight Search Terms]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '
PR_26171_BETA_022-messages-postgres-service-cutover by ToolboxAid · Pull Request #44 · ToolboxAid/HTML-JavaScript-Gaming · GitHub
Skip to content
PR_26171_BETA_022-messages-postgres-service-cutover#44

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

 Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and
privacy statement. We’ll occasionally send you account related emails.

Already on GitHub?
Sign in
to your account

 Merged

ToolboxAid
merged 1 commit into

from

Jun 21, 2026
 Merged


PR_26171_BETA_022-messages-postgres-service-cutover
#44
Show file tree
Hide file tree
 Changes from all commits
Commits

Show all changes


1 commit


Select commit


File filter

Filter by extension
Filter by extension
Conversations

Failed to load comments. 
Loading
Jump to

Jump to file


Failed to load files. 
Loading
 Diff view

 Diff view


2 changes: 1 addition & 1 deletion
docs_build/database/dml/DML_INDEX.md

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -14,7 +14,7 @@ Direct SQL setup is intentionally narrow. Account DEV users now require server-s
| Game Design | `game-design.sql` | Server-seed-owned | Server-side seed API |
| Game Journey | `game-journey.sql` | Server-seed-owned | Server-side seed API |
| Game Hub | `game-workspace.sql` | Server-seed-owned | Server-side seed API |
| Messages | `messages.sql` | Server-seed-owned | Messages Local API/server-side SQLite service |
| Messages | `messages.sql` | Server-seed-owned | Messages Local API/server-side Postgres service |
| Objects | `objects.sql` | Server-seed-owned | Server-side seed API |
| Palette | `palette.sql` | Server-seed-owned | Server-side seed API |
| Support Tickets | `support-tickets.sql` | Server-seed-owned | Admin Site Setup/server-side seed API |
Expand Down

2 changes: 1 addition & 1 deletion
docs_build/database/dml/messages.sql

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -6,7 +6,7 @@
-- Owned tables: messages_categories, messages_emotion_profiles, messages_tts_profiles, messages_records, messages_segments

-- DML status: Server-seed-owned.
-- Setup is performed through the Messages Local API / server-side SQLite service for this PR.
-- Setup is performed through the Messages Local API / server-side Postgres service for this PR.
-- Browser pages must not seed authoritative records.
-- The server/API layer generates all keys and audit fields.
-- This SQL file intentionally has no direct INSERT statements because direct SQL would bypass key/audit ownership.
Expand Down

2 changes: 1 addition & 1 deletion
docs_build/database/seed/messages.json

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -31,5 +31,5 @@
 "messages_records": [],
 "messages_segments": []
 },
 "note": "Seed names are applied by the server-side Messages SQLite service. Browser pages must not seed authoritative records."
 "note": "Seed names are applied by the server-side Messages Postgres service. Browser pages must not seed authoritative records."
}

16 changes: 16 additions & 0 deletions
..._BETA_022-messages-postgres-service-cutover-instruction-compliance-checklist.md

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -0,0 +1,16 @@
# PR_26171_BETA_022 Instruction Compliance Checklist

TEAM ownership: BETA.

- PASS: Read `docs_build/dev/PROJECT_INSTRUCTIONS.md`.
- PASS: Read `docs_build/dev/PROJECT_MULTI_PC.txt`.
- PASS: Verified Messages belongs to Team Beta ownership.
- PASS: Started from latest `main` before creating `team/BETA/messages`.
- PASS: Scope stayed within Messages Local API/service/tests and directly affected active database docs.
- PASS: Removed active Messages SQLite runtime dependency without touching unrelated SQLite work.
- PASS: Used targeted validation only.
- PASS: Did not run samples.
- PASS: Did not merge.
- PASS: Required shared reports are generated under `docs_build/dev/reports/`.
- PASS: Manual validation notes are present.
- PASS: Repo-structured ZIP is required under `tmp/`.

21 changes: 21 additions & 0 deletions
.../PR_26171_BETA_022-messages-postgres-service-cutover-manual-validation-notes.md

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -0,0 +1,21 @@
# PR_26171_BETA_022 Manual Validation Notes

TEAM ownership: BETA.

Manual validation performed:
- Confirmed the Messages page loads through the repo Playwright server with the injected Postgres client stub.
- Confirmed seeded Messages categories, emotion profiles, and TTS profiles are available from `/api/messages/*`.
- Confirmed creating a message preserves `message`, `persistence`, `categoryName`, and `emotionProfileName` response shape fields.
- Confirmed creating a message segment preserves `segment`, `messageName`, and `emotionProfileName` response shape fields.
- Confirmed playback behavior remains covered by the Messages Playwright spec.
- Confirmed active Messages SQLite runtime dependency strings are absent from scoped active paths.

Expected outcome:
- Messages Local API persists through the Postgres service contract.
- Messages tool behavior remains unchanged from the browser user's perspective.
- No `messages.sqlite` file is created or required by the Messages tool tests.

Out of scope:
- Full samples smoke.
- Broad runtime regression suite.
- Production database connectivity against a live Postgres instance.

39 changes: 39 additions & 0 deletions
docs_build/dev/reports/PR_26171_BETA_022-messages-postgres-service-cutover.md

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -0,0 +1,39 @@
# PR_26171_BETA_022-messages-postgres-service-cutover

## Summary

TEAM ownership: BETA.

Branch: `team/BETA/messages`.

Scope completed:
- Replaced the active Messages Local API persistence service with a Postgres-backed implementation.
- Routed `/api/messages/*` through `createMessagesPostgresService`.
- Preserved the existing Local API response envelopes for Messages, categories, emotion profiles, TTS profiles, and segments.
- Removed the active Messages `node:sqlite`, `DatabaseSync`, `messages.sqlite`, and `GAMEFOUNDRY_MESSAGES_SQLITE_PATH` runtime path.
- Updated targeted Messages Playwright coverage to use an injected Postgres client stub.
- Updated active Messages DML/seed documentation from SQLite service wording to Postgres service wording.

Compatibility wrapper:
- No wrapper was kept. Active imports were migrated to `messages-postgres-service.mjs`, and targeted text checks found no remaining active reference to `messages-sqlite-service`.

Postgres authority:
- The runtime service uses the existing `createPostgresConnectionClient` path, which reads `GAMEFOUNDRY_DATABASE_URL` for the server-side database connection.
- Tests inject `createMessagesPostgresClientStub()` to preserve deterministic Local API and Playwright behavior without creating a local SQLite file.

## Validation

Passed:
- `node --test --test-name-pattern "Messages Local API" tests/dev-runtime/DbSeedIntegrity.test.mjs`
- `npx playwright test tests/playwright/tools/MessagesTool.spec.mjs --config=codex_playwright_system_chrome.config.cjs --project=playwright`
- `git diff --check`
- Targeted stale SQLite dependency check across Messages service, Local API router, affected tests, helper stubs, and active Messages docs.
- Targeted Postgres wiring check for `GAMEFOUNDRY_DATABASE_URL`, `createPostgresConnectionClient`, and Postgres service references.

Skipped:
- Full samples smoke: out of scope for this Messages Local API cutover.
- Broad runtime test suite: out of scope; validation was limited to the affected Messages and Local API surfaces.

## Notes

Playwright V8 coverage reports were refreshed by the targeted Messages Playwright run. Server-side runtime files are listed as advisory warnings because browser V8 coverage does not collect Node-side Local API modules.

101 changes: 34 additions & 67 deletions
docs_build/dev/reports/codex_changed_files.txt

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -1,69 +1,36 @@
# PR_26171_GAMMA_019-admin-workstream-mergeability-recovery changed files
## git status --short
M docs_build/database/dml/DML_INDEX.md
M docs_build/database/dml/messages.sql
M docs_build/database/seed/messages.json
A docs_build/dev/reports/PR_26171_BETA_022-messages-postgres-service-cutover-instruction-compliance-checklist.md
A docs_build/dev/reports/PR_26171_BETA_022-messages-postgres-service-cutover-manual-validation-notes.md
A docs_build/dev/reports/PR_26171_BETA_022-messages-postgres-service-cutover.md
M docs_build/dev/reports/coverage_changed_js_guardrail.txt
M docs_build/dev/reports/playwright_v8_coverage_report.txt
A src/dev-runtime/messages/messages-postgres-service.mjs
D src/dev-runtime/messages/messages-sqlite-service.mjs
M src/dev-runtime/server/local-api-router.mjs
M tests/dev-runtime/DbSeedIntegrity.test.mjs
A tests/helpers/messagesPostgresClientStub.mjs
M tests/helpers/playwrightRepoServer.mjs
M tests/playwright/tools/MessagesTool.spec.mjs

Base: origin/main 35b04c02ea54da8b13c10354126f1ee8ddd14a89
Head before recovery merge commit: 1806adbf5df787f7072c6579d23b99bb4257466b
## git diff --cached --stat
 docs_build/database/dml/DML_INDEX.md | 2 +-
 docs_build/database/dml/messages.sql | 2 +-
 docs_build/database/seed/messages.json | 2 +-
 ...ice-cutover-instruction-compliance-checklist.md | 16 +
 ...gres-service-cutover-manual-validation-notes.md | 21 +
 ...1_BETA_022-messages-postgres-service-cutover.md | 39 +
 .../dev/reports/coverage_changed_js_guardrail.txt | 32 +-
 .../dev/reports/playwright_v8_coverage_report.txt | 91 +-
 .../messages/messages-postgres-service.mjs | 1117 +++++++++++++++++++
 .../messages/messages-sqlite-service.mjs | 1118 --------------------
 src/dev-runtime/server/local-api-router.mjs | 16 +-
 tests/dev-runtime/DbSeedIntegrity.test.mjs | 61 +-
 tests/helpers/messagesPostgresClientStub.mjs | 83 ++
 tests/helpers/playwrightRepoServer.mjs | 6 +-
 tests/playwright/tools/MessagesTool.spec.mjs | 39 +-
 15 files changed, 1377 insertions(+), 1268 deletions(-)

## 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

## 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(-)

## 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
## git diff --stat
Binary file modifieddocs_build/dev/reports/codex_review.diff

View file


Open in desktop


Binary file not shown.


32 changes: 2 additions & 30 deletions
docs_build/dev/reports/coverage_changed_js_guardrail.txt

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -6,37 +6,9 @@ Missing changed runtime JS files are WARN, not FAIL.
Source: Playwright/Chromium built-in V8 coverage from the active Playwright run.

Changed runtime JS files considered:
(0%) assets/theme-v2/js/account-achievements.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) src/dev-runtime/messages/messages-sqlite-service.mjs - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) src/dev-runtime/persistence/mock-db-store.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) src/dev-runtime/persistence/tool-repositories/game-journey-mock-repository.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) src/dev-runtime/seed/server-seed-loader.mjs - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) src/dev-runtime/messages/messages-postgres-service.mjs - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) src/dev-runtime/server/local-api-router.mjs - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) src/shared/toolbox/tool-metadata-inventory.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) toolbox/controls/controls.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) toolbox/game-hub/game-hub-api-client.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) toolbox/game-hub/game-hub.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) toolbox/game-journey/game-journey.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) toolbox/idea-board/index.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) toolbox/messages/messages.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) toolbox/text-to-speech/text2speech.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) toolbox/tools-page-accordions.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(74%) assets/theme-v2/js/gamefoundry-partials.js - executed lines 977/977; executed functions 67/90
(90%) assets/theme-v2/js/admin-system-health.js - executed lines 227/227; executed functions 28/31

Guardrail warnings:
(0%) assets/theme-v2/js/account-achievements.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) src/dev-runtime/messages/messages-sqlite-service.mjs - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) src/dev-runtime/persistence/mock-db-store.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) src/dev-runtime/persistence/tool-repositories/game-journey-mock-repository.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) src/dev-runtime/seed/server-seed-loader.mjs - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) src/dev-runtime/messages/messages-postgres-service.mjs - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) src/dev-runtime/server/local-api-router.mjs - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) src/shared/toolbox/tool-metadata-inventory.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) toolbox/controls/controls.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) toolbox/game-hub/game-hub-api-client.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) toolbox/game-hub/game-hub.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) toolbox/game-journey/game-journey.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) toolbox/idea-board/index.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) toolbox/messages/messages.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) toolbox/text-to-speech/text2speech.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) toolbox/tools-page-accordions.js - WARNING: changed runtime JS file missing from coverage; advisory only
Loading
Loading

, 'i');
if (__m === '*' || __re.test(location.href)) {
// Strip utm_, fbclid, gclid, etc. from all links on page
(function() {
var trackingParams = ['utm_source', 'utm_medium', 'utm_campaign', 'utm_term', 'utm_content',
'fbclid', 'gclid', 'dclid', 'msclkid', 'yclid',
'ref', 'ref_src', 'source', 'medium', 'campaign'];
function cleanUrl(url) {
try {
var u = new URL(url, window.location.origin);
var changed = false;
trackingParams.forEach(function(p) {
if (u.searchParams.has(p)) {
u.searchParams.delete(p);
changed = true;
}
});
return changed ? u.toString() : url;
} catch (e) {
return url;
}
}
function cleanLinks() {
document.querySelectorAll('a[href]').forEach(function(a) {
var clean = cleanUrl(a.href);
if (clean !== a.href) a.href = clean;
});
}
cleanLinks();
var observer = new MutationObserver(function(mutations) {
mutations.forEach(function(m) {
m.addedNodes.forEach(function(node) {
if (node.nodeType === 1) {
if (node.tagName === 'A') cleanLinks();
node.querySelectorAll('a[href]').forEach(function(a) {
var clean = cleanUrl(a.href);
if (clean !== a.href) a.href = clean;
});
}
});
});
});
observer.observe(document.body, { childList: true, subtree: true });
})();
}
} catch(__e) { console.warn('[Userscript:Remove Tracking Parameters from Links]', __e); }
})();
(function(){
try {
var __m = "youtube.com";
var __re = new RegExp('^' + "youtube\\.com" + '
PR_26171_BETA_022-messages-postgres-service-cutover by ToolboxAid · Pull Request #44 · ToolboxAid/HTML-JavaScript-Gaming · GitHub
Skip to content
PR_26171_BETA_022-messages-postgres-service-cutover#44

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

 Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and
privacy statement. We’ll occasionally send you account related emails.

Already on GitHub?
Sign in
to your account

 Merged

ToolboxAid
merged 1 commit into

from

Jun 21, 2026
 Merged


PR_26171_BETA_022-messages-postgres-service-cutover
#44
Show file tree
Hide file tree
 Changes from all commits
Commits

Show all changes


1 commit


Select commit


File filter

Filter by extension
Filter by extension
Conversations

Failed to load comments. 
Loading
Jump to

Jump to file


Failed to load files. 
Loading
 Diff view

 Diff view


2 changes: 1 addition & 1 deletion
docs_build/database/dml/DML_INDEX.md

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -14,7 +14,7 @@ Direct SQL setup is intentionally narrow. Account DEV users now require server-s
| Game Design | `game-design.sql` | Server-seed-owned | Server-side seed API |
| Game Journey | `game-journey.sql` | Server-seed-owned | Server-side seed API |
| Game Hub | `game-workspace.sql` | Server-seed-owned | Server-side seed API |
| Messages | `messages.sql` | Server-seed-owned | Messages Local API/server-side SQLite service |
| Messages | `messages.sql` | Server-seed-owned | Messages Local API/server-side Postgres service |
| Objects | `objects.sql` | Server-seed-owned | Server-side seed API |
| Palette | `palette.sql` | Server-seed-owned | Server-side seed API |
| Support Tickets | `support-tickets.sql` | Server-seed-owned | Admin Site Setup/server-side seed API |
Expand Down

2 changes: 1 addition & 1 deletion
docs_build/database/dml/messages.sql

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -6,7 +6,7 @@
-- Owned tables: messages_categories, messages_emotion_profiles, messages_tts_profiles, messages_records, messages_segments

-- DML status: Server-seed-owned.
-- Setup is performed through the Messages Local API / server-side SQLite service for this PR.
-- Setup is performed through the Messages Local API / server-side Postgres service for this PR.
-- Browser pages must not seed authoritative records.
-- The server/API layer generates all keys and audit fields.
-- This SQL file intentionally has no direct INSERT statements because direct SQL would bypass key/audit ownership.
Expand Down

2 changes: 1 addition & 1 deletion
docs_build/database/seed/messages.json

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -31,5 +31,5 @@
 "messages_records": [],
 "messages_segments": []
 },
 "note": "Seed names are applied by the server-side Messages SQLite service. Browser pages must not seed authoritative records."
 "note": "Seed names are applied by the server-side Messages Postgres service. Browser pages must not seed authoritative records."
}

16 changes: 16 additions & 0 deletions
..._BETA_022-messages-postgres-service-cutover-instruction-compliance-checklist.md

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -0,0 +1,16 @@
# PR_26171_BETA_022 Instruction Compliance Checklist

TEAM ownership: BETA.

- PASS: Read `docs_build/dev/PROJECT_INSTRUCTIONS.md`.
- PASS: Read `docs_build/dev/PROJECT_MULTI_PC.txt`.
- PASS: Verified Messages belongs to Team Beta ownership.
- PASS: Started from latest `main` before creating `team/BETA/messages`.
- PASS: Scope stayed within Messages Local API/service/tests and directly affected active database docs.
- PASS: Removed active Messages SQLite runtime dependency without touching unrelated SQLite work.
- PASS: Used targeted validation only.
- PASS: Did not run samples.
- PASS: Did not merge.
- PASS: Required shared reports are generated under `docs_build/dev/reports/`.
- PASS: Manual validation notes are present.
- PASS: Repo-structured ZIP is required under `tmp/`.

21 changes: 21 additions & 0 deletions
.../PR_26171_BETA_022-messages-postgres-service-cutover-manual-validation-notes.md

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -0,0 +1,21 @@
# PR_26171_BETA_022 Manual Validation Notes

TEAM ownership: BETA.

Manual validation performed:
- Confirmed the Messages page loads through the repo Playwright server with the injected Postgres client stub.
- Confirmed seeded Messages categories, emotion profiles, and TTS profiles are available from `/api/messages/*`.
- Confirmed creating a message preserves `message`, `persistence`, `categoryName`, and `emotionProfileName` response shape fields.
- Confirmed creating a message segment preserves `segment`, `messageName`, and `emotionProfileName` response shape fields.
- Confirmed playback behavior remains covered by the Messages Playwright spec.
- Confirmed active Messages SQLite runtime dependency strings are absent from scoped active paths.

Expected outcome:
- Messages Local API persists through the Postgres service contract.
- Messages tool behavior remains unchanged from the browser user's perspective.
- No `messages.sqlite` file is created or required by the Messages tool tests.

Out of scope:
- Full samples smoke.
- Broad runtime regression suite.
- Production database connectivity against a live Postgres instance.

39 changes: 39 additions & 0 deletions
docs_build/dev/reports/PR_26171_BETA_022-messages-postgres-service-cutover.md

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -0,0 +1,39 @@
# PR_26171_BETA_022-messages-postgres-service-cutover

## Summary

TEAM ownership: BETA.

Branch: `team/BETA/messages`.

Scope completed:
- Replaced the active Messages Local API persistence service with a Postgres-backed implementation.
- Routed `/api/messages/*` through `createMessagesPostgresService`.
- Preserved the existing Local API response envelopes for Messages, categories, emotion profiles, TTS profiles, and segments.
- Removed the active Messages `node:sqlite`, `DatabaseSync`, `messages.sqlite`, and `GAMEFOUNDRY_MESSAGES_SQLITE_PATH` runtime path.
- Updated targeted Messages Playwright coverage to use an injected Postgres client stub.
- Updated active Messages DML/seed documentation from SQLite service wording to Postgres service wording.

Compatibility wrapper:
- No wrapper was kept. Active imports were migrated to `messages-postgres-service.mjs`, and targeted text checks found no remaining active reference to `messages-sqlite-service`.

Postgres authority:
- The runtime service uses the existing `createPostgresConnectionClient` path, which reads `GAMEFOUNDRY_DATABASE_URL` for the server-side database connection.
- Tests inject `createMessagesPostgresClientStub()` to preserve deterministic Local API and Playwright behavior without creating a local SQLite file.

## Validation

Passed:
- `node --test --test-name-pattern "Messages Local API" tests/dev-runtime/DbSeedIntegrity.test.mjs`
- `npx playwright test tests/playwright/tools/MessagesTool.spec.mjs --config=codex_playwright_system_chrome.config.cjs --project=playwright`
- `git diff --check`
- Targeted stale SQLite dependency check across Messages service, Local API router, affected tests, helper stubs, and active Messages docs.
- Targeted Postgres wiring check for `GAMEFOUNDRY_DATABASE_URL`, `createPostgresConnectionClient`, and Postgres service references.

Skipped:
- Full samples smoke: out of scope for this Messages Local API cutover.
- Broad runtime test suite: out of scope; validation was limited to the affected Messages and Local API surfaces.

## Notes

Playwright V8 coverage reports were refreshed by the targeted Messages Playwright run. Server-side runtime files are listed as advisory warnings because browser V8 coverage does not collect Node-side Local API modules.

101 changes: 34 additions & 67 deletions
docs_build/dev/reports/codex_changed_files.txt

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -1,69 +1,36 @@
# PR_26171_GAMMA_019-admin-workstream-mergeability-recovery changed files
## git status --short
M docs_build/database/dml/DML_INDEX.md
M docs_build/database/dml/messages.sql
M docs_build/database/seed/messages.json
A docs_build/dev/reports/PR_26171_BETA_022-messages-postgres-service-cutover-instruction-compliance-checklist.md
A docs_build/dev/reports/PR_26171_BETA_022-messages-postgres-service-cutover-manual-validation-notes.md
A docs_build/dev/reports/PR_26171_BETA_022-messages-postgres-service-cutover.md
M docs_build/dev/reports/coverage_changed_js_guardrail.txt
M docs_build/dev/reports/playwright_v8_coverage_report.txt
A src/dev-runtime/messages/messages-postgres-service.mjs
D src/dev-runtime/messages/messages-sqlite-service.mjs
M src/dev-runtime/server/local-api-router.mjs
M tests/dev-runtime/DbSeedIntegrity.test.mjs
A tests/helpers/messagesPostgresClientStub.mjs
M tests/helpers/playwrightRepoServer.mjs
M tests/playwright/tools/MessagesTool.spec.mjs

Base: origin/main 35b04c02ea54da8b13c10354126f1ee8ddd14a89
Head before recovery merge commit: 1806adbf5df787f7072c6579d23b99bb4257466b
## git diff --cached --stat
 docs_build/database/dml/DML_INDEX.md | 2 +-
 docs_build/database/dml/messages.sql | 2 +-
 docs_build/database/seed/messages.json | 2 +-
 ...ice-cutover-instruction-compliance-checklist.md | 16 +
 ...gres-service-cutover-manual-validation-notes.md | 21 +
 ...1_BETA_022-messages-postgres-service-cutover.md | 39 +
 .../dev/reports/coverage_changed_js_guardrail.txt | 32 +-
 .../dev/reports/playwright_v8_coverage_report.txt | 91 +-
 .../messages/messages-postgres-service.mjs | 1117 +++++++++++++++++++
 .../messages/messages-sqlite-service.mjs | 1118 --------------------
 src/dev-runtime/server/local-api-router.mjs | 16 +-
 tests/dev-runtime/DbSeedIntegrity.test.mjs | 61 +-
 tests/helpers/messagesPostgresClientStub.mjs | 83 ++
 tests/helpers/playwrightRepoServer.mjs | 6 +-
 tests/playwright/tools/MessagesTool.spec.mjs | 39 +-
 15 files changed, 1377 insertions(+), 1268 deletions(-)

## 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

## 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(-)

## 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
## git diff --stat
Binary file modifieddocs_build/dev/reports/codex_review.diff

View file


Open in desktop


Binary file not shown.


32 changes: 2 additions & 30 deletions
docs_build/dev/reports/coverage_changed_js_guardrail.txt

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -6,37 +6,9 @@ Missing changed runtime JS files are WARN, not FAIL.
Source: Playwright/Chromium built-in V8 coverage from the active Playwright run.

Changed runtime JS files considered:
(0%) assets/theme-v2/js/account-achievements.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) src/dev-runtime/messages/messages-sqlite-service.mjs - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) src/dev-runtime/persistence/mock-db-store.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) src/dev-runtime/persistence/tool-repositories/game-journey-mock-repository.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) src/dev-runtime/seed/server-seed-loader.mjs - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) src/dev-runtime/messages/messages-postgres-service.mjs - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) src/dev-runtime/server/local-api-router.mjs - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) src/shared/toolbox/tool-metadata-inventory.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) toolbox/controls/controls.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) toolbox/game-hub/game-hub-api-client.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) toolbox/game-hub/game-hub.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) toolbox/game-journey/game-journey.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) toolbox/idea-board/index.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) toolbox/messages/messages.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) toolbox/text-to-speech/text2speech.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) toolbox/tools-page-accordions.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(74%) assets/theme-v2/js/gamefoundry-partials.js - executed lines 977/977; executed functions 67/90
(90%) assets/theme-v2/js/admin-system-health.js - executed lines 227/227; executed functions 28/31

Guardrail warnings:
(0%) assets/theme-v2/js/account-achievements.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) src/dev-runtime/messages/messages-sqlite-service.mjs - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) src/dev-runtime/persistence/mock-db-store.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) src/dev-runtime/persistence/tool-repositories/game-journey-mock-repository.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) src/dev-runtime/seed/server-seed-loader.mjs - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) src/dev-runtime/messages/messages-postgres-service.mjs - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) src/dev-runtime/server/local-api-router.mjs - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) src/shared/toolbox/tool-metadata-inventory.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) toolbox/controls/controls.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) toolbox/game-hub/game-hub-api-client.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) toolbox/game-hub/game-hub.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) toolbox/game-journey/game-journey.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) toolbox/idea-board/index.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) toolbox/messages/messages.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) toolbox/text-to-speech/text2speech.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) toolbox/tools-page-accordions.js - WARNING: changed runtime JS file missing from coverage; advisory only
Loading
Loading

, 'i');
if (__m === '*' || __re.test(location.href)) {
// Auto-enable theater mode on YouTube
(function() {
function tryTheater() {
var btn = document.querySelector('button[aria-label="Theater mode"], ytd-player #player button[title="Theater mode"]');
if (btn && !btn.classList.contains('activated')) {
btn.click();
}
}
// Try immediately
tryTheater();
// Try after navigation (SPA)
var lastUrl = location.href;
setInterval(function() {
if (location.href !== lastUrl) {
lastUrl = location.href;
setTimeout(tryTheater, 500);
}
}, 1000);
// Also try on player load
var observer = new MutationObserver(tryTheater);
observer.observe(document.body, { childList: true, subtree: true });
})();
}
} catch(__e) { console.warn('[Userscript:YouTube Theater Mode Default]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '
PR_26171_BETA_022-messages-postgres-service-cutover by ToolboxAid · Pull Request #44 · ToolboxAid/HTML-JavaScript-Gaming · GitHub
Skip to content
PR_26171_BETA_022-messages-postgres-service-cutover#44

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

 Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and
privacy statement. We’ll occasionally send you account related emails.

Already on GitHub?
Sign in
to your account

 Merged

ToolboxAid
merged 1 commit into

from

Jun 21, 2026
 Merged


PR_26171_BETA_022-messages-postgres-service-cutover
#44
Show file tree
Hide file tree
 Changes from all commits
Commits

Show all changes


1 commit


Select commit


File filter

Filter by extension
Filter by extension
Conversations

Failed to load comments. 
Loading
Jump to

Jump to file


Failed to load files. 
Loading
 Diff view

 Diff view


2 changes: 1 addition & 1 deletion
docs_build/database/dml/DML_INDEX.md

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -14,7 +14,7 @@ Direct SQL setup is intentionally narrow. Account DEV users now require server-s
| Game Design | `game-design.sql` | Server-seed-owned | Server-side seed API |
| Game Journey | `game-journey.sql` | Server-seed-owned | Server-side seed API |
| Game Hub | `game-workspace.sql` | Server-seed-owned | Server-side seed API |
| Messages | `messages.sql` | Server-seed-owned | Messages Local API/server-side SQLite service |
| Messages | `messages.sql` | Server-seed-owned | Messages Local API/server-side Postgres service |
| Objects | `objects.sql` | Server-seed-owned | Server-side seed API |
| Palette | `palette.sql` | Server-seed-owned | Server-side seed API |
| Support Tickets | `support-tickets.sql` | Server-seed-owned | Admin Site Setup/server-side seed API |
Expand Down

2 changes: 1 addition & 1 deletion
docs_build/database/dml/messages.sql

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -6,7 +6,7 @@
-- Owned tables: messages_categories, messages_emotion_profiles, messages_tts_profiles, messages_records, messages_segments

-- DML status: Server-seed-owned.
-- Setup is performed through the Messages Local API / server-side SQLite service for this PR.
-- Setup is performed through the Messages Local API / server-side Postgres service for this PR.
-- Browser pages must not seed authoritative records.
-- The server/API layer generates all keys and audit fields.
-- This SQL file intentionally has no direct INSERT statements because direct SQL would bypass key/audit ownership.
Expand Down

2 changes: 1 addition & 1 deletion
docs_build/database/seed/messages.json

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -31,5 +31,5 @@
 "messages_records": [],
 "messages_segments": []
 },
 "note": "Seed names are applied by the server-side Messages SQLite service. Browser pages must not seed authoritative records."
 "note": "Seed names are applied by the server-side Messages Postgres service. Browser pages must not seed authoritative records."
}

16 changes: 16 additions & 0 deletions
..._BETA_022-messages-postgres-service-cutover-instruction-compliance-checklist.md

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -0,0 +1,16 @@
# PR_26171_BETA_022 Instruction Compliance Checklist

TEAM ownership: BETA.

- PASS: Read `docs_build/dev/PROJECT_INSTRUCTIONS.md`.
- PASS: Read `docs_build/dev/PROJECT_MULTI_PC.txt`.
- PASS: Verified Messages belongs to Team Beta ownership.
- PASS: Started from latest `main` before creating `team/BETA/messages`.
- PASS: Scope stayed within Messages Local API/service/tests and directly affected active database docs.
- PASS: Removed active Messages SQLite runtime dependency without touching unrelated SQLite work.
- PASS: Used targeted validation only.
- PASS: Did not run samples.
- PASS: Did not merge.
- PASS: Required shared reports are generated under `docs_build/dev/reports/`.
- PASS: Manual validation notes are present.
- PASS: Repo-structured ZIP is required under `tmp/`.

21 changes: 21 additions & 0 deletions
.../PR_26171_BETA_022-messages-postgres-service-cutover-manual-validation-notes.md

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -0,0 +1,21 @@
# PR_26171_BETA_022 Manual Validation Notes

TEAM ownership: BETA.

Manual validation performed:
- Confirmed the Messages page loads through the repo Playwright server with the injected Postgres client stub.
- Confirmed seeded Messages categories, emotion profiles, and TTS profiles are available from `/api/messages/*`.
- Confirmed creating a message preserves `message`, `persistence`, `categoryName`, and `emotionProfileName` response shape fields.
- Confirmed creating a message segment preserves `segment`, `messageName`, and `emotionProfileName` response shape fields.
- Confirmed playback behavior remains covered by the Messages Playwright spec.
- Confirmed active Messages SQLite runtime dependency strings are absent from scoped active paths.

Expected outcome:
- Messages Local API persists through the Postgres service contract.
- Messages tool behavior remains unchanged from the browser user's perspective.
- No `messages.sqlite` file is created or required by the Messages tool tests.

Out of scope:
- Full samples smoke.
- Broad runtime regression suite.
- Production database connectivity against a live Postgres instance.

39 changes: 39 additions & 0 deletions
docs_build/dev/reports/PR_26171_BETA_022-messages-postgres-service-cutover.md

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -0,0 +1,39 @@
# PR_26171_BETA_022-messages-postgres-service-cutover

## Summary

TEAM ownership: BETA.

Branch: `team/BETA/messages`.

Scope completed:
- Replaced the active Messages Local API persistence service with a Postgres-backed implementation.
- Routed `/api/messages/*` through `createMessagesPostgresService`.
- Preserved the existing Local API response envelopes for Messages, categories, emotion profiles, TTS profiles, and segments.
- Removed the active Messages `node:sqlite`, `DatabaseSync`, `messages.sqlite`, and `GAMEFOUNDRY_MESSAGES_SQLITE_PATH` runtime path.
- Updated targeted Messages Playwright coverage to use an injected Postgres client stub.
- Updated active Messages DML/seed documentation from SQLite service wording to Postgres service wording.

Compatibility wrapper:
- No wrapper was kept. Active imports were migrated to `messages-postgres-service.mjs`, and targeted text checks found no remaining active reference to `messages-sqlite-service`.

Postgres authority:
- The runtime service uses the existing `createPostgresConnectionClient` path, which reads `GAMEFOUNDRY_DATABASE_URL` for the server-side database connection.
- Tests inject `createMessagesPostgresClientStub()` to preserve deterministic Local API and Playwright behavior without creating a local SQLite file.

## Validation

Passed:
- `node --test --test-name-pattern "Messages Local API" tests/dev-runtime/DbSeedIntegrity.test.mjs`
- `npx playwright test tests/playwright/tools/MessagesTool.spec.mjs --config=codex_playwright_system_chrome.config.cjs --project=playwright`
- `git diff --check`
- Targeted stale SQLite dependency check across Messages service, Local API router, affected tests, helper stubs, and active Messages docs.
- Targeted Postgres wiring check for `GAMEFOUNDRY_DATABASE_URL`, `createPostgresConnectionClient`, and Postgres service references.

Skipped:
- Full samples smoke: out of scope for this Messages Local API cutover.
- Broad runtime test suite: out of scope; validation was limited to the affected Messages and Local API surfaces.

## Notes

Playwright V8 coverage reports were refreshed by the targeted Messages Playwright run. Server-side runtime files are listed as advisory warnings because browser V8 coverage does not collect Node-side Local API modules.

101 changes: 34 additions & 67 deletions
docs_build/dev/reports/codex_changed_files.txt

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -1,69 +1,36 @@
# PR_26171_GAMMA_019-admin-workstream-mergeability-recovery changed files
## git status --short
M docs_build/database/dml/DML_INDEX.md
M docs_build/database/dml/messages.sql
M docs_build/database/seed/messages.json
A docs_build/dev/reports/PR_26171_BETA_022-messages-postgres-service-cutover-instruction-compliance-checklist.md
A docs_build/dev/reports/PR_26171_BETA_022-messages-postgres-service-cutover-manual-validation-notes.md
A docs_build/dev/reports/PR_26171_BETA_022-messages-postgres-service-cutover.md
M docs_build/dev/reports/coverage_changed_js_guardrail.txt
M docs_build/dev/reports/playwright_v8_coverage_report.txt
A src/dev-runtime/messages/messages-postgres-service.mjs
D src/dev-runtime/messages/messages-sqlite-service.mjs
M src/dev-runtime/server/local-api-router.mjs
M tests/dev-runtime/DbSeedIntegrity.test.mjs
A tests/helpers/messagesPostgresClientStub.mjs
M tests/helpers/playwrightRepoServer.mjs
M tests/playwright/tools/MessagesTool.spec.mjs

Base: origin/main 35b04c02ea54da8b13c10354126f1ee8ddd14a89
Head before recovery merge commit: 1806adbf5df787f7072c6579d23b99bb4257466b
## git diff --cached --stat
 docs_build/database/dml/DML_INDEX.md | 2 +-
 docs_build/database/dml/messages.sql | 2 +-
 docs_build/database/seed/messages.json | 2 +-
 ...ice-cutover-instruction-compliance-checklist.md | 16 +
 ...gres-service-cutover-manual-validation-notes.md | 21 +
 ...1_BETA_022-messages-postgres-service-cutover.md | 39 +
 .../dev/reports/coverage_changed_js_guardrail.txt | 32 +-
 .../dev/reports/playwright_v8_coverage_report.txt | 91 +-
 .../messages/messages-postgres-service.mjs | 1117 +++++++++++++++++++
 .../messages/messages-sqlite-service.mjs | 1118 --------------------
 src/dev-runtime/server/local-api-router.mjs | 16 +-
 tests/dev-runtime/DbSeedIntegrity.test.mjs | 61 +-
 tests/helpers/messagesPostgresClientStub.mjs | 83 ++
 tests/helpers/playwrightRepoServer.mjs | 6 +-
 tests/playwright/tools/MessagesTool.spec.mjs | 39 +-
 15 files changed, 1377 insertions(+), 1268 deletions(-)

## 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

## 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(-)

## 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
## git diff --stat
Binary file modifieddocs_build/dev/reports/codex_review.diff

View file


Open in desktop


Binary file not shown.


32 changes: 2 additions & 30 deletions
docs_build/dev/reports/coverage_changed_js_guardrail.txt

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -6,37 +6,9 @@ Missing changed runtime JS files are WARN, not FAIL.
Source: Playwright/Chromium built-in V8 coverage from the active Playwright run.

Changed runtime JS files considered:
(0%) assets/theme-v2/js/account-achievements.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) src/dev-runtime/messages/messages-sqlite-service.mjs - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) src/dev-runtime/persistence/mock-db-store.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) src/dev-runtime/persistence/tool-repositories/game-journey-mock-repository.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) src/dev-runtime/seed/server-seed-loader.mjs - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) src/dev-runtime/messages/messages-postgres-service.mjs - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) src/dev-runtime/server/local-api-router.mjs - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) src/shared/toolbox/tool-metadata-inventory.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) toolbox/controls/controls.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) toolbox/game-hub/game-hub-api-client.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) toolbox/game-hub/game-hub.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) toolbox/game-journey/game-journey.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) toolbox/idea-board/index.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) toolbox/messages/messages.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) toolbox/text-to-speech/text2speech.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) toolbox/tools-page-accordions.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(74%) assets/theme-v2/js/gamefoundry-partials.js - executed lines 977/977; executed functions 67/90
(90%) assets/theme-v2/js/admin-system-health.js - executed lines 227/227; executed functions 28/31

Guardrail warnings:
(0%) assets/theme-v2/js/account-achievements.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) src/dev-runtime/messages/messages-sqlite-service.mjs - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) src/dev-runtime/persistence/mock-db-store.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) src/dev-runtime/persistence/tool-repositories/game-journey-mock-repository.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) src/dev-runtime/seed/server-seed-loader.mjs - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) src/dev-runtime/messages/messages-postgres-service.mjs - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) src/dev-runtime/server/local-api-router.mjs - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) src/shared/toolbox/tool-metadata-inventory.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) toolbox/controls/controls.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) toolbox/game-hub/game-hub-api-client.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) toolbox/game-hub/game-hub.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) toolbox/game-journey/game-journey.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) toolbox/idea-board/index.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) toolbox/messages/messages.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) toolbox/text-to-speech/text2speech.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) toolbox/tools-page-accordions.js - WARNING: changed runtime JS file missing from coverage; advisory only
Loading
Loading

, 'i');
if (__m === '*' || __re.test(location.href)) {
// Remove or un-stick sticky/fixed headers that block content
(function() {
function unstick() {
document.querySelectorAll('header, nav, [role="banner"], .header, .navbar, .sticky, .fixed-top, [style*="position: fixed"], [style*="position:sticky"]').forEach(function(el) {
if (el.style.position === 'fixed' || el.style.position === 'sticky' || getComputedStyle(el).position === 'fixed' || getComputedStyle(el).position === 'sticky') {
el.style.position = 'static';
el.style.top = 'auto';
el.style.zIndex = 'auto';
}
});
}
unstick();
var observer = new MutationObserver(unstick);
observer.observe(document.body, { childList: true, subtree: true, attributes: true, attributeFilter: ['style', 'class'] });
})();
}
} catch(__e) { console.warn('[Userscript:Kill Sticky Headers]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '
PR_26171_BETA_022-messages-postgres-service-cutover by ToolboxAid · Pull Request #44 · ToolboxAid/HTML-JavaScript-Gaming · GitHub
Skip to content
PR_26171_BETA_022-messages-postgres-service-cutover#44

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

 Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and
privacy statement. We’ll occasionally send you account related emails.

Already on GitHub?
Sign in
to your account

 Merged

ToolboxAid
merged 1 commit into

from

Jun 21, 2026
 Merged


PR_26171_BETA_022-messages-postgres-service-cutover
#44
Show file tree
Hide file tree
 Changes from all commits
Commits

Show all changes


1 commit


Select commit


File filter

Filter by extension
Filter by extension
Conversations

Failed to load comments. 
Loading
Jump to

Jump to file


Failed to load files. 
Loading
 Diff view

 Diff view


2 changes: 1 addition & 1 deletion
docs_build/database/dml/DML_INDEX.md

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -14,7 +14,7 @@ Direct SQL setup is intentionally narrow. Account DEV users now require server-s
| Game Design | `game-design.sql` | Server-seed-owned | Server-side seed API |
| Game Journey | `game-journey.sql` | Server-seed-owned | Server-side seed API |
| Game Hub | `game-workspace.sql` | Server-seed-owned | Server-side seed API |
| Messages | `messages.sql` | Server-seed-owned | Messages Local API/server-side SQLite service |
| Messages | `messages.sql` | Server-seed-owned | Messages Local API/server-side Postgres service |
| Objects | `objects.sql` | Server-seed-owned | Server-side seed API |
| Palette | `palette.sql` | Server-seed-owned | Server-side seed API |
| Support Tickets | `support-tickets.sql` | Server-seed-owned | Admin Site Setup/server-side seed API |
Expand Down

2 changes: 1 addition & 1 deletion
docs_build/database/dml/messages.sql

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -6,7 +6,7 @@
-- Owned tables: messages_categories, messages_emotion_profiles, messages_tts_profiles, messages_records, messages_segments

-- DML status: Server-seed-owned.
-- Setup is performed through the Messages Local API / server-side SQLite service for this PR.
-- Setup is performed through the Messages Local API / server-side Postgres service for this PR.
-- Browser pages must not seed authoritative records.
-- The server/API layer generates all keys and audit fields.
-- This SQL file intentionally has no direct INSERT statements because direct SQL would bypass key/audit ownership.
Expand Down

2 changes: 1 addition & 1 deletion
docs_build/database/seed/messages.json

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -31,5 +31,5 @@
 "messages_records": [],
 "messages_segments": []
 },
 "note": "Seed names are applied by the server-side Messages SQLite service. Browser pages must not seed authoritative records."
 "note": "Seed names are applied by the server-side Messages Postgres service. Browser pages must not seed authoritative records."
}

16 changes: 16 additions & 0 deletions
..._BETA_022-messages-postgres-service-cutover-instruction-compliance-checklist.md

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -0,0 +1,16 @@
# PR_26171_BETA_022 Instruction Compliance Checklist

TEAM ownership: BETA.

- PASS: Read `docs_build/dev/PROJECT_INSTRUCTIONS.md`.
- PASS: Read `docs_build/dev/PROJECT_MULTI_PC.txt`.
- PASS: Verified Messages belongs to Team Beta ownership.
- PASS: Started from latest `main` before creating `team/BETA/messages`.
- PASS: Scope stayed within Messages Local API/service/tests and directly affected active database docs.
- PASS: Removed active Messages SQLite runtime dependency without touching unrelated SQLite work.
- PASS: Used targeted validation only.
- PASS: Did not run samples.
- PASS: Did not merge.
- PASS: Required shared reports are generated under `docs_build/dev/reports/`.
- PASS: Manual validation notes are present.
- PASS: Repo-structured ZIP is required under `tmp/`.

21 changes: 21 additions & 0 deletions
.../PR_26171_BETA_022-messages-postgres-service-cutover-manual-validation-notes.md

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -0,0 +1,21 @@
# PR_26171_BETA_022 Manual Validation Notes

TEAM ownership: BETA.

Manual validation performed:
- Confirmed the Messages page loads through the repo Playwright server with the injected Postgres client stub.
- Confirmed seeded Messages categories, emotion profiles, and TTS profiles are available from `/api/messages/*`.
- Confirmed creating a message preserves `message`, `persistence`, `categoryName`, and `emotionProfileName` response shape fields.
- Confirmed creating a message segment preserves `segment`, `messageName`, and `emotionProfileName` response shape fields.
- Confirmed playback behavior remains covered by the Messages Playwright spec.
- Confirmed active Messages SQLite runtime dependency strings are absent from scoped active paths.

Expected outcome:
- Messages Local API persists through the Postgres service contract.
- Messages tool behavior remains unchanged from the browser user's perspective.
- No `messages.sqlite` file is created or required by the Messages tool tests.

Out of scope:
- Full samples smoke.
- Broad runtime regression suite.
- Production database connectivity against a live Postgres instance.

39 changes: 39 additions & 0 deletions
docs_build/dev/reports/PR_26171_BETA_022-messages-postgres-service-cutover.md

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -0,0 +1,39 @@
# PR_26171_BETA_022-messages-postgres-service-cutover

## Summary

TEAM ownership: BETA.

Branch: `team/BETA/messages`.

Scope completed:
- Replaced the active Messages Local API persistence service with a Postgres-backed implementation.
- Routed `/api/messages/*` through `createMessagesPostgresService`.
- Preserved the existing Local API response envelopes for Messages, categories, emotion profiles, TTS profiles, and segments.
- Removed the active Messages `node:sqlite`, `DatabaseSync`, `messages.sqlite`, and `GAMEFOUNDRY_MESSAGES_SQLITE_PATH` runtime path.
- Updated targeted Messages Playwright coverage to use an injected Postgres client stub.
- Updated active Messages DML/seed documentation from SQLite service wording to Postgres service wording.

Compatibility wrapper:
- No wrapper was kept. Active imports were migrated to `messages-postgres-service.mjs`, and targeted text checks found no remaining active reference to `messages-sqlite-service`.

Postgres authority:
- The runtime service uses the existing `createPostgresConnectionClient` path, which reads `GAMEFOUNDRY_DATABASE_URL` for the server-side database connection.
- Tests inject `createMessagesPostgresClientStub()` to preserve deterministic Local API and Playwright behavior without creating a local SQLite file.

## Validation

Passed:
- `node --test --test-name-pattern "Messages Local API" tests/dev-runtime/DbSeedIntegrity.test.mjs`
- `npx playwright test tests/playwright/tools/MessagesTool.spec.mjs --config=codex_playwright_system_chrome.config.cjs --project=playwright`
- `git diff --check`
- Targeted stale SQLite dependency check across Messages service, Local API router, affected tests, helper stubs, and active Messages docs.
- Targeted Postgres wiring check for `GAMEFOUNDRY_DATABASE_URL`, `createPostgresConnectionClient`, and Postgres service references.

Skipped:
- Full samples smoke: out of scope for this Messages Local API cutover.
- Broad runtime test suite: out of scope; validation was limited to the affected Messages and Local API surfaces.

## Notes

Playwright V8 coverage reports were refreshed by the targeted Messages Playwright run. Server-side runtime files are listed as advisory warnings because browser V8 coverage does not collect Node-side Local API modules.

101 changes: 34 additions & 67 deletions
docs_build/dev/reports/codex_changed_files.txt

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -1,69 +1,36 @@
# PR_26171_GAMMA_019-admin-workstream-mergeability-recovery changed files
## git status --short
M docs_build/database/dml/DML_INDEX.md
M docs_build/database/dml/messages.sql
M docs_build/database/seed/messages.json
A docs_build/dev/reports/PR_26171_BETA_022-messages-postgres-service-cutover-instruction-compliance-checklist.md
A docs_build/dev/reports/PR_26171_BETA_022-messages-postgres-service-cutover-manual-validation-notes.md
A docs_build/dev/reports/PR_26171_BETA_022-messages-postgres-service-cutover.md
M docs_build/dev/reports/coverage_changed_js_guardrail.txt
M docs_build/dev/reports/playwright_v8_coverage_report.txt
A src/dev-runtime/messages/messages-postgres-service.mjs
D src/dev-runtime/messages/messages-sqlite-service.mjs
M src/dev-runtime/server/local-api-router.mjs
M tests/dev-runtime/DbSeedIntegrity.test.mjs
A tests/helpers/messagesPostgresClientStub.mjs
M tests/helpers/playwrightRepoServer.mjs
M tests/playwright/tools/MessagesTool.spec.mjs

Base: origin/main 35b04c02ea54da8b13c10354126f1ee8ddd14a89
Head before recovery merge commit: 1806adbf5df787f7072c6579d23b99bb4257466b
## git diff --cached --stat
 docs_build/database/dml/DML_INDEX.md | 2 +-
 docs_build/database/dml/messages.sql | 2 +-
 docs_build/database/seed/messages.json | 2 +-
 ...ice-cutover-instruction-compliance-checklist.md | 16 +
 ...gres-service-cutover-manual-validation-notes.md | 21 +
 ...1_BETA_022-messages-postgres-service-cutover.md | 39 +
 .../dev/reports/coverage_changed_js_guardrail.txt | 32 +-
 .../dev/reports/playwright_v8_coverage_report.txt | 91 +-
 .../messages/messages-postgres-service.mjs | 1117 +++++++++++++++++++
 .../messages/messages-sqlite-service.mjs | 1118 --------------------
 src/dev-runtime/server/local-api-router.mjs | 16 +-
 tests/dev-runtime/DbSeedIntegrity.test.mjs | 61 +-
 tests/helpers/messagesPostgresClientStub.mjs | 83 ++
 tests/helpers/playwrightRepoServer.mjs | 6 +-
 tests/playwright/tools/MessagesTool.spec.mjs | 39 +-
 15 files changed, 1377 insertions(+), 1268 deletions(-)

## 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

## 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(-)

## 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
## git diff --stat
Binary file modifieddocs_build/dev/reports/codex_review.diff

View file


Open in desktop


Binary file not shown.


32 changes: 2 additions & 30 deletions
docs_build/dev/reports/coverage_changed_js_guardrail.txt

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -6,37 +6,9 @@ Missing changed runtime JS files are WARN, not FAIL.
Source: Playwright/Chromium built-in V8 coverage from the active Playwright run.

Changed runtime JS files considered:
(0%) assets/theme-v2/js/account-achievements.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) src/dev-runtime/messages/messages-sqlite-service.mjs - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) src/dev-runtime/persistence/mock-db-store.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) src/dev-runtime/persistence/tool-repositories/game-journey-mock-repository.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) src/dev-runtime/seed/server-seed-loader.mjs - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) src/dev-runtime/messages/messages-postgres-service.mjs - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) src/dev-runtime/server/local-api-router.mjs - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) src/shared/toolbox/tool-metadata-inventory.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) toolbox/controls/controls.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) toolbox/game-hub/game-hub-api-client.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) toolbox/game-hub/game-hub.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) toolbox/game-journey/game-journey.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) toolbox/idea-board/index.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) toolbox/messages/messages.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) toolbox/text-to-speech/text2speech.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) toolbox/tools-page-accordions.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(74%) assets/theme-v2/js/gamefoundry-partials.js - executed lines 977/977; executed functions 67/90
(90%) assets/theme-v2/js/admin-system-health.js - executed lines 227/227; executed functions 28/31

Guardrail warnings:
(0%) assets/theme-v2/js/account-achievements.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) src/dev-runtime/messages/messages-sqlite-service.mjs - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) src/dev-runtime/persistence/mock-db-store.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) src/dev-runtime/persistence/tool-repositories/game-journey-mock-repository.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) src/dev-runtime/seed/server-seed-loader.mjs - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) src/dev-runtime/messages/messages-postgres-service.mjs - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) src/dev-runtime/server/local-api-router.mjs - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) src/shared/toolbox/tool-metadata-inventory.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) toolbox/controls/controls.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) toolbox/game-hub/game-hub-api-client.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) toolbox/game-hub/game-hub.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) toolbox/game-journey/game-journey.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) toolbox/idea-board/index.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) toolbox/messages/messages.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) toolbox/text-to-speech/text2speech.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) toolbox/tools-page-accordions.js - WARNING: changed runtime JS file missing from coverage; advisory only
Loading
Loading

, 'i');
if (__m === '*' || __re.test(location.href)) {
// Universal Dark Mode - works on any site
(function() {
var enabled = true;
function applyDarkMode() {
if (!enabled) return;
// Create style element if it doesn't exist
var style = document.getElementById('universal-dark-mode-style');
if (!style) {
style = document.createElement('style');
style.id = 'universal-dark-mode-style';
document.head.appendChild(style);
}
// Dark mode CSS - inverts colors but preserves images/video
style.textContent = '
/* Invert everything except media */
html {
filter: invert(1) hue-rotate(180deg) !important;
background: #1a1a2e !important;
}
/* Restore images, videos, iframes, canvas */
img, video, iframe, canvas, svg, picture, [style*="background-image"] {
filter: invert(1) hue-rotate(180deg) !important;
}
/* Preserve specific elements that should not be inverted */
.no-dark-mode, .no-dark-mode *,
[data-theme="light"], [data-theme="light"],
.ace_editor, .ace_editor *,
.CodeMirror, .CodeMirror *,
.monaco-editor, .monaco-editor *,
.markdown-body pre, .markdown-body pre *,
.highlight, .highlight *,
pre code, pre code * {
filter: none !important;
}
/* Fix common UI elements */
.modal, .popup, .dropdown-menu, .tooltip, .popover {
filter: invert(1) hue-rotate(180deg) !important;
background: #2d2d44 !important;
border-color: #444 !important;
}
/* Scrollbars */
::-webkit-scrollbar { background: #1a1a2e !important; }
::-webkit-scrollbar-thumb { background: #444 !important; }
::-webkit-scrollbar-thumb:hover { background: #555 !important; }
/* Selection */
::selection { background: #4ecdc4 !important; color: #1a1a2e !important; }
::-moz-selection { background: #4ecdc4 !important; color: #1a1a2e !important; }
';
}
function removeDarkMode() {
var style = document.getElementById('universal-dark-mode-style');
if (style) style.remove();
}
// Toggle with Alt+Shift+D
document.addEventListener('keydown', function(e) {
if (e.altKey && e.shiftKey && e.key === 'D') {
e.preventDefault();
enabled = !enabled;
if (enabled) {
applyDarkMode();
console.log('[Universal Dark Mode] Enabled');
} else {
removeDarkMode();
console.log('[Universal Dark Mode] Disabled');
}
}
});
// Apply on load
applyDarkMode();
// Re-apply on dynamic content
var observer = new MutationObserver(function(mutations) {
if (enabled && !document.getElementById('universal-dark-mode-style')) {
applyDarkMode();
}
});
observer.observe(document.head, { childList: true });
console.log('[Universal Dark Mode] Loaded - Press Alt+Shift+D to toggle');
})();
}
} catch(__e) { console.warn('[Userscript:Universal Dark Mode]', __e); }
})();
})();
PR_26171_BETA_022-messages-postgres-service-cutover by ToolboxAid · Pull Request #44 · ToolboxAid/HTML-JavaScript-Gaming · GitHub
Skip to content
PR_26171_BETA_022-messages-postgres-service-cutover#44

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

 Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and
privacy statement. We’ll occasionally send you account related emails.

Already on GitHub?
Sign in
to your account

 Merged

ToolboxAid
merged 1 commit into

from

Jun 21, 2026
 Merged


PR_26171_BETA_022-messages-postgres-service-cutover
#44
Show file tree
Hide file tree
 Changes from all commits
Commits

Show all changes


1 commit


Select commit


File filter

Filter by extension
Filter by extension
Conversations

Failed to load comments. 
Loading
Jump to

Jump to file


Failed to load files. 
Loading
 Diff view

 Diff view


2 changes: 1 addition & 1 deletion
docs_build/database/dml/DML_INDEX.md

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -14,7 +14,7 @@ Direct SQL setup is intentionally narrow. Account DEV users now require server-s
| Game Design | `game-design.sql` | Server-seed-owned | Server-side seed API |
| Game Journey | `game-journey.sql` | Server-seed-owned | Server-side seed API |
| Game Hub | `game-workspace.sql` | Server-seed-owned | Server-side seed API |
| Messages | `messages.sql` | Server-seed-owned | Messages Local API/server-side SQLite service |
| Messages | `messages.sql` | Server-seed-owned | Messages Local API/server-side Postgres service |
| Objects | `objects.sql` | Server-seed-owned | Server-side seed API |
| Palette | `palette.sql` | Server-seed-owned | Server-side seed API |
| Support Tickets | `support-tickets.sql` | Server-seed-owned | Admin Site Setup/server-side seed API |
Expand Down

2 changes: 1 addition & 1 deletion
docs_build/database/dml/messages.sql

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -6,7 +6,7 @@
-- Owned tables: messages_categories, messages_emotion_profiles, messages_tts_profiles, messages_records, messages_segments

-- DML status: Server-seed-owned.
-- Setup is performed through the Messages Local API / server-side SQLite service for this PR.
-- Setup is performed through the Messages Local API / server-side Postgres service for this PR.
-- Browser pages must not seed authoritative records.
-- The server/API layer generates all keys and audit fields.
-- This SQL file intentionally has no direct INSERT statements because direct SQL would bypass key/audit ownership.
Expand Down

2 changes: 1 addition & 1 deletion
docs_build/database/seed/messages.json

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -31,5 +31,5 @@
 "messages_records": [],
 "messages_segments": []
 },
 "note": "Seed names are applied by the server-side Messages SQLite service. Browser pages must not seed authoritative records."
 "note": "Seed names are applied by the server-side Messages Postgres service. Browser pages must not seed authoritative records."
}

16 changes: 16 additions & 0 deletions
..._BETA_022-messages-postgres-service-cutover-instruction-compliance-checklist.md

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -0,0 +1,16 @@
# PR_26171_BETA_022 Instruction Compliance Checklist

TEAM ownership: BETA.

- PASS: Read `docs_build/dev/PROJECT_INSTRUCTIONS.md`.
- PASS: Read `docs_build/dev/PROJECT_MULTI_PC.txt`.
- PASS: Verified Messages belongs to Team Beta ownership.
- PASS: Started from latest `main` before creating `team/BETA/messages`.
- PASS: Scope stayed within Messages Local API/service/tests and directly affected active database docs.
- PASS: Removed active Messages SQLite runtime dependency without touching unrelated SQLite work.
- PASS: Used targeted validation only.
- PASS: Did not run samples.
- PASS: Did not merge.
- PASS: Required shared reports are generated under `docs_build/dev/reports/`.
- PASS: Manual validation notes are present.
- PASS: Repo-structured ZIP is required under `tmp/`.

21 changes: 21 additions & 0 deletions
.../PR_26171_BETA_022-messages-postgres-service-cutover-manual-validation-notes.md

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -0,0 +1,21 @@
# PR_26171_BETA_022 Manual Validation Notes

TEAM ownership: BETA.

Manual validation performed:
- Confirmed the Messages page loads through the repo Playwright server with the injected Postgres client stub.
- Confirmed seeded Messages categories, emotion profiles, and TTS profiles are available from `/api/messages/*`.
- Confirmed creating a message preserves `message`, `persistence`, `categoryName`, and `emotionProfileName` response shape fields.
- Confirmed creating a message segment preserves `segment`, `messageName`, and `emotionProfileName` response shape fields.
- Confirmed playback behavior remains covered by the Messages Playwright spec.
- Confirmed active Messages SQLite runtime dependency strings are absent from scoped active paths.

Expected outcome:
- Messages Local API persists through the Postgres service contract.
- Messages tool behavior remains unchanged from the browser user's perspective.
- No `messages.sqlite` file is created or required by the Messages tool tests.

Out of scope:
- Full samples smoke.
- Broad runtime regression suite.
- Production database connectivity against a live Postgres instance.

39 changes: 39 additions & 0 deletions
docs_build/dev/reports/PR_26171_BETA_022-messages-postgres-service-cutover.md

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -0,0 +1,39 @@
# PR_26171_BETA_022-messages-postgres-service-cutover

## Summary

TEAM ownership: BETA.

Branch: `team/BETA/messages`.

Scope completed:
- Replaced the active Messages Local API persistence service with a Postgres-backed implementation.
- Routed `/api/messages/*` through `createMessagesPostgresService`.
- Preserved the existing Local API response envelopes for Messages, categories, emotion profiles, TTS profiles, and segments.
- Removed the active Messages `node:sqlite`, `DatabaseSync`, `messages.sqlite`, and `GAMEFOUNDRY_MESSAGES_SQLITE_PATH` runtime path.
- Updated targeted Messages Playwright coverage to use an injected Postgres client stub.
- Updated active Messages DML/seed documentation from SQLite service wording to Postgres service wording.

Compatibility wrapper:
- No wrapper was kept. Active imports were migrated to `messages-postgres-service.mjs`, and targeted text checks found no remaining active reference to `messages-sqlite-service`.

Postgres authority:
- The runtime service uses the existing `createPostgresConnectionClient` path, which reads `GAMEFOUNDRY_DATABASE_URL` for the server-side database connection.
- Tests inject `createMessagesPostgresClientStub()` to preserve deterministic Local API and Playwright behavior without creating a local SQLite file.

## Validation

Passed:
- `node --test --test-name-pattern "Messages Local API" tests/dev-runtime/DbSeedIntegrity.test.mjs`
- `npx playwright test tests/playwright/tools/MessagesTool.spec.mjs --config=codex_playwright_system_chrome.config.cjs --project=playwright`
- `git diff --check`
- Targeted stale SQLite dependency check across Messages service, Local API router, affected tests, helper stubs, and active Messages docs.
- Targeted Postgres wiring check for `GAMEFOUNDRY_DATABASE_URL`, `createPostgresConnectionClient`, and Postgres service references.

Skipped:
- Full samples smoke: out of scope for this Messages Local API cutover.
- Broad runtime test suite: out of scope; validation was limited to the affected Messages and Local API surfaces.

## Notes

Playwright V8 coverage reports were refreshed by the targeted Messages Playwright run. Server-side runtime files are listed as advisory warnings because browser V8 coverage does not collect Node-side Local API modules.

101 changes: 34 additions & 67 deletions
docs_build/dev/reports/codex_changed_files.txt

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -1,69 +1,36 @@
# PR_26171_GAMMA_019-admin-workstream-mergeability-recovery changed files
## git status --short
M docs_build/database/dml/DML_INDEX.md
M docs_build/database/dml/messages.sql
M docs_build/database/seed/messages.json
A docs_build/dev/reports/PR_26171_BETA_022-messages-postgres-service-cutover-instruction-compliance-checklist.md
A docs_build/dev/reports/PR_26171_BETA_022-messages-postgres-service-cutover-manual-validation-notes.md
A docs_build/dev/reports/PR_26171_BETA_022-messages-postgres-service-cutover.md
M docs_build/dev/reports/coverage_changed_js_guardrail.txt
M docs_build/dev/reports/playwright_v8_coverage_report.txt
A src/dev-runtime/messages/messages-postgres-service.mjs
D src/dev-runtime/messages/messages-sqlite-service.mjs
M src/dev-runtime/server/local-api-router.mjs
M tests/dev-runtime/DbSeedIntegrity.test.mjs
A tests/helpers/messagesPostgresClientStub.mjs
M tests/helpers/playwrightRepoServer.mjs
M tests/playwright/tools/MessagesTool.spec.mjs

Base: origin/main 35b04c02ea54da8b13c10354126f1ee8ddd14a89
Head before recovery merge commit: 1806adbf5df787f7072c6579d23b99bb4257466b
## git diff --cached --stat
 docs_build/database/dml/DML_INDEX.md | 2 +-
 docs_build/database/dml/messages.sql | 2 +-
 docs_build/database/seed/messages.json | 2 +-
 ...ice-cutover-instruction-compliance-checklist.md | 16 +
 ...gres-service-cutover-manual-validation-notes.md | 21 +
 ...1_BETA_022-messages-postgres-service-cutover.md | 39 +
 .../dev/reports/coverage_changed_js_guardrail.txt | 32 +-
 .../dev/reports/playwright_v8_coverage_report.txt | 91 +-
 .../messages/messages-postgres-service.mjs | 1117 +++++++++++++++++++
 .../messages/messages-sqlite-service.mjs | 1118 --------------------
 src/dev-runtime/server/local-api-router.mjs | 16 +-
 tests/dev-runtime/DbSeedIntegrity.test.mjs | 61 +-
 tests/helpers/messagesPostgresClientStub.mjs | 83 ++
 tests/helpers/playwrightRepoServer.mjs | 6 +-
 tests/playwright/tools/MessagesTool.spec.mjs | 39 +-
 15 files changed, 1377 insertions(+), 1268 deletions(-)

## 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

## 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(-)

## 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
## git diff --stat
Binary file modifieddocs_build/dev/reports/codex_review.diff

View file


Open in desktop


Binary file not shown.


32 changes: 2 additions & 30 deletions
docs_build/dev/reports/coverage_changed_js_guardrail.txt

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -6,37 +6,9 @@ Missing changed runtime JS files are WARN, not FAIL.
Source: Playwright/Chromium built-in V8 coverage from the active Playwright run.

Changed runtime JS files considered:
(0%) assets/theme-v2/js/account-achievements.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) src/dev-runtime/messages/messages-sqlite-service.mjs - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) src/dev-runtime/persistence/mock-db-store.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) src/dev-runtime/persistence/tool-repositories/game-journey-mock-repository.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) src/dev-runtime/seed/server-seed-loader.mjs - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) src/dev-runtime/messages/messages-postgres-service.mjs - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) src/dev-runtime/server/local-api-router.mjs - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) src/shared/toolbox/tool-metadata-inventory.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) toolbox/controls/controls.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) toolbox/game-hub/game-hub-api-client.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) toolbox/game-hub/game-hub.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) toolbox/game-journey/game-journey.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) toolbox/idea-board/index.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) toolbox/messages/messages.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) toolbox/text-to-speech/text2speech.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(0%) toolbox/tools-page-accordions.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
(74%) assets/theme-v2/js/gamefoundry-partials.js - executed lines 977/977; executed functions 67/90
(90%) assets/theme-v2/js/admin-system-health.js - executed lines 227/227; executed functions 28/31

Guardrail warnings:
(0%) assets/theme-v2/js/account-achievements.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) src/dev-runtime/messages/messages-sqlite-service.mjs - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) src/dev-runtime/persistence/mock-db-store.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) src/dev-runtime/persistence/tool-repositories/game-journey-mock-repository.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) src/dev-runtime/seed/server-seed-loader.mjs - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) src/dev-runtime/messages/messages-postgres-service.mjs - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) src/dev-runtime/server/local-api-router.mjs - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) src/shared/toolbox/tool-metadata-inventory.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) toolbox/controls/controls.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) toolbox/game-hub/game-hub-api-client.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) toolbox/game-hub/game-hub.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) toolbox/game-journey/game-journey.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) toolbox/idea-board/index.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) toolbox/messages/messages.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) toolbox/text-to-speech/text2speech.js - WARNING: changed runtime JS file missing from coverage; advisory only
(0%) toolbox/tools-page-accordions.js - WARNING: changed runtime JS file missing from coverage; advisory only
Loading
Loading