Skip to content

Commit 7407041

Browse files
author
DavidQ
committed
Accelerate asteroids_new with one larger non-destructive parallel adoption burst covering world, debug, and two entity files.
1 parent 77bca3f commit 7407041

File tree

9 files changed

+86
-13
lines changed

9 files changed

+86
-13
lines changed

docs/dev/CODEX_COMMANDS.md

Lines changed: 2 additions & 2 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_93A_ASTEROIDS_NEW_PARALLEL_BOOT_VALIDATION.md.
5+
Execute exactly docs/pr/BUILD_PR_GAMES_94A_ASTEROIDS_NEW_PARALLEL_BURST_ALPHA.md.
66
Modify only the exact target files listed in the PR doc.
77
Do not expand scope.
88
Package the delta zip to:
9-
<project folder>/tmp/BUILD_PR_GAMES_93A_ASTEROIDS_NEW_PARALLEL_BOOT_VALIDATION_delta.zip
9+
<project folder>/tmp/BUILD_PR_GAMES_94A_ASTEROIDS_NEW_PARALLEL_BURST_ALPHA_delta.zip

docs/dev/COMMIT_COMMENT.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
Validate and minimally fix the current asteroids_new parallel boot lane.
1+
Accelerate asteroids_new with one larger non-destructive parallel adoption burst covering world, debug, and two entity files.

docs/dev/NEXT_COMMAND.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
Next: inspect whether systems or one additional gameplay slice can be pulled over exactly.
1+
After commit, inspect the delta and either pull one additional gameplay slice or validate before widening into systems.
Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,2 +1,4 @@
1-
- focused validation and minimum-fix closeout for the current aggressive asteroids_new burst
2-
- no original Asteroids files touched
1+
- combine multiple previously separate asteroids_new parallel-adoption slices into one larger exact-file BUILD
2+
- copy world, debug shell, and two entity files into the parallel lane
3+
- allow only the minimum path/import fixes inside asteroids_new index/flow files
4+
- keep the original Asteroids game untouched

docs/dev/reports/file_tree.txt

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,18 @@
1-
docs/pr/BUILD_PR_GAMES_93A_ASTEROIDS_NEW_PARALLEL_BOOT_VALIDATION.md
1+
docs/pr/BUILD_PR_GAMES_94A_ASTEROIDS_NEW_PARALLEL_BURST_ALPHA.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
812
games/asteroids_new/index.js
913
games/asteroids_new/flow/attract.js
1014
games/asteroids_new/flow/intro.js
1115
games/asteroids_new/game/AsteroidsWorld.js
16+
games/asteroids_new/debug/asteroidsShowcaseDebug.js
1217
games/asteroids_new/entities/Bullet.js
13-
games/asteroids_new/entities/Ship.js
14-
games/asteroids_new/debug/asteroidsShowcaseDebug.js
18+
games/asteroids_new/entities/Ship.js
Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,7 @@
11
- single PR purpose only
22
- exact target files only
3-
- no new files
3+
- copy-only from original Asteroids files
44
- no original Asteroids file changes
5-
- minimum-fix validation only
5+
- no systems/levels/assets widening
6+
- minimum import/path fixes only
7+
- listed asteroids_new files parse and resolve together
Lines changed: 65 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,65 @@
1+
# BUILD PR — Asteroids New Parallel Burst Alpha
2+
3+
## Purpose
4+
Accelerate completion of `games/asteroids_new` with one larger exact-file, non-destructive burst.
5+
This PR intentionally combines multiple compatible slices into one Codex run so progress is materially faster.
6+
7+
## Exact Target Files
8+
9+
### Source files (copy from original Asteroids only)
10+
- `games/Asteroids/game/AsteroidsWorld.js`
11+
- `games/Asteroids/debug/asteroidsShowcaseDebug.js`
12+
- `games/Asteroids/entities/Bullet.js`
13+
- `games/Asteroids/entities/Ship.js`
14+
15+
### Destination / touched files
16+
- `games/asteroids_new/index.js`
17+
- `games/asteroids_new/flow/attract.js`
18+
- `games/asteroids_new/flow/intro.js`
19+
- `games/asteroids_new/game/AsteroidsWorld.js`
20+
- `games/asteroids_new/debug/asteroidsShowcaseDebug.js`
21+
- `games/asteroids_new/entities/Bullet.js`
22+
- `games/asteroids_new/entities/Ship.js`
23+
24+
## Required Code Changes
25+
1. Copy these exact source files into the parallel lane:
26+
- `games/Asteroids/game/AsteroidsWorld.js` -> `games/asteroids_new/game/AsteroidsWorld.js`
27+
- `games/Asteroids/debug/asteroidsShowcaseDebug.js` -> `games/asteroids_new/debug/asteroidsShowcaseDebug.js`
28+
- `games/Asteroids/entities/Bullet.js` -> `games/asteroids_new/entities/Bullet.js`
29+
- `games/Asteroids/entities/Ship.js` -> `games/asteroids_new/entities/Ship.js`
30+
31+
2. Update only these existing parallel-lane files as needed:
32+
- `games/asteroids_new/index.js`
33+
- `games/asteroids_new/flow/attract.js`
34+
- `games/asteroids_new/flow/intro.js`
35+
36+
3. In the destination parallel files listed above, make only the minimum import/path fixes required so the copied files parse and resolve together inside `games/asteroids_new`.
37+
38+
## Hard Constraints
39+
- exact files only
40+
- copy only from original Asteroids files; do not move or delete source files
41+
- do not modify any original `games/Asteroids/*` files
42+
- do not widen into systems, levels, assets, or unrelated entities
43+
- do not refactor gameplay behavior
44+
- preserve copied file behavior exactly except for minimum path/import fixes required in the parallel lane
45+
- no repo-wide cleanup
46+
- no unrelated formatting churn
47+
48+
## Validation Steps
49+
- confirm only the exact target files above changed
50+
- syntax-check:
51+
- `games/asteroids_new/index.js`
52+
- `games/asteroids_new/flow/attract.js`
53+
- `games/asteroids_new/flow/intro.js`
54+
- `games/asteroids_new/game/AsteroidsWorld.js`
55+
- `games/asteroids_new/debug/asteroidsShowcaseDebug.js`
56+
- `games/asteroids_new/entities/Bullet.js`
57+
- `games/asteroids_new/entities/Ship.js`
58+
- confirm imports across the listed `games/asteroids_new` files resolve cleanly
59+
- confirm no original `games/Asteroids/*` files changed
60+
61+
## Acceptance Criteria
62+
- the copied world, debug, and two-entity slice exist in `games/asteroids_new`
63+
- `games/asteroids_new/index.js` and the two flow files resolve against the parallel lane files
64+
- the listed parallel-lane files parse cleanly together
65+
- original Asteroids files remain untouched

games/asteroids_new/entities/Bullet.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ David Quesenberry
44
03/22/2026
55
Bullet.js
66
*/
7-
import { wrap } from "../../asteroids/utils/math.js";
7+
import { wrap } from '../../Asteroids/utils/math.js';
88

99
export default class Bullet {
1010
constructor(x, y, vx, vy, life = 1.1) {

games/asteroids_new/entities/Ship.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ David Quesenberry
44
03/22/2026
55
Ship.js
66
*/
7-
import { wrap } from "../../asteroids/utils/math.js";
7+
import { wrap } from '../../Asteroids/utils/math.js';
88
import { transformPoints } from '../../../src/engine/vector/index.js';
99

1010
const SMALL_VECTOR_MAP = [

0 commit comments

Comments
 (0)