Skip to content

Commit e828bf5

Browse files
ToolboxAidToolboxAid
committed
Use the individual commit comment generated for each PR. Do not squash PR_26152_119 through PR_26152_123 into a single commit.
1 parent 2bafab4 commit e828bf5

5 files changed

Lines changed: 416 additions & 0 deletions

‎docs/dev/reports/tool_migration_inventory.md‎

Lines changed: 116 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,116 @@
1+
# Tool Migration Inventory
2+
3+
PR: PR_26152_119-tool-migration-inventory
4+
Date: 2026-06-02
5+
6+
## Scope
7+
8+
- Inventoried migrated vs unmigrated tools.
9+
- Marked unmigrated tools as SKIP / not migrated.
10+
- Did not classify unmigrated tools as failures.
11+
- Did not modify tool runtime code.
12+
- Did not touch samples.
13+
14+
## Sources
15+
16+
- `src/shared/contracts/tools/toolContractsIndex.js`
17+
- `tools/toolRegistry.js`
18+
- `tools/workspace-manager-v2/js/services/WorkspaceManagerV2ContextService.js`
19+
20+
## Summary
21+
22+
- First-class tool contracts: 34.
23+
- Active tool registry entries: 23.
24+
- Current ProjectWorkspace child-launchable tools: 11.
25+
- ProjectWorkspace host/bootstrap tool: 1.
26+
- Active registry tools not yet ProjectWorkspace child-launchable: 11, SKIP / not migrated.
27+
- Contract-only backlog tools not active in the current registry: 11, SKIP / not migrated.
28+
- `templates-v2` appears in the ProjectWorkspace launch list as a support template, not a first-class tool contract.
29+
30+
## ProjectWorkspace Migrated / Launchable
31+
32+
| Tool ID | Status | Notes |
33+
| --- | --- | --- |
34+
| `asset-manager-v2` | migrated / ProjectWorkspace launchable | Current launch list includes explicit ProjectWorkspace child launch path. |
35+
| `palette-manager-v2` | migrated / ProjectWorkspace launchable | Current launch list includes explicit ProjectWorkspace child launch path. |
36+
| `object-vector-studio-v2` | migrated / ProjectWorkspace launchable | Current launch list includes explicit ProjectWorkspace child launch path. |
37+
| `world-vector-studio-v2` | migrated / ProjectWorkspace launchable | Current launch list includes explicit ProjectWorkspace child launch path. |
38+
| `collision-inspector-v2` | migrated / ProjectWorkspace launchable | Current launch list includes explicit ProjectWorkspace child launch path. |
39+
| `input-mapping-v2` | migrated / ProjectWorkspace launchable | Current launch list includes explicit ProjectWorkspace child launch path. |
40+
| `preview-generator-v2` | migrated / ProjectWorkspace launchable | Current launch list includes explicit ProjectWorkspace child launch path. |
41+
| `text2speech-V2` | migrated / ProjectWorkspace launchable | Current launch list includes explicit ProjectWorkspace child launch path. |
42+
| `audio-sfx-playground-v2` | migrated / ProjectWorkspace launchable | Current launch list includes explicit ProjectWorkspace child launch path. |
43+
| `midi-studio-v2` | migrated / ProjectWorkspace launchable | Current launch list includes explicit ProjectWorkspace child launch path. |
44+
| `storage-inspector-v2` | migrated / ProjectWorkspace launchable | Current launch list includes explicit ProjectWorkspace child launch path. |
45+
46+
## ProjectWorkspace Host / Support
47+
48+
| Tool ID | Status | Notes |
49+
| --- | --- | --- |
50+
| `workspace-manager-v2` | ProjectWorkspace host/bootstrap | Coordinates ProjectWorkspace launch context; not treated as a child-launched tool. |
51+
| `templates-v2` | support template | Launchable template surface; not a first-class tool contract. |
52+
53+
## Active Registry Tools Not Yet Migrated
54+
55+
These tools are SKIP / not migrated / out of scope for this inventory PR. They are not failures.
56+
57+
| Tool ID | Status | Notes |
58+
| --- | --- | --- |
59+
| `tile-map-editor` | SKIP / not migrated | Active registry entry, not in current ProjectWorkspace launch list. |
60+
| `parallax-editor` | SKIP / not migrated | Active registry entry, not in current ProjectWorkspace launch list. |
61+
| `sprite-editor` | SKIP / not migrated | Active registry entry, not in current ProjectWorkspace launch list. |
62+
| `state-inspector` | SKIP / not migrated | Active registry entry, not in current ProjectWorkspace launch list. |
63+
| `replay-visualizer` | SKIP / not migrated | Active registry entry, not in current ProjectWorkspace launch list. |
64+
| `performance-profiler` | SKIP / not migrated | Active registry entry, not in current ProjectWorkspace launch list. |
65+
| `physics-sandbox` | SKIP / not migrated | Active registry entry, not in current ProjectWorkspace launch list. |
66+
| `asset-pipeline` | SKIP / not migrated | Active registry entry, not in current ProjectWorkspace launch list. |
67+
| `3d-json-payload` | SKIP / not migrated | Active registry entry, not in current ProjectWorkspace launch list. |
68+
| `3d-asset-viewer` | SKIP / not migrated | Active registry entry, not in current ProjectWorkspace launch list. |
69+
| `3d-camera-path-editor` | SKIP / not migrated | Active registry entry, not in current ProjectWorkspace launch list. |
70+
71+
## Contract-Only Backlog
72+
73+
These first-class tool contracts are SKIP / not migrated until a future PR activates them in the current registry or explicitly scopes them into a migration wave. They are not failures.
74+
75+
| Tool ID | Status |
76+
| --- | --- |
77+
| `asset-studio` | SKIP / not migrated |
78+
| `game-builder` | SKIP / not migrated |
79+
| `game-design-studio` | SKIP / not migrated |
80+
| `publish-studio` | SKIP / not migrated |
81+
| `animation-studio` | SKIP / not migrated |
82+
| `particle-studio` | SKIP / not migrated |
83+
| `sound-studio` | SKIP / not migrated |
84+
| `ai-assistant` | SKIP / not migrated |
85+
| `code-studio` | SKIP / not migrated |
86+
| `input-studio` | SKIP / not migrated |
87+
| `localization-studio` | SKIP / not migrated |
88+
89+
## Validation
90+
91+
Command:
92+
93+
```powershell
94+
git diff --check
95+
```
96+
97+
Result: PASS.
98+
99+
## Lanes Executed
100+
101+
- docs/report inventory only.
102+
103+
## Lanes Skipped
104+
105+
- runtime - no tool runtime code changed.
106+
- tool runtime tests - not required and not run.
107+
- samples - SKIP / pending rebuild.
108+
- Playwright - not impacted.
109+
110+
## Samples Decision
111+
112+
SKIP / pending rebuild. No samples were touched.
113+
114+
## Tools Decision
115+
116+
Unmigrated tools are SKIP / not migrated / out of scope and are not failures.

