Skip to content

Commit 41624db

Browse files
author
DavidQ
committed
Full subfolder copy of Asteroids into asteroids_new for testable state: Flatten duplicated nested asteroids_new subfolders and keep only the correct root-level copies.
1 parent fb17c8e commit 41624db

47 files changed

Lines changed: 2030 additions & 35 deletions

Some content is hidden

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

docs/dev/CODEX_COMMANDS.md

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,8 +2,8 @@ MODEL: GPT-5.4-codex
22
REASONING: high
33

44
COMMAND:
5-
Execute exactly docs/pr/BUILD_PR_GAMES_95B_ASTEROIDS_NEW_TESTABLE_VERTICAL_SLICE_EXISTING_SOURCES_ONLY.md.
6-
Modify only the exact target files listed in the PR doc.
5+
Execute exactly docs/pr/BUILD_PR_GAMES_96A_ASTEROIDS_NEW_FLATTEN_DUPLICATE_SUBFOLDERS.md.
6+
Modify only the exact target files/folders listed in the PR doc.
77
Do not expand scope.
88
Package the delta zip to:
9-
<project folder>/tmp/BUILD_PR_GAMES_95B_ASTEROIDS_NEW_TESTABLE_VERTICAL_SLICE_EXISTING_SOURCES_ONLY_delta.zip
9+
<project folder>/tmp/BUILD_PR_GAMES_96A_ASTEROIDS_NEW_FLATTEN_DUPLICATE_SUBFOLDERS_delta.zip

docs/dev/COMMIT_COMMENT.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
Recover blocked 95A by building a smoke-testable asteroids_new vertical slice from existing Asteroids source files only.
1+
Flatten duplicated nested asteroids_new subfolders and keep only the correct root-level copies.

docs/dev/NEXT_COMMAND.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
After commit, inspect whether one additional exact gameplay dependency is needed or whether the vertical slice is already smoke-testable enough to validate.
1+
After commit, validate asteroids_new runtime/imports and then continue with the next testable vertical slice only.
Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,3 @@
1-
- replace blocked 95A with an existing-sources-only vertical slice BUILD
2-
- use current asteroids_new boot/flow shell instead of missing original Asteroids boot/flow files
3-
- copy only world/debug/core-entity files that actually exist in the repo
4-
- keep original Asteroids files untouched
1+
- fix duplicated nested folders created by the full-copy burst
2+
- keep only the intended root-level asteroids_new files
3+
- preserve the smoke-testable parallel lane

docs/dev/reports/file_tree.txt

