Skip to content

Commit f321420

Browse files
author
DavidQ
committed
Combine remaining section-1 work into a near-closeout structure PR
BUILD_PR_LEVEL_01_COMBINED_STRUCTURE_CLOSEOUT
1 parent 86f4b61 commit f321420

7 files changed

Lines changed: 107 additions & 48 deletions

‎docs/dev/CODEX_COMMANDS.md‎

Lines changed: 25 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -2,39 +2,37 @@ MODEL: GPT-5.4
22
REASONING: high
33

44
COMMAND:
5-
Create `BUILD_PR_LEVEL_01_STRUCTURE_FINALIZATION_AND_ROADMAP_CORRECTION`.
5+
Create `BUILD_PR_LEVEL_01_COMBINED_STRUCTURE_CLOSEOUT` as a combined section-1 closeout PR.
66

7-
Goals:
8-
1. Finalize the remaining planning/definition work for roadmap section 1.
9-
2. Re-anchor execution order to the correct top-down next lane.
10-
3. Keep this PR docs-first and surgical.
7+
Goal:
8+
Finish as much of roadmap section 1 as truthfully possible in one PR to reduce follow-on PR count.
119

1210
Required work:
13-
1. Complete the remaining section-1 definition items where possible:
14-
- map current folder inventory to target homes
15-
- define remaining move-map lanes
16-
- define ambiguous-name rename map where still needed
17-
- define legacy migration map where still needed
18-
- confirm structure targets and boundaries against current repo state
19-
20-
2. Update roadmap status markers only where truthfully supported.
11+
1. Validate the current state of roadmap section 1 against the actual repo.
12+
2. Close all section-1 items that are already supported by repo reality.
13+
3. If a small number of remaining structure issues block closure:
14+
- fix only the smallest surgical items needed
15+
- avoid broad folder churn
16+
- avoid unrelated scope
17+
4. Confirm and document current structure boundaries for:
18+
- src/engine
19+
- src/shared
20+
- games
21+
- games/_template
22+
- samples
23+
- tools
24+
- tools/shared
25+
5. Update roadmap status markers only.
2126
- do NOT rewrite roadmap text
22-
- do NOT jump to already-complete sections as the next lane
23-
- section 1 should become the active anchor if validation supports it
24-
25-
3. Produce clear handoff guidance for later implementation PRs:
26-
- exact move-map lanes
27-
- exact rename-map lanes
28-
- exact validation gates
29-
- explicit no-go areas for now
27+
6. If section 1 still cannot be fully closed:
28+
- leave only the truly incomplete items open
29+
- report exact blockers
3030

31-
4. Final packaging step is REQUIRED:
32-
- package ALL changed files into this exact repo-structured ZIP:
33-
`<project folder>/tmp/BUILD_PR_LEVEL_01_STRUCTURE_FINALIZATION_AND_ROADMAP_CORRECTION.zip`
31+
Final packaging step is REQUIRED:
32+
`<project folder>/tmp/BUILD_PR_LEVEL_01_COMBINED_STRUCTURE_CLOSEOUT.zip`
3433

3534
Hard rules:
36-
- docs-first only
37-
- no implementation code by ChatGPT
38-
- no broad repo churn
35+
- reduce PR count by combining the remaining section-1 work here
36+
- keep fixes surgical
3937
- no unrelated repo changes
4038
- no missing ZIP

‎docs/dev/COMMIT_COMMENT.txt‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,2 +1,2 @@
1-
Finalize section 1 structure planning and re-anchor roadmap to top-down execution
2-
BUILD_PR_LEVEL_01_STRUCTURE_FINALIZATION_AND_ROADMAP_CORRECTION
1+
Combine remaining section-1 work into a near-closeout structure PR
2+
BUILD_PR_LEVEL_01_COMBINED_STRUCTURE_CLOSEOUT

‎docs/dev/NEXT_COMMAND.txt‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
BUILD_PR_LEVEL_01_MOVE_MAP_IMPLEMENTATION_CANDIDATE
1+
BUILD_PR_LEVEL_01_FINAL_RESIDUE_ONLY

‎docs/dev/reports/change_summary.txt‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,3 @@
1-
- Added section-1 finalization PR as the correct top-down next lane
2-
- Re-anchored execution away from already-complete section 11
3-
- Focused on move-map, rename-map, legacy-map, and validation-gate definition only
1+
- Combined the remaining section-1 planning and closeout work into one PR
2+
- Intended to finish most or all of repo structure normalization in a single pass
3+
- Allows only small surgical corrections if they are required for truthful closure

‎docs/dev/reports/validation_checklist.txt‎

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
1-
- Section 1 is treated as the active next lane if roadmap truth supports it
2-
- Remaining section-1 definition items are addressed
1+
- Section 1 validated against actual repo state
2+
- Supported section-1 items closed
3+
- Any remaining open items are explicit and minimal
34
- Roadmap updates are status-only
4-
- No broad file moves are performed in this PR
5-
- Later implementation lanes are clearly defined
5+
- No broad structure churn introduced
66
- Output ZIP created at:
7-
<project folder>/tmp/BUILD_PR_LEVEL_01_STRUCTURE_FINALIZATION_AND_ROADMAP_CORRECTION.zip
7+
<project folder>/tmp/BUILD_PR_LEVEL_01_COMBINED_STRUCTURE_CLOSEOUT.zip

