Link Build Path tool names and show badges - PR_26156_119-build-path-tool-links-badges

Author: qbytesdq-jpg

docs_build/dev/codex_commands.md

@@ -590,3 +590,58 @@ 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_118
+
+Changes:
+- Read `docs_build/dev/PROJECT_INSTRUCTIONS.md`.
+- Updated Toolbox card rendering so tool names render as `h3 > a` links.
+- Sourced name-link targets from the registered Toolbox route via `getToolRoute()`.
+- Preserved existing preview-image links, Open Tool launch actions, card layout, status badges, and image diagnostics.
+- Added targeted Toolbox navigation/runtime Playwright/MSJ coverage for name-link route resolution.
+- Did not modify Toolbox registry metadata.
+- Did not add or modify CSS.
+- Did not modify `start_of_day`.
+
+Validation:
+- `node --check toolbox/tools-page-accordions.js`
+- `node --check tests/playwright/tools/ToolNavigationPrevNext.spec.mjs`
+- `npm run test:lane:tool-navigation`
+- Scoped `git diff --check` for changed implementation/test files.
+- Targeted changed-file scan confirmed no local CSS, inline styles, script blocks, or `start_of_day` references were introduced.
+- Full samples smoke: skipped by request.
+
+Required reports:
+- `docs_build/dev/reports/toolbox-name-link-navigation-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`
+
+
+## PR_26156_119
+
+Changes:
+- Read `docs_build/dev/PROJECT_INSTRUCTIONS.md`.
+- Updated Build Path rows on `toolbox/index.html` so the Tool cell renders a registry-backed badge image and linked tool name.
+- Sourced Build Path tool-name links from `getToolRoute()`.
+- Sourced Build Path badge images through the approved Toolbox registry image path.
+- Reused the existing visible image diagnostic behavior for missing badge fallback.
+- Preserved Build Path workflow order, status labels, completion values, and progress guidance.
+- Added targeted Build Path Playwright/MSJ coverage for links, badges, and visible missing-badge diagnostics.
+- Did not add inline styles, style blocks, script blocks, inline event handlers, or CSS.
+- Did not modify `start_of_day`.
+
+Validation:
+- `node --check toolbox/tools-page-accordions.js`
+- `node --check tests/playwright/tools/BuildPathProgressSimplification.spec.mjs`
+- `npm run test:lane:build-path`
+- Scoped `git diff --check` for changed implementation/test files.
+- Targeted changed-file scan confirmed no inline styles, style blocks, script blocks, inline event handlers, or `start_of_day` references were introduced.
+- Full samples smoke: skipped by request.
+
+Required reports:
+- `docs_build/dev/reports/build-path-tool-links-badges-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`

docs_build/dev/commit_comment.txt

@@ -1 +1 @@
-Close Toolbox image UI diagnostics - PR_26156_114-117-tool-image-ui-closeout
+Link Build Path tools and badges - PR_26156_119-build-path-tool-links-badges

docs_build/dev/reports/build-path-tool-links-badges-report.md

