Skip to content

Commit b7919a8

Browse files
author
DavidQ
committed
Classify game entries that are actually sample/demo candidates and recommend phase move targets.
BUILD_PR_LEVEL_18_3_GAME_TO_SAMPLE_RECLASSIFICATION_RECOMMENDATIONS - inspect listed games for true game vs sample/demo classification - recommend best-fit phase target for sample/demo entries - update master roadmap in place under roadmap guard rules
1 parent 11d7add commit b7919a8

61 files changed

Lines changed: 188 additions & 319 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: 36 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,39 @@
11
MODEL: GPT-5.3-codex
22
REASONING: high
33

4-
Execute BUILD_PR_LEVEL_18_2_BOUNDARY_HARDENING_ENFORCEMENT:
5-
6-
1. Inspect engine/shared/games/tools boundaries.
7-
2. Identify cross-layer leakage and dependency-direction violations.
8-
3. Apply the smallest valid implementation needed to harden boundaries.
9-
4. Remove or relocate accidental coupling without broad refactors.
10-
5. Re-run validation and document exact findings in docs/dev/reports.
11-
6. If roadmap status is execution-backed, update status markers only in place.
12-
7. Package final ZIP to:
13-
<project folder>/tmp/BUILD_PR_LEVEL_18_2_BOUNDARY_HARDENING_ENFORCEMENT.zip
4+
Execute BUILD_PR_LEVEL_18_3_GAME_TO_SAMPLE_RECLASSIFICATION_RECOMMENDATIONS.
5+
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
16+
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

docs/dev/COMMIT_COMMENT.txt

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
1-
Boundary hardening enforcement for Level 18 Track B
1+
Classify game entries that are actually sample/demo candidates and recommend phase move targets.
22

3-
BUILD_PR_LEVEL_18_2_BOUNDARY_HARDENING_ENFORCEMENT
4-
- enforce engine/shared/games/tools boundaries
5-
- reduce cross-layer leakage
6-
- validate dependency direction after changes
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
6+
- update master roadmap in place under roadmap guard rules
Lines changed: 20 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,20 @@
1+
# BUILD_PR_LEVEL_18_3_GAME_TO_SAMPLE_RECLASSIFICATION_RECOMMENDATIONS
2+
3+
## Classification Matrix
4+
5+
| Entry | Classification | Recommended `phase-xx` Target (if move) | Rationale |
6+
|---|---|---|---|
7+
| `games/Bouncing-ball` | Remain in `games/` | n/a | Serves as the canonical minimal fully-playable arcade game in the Level 1 progression and is not marked as sample-track in the games roadmap carding. |
8+
| `games/Gravity` | Recommend move to `samples/` | `phase-03` | Primarily a focused gravity-mechanics demonstration (force, damping, rebound behavior), which aligns better with gameplay-physics teaching slices than product-facing game catalog placement. |
9+
| `games/Thruster` | Recommend move to `samples/` | `phase-04` | Core value is movement feel/input-response exploration (rotation + thrust + drift), which fits camera/feel/input tuning samples better than standalone game inventory. |
10+
| `games/ProjectileLab` | Recommend move to `samples/` | `phase-02` | Explicitly a projectile behavior lab; best aligned to early world/system-building sample phases that already introduce projectile runtime patterns. |
11+
| `games/Orbit` | Recommend move to `samples/` | `phase-06` | Orbits/trajectory shaping are simulation-first mechanics with educational value; this is a better fit for world-simulation sample sequencing than game catalog placement. |
12+
| `games/PaddleIntercept` | Recommend move to `samples/` | `phase-03` | Intercept prediction and collision-response behavior are collision mastery demonstrations and map cleanly to gameplay-systems sample instruction. |
13+
| `games/MultiBallChaos` | Recommend move to `samples/` | `phase-03` | Multi-entity collision/rebound stress is a systems demo for collision stability and pacing, best suited to collision-focused sample phases. |
14+
| `games/PacmanLite` | Recommend move to `samples/` | `phase-13` | Already has a direct curriculum analogue (`Sample 1313 - Pacman Lite World Systems`), so the strongest fit is phase-13 sample treatment rather than duplicated game-track ownership. |
15+
16+
## Notes
17+
18+
- This BUILD bundle performs classification/recommendation only.
19+
- No game-to-sample moves are executed in this PR.
20+
- Roadmap status update is execution-backed by this completed matrix.
Lines changed: 5 additions & 36 deletions
Original file line numberDiff line numberDiff line change
@@ -1,37 +1,6 @@
1-
BUILD_PR_LEVEL_18_2_BOUNDARY_HARDENING_ENFORCEMENT
2-
Date: 2026-04-17
3-
Scope: Engine/shared/games/tools boundary hardening with smallest valid coupling removal.
1+
Docs-only PR bundle for classification of selected game entries as true games vs sample/demo candidates.
42

