Skip to content

Commit 377c8c8

Browse files
committed
Simplify Build Path and rename admin progress to Tools Progress - PR_26155_088-091-build-path-simplification
1 parent f841105 commit 377c8c8

59 files changed

Lines changed: 1528 additions & 295 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

admin/tools-progress.html

Lines changed: 94 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,94 @@
1+
<!doctype html>
2+
<html lang="en">
3+
4+
<head>
5+
<meta charset="utf-8">
6+
<base href="/">
7+
<meta name="viewport" content="width=device-width, initial-scale=1">
8+
<title>Tools Progress - GameFoundryStudio</title>
9+
<meta name="description" content="Admin planning page for platform-development Tools Progress.">
10+
<link rel="icon" href="/favicon.svg">
11+
<link rel="stylesheet" href="assets/theme-v2/css/theme.css">
12+
</head>
13+
14+
<body>
15+
<div data-partial="header-nav"></div>
16+
<main>
17+
<section class="page-title">
18+
<div class="container">
19+
<div class="kicker">Admin</div>
20+
<h1>Tools Progress</h1>
21+
<p class="lede">Platform-development progress for deciding what to build next for the Toolbox.</p>
22+
</div>
23+
</section>
24+
<section class="section">
25+
<div class="container account-panel">
26+
<aside class="side-menu" aria-label="Admin pages">
27+
<a href="admin/analytics.html">Analytics</a>
28+
<a href="admin/branding.html">Branding</a>
29+
<a href="admin/controls.html">Controls</a>
30+
<a href="admin/design-system.html">Design System</a>
31+
<a href="toolbox/environments/index.html">Environments</a>
32+
<a href="toolbox/game-migration/index.html">Game Migration</a>
33+
<a href="admin/grouping-colors.html">Grouping Colors</a>
34+
<a href="admin/moderation.html">Moderation</a>
35+
<a href="toolbox/platform-settings/index.html">Platform Settings</a>
36+
<a href="admin/ratings.html">Ratings</a>
37+
<a href="admin/roles.html">Roles</a>
38+
<a href="admin/site-settings.html">Site Settings</a>
39+
<a href="admin/themes.html">Themes</a>
40+
<a class="active" aria-current="page" href="admin/tools-progress.html">Tools Progress</a>
41+
<a href="admin/users.html">Users</a>
42+
</aside>
43+
<div class="card">
44+
<div class="card-body content-stack">
45+
<div>
46+
<div class="kicker">Platform Development</div>
47+
<h2>What should I build next for the platform?</h2>
48+
</div>
49+
<p>Tools Progress is an Admin-only planning surface for platform-development readiness. Project completion belongs in Project Build Path, not here.</p>
50+
<div class="table-wrapper">
51+
<table class="data-table">
52+
<caption>Platform Tooling Progress</caption>
53+
<thead>
54+
<tr>
55+
<th>Area</th>
56+
<th>Status</th>
57+
<th>Next Platform Build Item</th>
58+
</tr>
59+
</thead>
60+
<tbody>
61+
<tr>
62+
<td>Project Workspace</td>
63+
<td>In Progress</td>
64+
<td>Keep Project Build Path handoff stable.</td>
65+
</tr>
66+
<tr>
67+
<td>Game Design</td>
68+
<td>In Progress</td>
69+
<td>Preserve handoff into Game Configuration.</td>
70+
</tr>
71+
<tr>
72+
<td>Game Configuration</td>
73+
<td>In Progress</td>
74+
<td>Prepare Assets implementation readiness.</td>
75+
</tr>
76+
<tr>
77+
<td>Assets</td>
78+
<td>Not Started</td>
79+
<td>Start the next first-class tool rebuild.</td>
80+
</tr>
81+
</tbody>
82+
</table>
83+
</div>
84+
<div class="status" role="status">Admin Tools Progress is platform planning only. It does not replace Project Build Path.</div>
85+
</div>
86+
</div>
87+
</div>
88+
</section>
89+
</main>
90+
<div data-partial="footer"></div>
91+
<script src="assets/theme-v2/js/gamefoundry-partials.js" defer></script>
92+
</body>
93+
94+
</html>