Lines changed: 6 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -1,22 +1,12 @@
1-
docs/pr/BUILD_PR_GAMES_95B_ASTEROIDS_NEW_TESTABLE_VERTICAL_SLICE_EXISTING_SOURCES_ONLY.md
1+
docs/pr/BUILD_PR_GAMES_96A_ASTEROIDS_NEW_FLATTEN_DUPLICATE_SUBFOLDERS.md
22
docs/dev/codex_commands.md
33
docs/dev/commit_comment.txt
44
docs/dev/next_command.txt
55
docs/dev/reports/file_tree.txt
66
docs/dev/reports/change_summary.txt
77
docs/dev/reports/validation_checklist.txt
8-
games/Asteroids/game/AsteroidsWorld.js
9-
games/Asteroids/debug/asteroidsShowcaseDebug.js
10-
games/Asteroids/entities/Bullet.js
11-
games/Asteroids/entities/Ship.js
12-
games/Asteroids/entities/Asteroid.js
13-
games/Asteroids/entities/Ufo.js
14-
games/asteroids_new/index.js
15-
games/asteroids_new/flow/attract.js
16-
games/asteroids_new/flow/intro.js
17-
games/asteroids_new/game/AsteroidsWorld.js
18-
games/asteroids_new/debug/asteroidsShowcaseDebug.js
19-
games/asteroids_new/entities/Bullet.js
20-
games/asteroids_new/entities/Ship.js
21-
games/asteroids_new/entities/Asteroid.js
22-
games/asteroids_new/entities/Ufo.js
8+
games/asteroids_new/assets/assets/*
9+
games/asteroids_new/debug/debug/*
10+
games/asteroids_new/entities/entities/*
11+
games/asteroids_new/game/game/*
12+
games/asteroids_new/systems/systems/*
Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
- single PR purpose only
2-
- exact target files only
3-
- existing source files only
2+
- exact folders/files only
43
- no original Asteroids file changes
5-
- no missing source-file assumptions
6-
- asteroids_new files parse and import cleanly together
7-
- result is substantial enough for a real smoke test
4+
- no content changes to correct root-level files
5+
- duplicate nested folders removed
6+
- asteroids_new remains smoke-testable
Lines changed: 79 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,79 @@
1+
# BUILD PR — Asteroids New Flatten Duplicate Subfolders
2+
3+
## Purpose
4+
Fix the incorrectly duplicated nested directories created during the full-copy burst so `games/asteroids_new` uses the intended template-aligned structure and remains testable.
5+
6+
## Exact Target Files
7+
8+
### Files currently in correct root-level folders (must remain)
9+
- `games/asteroids_new/assets/Asteroids Deluxe.png`
10+
- `games/asteroids_new/assets/asteroids-bezel.png`
11+
- `games/asteroids_new/assets/bangLarge.wav`
12+
- `games/asteroids_new/assets/bangMedium.wav`
13+
- `games/asteroids_new/assets/bangSmall.wav`
14+
- `games/asteroids_new/assets/beat1.wav`
15+
- `games/asteroids_new/assets/beat2.wav`
16+
- `games/asteroids_new/assets/extraShip.wav`
17+
- `games/asteroids_new/assets/fire.wav`
18+
- `games/asteroids_new/assets/preview.png`
19+
- `games/asteroids_new/assets/saucerBig.wav`
20+
- `games/asteroids_new/assets/saucerSmall.wav`
21+
- `games/asteroids_new/assets/thrust.wav`
22+
- `games/asteroids_new/assets/vector_battle.ttf`
23+
- `games/asteroids_new/debug/asteroidsShowcaseDebug.js`
24+
- `games/asteroids_new/entities/Asteroid.js`
25+
- `games/asteroids_new/entities/Bullet.js`
26+
- `games/asteroids_new/entities/Ship.js`
27+
- `games/asteroids_new/entities/Ufo.js`
28+
- `games/asteroids_new/game/AsteroidsAttractAdapter.js`
29+
- `games/asteroids_new/game/AsteroidsGameScene.js`
30+
- `games/asteroids_new/game/AsteroidsSession.js`
31+
- `games/asteroids_new/game/AsteroidsWorld.js`
32+
- `games/asteroids_new/systems/AsteroidsAudio.js`
33+
- `games/asteroids_new/systems/AsteroidsHighScoreService.js`
34+
- `games/asteroids_new/systems/AsteroidsInitialsEntry.js`
35+
- `games/asteroids_new/systems/HighScoreStore.js`
36+
- `games/asteroids_new/systems/ShipDebrisSystem.js`
37+
38+
### Incorrect duplicate nested folders/files to remove
39+
- `games/asteroids_new/assets/assets/*`
40+
- `games/asteroids_new/debug/debug/*`
41+
- `games/asteroids_new/entities/entities/*`
42+
- `games/asteroids_new/game/game/*`
43+
- `games/asteroids_new/systems/systems/*`
44+
45+
## Required Code Changes
46+
1. Verify the root-level copies listed above are present and intact.
47+
2. Remove the duplicate nested folders and their duplicate contents:
48+
- `assets/assets`
49+
- `debug/debug`
50+
- `entities/entities`
51+
- `game/game`
52+
- `systems/systems`
53+
3. Do not move, rename, or modify the correct root-level copies.
54+
4. Do not modify unrelated folders such as:
55+
- `config`
56+
- `flow`
57+
- `levels`
58+
- `platform`
59+
- `ui`
60+
- `utils`
61+
62+
## Hard Constraints
63+
- exact scope only
64+
- do not touch original `games/Asteroids/*`
65+
- do not change file contents in the correct root-level copies
66+
- do not widen into import refactors unless removal of duplicate nested paths requires a minimum fix in an already-listed `games/asteroids_new` file
67+
- no repo-wide cleanup
68+
- no formatting-only churn
69+
70+
## Validation Steps
71+
- confirm the duplicate nested folders no longer exist
72+
- confirm the root-level copies still exist
73+
- confirm `games/asteroids_new` still parses/imports cleanly after duplicate-folder removal
74+
- confirm no original `games/Asteroids/*` files changed
75+
76+
## Acceptance Criteria
77+
- `games/asteroids_new` has one intended copy of each file in the correct folder
78+
- no `assets/assets`, `debug/debug`, `entities/entities`, `game/game`, or `systems/systems` folders remain
79+
- the parallel lane remains smoke-testable
2.17 MB
Loading
389 KB
Loading
9.34 KB
Binary file not shown.

0 commit comments

Comments
 (0)