Clean up old toolbox beta path and add status kickers - PR_26159_053-tool-status-kickers-beta-access

Author: ToolboxAid

docs_build/dev/admin-notes/engine/game config.txt

@@ -0,0 +1,4 @@
+[ ] context disable
+[ ] windows key disable
+[ ] disable keyboard special (function key, not fuction key actions
+[ ]public or private

docs_build/dev/reports/tool-status-kickers-beta-access-report.md

@@ -0,0 +1,73 @@
+# PR_26159_053 Tool Status Kickers Beta Access
+
+## Executive Summary
+
+Status kickers are now registry-driven on the Toolbox index. Existing internal `status` metadata is preserved for compatibility, while user-facing tiles derive `releaseChannel`, `releaseChannelLabel`, and exact hover/help text. AI Assistant is the first beta-restricted tool through `requiredRole: "beta"`, with User 2 seeded as the local beta tester.
+
+Playwright impacted: Yes.
+
+## Branch Guard
+
+| Check | Status | Evidence |
+| --- | --- | --- |
+| Current branch is main | PASS | `git branch --show-current` returned `main`. |
+| Local branch list checked | PASS | `git branch --list` returned `* main`. |
+| Worktree inspected before edits | PASS | Initial `git status --short --untracked-files=all` was clean. |
+
+## Old Status/Beta Cleanup
+
+| Area | Result |
+| --- | --- |
+| Old code found | Existing active Toolbox code used `TOOL_STATUS_MODEL` and visible tile labels `Ready`, `Wireframe`, `Planned`, `Hidden`; `tools-page-accordions.js` filtered creator tiles with `tool.status === "Ready"`. |
+| Files cleaned | `toolbox/tools-page-accordions.js` now derives display/status filtering from release channels; the old visible `Ready` pill text is replaced by `Complete`. |
+| No obsolete beta route found | No active `{tool}-beta` routes, beta-only pages, beta-only launch code, or voting code existed before this PR. |
+| Intentionally preserved | `TOOL_STATUS_MODEL`, `status`, readiness metadata, Build Path status logic, `adminOnly`, `hidden`, and existing stable tool routes are preserved for compatibility. |
+| Why preserved | Other admin/progress/build-path tooling still consumes legacy registry status fields; release channels are added as the clean user-facing access layer. |
+
+## Requirement Checklist
+
+| Requirement | Status | Evidence |
+| --- | --- | --- |
+| Read project instructions first | PASS | `docs_build/dev/PROJECT_INSTRUCTIONS.md` read before edits. |
+| Hard stop unless on main | PASS | Branch guard passed on `main`. |
+| Remove/report obsolete status/beta/vote/filter code | PASS | Cleanup section above; no stale beta/vote implementation found. |
+| Toolbox tiles show Planned/Wireframe/Beta/Complete kickers | PASS | `toolbox/toolRegistry.js` derives release channel labels; `toolbox/tools-page-accordions.js` renders `data-toolbox-kicker`. |
+| Kicker hover/help text matches requested wording | PASS | Help text constants in `toolbox/toolRegistry.js` and Playwright title assertions in `ToolboxRoutePages.spec.mjs`. |
+| Status count/filter controls side by side | PASS | `toolbox/index.html` adds `data-toolbox-status-filters`; Playwright verifies same-row button positions. |
+| Default filters Complete, Beta, Wireframe | PASS | `visibleReleaseChannels` defaults to `complete`, `beta`, `wireframe`; Playwright checks `aria-pressed`. |
+| Planned visible only when selected | PASS | Planned filter defaults false; Admin Build Game appears only after toggling Planned. |
+| No `{tool}-beta` files/routes | PASS | Static `rg --files` check found no `-beta` route/file; stable hrefs asserted. |
+| Registry-driven beta access | PASS | AI Assistant has `releaseChannel: "beta"` and `requiredRole: "beta"` in `toolbox/toolRegistry.js`. |
+| Non-beta beta launch blocked | PASS | User 1 click shows `This tool is in beta. Request beta access to try it.` |
+| Beta role launches beta tool | PASS | User 2 has seeded `beta` role and launches `/toolbox/ai-assistant/index.html`. |
+| Admin launches beta tool | PASS | DavidQ/admin launches `/toolbox/ai-assistant/index.html`. |
+| Complete tools launch normally | PASS | Existing stable Complete tool URLs remain unchanged; Colors remains `/toolbox/colors/index.html`. |
+| Wireframe tools open preview pages | PASS | Wireframe action links are not blocked and are labeled `Open Preview`. |
+| Planned tools show details/vote/feedback instead of runtime launch | PASS | Planned click stays on Toolbox and updates planned status; planned card includes non-persistent feedback controls. |
+| Planned/Wireframe voting controls render only there | PASS | Playwright verifies Build Game and Fonts vote controls; renderer returns no controls for Beta/Complete. |
+| No hidden vote persistence invented | PASS | Vote handlers update only the page status message. |
+| No inline script/style/event handlers | PASS | Toolbox source checked in Playwright; no inline script/style/onclick added. |
+| No console errors | PASS | Toolbox Playwright lane captured no repo-owned console errors. |
+
+## Validation Evidence
+
+| Lane | Status | Evidence |
+| --- | --- | --- |
+| Changed-file syntax | PASS | `node --check toolbox/tools-page-accordions.js`, `toolbox/toolRegistry.js`, `src/dev-runtime/persistence/mock-db-store.js`, and `tests/playwright/tools/ToolboxRoutePages.spec.mjs`. |
+| Toolbox page/runtime Playwright | PASS | `npx playwright test tests/playwright/tools/ToolboxRoutePages.spec.mjs --reporter=line` passed 3/3. |
+| Registry validation | PASS | `node scripts/validate-tool-registry.mjs` returned `TOOL_REGISTRY_VALID`. |
+| Diff whitespace | PASS | `git diff --check` passed; only existing LF/CRLF warnings were reported. |
+| `{tool}-beta` static audit | PASS | `rg --files | rg "(^|/|\\\\)[^/\\\\]+-beta(\\.|/|\\\\)|beta-only|tool-vote|vote-tool"` returned no matches. |
+
+## Skipped Lanes
+
+| Lane | Why Safe to Skip |
+| --- | --- |
+| Full samples validation | PR touches Toolbox registry/index behavior and local seed roles only; no sample loader/runtime framework changed. |
+| Full Playwright suite | Targeted Toolbox route/status/access lane covered the affected surface; full suite is outside this PR scope. |
+
+## Notes
+
+- Planned tools are intentionally not launched from Toolbox tiles, including for admin, because this PR explicitly says planned tools should show planned details/vote/feedback instead of launching unfinished runtime.
+- A pre-existing/generated `docs_build/dev/reports/tool_registry_validation.txt` was updated by the registry validation lane.
+- An unrelated untracked file, `docs_build/dev/admin-notes/engine/game setup.txt`, is present in the worktree and was not modified or packaged.

docs_build/dev/reports/tool_registry_validation.txt

@@ -1,45 +1,89 @@
 TOOL_REGISTRY_VALIDATION
 status=PASS
-active=AI Assistant, Animation, Assets, Cloud, Custom Extensions, Game Design, Input, Localization, MIDI, Object Vector, Palette Manager, Particles, Publish, Sound, Storage Inspector, World Vector, Game Configuration, Project Workspace
+active=AI Assistant, Project Workspace, Project Journey, Game Design, Game Configuration, Assets, Colors, Fonts, Sprites, Characters, Objects, Worlds, Animations, Audio, Music, Voices, Videos, Build Game, Game Testing, Controls, Hitboxes, Saved Data, Debug, Performance, Events, Publish, Marketplace, Community, Languages, Achievements, Ratings, Cloud, Custom Extensions, MIDI, Particles, Audio Effects, Voice Capture, Voice Output
 Filesystem Active Tool Directories
+- achievements
 - ai-assistant
-- animation
+- animations
 - assets
+- audio
+- audio-effects
+- build-game
+- characters
 - cloud
 - code
+- colors
+- community
+- controls
+- debug
+- events
+- fonts
 - game-configuration
 - game-design
-- input
-- localization
+- game-testing
+- hitboxes
+- languages
+- marketplace
 - midi
-- object-vector
-- palette
+- music
+- objects
 - particles
+- performance
+- project-journey
 - project-workspace
 - publish
-- sound
-- storage
-- world-vector
+- ratings
+- saved-data
+- speech-to-text
+- sprites
+- text-to-speech
+- videos
+- voices
+- worlds
 Registry Folders
 - ai-assistant
-- animation
+- project-workspace
+- project-journey
+- game-design
+- game-configuration
 - assets
+- colors
+- fonts
+- sprites
+- characters
+- objects
+- worlds
+- animations
+- audio
+- music
+- voices
+- videos
+- build-game
+- game-testing
+- controls
+- hitboxes
+- saved-data
+- debug
+- performance
+- events
+- publish
+- marketplace
+- community
+- languages
+- achievements
+- ratings
 - cloud
 - code
-- game-design
-- game-configuration
-- input
-- localization
 - midi
-- object-vector
-- palette
 - particles
-- publish
-- project-workspace
-- sound
-- storage
-- world-vector
+- audio-effects
+- speech-to-text
+- text-to-speech
+- users
+- environments
+- game-migration
+- platform-settings
 Issues
 - none
 Notes
-- registry, toolbox folders, and toolbox index routes are aligned
+- registry, toolbox folders, and registry-driven toolbox index routes are aligned

src/dev-runtime/persistence/mock-db-store.js

@@ -21,6 +21,7 @@ export const MOCK_DB_KEYS = Object.freeze({
     user: makeMockUlid(72),
     admin: makeMockUlid(73),
     system: makeMockUlid(74),
+    beta: makeMockUlid(75),
   }),
   userRoles: Object.freeze({
     user1User: makeMockUlid(82),
@@ -29,6 +30,7 @@ export const MOCK_DB_KEYS = Object.freeze({
     adminUser: makeMockUlid(85),
     adminAdmin: makeMockUlid(86),
     forgeBotSystem: makeMockUlid(87),
+    user2Beta: makeMockUlid(88),
   }),
 });
 
@@ -730,14 +732,24 @@ export function getStandaloneMockDbSeedTables() {
         isActive: true,
         ...standaloneAudit(8),
       },
+      {
+        key: MOCK_DB_KEYS.roles.beta,
+        roleSlug: "beta",
+        name: "beta",
+        description: "Local beta access tester.",
+        isSystemRole: false,
+        isActive: true,
+        ...standaloneAudit(9),
+      },
     ],
     user_roles: [
-      { key: MOCK_DB_KEYS.userRoles.user1User, userKey: MOCK_DB_KEYS.users.user1, roleKey: MOCK_DB_KEYS.roles.user, ...standaloneAudit(9) },
-      { key: MOCK_DB_KEYS.userRoles.user2User, userKey: MOCK_DB_KEYS.users.user2, roleKey: MOCK_DB_KEYS.roles.user, ...standaloneAudit(10) },
-      { key: MOCK_DB_KEYS.userRoles.user3User, userKey: MOCK_DB_KEYS.users.user3, roleKey: MOCK_DB_KEYS.roles.user, ...standaloneAudit(11) },
-      { key: MOCK_DB_KEYS.userRoles.adminUser, userKey: MOCK_DB_KEYS.users.admin, roleKey: MOCK_DB_KEYS.roles.user, ...standaloneAudit(12) },
-      { key: MOCK_DB_KEYS.userRoles.adminAdmin, userKey: MOCK_DB_KEYS.users.admin, roleKey: MOCK_DB_KEYS.roles.admin, ...standaloneAudit(13) },
-      { key: MOCK_DB_KEYS.userRoles.forgeBotSystem, userKey: MOCK_DB_KEYS.users.forgeBot, roleKey: MOCK_DB_KEYS.roles.system, ...standaloneAudit(14) },
+      { key: MOCK_DB_KEYS.userRoles.user1User, userKey: MOCK_DB_KEYS.users.user1, roleKey: MOCK_DB_KEYS.roles.user, ...standaloneAudit(10) },
+      { key: MOCK_DB_KEYS.userRoles.user2User, userKey: MOCK_DB_KEYS.users.user2, roleKey: MOCK_DB_KEYS.roles.user, ...standaloneAudit(11) },
+      { key: MOCK_DB_KEYS.userRoles.user2Beta, userKey: MOCK_DB_KEYS.users.user2, roleKey: MOCK_DB_KEYS.roles.beta, ...standaloneAudit(12) },
+      { key: MOCK_DB_KEYS.userRoles.user3User, userKey: MOCK_DB_KEYS.users.user3, roleKey: MOCK_DB_KEYS.roles.user, ...standaloneAudit(13) },
+      { key: MOCK_DB_KEYS.userRoles.adminUser, userKey: MOCK_DB_KEYS.users.admin, roleKey: MOCK_DB_KEYS.roles.user, ...standaloneAudit(14) },
+      { key: MOCK_DB_KEYS.userRoles.adminAdmin, userKey: MOCK_DB_KEYS.users.admin, roleKey: MOCK_DB_KEYS.roles.admin, ...standaloneAudit(15) },
+      { key: MOCK_DB_KEYS.userRoles.forgeBotSystem, userKey: MOCK_DB_KEYS.users.forgeBot, roleKey: MOCK_DB_KEYS.roles.system, ...standaloneAudit(16) },
     ],
   };
 }