5-
Summary
6-
1) Added boundary guard implementation
7-
- Added tools/dev/checkBoundaryHardeningGuard.mjs
8-
- Scans src/engine, src/shared, games, tools for dependency-direction violations.
9-
- Enforces:
10-
- engine must not depend on games/tools/samples/docs
11-
- shared must not depend on engine/games/tools/samples/docs
12-
- games must not depend on tools/samples/docs
13-
- tools must not depend on games/samples/docs
14-
15-
2) Findings before fix (execution evidence)
16-
- games/breakout/main.js imported ../../tools/dev/devConsoleIntegration.js
17-
- games/Asteroids/index.js imported ../../tools/dev/devConsoleIntegration.js
18-
- These were cross-layer leakage from games -> tools.
19-
20-
3) Smallest valid hardening implementation applied
21-
- Added src/shared/utils/createNoopDevConsoleIntegration.js as a boundary-safe default integration surface.
22-
- Updated games/breakout/main.js to use createNoopDevConsoleIntegration default.
23-
- Updated games/Asteroids/index.js to use createNoopDevConsoleIntegration default.
24-
- Removed direct games -> tools imports for dev console integration.
25-
26-
4) Roadmap status-only update (execution-backed)
27-
- docs/dev/roadmaps/MASTER_ROADMAP_HIGH_LEVEL.md
28-
- Level 18 Track B updated in place:
29-
- [.] -> [x] enforce engine vs shared vs game vs tool boundaries
30-
- [.] -> [x] eliminate cross-layer leakage
31-
- [.] -> [x] validate dependency direction rules across repo
32-
- [ ] -> [x] remove accidental coupling
33-
34-
Notes
35-
- No broad refactor performed.
36-
- No start_of_day edits.
37-
- No engine-core behavior changes introduced beyond boundary-safe debug integration defaulting for two games.
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

docs/dev/reports/file_tree.txt

Lines changed: 9 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -1,26 +1,10 @@
1-
BUILD_PR_LEVEL_18_2_BOUNDARY_HARDENING_ENFORCEMENT file tree
2-
3-
src/
4-
shared/
5-
utils/
6-
createNoopDevConsoleIntegration.js (added)
7-
8-
games/
9-
Asteroids/
10-
index.js (modified)
11-
Breakout/
12-
main.js (modified)
13-
14-
tools/
15-
dev/
16-
checkBoundaryHardeningGuard.mjs (added)
17-
181
docs/
19-
dev/
20-
roadmaps/
21-
MASTER_ROADMAP_HIGH_LEVEL.md (modified: Track B status markers only)
22-
reports/
23-
BUILD_PR_LEVEL_18_2_BOUNDARY_HARDENING_ENFORCEMENT_report.md (added)
24-
change_summary.txt (updated)
25-
validation_checklist.txt (updated)
26-
file_tree.txt (updated)
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
7+
docs/dev/reports/
8+
docs/dev/reports/change_summary.txt
9+
docs/dev/reports/validation_checklist.txt
10+
docs/dev/reports/file_tree.txt
Lines changed: 6 additions & 44 deletions
Original file line numberDiff line numberDiff line change
@@ -1,44 +1,6 @@
1-
BUILD_PR_LEVEL_18_2_BOUNDARY_HARDENING_ENFORCEMENT - Validation Checklist
2-
Date: 2026-04-17
3-
4-
[PASS] Build input read
5-
- docs/pr/BUILD_PR_LEVEL_18_2_BOUNDARY_HARDENING_ENFORCEMENT.md
6-
7-
[PASS] Targeted boundary scan (pre-fix)
8-
Command:
9-
- node tools/dev/checkBoundaryHardeningGuard.mjs
10-
Result:
11-
- scanned files: 712
12-
- violations: 2
13-
- games/breakout/main.js -> ../../tools/dev/devConsoleIntegration.js
14-
- games/Asteroids/index.js -> ../../tools/dev/devConsoleIntegration.js
15-
16-
[PASS] Boundary hardening implementation applied
17-
- Added boundary guard script
18-
- Removed games -> tools coupling in the two violating files
19-
- Added boundary-safe shared no-op integration default
20-
21-
[PASS] Targeted boundary scan (post-fix)
22-
Command:
23-
- node tools/dev/checkBoundaryHardeningGuard.mjs
24-
Result:
25-
- scanned files: 713
26-
- violations: 0
27-
28-
[PASS] Focused regression checks
29-
Commands:
30-
- node --input-type=module -e "... import('./tests/games/AsteroidsValidation.test.mjs') ..."
31-
- node --input-type=module -e "... import('./tests/games/BreakoutValidation.test.mjs') ..."
32-
- node --input-type=module -e "... import('./tests/tools/ToolBoundaryEnforcement.test.mjs') ..."
33-
Results:
34-
- PASS AsteroidsValidation
35-
- PASS BreakoutValidation
36-
- PASS ToolBoundaryEnforcement
37-
38-
[PASS] Roadmap status-only update
39-
- Updated Level 18 Track B markers only in place.
40-
- No roadmap text rewrite.
41-
42-
Validation conclusion
43-
- Cross-layer leakage identified and removed in scoped targets.
44-
- Dependency-direction rules are now enforced by a concrete guard and validated clean.
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

