Skip to content

Commit 11c0867

Browse files
author
DavidQ
committed
PR 8.16: First-class tool inventory and remaining work
- Confirmed all 17 first-class tools are in scope - Added remaining-work checklist - Added inventory template for canonical tool/schema coverage - Preserved docs-only workflow
1 parent b2ad89e commit 11c0867

5 files changed

Lines changed: 129 additions & 18 deletions

‎docs/dev/codex_commands.md‎

Lines changed: 21 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -1,20 +1,27 @@
11
MODEL: GPT-5.3-codex
2-
REASONING: low
2+
REASONING: medium
33

44
TASK:
5-
Document-only PR.
5+
Apply PR 8.16: document remaining work and canonical first-class tool inventory.
66

7-
- Do NOT rename files
8-
- Do NOT modify payloads
9-
- Do NOT modify runtime
10-
11-
Verify:
12-
- sample.<id>.<tool>.json are tool payloads
13-
- sample.<id>.palette.json are palette data files
14-
15-
Ensure:
16-
- no new invalid naming patterns introduced
7+
STEPS:
8+
1. Inspect the repo's tool folders and schemas.
9+
2. Create/update a canonical 17-tool inventory document.
10+
3. Fill all 17 rows with:
11+
- tool id
12+
- display name
13+
- folder path
14+
- schema path
15+
- palette/data usage
16+
4. Verify each first-class tool has exactly one schema.
17+
5. Do not invent tools. Use the repo's actual first-class tool set.
18+
6. Keep this PR docs/inventory only.
19+
7. Do NOT modify runtime logic.
20+
8. Do NOT add validators.
21+
9. Do NOT modify start_of_day.
1722

1823
ACCEPTANCE:
19-
- Naming rules clearly documented
20-
- No file changes required
24+
- 17 first-class tools are listed.
25+
- Tool schema paths are explicit.
26+
- Remaining work checklist is updated.
27+
- No runtime/start_of_day changes.

‎docs/dev/commit_comment.txt‎

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
1-
PR 8.15: Naming convention standardization
1+
PR 8.16: First-class tool inventory and remaining work
22

3-
- Clarified difference between tool payload and palette data files
4-
- Locked naming conventions for future consistency
5-
- No file changes required
3+
- Confirmed all 17 first-class tools are in scope
4+
- Added remaining-work checklist
5+
- Added inventory template for canonical tool/schema coverage
6+
- Preserved docs-only workflow

‎docs/pr/PR_8.16_FIRST_CLASS_TOOL_INVENTORY_AND_REMAINING_WORK.md‎