assets/theme-v2/js/gamefoundry-partials.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -78,6 +78,7 @@
7878
"admin-roles": "admin/roles.html",
7979
"admin-moderation": "admin/moderation.html",
8080
"admin-analytics": "admin/analytics.html",
81+
"admin-tools-progress": "admin/tools-progress.html",
8182
"cookie-policy": "legal/cookie-policy.html",
8283
disclaimer: "legal/disclaimer.html",
8384
"privacy-policy": "legal/privacy-policy.html",

assets/theme-v2/partials/header-nav.html

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -122,6 +122,7 @@
122122
<a data-nav-link data-route="admin-roles" href="admin/roles.html">Roles</a>
123123
<a data-nav-link data-route="admin-site-settings" href="admin/site-settings.html">Site Settings</a>
124124
<a data-nav-link data-route="admin-themes" href="admin/themes.html">Themes</a>
125+
<a data-nav-link data-route="admin-tools-progress" href="admin/tools-progress.html">Tools Progress</a>
125126
<a data-nav-link data-route="admin-users" href="admin/users.html">Users</a>
126127
</div>
127128
</div>
Lines changed: 21 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,21 @@
1+
# Admin Tools Progress Rename
2+
3+
PR: PR_26155_090-admin-tools-progress-rename
4+
5+
## Summary
6+
- Added Admin navigation entry `Tools Progress`.
7+
- Added `admin/tools-progress.html` as a Theme V2 Admin planning page.
8+
- Added `admin-tools-progress` to shared route rewriting in `assets/theme-v2/js/gamefoundry-partials.js`.
9+
- No `Admin -> Project Progress` navigation exists in the affected shared header.
10+
11+
## Purpose
12+
- Tools Progress is platform-development progress only.
13+
- Tools Progress answers: `What should I build next for the platform?`
14+
- Project completion remains separate under Project Build Path.
15+
16+
## Validation Notes
17+
- Targeted Admin navigation checks are covered by `tests/playwright/tools/BuildPathProgressSimplification.spec.mjs`.
18+
- `npm run test:workspace-v2` passed because shared header navigation changed.
19+
- Manual test notes: open `admin/tools-progress.html`, confirm the page loads, header Admin submenu includes Tools Progress, and Project Progress is absent from Admin navigation.
20+
- No page-local CSS, tool-local CSS, inline styles, style blocks, or inline event handlers were added.
21+
- Theme V2 gap findings: none.
Lines changed: 24 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,24 @@
1+
# Build Path Progress Simplification
2+
3+
PR: PR_26155_088-build-path-progress-simplification
4+
5+
## Summary
6+
- Removed the separate Toolbox `Progress` view control from `toolbox/index.html`.
7+
- Removed the Progress render path from `toolbox/tools-page-accordions.js`.
8+
- Preserved progress data/model fields that Build Path still consumes.
9+
- Build Path now carries the project workflow guidance and project completion summary.
10+
11+
## Current Behavior
12+
- Project workflow guidance lives under Toolbox `Build Path`.
13+
- Build Path answers: `What should I do next?`
14+
- Build Path shows active project, project completion, publishing progress, current focus, and top-to-bottom workflow guidance.
15+
- No `Project -> Progress` navigation remains in the affected active Toolbox surface.
16+
17+
## Validation Notes
18+
- Impacted lane: `build-path`.
19+
- Additional affected lanes run: `workspace-contract` through legacy command `npm run test:workspace-v2`, `project-workspace`, `game-configuration`, and `game-design`.
20+
- `npm run test:workspace-v2` was run because shared header/navigation wiring changed. The command name is legacy; user-facing naming remains Project Workspace.
21+
- Skipped lanes: engine, games, samples, integration, old archive material.
22+
- Skipped-lane rationale: no engine, game runtime, sample JSON, or archive behavior changed.
23+
- Manual test notes: open `toolbox/index.html?role=user`, confirm `Progress` is absent, click `Build Path`, and confirm the guidance summary and workflow table render without console errors.
24+
- Theme V2 gap findings: none.
Lines changed: 31 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,31 @@
1+
# Build Path Status Table
2+
3+
PR: PR_26155_089-build-path-status-table
4+
5+
## Summary
6+
- Converted Build Path from accordion/card groups into a workflow-order table.
7+
- The table columns are:
8+
- Order
9+
- Tool
10+
- Status
11+
- Complete
12+
- Workflow rows render in explicit build order, not alphabetical order.
13+
14+
## Status Indicators
15+
- 🟢 Complete
16+
- 🟡 In Progress
17+
- 🔴 Not Started
18+
- ⚪ N/A
19+
20+
## Rules Applied
21+
- Users work top-to-bottom and left-to-right.
22+
- `Publish` is never allowed to render as `N/A`.
23+
- `N/A` appears only for non-required tools in focused contributor views, such as a role-focused Audio Creator view.
24+
- Project Workspace completion is derived from active project presence.
25+
26+
## Validation Notes
27+
- `npm run test:lane:build-path` passed with 3 tests.
28+
- Targeted checks verified table headers, row order, status text, completion values, contributor `N/A`, and `Publish` never rendering as `N/A`.
29+
- Combined narrow coverage pass also passed with 11 tests across Build Path, Game Configuration, and Game Design.
30+
- Manual test notes: open `toolbox/index.html?role=user&memberRole=Audio%20Creator`, click Build Path, confirm non-required rows can show `⚪ N/A` while Publish remains a required status row.
31+
- Theme V2 gap findings: none; existing `table-wrapper` and `data-table` classes were sufficient.
Lines changed: 38 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,38 @@
1+
# Build Path Targeted MSJ Tests
2+
3+
PR: PR_26155_091-build-path-targeted-msj-tests
4+
5+
## Test Coverage Added
6+
- Added `tests/playwright/tools/BuildPathProgressSimplification.spec.mjs`.
7+
- Added `test:lane:build-path` to `package.json`.
8+
- Registered the `build-path` lane in `scripts/run-targeted-test-lanes.mjs`.
9+
10+
## Behaviors Covered
11+
- Project Progress view/control is removed.
12+
- Project Build Path remains available.
13+
- Admin Tools Progress exists.
14+
- Admin Project Progress is absent.
15+
- Build Path table renders Order, Tool, Status, and Complete columns.
16+
- Publish never renders as `N/A`.
17+
- `N/A` renders for non-required tools in a contributor-focused Build Path view.
18+
- No duplicate Admin is exposed under Toolbox.
19+
- No console errors are emitted during targeted checks.
20+
21+
## Validation Run
22+
- `npm run test:lane:build-path`: PASS, 3 tests.
23+
- `npm run test:workspace-v2`: PASS, 4 tests. Legacy command name; user-facing naming remains Project Workspace.
24+
- `npm run test:lane:project-workspace`: PASS, 8 tests.
25+
- `npm run test:lane:game-configuration`: PASS, 4 tests.
26+
- `npm run test:lane:game-design`: PASS, 4 tests.
27+
- Combined narrow coverage pass: PASS, 11 tests.
28+
- `git diff --check`: PASS.
29+
30+
## Skipped Lanes
31+
- Engine, games, samples, integration, archive validation, and full samples smoke were skipped.
32+
- Skipped-lane rationale: this bundle changes active Toolbox Build Path rendering, shared Admin navigation, targeted tests, and generated validation reports only.
33+
34+
## Manual Test Notes
35+
- Open `toolbox/index.html?role=user`, confirm `Progress` is gone and `Build Path` remains.
36+
- Click Build Path and confirm the workflow table and completion summary render.
37+
- Open `toolbox/index.html?role=user&memberRole=Audio%20Creator`, confirm a non-required row can show `⚪ N/A` and Publish does not.
38+
- Open `admin/tools-progress.html`, confirm Tools Progress is platform-development planning only.

