Skip to content

Commit 49a167a

Browse files
author
DavidQ
committed
PLAN_PR_GAMES_PACMANLITE_MIGRATION
Planned migration of PacmanLite using the validated pipeline. - template baseline - _next staging - controlled gameplay migration - canonical promotion - cleanup
1 parent 138fab5 commit 49a167a

9 files changed

Lines changed: 206 additions & 25 deletions

‎docs/dev/CODEX_COMMANDS.md‎

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

44
COMMAND:
5-
Execute BUILD_PR_GAMES_SPACE_INVADERS_REMOVE_NEXT exactly as written.
5+
Execute BUILD_PR_GAMES_PACMANLITE_NEXT_TEMPLATE_BASELINE exactly as written.
66

77
Rules:
8-
- Delete ONLY `games/SpaceInvaders_next/**`
9-
- Remove `games/SpaceInvaders_next/` after contents are deleted
10-
- Do NOT touch `games/SpaceInvaders/**`
11-
- Do NOT add or restore preview files
12-
- Do NOT modify engine/shared code
13-
- Fail fast on ambiguity
8+
- use `games/_template/**` as source of truth
9+
- create `games/PacmanLite_next/**`
10+
- keep `games/PacmanLite/**` unchanged
11+
- keep `_next` baseline non-playable
12+
- required status text must render on canvas
13+
- fail fast on ambiguity
1414

1515
Package output to:
16-
<project folder>/tmp/BUILD_PR_GAMES_SPACE_INVADERS_REMOVE_NEXT_delta.zip
16+
<project folder>/tmp/BUILD_PR_GAMES_PACMANLITE_NEXT_TEMPLATE_BASELINE_delta.zip

‎docs/dev/COMMIT_COMMENT.txt‎

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
1-
APPLY_PR_GAMES_SPACE_INVADERS_REMOVE_NEXT
1+
BUILD_PR_GAMES_PACMANLITE_NEXT_TEMPLATE_BASELINE
22

3-
Accepted removal of SpaceInvaders_next scaffold.
3+
Prepared PacmanLite next baseline BUILD from template.
44

5-
- canonical SpaceInvaders preserved
6-
- migration pipeline completed
7-
- repo cleaned
5+
- source of truth: games/_template
6+
- destination: games/PacmanLite_next
7+
- non-playable shell with canvas status text

‎docs/dev/NEXT_COMMAND.txt‎

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
1-
Create PLAN_PR_NEXT_GAME_MIGRATION
1+
Create APPLY_PR_GAMES_PACMANLITE_NEXT_TEMPLATE_BASELINE
22

33
Scope:
4-
- select next game (e.g., Puckman)
5-
- repeat template → _next → migration → promotion pipeline
4+
- accept baseline copy
5+
- verify non-playable shell behavior
6+
- verify required canvas status text

‎docs/dev/reports/change_summary.txt‎

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1,5 @@
1-
Wire debug init + keybindings
1+
Planned PacmanLite migration handoff to first BUILD step.
2+
3+
- Added BUILD_PR_GAMES_PACMANLITE_NEXT_TEMPLATE_BASELINE
4+
- Updated active command to execute baseline BUILD
5+
- Updated next command to APPLY baseline acceptance

‎docs/dev/reports/file_tree.txt‎

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,8 @@
1-
docs/pr/BUILD_PR_GAMES_102C_ASTEROIDS_NEW_GAME_OVER_SCREEN_TIMER.md
1+
docs/pr/PLAN_PR_GAMES_PACMANLITE_MIGRATION.md
2+
docs/pr/BUILD_PR_GAMES_PACMANLITE_NEXT_TEMPLATE_BASELINE.md
23
docs/dev/codex_commands.md
34
docs/dev/commit_comment.txt
45
docs/dev/next_command.txt
56
docs/dev/reports/file_tree.txt
67
docs/dev/reports/change_summary.txt
78
docs/dev/reports/validation_checklist.txt
8-
games/asteroids_new/game/AsteroidsGameScene.js
9-
games/asteroids_new/game/AsteroidsSession.js

‎docs/dev/reports/validation_checklist.txt‎

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,9 @@
1-
Validation Checklist — APPLY Games Space Invaders Remove Next
1+
Validation Checklist — PLAN PR Games PacmanLite Migration
22

