Add confirmed independent DB reseed control to login page - PR_26158_038-login-db-reseed-control

Author: ToolboxAid

assets/theme-v2/js/login-session.js

@@ -5,6 +5,7 @@ import {
   setSessionMode,
   setSessionUser,
 } from "../../../src/engine/api/session-api-client.js";
+import { seedMockDb } from "../../../src/engine/api/mock-db-api-client.js";
 
 const modeButtons = Array.from(document.querySelectorAll("[data-login-mode]"));
 const modeTitle = document.querySelector("[data-login-mode-title]");
@@ -14,11 +15,21 @@ const modeDisabledMessage = document.querySelector("[data-login-mode-disabled-me
 const userControls = document.querySelector("[data-login-user-controls]");
 const userStatus = document.querySelector("[data-login-user-status]");
 const continueLink = document.querySelector("[data-login-continue]");
+const reseedFields = {
+  activeMode: document.querySelector("[data-login-reseed-active-mode]"),
+  cancelButton: document.querySelector("[data-login-reseed-cancel]"),
+  confirmButton: document.querySelector("[data-login-reseed-confirm]"),
+  startButton: document.querySelector("[data-login-reseed-start]"),
+  status: document.querySelector("[data-login-reseed-status]"),
+  target: document.querySelector("[data-login-reseed-target]"),
+};
 const localApiStartCommand = "npm run dev:local-api";
 const localApiLoginUrl = "http://127.0.0.1:5501/login.html";
 const expectedSessionEndpoint = "/api/session/current";
 const apiBackedLoginDiagnostic = `Use the API-backed local server for login. Run ${localApiStartCommand} and open ${localApiLoginUrl}.`;
 const staticModeDisabledMessage = `Use the API-backed local server for login. Run ${localApiStartCommand} and open ${localApiLoginUrl}. Local Mem and Local DB are disabled until the local API server is running.`;
+let reseedConfirmationPending = false;
+let reseedStatusMessage = "";
 const localStatusFields = {
   api: document.querySelector("[data-login-status-api]"),
   apiUrl: document.querySelector("[data-login-status-api-url]"),
@@ -68,6 +79,55 @@ function updateLocalDevelopmentStatus({ apiAvailability, disabledReason, serverM
   }
 }
 
+function setReseedButtonState(button, disabled) {
+  if (!button) {
+    return;
+  }
+  button.disabled = disabled;
+  if (disabled) {
+    button.setAttribute("aria-disabled", "true");
+  } else {
+    button.removeAttribute("aria-disabled");
+  }
+}
+
+function reseedModeLabel(mode) {
+  return mode?.label || mode?.environment || mode?.id || "selected DB mode";
+}
+
+function updateReseedControls({ apiAvailable, mode, statusMessage }) {
+  const modeLabel = apiAvailable ? reseedModeLabel(mode) : "Unavailable";
+  const message = statusMessage || (apiAvailable
+    ? `Ready to reseed ${modeLabel} only.`
+    : "Reseed unavailable until the Local API is available.");
+  if (reseedFields.activeMode) {
+    reseedFields.activeMode.textContent = modeLabel;
+  }
+  if (reseedFields.target) {
+    reseedFields.target.textContent = modeLabel;
+  }
+  if (reseedFields.status) {
+    reseedFields.status.textContent = message;
+  }
+  setReseedButtonState(reseedFields.startButton, !apiAvailable || reseedConfirmationPending);
+  setReseedButtonState(reseedFields.confirmButton, !apiAvailable || !reseedConfirmationPending);
+  setReseedButtonState(reseedFields.cancelButton, !apiAvailable || !reseedConfirmationPending);
+  if (reseedFields.confirmButton) {
+    reseedFields.confirmButton.hidden = !apiAvailable || !reseedConfirmationPending;
+  }
+  if (reseedFields.cancelButton) {
+    reseedFields.cancelButton.hidden = !apiAvailable || !reseedConfirmationPending;
+  }
+}
+
+function currentModeForReseed() {
+  const session = getSessionCurrent();
+  return getSessionModes().find((item) => item.id === session.mode) || {
+    id: session.mode,
+    label: session.environment || session.mode,
+  };
+}
+
 function dispatchSessionChanged() {
   window.dispatchEvent(new CustomEvent("gamefoundry:mock-db-session-user-changed", {
     detail: getSessionCurrent(),
@@ -147,6 +207,8 @@ function renderError(error) {
   const disabledReason = message === apiBackedLoginDiagnostic
     ? staticModeDisabledMessage
     : `Local Mem and Local DB are disabled because ${message}`;
+  reseedConfirmationPending = false;
+  reseedStatusMessage = "Reseed unavailable until the Local API is available.";
   modeButtons.forEach((button) => {
     button.disabled = true;
     button.setAttribute("aria-disabled", "true");
@@ -177,6 +239,11 @@ function renderError(error) {
     disabledReason,
     serverMode,
   });
+  updateReseedControls({
+    apiAvailable: false,
+    mode: null,
+    statusMessage: reseedStatusMessage,
+  });
   updateContinueLink();
 }
 
@@ -216,6 +283,11 @@ function render() {
       disabledReason: "Local Mem and Local DB are enabled because the Local API is available.",
       serverMode: `API-backed local server (${mode.label || session.mode})`,
     });
+    updateReseedControls({
+      apiAvailable: true,
+      mode,
+      statusMessage: reseedStatusMessage,
+    });
     if (modeDisabledMessage) {
       modeDisabledMessage.hidden = true;
       modeDisabledMessage.textContent = "";
@@ -235,6 +307,8 @@ modeButtons.forEach((button) => {
       return;
     }
     try {
+      reseedConfirmationPending = false;
+      reseedStatusMessage = "";
       setSessionMode(modeId);
       dispatchModeChanged();
       render();
@@ -258,4 +332,51 @@ userControls?.addEventListener("click", (event) => {
   }
 });
 
+reseedFields.startButton?.addEventListener("click", () => {
+  try {
+    const mode = currentModeForReseed();
+    const modeLabel = reseedModeLabel(mode);
+    reseedConfirmationPending = true;
+    reseedStatusMessage = `Confirm reseed for ${modeLabel} only. The other DB mode will not be reseeded.`;
+    render();
+  } catch (error) {
+    reseedConfirmationPending = false;
+    reseedStatusMessage = `Reseed unavailable: ${errorMessage(error)}`;
+    renderError(error);
+  }
+});
+
+reseedFields.cancelButton?.addEventListener("click", () => {
+  try {
+    const modeLabel = reseedModeLabel(currentModeForReseed());
+    reseedConfirmationPending = false;
+    reseedStatusMessage = `Reseed canceled for ${modeLabel}.`;
+    render();
+  } catch (error) {
+    reseedConfirmationPending = false;
+    reseedStatusMessage = `Reseed canceled. ${errorMessage(error)}`;
+    renderError(error);
+  }
+});
+
+reseedFields.confirmButton?.addEventListener("click", () => {
+  try {
+    const modeLabel = reseedModeLabel(currentModeForReseed());
+    seedMockDb();
+    reseedConfirmationPending = false;
+    reseedStatusMessage = `Reseed complete for ${modeLabel}. Only ${modeLabel} was reseeded.`;
+    dispatchModeChanged();
+    render();
+  } catch (error) {
+    const message = errorMessage(error);
+    let modeLabel = "selected DB mode";
+    try {
+      modeLabel = reseedModeLabel(currentModeForReseed());
+    } catch {}
+    reseedConfirmationPending = false;
+    reseedStatusMessage = `Reseed failed for ${modeLabel}: ${message}`;
+    render();
+  }
+});
+
 render();

docs_build/dev/reports/coverage_changed_js_guardrail.txt

@@ -11,6 +11,7 @@ Changed runtime JS files considered:
 (0%) toolbox/assets/assets-mock-repository.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
 (0%) toolbox/colors/palette-workspace-repository.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
 (0%) toolbox/project-journey/project-journey-mock-repository.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
+(80%) src/engine/api/mock-db-api-client.js - executed lines 19/19; executed functions 4/5
 (87%) src/engine/api/mock-db-viewer-ui.js - executed lines 517/517; executed functions 85/98
 
 Guardrail warnings:

docs_build/dev/reports/login-db-reseed-control-report.md

@@ -0,0 +1,65 @@
+# PR_26158_038 Login DB Reseed Control Report
+
+## Summary
+
+Added a two-step Reseed control to the login page Local Development Status section. The control uses the existing server API boundary (`/api/mock-db/seed`) and reseeds only the active DB mode: Local Mem or Local DB.
+
+## Files Changed
+
+| File | Change |
+| --- | --- |
+| `login.html` | Added Local Development Status reseed fields, start/confirm/cancel controls, and visible reseed status text. |
+| `assets/theme-v2/js/login-session.js` | Wired reseed UI state, confirmation, cancel, success, failure, and API-unavailable disabled handling. |
+| `src/engine/api/mock-db-api-client.js` | Updated seed client context text from Local Mem-specific wording to generic Mock DB seed wording. |
+| `src/dev-runtime/server/mock-api-router.mjs` | Updated seed action diagnostics to name Local Mem or Local DB based on the active mode. |
+| `tests/dev-runtime/DbSeedIntegrity.test.mjs` | Added independent Local Mem/Local DB reseed API validation. |
+| `tests/playwright/tools/LoginSessionMode.spec.mjs` | Added UI coverage for reseed confirmation, cancel, Local Mem success, Local DB success, and visible failure status. |
+
+## Requirement Checklist
+
+| Requirement | Status | Evidence |
+| --- | --- | --- |
+| Read `docs_build/dev/PROJECT_INSTRUCTIONS.md` first. | PASS | Read before implementation; applied PR completion, HTML, validation, and ZIP/report rules. |
+| Add a Reseed control to the login page Local Development Status section. | PASS | `login.html` adds `data-login-reseed-*` controls inside the Local Development Status card. |
+| Require confirmation before reseed executes. | PASS | `login-session.js` only calls `seedMockDb()` from the confirm handler; LoginSessionMode verifies start -> confirm/cancel state. |
+| Local Mem reseeds only Local Mem. | PASS | Node reseed test mutates Local DB, reseeds Local Mem, and verifies Local DB remains mutated. |
+| Local DB reseeds only Local DB. | PASS | Node reseed test reseeds Local DB separately and verifies Local Mem remains unchanged by that action. |
+| Do not reseed both DBs from one action. | PASS | `LocalDevMockDataSource.seed()` persists only the current adapter state; Node test validates both directions. |
+| Show active DB mode visibly. | PASS | `data-login-reseed-active-mode`; LoginSessionMode verifies Local Mem and Local DB text. |
+| Show reseed target visibly. | PASS | `data-login-reseed-target`; LoginSessionMode verifies Local Mem and Local DB text. |
+| Show success visibly. | PASS | LoginSessionMode verifies `Reseed complete for Local Mem...` and `Reseed complete for Local DB...`. |
+| Show failure visibly. | PASS | LoginSessionMode forces `/api/mock-db/seed` to return `ok:false` and verifies visible failure text. |
+| Show canceled status visibly. | PASS | LoginSessionMode verifies `Reseed canceled for Local Mem.` |
+| Preserve real runtime timestamps during reseed. | PASS | `DbSeedIntegrity.test.mjs` reuses runtime timestamp assertions after Local Mem and Local DB reseed. |
+| Preserve guest tool samples from PR_26158_037. | PASS | Existing seed integrity test still verifies guest samples for every active tool. |
+| Preserve unique per-user seeded data from PR_26158_037. | PASS | Existing seed integrity test still verifies unique user-owned project/tool state keys. |
+| Preserve SQLite-backed Local DB behind API boundary. | PASS | Local DB reseed and AdminDbViewer tests access Local DB through `/api/*`; no browser DB implementation imports were added. |
+| Do not add fallback behavior. | PASS | Static/API-unavailable behavior remains disabled; reseed controls disable through `renderError()`. |
+| Do not add UAT/Prod behavior. | PASS | Login options and seed behavior remain Local Mem / Local DB only. |
+| Do not add CSS. | PASS | No CSS files or inline/page-local styles were changed. |
+
+## Validation Results
+
+| Validation | Result |
+| --- | --- |
+| Changed-file syntax checks | PASS |
+| Targeted reseed API/DB validation | PASS, `node --test tests/dev-runtime/DbSeedIntegrity.test.mjs` 2/2 |
+| LoginSessionMode Playwright | PASS, 6/6 |
+| AdminDbViewer Playwright | PASS, 7/7 |
+| Changed-file/static validation | PASS |
+| Playwright V8 coverage | PASS/WARN advisory report generated |
+
+## Skipped Validation
+
+| Lane | Decision | Reason |
+| --- | --- | --- |
+| Full samples smoke | SKIP | No sample runtime or loader changed. |
+| Full Playwright suite | SKIP | Targeted login, DB Viewer, and reseed API lanes cover this PR scope. |
+| ProjectJourneyTool Playwright | SKIP | Project Journey runtime UI was not changed. |
+| ToolboxRoutePages Playwright | SKIP | Tool route rendering was not changed. |
+
+## Notes
+
+- The first failure-case Playwright attempt used an intentional HTTP 500 and produced an expected browser resource console error. The test was corrected to use a JSON API failure (`ok:false`) and then passed.
+- Existing Node SQLite experimental warnings and seed-only audit fallback diagnostics appeared during validation and did not indicate PR_26158_038 failures.
+- The generated V8 coverage report includes advisory entries from the existing stacked HEAD-diff coverage helper; current PR files are listed in `codex_changed_files.txt`.

docs_build/dev/reports/playwright_v8_coverage_report.txt

@@ -14,27 +14,28 @@ Note: coverage entries are aggregated across every page/tool where coverageRepor
 Exercised tool entry points detected:
 (61%) Toolbox Index - exercised 3 runtime JS files
 (0%) Tool Template V2 - not exercised by this Playwright run
-(81%) Theme V2 Shared JS - exercised 3 runtime JS files
+(84%) Theme V2 Shared JS - exercised 3 runtime JS files
 
 Changed runtime JS files covered:
 (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%) toolbox/assets/assets-mock-repository.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
 (0%) toolbox/colors/palette-workspace-repository.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
 (0%) toolbox/project-journey/project-journey-mock-repository.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
+(80%) src/engine/api/mock-db-api-client.js - executed lines 19/19; executed functions 4/5
 (87%) src/engine/api/mock-db-viewer-ui.js - executed lines 517/517; executed functions 85/98
 
 Files with executed line/function counts where available:
-(53%) src/engine/api/server-api-client.js - executed lines 159/159; executed functions 10/19
 (55%) toolbox/project-journey/project-journey.js - executed lines 1003/1003; executed functions 54/99
-(60%) src/engine/api/mock-db-api-client.js - executed lines 19/19; executed functions 3/5
+(58%) src/engine/api/server-api-client.js - executed lines 159/159; executed functions 11/19
 (64%) assets/theme-v2/js/tool-display-mode.js - executed lines 201/201; executed functions 9/14
 (67%) admin/db-viewer.js - executed lines 53/53; executed functions 4/6
+(80%) src/engine/api/mock-db-api-client.js - executed lines 19/19; executed functions 4/5
 (81%) toolbox/tool-registry-api-client.js - executed lines 148/148; executed functions 22/27
 (83%) assets/theme-v2/js/gamefoundry-partials.js - executed lines 442/442; executed functions 33/40
 (87%) src/engine/api/mock-db-viewer-ui.js - executed lines 517/517; executed functions 85/98
 (88%) src/engine/api/session-api-client.js - executed lines 34/34; executed functions 7/8
-(89%) assets/theme-v2/js/login-session.js - executed lines 243/243; executed functions 17/19
+(96%) assets/theme-v2/js/login-session.js - executed lines 357/357; executed functions 26/27
 (100%) toolbox/project-journey/project-journey-api-client.js - executed lines 12/12; executed functions 2/2
 
 Uncovered or low-coverage changed JS files:
@@ -50,9 +51,9 @@ Changed JS files considered:
 (0%) tests/dev-runtime/DbSeedIntegrity.test.mjs - changed JS file not collected as browser runtime coverage
 (0%) tests/playwright/tools/AdminDbViewer.spec.mjs - changed JS file not collected as browser runtime coverage
 (0%) tests/playwright/tools/LoginSessionMode.spec.mjs - changed JS file not collected as browser runtime coverage
-(0%) tests/playwright/tools/StaticOnlyLoginApiRequired.spec.mjs - changed JS file not collected as browser runtime coverage
 (0%) toolbox/assets/assets-mock-repository.js - changed JS file not collected as browser runtime coverage
 (0%) toolbox/colors/palette-workspace-repository.js - changed JS file not collected as browser runtime coverage
 (0%) toolbox/project-journey/project-journey-mock-repository.js - changed JS file not collected as browser runtime coverage
+(80%) src/engine/api/mock-db-api-client.js - changed JS file with browser V8 coverage
 (87%) src/engine/api/mock-db-viewer-ui.js - changed JS file with browser V8 coverage
-(89%) assets/theme-v2/js/login-session.js - changed JS file with browser V8 coverage
+(96%) assets/theme-v2/js/login-session.js - changed JS file with browser V8 coverage