docs_build/dev/reports/coverage_changed_js_guardrail.txt

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,10 +6,9 @@ Missing changed runtime JS files are WARN, not FAIL.
66
Source: Playwright/Chromium built-in V8 coverage from the active Playwright run.
77

88
Changed runtime JS files considered:
9-
(89%) toolbox/tools-page-accordions.js - executed lines 1692/1692; executed functions 51/57
9+
(89%) toolbox/tools-page-accordions.js - executed lines 1790/1790; executed functions 57/64
1010
(94%) toolbox/game-configuration/game-configuration-mock-repository.js - executed lines 283/283; executed functions 31/33
1111
(100%) toolbox/game-configuration/game-configuration.js - executed lines 168/168; executed functions 18/18
12-
(100%) toolbox/game-design/game-design.js - executed lines 224/224; executed functions 21/21
1312

1413
Guardrail warnings:
1514
(100%) none - no changed runtime JS coverage warnings

docs_build/dev/reports/dependency_gating_report.md

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
# Dependency Gating Report
22

3-
Generated: 2026-06-04T15:40:53.232Z
3+
Generated: 2026-06-05T00:18:28.233Z
44
Status: PASS
55

66
## Gate Order
@@ -14,7 +14,11 @@ Status: PASS
1414

1515
| Lane | Selected | Status | Dependencies | Affected Surface | Reason |
1616
| --- | --- | --- | --- | --- | --- |
17-
| workspace-contract | Yes | PASS | none | Root tools future-state navigation and Tool Template V2 contract | Lane has no lane dependencies and is eligible after preflight and compilation pass. |
17+
| workspace-contract | No | SKIP | none | Root tools future-state navigation and Tool Template V2 contract | Lane was not selected, so dependency-gated runtime scheduling skipped it. |
18+
| 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. |
19+
| game-design | Yes | PASS | none | Game Design mock repository, project purpose flow, validation overlay, capability demo authoring, and Toolbox progress handoff | Lane has no lane dependencies and is eligible after preflight and compilation pass. |
20+
| 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. |
21+
| 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. |
1822
| tool-runtime | No | SKIP | none | Active public toolbox and Tool Template V2 contract | Lane was not selected, so dependency-gated runtime scheduling skipped it. |
1923
| game-runtime | No | SKIP | none | Deprecated archive/v1-v2/games reference coverage | Lane was not selected, so dependency-gated runtime scheduling skipped it. |
2024
| integration | No | SKIP | none | Integration handoff behavior | Lane was not selected, so dependency-gated runtime scheduling skipped it. |

