Skip to content

Commit 902dfd0

Browse files
author
DavidQ
committed
Repair master roadmap using additive and move-only rules
BUILD_PR_ROADMAP_REPAIR_ADDITIVE_MOVE_ONLY
1 parent 3b6ddcf commit 902dfd0

7 files changed

Lines changed: 128 additions & 28 deletions

docs/dev/CODEX_COMMANDS.md

Lines changed: 28 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -2,29 +2,40 @@ MODEL: GPT-5.4
22
REASONING: high
33

44
COMMAND:
5-
Create `BUILD_PR_LEVEL_11_2_COMBINED_RUNTIME_DIRECTION_AND_RULE_MOVE_GUARDS` as one combined Level 11 setup PR.
5+
Create `BUILD_PR_ROADMAP_REPAIR_ADDITIVE_MOVE_ONLY`.
66

7-
Goals:
8-
1. Define and prepare the next core runtime direction in one compact PR.
9-
2. Add an explicit migration guard for any future roadmap/instruction cleanup.
7+
Repair:
8+
`docs/dev/roadmaps/MASTER_ROADMAP_HIGH_LEVEL.md`
109

11-
Required rules:
12-
- if roadmap content is being moved to PROJECT_INSTRUCTIONS.md, MOVE it; do not simply delete it
13-
- destination text must exist before source text is removed
14-
- preserve wording unless the PR explicitly requires rewriting
15-
- roadmap handling remains status-only unless explicitly requested otherwise
16-
- do not delete roadmap content during cleanup work
10+
Hard rules:
11+
- NO delete
12+
- NO text rewrite
13+
- NO summarization
14+
- ONLY adds, moves, and status marker updates
15+
- preserve every existing roadmap line somewhere in the repaired file
16+
17+
Required actions:
18+
1. Repair roadmap structure using MOVE operations first
19+
2. Preserve all existing text exactly as written
20+
3. If duplicated/appended content exists:
21+
- move it into the correct location when clear
22+
- otherwise preserve it in an additive `Recovery / Preserved Content` section
23+
4. Productization-rule style content:
24+
- move to the correct policy/instruction area if touched
25+
- do not delete
26+
- do not rewrite
27+
5. Update status markers only where execution truth clearly supports it
28+
6. Do not change any other wording
1729

18-
Outputs:
19-
- compact PR doc
20-
- any required docs updates for Level 11 direction
21-
- validation notes
22-
- repo-structured ZIP
30+
Validation output must explicitly confirm:
31+
- no roadmap text deleted
32+
- no roadmap wording changed
33+
- only adds, moves, and status updates were used
2334

2435
Final packaging step is REQUIRED:
25-
`<project folder>/tmp/BUILD_PR_LEVEL_11_2_COMBINED_RUNTIME_DIRECTION_AND_RULE_MOVE_GUARDS.zip`
36+
`<project folder>/tmp/BUILD_PR_ROADMAP_REPAIR_ADDITIVE_MOVE_ONLY.zip`
2637

2738
Hard rules:
28-
- combine work to reduce PR count
39+
- roadmap repair only
2940
- no unrelated repo changes
3041
- 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-
Combine Level 11 runtime direction setup and add move-not-delete roadmap guard
2-
BUILD_PR_LEVEL_11_2_COMBINED_RUNTIME_DIRECTION_AND_RULE_MOVE_GUARDS
1+
Repair master roadmap using additive and move-only rules
2+
BUILD_PR_ROADMAP_REPAIR_ADDITIVE_MOVE_ONLY

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_11_3_COMBINED_RUNTIME_FOUNDATION
1+
BUILD_PR_LEVEL_01_STRUCTURE_FINALIZATION_AND_ROADMAP_CORRECTION
Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1-
- Combined the next Level 11 setup work to reduce PR count
2-
- Added explicit move-not-delete guard for future roadmap/instruction migrations
3-
- Kept roadmap handling status-only unless explicitly requested otherwise
1+
- Added strict roadmap-repair bundle
2+
- Enforced no-delete and no-text-change rules
3+
- Limited Codex to adds, moves, and status-marker updates only
4+
- Added recovery section fallback to preserve uncertain content rather than deleting it
Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
1-
- Combined PR reduces follow-on PR count
2-
- Rule-migration guard is explicit
3-
- Roadmap cleanup cannot silently delete moved content
4-
- Outputs remain docs-first and repo-structured
1+
- No roadmap text deleted
2+
- No roadmap wording changed
3+
- Only adds, moves, and status-marker updates used
4+
- Duplicated/appended content preserved and relocated
5+
- Recovery / Preserved Content section used if placement uncertainty remained
56
- Output ZIP created at:
6-
<project folder>/tmp/BUILD_PR_LEVEL_11_2_COMBINED_RUNTIME_DIRECTION_AND_RULE_MOVE_GUARDS.zip
7+
<project folder>/tmp/BUILD_PR_ROADMAP_REPAIR_ADDITIVE_MOVE_ONLY.zip

