Enable readonly Local DB inspection in Admin DB Viewer through API - PR_26158_026-local-db-viewer-readonly

Author: ToolboxAid

admin/db-viewer.html

@@ -5,8 +5,8 @@
     <meta charset="utf-8">
     <base href="/">
     <meta name="viewport" content="width=device-width, initial-scale=1">
-    <title>Local Mem DB - GameFoundryStudio</title>
-    <meta name="description" content="Admin-only read-only Local Mem DB viewer for GameFoundryStudio project data diagnostics.">
+    <title>DB Viewer - GameFoundryStudio</title>
+    <meta name="description" content="Admin-only read-only DB viewer for GameFoundryStudio project data diagnostics.">
     <link rel="icon" href="/favicon.svg">
     <link rel="stylesheet" href="assets/theme-v2/css/theme.css">
 </head>
@@ -16,23 +16,23 @@
     <main data-admin-only="true">
         <section class="page-title">
             <div class="container">
-                <div class="kicker">Admin Only / Local Mem DB</div>
-                <h1>Local Mem DB</h1>
-                <p class="lede">Read-only Local Mem DB dump for project tables, relationships, and data diagnostics.</p>
+                <div class="kicker" data-admin-db-mode-kicker>Admin Only / Local Mem DB</div>
+                <h1 data-admin-db-mode-title>Local Mem DB</h1>
+                <p class="lede" data-admin-db-mode-description>Read-only Local Mem DB dump for project tables, relationships, and data diagnostics.</p>
             </div>
         </section>
         <section class="section">
             <div class="container container--tool-wide">
                 <div class="tool-workspace tool-workspace--wide">
                     <aside class="tool-column tool-group-build">
                         <div class="tool-column-header">
-                            <h2>Local Mem DB</h2>
+                            <h2 data-admin-db-mode-title>Local Mem DB</h2>
                         </div>
                         <div class="accordion-stack">
                             <details class="vertical-accordion" open>
                                 <summary>Scope</summary>
                                 <div class="accordion-body content-stack">
-                                    <p>Current tool Local Mem DB records are displayed as a read-only human-readable dump.</p>
+                                    <p data-admin-db-scope-description>Current tool Local Mem DB records are displayed as a read-only human-readable dump.</p>
                                     <p>Use the diagnostics panel to confirm table attribution, relationships, and table bleed checks.</p>
                                     <button class="btn btn--compact" type="button" data-admin-db-clear>Clear Local Mem DB</button>
                                 </div>
@@ -43,8 +43,8 @@ <h2>Local Mem DB</h2>
                     <section class="tool-center-panel content-stack" aria-labelledby="admin-db-viewer-title" data-admin-db-viewer>
                         <div class="content-cluster">
                             <div>
-                                <div class="kicker">Local Mem DB</div>
-                                <h2 id="admin-db-viewer-title">Local Mem DB</h2>
+                                <div class="kicker" data-admin-db-mode-kicker>Local Mem DB</div>
+                                <h2 id="admin-db-viewer-title" data-admin-db-mode-title>Local Mem DB</h2>
                             </div>
                             <span class="status" role="status" data-admin-db-status>Loading Local Mem DB records.</span>
                         </div>

admin/db-viewer.js

@@ -6,16 +6,25 @@ function devRuntimeAllowed() {
     host === "127.0.0.1";
 }
 
