Skip to content

Commit 20a538a

Browse files
author
DavidQ
committed
Complete tool testing documentation and report standardization
- inventoried active tools - added full testing documentation for each active tool - defined manual test cases per tool - standardized validation report format under docs/dev/reports - preserved unrelated working-tree changes PR Details - advances `create full testing documentation for each tool` from [ ] to [x] - advances `define manual test cases per tool` from [ ] to [x] - advances `standardize validation reports under docs/dev/reports` from [ ] to [x]
1 parent c8cb8a9 commit 20a538a

28 files changed

Lines changed: 1175 additions & 52 deletions

docs/README.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,8 @@ This is the root onboarding entry for active project documentation.
1515
4. Tooling docs:
1616
- `docs/operations/dev/POWERSHELL_SCRIPT_STRUCTURE.md`
1717
- `docs/operations/dev/CODEX_WEBSITE_REPO_DEPLOYMENT_SCRIPTING.md`
18+
- `docs/tools/README.md`
19+
- `docs/tools/testing/README.md`
1820
5. PR and capability review surfaces:
1921
- `docs/pr/capabilities/README.md`
2022
6. Active reports:
Lines changed: 23 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,23 @@
1+
# BUILD_PR_LEVEL_21_2_TOOL_TESTING_DOCUMENTATION_AND_REPORT_STANDARDIZATION_TOOL_INVENTORY
2+
3+
- Source of truth: `tools/toolRegistry.js` (`getActiveToolRegistry()`).
4+
- Active tool count: 16
5+
6+
| # | Tool | Tool Id | Entry Point | Testing Doc |
7+
| --- | --- | --- | --- | --- |
8+
| 1 | Vector Map Editor | `vector-map-editor` | `tools/Vector Map Editor/index.html` | `docs/tools/testing/vector-map-editor.md` |
9+
| 2 | Vector Asset Studio | `vector-asset-studio` | `tools/Vector Asset Studio/index.html` | `docs/tools/testing/vector-asset-studio.md` |
10+
| 3 | Tilemap Studio | `tile-map-editor` | `tools/Tilemap Studio/index.html` | `docs/tools/testing/tile-map-editor.md` |
11+
| 4 | Parallax Scene Studio | `parallax-editor` | `tools/Parallax Scene Studio/index.html` | `docs/tools/testing/parallax-editor.md` |
12+
| 5 | Sprite Editor | `sprite-editor` | `tools/Sprite Editor/index.html` | `docs/tools/testing/sprite-editor.md` |
13+
| 6 | Asset Browser / Import Hub | `asset-browser` | `tools/Asset Browser/index.html` | `docs/tools/testing/asset-browser.md` |
14+
| 7 | Palette Browser / Manager | `palette-browser` | `tools/Palette Browser/index.html` | `docs/tools/testing/palette-browser.md` |
15+
| 8 | State Inspector | `state-inspector` | `tools/State Inspector/index.html` | `docs/tools/testing/state-inspector.md` |
16+
| 9 | Replay Visualizer | `replay-visualizer` | `tools/Replay Visualizer/index.html` | `docs/tools/testing/replay-visualizer.md` |
17+
| 10 | Performance Profiler | `performance-profiler` | `tools/Performance Profiler/index.html` | `docs/tools/testing/performance-profiler.md` |
18+
| 11 | Physics Sandbox | `physics-sandbox` | `tools/Physics Sandbox/index.html` | `docs/tools/testing/physics-sandbox.md` |
19+
| 12 | Asset Pipeline Tool | `asset-pipeline-tool` | `tools/Asset Pipeline Tool/index.html` | `docs/tools/testing/asset-pipeline-tool.md` |
20+
| 13 | Tile Model Converter | `tile-model-converter` | `tools/Tile Model Converter/index.html` | `docs/tools/testing/tile-model-converter.md` |
21+
| 14 | 3D Map Editor | `3d-map-editor` | `tools/3D Map Editor/index.html` | `docs/tools/testing/3d-map-editor.md` |
22+
| 15 | 3D Asset Viewer | `3d-asset-viewer` | `tools/3D Asset Viewer/index.html` | `docs/tools/testing/3d-asset-viewer.md` |
23+
| 16 | 3D Camera Path Editor | `3d-camera-path-editor` | `tools/3D Camera Path Editor/index.html` | `docs/tools/testing/3d-camera-path-editor.md` |
Lines changed: 34 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,34 @@
1+
# BUILD_PR_LEVEL_21_2_TOOL_TESTING_DOCUMENTATION_AND_REPORT_STANDARDIZATION_VALIDATION
2+
3+
## Commands Run
4+
1. `node --input-type=module -` (reads `getActiveToolRegistry()` from `tools/toolRegistry.js` and validates per-tool docs + required sections)
5+
2. `Get-ChildItem docs/tools/testing -Filter *.md | Where-Object { $_.Name -ne 'README.md' } | Select-String -Pattern '^## Manual Test Cases' | Measure-Object`
6+
3. `git status --short -- docs/dev/start_of_day`
7+
8+
## Scope Validation Results
9+
- Active tool registry inventory: **PASS** (`16` active tools).
10+
- Per-tool testing docs present: **PASS** (`16/16`).
11+
- Required sections present per tool doc: **PASS** (`missingSectionCount=0`).
12+
- Manual test-case section coverage: **PASS** (`16` docs include `## Manual Test Cases`).
13+
- Standardized template exists: **PASS** (`docs/dev/reports/tool_validation_report_template.md`).
14+
- Tool testing index exists: **PASS** (`docs/tools/testing/README.md`).
15+
16+
## Guardrail Validation
17+
- No `start_of_day` modifications: **PASS**.
18+
- Docs-first only (no runtime/feature implementation work): **PASS**.
19+
- `add automated validation where possible` was intentionally not updated in roadmap: **PASS**.
20+
21+
## Navigation Surfaces Updated
22+
- `docs/README.md`
23+
- `docs/reference/root/README.md`
24+
- `docs/tools/README.md`
25+
- `docs/tools/testing/README.md`
26+
27+
## Roadmap Status Updates (Execution-Backed)
28+
Updated in `docs/dev/roadmaps/MASTER_ROADMAP_TOOLS.md`:
29+
- `create full testing documentation for each tool` `[ ] -> [x]`
30+
- `define manual test cases per tool` `[ ] -> [x]`
31+
- `standardize validation reports under docs/dev/reports` `[ ] -> [x]`
32+
33+
Unchanged by design:
34+
- `add automated validation where possible` remains `[ ]`.
Lines changed: 2 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -1,21 +1,2 @@
1-
BUILD: FINAL REPO CLEANUP + ROADMAP RENAME (V2)
2-
3-
Scope executed:
4-
- scanned for empty directories with explicit exclusion of games/_template/**
5-
- removed empty directories outside template scope
6-
- removed unnecessary .keep files outside template scope
7-
- renamed roadmap file to docs/dev/roadmaps/MASTER_ROADMAP_ENGINE.md
8-
- moved remaining roadmap support docs from docs/roadmaps/ to docs/dev/roadmaps/
9-
- updated references to the renamed/moved roadmap surfaces
10-
11-
Key results:
12-
- removed docs/roadmaps/MASTER_ROADMAP_HIGH_LEVEL.md (replaced by docs/dev/roadmaps/MASTER_ROADMAP_ENGINE.md)
13-
- moved docs/roadmaps/README.md -> docs/dev/roadmaps/README.md
14-
- moved docs/roadmaps/phases.txt -> docs/dev/roadmaps/phases.txt
15-
- docs/roadmaps/ directory removed after migration
16-
- .keep files outside template: none found
17-
- empty directories outside template after cleanup: 0
18-
19-
Notes:
20-
- preserved unrelated working-tree changes
21-
- preserved template structure (no changes under games/_template)
1+
This bundle starts the Tools roadmap testing lane with execution-capable documentation and report standardization work.
2+
It intentionally leaves automated validation for the next PR.
Lines changed: 38 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,38 @@
1+
# tool_validation_report_template
2+
3+
## Tool Metadata
4+
- Tool name:
5+
- Tool id:
6+
- Entry point:
7+
- Validator:
8+
- Date/time:
9+
- Environment (OS/browser/build):
10+
11+
## Run Context
12+
- Branch/worktree context:
13+
- Related BUILD_PR:
14+
- Data set(s) used:
15+
16+
## Cases Executed
17+
| Case ID | Case Name | Result (PASS/FAIL/BLOCKED) | Evidence |
18+
| --- | --- | --- | --- |
19+
| TC-01 | Launch/Boot | | |
20+
| TC-02 | Open/Load | | |
21+
| TC-03 | Create/Edit | | |
22+
| TC-04 | Save/Export Applicability | | |
23+
| TC-05 | Invalid Input/Error Handling | | |
24+
| TC-06 | UI Persistence/State Restoration | | |
25+
| TC-07 | Integration Handoff | | |
26+
27+
## Blockers
28+
- None / list blockers with reproduction details.
29+
30+
## Artifacts
31+
- Screenshots:
32+
- Exported files or payload references:
33+
- Console/error logs:
34+
35+
## Summary / Recommendation
36+
- Overall status:
37+
- Risks:
38+
- Recommended next action:
Lines changed: 8 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -1,28 +1,8 @@
1-
Validation Checklist
2-
3-
1) Template structure untouched
4-
- command: git status --short -- games/_template
5-
- result: PASS (no output; no template changes)
6-
7-
2) Docs structure and roadmap location guard
8-
- command: node tools/dev/checkDocsStructureGuard.mjs
9-
- result: PASS
10-
- no invalid docs/roadmaps/ directory
11-
- docs/dev/roadmaps exists
12-
- docs structure validation complete
13-
14-
3) Empty directory validation (excluding template)
15-
- command: python empty-dir scan excluding games/_template/**, .git, node_modules, tmp
16-
- result: PASS (0 empty directories remain)
17-
18-
4) .keep audit (excluding template)
19-
- command: rg --files -g "**/.keep" --glob "!tmp/**" --glob "!.git/**" --glob "!node_modules/**"
20-
- result: PASS (no matches)
21-
22-
5) Legacy roadmap path reference check
23-
- command: rg -n "docs[/\]roadmaps[/\](MASTER_ROADMAP_HIGH_LEVEL\.md|README\.md|phases\.txt)" . --glob "!tmp/**" --glob "!.git/**" --glob "!node_modules/**"
24-
- result: PASS (no matches)
25-
26-
6) Repo cleanliness note
27-
- command: git status --porcelain
28-
- result: INFO (working tree contains intentional scoped changes for this BUILD; unrelated pre-existing changes preserved)
1+
- [ ] active tool inventory completed
2+
- [ ] testing doc exists for every active tool
3+
- [ ] manual test cases exist for every active tool
4+
- [ ] standardized validation template created under docs/dev/reports
5+
- [ ] affected tool-doc navigation updated if needed
6+
- [ ] no start_of_day changes
7+
- [ ] unrelated working-tree changes preserved
8+
- [ ] roadmap updated only for completed items

docs/dev/roadmaps/MASTER_ROADMAP_TOOLS.md

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -7,10 +7,10 @@ Track all tool-related improvements, gaps, and future work.
77
---
88

99
## 1. Testing & Validation
10-
- [ ] create full testing documentation for each tool
11-
- [ ] define manual test cases per tool
10+
- [x] create full testing documentation for each tool
11+
- [x] define manual test cases per tool
1212
- [ ] add automated validation where possible
13-
- [ ] standardize validation reports under docs/dev/reports
13+
- [x] standardize validation reports under docs/dev/reports
1414

1515
---
1616

Lines changed: 121 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,121 @@
1+
# BUILD_PR_LEVEL_21_2_TOOL_TESTING_DOCUMENTATION_AND_REPORT_STANDARDIZATION
2+
3+
## Purpose
4+
Execute the first Tools roadmap testing lane by completing the documentation and reporting work that can be finished cleanly in one PR:
5+
6+
- create full testing documentation for each tool
7+
- define manual test cases per tool
8+
- standardize validation reports under `docs/dev/reports`
9+
10+
This PR does **not** attempt to complete automated validation. That remains the next lane.
11+
12+
## Roadmap intent
13+
This PR may advance the following items only if fully executed and validated:
14+
15+
- `create full testing documentation for each tool` `[ ] -> [x]`
16+
- `define manual test cases per tool` `[ ] -> [x]`
17+
- `standardize validation reports under docs/dev/reports` `[ ] -> [x]`
18+
19+
Do **not** advance:
20+
- `add automated validation where possible`
21+
22+
unless that work is actually completed in this same PR.
23+
24+
## Scope
25+
Included:
26+
- inventory active tools
27+
- create or normalize a full testing document for each active tool
28+
- add manual test cases per tool
29+
- standardize validation report structure under `docs/dev/reports`
30+
- add a reusable report template for tool validation
31+
- update tool doc navigation where needed
32+
- preserve unrelated working-tree changes
33+
34+
Excluded:
35+
- no feature work
36+
- no runtime changes unless strictly required for test harness discovery
37+
- no broad repo cleanup
38+
- no roadmap rewrites
39+
- no `start_of_day` changes
40+
- no speculative automation work beyond documenting automation opportunities
41+
42+
## Active tools baseline
43+
Ensure the tools set explicitly includes current active tool surfaces, including:
44+
- Tile Map Editor
45+
- Parallax Editor
46+
- Vector Map Editor
47+
- Vector Asset Studio
48+
49+
If additional active tools are present in the repo and are clearly in-scope, include them in the testing-doc set and record them in the inventory.
50+
51+
## Required outputs
52+
Codex must create/update:
53+
54+
- `docs/dev/reports/BUILD_PR_LEVEL_21_2_TOOL_TESTING_DOCUMENTATION_AND_REPORT_STANDARDIZATION_TOOL_INVENTORY.md`
55+
- `docs/dev/reports/BUILD_PR_LEVEL_21_2_TOOL_TESTING_DOCUMENTATION_AND_REPORT_STANDARDIZATION_VALIDATION.md`
56+
- `docs/dev/reports/tool_validation_report_template.md`
57+
58+
Per-tool testing docs under a consistent location, for example:
59+
- `docs/tools/<tool-name>/testing.md`
60+
or the current tool-doc structure if already established and coherent
61+
62+
Each per-tool testing doc must contain:
63+
- purpose of testing
64+
- prerequisites
65+
- setup steps
66+
- smoke tests
67+
- core workflow tests
68+
- edge case tests
69+
- regression checks
70+
- known limitations / non-goals
71+
- expected validation artifacts
72+
73+
## Manual test-case requirements
74+
Each active tool must have explicit manual test cases covering:
75+
- launch / boot
76+
- open/load workflow
77+
- create/edit workflow
78+
- save/export workflow if applicable
79+
- invalid input / error handling
80+
- UI persistence or state restoration if applicable
81+
- integration handoff to repo/runtime where applicable
82+
83+
## Validation report standardization requirements
84+
Create or normalize a common validation report shape that includes:
85+
- tool name
86+
- date / run context
87+
- environment
88+
- cases executed
89+
- pass/fail per case
90+
- blockers
91+
- screenshots or artifact references if applicable
92+
- summary / recommendation
93+
94+
## Acceptance
95+
- every active tool has a testing document
96+
- every active tool has manual test cases
97+
- validation reporting format is standardized under `docs/dev/reports`
98+
- tool inventory is explicit and current
99+
- no unrelated repo churn
100+
- roadmap updates are status-only and execution-backed
101+
102+
## Validation requirements
103+
Validation must confirm:
104+
- active tools inventory is complete
105+
- testing doc exists for every active tool
106+
- manual cases exist for every active tool
107+
- validation template exists and is reusable
108+
- no `start_of_day` changes
109+
- unrelated working-tree changes preserved
110+
111+
## Roadmap update rules
112+
Only update status markers in:
113+
- `docs/dev/roadmaps/MASTER_ROADMAP_TOOLS.md`
114+
115+
Allowed transitions only if fully executed and validated:
116+
- `create full testing documentation for each tool` `[ ] -> [x]`
117+
- `define manual test cases per tool` `[ ] -> [x]`
118+
- `standardize validation reports under docs/dev/reports` `[ ] -> [x]`
119+
120+
Do not update:
121+
- `add automated validation where possible`

docs/reference/root/README.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,7 @@ README.md
1919
- [Docs Home](../../README.md)
2020
- [Workflow Surface](../../operations/dev/README.md)
2121
- [Master Roadmap](../../dev/roadmaps/MASTER_ROADMAP_ENGINE.md)
22+
- [Tool Testing Docs](../../tools/testing/README.md)
2223
- [Architecture Overview](../architecture-standards/architecture/README.md)
2324
- [Standards](../architecture-standards/standards/README.md)
2425
- [PR Capability Bundles](../../pr/capabilities/README.md)

docs/tools/README.md

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,9 @@
1+
# Tools Documentation
2+
3+
This folder contains active tool documentation and testing guides.
4+
5+
## Testing Guides
6+
- [Testing Index](./testing/README.md)
7+
8+
## Report Template
9+
- [Tool Validation Report Template](../dev/reports/tool_validation_report_template.md)

0 commit comments

Comments
 (0)