@@ -0,0 +1,47 @@
+# PR_26156_119 Build Path Tool Links Badges
+
+## Scope
+- Read `docs_build/dev/PROJECT_INSTRUCTIONS.md` before implementation.
+- Updated the Build Path screen rendered on `toolbox/index.html`.
+- Kept existing Build Path workflow order, status labels, completion values, and progress guidance intact.
+- Did not add inline styles, `<style>` blocks, `<script>` blocks, or inline event handlers.
+- Did not modify unrelated Toolbox screens.
+- Did not modify `start_of_day`.
+
+## Implementation Notes
+- `toolbox/tools-page-accordions.js`
+  - Replaced the Build Path Tool cell's plain text with a compact registry-backed badge and tool-name link.
+  - Each Build Path badge uses the approved registry badge source from `/assets/theme-v2/images/badges/`.
+  - Each Build Path tool-name link uses the registered route from `getToolRoute()` and stores it in `data-registered-tool-route`.
+  - Browser navigation uses root-relative hrefs such as `/toolbox/game-design/index.html` so links resolve correctly from `/toolbox/index.html`.
+  - Missing badge fallback reuses the existing visible image diagnostic behavior and `/assets/theme-v2/images/image-missing.svg`.
+- `tests/playwright/tools/BuildPathProgressSimplification.spec.mjs`
+  - Added targeted Build Path coverage for badge image paths.
+  - Added targeted Build Path coverage for registered tool-name links.
+  - Simulated a missing badge image failure and verified the visible diagnostic.
+  - Click-checked the Game Design Build Path link.
+
+## Validation Notes
+- Impacted lane: `build-path`.
+- Command: `npm run test:lane:build-path`.
+- Result: PASS, 4 Playwright tests.
+- Static checks:
+  - `node --check toolbox/tools-page-accordions.js` - PASS
+  - `node --check tests/playwright/tools/BuildPathProgressSimplification.spec.mjs` - PASS
+  - Scoped `git diff --check` for changed implementation/test files - PASS
+- Targeted guard checks:
+  - No inline styles, `<style>` blocks, `<script>` blocks, inline event handlers, or `start_of_day` references were introduced in changed implementation/test files.
+
+## Manual Test Notes
+- Toolbox Build Path view was exercised at `/toolbox/index.html?role=admin`.
+- Every Build Path row retained its existing workflow order.
+- Every Build Path tool name rendered as a link with a registered route.
+- Every Build Path badge rendered from the approved Theme V2 badge path for the tool.
+- Simulated Game Design badge failure displayed `Badge image missing; fallback shown.`.
+- Game Design Build Path link navigated to `/toolbox/game-design/index.html`.
+- No console errors or failed page requests were reported by the targeted lane.
+
+## Skipped Lanes
+- Full samples smoke: skipped by request.
+- Tool image lane: skipped because this PR consumes existing approved badge paths and the Build Path lane directly validates the affected UI.
+- Broad Project Workspace/runtime suites: skipped because this PR only changes Build Path table UI and preserves existing progress/status content.

docs_build/dev/reports/coverage_changed_js_guardrail.txt

@@ -6,8 +6,8 @@ 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:
-(66%) toolbox/tools-page-accordions.js - executed lines 1869/1869; executed functions 40/61
-(88%) toolbox/toolRegistry.js - executed lines 1598/1598; executed functions 28/32
+(81%) toolbox/toolRegistry.js - executed lines 1598/1598; executed functions 26/32
+(90%) toolbox/tools-page-accordions.js - executed lines 1934/1934; executed functions 70/78
 
 Guardrail warnings:
 (100%) none - no changed runtime JS coverage warnings

docs_build/dev/reports/dependency_gating_report.md

@@ -1,6 +1,6 @@
 # Dependency Gating Report
 
-Generated: 2026-06-05T12:32:34.997Z
+Generated: 2026-06-05T12:42:55.005Z
 Status: PASS
 
 ## Gate Order
@@ -18,9 +18,9 @@ Status: PASS
 | project-workspace | No | SKIP | none | Project Workspace mock repository, Project Workspace UI, and Toolbox Progress/Build Path project-state bridge | Lane was not selected, so dependency-gated runtime scheduling skipped it. |
 | game-design | No | SKIP | none | Game Design mock repository, project purpose flow, validation overlay, capability demo authoring, and Toolbox progress handoff | Lane was not selected, so dependency-gated runtime scheduling skipped it. |
 | game-configuration | No | SKIP | none | Game Configuration mock repository, Game Design handoff, configuration validation, user-facing output, and Toolbox progress handoff | Lane was not selected, so dependency-gated runtime scheduling skipped it. |
-| build-path | No | SKIP | none | Toolbox Build Path simplification, workflow status table, and Admin Tools Progress navigation | Lane was not selected, so dependency-gated runtime scheduling skipped it. |
+| build-path | Yes | PASS | none | Toolbox Build Path simplification, workflow status table, and Admin Tools Progress navigation | Lane has no lane dependencies and is eligible after preflight and compilation pass. |
 | tools-progress | No | SKIP | none | Admin Tools Progress hydration, Toolbox Group view color model, and Project Build Path separation | Lane was not selected, so dependency-gated runtime scheduling skipped it. |
-| tool-navigation | Yes | PASS | none | Admin Tools Progress tool route links, Tool Display Mode build-order previous/next controls, and Toolbox group fallback routing | Lane has no lane dependencies and is eligible after preflight and compilation pass. |
+| tool-navigation | No | SKIP | none | Admin Tools Progress tool route links, Tool Display Mode build-order previous/next controls, and Toolbox group fallback routing | Lane was not selected, so dependency-gated runtime scheduling skipped it. |
 | tool-display-mode | No | SKIP | none | Tool Display Mode identity row, registry-owned previous/next links, disabled text fallback, and multi-path group routing | Lane was not selected, so dependency-gated runtime scheduling skipped it. |
 | tool-images | No | SKIP | none | Toolbox registry image contract, Toolbox card image rendering, and Tool Display Mode image fallback | Lane was not selected, so dependency-gated runtime scheduling skipped it. |
 | tool-runtime | No | SKIP | none | Active public toolbox and Tool Template V2 contract | Lane was not selected, so dependency-gated runtime scheduling skipped it. |

