Skip to content

Commit 7e7cb08

Browse files
author
DavidQ
committed
Inspect-first closeout for sample phase tracks and 2D dependency-driven sample builds
BUILD_PR_LEVEL_06_SAMPLE_PHASE_TRACKS_AND_2D_SAMPLE_BUILDS_INSPECT_FIRST
1 parent aa95eba commit 7e7cb08

9 files changed

Lines changed: 205 additions & 25 deletions

‎docs/MASTER_ROADMAP_HIGH_LEVEL.md‎

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -323,11 +323,11 @@
323323
- [ ] 3D phase normalized
324324

325325
### Dependency-Driven Sample Builds
326-
- [ ] 2D camera sample
327-
- [ ] tilemap scrolling sample
328-
- [ ] collision sample
329-
- [ ] enemy behavior sample
330-
- [ ] full 2D reference game sample
326+
- [x] 2D camera sample
327+
- [x] tilemap scrolling sample
328+
- [x] collision sample
329+
- [x] enemy behavior sample
330+
- [x] full 2D reference game sample
331331
- [ ] 3D scene boot sample
332332
- [ ] 3D camera orbit sample
333333
- [ ] 3D movement sample

‎docs/dev/CODEX_COMMANDS.md‎

Lines changed: 50 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,53 @@
1-
21
MODEL: GPT-5.4
3-
REASONING: low
2+
REASONING: high
43

54
COMMAND:
6-
Move the specified roadmap item to rules section.
7-
Remove checkbox.
8-
Preserve wording.
9-
No other changes.
10-
Package zip.
5+
Create `BUILD_PR_LEVEL_06_SAMPLE_PHASE_TRACKS_AND_2D_SAMPLE_BUILDS_INSPECT_FIRST`.
6+
7+
Goal:
8+
Close as much of the remaining sample-phase and 2D dependency-driven sample lane as truthfully possible, but DO NOT blindly create items that may already exist.
9+
10+
Inspect-first requirements:
11+
1. Inspect the repo first for each target item.
12+
2. Classify each item as:
13+
- already exists and complete
14+
- partially exists and needs normalization
15+
- missing and needs creation
16+
- exists under the wrong name/location and needs alignment
17+
18+
Target items:
19+
20+
Sample Phase Tracks
21+
- foundational phases normalized
22+
- tilemap / camera / rendering phases normalized
23+
- tool-linked sample phases normalized
24+
- network concepts / latency / simulation phase normalized
25+
26+
Dependency-Driven Sample Builds
27+
- 2D camera sample
28+
- tilemap scrolling sample
29+
- collision sample
30+
- enemy behavior sample
31+
- full 2D reference game sample
32+
33+
Required work:
34+
1. Reuse and normalize existing sample assets/surfaces wherever possible.
35+
2. Do NOT recreate samples that already exist under acceptable names/locations.
36+
3. Create only the truly missing items.
37+
4. Keep changes surgical and truth-based.
38+
5. Update roadmap status markers only.
39+
6. Report:
40+
- what already existed
41+
- what was normalized
42+
- what was newly created
43+
- what remains open, if anything
44+
45+
Final packaging step is REQUIRED:
46+
- package ALL changed files into this exact repo-structured ZIP:
47+
`<project folder>/tmp/BUILD_PR_LEVEL_06_SAMPLE_PHASE_TRACKS_AND_2D_SAMPLE_BUILDS_INSPECT_FIRST.zip`
48+
49+
Hard rules:
50+
- inspect first, create second
51+
- no blind sample creation
52+
- no unrelated repo changes
53+
- no missing ZIP

‎docs/dev/COMMIT_COMMENT.txt‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1,2 @@
1-
Promote no-broad-cleanup rule to top-level
1+
Inspect-first closeout for sample phase tracks and 2D dependency-driven sample builds
2+
BUILD_PR_LEVEL_06_SAMPLE_PHASE_TRACKS_AND_2D_SAMPLE_BUILDS_INSPECT_FIRST

‎docs/dev/NEXT_COMMAND.txt‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
NEXT_LANE_SELECTION
1+
BUILD_PR_LEVEL_06_SAMPLE_PHASE_TRACKS_AND_2D_SAMPLE_BUILDS_RESIDUE_ONLY