docs_build/dev/reports/dependency_hydration_reuse_report.md

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,22 +1,22 @@
11
# Dependency Hydration Reuse Report
22

3-
Generated: 2026-06-04T15:40:53.233Z
3+
Generated: 2026-06-05T00:18:28.233Z
44
Status: PASS
55

66
## Summary
77

8-
Reused dependency hydration: 1
8+
Reused dependency hydration: 0
99
Invalidated dependency hydration: 0
10-
Generated dependency hydration: 0
11-
Prevented dependency graph hydration: 1
12-
Prevented helper resolution passes: 4
10+
Generated dependency hydration: 1
11+
Prevented dependency graph hydration: 0
12+
Prevented helper resolution passes: 0
1313
Prevented fixture ownership traversal: 0
1414

1515
## Hydration Decisions
1616

1717
| Lane | Status | Helpers | Fixtures | Imports | Dependency Hydration Hash | Reason |
1818
| --- | --- | --- | --- | --- | --- | --- |
19-
| workspace-contract | REUSED | 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 | 0ff8fc92c789456d | Dependency hydration reused from validated warm-start state. |
19+
| game-design | GENERATED | 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 | 1a5eda0dbb35dbd2 | Dependency hydration was recorded for future reuse. |
2020

2121
## Safeguards
2222

0 commit comments

Comments
 (0)