Skip to content

Commit 9a30554

Browse files
author
DavidQ
committed
Execute approved game-to-sample reclassification moves.
BUILD_PR_LEVEL_18_4_GAME_TO_SAMPLE_RECLASSIFICATION_EXECUTION - move approved game/demo entries into recommended sample phase targets - leave PacmanLite unchanged - update exact references required by the moves - update master roadmap in place under roadmap guard rules
1 parent b7919a8 commit 9a30554

210 files changed

Lines changed: 458 additions & 199 deletions

File tree

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: 25 additions & 34 deletions
Original file line numberDiff line numberDiff line change
@@ -1,39 +1,30 @@
11
MODEL: GPT-5.3-codex
22
REASONING: high
33

4-
Execute BUILD_PR_LEVEL_18_3_GAME_TO_SAMPLE_RECLASSIFICATION_RECOMMENDATIONS.
4+
Execute BUILD_PR_LEVEL_18_4_GAME_TO_SAMPLE_RECLASSIFICATION_EXECUTION.
55

6-
Primary task:
7-
- inspect and classify these entries:
8-
- games/Bouncing-ball
9-
- games/Gravity
10-
- games/Thruster
11-
- games/ProjectileLab
12-
- games/Orbit
13-
- games/PaddleIntercept
14-
- games/MultiBallChaos
15-
- games/PacmanLite
6+
Required behavior:
7+
1. Use the prior reviewed recommendations as the source of truth.
8+
2. Execute only the approved reclassification moves for:
9+
- games/Bouncing-ball
10+
- games/Gravity
11+
- games/Thruster
12+
- games/ProjectileLab
13+
- games/Orbit
14+
- games/PaddleIntercept
15+
- games/MultiBallChaos
16+
3. Do not change games/PacmanLite.
17+
4. Apply only the exact path/reference updates needed to keep the moved entries valid.
18+
5. Do not expand into unrelated cleanup or refactors.
19+
6. Write execution-backed reports to docs/dev/reports.
20+
7. Update docs/dev/roadmaps/MASTER_ROADMAP_HIGH_LEVEL.md in place under these rules:
21+
- never delete roadmap content
22+
- never rewrite roadmap text
23+
- only update status markers:
24+
- [ ] -> [.]
25+
- [.] -> [x]
26+
- append additive content only if explicitly required by this PR
1627

17-
Required output:
18-
1. Determine for each entry:
19-
- remain in games
20-
- or recommend move to samples
21-
2. For each sample/demo recommendation, assign the best-fit `phase-xx` move target.
22-
3. Write the classification matrix and rationale to docs/dev/reports.
23-
24-
Roadmap requirement for this bundle:
25-
- update docs/dev/roadmaps/MASTER_ROADMAP_HIGH_LEVEL.md in place
26-
- preserve all existing roadmap text
27-
- never delete roadmap content
28-
- never rewrite existing roadmap text
29-
- only update status markers using:
30-
- [ ] -> [.]
31-
- [.] -> [x]
32-
- append additive roadmap content only if explicitly required by this PR
33-
34-
Constraints:
35-
- no broad repo cleanup
36-
- no unrelated refactors
37-
- keep scope tightly limited to classification + roadmap update
38-
- package final ZIP to:
39-
<project folder>/tmp/BUILD_PR_LEVEL_18_3_GAME_TO_SAMPLE_RECLASSIFICATION_RECOMMENDATIONS.zip
28+
Packaging:
29+
- produce final ZIP at:
30+
<project folder>/tmp/BUILD_PR_LEVEL_18_4_GAME_TO_SAMPLE_RECLASSIFICATION_EXECUTION.zip

‎docs/dev/COMMIT_COMMENT.txt‎

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
1-
Classify game entries that are actually sample/demo candidates and recommend phase move targets.
1+
Execute approved game-to-sample reclassification moves.
22

3-
BUILD_PR_LEVEL_18_3_GAME_TO_SAMPLE_RECLASSIFICATION_RECOMMENDATIONS
4-
- inspect listed games for true game vs sample/demo classification
5-
- recommend best-fit phase target for sample/demo entries
3+
BUILD_PR_LEVEL_18_4_GAME_TO_SAMPLE_RECLASSIFICATION_EXECUTION
4+
- move approved game/demo entries into recommended sample phase targets
5+
- leave PacmanLite unchanged
6+
- update exact references required by the moves
67
- update master roadmap in place under roadmap guard rules