Lines changed: 51 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,51 @@
1+
# PR 8.16 - First-Class Tool Inventory And Remaining Work
2+
3+
## Purpose
4+
Provide one canonical, docs-only inventory for the repo's first-class tool set and record remaining schema alignment work.
5+
6+
## Sources Used
7+
- `tools/toolRegistry.js` (`getVisibleActiveToolRegistry()` for canonical 17-tool set)
8+
- `tools/<tool folder>/...` (folder existence)
9+
- `tools/schemas/tools/*.schema.json` (tool schema inventory)
10+
- `samples/**/sample.<id>.<tool>.json` (observed payload/config usage hints)
11+
12+
## Canonical First-Class Tool Inventory (17)
13+
14+
| # | Tool ID | Display Name | Folder Path | Schema Path | Palette Usage | Data Usage | Schema Count |
15+
| --- | --- | --- | --- | --- | --- | --- | --- |
16+
| 1 | `vector-map-editor` | Vector Map Editor | `tools/Vector Map Editor` | `tools/schemas/tools/vector-map-editor.schema.json` | Optional/shared | `vectorMapDocument` | 1 |
17+
| 2 | `vector-asset-studio` | Vector Asset Studio | `tools/Vector Asset Studio` | `tools/schemas/tools/vector-asset-studio.schema.json` | Optional/shared | `vectorAssetDocument` | 1 |
18+
| 3 | `tile-map-editor` | Tilemap Studio | `tools/Tilemap Studio` | `tools/schemas/tools/tile-map-editor.schema.json` | Optional/shared | `tileMapDocumentPath`, `tilemapDocumentPath`, `parallaxDocument` | 1 |
19+
| 4 | `parallax-editor` | Parallax Scene Studio | `tools/Parallax Scene Studio` | `tools/schemas/tools/parallax-editor.schema.json` | Optional/shared | `parallaxDocument`, `tilemapDocumentPath`, `vectorAssetSvgPath` | 1 |
20+
| 5 | `sprite-editor` | Sprite Editor | `tools/Sprite Editor` | `tools/schemas/tools/sprite-editor.schema.json` | Yes | `spriteProject` | 1 |
21+
| 6 | `skin-editor` | Primitive Skin Editor | `tools/Skin Editor` | `MISSING (expected: tools/schemas/tools/skin-editor.schema.json)` | Yes (color presets) | Skin preset config (no sample payload lane currently) | 0 |
22+
| 7 | `asset-browser` | Asset Browser / Import Hub | `tools/Asset Browser` | `tools/schemas/tools/asset-browser.schema.json` | Yes (browse/reference) | `assetBrowserPreset` | 1 |
23+
| 8 | `palette-browser` | Palette Browser / Manager | `tools/Palette Browser` | `tools/schemas/tools/palette-browser.schema.json` | Yes (primary) | Palette dataset/swatches | 1 |
24+
| 9 | `state-inspector` | State Inspector | `tools/State Inspector` | `tools/schemas/tools/state-inspector.schema.json` | No | `snapshot` | 1 |
25+
| 10 | `replay-visualizer` | Replay Visualizer | `tools/Replay Visualizer` | `tools/schemas/tools/replay-visualizer.schema.json` | No | `events` timeline | 1 |
26+
| 11 | `performance-profiler` | Performance Profiler | `tools/Performance Profiler` | `tools/schemas/tools/performance-profiler.schema.json` | No | `profileSettings` | 1 |
27+
| 12 | `physics-sandbox` | Physics Sandbox | `tools/Physics Sandbox` | `tools/schemas/tools/physics-sandbox.schema.json` | No | `physicsBody` | 1 |
28+
| 13 | `asset-pipeline-tool` | Asset Pipeline Tool | `tools/Asset Pipeline Tool` | `tools/schemas/tools/asset-pipeline-tool.schema.json` | No direct palette payload | `pipelinePayload` | 1 |
29+
| 14 | `tile-model-converter` | Tile Model Converter | `tools/Tile Model Converter` | `tools/schemas/tools/tile-model-converter.schema.json` | No direct palette payload | `candidate`, `conversion` | 1 |
30+
| 15 | `3d-json-payload-normalizer` | 3D JSON Payload Normalizer | `tools/3D JSON Payload Normalizer` | `tools/schemas/tools/3d-json-payload-normalizer.schema.json` | No | `mapPayload` | 1 |
31+
| 16 | `3d-asset-viewer` | 3D Asset Viewer | `tools/3D Asset Viewer` | `tools/schemas/tools/3d-asset-viewer.schema.json` | No | `asset3d` | 1 |
32+
| 17 | `3d-camera-path-editor` | 3D Camera Path Editor | `tools/3D Camera Path Editor` | `tools/schemas/tools/3d-camera-path-editor.schema.json` | No | `cameraPath` | 1 |
33+
34+
## Verification Summary
35+
- First-class tools listed: 17/17.
36+
- Tool folders present: 17/17.
37+
- Tools with exactly one `<toolId>.schema.json`: 16/17.
38+
- Gap: `skin-editor` has no `tools/schemas/tools/skin-editor.schema.json`.
39+
40+
## Remaining Work Checklist (Updated)
41+
- [x] Canonical 17-tool inventory documented.
42+
- [x] Tool folder paths verified for all 17.
43+
- [x] Explicit schema paths recorded per tool.
44+
- [ ] Close schema gap for `skin-editor` by adding `tools/schemas/tools/skin-editor.schema.json`.
45+
- [ ] Re-run first-class schema count check and confirm 17/17 have exactly one schema.
46+
47+
## Scope Guard
48+
- Docs/inventory only.
49+
- No runtime logic changes.
50+
- No validator additions.
51+
- No `start_of_day` changes.

