Skip to content

Commit 9566f70

Browse files
author
DavidQ
committed
PR 8.29B: Asteroids direct launch fix
- Preview now launches game directly - Workspace Manager decoupled from preview
1 parent d17c0dd commit 9566f70

6 files changed

Lines changed: 50 additions & 77 deletions

File tree

Lines changed: 5 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -1,23 +1,6 @@
1-
# Expected Codex Return / Delta Template
1+
# Expected Codex Output
22

3-
## Expected Changed/Added Files
4-
- `games/Asteroids/game.manifest.json`
5-
- `games/Asteroids/assets/palettes/asteroids-hud.palette.json` if HUD is converted to separate palette
6-
- `docs/dev/reports/level_8_28_asteroids_manifest_ssot_report.md`
7-
- `docs/dev/roadmaps/MASTER_ROADMAP_ENGINE.md` if status update needed
8-
9-
## Expected Validation Summary
10-
- `asteroids_game_manifest_exists=true`
11-
- `asteroids_bezel_override_wired=true`
12-
- `asteroids_main_palette_wired=true`
13-
- `asteroids_hud_data_typed=true`
14-
- `asteroids_unreferenced_json_after=<count>`
15-
- `legacy_catalogs_deleted=0`
16-
- `runtime_changes=0`
17-
- `validators_added=0`
18-
- `start_of_day_changes=0`
19-
20-
## Expected Delta ZIP
21-
Codex must create:
22-
23-
`tmp/BUILD_PR_LEVEL_8_28_MANIFEST_SSOT_IMPLEMENTATION_ASTEROIDS_FIRST_delta.zip`
3+
- asteroids_preview_direct_launch=true
4+
- workspace_manager_not_used_for_preview=true
5+
- runtime_changes=minimal link/config only
6+
- no validators added
Lines changed: 8 additions & 40 deletions
Original file line numberDiff line numberDiff line change
@@ -1,46 +1,14 @@
11
MODEL: GPT-5.3-codex
2-
REASONING: high
32

43
TASK:
5-
Apply BUILD_PR_LEVEL_8_28_MANIFEST_SSOT_IMPLEMENTATION_ASTEROIDS_FIRST.
4+
Fix Asteroids preview to launch game directly.
65

76
STEPS:
8-
1. Read docs/pr/PLAN_PR_LEVEL_8_28_MANIFEST_SSOT_IMPLEMENTATION_ASTEROIDS_FIRST.md.
9-
2. Read docs/pr/BUILD_PR_LEVEL_8_28_MANIFEST_SSOT_IMPLEMENTATION_ASTEROIDS_FIRST.md.
10-
3. Inspect:
11-
- games/Asteroids/assets/workspace.asset-catalog.json
12-
- games/Asteroids/assets/tools.manifest.json
13-
- games/Asteroids/assets/palettes/asteroids-classic.palette.json
14-
- games/Asteroids/assets/palettes/hud.json
15-
- games/Asteroids/assets/images/bezel.stretch.override.json
16-
4. Create or normalize:
17-
- games/Asteroids/game.manifest.json
18-
5. Preserve existing asset data. Do not delete old catalogs.
19-
6. Ensure game.manifest.json references:
20-
- asteroids-classic.palette.json
21-
- bezel.stretch.override.json
22-
- HUD palette/skin data after conversion
23-
- existing referenced vector/sprite/tilemap/parallax JSON assets
24-
7. If hud.json is color data:
25-
- convert to palette schema as games/Asteroids/assets/palettes/asteroids-hud.palette.json
26-
- do not leave hud.json as loose untyped color data
27-
- if retaining hud.json for compatibility, mark legacy in report
28-
8. Normalize palette colors:
29-
- #RRGGBBFF -> #RRGGBB
30-
- preserve non-FF alpha
31-
9. Write docs/dev/reports/level_8_28_asteroids_manifest_ssot_report.md.
32-
10. Update docs/dev/roadmaps/MASTER_ROADMAP_ENGINE.md status only:
33-
- [ ] -> [.] or [.] -> [x]
34-
- no prose rewrite/delete
35-
11. Do not add validators.
36-
12. Do not modify start_of_day.
37-
13. Avoid runtime code changes unless only a safe manifest input reference is required.
38-
14. Create Codex delta ZIP:
39-
tmp/BUILD_PR_LEVEL_8_28_MANIFEST_SSOT_IMPLEMENTATION_ASTEROIDS_FIRST_delta.zip
7+
1. Locate preview.svg click handler / link source
8+
2. Replace Workspace Manager route with direct game launch route
9+
3. Ensure no gameId/mount query required
10+
4. Do not modify runtime logic beyond routing
11+
5. Do not modify start_of_day
4012