docs_build/dev/reports/dependency_hydration_reuse_report.md

@@ -1,6 +1,6 @@
 # Dependency Hydration Reuse Report
 
-Generated: 2026-06-05T12:32:34.998Z
+Generated: 2026-06-05T12:42:55.005Z
 Status: PASS
 
 ## Summary
@@ -16,7 +16,7 @@ Prevented fixture ownership traversal: 0
 
 | Lane | Status | Helpers | Fixtures | Imports | Dependency Hydration Hash | Reason |
 | --- | --- | --- | --- | --- | --- | --- |
-| tool-navigation | INVALIDATED | tests/helpers/playwrightRepoServer.mjs; tests/helpers/playwrightStorageIsolation.mjs; tests/helpers/playwrightV8CoverageReporter.mjs; tests/helpers/workspaceV2CoverageReporter.mjs | none | tests/helpers/playwrightRepoServer.mjs; tests/helpers/playwrightStorageIsolation.mjs; tests/helpers/playwrightV8CoverageReporter.mjs; tests/helpers/workspaceV2CoverageReporter.mjs; toolbox/toolRegistry.js | 0074b6ff936b5572 | Dependency hydration was refreshed after warm-start invalidation. |
+| build-path | INVALIDATED | tests/helpers/playwrightRepoServer.mjs; tests/helpers/playwrightStorageIsolation.mjs; tests/helpers/playwrightV8CoverageReporter.mjs; tests/helpers/workspaceV2CoverageReporter.mjs | none | tests/helpers/playwrightRepoServer.mjs; tests/helpers/playwrightStorageIsolation.mjs; tests/helpers/playwrightV8CoverageReporter.mjs; tests/helpers/workspaceV2CoverageReporter.mjs; toolbox/toolRegistry.js | 63d18ef7748baba0 | Dependency hydration was refreshed after warm-start invalidation. |
 
 ## Safeguards
 

docs_build/dev/reports/execution_graph_reuse_report.md

@@ -1,6 +1,6 @@
 # Execution Graph Reuse Report
 
-Generated: 2026-06-05T12:32:34.998Z
+Generated: 2026-06-05T12:42:55.006Z
 Status: PASS
 
 ## Summary
@@ -16,7 +16,7 @@ Prevented targeted scheduling work: 0
 
 | Lane | Status | Snapshot Status | Execution Graph Hash | Reason |
 | --- | --- | --- | --- | --- |
-| tool-navigation | INVALIDATED | INVALIDATED | 7c80d9ed012db476 | Lane snapshot is part of the selected targeted execution graph. |
+| build-path | INVALIDATED | INVALIDATED | 6d25c5bf0f297b47 | Lane snapshot is part of the selected targeted execution graph. |
 
 ## Safeguards
 

docs_build/dev/reports/failure_fingerprint_report.md

@@ -1,6 +1,6 @@
 # Failure Fingerprint Report
 
-Generated: 2026-06-05T12:32:34.998Z
+Generated: 2026-06-05T12:43:20.765Z
 Status: PASS
 
 ## Summary

docs_build/dev/reports/filesystem_scan_reduction_report.md

@@ -1,6 +1,6 @@
 # Filesystem Scan Reduction Report
 
-Generated: 2026-06-05T12:32:34.965Z
+Generated: 2026-06-05T12:42:54.984Z
 Status: PASS
 
 ## Scan Enforcement

docs_build/dev/reports/incremental_validation_report.md

@@ -1,6 +1,6 @@
 # Incremental Validation Report
 
-Generated: 2026-06-05T12:32:34.998Z
+Generated: 2026-06-05T12:42:55.006Z
 Status: PASS
 
 ## Reuse Summary
@@ -18,7 +18,7 @@ Prevented fixture resolution passes: 0
 
 | Lane | Decision | Invalidated By | Runtime Savings Observation |
 | --- | --- | --- | --- |
-| tool-navigation | INVALIDATED | Persistent manifest lane definition hash changed for tool-navigation. | Manifest was regenerated or skipped; no reuse savings for this lane. |
+| build-path | INVALIDATED | Persistent manifest lane definition hash changed for build-path. | Manifest was regenerated or skipped; no reuse savings for this lane. |
 
 ## Invalidation Rules