Clean up Toolbox cards status visibility and Game Configuration form layout - PR_26156_127-toolbox-card-and-status-cleanup

Author: ToolboxAid

assets/theme-v2/images/logo-merch-1.png

@@ -859,3 +859,40 @@ Required reports:
 - `docs_build/dev/reports/testing_lane_execution_report.md`
 - `docs_build/dev/reports/codex_changed_files.txt`
 - `docs_build/dev/reports/codex_review.diff`
+
+
+## PR_26156_127
+
+Changes:
+- Read `docs_build/dev/PROJECT_INSTRUCTIONS.md`.
+- Updated `toolbox/game-configuration/index.html` Playable Setup fields to use the reusable Theme V2 `table-wrapper`, `data-table`, and `tool-form-table` form table pattern.
+- Preserved Game Configuration field IDs, data hooks, form behavior, and submit behavior.
+- Set Game Configuration Playable Setup textareas to 4 rows.
+- Updated Toolbox card rendering in `toolbox/tools-page-accordions.js`.
+- Removed duplicate local Toolbox status ownership and enriched Toolbox cards from the active registry data used by Admin Tools Progress.
+- Updated normal role visibility so only Ready tools are visible outside Admin role.
+- Kept non-Ready tools visible in Admin role for planning/status review.
+- Converted Toolbox card bullet/list content to comma-separated bottom values.
+- Updated Toolbox tile action rows to one line with badge, Open Tool/Open Page link, brand color swatch, and status tag.
+- Updated targeted Playwright expectations for the changed Toolbox status/card rendering and Game Configuration table layout.
+- Did not add CSS.
+- Did not modify archived V1/V2 pages.
+- Did not modify `start_of_day`.
+
+Validation:
+- `node --check toolbox/tools-page-accordions.js`
+- `node --check tests/playwright/tools/RootToolsFutureState.spec.mjs`
+- `node --check tests/playwright/tools/ToolsProgressHydration.spec.mjs`
+- `node --check tests/playwright/tools/ToolNavigationPrevNext.spec.mjs`
+- `node --check tests/playwright/tools/GameConfigurationMockRepository.spec.mjs`
+- `node --check tests/playwright/tools/ProjectWorkspaceMockRepository.spec.mjs`
+- `node ./scripts/run-targeted-test-lanes.mjs --lane tool-runtime --lane tools-progress --lane game-configuration --lane tool-navigation`
+- `git diff --check`
+- Changed-file static validation for forbidden archive/start_of_day paths and inline style/script/event-handler additions.
+- Full samples smoke: skipped by request.
+
+Required reports:
+- `docs_build/dev/reports/toolbox-card-and-status-cleanup-report.md`
+- `docs_build/dev/reports/testing_lane_execution_report.md`
+- `docs_build/dev/reports/codex_changed_files.txt`
+- `docs_build/dev/reports/codex_review.diff`

assets/theme-v2/images/logo-merch-2.png

@@ -1 +1 @@
-Consolidate Toolbox group color SSoT - PR_26156_126-tool-group-color-ssot-consolidation
+Clean up Toolbox cards and status visibility - PR_26156_127-toolbox-card-and-status-cleanup

assets/theme-v2/images/logo-merch-3.png

@@ -1,16 +1,16 @@
 # Testing Lane Execution Report
 
-Generated: 2026-06-05T14:21:16.547Z
+Generated: 2026-06-05T14:53:22.656Z
 Dry run: No
 
 ## Summary
 
-PASS: 1
+PASS: 4
 WARN: 0
 FAIL: 0
-SKIP: 13
-Total lane elapsed time: 81.90s
-Actual browser launches: 1
+SKIP: 10
+Total lane elapsed time: 136.67s
+Actual browser launches: 4
 
 ## Full Samples Smoke
 
@@ -21,7 +21,7 @@ Reason: Skipped because changed files do not modify sample JSON or shared sample
 
 Status: PASS
 Reason: Runner preflight and Playwright structure audit passed before expensive lane execution.
-Command: C:\nvm4w\nodejs\node.exe scripts/audit-playwright-test-locations.mjs --discovery-report docs_build/dev/reports/playwright_discovery_ownership_report.md --scope-report docs_build/dev/reports/playwright_discovery_scope_report.md --scan-report docs_build/dev/reports/filesystem_scan_reduction_report.md --lanes tool-runtime --targets tests/playwright/tools/RootToolsFutureState.spec.mjs --helpers tests/helpers/playwrightRepoServer.mjs,tests/helpers/playwrightStorageIsolation.mjs,tests/helpers/playwrightV8CoverageReporter.mjs,tests/helpers/workspaceV2CoverageReporter.mjs
+Command: C:\nvm4w\nodejs\node.exe scripts/audit-playwright-test-locations.mjs --discovery-report docs_build/dev/reports/playwright_discovery_ownership_report.md --scope-report docs_build/dev/reports/playwright_discovery_scope_report.md --scan-report docs_build/dev/reports/filesystem_scan_reduction_report.md --lanes tool-runtime,tools-progress,game-configuration,tool-navigation --targets tests/playwright/tools/GameConfigurationMockRepository.spec.mjs,tests/playwright/tools/RootToolsFutureState.spec.mjs,tests/playwright/tools/ToolNavigationPrevNext.spec.mjs,tests/playwright/tools/ToolsProgressHydration.spec.mjs --helpers tests/helpers/playwrightRepoServer.mjs,tests/helpers/playwrightStorageIsolation.mjs,tests/helpers/playwrightV8CoverageReporter.mjs,tests/helpers/workspaceV2CoverageReporter.mjs
 Details: none
 
 ## Dependency Gate