tests/playwright/tools/ToolboxRoutePages.spec.mjs

@@ -92,6 +92,7 @@ test("toolbox index shows all registered list tools for Admin and preserves crea
   try {
     await setServerSession(server, MOCK_DB_KEYS.users.user1);
     await page.goto(`${server.baseUrl}/toolbox/index.html`, { waitUntil: "networkidle" });
+    await expect(page.locator("[data-toolbox-tool-name-link='AI Assistant']")).toBeVisible();
     await expect(page.locator("[data-toolbox-tool-name-link='Colors']")).toBeVisible();
     await expect(page.locator("[data-toolbox-tool-name-link='Fonts']")).toHaveCount(0);
     await expect(page.locator("[data-toolbox-tool-name-link='Assets']")).toBeVisible();
@@ -102,7 +103,119 @@ test("toolbox index shows all registered list tools for Admin and preserves crea
     await expect(page.locator("[data-toolbox-tool-name-link='Colors']")).toHaveAttribute("href", "/toolbox/colors/index.html");
     await expect(page.locator("[data-toolbox-tool-name-link='Fonts']")).toBeVisible();
     const colorsCard = page.locator("[data-toolbox-tool-name-link='Colors']").locator("xpath=ancestor::article[1]");
-    await expect(colorsCard.locator("[data-toolbox-readiness]")).toHaveText("Ready");
+    await expect(colorsCard.locator("[data-toolbox-readiness]")).toHaveText("Complete");
+
+    expect(failedRequests).toEqual([]);
+    expect(pageErrors).toEqual([]);
+    expect(consoleErrors).toEqual([]);
+  } finally {
+    await server.close();
+  }
+});
+
+test("toolbox status kickers, filters, beta access, and voting controls work from registry metadata", async ({ page }) => {
+  const server = await startRepoServer();
+  const failedRequests = [];
+  const pageErrors = [];
+  const consoleErrors = [];
+
+  page.on("response", (response) => {
+    if (response.status() >= 400) {
+      failedRequests.push(`${response.status()} ${response.url()}`);
+    }
+  });
+  page.on("requestfailed", (request) => {
+    failedRequests.push(`FAILED ${request.url()}`);
+  });
+  page.on("pageerror", (error) => {
+    const text = error.stack || error.message;
+    if (!isBrowserExtensionNoise(text)) {
+      pageErrors.push(error.message);
+    }
+  });
+  page.on("console", (message) => {
+    if (message.type() === "error" && !isBrowserExtensionNoise(message.text())) {
+      consoleErrors.push(message.text());
+    }
+  });
+
+  try {
+    await setServerSession(server, MOCK_DB_KEYS.users.user1);
+    await page.goto(`${server.baseUrl}/toolbox/index.html`, { waitUntil: "networkidle" });
+
+    await expect(page.locator("[data-toolbox-status-filter]")).toHaveText([
+      /Complete \(\d+\)/,
+      /Beta \(\d+\)/,
+      /Wireframe \(\d+\)/,
+      /Planned \(\d+\)/,
+    ]);
+    await expect(page.locator("[data-toolbox-status-filter='complete']")).toHaveAttribute("aria-pressed", "true");
+    await expect(page.locator("[data-toolbox-status-filter='beta']")).toHaveAttribute("aria-pressed", "true");
+    await expect(page.locator("[data-toolbox-status-filter='wireframe']")).toHaveAttribute("aria-pressed", "true");
+    await expect(page.locator("[data-toolbox-status-filter='planned']")).toHaveAttribute("aria-pressed", "false");
+    const statusFilterTopPositions = await page.locator("[data-toolbox-status-filter]").evaluateAll((buttons) => (
+      buttons.map((button) => Math.round(button.getBoundingClientRect().top))
+    ));
+    expect(new Set(statusFilterTopPositions).size).toBe(1);
+
+    const betaCard = page.locator("[data-toolbox-tool-card='AI Assistant']");
+    await expect(betaCard).toBeVisible();
+    await expect(betaCard.locator("[data-toolbox-kicker]")).toHaveText("Beta");
+    await expect(betaCard.locator("[data-toolbox-kicker]")).toHaveAttribute(
+      "title",
+      "Ready to try.\nFeatures, layout, and workflows may change based on feedback.",
+    );
+    await expect(betaCard.locator("[data-toolbox-launch-link='AI Assistant']").first()).not.toHaveAttribute("href", /-beta/);
+    await betaCard.locator("[data-toolbox-tile-action-row='AI Assistant'] a.btn").click();
+    await expect(page.locator("[data-toolbox-launch-status]")).toHaveText("This tool is in beta. Request beta access to try it.");
+
+    await expect(page.locator("[data-toolbox-tool-card='Build Game']")).toHaveCount(0);
+    await page.locator("[data-toolbox-status-filter='planned']").click();
+    await expect(page.locator("[data-toolbox-tool-card='Build Game']")).toHaveCount(0);
+
+    await setServerSession(server, MOCK_DB_KEYS.users.user2);
+    await page.goto(`${server.baseUrl}/toolbox/index.html`, { waitUntil: "networkidle" });
+    await page.locator("[data-toolbox-tool-card='AI Assistant'] [data-toolbox-tile-action-row='AI Assistant'] a.btn").click();
+    await page.waitForURL(/\/toolbox\/ai-assistant\/index\.html$/);
+
+    await setServerSession(server, MOCK_DB_KEYS.users.admin);
+    await page.goto(`${server.baseUrl}/toolbox/index.html`, { waitUntil: "networkidle" });
+    await expect(page.locator("[data-toolbox-tool-card='Build Game']")).toHaveCount(0);
+    await page.locator("[data-toolbox-status-filter='planned']").click();
+    const plannedCard = page.locator("[data-toolbox-tool-card='Build Game']");
+    await expect(plannedCard).toBeVisible();
+    await expect(plannedCard.locator("[data-toolbox-kicker]")).toHaveText("Planned");
+    await expect(plannedCard.locator("[data-toolbox-kicker]")).toHaveAttribute("title", "Idea exists.\nNot yet available.");
+    await expect(plannedCard.locator("[data-toolbox-vote-controls='Build Game']")).toBeVisible();
+    await plannedCard.getByRole("button", { name: "Up vote Build Game" }).click();
+    await expect(page.locator("[data-toolbox-launch-status]")).toHaveText("Build Game up vote noted as a non-persistent wireframe control.");
+    await plannedCard.locator("[data-toolbox-tile-action-row='Build Game'] a.btn").click();
+    await expect(page).toHaveURL(/\/toolbox\/index\.html$/);
+    await expect(page.locator("[data-toolbox-launch-status]")).toContainText("Build Game is planned.");
+
+    const wireframeCard = page.locator("[data-toolbox-tool-card='Fonts']");
+    await expect(wireframeCard.locator("[data-toolbox-kicker]")).toHaveText("Wireframe");
+    await expect(wireframeCard.locator("[data-toolbox-kicker]")).toHaveAttribute(
+      "title",
+      "Preview the planned workflow and layout.\nHelp shape the design before development begins.",
+    );
+    await expect(wireframeCard.locator("[data-toolbox-vote-controls='Fonts']")).toBeVisible();
+    await expect(page.locator("[data-toolbox-tool-card='Colors'] [data-toolbox-kicker]")).toHaveAttribute(
+      "title",
+      "Production ready and fully supported.",
+    );
+
+    await page.goto(`${server.baseUrl}/toolbox/index.html`, { waitUntil: "networkidle" });
+    await page.locator("[data-toolbox-tool-card='AI Assistant'] [data-toolbox-tile-action-row='AI Assistant'] a.btn").click();
+    await page.waitForURL(/\/toolbox\/ai-assistant\/index\.html$/);
+
+    const toolboxSource = await page.evaluate(async () => {
+      const response = await fetch("/toolbox/index.html");
+      return response.text();
+    });
+    expect(toolboxSource).not.toMatch(/<script(?![^>]+src=)[^>]*>/i);
+    expect(toolboxSource).not.toMatch(/<style[\s>]/i);
+    expect(toolboxSource).not.toContain("onclick=");
 
     expect(failedRequests).toEqual([]);
     expect(pageErrors).toEqual([]);

toolbox/index.html

@@ -27,6 +27,8 @@ <h1>Creator tools organized by build surface.</h1>
                     <button class="btn" type="button" data-tools-view="build-path" aria-pressed="false">Build Path</button>
                     <span class="pill" data-tools-count>Tool Count: 0/0</span>
                 </div>
+                <div class="hero-actions" aria-label="Tool status filters" data-toolbox-status-filters></div>
+                <p class="status" role="status" data-toolbox-launch-status>Showing Complete, Beta, and Wireframe tools. Select Planned to include planned tools.</p>
                 <div class="accordion-group" data-tools-accordion-list></div>
             </div>
         </section>