-function localModeSelected() {
+function currentSession() {
   try {
     const request = new XMLHttpRequest();
     request.open("GET", "/api/session/current", false);
     request.setRequestHeader("Accept", "application/json");
     request.send(null);
     const payload = request.responseText ? JSON.parse(request.responseText) : null;
-    return request.status >= 200 && request.status < 300 && payload?.data?.mode === "local-mem";
-  } catch {
-    return false;
+    if (request.status >= 200 && request.status < 300 && payload?.data) {
+      return payload.data;
+    }
+    return {
+      diagnostic: payload?.error || "Unable to read current DB Viewer session from the server API.",
+      mode: "",
+    };
+  } catch (error) {
+    return {
+      diagnostic: error instanceof Error ? error.message : String(error || "Unable to read current DB Viewer session."),
+      mode: "",
+    };
   }
 }
 
@@ -31,17 +40,18 @@ async function loadLocalDbViewer() {
     return;
   }
   if (!devRuntimeAllowed()) {
-    showGatewayStatus("Local Mem DB is available only in the local dev runtime.");
+    showGatewayStatus("DB Viewer is available only in the local dev runtime.");
     return;
   }
-  if (!localModeSelected()) {
-    showGatewayStatus("Local Mem DB Viewer is available only in Local Mem mode.");
+  const session = currentSession();
+  if (session.mode !== "local-mem" && session.mode !== "local-db") {
+    showGatewayStatus(session.diagnostic || "DB Viewer is available only in Local Mem or Local DB mode.");
     return;
   }
   const module = await import("../src/engine/api/mock-db-viewer-ui.js");
-  module.startMockDbViewer(document);
+  module.startMockDbViewer(document, { session });
 }
 
 loadLocalDbViewer().catch((error) => {
-  console.error("Unable to load Local Mem DB viewer.", error);
+  console.error("Unable to load DB Viewer.", error);
 });

docs_build/dev/reports/browser_mock_remaining_audit.md

@@ -1,34 +1,36 @@
-# PR_26158_025 Browser Mock Remaining Audit
+# PR_26158_026 Browser Mock Remaining Audit
 
 ## Scope
 
-Focused audit for the Local DB server adapter step:
+Focused audit for enabling Local DB read-only inspection through Admin DB Viewer:
 
+- `admin/db-viewer.html`
 - `admin/db-viewer.js`
-- `src/dev-runtime/server/mock-api-router.mjs`
-- `src/dev-runtime/persistence/mock-db-store.js`
-- `tests/playwright/tools/LoginSessionMode.spec.mjs`
+- `src/engine/api/mock-db-api-client.js`
+- `src/engine/api/mock-db-viewer-ui.js`
+- `tests/playwright/tools/AdminDbViewer.spec.mjs`
 
 ## Static Import Boundary
 
 | Check | Status | Evidence |
 | --- | --- | --- |
-| Browser files import `src/dev-runtime` directly. | PASS | `rg -n 'src/dev-runtime' admin assets toolbox src -g '*.js' -g '*.mjs' -g '!src/dev-runtime/**' -g '!**/*-mock-repository.js' -g '!toolbox/colors/palette-workspace-repository.js' ...` returned no matches. |
-| Browser files import mock repositories directly. | PASS | `rg -n 'mock-.*repository' ...` excluding repository implementation files returned no matches. |
-| Browser files import static `toolRegistry.js` directly. | PASS | `rg -n 'import .*toolRegistry\\.js|from .*toolRegistry\\.js|import\\(.*toolRegistry\\.js' ...` returned no matches. |
+| Browser files import `src/dev-runtime` directly. | PASS | `rg -n 'src/dev-runtime' admin assets toolbox src -g '*.js' -g '*.mjs' -g '!src/dev-runtime/**' -g '!**/*-mock-repository.js' ...` returned no matches. |
+| Browser files import mock repositories directly. | PASS | `rg -n 'mock-.*repository|LocalDbAdapter|DB implementation' ...` excluding repository implementation files returned no matches. |
+| Browser files import `LocalDbAdapter` or DB implementation modules directly. | PASS | Same focused `rg` returned no browser matches. |
 
-## Local DB Boundary
+## Local DB Viewer Boundary
 
 | Check | Status | Evidence |
 | --- | --- | --- |
-| Browser flow remains Browser -> API client -> server API -> data source. | PASS | Local DB implementation lives in `src/dev-runtime/server/mock-api-router.mjs`; login and DB Viewer continue to use `/api/session/*` and `/api/mock-db/*`. |
-| Browser code does not import Local DB implementation modules. | PASS | No browser import-boundary matches were found. |
-| Local Mem behavior remains available. | PASS | LoginSessionMode and AdminDbViewer Playwright lanes passed against Local Mem. |
-| Local DB is configured behind the existing server API boundary. | PASS | API contract probe selected `local-db`, resolved users/roles, read snapshots, and verified persisted clear across server restart. |
-| Local DB failure is visible and actionable. | PASS | API contract probe with `GAMEFOUNDRY_LOCAL_DB_DISABLE=1` returned a 500 diagnostic naming `Local DB adapter not configured` and the disabled storage cause. |
-| UAT/Prod are not local login choices. | PASS | LoginSessionMode Playwright and API contract probe assert only Local Mem and Local DB selector modes. |
+| Browser flow remains Browser -> API client -> server API -> data source. | PASS | DB Viewer still uses `src/engine/api/mock-db-api-client.js` and `/api/mock-db/snapshot`; no browser Local DB imports were added. |
+| Local Mem behavior is preserved. | PASS | AdminDbViewer Playwright retained Local Mem clear/seed, filters, diagnostics, and live persisted tool-record coverage. |
+| Local DB behavior is read-only. | PASS | AdminDbViewer Playwright asserts no Local DB clear/seed/write controls are visible or available. |
+| Local DB renders live server-backed adapter state. | PASS | Playwright and API contract probe mutate/read server state through API routes and verify DB Viewer output. |
+| Local DB missing storage fails visibly. | PASS | Playwright and API contract probe assert `Local DB adapter not configured` diagnostics. |
+| UAT/Prod are not local login choices. | PASS | `/api/session/modes` probe returned exactly `Local Mem` and `Local DB`; UAT/Prod remain server-side deployment metadata only. |
 
 ## Remaining Internal Matches
 
+- UAT/Prod strings remain only in the server adapter contract as deployment-only metadata.
 - `toolbox/*/*-mock-repository.js` and `toolbox/colors/palette-workspace-repository.js` remain server/data-source implementation modules, not active browser entry imports.
 - `/api/mock-db/*` route names and `mock-db-*` module filenames remain stable internal API contracts.

docs_build/dev/reports/coverage_changed_js_guardrail.txt

@@ -8,20 +8,9 @@ Source: Playwright/Chromium built-in V8 coverage from the active Playwright run.
 Changed runtime JS files considered:
 (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/server/mock-api-router.mjs - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
-(0%) src/shared/toolbox/assetUsageIntegration.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
-(0%) src/shared/toolbox/platformShell.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
-(0%) src/shared/toolbox/projectSystem.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
-(0%) src/shared/toolbox/toolHostManifest.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
-(0%) src/shared/toolbox/toolLaunchSSoTData.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
-(56%) src/engine/api/server-api-client.js - executed lines 154/154; executed functions 10/18
-(60%) src/engine/api/mock-db-api-client.js - executed lines 19/19; executed functions 3/5
-(86%) src/engine/api/mock-db-viewer-ui.js - executed lines 478/478; executed functions 78/91
+(96%) src/engine/api/mock-db-viewer-ui.js - executed lines 510/510; executed functions 92/96
+(100%) src/engine/api/mock-db-api-client.js - executed lines 19/19; executed functions 5/5
 
 Guardrail warnings:
 (0%) src/dev-runtime/persistence/mock-db-store.js - WARNING: changed runtime JS file missing from coverage; advisory only
 (0%) src/dev-runtime/server/mock-api-router.mjs - WARNING: changed runtime JS file missing from coverage; advisory only
-(0%) src/shared/toolbox/assetUsageIntegration.js - WARNING: changed runtime JS file missing from coverage; advisory only
-(0%) src/shared/toolbox/platformShell.js - WARNING: changed runtime JS file missing from coverage; advisory only
-(0%) src/shared/toolbox/projectSystem.js - WARNING: changed runtime JS file missing from coverage; advisory only
-(0%) src/shared/toolbox/toolHostManifest.js - WARNING: changed runtime JS file missing from coverage; advisory only
-(0%) src/shared/toolbox/toolLaunchSSoTData.js - WARNING: changed runtime JS file missing from coverage; advisory only

docs_build/dev/reports/local-db-viewer-readonly-report.md

@@ -0,0 +1,47 @@
+# PR_26158_026 Local DB Viewer Readonly Report
+
+## Executive Summary
+
+Enabled Admin DB Viewer read-only inspection for Local DB through existing server API routes. Local Mem DB Viewer behavior remains intact, while Local DB now clearly identifies the active data source, renders live adapter snapshots and schemas, and hides write controls.
+
+## Requirement Checklist
+
+| Requirement | Status | Evidence |
+| --- | --- | --- |
+| Read `docs_build/dev/PROJECT_INSTRUCTIONS.md` first. | PASS | Instructions were read before implementation. |
+| Enable Admin DB Viewer read-only inspection for Local DB through existing server API routes. | PASS | `admin/db-viewer.js` now allows `local-db`; `src/engine/api/mock-db-viewer-ui.js` reads snapshots via existing API client. |
+| Preserve existing Local Mem DB Viewer behavior. | PASS | AdminDbViewer Playwright Local Mem coverage passed, including clear/seed and live persisted tool records. |
+| Do not let browser code import dev-runtime, DB repositories, LocalDbAdapter, or DB implementation modules directly. | PASS | Static import-boundary checks returned no browser matches. |
+| DB Viewer clearly shows active data source mode. | PASS | `admin/db-viewer.html` and `mock-db-viewer-ui.js` update headings, kicker text, status, labels, and document title to Local Mem DB or Local DB. |
+| Local DB Viewer renders live adapter state and schemas, including empty tables with headers. | PASS | API contract probe and Playwright verify Local DB snapshot rendering, live state update, and empty table schema headers. |
+| Local DB Viewer fails visibly with actionable diagnostics when Local DB is unavailable. | PASS | Playwright and API contract probe verify `Local DB adapter not configured` plus storage diagnostic text. |
+| Do not add edit/write controls for Local DB. | PASS | Local DB mode removes the clear/seed control and Playwright asserts no non-filter buttons or form controls in the viewer. |
+| Do not expose UAT or Prod as local login choices. | PASS | `/api/session/modes` probe returned exactly `Local Mem` and `Local DB`. |
+| Do not modify `start_of_day` folders. | PASS | No `start_of_day` files changed. |
+
+## Implementation Notes
+
+| Area | Evidence |
+| --- | --- |
+| Gateway | `admin/db-viewer.js` now reads `/api/session/current`, allows `local-mem` and `local-db`, and passes session mode into the viewer UI. |
+| Shell copy | `admin/db-viewer.html` uses mode-aware data attributes so runtime text can show Local Mem DB or Local DB. |
+| Viewer UI | `src/engine/api/mock-db-viewer-ui.js` adds mode-aware labels, Local DB read-only behavior, Local DB empty-table messages, and mode-specific error diagnostics. |
+| API client | `src/engine/api/mock-db-api-client.js` uses generic DB Viewer snapshot wording while retaining Local Mem clear/seed methods. |
+| Validation | `tests/playwright/tools/AdminDbViewer.spec.mjs` adds Local DB read-only and Local DB unavailable tests. |
+
+## Validation Evidence
+
+| Validation | Result |
+| --- | --- |
+| Changed-file syntax checks | PASS |
+| Local DB/API snapshot contract validation | PASS |
+| AdminDbViewer Playwright | PASS, 7/7 |
+| Local login mode check | PASS |
+| Static import boundary audit | PASS |
+| `git diff --check` | PASS, Git line-ending warnings only |
+
+## Remaining Migration Notes
+
+- DB Viewer remains Local-only/admin-only; Local DB is read-only in this PR.
+- Local Mem clear/seed remains available only for Local Mem mode.
+- UAT/Prod remain deployment-only adapter contract metadata, not local login choices.