@@ -32,15 +32,15 @@ Reason: No deterministic dependency failures before runtime.
 ## Runtime Scheduling
 
 Status: PASS
-Scheduled lane order: tool-runtime
-Reused runtime sessions: 0
-Reused lane snapshots: 0
-Reused warm-start lanes: 0
-Reused dependency hydration: 0
-Prevented graph rebuilds: 0
-Prevented redundant initialization: 0
+Scheduled lane order: game-configuration, tools-progress, tool-navigation, tool-runtime
+Reused runtime sessions: 1
+Reused lane snapshots: 3
+Reused warm-start lanes: 3
+Reused dependency hydration: 3
+Prevented graph rebuilds: 3
+Prevented redundant initialization: 3
 Prevented redundant browser launches: 0
-Prevented redundant lane execution: 13
+Prevented redundant lane execution: 10
 
 ## Validation Cache
 
@@ -61,7 +61,7 @@ Prevented broad lane escalation: 0
 ## Discovery Scope
 
 Status: PASS
-Target files: tests/playwright/tools/RootToolsFutureState.spec.mjs
+Target files: tests/playwright/tools/GameConfigurationMockRepository.spec.mjs, tests/playwright/tools/RootToolsFutureState.spec.mjs, tests/playwright/tools/ToolNavigationPrevNext.spec.mjs, tests/playwright/tools/ToolsProgressHydration.spec.mjs
 Required shared helpers: tests/helpers/playwrightRepoServer.mjs, tests/helpers/playwrightStorageIsolation.mjs, tests/helpers/playwrightV8CoverageReporter.mjs, tests/helpers/workspaceV2CoverageReporter.mjs
 Required fixtures: none
 Targeted file/helper reads: 5
@@ -71,29 +71,29 @@ Prevented fallback expansion: Yes; no ownership or scope blocker widened into br
 ## Targeted File Manifests
 
 Status: PASS
-Generated manifests: tool-runtime:PASS
+Generated manifests: tool-runtime:PASS, tools-progress:PASS, game-configuration:PASS, tool-navigation:PASS
 Prevented discovery expansion: Yes
-Prevented redundant scans: 4
-Persistent manifest events: tool-runtime:INVALIDATED
+Prevented redundant scans: 1
+Persistent manifest events: tool-runtime:REUSED, tools-progress:REUSED, game-configuration:REUSED, tool-navigation:INVALIDATED
 
 ## Warm-Start Reuse
 
 Status: PASS
-Warm-start events: tool-runtime:INVALIDATED
-Dependency hydration events: tool-runtime:INVALIDATED
-Prevented redundant initialization: 0
-Prevented helper resolution passes: 0
+Warm-start events: tool-runtime:REUSED, tools-progress:REUSED, game-configuration:REUSED, tool-navigation:INVALIDATED
+Dependency hydration events: tool-runtime:REUSED, tools-progress:REUSED, game-configuration:REUSED, tool-navigation:INVALIDATED
+Prevented redundant initialization: 3
+Prevented helper resolution passes: 12
 Prevented fixture ownership traversal: 0
 
 ## Lane Snapshots
 
 Status: PASS
-Snapshot events: tool-runtime:INVALIDATED
-Reused snapshots: 0
+Snapshot events: tool-runtime:REUSED, tools-progress:REUSED, game-configuration:REUSED, tool-navigation:INVALIDATED
+Reused snapshots: 3
 Invalidated snapshots: 1
-Prevented graph rebuilds: 0
-Prevented redundant dependency traversal: 0
-Prevented fixture/helper graph assembly: 0
+Prevented graph rebuilds: 3
+Prevented redundant dependency traversal: 3
+Prevented fixture/helper graph assembly: 12
 
 ## Lane Deduplication
 
