Skip to content

Commit 8ad3e5a

Browse files
author
DavidQ
committed
Phase 08.07 — correct roadmap scope to core-games complete, repo-wide adoption pending
1 parent e2ba2d3 commit 8ad3e5a

8 files changed

Lines changed: 133 additions & 64 deletions

docs/MASTER_ROADMAP_HIGH_LEVEL.md

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -421,13 +421,15 @@
421421

422422
## 8. Games Layer
423423
- [x] `games/_template/` created
424-
- [ ] game flow pattern standardized (`flow/attract.js`, `flow/intro.js`, `flow/highscore.js`)
424+
- [x] core target games normalized (`Asteroids`, `Puckman`, `SpaceInvaders`)
425425
- [.] per-game structure normalized
426-
- [ ] gameplay/entities/levels/rules/assets boundaries normalized
427-
- [.] current games migrated to target structure
426+
- [.] current games migrated to target structure (repo-wide adoption pending)
428427
- [.] game-specific asset ownership enforced
429-
- [ ] shared-vs-game utility boundaries enforced
430-
- [ ] space_invaders normalized
428+
- [ ] game flow pattern standardized repo-wide (`flow/attract.js`, `flow/intro.js`, `flow/highscore.js`)
429+
- [ ] gameplay/entities/levels/rules/assets boundaries normalized repo-wide
430+
- [ ] shared-vs-game utility boundaries enforced repo-wide
431+
- [x] asteroids normalized
432+
- [x] space_invaders normalized
431433
- [x] puckman normalized
432434
- [ ] future games follow template-first path
433435

docs/dev/CODEX_COMMANDS.md

Lines changed: 22 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,26 @@
1-
MODEL: GPT-5.3-codex
1+
MODEL: GPT-5.4
22
REASONING: medium
33

