Skip to content

Commit 24ddad7

Browse files
author
DavidQ
committed
Normalize classes_old_keep references to docs-only placeholder
1 parent 24cab0e commit 24ddad7

27 files changed

+298
-147
lines changed

docs/dev/CODEX_COMMANDS.md

Lines changed: 10 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -2,26 +2,15 @@ MODEL: GPT-5.3-codex
22
REASONING: medium
33

44
COMMAND:
5-
Execute BUILD_PR_TARGETED_REPO_CLEANUP_PASS_5_CLASSES_OLD_KEEP_POLICY_PS_FIRST.
5+
Execute BUILD_PR_TARGETED_REPO_CLEANUP_PASS_6_CLASSES_OLD_KEEP_REFERENCE_NORMALIZATION.
66

7-
Use the user-supplied PowerShell scan evidence as the primary source of truth:
8-
- `classes_old_keep` appears only in docs/planning/generated-doc files
9-
- no `classes_old_keep` directory exists on disk
10-
- no active runtime/code references were found in the supplied scan output
7+
- Normalize wording of classes_old_keep references across docs
8+
- Do not change meaning, only clarify as docs-only placeholder
9+
- Do not create/delete/move any files or folders
10+
- Generate:
11+
docs/dev/reports/classes_old_keep_normalization_report.md
12+
docs/dev/reports/BUILD_PR_TARGETED_REPO_CLEANUP_PASS_6_CLASSES_OLD_KEEP_REFERENCE_NORMALIZATION_report.md
13+
docs/dev/reports/validation_checklist.txt
1114

12-
Create exactly:
13-
- docs/dev/reports/classes_old_keep_policy_inventory.md
14-
- docs/dev/reports/classes_old_keep_policy_decision.md
15-
- docs/dev/reports/classes_old_keep_validation_guard.md
16-
- docs/dev/reports/BUILD_PR_TARGETED_REPO_CLEANUP_PASS_5_CLASSES_OLD_KEEP_POLICY_PS_FIRST_report.md
17-
- docs/dev/reports/validation_checklist.txt
18-
19-
Constraints:
20-
- Do NOT rerun broad repo discovery scans already completed by the user unless needed only to validate a specific contradiction.
21-
- Do NOT create, move, rename, or delete any `classes_old_keep/` directory.
22-
- Do NOT modify templates/, docs/archive/, or SpriteEditor archive surfaces.
23-
- Do NOT modify runtime code or repo structure.
24-
- If roadmap updates are needed, apply bracket-only changes only where exact wording already exists.
25-
26-
Package output to:
27-
<project folder>/tmp/BUILD_PR_TARGETED_REPO_CLEANUP_PASS_5_CLASSES_OLD_KEEP_POLICY_PS_FIRST.zip
15+
Package to:
16+
<project folder>/tmp/BUILD_PR_TARGETED_REPO_CLEANUP_PASS_6_CLASSES_OLD_KEEP_REFERENCE_NORMALIZATION.zip

docs/dev/COMMIT_COMMENT.txt

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1 @@
1-
BUILD_PR_TARGETED_REPO_CLEANUP_PASS_5_CLASSES_OLD_KEEP_POLICY_PS_FIRST
2-
3-
Convert user-supplied PowerShell evidence into docs-only policy, inventory, and validation guard for classes_old_keep.
1+
Normalize classes_old_keep references to docs-only placeholder

docs/dev/NEXT_COMMAND.txt

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,2 +1 @@
1-
After Codex completes and validation passes, proceed to:
2-
BUILD_PR_TARGETED_REPO_CLEANUP_PASS_6_CLASSES_OLD_KEEP_REFERENCE_NORMALIZATION
1+
Next: BUILD_PR_TARGETED_REPO_CLEANUP_PASS_7_FINAL_REVIEW

docs/dev/reports/BUILD_PR_REPO_CLEANUP_AND_ROADMAP_UPDATE_report.md

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ Per BUILD constraints, wording was left intact and no forced rewrite was perform
3535
## Cleanup Inventory Findings Summary
3636
- `templates/`: exists; live references in active tooling/tests/docs; classification requires manual review in later cleanup lane.
3737
- `docs/archive/tools/SpriteEditor_old_keep/`: exists; live references in registry/spec/report docs; remains keep-only legacy candidate for later policy PR.
38-
- `classes_old_keep`: path does not exist; docs-only planning references present.
38+
- `classes_old_keep (docs-only placeholder, no on-disk path)`: path does not exist; docs-only planning references present.
3939
- `docs/archive/` archived notes area: exists and actively referenced in documentation structure/policy docs.
4040
- Legacy import path scan (`/engine/`, `../engine/`, `./engine/` in tools/src/games/samples): no direct matches.
4141
- Legacy-retirement candidates are present and tracked in roadmap + cleanup planning docs.
@@ -56,3 +56,4 @@ Per BUILD constraints, wording was left intact and no forced rewrite was perform
5656
- Evidence reports created: PASS
5757
- No unrelated repo files changed (bundle-scoped): PASS
5858