‎docs/dev/reports/tool_migration_lane_bootstrap.md‎

Lines changed: 84 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,84 @@
1+
# Tool Migration Lane Bootstrap
2+
3+
PR: PR_26152_123-tool-migration-lane-bootstrap
4+
Date: 2026-06-02
5+
6+
## Scope
7+
8+
- Bootstrapped tool migration lane governance.
9+
- Defined how future tool migration PRs validate migrated tools only.
10+
- Kept samples as SKIP / pending rebuild.
11+
- Kept unmigrated tools as SKIP / out of scope.
12+
- Used ProjectWorkspace terminology.
13+
14+
## Future Tool Migration PR Rules
15+
16+
- Name the exact tool or tools in scope.
17+
- Validate only those migrated tools.
18+
- Keep all other unmigrated tools SKIP / not migrated / out of scope.
19+
- Do not classify unmigrated tools as failures.
20+
- Require explicit ProjectWorkspace launch inputs.
21+
- Require manifest handoff to use declared fields only.
22+
- Require Tool State to remain the saved editing source.
23+
- Require ProjectWorkspace to remain coordination-only.
24+
- Require palette and asset references to stay explicit.
25+
- Reject hidden fallback data, sample JSON, `localStorage`, or `sessionStorage` assumptions.
26+
27+
## Future Validation Ladder
28+
29+
Use only the lanes named by the future migration PR:
30+
31+
1. `git diff --check`
32+
2. ProjectWorkspace launch boundary validation for the migrated tool.
33+
3. ProjectWorkspace manifest handoff boundary validation for the migrated tool.
34+
4. ProjectWorkspace state/lifecycle boundary validation for the migrated tool.
35+
5. Targeted tool runtime test only when the future PR explicitly requires runtime validation for that migrated tool.
36+
37+
Do not use tool runtime tests as blockers for tools that are not in scope.
38+
39+
## Samples Rule
40+
41+
Samples remain SKIP / pending rebuild until a dedicated sample rebuild PR explicitly scopes them.
42+
43+
Future tool migration PRs must not use sample JSON as hidden fallback data.
44+
45+
## Report Rule
46+
47+
Future tool migration reports should state:
48+
49+
- exact migrated tools
50+
- exact skipped tools
51+
- lanes executed
52+
- lanes skipped
53+
- samples decision
54+
- whether runtime validation was in scope
55+
- whether ProjectWorkspace launch/manifest/state/lifecycle boundaries passed
56+
57+
## Validation
58+
59+
Command:
60+
61+
```powershell
62+
git diff --check
63+
```
64+
65+
Result: PASS.
66+
67+
## Lanes Executed
68+
69+
- docs/report lane bootstrap only.
70+
71+
## Lanes Skipped
72+
73+
- runtime - no tool runtime code changed.
74+
- tool runtime tests - not required and not run.
75+
- samples - SKIP / pending rebuild.
76+
- Playwright - not impacted.
77+
78+
## Samples Decision
79+
80+
SKIP / pending rebuild. No samples were touched.
81+
82+
## Tools Decision
83+
84+
Future migration PRs validate migrated tools only. Unmigrated tools remain SKIP / out of scope.