4-
TASK:
5-
Scan all games/* and report boundary violations.
4+
COMMAND GOAL:
5+
Create a docs-only PR that corrects Section 8 roadmap wording and markers to match the completed
6+
boundary scan.
67

7-
DO NOT FIX.
8-
DO NOT MODIFY.
8+
CONSTRAINTS:
9+
- Docs only
10+
- No code changes
11+
- No repo-wide restructuring
12+
- Preserve completed status for Asteroids, Puckman, and SpaceInvaders
13+
- Distinguish core-games completion from repo-wide adoption
914

10-
OUTPUT:
11-
Return report only.
15+
REQUIRED OUTPUT:
16+
Return ZIP to <project folder>/tmp/ with exact repo-relative structure.
17+
18+
FILES TO UPDATE:
19+
- docs/MASTER_ROADMAP_HIGH_LEVEL.md
20+
- docs/dev/ROADMAP_RULES.md
21+
- docs/pr/BUILD_PR_LEVEL_08_07_ROADMAP_SCOPE_CORRECTION.md
22+
23+
VALIDATION INPUT:
24+
Use the latest boundary scan results:
25+
- No violations: Asteroids, Puckman, SpaceInvaders
26+
- Violations remain across multiple other game folders

docs/dev/COMMIT_COMMENT.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
Phase 08.06Boundary enforcement rules and validation
1+
Phase 08.07correct roadmap scope to core-games complete, repo-wide adoption pending

docs/dev/ROADMAP_RULES.md

Lines changed: 34 additions & 48 deletions
Original file line numberDiff line numberDiff line change
@@ -1,48 +1,34 @@
1-
# Roadmap Rules
2-
3-
## Purpose
4-
Prevent roadmap drift by separating implementation work from roadmap truth updates.
5-
6-
## Ownership
7-
- Implementation PRs build or refactor code.
8-
- Validation PRs verify repo reality against roadmap criteria.
9-
- Roadmap status changes happen only in a dedicated validation or planning PR.
10-
11-
## Status Definitions
12-
- `[x]` Complete: every acceptance criterion is satisfied and evidenced.
13-
- `[.]` In Progress: meaningful partial completion exists, but at least one acceptance criterion is still unmet.
14-
- `[ ]` Not Complete: no sufficient evidence exists to claim progress or completion.
15-
16-
## Evidence Standard
17-
Before changing any roadmap line, collect all of the following:
18-
1. scoped acceptance criteria
19-
2. repo evidence for each criterion
20-
3. a pass/fail checklist
21-
4. missing-items summary
22-
5. recommendation with justification
23-
24-
## Forbidden Shortcuts
25-
Do not move a roadmap line to `[x]` because of:
26-
- folder placeholders
27-
- naming conventions alone
28-
- one-off samples when the line is repo-wide
29-
- inferred intent
30-
- partial migration without runtime boundary proof
31-
32-
## Validator Behavior
33-
Validators must report facts only:
34-
- what files exist
35-
- what boundaries are present
36-
- what criteria pass
37-
- what criteria fail
38-
39-
Validators must not:
40-
- decide roadmap status without explicit acceptance criteria
41-
- broaden scope
42-
- merge multiple roadmap lines into one conclusion
43-
44-
## Review Rule
45-
When there is uncertainty, preserve the stricter status.
46-
47-
## Phase-Based Rule
48-
A line that names multiple games, systems, or structures stays below `[x]` until all named targets are validated.
1+
# ROADMAP RULES
2+
3+
## Status authority
4+
Roadmap status must be evidence-backed and scoped.
5+
6+
## Required distinction
7+
A line may be marked complete for:
8+
- a named game
9+
- a defined subset (for example, core games)
10+
- the full repo
11+
12+
These scopes must not be collapsed into one another.
13+
14+
## Placeholder rule
15+
`.gitkeep` folders and placeholder-only directories do not count as completion.
16+
17+
## Repo-wide completion rule
18+
Repo-wide boundary completion requires all in-scope game folders to pass the active contract.
19+
If a contract is intended only for target/core games, the roadmap must say so explicitly.
20+
21+
## Games-layer scope rule
22+
Section 8 lines must explicitly state scope:
23+
- core/target-game completion
24+
- or repo-wide adoption
25+
Do not mark repo-wide lines complete from core-game evidence alone.
26+
27+
## Validation rule
28+
Validation reports may provide facts and recommendations.
29+
Roadmap status changes require an explicit docs update aligned to the real scope.
30+
31+
## Boundary scan interpretation rule
32+
Boundary scan facts must be recorded line-by-line:
33+
- games with no violations may support game-specific `[x]` lines
34+
- remaining violations in other game folders keep repo-wide lines below `[x]`
Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1,3 @@
1-
Adds enforcement rules for Phase 08 completion.
1+
Adds a docs-only roadmap correction PR after boundary enforcement scan.
2+
The correction preserves earned completion for Asteroids, Puckman, and SpaceInvaders while removing
3+
any implication that all games now satisfy the strict boundary contract.

docs/dev/reports/file_tree.txt

Lines changed: 11 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1,11 @@
1-
docs/pr/BUILD_PR_LEVEL_08_06_BOUNDARY_ENFORCEMENT.md
1+
docs/
2+
dev/
3+
ROADMAP_RULES.md
4+
codex_commands.md
5+
commit_comment.txt
6+
reports/
7+
change_summary.txt
8+
file_tree.txt
9+
validation_checklist.txt
10+
pr/
11+
BUILD_PR_LEVEL_08_07_ROADMAP_SCOPE_CORRECTION.md
Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1,7 @@
1-
Check all games follow flow/game/rules separation.
1+
Roadmap correction checklist
2+
3+
- Confirm Section 8 no longer implies repo-wide completion
4+
- Confirm normalized core games are called out explicitly
5+
- Confirm placeholder-only directories are not counted
6+
- Confirm future template-first adoption remains pending
7+
- Confirm no code files changed
Lines changed: 48 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,48 @@
1+
# BUILD_PR — LEVEL 08.07 ROADMAP SCOPE CORRECTION
2+
3+
## Purpose
4+
Correct Section 8 roadmap status so it reflects the actual repo state after boundary validation.
5+
6+
## Why this PR exists
7+
The repo-wide boundary scan shows that only the normalized core games currently satisfy the strict
8+
`flow/game/rules` boundary contract:
9+
- Asteroids
10+
- Puckman
11+
- SpaceInvaders
12+
13+
Many other game folders still contain placeholder-only `rules/` directories and do not contain the
14+
required `flow/{attract.js,intro.js,highscore.js}` files. Therefore, repo-wide enforcement has not
15+
been earned yet.
16+
17+
## Scope
18+
Docs only.
19+
- Update roadmap markers and wording
20+
- Update roadmap rules wording for explicit scope handling
21+
- Preserve completed work for the normalized core games
22+
- Do not modify game code
23+
- Do not force all sample/prototype games into the same contract in this PR
24+
25+
## Required roadmap correction
26+
Section 8 should distinguish between:
27+
1. **Core game normalization completed**
28+
2. **Repo-wide adoption incomplete**
29+
3. **Future template-first rule still pending**
30+
31+
## Recommended status model
32+
- [x] `games/_template/` created
33+
- [x] core target games normalized (`Asteroids`, `Puckman`, `SpaceInvaders`)
34+
- [.] per-game structure normalized
35+
- [.] current games migrated to target structure
36+
- [ ] game flow pattern standardized repo-wide
37+
- [ ] gameplay/entities/levels/rules/assets boundaries normalized repo-wide
38+
- [ ] shared-vs-game utility boundaries enforced repo-wide
39+
- [ ] future games follow template-first path
40+
41+
## Notes
42+
Do not mark repo-wide items complete based only on the three normalized games.
43+
Do not count placeholder folders as completion.
44+
Do not expand this PR into code changes.
45+
46+
## Validation input used
47+
- Latest boundary scan result: no violations for `Asteroids`, `Puckman`, `SpaceInvaders`.
48+
- Boundary violations still present in multiple other `games/*` folders.

0 commit comments

Comments
 (0)