docs/dev/roadmaps/MASTER_ROADMAP_HIGH_LEVEL.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -715,7 +715,7 @@
715715
- all changes must be validation-backed
716716
- no template-driven work
717717

718-
[ ] Some games are actually samples/demos; identify and recommend a `phase-xx` move target.
718+
[x] Some games are actually samples/demos; identify and recommend a `phase-xx` move target.
719719

720720
[ ] Lock APIs
721721
[ ] Clean boundaries
Lines changed: 44 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,44 @@
1+
# BUILD_PR_LEVEL_18_3_GAME_TO_SAMPLE_RECLASSIFICATION_RECOMMENDATIONS
2+
3+
## Purpose
4+
Advance Level 18 by identifying game entries that are actually sample/demo candidates and recommending the correct `phase-xx` move target for each.
5+
6+
## Requested Candidates
7+
- `games/Bouncing-ball`
8+
- `games/Gravity`
9+
- `games/Thruster`
10+
- `games/ProjectileLab`
11+
- `games/Orbit`
12+
- `games/PaddleIntercept`
13+
- `games/MultiBallChaos`
14+
- `games/PacmanLite`
15+
16+
## Scope
17+
- one PR purpose only
18+
- classification + recommendation only
19+
- docs-first bundle
20+
- no implementation code authored by ChatGPT
21+
- no moves in this PR unless Codex determines a tiny execution-backed move is required to keep the PR testable and roadmap-improving
22+
- roadmap must be updated in place as part of this bundle if execution-backed
23+
24+
## Codex Responsibilities
25+
1. Inspect each listed game entry.
26+
2. Determine whether it is:
27+
- a true game that should remain in `games/`
28+
- a sample/demo that should move to `samples/`
29+
3. For each item classified as a sample/demo, recommend the best-fit `phase-xx` target and explain why.
30+
4. Produce a concise classification matrix in `docs/dev/reports`.
31+
5. Update `docs/dev/roadmaps/MASTER_ROADMAP_HIGH_LEVEL.md` in place as part of this bundle:
32+
- preserve all existing roadmap text
33+
- do not delete content
34+
- do not rewrite existing roadmap wording
35+
- only update status markers in place when execution-backed
36+
- only append additive roadmap content when explicitly required by this PR
37+
6. Keep the change tightly scoped to classification and roadmap update behavior.
38+
39+
## Acceptance
40+
- all listed entries are classified
41+
- each sample/demo candidate has a recommended `phase-xx` target
42+
- rationale is documented
43+
- roadmap is updated in place by Codex under the guard rules when execution-backed
44+
- reports are written under `docs/dev/reports`

games/Pacman/assets/.gitkeep

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+

File renamed without changes.

0 commit comments

Comments
 (0)