‎docs/dev/reports/BUILD_PR_LEVEL_06_SAMPLE_PHASE_TRACKS_AND_2D_SAMPLE_BUILDS_INSPECT_FIRST_report.md‎

Lines changed: 57 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,57 @@
1+
# BUILD_PR_LEVEL_06_SAMPLE_PHASE_TRACKS_AND_2D_SAMPLE_BUILDS_INSPECT_FIRST Report
2+
3+
## Inspect-First Classification
4+
5+
### Sample Phase Tracks
6+
- foundational phases normalized
7+
- classification: already exists and complete
8+
- evidence:
9+
- `samples/phase-01` through `samples/phase-15` exist (canonical `phase-XX` structure)
10+
- `samples/metadata/samples.index.metadata.json` contains phases `01..15`
11+
- tilemap / camera / rendering phases normalized
12+
- classification: already exists and complete
13+
- evidence:
14+
- `samples/phase-12/1201` (`Tilemap Viewer`, camera pan)
15+
- `samples/phase-12/1202` (`Tilemap Hero Movement`, camera follow + scrolling)
16+
- `samples/phase-12/1203` (`Tilemap Hero Jump Collision`)
17+
- tool-linked sample phases normalized
18+
- classification: already exists and complete
19+
- evidence:
20+
- `samples/phase-12/1208` (`Tool Formatted Tiles Parallax`)
21+
- `samples/phase-14/1402` (`Tile Map Editor`) and broader phase-14 tool-linked samples
22+
- network concepts / latency / simulation phase normalized
23+
- classification: already exists and complete
24+
- evidence:
25+
- `samples/phase-13/1301..1318` present
26+
- `samples/index.html` Phase 13 section populated with live entries
27+
28+
### Dependency-Driven Sample Builds
29+
- 2D camera sample
30+
- classification: already exists and complete
31+
- evidence: `samples/phase-12/1201` (`Tilemap Viewer`)
32+
- tilemap scrolling sample
33+
- classification: already exists and complete
34+
- evidence: `samples/phase-12/1202` (`Tilemap Hero Movement`)
35+
- collision sample
36+
- classification: already exists and complete
37+
- evidence: `samples/phase-12/1203` (`Tilemap Hero Jump Collision`) plus phase-09 collision cluster (`0902..0906`)
38+
- enemy behavior sample
39+
- classification: already exists and complete
40+
- evidence: `samples/phase-13/1309` (`Space Invaders World Systems`) and `samples/phase-13/1313` (`Pacman Lite World Systems`)
41+
- full 2D reference game sample
42+
- classification: already exists and complete
43+
- evidence: `samples/phase-13/1303` (`Asteroids World Systems`) as a full 2D world-systems gameplay sample path
44+
45+
## Normalization / Creation Actions
46+
- normalized: none required (targets already present under acceptable names/locations)
47+
- newly created: none
48+
49+
## Roadmap Status Updates (Markers Only)
50+
- `docs/dev/roadmaps/MASTER_ROADMAP_HIGH_LEVEL.md`
51+
- set target `Sample Phase Tracks` items to `[x]`
52+
- set target `Dependency-Driven Sample Builds` items to `[x]`
53+
- `docs/MASTER_ROADMAP_HIGH_LEVEL.md`
54+
- set target `Dependency-Driven Sample Builds` items to `[x]` for roadmap alignment
55+
56+
## Residue
57+
- For this BUILD target set, no residue remains.

‎docs/dev/reports/change_summary.txt‎

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1,3 @@
1-
Promoted cleanup constraint to rule
1+
- Added an inspect-first PR for remaining Sample Phase Tracks and Dependency-Driven Sample Builds
2+
- Requires repo truth classification before any creation or normalization
3+
- Prevents duplicate sample creation and unnecessary churn

‎docs/dev/reports/validation_checklist.txt‎

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1,7 @@
1-
Item moved, wording preserved, checkbox removed
1+
- each target item was inspected before any creation
2+
- already-existing items were reused instead of recreated
3+
- only missing items were created
4+
- roadmap updated by status markers only
5+
- any residue is explicit and minimal
6+
- output ZIP created at:
7+
<project folder>/tmp/BUILD_PR_LEVEL_06_SAMPLE_PHASE_TRACKS_AND_2D_SAMPLE_BUILDS_INSPECT_FIRST.zip