3-
[x] games/SpaceInvaders_next removed
4-
[x] games/SpaceInvaders unchanged
5-
[x] No preview restoration
6-
[x] No unrelated changes
3+
[x] Validated migration pipeline referenced
4+
[x] Sequence preserved with no skipped steps
5+
[x] Scope constrained to first BUILD handoff
6+
[x] Fail-fast rules retained in generated BUILD doc
7+
[x] No runtime/gameplay code touched
78

89
Status: PASS

‎docs/pr/BUILD_PR_GAMES_PACMANLITE_NEXT_TEMPLATE_BASELINE.md‎

Lines changed: 97 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,97 @@
1+
# BUILD PR — Games PacmanLite Next Template Baseline
2+
3+
## Purpose
4+
Create a safe, non-destructive template-aligned baseline alongside the existing PacmanLite game.
5+
6+
## Why This Shape
7+
We want less back-and-forth, but we still need one clear PR purpose.
8+
This BUILD bundles all baseline setup work needed for the next migration lane while keeping the current playable game intact.
9+
10+
## Single PR Purpose
11+
Establish `games/PacmanLite_next/**` as the accepted template-based migration baseline.
12+
13+
## Scope (STRICT)
14+
- Create `games/PacmanLite_next/**`
15+
- Use `games/_template/**` as the source of truth
16+
- Preserve the current `games/PacmanLite/**` implementation unchanged
17+
- Keep `games/PacmanLite_next/**` non-playable in this PR
18+
- Make the new baseline immediately testable in browser
19+
20+
## Source Of Truth
21+
- `games/_template/**`
22+
23+
## Target Paths (EXACT)
24+
Source:
25+
- `games/_template/**`
26+
27+
Destination:
28+
- `games/PacmanLite_next/**`
29+
30+
## Bundled Tasks Included In This BUILD
31+
1. Copy/adapt the accepted template baseline into `games/PacmanLite_next/**`
32+
2. Preserve the minimal reusable folder skeleton
33+
3. Preserve the aligned engine/theme shell baseline
34+
4. Ensure `games/PacmanLite_next/index.html` is browser-openable and testable
35+
5. Ensure the required status text renders on the canvas
36+
6. Ensure no gameplay boots
37+
7. Ensure no current PacmanLite files are modified
38+
39+
## Required Behavior
40+
`games/PacmanLite_next/index.html` MUST:
41+
- show the aligned shell/theme baseline from the accepted template
42+
- render a visible canvas
43+
- render this text on the canvas:
44+
45+
HTML Says
46+
Template Status
47+
This template intentionally does not boot gameplay.
48+
49+
- not boot gameplay
50+
- not load PacmanLite gameplay
51+
- not load Asteroids gameplay
52+
- not depend on copied game-specific assets
53+
54+
## Allowed Operations
55+
- create `games/PacmanLite_next/**`
56+
- copy/adapt files from `games/_template/**`
57+
- keep `.gitkeep` placeholders
58+
- adjust only what is required so the copied baseline works correctly at the new destination path
59+
60+
## Explicit Non-Goals
61+
- DO NOT modify `games/PacmanLite/**`
62+
- DO NOT migrate gameplay in this PR
63+
- DO NOT perform canonical replacement in this PR
64+
- DO NOT refactor engine/shared code
65+
- DO NOT change other games
66+
- DO NOT add PacmanLite assets, ghosts, maps, scoring, audio, or rules
67+
- DO NOT expand `_next` beyond the accepted template baseline
68+
69+
## Fail-Fast Conditions
70+
STOP with no changes if any of the following are true:
71+
- creating `games/PacmanLite_next/**` would require modifying `games/PacmanLite/**`
72+
- the baseline cannot work at the new destination without gameplay code
73+
- any path outside `games/PacmanLite_next/**` must be changed, other than reading `games/_template/**`
74+
- more than baseline/template alignment is required
75+
76+
## Acceptance Criteria
77+
- `games/PacmanLite_next/**` exists
78+
- it matches the accepted template baseline structure
79+
- canvas is visible
80+
- required status text renders on the canvas
81+
- no gameplay boots
82+
- no console errors
83+
- `games/PacmanLite/**` remains unchanged
84+
85+
## Validation Steps
86+
1. Open `games/PacmanLite_next/index.html`
87+
2. Confirm:
88+
- canvas is visible
89+
- shell/theme baseline is present
90+
- the three required lines render on the canvas
91+
- no gameplay starts
92+
- no console errors
93+
3. Confirm `games/PacmanLite/**` was not modified
94+
95+
## Output Requirement
96+
Codex must package:
97+
`<project folder>/tmp/BUILD_PR_GAMES_PACMANLITE_NEXT_TEMPLATE_BASELINE_delta.zip`