‎docs/dev/reports/BUILD_PR_LEVEL_18_4_GAME_TO_SAMPLE_RECLASSIFICATION_EXECUTION_summary.md‎

Lines changed: 38 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,38 @@
1+
# BUILD_PR_LEVEL_18_4_GAME_TO_SAMPLE_RECLASSIFICATION_EXECUTION
2+
3+
## Execution Summary
4+
5+
Executed approved game-to-sample reclassification moves from the prior reviewed matrix.
6+
7+
### Approved Outcomes Applied
8+
9+
| Source | Action | Destination |
10+
|---|---|---|
11+
| `games/Bouncing-ball` | remain in `games` | n/a |
12+
| `games/Gravity` | moved to `samples` | `samples/phase-03/0325` |
13+
| `games/Thruster` | moved to `samples` | `samples/phase-04/0413` |
14+
| `games/ProjectileLab` | moved to `samples` | `samples/phase-02/0225` |
15+
| `games/Orbit` | moved to `samples` | `samples/phase-06/0614` |
16+
| `games/PaddleIntercept` | moved to `samples` | `samples/phase-03/0326` |
17+
| `games/MultiBallChaos` | moved to `samples` | `samples/phase-03/0327` |
18+
19+
### Explicit Exclusion Preserved
20+
21+
- `games/PacmanLite` was not modified and was not moved.
22+
23+
### Exact Reference Updates Applied
24+
25+
- Removed moved entries from `games/index.html` so no stale `/games/...` links remain for moved entries.
26+
- Added new sample entries in `samples/index.html` for:
27+
- `0225` Projectile Lab
28+
- `0325` Gravity Arcade
29+
- `0326` Paddle Intercept
30+
- `0327` Multi-Ball Chaos
31+
- `0413` Thruster Inertia Lab
32+
- `0614` Orbit Lab
33+
- Updated focused test imports to the new sample paths for Gravity/Thruster/PaddleIntercept/MultiBallChaos/ProjectileLab/Orbit surfaces.
34+
35+
### Roadmap Handling
36+
37+
- Roadmap file preserved in place with no text rewrite/deletion.
38+
- No new status-marker transition was applied in this PR because no additional eligible `[ ]` or `[.]` marker directly represented this already-approved execution lane without introducing unrelated scope.

‎docs/dev/reports/BUILD_PR_LEVEL_18_4_GAME_TO_SAMPLE_RECLASSIFICATION_EXECUTION_validation.md‎

Lines changed: 29 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,29 @@
1+
# BUILD_PR_LEVEL_18_4_GAME_TO_SAMPLE_RECLASSIFICATION_EXECUTION Validation
2+
3+
## Commands Run
4+
5+
1. Targeted move validation (existence checks):
6+
- verified moved source paths no longer exist under `games/`
7+
- verified new sample targets exist under `samples/phase-02/0225`, `samples/phase-03/0325-0327`, `samples/phase-04/0413`, `samples/phase-06/0614`
8+
2. Targeted stale-reference scans:
9+
- `games/index.html` for removed `/games/(Gravity|Thruster|ProjectileLab|Orbit|PaddleIntercept|MultiBallChaos)/` links
10+
- `tests/games` + `tests/samples` for stale `../../games/...` imports
11+
3. Focused runtime/unit validation via alias-hook runner:
12+
- `tests/games/GravityValidation.test.mjs`
13+
- `tests/games/GravityWorld.test.mjs`
14+
- `tests/games/ThrusterValidation.test.mjs`
15+
- `tests/games/ThrusterWorld.test.mjs`
16+
- `tests/games/PaddleInterceptValidation.test.mjs`
17+
- `tests/games/PaddleInterceptWorld.test.mjs`
18+
- `tests/games/MultiBallChaosValidation.test.mjs`
19+
- `tests/games/MultiBallChaosWorld.test.mjs`
20+
- `tests/samples/ProjectileLabModel.test.mjs`
21+
- `tests/samples/ProjectileLabScene.test.mjs`
22+
- `tests/samples/OrbitLabModel.test.mjs`
23+
- `tests/samples/OrbitLabScene.test.mjs`
24+
25+
## Results
26+
27+
- All 12 targeted tests passed.
28+
- No stale moved-entry links remained in `games/index.html` for the reclassified entries.
29+
- No stale `../../games/(Gravity|Thruster|ProjectileLab|Orbit|PaddleIntercept|MultiBallChaos)/` imports remained in focused test files.

‎docs/dev/reports/change_summary.txt‎