‎docs/dev/reports/tool_migration_prioritization.md‎

Lines changed: 80 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,80 @@
1+
# Tool Migration Prioritization
2+
3+
PR: PR_26152_120-tool-migration-prioritization
4+
Date: 2026-06-02
5+
6+
## Scope
7+
8+
- Defined tool migration priority order.
9+
- Grouped tools by dependency, risk, and ProjectWorkspace readiness.
10+
- Did not fix tools.
11+
- Did not run tool runtime tests as blockers.
12+
13+
## Prioritization Rules
14+
15+
- Prefer tools with read-only or explicit input/output boundaries before content editors.
16+
- Prefer tools with existing V2 folders and schema references before older folder-based tools.
17+
- Prefer low ProjectWorkspace payload ownership risk before tools that own assets, palettes, maps, or publishable content.
18+
- Keep currently ProjectWorkspace-launchable tools out of migration blocker lists.
19+
- Keep unmigrated tools as SKIP / not migrated until their exact PR names them.
20+
21+
## Already ProjectWorkspace Launchable
22+
23+
No migration priority is assigned to these tools in this lane because they are already in the current ProjectWorkspace child launch list:
24+
25+
- `asset-manager-v2`
26+
- `palette-manager-v2`
27+
- `object-vector-studio-v2`
28+
- `world-vector-studio-v2`
29+
- `collision-inspector-v2`
30+
- `input-mapping-v2`
31+
- `preview-generator-v2`
32+
- `text2speech-V2`
33+
- `audio-sfx-playground-v2`
34+
- `midi-studio-v2`
35+
- `storage-inspector-v2`
36+
37+
`workspace-manager-v2` remains the ProjectWorkspace host/bootstrap surface.
38+
39+
## Priority Order
40+
41+
| Priority | Tools | Dependency / Risk / Readiness |
42+
| --- | --- | --- |
43+
| Wave 1 | `state-inspector`, `replay-visualizer`, `performance-profiler`, `physics-sandbox`, `3d-json-payload`, `3d-asset-viewer` | Lower write risk, mostly viewer/inspector/utility surfaces, suitable for explicit ProjectWorkspace launch inputs and boundary validation. |
44+
| Wave 2 | `tile-map-editor`, `parallax-editor`, `sprite-editor`, `asset-pipeline`, `3d-camera-path-editor` | Higher content ownership and Tool State/payload risk; requires stricter save/open/palette/asset boundaries. |
45+
| Future activation backlog | `asset-studio`, `game-builder`, `game-design-studio`, `publish-studio`, `animation-studio`, `particle-studio`, `sound-studio`, `ai-assistant`, `code-studio`, `input-studio`, `localization-studio` | Contracted planning surfaces not active in the current registry; scope when a future PR explicitly activates or migrates them. |
46+
47+
## Non-Blockers
48+
49+
- Tool runtime tests are not blockers for this prioritization PR.
50+
- Unmigrated tools are not failures.
51+
- Samples remain SKIP / pending rebuild.
52+
53+
## Validation
54+
55+
Command:
56+
57+
```powershell
58+
git diff --check
59+
```
60+
61+
Result: PASS.
62+
63+
## Lanes Executed
64+
65+
- docs/report prioritization only.
66+
67+
## Lanes Skipped
68+
69+
- runtime - no tool runtime code changed.
70+
- tool runtime tests - not required and not run.
71+
- samples - SKIP / pending rebuild.
72+
- Playwright - not impacted.
73+
74+
## Samples Decision
75+
76+
SKIP / pending rebuild. No samples were touched.
77+
78+
## Tools Decision
79+
80+
Only future PRs that name exact tools should validate tool runtime behavior, and only for those migrated tools.