@@ -108,13 +108,13 @@ Prevented Workspace lane reruns: 0
 | workspace-contract | SKIP | 0ms | 0 | Lane was not selected for this targeted run. | Root tools future-state navigation and Tool Template V2 contract | repo-served root tools page; Tool Template V2 future-state page; Theme V2 shared partials and assets |
 | project-workspace | SKIP | 0ms | 0 | Lane was not selected for this targeted run. | Project Workspace mock repository, Project Workspace UI, and Toolbox Progress/Build Path project-state bridge | repo-served Project Workspace page; repo-served Toolbox page with role simulation; in-memory SQL-shaped mock project repository |
 | game-design | SKIP | 0ms | 0 | Lane was not selected for this targeted run. | Game Design mock repository, project purpose flow, validation overlay, capability demo authoring, and Toolbox progress handoff | repo-served Game Design page; repo-served Toolbox Progress and Build Path views; in-memory SQL-shaped Game Design mock repository; Project Workspace mock project context |
-| game-configuration | SKIP | 0ms | 0 | Lane was not selected for this targeted run. | Game Configuration mock repository, Game Design handoff, configuration validation, user-facing output, and Toolbox progress handoff | repo-served Game Configuration page; repo-served Game Design page for handoff checks; repo-served Toolbox Progress and Build Path views; in-memory SQL-shaped Game Configuration mock repository; Game Design mock repository handoff |
+| game-configuration | PASS | 41.82s | 1 | Game Configuration rebuild slice validates the valid Game Design handoff, blocked invalid handoffs, configuration save/update, actionable validation, user-facing output, and Toolbox progress handoff without exercising unrelated toolbox routes. | Game Configuration mock repository, Game Design handoff, configuration validation, user-facing output, and Toolbox progress handoff | repo-served Game Configuration page; repo-served Game Design page for handoff checks; repo-served Toolbox Progress and Build Path views; in-memory SQL-shaped Game Configuration mock repository; Game Design mock repository handoff |
 | build-path | SKIP | 0ms | 0 | Lane was not selected for this targeted run. | Toolbox Build Path simplification, workflow status table, and Admin Tools Progress navigation | repo-served Toolbox page; repo-served Admin Tools Progress page; Project Workspace mock project context; Toolbox role simulation |
-| tools-progress | SKIP | 0ms | 0 | Lane was not selected for this targeted run. | Admin Tools Progress hydration, Toolbox Group view color model, and Project Build Path separation | repo-served Admin Tools Progress page; repo-served Toolbox Group view; Toolbox registry build sequence; Project Build Path workflow table |
-| tool-navigation | SKIP | 0ms | 0 | Lane was not selected for this targeted run. | Admin Tools Progress tool route links, Tool Display Mode build-order previous/next controls, and Toolbox group fallback routing | repo-served Admin Tools Progress page; repo-served Project Workspace, Game Design, and Game Configuration tool pages; repo-served Toolbox Group view with URL-selected accordion; Toolbox registry build sequence and route metadata |
+| tools-progress | PASS | 18.23s | 1 | Tools Progress validates that Admin platform progress hydrates every planned/active Toolbox registry entry in build order, the restored semantic group colors render in Toolbox Group view, and Project Build Path stays workflow-order and project-specific. | Admin Tools Progress hydration, Toolbox Group view color model, and Project Build Path separation | repo-served Admin Tools Progress page; repo-served Toolbox Group view; Toolbox registry build sequence; Project Build Path workflow table |
+| tool-navigation | PASS | 28.57s | 1 | Tool navigation validates registry-owned tool routes, disabled rendering for route-less tools, build-order previous/next controls, multi-path fallback to Toolbox Group view, and role query preservation without exercising unrelated toolbox routes. | Admin Tools Progress tool route links, Tool Display Mode build-order previous/next controls, and Toolbox group fallback routing | repo-served Admin Tools Progress page; repo-served Project Workspace, Game Design, and Game Configuration tool pages; repo-served Toolbox Group view with URL-selected accordion; Toolbox registry build sequence and route metadata |
 | tool-display-mode | SKIP | 0ms | 0 | Lane was not selected for this targeted run. | Tool Display Mode identity row, registry-owned previous/next links, disabled text fallback, and multi-path group routing | repo-served Project Workspace, Game Design, Game Configuration, and AI Assistant tool pages; repo-served Toolbox Group view with URL-selected accordion; Toolbox registry build sequence and route metadata; shared Theme V2 Tool Display Mode script |
 | tool-images | SKIP | 0ms | 0 | Lane was not selected for this targeted run. | Toolbox registry image contract, Toolbox card image rendering, and Tool Display Mode image fallback | Toolbox registry badge/tool image contract; repo-served Toolbox page; repo-served representative Toolbox tool pages; shared registry image fallback |