‎docs/pr/PLAN_PR_GAMES_PACMANLITE_MIGRATION.md‎

Lines changed: 42 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,42 @@
1+
# PLAN PR — Games PacmanLite Migration
2+
3+
## Purpose
4+
Plan migration of `games/PacmanLite/**` using the validated template → _next → migration → promotion → cleanup pipeline.
5+
6+
## Selected Game
7+
- `games/PacmanLite/**`
8+
9+
## Why PacmanLite
10+
- lower-risk next migration than `PacmanFullAI`
11+
- better fit for the now-proven migration pipeline
12+
- keeps scope controlled while extending the standard to another real game
13+
14+
## Validated Pipeline To Reuse
15+
`games/_template`
16+
→ `games/PacmanLite_next` baseline
17+
→ gameplay migration into `_next`
18+
→ clear canonical destination
19+
→ copy `_next` to canonical
20+
→ remove `_next`
21+
22+
## Planned Sequence
23+
1. `BUILD_PR_GAMES_PACMANLITE_NEXT_TEMPLATE_BASELINE`
24+
2. `APPLY_PR_GAMES_PACMANLITE_NEXT_TEMPLATE_BASELINE`
25+
3. `BUILD_PR_GAMES_PACMANLITE_GAMEPLAY_MIGRATION_TO_NEXT`
26+
4. `APPLY_PR_GAMES_PACMANLITE_GAMEPLAY_MIGRATION_TO_NEXT`
27+
5. `BUILD_PR_GAMES_PACMANLITE_CLEAR_DESTINATION`
28+
6. `BUILD_PR_GAMES_PACMANLITE_COPY_FROM_NEXT`
29+
7. `APPLY_PR_GAMES_PACMANLITE_COPY_FROM_NEXT`
30+
8. `BUILD_PR_GAMES_PACMANLITE_REMOVE_NEXT`
31+
9. `APPLY_PR_GAMES_PACMANLITE_REMOVE_NEXT`
32+
33+
## Rules
34+
- no direct edits to canonical until promotion
35+
- no gameplay during baseline step
36+
- no guessing missing files
37+
- fail fast on ambiguity
38+
- docs-first always
39+
- keep one PR purpose per step
40+
41+
## Output Requirement
42+
Follow the BUILD/APPLY sequence exactly as listed.

‎docs/pr/PLAN_PR_GAMES_PUCKMAN_MIGRATION.md‎

Lines changed: 37 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,37 @@
1+
# PLAN PR — Games Puckman Migration
2+
3+
## Purpose
4+
Plan migration of Puckman using the proven template → _next → migration → promotion pipeline.
5+
6+
## Selected Game
7+
- games/Puckman/**
8+
9+
## Strategy
10+
Replicate validated SpaceInvaders/Asteroids pipeline:
11+
12+
_template
13+
→ Puckman_next (baseline)
14+
→ gameplay migration
15+
→ canonical promotion
16+
→ cleanup
17+
18+
## Phases
19+
1. BUILD_PR_GAMES_PUCKMAN_NEXT_TEMPLATE_BASELINE
20+
2. APPLY
21+
3. BUILD_PR_GAMES_PUCKMAN_GAMEPLAY_MIGRATION_TO_NEXT
22+
4. APPLY
23+
5. BUILD_PR_GAMES_PUCKMAN_CLEAR_DESTINATION
24+
6. BUILD_PR_GAMES_PUCKMAN_COPY_FROM_NEXT
25+
7. APPLY
26+
8. BUILD_PR_GAMES_PUCKMAN_REMOVE_NEXT
27+
9. APPLY
28+
29+
## Rules
30+
- no direct edits to canonical until promotion
31+
- no gameplay during baseline step
32+
- no guessing files
33+
- fail fast on ambiguity
34+
- docs-first always
35+
36+
## Output Requirement
37+
Codex must follow BUILD sequence exactly per phase.

0 commit comments

Comments
 (0)