Skip to content

Commit ee0ea80

Browse files
author
DavidQ
committed
Add runtime session validation guard for V2 tools with explicit INVALID state handling - PR 11.219
1 parent af7218a commit ee0ea80

2 files changed

Lines changed: 420 additions & 0 deletions

File tree

Lines changed: 58 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,58 @@
1+
# PR_11_219 Report - V2 Session Schema Validation (Runtime Guard)
2+
3+
## Files Changed
4+
- `tests/runtime/V2SessionValidation.test.mjs`
5+
- `docs/dev/reports/PR_11_219_report.md`
6+
7+
## Validation Cases Tested
8+
Per tool (`asset-browser-v2`, `palette-manager-v2`, `svg-asset-studio-v2`, `tilemap-studio-v2`, `vector-map-editor-v2`), the runtime harness tested:
9+
1. valid fixture payload -> expected `VALID`
10+
2. malformed JSON -> expected `INVALID`
11+
3. missing required fields -> expected `INVALID`
12+
4. empty payload object -> expected `INVALID`
13+
14+
## Validation Commands Run
15+
1. `node --check tests/runtime/V2SessionValidation.test.mjs`
16+
Result: **PASS**
17+
2. `node tests/runtime/V2SessionValidation.test.mjs`
18+
Result: **PASS** (writes `tmp/v2-session-validation-results.json`)
19+
3. `node --check tools/*-v2/index.js`
20+
Result: **FAIL** on Windows/Node CLI wildcard resolution (`MODULE_NOT_FOUND` for literal `tools\\*-v2\\index.js`)
21+
4. Explicit equivalent per-file syntax sweep for `tools/*-v2/index.js`
22+
Result: **PASS** for:
23+
- `tools/asset-browser-v2/index.js`
24+
- `tools/palette-manager-v2/index.js`
25+
- `tools/svg-asset-studio-v2/index.js`
26+
- `tools/tilemap-studio-v2/index.js`
27+
- `tools/vector-map-editor-v2/index.js`
28+
- `tools/workspace-v2/index.js`
29+
30+
## Pass/Fail By Case
31+
All five V2 tools passed all four required cases:
32+
- valid fixture: **PASS**
33+
- malformed JSON: **PASS**
34+
- missing required fields: **PASS**
35+
- empty payload: **PASS**
36+
37+
No tool-level failures were reported in `tmp/v2-session-validation-results.json`.
38+
39+
## Example INVALID Messages
40+
- `Asset Browser V2 session data is invalid. Expected payloadJson.assetCatalog.`
41+
- `Palette Manager V2 session data is invalid. Expected paletteJson only.`
42+
- `SVG Asset Studio V2 session data is invalid. Expected payloadJson.vectorAssetDocument.`
43+
- `Tilemap session data is invalid. Expected payloadJson.tileMapDocument.`
44+
- `Vector Map Editor V2 session data is invalid. Expected payloadJson.vectorMapDocument.`
45+
46+
## No Fallback Confirmation
47+
- No fallback/default/demo session payloads were introduced.
48+
- No hidden sample loading was introduced.
49+
- No alternate session source was added; validation remains explicit and deterministic.
50+
51+
## Scope Confirmation
52+
- No schema files changed.
53+
- No samples changed.
54+
- No games changed.
55+
- No Workspace Manager v1 changes.
56+
- No platformShell changes.
57+
- No `tools/shared/*` changes.
58+
- No V2 tool HTML structure changes.

0 commit comments

Comments
 (0)