‎docs/dev/reports/tool_migration_wave_1_scope.md‎

Lines changed: 68 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,68 @@
1+
# Tool Migration Wave 1 Scope
2+
3+
PR: PR_26152_121-tool-migration-wave-1-scope
4+
Date: 2026-06-02
5+
6+
## Scope
7+
8+
- Defined Wave 1 migration scope.
9+
- Identified exact tools in Wave 1.
10+
- Defined expected ProjectWorkspace integration boundaries.
11+
- No implementation.
12+
13+
## Wave 1 Tools
14+
15+
| Tool ID | Type | Expected ProjectWorkspace Boundary |
16+
| --- | --- | --- |
17+
| `state-inspector` | inspector | Read explicit ProjectWorkspace/toolState inputs; inspect state without owning or persisting ProjectWorkspace data. |
18+
| `replay-visualizer` | viewer | Read explicit replay payload/toolState input; do not capture or require live runtime event streams. |
19+
| `performance-profiler` | inspector | Read explicit profiling payload/toolState input; do not require live engine runtime UAT in migration PR. |
20+
| `physics-sandbox` | utility | Read explicit physics config/toolState input; save only through Tool State boundary when scoped. |
21+
| `3d-json-payload` | utility | Read explicit JSON payload/toolState input; reject hidden fallback/sample data. |
22+
| `3d-asset-viewer` | viewer | Read explicit asset references/toolState input; remain read-only unless a later PR scopes editing. |
23+
24+
## Expected Validation Boundaries For Future Wave 1 PRs
25+
26+
- Validate migrated tool launch uses explicit ProjectWorkspace inputs.
27+
- Validate manifest handoff consumes declared fields only.
28+
- Validate Tool State is the saved editing source.
29+
- Validate ProjectWorkspace does not persist payloads, fallback data, sample JSON, `localStorage`, or `sessionStorage`.
30+
- Validate only the exact migrated tool or tools named in the PR.
31+
- Keep unmigrated tools SKIP / not migrated / out of scope.
32+
33+
## Out Of Scope
34+
35+
- No tool runtime implementation in this PR.
36+
- No samples.
37+
- No tool runtime tests as blockers.
38+
- No Wave 2 tool migration.
39+
- No contract-only backlog activation.
40+
41+
## Validation
42+
43+
Command:
44+
45+
```powershell
46+
git diff --check
47+
```
48+
49+
Result: PASS.
50+
51+
## Lanes Executed
52+
53+
- docs/report Wave 1 scoping only.
54+
55+
## Lanes Skipped
56+
57+
- runtime - no tool runtime code changed.
58+
- tool runtime tests - not required and not run.
59+
- samples - SKIP / pending rebuild.
60+
- Playwright - not impacted.
61+
62+
## Samples Decision
63+
64+
SKIP / pending rebuild. No samples were touched.
65+
66+
## Tools Decision
67+
68+
Wave 1 tools are scoped for future migration PRs only. They are SKIP / not migrated until those PRs implement and validate them.

0 commit comments

Comments
 (0)