‎docs/pr/first_class_tool_inventory_template.md‎

Lines changed: 23 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,23 @@
1+
# First-Class Tool Inventory Template
2+
3+
Use this document to lock the 17-tool list.
4+
5+
| # | Tool ID | Display Name | Tool Folder | Schema Path | Uses Palette | Uses Asset/Data Object | Notes |
6+
|---|---|---|---|---|---|---|---|
7+
| 1 | palette-browser | Palette Browser | tools/palette-browser | tools/schemas/tools/palette-browser.schema.json | yes | palette | |
8+
| 2 | sprite-editor | Sprite Editor | tools/sprite-editor | tools/schemas/tools/sprite-editor.schema.json | yes | sprite/palette | |
9+
| 3 | vector-map-editor | Vector Map Editor | tools/vector-map-editor | tools/schemas/tools/vector-map-editor.schema.json | yes | map/palette | |
10+
| 4 | vector-asset-studio | Vector Asset Studio | tools/vector-asset-studio | tools/schemas/tools/vector-asset-studio.schema.json | yes | vector assets | |
11+
| 5 | TBD | TBD | TBD | TBD | TBD | TBD | fill from repo |
12+
| 6 | TBD | TBD | TBD | TBD | TBD | TBD | fill from repo |
13+
| 7 | TBD | TBD | TBD | TBD | TBD | TBD | fill from repo |
14+
| 8 | TBD | TBD | TBD | TBD | TBD | TBD | fill from repo |
15+
| 9 | TBD | TBD | TBD | TBD | TBD | TBD | fill from repo |
16+
| 10 | TBD | TBD | TBD | TBD | TBD | TBD | fill from repo |
17+
| 11 | TBD | TBD | TBD | TBD | TBD | TBD | fill from repo |
18+
| 12 | TBD | TBD | TBD | TBD | TBD | TBD | fill from repo |
19+
| 13 | TBD | TBD | TBD | TBD | TBD | TBD | fill from repo |
20+
| 14 | TBD | TBD | TBD | TBD | TBD | TBD | fill from repo |
21+
| 15 | TBD | TBD | TBD | TBD | TBD | TBD | fill from repo |
22+
| 16 | TBD | TBD | TBD | TBD | TBD | TBD | fill from repo |
23+
| 17 | TBD | TBD | TBD | TBD | TBD | TBD | fill from repo |

‎docs/pr/remaining_work_checklist.md‎

Lines changed: 29 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,29 @@
1+
# Remaining Work Checklist
2+
3+
## First-Class Tool Coverage
4+
- [x] Create canonical list of 17 first-class tools.
5+
- [x] Verify each tool folder exists.
6+
- [ ] Verify each tool has exactly one schema. (Current status: 16/17; missing `skin-editor` schema.)
7+
- [ ] Verify each schema path follows `tools/schemas/tools/<tool>.schema.json`. (Blocked by missing `tools/schemas/tools/skin-editor.schema.json`.)
8+
9+
## Sample Payload Cleanup
10+
- [ ] Verify all sample tool files use `sample.<id>.<tool>.json`.
11+
- [ ] Verify no old `sample-<id>-<tool>.json` payload files remain except intentional `*-document.json`.
12+
- [ ] Verify each sample tool file has `$schema`, `tool`, `version`, and `config`.
13+
- [ ] Verify no sample tool payload embeds palette data.
14+
15+
## Palette/Data Cleanup
16+
- [ ] Verify palette files are named `sample.<id>.palette.json`.
17+
- [ ] Verify palette files use `swatches`.
18+
- [ ] Verify each swatch has single-character `symbol`.
19+
- [ ] Verify locked/source metadata lives in palette JSON only.
20+
21+
## Workspace/Game Alignment
22+
- [ ] Verify workspace/game `tools[]` entries match sample tool payload shape.
23+
- [ ] Verify workspace schema does not contain sample-only fields.
24+
- [ ] Verify sample concepts do not leak into workspace schema.
25+
26+
## Docs
27+
- [ ] Document launcher pairing contract.
28+
- [ ] Document samples as one-off tool launch files.
29+
- [ ] Document games/workspaces as manifest-owned collections.

0 commit comments

Comments
 (0)