Lines changed: 37 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,38 @@
1-
Docs-only PR bundle for classification of selected game entries as true games vs sample/demo candidates.
1+
BUILD_PR_LEVEL_18_4_GAME_TO_SAMPLE_RECLASSIFICATION_EXECUTION
22

3-
This PR asks Codex to:
4-
- inspect the listed entries
5-
- recommend phase move targets for sample/demo candidates
6-
- update the master roadmap in place under the existing roadmap guard
3+
Scope executed:
4+
- Applied only approved reclassification moves from prior reviewed recommendations for the requested set.
5+
- Left `games/Bouncing-ball` in `games` (approved remain-in-games decision).
6+
- Left `games/PacmanLite` untouched (explicit exclusion).
7+
8+
Executed moves:
9+
- `games/ProjectileLab` -> `samples/phase-02/0225`
10+
- `games/Gravity` -> `samples/phase-03/0325`
11+
- `games/PaddleIntercept` -> `samples/phase-03/0326`
12+
- `games/MultiBallChaos` -> `samples/phase-03/0327`
13+
- `games/Thruster` -> `samples/phase-04/0413`
14+
- `games/Orbit` -> `samples/phase-06/0614`
15+
16+
Exact reference updates applied:
17+
- `games/index.html`
18+
- removed cards and list references for moved entries
19+
- `samples/index.html`
20+
- added sample entries 0225, 0325, 0326, 0327, 0413, 0614
21+
- updated focused imports in tests:
22+
- `tests/games/GravityValidation.test.mjs`
23+
- `tests/games/GravityWorld.test.mjs`
24+
- `tests/games/ThrusterValidation.test.mjs`
25+
- `tests/games/ThrusterWorld.test.mjs`
26+
- `tests/games/PaddleInterceptValidation.test.mjs`
27+
- `tests/games/PaddleInterceptWorld.test.mjs`
28+
- `tests/games/MultiBallChaosValidation.test.mjs`
29+
- `tests/games/MultiBallChaosWorld.test.mjs`
30+
- `tests/samples/ProjectileLabModel.test.mjs`
31+
- `tests/samples/ProjectileLabScene.test.mjs`
32+
- `tests/samples/OrbitLabModel.test.mjs`
33+
- `tests/samples/OrbitLabScene.test.mjs`
34+
35+
Roadmap handling:
36+
- roadmap file preserved in place.
37+
- no text rewrite/deletion performed.
38+
- no additional marker transition applied in this PR because no directly matching pending status marker for this execution lane was present without expanding scope.

‎docs/dev/reports/file_tree.txt‎

Lines changed: 30 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,34 @@
1-
docs/
2-
docs/pr/
3-
docs/pr/BUILD_PR_LEVEL_18_3_GAME_TO_SAMPLE_RECLASSIFICATION_RECOMMENDATIONS.md
4-
docs/dev/
5-
docs/dev/codex_commands.md
6-
docs/dev/commit_comment.txt
71
docs/dev/reports/
82
docs/dev/reports/change_summary.txt
93
docs/dev/reports/validation_checklist.txt
104
docs/dev/reports/file_tree.txt
5+
docs/dev/reports/BUILD_PR_LEVEL_18_4_GAME_TO_SAMPLE_RECLASSIFICATION_EXECUTION_summary.md
6+
docs/dev/reports/BUILD_PR_LEVEL_18_4_GAME_TO_SAMPLE_RECLASSIFICATION_EXECUTION_validation.md
7+
8+
games/
9+
games/index.html
10+
11+
samples/
12+
samples/index.html
13+
samples/phase-02/0225/ (moved full folder from games/ProjectileLab)
14+
samples/phase-03/0325/ (moved full folder from games/Gravity)
15+
samples/phase-03/0326/ (moved full folder from games/PaddleIntercept)
16+
samples/phase-03/0327/ (moved full folder from games/MultiBallChaos)
17+
samples/phase-04/0413/ (moved full folder from games/Thruster)
18+
samples/phase-06/0614/ (moved full folder from games/Orbit)
19+
20+
tests/games/
21+
tests/games/GravityValidation.test.mjs
22+
tests/games/GravityWorld.test.mjs
23+
tests/games/ThrusterValidation.test.mjs
24+
tests/games/ThrusterWorld.test.mjs
25+
tests/games/PaddleInterceptValidation.test.mjs
26+
tests/games/PaddleInterceptWorld.test.mjs
27+
tests/games/MultiBallChaosValidation.test.mjs
28+
tests/games/MultiBallChaosWorld.test.mjs
29+
30+
tests/samples/
31+
tests/samples/ProjectileLabModel.test.mjs
32+
tests/samples/ProjectileLabScene.test.mjs
33+
tests/samples/OrbitLabModel.test.mjs
34+
tests/samples/OrbitLabScene.test.mjs