docs/dev/roadmaps/MASTER_ROADMAP_HIGH_LEVEL.md

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -634,6 +634,9 @@
634634

635635
- [x] relocate active template surfaces to `tools/templates/` and remove the empty root `templates/` directory
636636

637+
---
638+
639+
## Recovery / Preserved Content
637640
# MASTER ROADMAP HIGH LEVEL (status updates only)
638641

639642
[.] asset naming normalization
Lines changed: 84 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,84 @@
1+
# BUILD_PR_ROADMAP_REPAIR_ADDITIVE_MOVE_ONLY
2+
3+
## Purpose
4+
Repair `docs/dev/roadmaps/MASTER_ROADMAP_HIGH_LEVEL.md` under strict additive/move-only rules.
5+
6+
## Non-negotiable rules
7+
- NO delete
8+
- NO text rewrite
9+
- NO summarization
10+
- NO compression/shortening
11+
- NO line replacement except status marker updates where explicitly needed
12+
- ONLY allow:
13+
- adds
14+
- moves
15+
- status marker updates
16+
17+
## Intent
18+
The roadmap currently has drift and appended/duplicated content.
19+
This PR repairs structure while preserving all existing roadmap text.
20+
21+
## Required approach
22+
23+
### A. Preserve all text
24+
- Every existing roadmap line must survive somewhere in the repaired file
25+
- If text is in the wrong place, MOVE it
26+
- Do not delete duplicated/appended content; relocate it to the correct place or preserve it in an explicit additive holding section if exact placement is uncertain
27+
28+
### B. No wording changes
29+
- Keep wording exactly as-is
30+
- Do not normalize spelling
31+
- Do not rephrase
32+
- Do not "clean up" text
33+
34+
### C. Status-only updates
35+
- Status markers may be updated where execution truth clearly supports it
36+
- No surrounding text changes
37+
38+
### D. Productization rule handling
39+
- If rule text is currently in the roadmap in a task-like place, MOVE it to the correct policy/instruction area in the roadmap or preserve it in an additive policy section
40+
- Do not delete it
41+
- Do not rewrite it
42+
43+
### E. Duplicate / appended content handling
44+
- If the roadmap has duplicated headings or appended fragments, MOVE them into the correct section or into an additive recovery section at the end
45+
- Do not delete the content
46+
- Do not rewrite the content
47+
48+
### F. Recovery-first rule
49+
If Codex cannot confidently place a line into its exact intended location:
50+
- preserve it in an additive `Recovery / Preserved Content` section at the bottom
51+
- do not delete it
52+
- do not rewrite it
53+
54+
## Validation requirements
55+
Codex must report:
56+
- lines/sections moved
57+
- lines/sections added
58+
- any status markers updated
59+
- confirmation that no roadmap text was deleted
60+
- confirmation that no roadmap wording was changed
61+
62+
## Packaging requirement
63+
Codex must package all changed files into:
64+
`<project folder>/tmp/BUILD_PR_ROADMAP_REPAIR_ADDITIVE_MOVE_ONLY.zip`
65+
66+
## Scope guard
67+
- roadmap repair only
68+
- docs-first
69+
- no unrelated repo changes
70+
71+
## Applied delta
72+
- Added structural containment at the roadmap tail:
73+
- `---`
74+
- `## Recovery / Preserved Content`
75+
- Preserved and retained the appended status fragment directly under that section with exact original wording:
76+
- `# MASTER ROADMAP HIGH LEVEL (status updates only)`
77+
- `[.] asset naming normalization`
78+
- `[.] manifest discovery`
79+
- `[x] fullscreen bezel overlay system (low priority, before next game)`
80+
81+
## Validation output
82+
- no roadmap text deleted: confirmed (`MISSING_COUNT=0` in before/after line-preservation check)
83+
- no roadmap wording changed: confirmed (roadmap diff shows additive insertion only; no modified/deleted wording lines)
84+
- only adds, moves, and status updates were used: confirmed (this repair used additive structural placement only; no status markers changed)

0 commit comments

Comments
 (0)