Skip to content

Commit aba2a07

Browse files
author
DavidQ
committed
Add an HTML entrypoint for asteroids_new so the parallel game can actually be launched. issue with all paths to inspectorUtils.js
1 parent e4f7d4f commit aba2a07

51 files changed

Lines changed: 140 additions & 66 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: 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_99A_ASTEROIDS_NEW_BOOT_ENTRY_VISIBILITY_AND_EXECUTION_CHECK.md.
5+
Execute exactly docs/pr/BUILD_PR_GAMES_100A_ASTEROIDS_NEW_HTML_ENTRYPOINT.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_99A_ASTEROIDS_NEW_BOOT_ENTRY_VISIBILITY_AND_EXECUTION_CHECK_delta.zip
9+
<project folder>/tmp/BUILD_PR_GAMES_100A_ASTEROIDS_NEW_HTML_ENTRYPOINT_delta.zip

docs/dev/COMMIT_COMMENT.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
Add explicit boot-entry visibility to asteroids_new so execution or silent failure can be confirmed immediately.
1+
Add an HTML entrypoint for asteroids_new so the parallel game can actually be launched.

docs/dev/NEXT_COMMAND.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
After commit, report exactly what becomes visible at boot; use that result to choose either boot-fix or gameplay continuation.
1+
After commit, open games/asteroids_new/index.html and report whether you see execution or a concrete runtime error.
Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,3 @@
1-
- narrow the next PR to boot-entry visibility only
2-
- expose whether asteroids_new index is executing at all
3-
- expose the first reached or failing stage in the runtime path
1+
- add the missing HTML launch entrypoint for asteroids_new
2+
- wire the HTML page to the existing parallel JS entry
3+
- keep scope limited to launchability only

docs/dev/reports/file_tree.txt

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,9 @@
1-
docs/pr/BUILD_PR_GAMES_99A_ASTEROIDS_NEW_BOOT_ENTRY_VISIBILITY_AND_EXECUTION_CHECK.md
1+
docs/pr/BUILD_PR_GAMES_100A_ASTEROIDS_NEW_HTML_ENTRYPOINT.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_new/index.js
9-
games/asteroids_new/game/AsteroidsGameScene.js
10-
games/asteroids_new/game/AsteroidsSession.js
11-
games/asteroids_new/game/AsteroidsWorld.js
8+
games/asteroids_new/index.html
9+
games/asteroids_new/index.js
Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
- single PR purpose only
22
- exact target files only
33
- no original Asteroids file changes
4-
- visible execution proof added
5-
- silent failure point exposed if boot still fails
4+
- asteroids_new has its own HTML entrypoint
5+
- opening the HTML page produces execution or a real visible error
Lines changed: 43 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,43 @@
1+
# BUILD PR — Asteroids New HTML Entrypoint
2+
3+
## Purpose
4+
Make `games/asteroids_new` actually launchable by adding the missing HTML entrypoint that matches the original Asteroids launch pattern.
5+
6+
## Why This PR Exists
7+
Current evidence shows the repo launches Asteroids through HTML pages such as:
8+
- `games/Asteroids/index.html`
9+
- `games/Asteroids/capture-preview.html`
10+
11+
The parallel `games/asteroids_new` lane has JS files, but no confirmed HTML entrypoint wired into that launch pattern. Without an HTML entry, the game code will not execute from the normal host flow.
12+
13+
## Exact Target Files
14+
- `games/asteroids_new/index.html`
15+
- `games/asteroids_new/index.js`
16+
17+
## Required Code Changes
18+
1. Create `games/asteroids_new/index.html` using the original Asteroids HTML launch pattern as the model.
19+
2. Wire the new HTML entrypoint to the existing `games/asteroids_new/index.js`.
20+
3. Update `games/asteroids_new/index.js` only as needed so it matches the HTML entry pattern.
21+
4. Keep the result minimal and launch-focused.
22+
23+
## Hard Constraints
24+
- exact files only
25+
- do not modify any original `games/Asteroids/*` files
26+
- do not modify root `index.html`
27+
- do not modify `games/index.html`
28+
- do not widen into gameplay fixes, assets, debug UI redesign, or registry work
29+
- do not refactor unrelated logic
30+
- focus only on making `games/asteroids_new` reachable and executable through its own HTML page
31+
32+
## Validation Steps
33+
- confirm only the exact target files changed
34+
- confirm `games/asteroids_new/index.html` loads `games/asteroids_new/index.js`
35+
- confirm opening `games/asteroids_new/index.html` produces visible execution or real runtime errors instead of silence
36+
- confirm no original `games/Asteroids/*` files changed
37+
38+
## Acceptance Criteria
39+
- `games/asteroids_new/index.html` exists
40+
- it launches the parallel game entry
41+
- `games/asteroids_new` is now reachable through an HTML page
42+
- execution is visible or real runtime failures are exposed
43+
- original Asteroids files remain untouched

games/Asteroids/debug/asteroidsShowcaseDebug.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ David Quesenberry
55
asteroidsShowcaseDebug.js
66
*/
77

8-
import { asArray, asObject } from "../../../src/engine/debug/inspectors/shared/inspectorUtils.js";
8+
import { asArray, asObject } from "/src/engine/debug/inspectors/shared/inspectorUtils.js";
99

1010
function sanitizeText(value) {
1111
return typeof value === "string" ? value.trim() : "";

games/asteroids_new/debug/asteroidsShowcaseDebug.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ David Quesenberry
55
asteroidsShowcaseDebug.js
66
*/
77

8-
import { asArray, asObject } from "../../../src/engine/debug/inspectors/shared/inspectorUtils.js";
8+
import { asArray, asObject } from "/src/engine/debug/inspectors/shared/inspectorUtils.js";
99

1010
function sanitizeText(value) {
1111
return typeof value === "string" ? value.trim() : "";

games/asteroids_new/entities/Asteroid.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ David Quesenberry
55
Asteroid.js
66
*/
77
import { TAU, randomRange, wrap } from '../utils/math.js';
8-
import { transformPoints } from '../../../src/engine/vector/index.js';
8+
import { transformPoints } from '/src/engine/vector/index.js';
99

1010
const BASE_VECTOR_MAP = [
1111
{ x: 10, y: 40 },

0 commit comments

Comments
 (0)