‎docs/dev/reports/validation_checklist.txt‎

Lines changed: 17 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,17 @@
1-
[ ] All listed entries inspected
2-
[ ] Each entry classified as remain-in-games or move-to-samples
3-
[ ] Each sample/demo candidate mapped to a best-fit phase target
4-
[ ] Classification rationale written to docs/dev/reports
5-
[ ] Master roadmap updated in place by Codex only under roadmap guard rules
6-
[ ] No roadmap text deleted or rewritten
1+
[x] Read BUILD doc: `docs/pr/BUILD_PR_LEVEL_18_4_GAME_TO_SAMPLE_RECLASSIFICATION_EXECUTION.md`
2+
[x] Used prior approved recommendation matrix as source of truth
3+
[x] Applied approved moves only for requested set (excluding PacmanLite)
4+
[x] Kept `games/Bouncing-ball` in games (approved remain-in-games)
5+
[x] Left `games/PacmanLite` unchanged
6+
[x] Updated only required references:
7+
- `games/index.html`
8+
- `samples/index.html`
9+
- focused moved-path test imports
10+
[x] Wrote execution-backed reports:
11+
- `docs/dev/reports/BUILD_PR_LEVEL_18_4_GAME_TO_SAMPLE_RECLASSIFICATION_EXECUTION_summary.md`
12+
- `docs/dev/reports/BUILD_PR_LEVEL_18_4_GAME_TO_SAMPLE_RECLASSIFICATION_EXECUTION_validation.md`
13+
[x] Validation: moved-path existence checks passed
14+
[x] Validation: stale moved-path reference scan passed for targeted files
15+
[x] Validation: 12 focused tests passed
16+
[ ] Roadmap status transition applied
17+
- no eligible directly matching pending marker existed for this execution lane without unrelated scope expansion; roadmap text was preserved unchanged

‎docs/pr/BUILD_PR_LEVEL_18_4_GAME_TO_SAMPLE_RECLASSIFICATION_EXECUTION.md‎

Lines changed: 46 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,46 @@
1+
# BUILD_PR_LEVEL_18_4_GAME_TO_SAMPLE_RECLASSIFICATION_EXECUTION
2+
3+
## Purpose
4+
Complete the Level 18 game-to-sample reclassification lane by executing the approved move recommendations from the prior review.
5+
6+
## Explicit Exclusion
7+
- `games/PacmanLite` must be left alone in this PR.
8+
9+
## In-Scope Candidates
10+
Execute only the approved reclassification work for the reviewed set, excluding PacmanLite:
11+
- `games/Bouncing-ball`
12+
- `games/Gravity`
13+
- `games/Thruster`
14+
- `games/ProjectileLab`
15+
- `games/Orbit`
16+
- `games/PaddleIntercept`
17+
- `games/MultiBallChaos`
18+
19+
## Scope
20+
- one PR purpose only
21+
- execute approved game-to-sample reclassification work
22+
- docs-first bundle
23+
- no implementation code authored by ChatGPT
24+
- PacmanLite excluded
25+
- roadmap update must be performed in place by Codex under roadmap guard rules
26+
27+
## Codex Responsibilities
28+
1. Read the prior recommendation outputs and user-approved direction.
29+
2. Reclassify only the approved entries from `games/` to their recommended `samples/phase-xx` targets.
30+
3. Leave `games/PacmanLite` untouched.
31+
4. Preserve repo structure consistency and fix only the exact references required by these moves.
32+
5. Keep scope tightly limited to the approved reclassification set.
33+
6. Write execution-backed reports under `docs/dev/reports`.
34+
7. Update `docs/dev/roadmaps/MASTER_ROADMAP_HIGH_LEVEL.md` in place:
35+
- preserve existing text
36+
- do not delete content
37+
- do not rewrite text
38+
- update status markers only when execution-backed
39+
- append only if explicitly required by this PR
40+
41+
## Acceptance
42+
- approved reclassification moves are executed
43+
- `games/PacmanLite` remains unchanged
44+
- exact path/reference updates required by the moves are completed
45+
- reports are written under `docs/dev/reports`
46+
- roadmap is updated in place under guard rules

0 commit comments

Comments
 (0)