‎docs/dev/roadmaps/MASTER_ROADMAP_HIGH_LEVEL.md‎

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -91,16 +91,16 @@
9191
---
9292

9393
## 1. Repo Structure Normalization
94-
- [.] target structure defined at high level
95-
- [.] `src/engine` target established
96-
- [.] `src/shared` target established
97-
- [.] `games/` target established
98-
- [.] `games/_template/` target established
99-
- [.] `tools/shared` target established
100-
- [.] phase-based `samples/` grouping target established
101-
- [.] dependency direction rules defined
102-
- [.] shared asset promotion rules defined
103-
- [.] network samples classified as sample-phase content
94+
- [x] target structure defined at high level
95+
- [x] `src/engine` target established
96+
- [x] `src/shared` target established
97+
- [x] `games/` target established
98+
- [x] `games/_template/` target established
99+
- [x] `tools/shared` target established
100+
- [x] phase-based `samples/` grouping target established
101+
- [x] dependency direction rules defined
102+
- [x] shared asset promotion rules defined
103+
- [x] network samples classified as sample-phase content
104104
- [x] current folder inventory mapped to target homes
105105
- [x] move-map defined for root `engine/` -> `src/engine/`
106106
- [x] duplicate-helper migration map defined

‎docs/pr/BUILD_PR_LEVEL_01_COMBINED_STRUCTURE_CLOSEOUT.md‎

Lines changed: 61 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,61 @@
1+
# BUILD_PR_LEVEL_01_COMBINED_STRUCTURE_CLOSEOUT
2+
3+
## Purpose
4+
Reduce PR count by combining the remaining section-1 work into one closeout-oriented PR.
5+
6+
## Applied Delta
7+
- Roadmap status-marker closeout pass for section 1 completed where directly supported by repo state.
8+
- Structure boundaries validated and documented.
9+
- Incomplete section-1 items left open with exact blockers.
10+
11+
## Boundary Confirmation (Current Repo)
12+
- `src/engine/` exists and is the active runtime structure root.
13+
- `src/shared/` exists and is the active cross-domain shared structure root.
14+
- `games/` exists and is the game ownership root.
15+
- `games/_template/` exists and remains the canonical game template root.
16+
- `samples/` exists and includes phase-grouped folders (including `phase-01` and `phase-13`).
17+
- `tools/` exists and remains the tool ownership root.
18+
- `tools/shared/` exists and remains the shared tool surface.
19+
20+
## Section-1 Items Closed In This PR
21+
- `target structure defined at high level` -> `[x]`
22+
- `src/engine` target established -> `[x]`
23+
- `src/shared` target established -> `[x]`
24+
- `games/` target established -> `[x]`
25+
- `games/_template/` target established -> `[x]`
26+
- `tools/shared` target established -> `[x]`
27+
- `phase-based samples grouping target established` -> `[x]`
28+
- `dependency direction rules defined` -> `[x]`
29+
- `shared asset promotion rules defined` -> `[x]`
30+
- `network samples classified as sample-phase content` -> `[x]`
31+
32+
## Section-1 Items Still Open (Truthful Residue)
33+
- `src/engine/rendering` remains `[ ]`
34+
- `src/engine/physics` remains `[ ]`
35+
- `src/engine/scene` remains `[ ]`
36+
- `implementation PRs executed` remains `[.]`
37+
38+
## Exact Blockers
39+
- Current engine folders are `src/engine/render/` and `src/engine/scenes/`, not the target names `rendering` and `scene`.
40+
- No `src/engine/physics/` target folder is present yet.
41+
- Closing these requires move/rename PRs with import/path normalization and focused smoke validation, which is implementation scope beyond this docs-first closeout pass.
42+
43+
## Handoff Guidance For Follow-On Surgical PRs
44+
1. `MOVE_MAP_L1_ENGINE_RENDER_AND_SCENE_ALIGNMENT`: Move/rename `src/engine/render` -> `src/engine/rendering` and `src/engine/scenes` -> `src/engine/scene`.
45+
2. `MOVE_MAP_L1_ENGINE_PHYSICS_TARGET_BOOTSTRAP`: Establish `src/engine/physics` target structure in a minimal scoped PR.
46+
3. `VALIDATE_L1_STRUCTURE_ALIGNMENT`: Run path/import normalization checks and focused smoke checks on touched games/tools only.
47+
48+
## Validation Notes
49+
- Roadmap edits are status-marker-only.
50+
- No roadmap prose was rewritten.
51+
- No implementation code changes were made in this PR.
52+
- Section 1 is not fully complete yet; only the truly incomplete items remain open.
53+
54+
## Packaging
55+
`<project folder>/tmp/BUILD_PR_LEVEL_01_COMBINED_STRUCTURE_CLOSEOUT.zip`
56+
57+
## Scope guard
58+
- combine work to reduce PR count
59+
- docs-first planning/build bundle
60+
- implementation deferred to follow-on surgical PRs for remaining blockers
61+
- no unrelated repo changes

0 commit comments

Comments
 (0)