Skip to content

Commit aa5bb6a

Browse files
author
DavidQ
committed
Run interactive UAT on stabilized tools
- Validate Vector Map Editor behavior - Validate paint/stroke interactions - Validate State Inspector JSON handling - Confirm header UX consistency - Capture issues without modifying behavior
1 parent e1c645d commit aa5bb6a

3 files changed

Lines changed: 118 additions & 61 deletions

File tree

docs/dev/codex_commands.md

Lines changed: 7 additions & 55 deletions
Original file line numberDiff line numberDiff line change
@@ -1,60 +1,12 @@
11
MODEL: GPT-5.3-codex
22
REASONING: low
33

4-
PR purpose:
5-
Stabilize existing tools only. Do not add new features.
4+
Run interactive UAT:
65

7-
Priority:
8-
1. Fix Vector Map Editor default selection behavior first.
9-
2. Fix paint/fill and stroke controls.
10-
3. Improve State Inspector JSON usability.
11-
4. Remove silent fallback data.
12-
5. Apply header polish follow-up.
6+
- Open each tool in browser
7+
- Execute manual test steps from PR doc
8+
- Record PASS/FAIL per tool
9+
- Capture issues without fixing them
1310

14-
Hard rules:
15-
- Do not use silent fallback data.
16-
- Do not auto-load hidden/default sample assets.
17-
- Do not add safe fallback behavior that hides missing tool configuration.
18-
- If required data or configuration is missing, show an actionable error message.
19-
- Do not modify start_of_day folders.
20-
- Do not modify sample games or unrelated engine runtime files.
21-
22-
Header follow-up:
23-
- Keep fullscreen header single-line format:
24-
<Tool Name> — <Short Description>
25-
- If tool.name or tool.shortDescription is missing, show an actionable configuration error.
26-
- Add hard max-width / nowrap / overflow hidden / text-overflow ellipsis.
27-
- Lock font size and line height.
28-
- Add data-tool-id to header DOM.
29-
30-
Vector Map Editor:
31-
- Make initial selection/mode explicit and visible.
32-
- Do not silently select an object.
33-
- Do not silently load default/sample data.
34-
- Show empty state when no map/manifest is loaded.
35-
- Confirm selection clears correctly.
36-
37-
Paint / stroke:
38-
- Ensure fill/paint controls update selected editable objects.
39-
- Ensure stroke controls update selected editable objects.
40-
- Disable controls only when there is no valid selection.
41-
- Disabled controls must explain why.
42-
43-
State Inspector:
44-
- Format valid JSON.
45-
- Show clear parse errors for invalid JSON.
46-
- Show safe empty state for missing/empty state.
47-
- Do not inject sample state.
48-
49-
Validation:
50-
- Run node --check on changed JavaScript files.
51-
- Manually verify the four target tools:
52-
- Vector Map Editor
53-
- Vector Asset Studio
54-
- Sprite Editor
55-
- State Inspector
56-
57-
Report:
58-
- List changed files.
59-
- List validation commands/results.
60-
- Note any remaining issues without masking them.
11+
Do not modify code in this PR.
12+
Only report findings.

docs/dev/commit_comment.txt

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
1-
Stabilize tool UX and remove silent fallback behavior
1+
Run interactive UAT on stabilized tools
22

3-
- Fix Vector Map Editor startup/selection behavior
4-
- Restore reliable paint/fill and stroke control behavior
5-
- Improve State Inspector JSON empty/error handling
6-
- Replace silent fallback data with actionable messages
7-
- Tighten fullscreen header sizing and add data-tool-id for diagnostics
3+
- Validate Vector Map Editor behavior
4+
- Validate paint/stroke interactions
5+
- Validate State Inspector JSON handling
6+
- Confirm header UX consistency
7+
- Capture issues without modifying behavior
Lines changed: 105 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,105 @@
1+
# PR — Tool Interactive UAT
2+
3+
## Purpose
4+
Perform browser-based interactive validation of core tools after stabilization.
5+
6+
This PR is validation-only. Do not add features or change behavior.
7+
8+
---
9+
10+
## Scope
11+
12+
Tools:
13+
- Vector Map Editor
14+
- Vector Asset Studio
15+
- Sprite Editor
16+
- State Inspector
17+
18+
---
19+
20+
## Test Matrix
21+
22+
### 1. Vector Map Editor
23+
24+
- Launch tool
25+
- Confirm:
26+
- No auto-selection
27+
- No hidden data loaded
28+
- Empty state visible
29+
30+
- Load/select objects (if available)
31+
- Verify:
32+
- Selection changes visually
33+
- Selection clears correctly
34+
35+
---
36+
37+
### 2. Vector Asset Studio
38+
39+
- Select editable object
40+
- Modify:
41+
- Fill / paint
42+
- Stroke
43+
44+
- Verify:
45+
- Visual updates occur
46+
- Controls disable when no selection
47+
- Disabled controls show reason
48+
49+
- Remove required config (test)
50+
- Verify:
51+
- Actionable error displayed
52+
- No silent fallback
53+
54+
---
55+
56+
### 3. State Inspector
57+
58+
- Input valid JSON
59+
- Confirm pretty formatting
60+
61+
- Input invalid JSON
62+
- Confirm clear parse error
63+
64+
- Input non-object JSON
65+
- Confirm validation message
66+
67+
- Empty input
68+
- Confirm empty-state message
69+
70+
---
71+
72+
### 4. Sprite Editor
73+
74+
- Launch tool
75+
- Verify:
76+
- Header format correct
77+
- No fallback data
78+
- No unexpected errors
79+
80+
---
81+
82+
## Reporting
83+
84+
For each tool:
85+
- PASS / FAIL
86+
- Notes on behavior
87+
- Screenshots (optional)
88+
- List of issues discovered
89+
90+
---
91+
92+
## Acceptance Criteria
93+
94+
- All tools pass interactive validation
95+
- No silent fallback behavior observed
96+
- Errors are actionable and visible
97+
- Header UX consistent across tools
98+
99+
---
100+
101+
## Out of Scope
102+
103+
- New features
104+
- Refactoring
105+
- Game engine changes

0 commit comments

Comments
 (0)