Skip to content

Commit 2113749

Browse files
author
DavidQ
committed
Normalize all asset filenames to remove redundant game/type coupling
BUILD_PR_LEVEL_10_12_ASSET_FILE_NAMING_NORMALIZATION_GLOBAL & Remove duplicate asset definitions and add bezel roadmap entry BUILD_PR_LEVEL_10_13_ASSET_DEDUP_AND_BEZEL_ROADMAP
1 parent 69b465e commit 2113749

44 files changed

Lines changed: 224 additions & 183 deletions

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

‎asteroids.txt‎

-3.22 KB
Binary file not shown.

‎docs/dev/CODEX_COMMANDS.md‎

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,10 @@
11
MODEL: GPT-5.3-codex
2-
REASONING: high
3-
COMMAND: Normalize shared manifest loader to be game-agnostic, rename file and APIs, parameterize gameId, update tests accordingly
2+
REASONING: medium
3+
4+
COMMAND:
5+
1. Remove assets.json
6+
2. Ensure tools.manifest.json is sole source of truth
7+
3. Remove duplicate data structures if present
8+
4. Validate all loaders/tests still pass
9+
5. Package repo ZIP to:
10+
<project folder>/tmp/BUILD_PR_LEVEL_10_13_ASSET_DEDUP_AND_BEZEL_ROADMAP.zip

‎docs/dev/CODEX_TEMPLATE_GAME_CREATION_SCRIPTING.md‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ Use `scripts/PS/New-Game-from-Template.ps1` to create a new game scaffold from `
1717
- `tilemaps`
1818
- `parallax`
1919
- `vectors`
20-
- `games/<game>/assets/<game>.assets.json` (manifest scaffold)
20+
- `games/<game>/assets/tools.manifest.json` (manifest scaffold)
2121
- `games/<game>/config/<game>.project.json` (tool project scaffold)
2222

2323
## Guardrails

‎docs/dev/COMMIT_COMMENT.txt‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,2 +1,2 @@
1-
Normalize shared manifest loader to remove Asteroids-specific coupling
2-
BUILD_PR_LEVEL_10_10_SHARED_MANIFEST_LOADER_NORMALIZATION
1+
Remove duplicate asset definitions and add bezel roadmap entry
2+
BUILD_PR_LEVEL_10_13_ASSET_DEDUP_AND_BEZEL_ROADMAP

‎docs/dev/NEXT_COMMAND.txt‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
BUILD_PR_LEVEL_10_11_ASTEROIDS_MANIFEST_RUNTIME_HOOK
1+
BUILD_PR_LEVEL_10_14_ASTEROIDS_RUNTIME_VALIDATION_PASS

‎docs/dev/reports/change_summary.txt‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
Identified and corrected shared pipeline coupling issue
1+
Deduplicated assets + added bezel roadmap

‎docs/dev/reports/launch_smoke_report.md‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
# Launch Smoke Report
22

3-
Generated: 2026-04-13T21:22:09.006Z
3+
Generated: 2026-04-14T18:49:10.899Z
44

55
Filters: games=true, samples=true, tools=true, sampleRange=all
66

‎docs/dev/reports/validation_checklist.txt‎

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1-
- No game-specific naming in shared loader
2-
- API parameterized
3-
- Tests updated
1+
- assets.json removed
2+
- single manifest
3+
- loaders pass
4+
- roadmap updated

‎docs/dev/roadmaps/MASTER_ROADMAP_HIGH_LEVEL.md‎

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -520,7 +520,7 @@
520520
- [ ] 3D Movement Sample
521521
- [ ] Basic Collision Sample
522522

523-
### Core Sample Track (1601–1608)
523+
### Core Sample Track (1601 - 1608)
524524
- [ ] 1601 - 3D Cube Explorer
525525
- [ ] 1602 - 3D Maze Runner
526526
- [ ] 1603 - First Person Walkthrough
@@ -631,3 +631,9 @@
631631
- [ ] avoid broad repo-wide cleanup passes until the active lanes above are materially further along
632632

633633
- [x] relocate active template surfaces to `tools/templates/` and remove the empty root `templates/` directory
634+
635+
# MASTER ROADMAP HIGH LEVEL (status updates only)
636+
637+
[.] asset naming normalization
638+
[.] manifest discovery
639+
[ ] fullscreen bezel overlay system (low priority, before next game)

‎docs/pr/BUILD_PR_LEVEL_10_12_ASSET_FILE_NAMING_NORMALIZATION_GLOBAL.md‎

Lines changed: 65 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,65 @@
1+
# BUILD_PR_LEVEL_10_12_ASSET_FILE_NAMING_NORMALIZATION_GLOBAL
2+
3+
## Problem
4+
Asset files redundantly include game and asset context in filenames:
5+
- asteroids-*.json
6+
- asteroids-*.data.json
7+
8+
Directory structure already provides:
9+
games/<gameId>/assets/<type>/
10+
11+
This creates duplication and coupling.
12+
13+
## Purpose
14+
Normalize ALL asset filenames to be context-driven by directory only.
15+
16+
## Scope
17+
18+
Apply to ALL files under:
19+
games/Asteroids/assets/**
20+
21+
### Rename Rules
22+
23+
Remove:
24+
- game prefix (asteroids-)
25+
- redundant asset descriptors already in folder path
26+
27+
Examples:
28+
29+
sprites/data/asteroids-demo.sprite.data.json
30+
→ sprites/data/demo.data.json
31+
32+
tilemaps/data/asteroids-stage.tilemap.data.json
33+
→ tilemaps/data/stage.data.json
34+
35+
parallax/data/asteroids-title.parallax.data.json
36+
→ parallax/data/title.data.json
37+
38+
vectors/data/asteroids-vectors.library.data.json
39+
→ vectors/data/library.data.json
40+
41+
manifest:
42+
asteroids.asset.manifest.json
43+
→ tools.manifest.json
44+
45+
### Rules
46+
47+
- File names must be minimal and local-context only
48+
- No game name in file
49+
- No type duplication (folder defines type)
50+
- Preserve uniqueness within folder only
51+
52+
## Loader Impact
53+
54+
Loader must:
55+
- NOT rely on filename patterns containing gameId
56+
- Use directory + manifest references only
57+
58+
## Testable Outcome
59+
60+
- All asset files renamed consistently
61+
- Loader resolves via manifest + directory, not naming
62+
- No "asteroids" in filenames
63+
64+
## Non-Goals
65+
- No engine modification

0 commit comments

Comments
 (0)