-| tool-runtime | PASS | 81.90s | 1 | Tool runtime lane now validates the active public toolbox/template surface and excludes removed V2 tool routes. | Active public toolbox and Tool Template V2 contract | repo-served root toolbox page; Tool Template V2 public page; Theme V2 shared partials and assets |
+| tool-runtime | PASS | 48.06s | 1 | Tool runtime lane now validates the active public toolbox/template surface and excludes removed V2 tool routes. | Active public toolbox and Tool Template V2 contract | repo-served root toolbox page; Tool Template V2 public page; Theme V2 shared partials and assets |
 | game-runtime | SKIP | 0ms | 0 | Lane was not selected for this targeted run. | Deprecated archive/v1-v2/games reference coverage |  |
 | integration | SKIP | 0ms | 0 | Lane was not selected for this targeted run. | Integration handoff behavior | No active integration Playwright specs after removal of stale V2 tool and removed game manifest routes. |
 | engine-src | SKIP | 0ms | 0 | Lane was not selected for this targeted run. | src/ engine and shared runtime capability behavior | explicit node unit fixtures; fresh in-memory localStorage/sessionStorage mocks per file |
@@ -124,11 +124,11 @@ Prevented Workspace lane reruns: 0
 
 | Lane | Duration | Test |
 | --- | --- | --- |
-| tool-runtime | 38.50s | tests\playwright\tools\RootToolsFutureState.spec.mjs:508:1 > active tool pages align center cleanup and registry group colors |
-| tool-runtime | 14.00s | tests\playwright\tools\RootToolsFutureState.spec.mjs:69:1 > root tools surface links current tool pages without old_* routes |
-| tool-runtime | 9.20s | tests\playwright\tools\RootToolsFutureState.spec.mjs:408:1 > learn wireframe pages load with shared Theme V2 structure |
-| tool-runtime | 8.00s | tests\playwright\tools\RootToolsFutureState.spec.mjs:323:1 > common header renders primary navigation order across active pages |
-| tool-runtime | 1.20s | tests\playwright\tools\RootToolsFutureState.spec.mjs:486:1 > tool template future-state page loads from root Theme V2 paths |
+| tool-runtime | 10.10s | tests\playwright\tools\RootToolsFutureState.spec.mjs:517:1 > representative active tool pages align center cleanup and registry group colors |
+| tool-runtime | 9.50s | tests\playwright\tools\RootToolsFutureState.spec.mjs:69:1 > root tools surface links current tool pages without old_* routes |
+| tool-runtime | 9.20s | tests\playwright\tools\RootToolsFutureState.spec.mjs:417:1 > learn wireframe pages load with shared Theme V2 structure |
+| tool-runtime | 7.70s | tests\playwright\tools\RootToolsFutureState.spec.mjs:332:1 > common header renders primary navigation order across active pages |
+| tool-navigation | 7.50s | tests\playwright\tools\ToolNavigationPrevNext.spec.mjs:101:1 > Toolbox card names link to registered tool routes without duplicating launch actions |
 
 ## Commands
 
@@ -142,16 +142,16 @@ Prevented Workspace lane reruns: 0
 - SKIP
 
 ### game-configuration
-- SKIP
+- PASS 41.82s C:\nvm4w\nodejs\node.exe C:\Users\davidq\Documents\GitHub\HTML-JavaScript-Gaming\node_modules\@playwright\test\cli.js test tests/playwright/tools/GameConfigurationMockRepository.spec.mjs --project=playwright --workers=1 --reporter=list
 
 ### build-path
 - SKIP
 
 ### tools-progress
-- SKIP
+- PASS 18.23s C:\nvm4w\nodejs\node.exe C:\Users\davidq\Documents\GitHub\HTML-JavaScript-Gaming\node_modules\@playwright\test\cli.js test tests/playwright/tools/ToolsProgressHydration.spec.mjs --project=playwright --workers=1 --reporter=list
 
 ### tool-navigation
-- SKIP
+- PASS 28.57s C:\nvm4w\nodejs\node.exe C:\Users\davidq\Documents\GitHub\HTML-JavaScript-Gaming\node_modules\@playwright\test\cli.js test tests/playwright/tools/ToolNavigationPrevNext.spec.mjs --project=playwright --workers=1 --reporter=list
 
 ### tool-display-mode
 - SKIP
@@ -160,7 +160,7 @@ Prevented Workspace lane reruns: 0
 - SKIP
 
 ### tool-runtime
-- PASS 81.89s C:\nvm4w\nodejs\node.exe C:\Users\davidq\Documents\GitHub\HTML-JavaScript-Gaming\node_modules\@playwright\test\cli.js test tests/playwright/tools/RootToolsFutureState.spec.mjs --project=playwright --workers=1 --reporter=list
+- PASS 48.06s C:\nvm4w\nodejs\node.exe C:\Users\davidq\Documents\GitHub\HTML-JavaScript-Gaming\node_modules\@playwright\test\cli.js test tests/playwright/tools/RootToolsFutureState.spec.mjs --project=playwright --workers=1 --reporter=list
 
 ### game-runtime
 - SKIP