59+

docs/dev/reports/BUILD_PR_TARGETED_REPO_CLEANUP_PASS_1_report.md

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ Bundle type: execution-ready BUILD docs
1616
- Formalized enforcement for exactly six approved cleanup targets:
1717
- `templates/`
1818
- `docs/archive/tools/SpriteEditor_old_keep/` policy target
19-
- `classes_old_keep` policy target
19+
- `classes_old_keep (docs-only placeholder, no on-disk path)` policy target
2020
- `docs/archive/` archived-notes policy target
2121
- legacy import path patterns
2222
- eventual legacy-retirement planning target
@@ -79,3 +79,4 @@ Bundle type: execution-ready BUILD docs
7979
- Result: no entries.
8080
- Guard result: `RUNTIME_TEST_SURFACE_CHECK: PASS`.
8181

82+

docs/dev/reports/BUILD_PR_TARGETED_REPO_CLEANUP_PASS_5_CLASSES_OLD_KEEP_POLICY_PS_FIRST_report.md

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -13,8 +13,8 @@ Bundle type: execution-ready BUILD docs (classes_old_keep policy, PS-first evide
1313
- `docs/dev/reports/validation_checklist.txt`
1414

1515
## User-Supplied Scan Evidence Used (Primary)
16-
- `classes_old_keep` appears only in docs/planning/generated-doc files.
17-
- no `classes_old_keep` directory exists on disk.
16+
- `classes_old_keep (docs-only placeholder, no on-disk path)` appears only in docs/planning/generated-doc files.
17+
- no `classes_old_keep (docs-only placeholder, no on-disk path)` directory exists on disk.
1818
- no active runtime/code references found in supplied scan output.
1919

2020
## Supporting Existing Cleanup Evidence
@@ -41,11 +41,12 @@ Bundle type: execution-ready BUILD docs (classes_old_keep policy, PS-first evide
4141
- No runtime code or repo-structure changes were made.
4242

4343
## Validation Results
44-
1. Verify no `classes_old_keep` directory exists on disk:
44+
1. Verify no `classes_old_keep (docs-only placeholder, no on-disk path)` directory exists on disk:
4545
- `Test-Path classes_old_keep` -> `False`
4646
2. Verify required report files exist:
4747
- all required files present in `docs/dev/reports/`
4848
3. Confirm protected start_of_day directories unchanged:
4949
- `git status --short -- docs/dev/start_of_day/chatGPT docs/dev/start_of_day/codex` -> no entries
5050
4. Roadmap diff check (if changed):
5151
- not required; roadmap unchanged in this lane.
52+
Lines changed: 62 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,62 @@
1+
# BUILD_PR_TARGETED_REPO_CLEANUP_PASS_6_CLASSES_OLD_KEEP_REFERENCE_NORMALIZATION Report
2+
3+
Generated: 2026-04-12
4+
Bundle type: execution-ready BUILD docs (wording normalization)
5+
6+
## Exact Files Created
7+
- `docs/dev/reports/classes_old_keep_normalization_report.md`
8+
- `docs/dev/reports/BUILD_PR_TARGETED_REPO_CLEANUP_PASS_6_CLASSES_OLD_KEEP_REFERENCE_NORMALIZATION_report.md`
9+
10+
## Exact Files Changed
11+
- `docs/dev/reports/archived_notes_policy_decision.md`
12+
- `docs/dev/reports/archived_notes_validation_guard.md`
13+
- `docs/dev/reports/BUILD_PR_REPO_CLEANUP_AND_ROADMAP_UPDATE_report.md`
14+
- `docs/dev/reports/BUILD_PR_TARGETED_REPO_CLEANUP_PASS_1_report.md`
15+
- `docs/dev/reports/BUILD_PR_TARGETED_REPO_CLEANUP_PASS_5_CLASSES_OLD_KEEP_POLICY_PS_FIRST_report.md`
16+
- `docs/dev/reports/classes_old_keep_policy_decision.md`
17+
- `docs/dev/reports/classes_old_keep_policy_inventory.md`
18+
- `docs/dev/reports/classes_old_keep_validation_guard.md`
19+
- `docs/dev/reports/cleanup_keep_move_future_delete_matrix.md`
20+
- `docs/dev/reports/cleanup_live_reference_inventory.txt`
21+
- `docs/dev/reports/cleanup_target_enforcement_map.md`
22+
- `docs/dev/reports/cleanup_target_normalization_report.md`
23+
- `docs/dev/reports/repo_cleanup_targets.txt`
24+
- `docs/dev/roadmaps/MASTER_ROADMAP_HIGH_LEVEL.md`
25+
- `docs/pr/BUILD_PR_REPO_CLEANUP_AND_ROADMAP_UPDATE.md`
26+
- `docs/pr/BUILD_PR_TARGETED_REPO_CLEANUP_PASS_1.md`
27+
- `docs/pr/BUILD_PR_TARGETED_REPO_CLEANUP_PASS_3_ARCHIVED_NOTES_POLICY.md`
28+
- `docs/pr/BUILD_PR_TARGETED_REPO_CLEANUP_PASS_5_CLASSES_OLD_KEEP_POLICY_PS_FIRST.md`
29+
- `docs/pr/BUILD_PR_TARGETED_REPO_CLEANUP_PASS_6_CLASSES_OLD_KEEP_REFERENCE_NORMALIZATION.md`
30+
- `docs/pr/BUILD_PR_TEMPLATES_POLICY_CLASSIFICATION.md`
31+
- `docs/dev/reports/validation_checklist.txt`
32+
33+
## Normalization Summary
34+
- Canonical phrase applied for wording normalization:
35+
- `classes_old_keep (docs-only placeholder, no on-disk path)`
36+
- Normalization was docs-only and meaning-preserving.
37+
- No cleanup behavior or policy intent was changed.
38+
39+
## Residual References Summary
40+
- Residual raw token occurrences remain in token-literal contexts (for example command metadata, search token examples, and literal directory check lines).
41+
- Residual raw-token match count (non-archive docs): 42.
42+
- Residual details are documented in `classes_old_keep_normalization_report.md`.
43+
44+
## Safety Assertions
45+
- No file/folder move, rename, or delete executed.
46+
- No `classes_old_keep/` path was created.
47+
- `templates/` untouched.
48+
- `docs/archive/` untouched.
49+
- SpriteEditor archive surfaces untouched.
50+
- No runtime code/repo-structure changes.
51+
52+
## Validation Results
53+
1. Move/Delete/Rename guard:
54+
- `git diff --name-status` + filter -> `MOVE_DELETE_RENAME_CHECK: PASS`
55+
2. Runtime/code surfaces unchanged:
56+
- `git diff --name-only -- tools src games samples tests` -> no entries
57+
3. Protected surfaces unchanged:
58+
- `git diff --name-status -- templates docs/archive docs/archive/tools/SpriteEditor_old_keep` -> no entries
59+
4. Protected start_of_day unchanged:
60+
- `git status --short -- docs/dev/start_of_day/chatGPT docs/dev/start_of_day/codex` -> no entries
61+
5. Canonical phrase presence:
62+
- `rg -n "classes_old_keep \(docs-only placeholder, no on-disk path\)" docs --glob "!docs/archive/**"` -> 57 matches

docs/dev/reports/archived_notes_policy_decision.md

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ Given this coupling, move/delete/repath actions are unsafe without synchronized
3030

3131
## Forbidden Now
3232
- Move, rename, delete, or rewrite any file/path under `docs/archive/` in this lane.
33-
- Structural cleanup mixed with unrelated targets (`templates`, `SpriteEditor_old_keep`, `classes_old_keep`, legacy import guard work).
33+
- Structural cleanup mixed with unrelated targets (`templates`, `SpriteEditor_old_keep`, `classes_old_keep (docs-only placeholder, no on-disk path)`, legacy import guard work).
3434
- Runtime or repo-structure edits under this policy-only BUILD.
3535

3636
## Exact Prerequisites For Future Migration/Removal Lane
@@ -55,3 +55,4 @@ Policy may move to deletion-oriented consideration only when:
5555
## Decision Summary
5656
- Archived notes remain retained in place for now.
5757
- This BUILD is policy/guard documentation only and performs no structural repo change.
58+

docs/dev/reports/archived_notes_validation_guard.md

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -70,7 +70,8 @@ Stop cleanup execution immediately if:
7070
- `docs/archive/` has move/delete/rename entries without explicit destination and rollback map;
7171
- protected start_of_day directories show any change;
7272
- roadmap requires wording rewrite (non bracket-only) to reflect status;
73-
- cleanup scope expands into unrelated targets (`templates`, `SpriteEditor_old_keep`, `classes_old_keep`, legacy import guard) in the same lane.
73+
- cleanup scope expands into unrelated targets (`templates`, `SpriteEditor_old_keep`, `classes_old_keep (docs-only placeholder, no on-disk path)`, legacy import guard) in the same lane.
7474

7575
## Guard Summary
7676
No future archived-notes structural cleanup should proceed without full reference proof, synchronized docs updates, structural integrity checks, and rollback-ready execution evidence.
77+
Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1 @@
1-
- Uses user-supplied Windows PowerShell scan results instead of spending Codex tokens on broad repo search.
2-
- Narrows the lane to docs-only policy work for classes_old_keep.
3-
- Confirms no on-disk classes_old_keep directory exists in the supplied evidence.
4-
- Keeps templates/, docs/archive/, and SpriteEditor archive surfaces untouched.
1+
classes_old_keep reference normalization

0 commit comments

Comments
 (0)