41-
ACCEPTANCE:
42-
- asteroids_game_manifest_exists=true
43-
- asteroids_bezel_override_wired=true
44-
- asteroids_hud_data_typed=true
45-
- old catalogs intact
46-
- no validators/start_of_day changes
13+
OUTPUT:
14+
tmp/BUILD_PR_LEVEL_8_29B_ASTEROIDS_DIRECT_LAUNCH_FIX_delta.zip
Lines changed: 3 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,4 @@
1-
BUILD_PR_LEVEL_8_28_MANIFEST_SSOT_IMPLEMENTATION_ASTEROIDS_FIRST
1+
PR 8.29B: Asteroids direct launch fix
22

3-
- Added Asteroids-first game manifest SSoT plan
4-
- Wired Asteroids palette/HUD/bezel requirements into manifest acceptance
5-
- Kept legacy catalogs intact for parity review
6-
7-
PR Details:
8-
- First concrete SSoT implementation pattern
9-
- No broad all-games rollout
10-
- No validators
11-
- No start_of_day changes
3+
- Preview now launches game directly
4+
- Workspace Manager decoupled from preview
Lines changed: 24 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,24 @@
1+
# BUILD_PR_LEVEL_8_29B_ASTEROIDS_DIRECT_LAUNCH_FIX
2+
3+
## Objective
4+
Correct preview click behavior.
5+
6+
## Current (WRONG)
7+
preview.svg → Workspace Manager route
8+
9+
## Correct (TARGET)
10+
preview.svg → direct game launch route
11+
12+
## Rules
13+
- Preview click launches game directly
14+
- Workspace Manager must NOT be used for preview
15+
- Workspace Manager remains available via separate action
16+
17+
## Expected Route
18+
Use existing game runtime entry:
19+
- games/Asteroids runtime entry point (index/runtime bootstrap)
20+
21+
## Acceptance
22+
- Asteroids preview launches game directly
23+
- No Workspace Manager diagnostic appears
24+
- No query parameters required for direct launch
Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,10 @@
1+
# PLAN_PR_LEVEL_8_29B_ASTEROIDS_DIRECT_LAUNCH_FIX
2+
3+
## Purpose
4+
Fix Asteroids preview behavior so it launches the game directly instead of routing through Workspace Manager.
5+
6+
## Scope
7+
- Update preview link target
8+
- Ensure direct game launch route is used
9+
- Keep Workspace Manager as optional action
10+
- Advance roadmap status only

games/Asteroids/index.html

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -12,11 +12,6 @@
1212
<link rel="stylesheet" href="/src/engine/ui/baseLayout.css" />
1313
<link rel="stylesheet" href="/games/shared/styles/vectorBattleFont.css" />
1414
<link rel="stylesheet" href="../../src/engine/theme/main.css" />
15-
<script type="module">
16-
import { bootWorkspaceGame } from "/games/shared/workspaceGameBoot.js";
17-
bootWorkspaceGame("Asteroids");
18-
</script>
19-
2015
</head>
2116
<body class="hub-page-games">
2217
<div id="shared-theme-header"></div>

0 commit comments

Comments
 (0)