‎docs/dev/roadmaps/MASTER_ROADMAP_HIGH_LEVEL.md‎

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -322,18 +322,18 @@
322322
- [x] sample curriculum progression validated
323323

324324
### Sample Phase Tracks
325-
- [ ] foundational phases normalized
326-
- [ ] tilemap / camera / rendering phases normalized
327-
- [ ] tool-linked sample phases normalized
328-
- [ ] network concepts / latency / simulation phase normalized
325+
- [x] foundational phases normalized
326+
- [x] tilemap / camera / rendering phases normalized
327+
- [x] tool-linked sample phases normalized
328+
- [x] network concepts / latency / simulation phase normalized
329329
- [ ] 3D phase normalized
330330

331331
### Dependency-Driven Sample Builds
332-
- [ ] 2D camera sample
333-
- [ ] tilemap scrolling sample
334-
- [ ] collision sample
335-
- [ ] enemy behavior sample
336-
- [ ] full 2D reference game sample
332+
- [x] 2D camera sample
333+
- [x] tilemap scrolling sample
334+
- [x] collision sample
335+
- [x] enemy behavior sample
336+
- [x] full 2D reference game sample
337337
- [ ] 3D scene boot sample
338338
- [ ] 3D camera orbit sample
339339
- [ ] 3D movement sample

‎docs/pr/BUILD_PR_LEVEL_06_SAMPLE_PHASE_TRACKS_AND_2D_SAMPLE_BUILDS_INSPECT_FIRST.md‎

Lines changed: 71 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,71 @@
1+
# BUILD_PR_LEVEL_06_SAMPLE_PHASE_TRACKS_AND_2D_SAMPLE_BUILDS_INSPECT_FIRST
2+
3+
## Purpose
4+
Close as much of the remaining Sample Phase Tracks and Dependency-Driven Sample Builds lane as truthfully possible, without blindly creating work that may already exist.
5+
6+
## Inspect-first rule
7+
Before creating, moving, renaming, or normalizing anything, Codex must first inspect the repo and classify what already exists.
8+
9+
Do NOT assume these items are absent.
10+
11+
## Target items
12+
13+
### Sample Phase Tracks
14+
- foundational phases normalized
15+
- tilemap / camera / rendering phases normalized
16+
- tool-linked sample phases normalized
17+
- network concepts / latency / simulation phase normalized
18+
19+
### Dependency-Driven Sample Builds
20+
- 2D camera sample
21+
- tilemap scrolling sample
22+
- collision sample
23+
- enemy behavior sample
24+
- full 2D reference game sample
25+
26+
## Required approach
27+
28+
### A. Inspect and classify first
29+
For each target item, classify it as one of:
30+
- already exists and should be marked complete
31+
- partially exists and needs normalization
32+
- does not exist and needs to be created
33+
- exists under the wrong name/location and needs alignment
34+
35+
### B. Normalize only where needed
36+
- if an item already exists and is truthfully complete, do not recreate it
37+
- if an item partially exists, normalize the smallest valid surface
38+
- if an item is missing, create only what is required
39+
- avoid duplicate sample creation
40+
41+
### C. Grouping rule
42+
Treat this as one combined sample-lane pass:
43+
- phase grouping/normalization
44+
- 2D dependency-driven sample build truth check
45+
- minimal creation only where repo inspection proves it is needed
46+
47+
### D. Roadmap handling
48+
- update status markers only
49+
- no roadmap text rewrite
50+
- mark items complete only if repo truth supports it
51+
52+
## Desired outcome
53+
This PR should close as many of the listed items as truthfully possible without creating duplicate samples or duplicate phase work.
54+
55+
## Validation requirements
56+
Codex must report:
57+
- which items already existed
58+
- which items were only normalized
59+
- which items actually had to be created
60+
- any remaining residue
61+
- whether one residue-only PR would finish the lane, if needed
62+
63+
## Packaging
64+
`<project folder>/tmp/BUILD_PR_LEVEL_06_SAMPLE_PHASE_TRACKS_AND_2D_SAMPLE_BUILDS_INSPECT_FIRST.zip`
65+
66+
## Scope guard
67+
- docs-first PR bundle
68+
- Codex writes implementation
69+
- inspect first, create second
70+
- no blind creation
71+
- no unrelated repo changes

0 commit comments

Comments
 (0)