Skip to content

Commit 676531d

Browse files
author
DavidQ
committed
Apply BUILD_PR_LEVEL_8_31_ASTEROIDS_JSON_CLEANUP_AND_MANIFEST_WIRING. V2
1 parent dfb8d09 commit 676531d

2 files changed

Lines changed: 105 additions & 8 deletions

File tree

Lines changed: 103 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,105 @@
11
# Level 8.31 Asteroids JSON Cleanup Report
22

3-
## To Be Completed By Codex
4-
- Inventory all Asteroids JSON files before cleanup.
5-
- Verify manifest coverage.
6-
- Identify files safe to delete.
7-
- Identify files that must be retained as legacy/runtime dependencies.
8-
- Identify deferred moves from `assets/<tool>/data/*`.
9-
- Confirm final unreferenced JSON count.
10-
- Confirm no runtime/start_of_day changes.
3+
## JSON File Inventory
4+
Asteroids JSON inventory (`games/Asteroids/**/*.json`) contains 22 files (including SSoT):
5+
6+
1. `games/Asteroids/assets/images/bezel.stretch.override.json`
7+
2. `games/Asteroids/assets/palettes/asteroids-classic.palette.json`
8+
3. `games/Asteroids/assets/palettes/asteroids-hud.palette.json`
9+
4. `games/Asteroids/assets/palettes/hud.json`
10+
5. `games/Asteroids/assets/parallax/data/overlay.data.json`
11+
6. `games/Asteroids/assets/parallax/data/title.data.json`
12+
7. `games/Asteroids/assets/parallax/overlay.json`
13+
8. `games/Asteroids/assets/parallax/title.json`
14+
9. `games/Asteroids/assets/sprites/data/demo.data.json`
15+
10. `games/Asteroids/assets/sprites/demo.json`
16+
11. `games/Asteroids/assets/tilemaps/data/stage.data.json`
17+
12. `games/Asteroids/assets/tilemaps/stage.json`
18+
13. `games/Asteroids/assets/tilesets/ui.json`
19+
14. `games/Asteroids/assets/tools.manifest.json`
20+
15. `games/Asteroids/assets/vectors/asteroid-large.json`
21+
16. `games/Asteroids/assets/vectors/asteroid-medium.json`
22+
17. `games/Asteroids/assets/vectors/asteroid-small.json`
23+
18. `games/Asteroids/assets/vectors/data/library.data.json`
24+
19. `games/Asteroids/assets/vectors/ship.json`
25+
20. `games/Asteroids/assets/vectors/title.json`
26+
21. `games/Asteroids/assets/workspace.asset-catalog.json`
27+
22. `games/Asteroids/game.manifest.json` (SSoT)
28+
29+
## SSoT Manifest Coverage
30+
- `games/Asteroids/game.manifest.json` is treated as SSoT.
31+
- Coverage check result:
32+
- every non-manifest Asteroids JSON file path is referenced by the manifest (`21/21` wired).
33+
- `assets/tilesets/ui.json` is wired through:
34+
- `assets.tileSets[].runtimePath`
35+
- and is also referenced in `tools/shared/asteroidsPlatformDemo.js`.
36+
37+
## Duplicate HUD Data Audit
38+
Compared:
39+
- `games/Asteroids/assets/palettes/hud.json`
40+
- `games/Asteroids/assets/palettes/asteroids-hud.palette.json`
41+
42+
Findings:
43+
- both are typed palette-schema JSON (`schema=html-js-gaming.palette`)
44+
- swatches are equal (`4` swatches, identical values)
45+
46+
Deletion decision for `hud.json`:
47+
- **retained** (not safe to delete yet)
48+
- reasons:
49+
- still referenced by `tools/shared/asteroidsPlatformDemo.js`
50+
- still referenced by `games/Asteroids/game.manifest.json` legacy compatibility fields
51+
- still referenced by active docs/build records
52+
53+
## Legacy Catalog Audit
54+
Audited:
55+
- `games/Asteroids/assets/tools.manifest.json`
56+
- `games/Asteroids/assets/workspace.asset-catalog.json`
57+
58+
Deletion decision:
59+
- **retained** (not safe to delete yet)
60+
61+
Why retained:
62+
- `tools.manifest.json` has active script/test/tooling references (for example `scripts/validate-asset-ownership-strategy.mjs`, `tests/tools/GameAssetManifestDiscovery.test.mjs`, pipeline tooling paths).
63+
- `workspace.asset-catalog.json` remains part of runtime/tooling filename-based discovery conventions (for example game skin loading and workspace/tool catalog discovery patterns), and is still listed in manifest lineage.
64+
65+
## Generic `assets/*/data/*` Folder Audit
66+
Audited:
67+
- `assets/parallax/data/*`
68+
- `assets/sprites/data/*`
69+
- `assets/tilemaps/data/*`
70+
- `assets/vectors/data/*`
71+
72+
Move decision:
73+
- **deferred** (not safe in this PR)
74+
75+
Why deferred:
76+
- these paths are wired in `game.manifest.json` and `assets/tools.manifest.json`
77+
- multiple tests/tooling conventions still reference `.../data/...` toolData paths
78+
- moving now would require broader runtime/tooling updates, outside this PR scope
79+
80+
## Deleted Files
81+
- none
82+
83+
## Retained Legacy Files
84+
- `games/Asteroids/assets/palettes/hud.json` (legacy compatibility)
85+
- `games/Asteroids/assets/tools.manifest.json` (runtime/tooling/test dependency)
86+
- `games/Asteroids/assets/workspace.asset-catalog.json` (runtime/tool discovery dependency)
87+
88+
## Runtime/Script References Found
89+
Representative references that block safe deletion/moves:
90+
- `tools/shared/asteroidsPlatformDemo.js` -> `games/Asteroids/assets/palettes/hud.json`, `games/Asteroids/assets/tilesets/ui.json`
91+
- `scripts/validate-asset-ownership-strategy.mjs` -> `games/Asteroids/assets/tools.manifest.json`
92+
- `tests/tools/GameAssetManifestDiscovery.test.mjs` -> `games/Asteroids/assets/tools.manifest.json`
93+
- runtime/tooling conventions loading `workspace.asset-catalog.json` via filename-based discovery (game skin loader/platform shell/tooling)
94+
95+
## Palette Alpha Normalization
96+
- `#RRGGBBFF` occurrences in Asteroids JSON: `0`
97+
- no palette alpha normalization changes were required in this PR.
98+
99+
## Final Unreferenced JSON Count
100+
- `final_unreferenced_json_count=0` (all remaining Asteroids JSON are manifest-wired, runtime-required legacy, or deferred with documented dependency rationale)
101+
102+
## Constraint Check
103+
- `validators_added=0`
104+
- `runtime_code_changes=0`
105+
- `start_of_day_changes=0`

docs/dev/roadmaps/MASTER_ROADMAP_ENGINE.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -881,6 +881,8 @@
881881
- [.] Verify workspace schema has no sample-only concepts.
882882
- [x] Manifest SSoT audit across game workspace catalogs/tool manifests is complete.
883883
- [x] Asteroids-first `game.manifest.json` SSoT example implemented (palette + HUD typing + bezel override wiring).
884+
- [.] Asteroids JSON cleanup and manifest wiring audit is in progress.
885+
- [.] Asteroids legacy catalog parity verification is in progress.
884886
- [x] All-games `game.manifest.json` SSoT rollout completed with legacy catalogs retained.
885887
- [x] Asteroids preview direct launch separation from Workspace Manager remains valid.
886888
- [.] Unused JSON cleanup audit (tools + explicit deletion candidates) is in progress.

0 commit comments

Comments
 (0)