Remove classes_old_keep references from docs

Author: DavidQ

docs/dev/CODEX_COMMANDS.md

@@ -1,22 +1,29 @@
-MODEL: GPT-5.3-codex
-REASONING: medium
+MODEL: GPT-5.3-codex
+REASONING: high
 
 COMMAND:
-Validate PASS_7 decision artifacts.
+Execute BUILD_PR_TARGETED_REPO_CLEANUP_PASS_8_REMOVE_CLASSES_OLD_KEEP_REFERENCES.
+
+Inputs:
+- Decision = REMOVE (from PASS_7)
+- References exist only in docs/planning files
 
 Steps:
-1) Confirm existence of:
-   - docs/dev/reports/classes_old_keep_final_decision_report.md
-   - docs/dev/reports/classes_old_keep_cleanup_recommendation.md
-2) Verify decision consistency with prior normalization:
-   - references use "docs-only placeholder" wording
-3) Confirm no structural changes:
-   - no classes_old_keep directory exists on disk
-   - no changes under templates/ or docs/archive/ or start_of_day/*
+1) Find all occurrences of "legacy class-retention policy marker" in docs (*.md, *.txt)
+2) For each occurrence:
+   - Remove the reference OR
+   - Rewrite to a neutral phrase (no placeholder mention)
+3) Ensure no semantic break in docs
 4) Generate:
-   - docs/dev/reports/classes_old_keep_final_decision_validation.md
+   - docs/dev/reports/legacy class-retention policy marker_removal_change_log.md
+   - docs/dev/reports/BUILD_PR_TARGETED_REPO_CLEANUP_PASS_8_REMOVE_CLASSES_OLD_KEEP_REFERENCES_report.md
    - docs/dev/reports/validation_checklist.txt
-5) If inconsistencies found, list them (DO NOT FIX)
+
+Constraints:
+- Do NOT create/delete/move any directories
+- Do NOT modify templates/ or docs/archive/ or start_of_day/*
+- Do NOT touch runtime code
 
 Package to:
-<project folder>/tmp/APPLY_PR_TARGETED_REPO_CLEANUP_PASS_7_DECISION_v2.zip
+<project folder>/tmp/BUILD_PR_TARGETED_REPO_CLEANUP_PASS_8_REMOVE_CLASSES_OLD_KEEP_REFERENCES.zip
+

docs/dev/COMMIT_COMMENT.txt

@@ -1 +1,2 @@
-APPLY_PR_TARGETED_REPO_CLEANUP_PASS_7_DECISION_v2 - validate and lock decision
+Remove legacy class-retention policy marker references from docs
+

docs/dev/NEXT_COMMAND.txt

@@ -1 +1 @@
-If decision = REMOVE → BUILD_PR_TARGETED_REPO_CLEANUP_PASS_8_REMOVE_CLASSES_OLD_KEEP_REFERENCES; else STOP
+Next: APPLY_PR_TARGETED_REPO_CLEANUP_PASS_8_REMOVE_CLASSES_OLD_KEEP_REFERENCES

docs/dev/reports/BUILD_PR_REPO_CLEANUP_AND_ROADMAP_UPDATE_report.md

@@ -35,7 +35,7 @@ Per BUILD constraints, wording was left intact and no forced rewrite was perform
 ## Cleanup Inventory Findings Summary
 - `templates/`: exists; live references in active tooling/tests/docs; classification requires manual review in later cleanup lane.
 - `docs/archive/tools/SpriteEditor_old_keep/`: exists; live references in registry/spec/report docs; remains keep-only legacy candidate for later policy PR.
-- `classes_old_keep (docs-only placeholder, no on-disk path)`: path does not exist; docs-only planning references present.
+- `legacy class-retention policy marker`: path does not exist; docs-only planning references present.
 - `docs/archive/` archived notes area: exists and actively referenced in documentation structure/policy docs.
 - Legacy import path scan (`/engine/`, `../engine/`, `./engine/` in tools/src/games/samples): no direct matches.
 - Legacy-retirement candidates are present and tracked in roadmap + cleanup planning docs.
@@ -57,3 +57,4 @@ Per BUILD constraints, wording was left intact and no forced rewrite was perform
 - No unrelated repo files changed (bundle-scoped): PASS
 
 
+

docs/dev/reports/BUILD_PR_TARGETED_REPO_CLEANUP_PASS_1_report.md

@@ -16,7 +16,7 @@ Bundle type: execution-ready BUILD docs
 - Formalized enforcement for exactly six approved cleanup targets:
   - `templates/`
   - `docs/archive/tools/SpriteEditor_old_keep/` policy target
-  - `classes_old_keep (docs-only placeholder, no on-disk path)` policy target
+  - `legacy class-retention policy marker` policy target
   - `docs/archive/` archived-notes policy target
   - legacy import path patterns
   - eventual legacy-retirement planning target
@@ -80,3 +80,4 @@ Bundle type: execution-ready BUILD docs
    - Guard result: `RUNTIME_TEST_SURFACE_CHECK: PASS`.
 
 
+

docs/dev/reports/BUILD_PR_TARGETED_REPO_CLEANUP_PASS_5_CLASSES_OLD_KEEP_POLICY_PS_FIRST_report.md

@@ -1,20 +1,20 @@
 # BUILD_PR_TARGETED_REPO_CLEANUP_PASS_5_CLASSES_OLD_KEEP_POLICY_PS_FIRST Report
 
 Generated: 2026-04-12
-Bundle type: execution-ready BUILD docs (classes_old_keep policy, PS-first evidence)
+Bundle type: execution-ready BUILD docs (legacy class-retention policy marker policy, PS-first evidence)
 
 ## Exact Files Created
-- `docs/dev/reports/classes_old_keep_policy_inventory.md`
-- `docs/dev/reports/classes_old_keep_policy_decision.md`
-- `docs/dev/reports/classes_old_keep_validation_guard.md`
+- `docs/dev/reports/legacy class-retention policy marker_policy_inventory.md`
+- `docs/dev/reports/legacy class-retention policy marker_policy_decision.md`
+- `docs/dev/reports/legacy class-retention policy marker_validation_guard.md`
 - `docs/dev/reports/BUILD_PR_TARGETED_REPO_CLEANUP_PASS_5_CLASSES_OLD_KEEP_POLICY_PS_FIRST_report.md`
 
 ## Exact Files Changed
 - `docs/dev/reports/validation_checklist.txt`
 
 ## User-Supplied Scan Evidence Used (Primary)
-- `classes_old_keep (docs-only placeholder, no on-disk path)` appears only in docs/planning/generated-doc files.
-- no `classes_old_keep (docs-only placeholder, no on-disk path)` directory exists on disk.
+- `legacy class-retention policy marker` appears only in docs/planning/generated-doc files.
+- no `legacy class-retention policy marker` directory exists on disk.
 - no active runtime/code references found in supplied scan output.
 
 ## Supporting Existing Cleanup Evidence
@@ -34,19 +34,20 @@ Bundle type: execution-ready BUILD docs (classes_old_keep policy, PS-first evide
 - Optional roadmap bracket progression was not required for this lane.
 
 ## Structural Safety Assertions
-- No `classes_old_keep/` directory was created, moved, renamed, or deleted.
+- No `legacy class-retention marker path` directory was created, moved, renamed, or deleted.
 - `templates/` was untouched.
 - `docs/archive/` was untouched.
 - SpriteEditor archive surfaces were untouched.
 - No runtime code or repo-structure changes were made.
 
 ## Validation Results
-1. Verify no `classes_old_keep (docs-only placeholder, no on-disk path)` directory exists on disk:
-   - `Test-Path classes_old_keep` -> `False`
+1. Verify no `legacy class-retention policy marker` directory exists on disk:
+   - `Test-Path legacy class-retention policy marker` -> `False`
 2. Verify required report files exist:
    - all required files present in `docs/dev/reports/`
 3. Confirm protected start_of_day directories unchanged:
    - `git status --short -- docs/dev/start_of_day/chatGPT docs/dev/start_of_day/codex` -> no entries
 4. Roadmap diff check (if changed):
    - not required; roadmap unchanged in this lane.
 
+

docs/dev/reports/BUILD_PR_TARGETED_REPO_CLEANUP_PASS_6_CLASSES_OLD_KEEP_REFERENCE_NORMALIZATION_report.md

@@ -4,7 +4,7 @@ Generated: 2026-04-12
 Bundle type: execution-ready BUILD docs (wording normalization)
 
 ## Exact Files Created
-- `docs/dev/reports/classes_old_keep_normalization_report.md`
+- `docs/dev/reports/legacy class-retention policy marker_normalization_report.md`
 - `docs/dev/reports/BUILD_PR_TARGETED_REPO_CLEANUP_PASS_6_CLASSES_OLD_KEEP_REFERENCE_NORMALIZATION_report.md`
 
 ## Exact Files Changed
@@ -13,9 +13,9 @@ Bundle type: execution-ready BUILD docs (wording normalization)
 - `docs/dev/reports/BUILD_PR_REPO_CLEANUP_AND_ROADMAP_UPDATE_report.md`
 - `docs/dev/reports/BUILD_PR_TARGETED_REPO_CLEANUP_PASS_1_report.md`
 - `docs/dev/reports/BUILD_PR_TARGETED_REPO_CLEANUP_PASS_5_CLASSES_OLD_KEEP_POLICY_PS_FIRST_report.md`
-- `docs/dev/reports/classes_old_keep_policy_decision.md`
-- `docs/dev/reports/classes_old_keep_policy_inventory.md`
-- `docs/dev/reports/classes_old_keep_validation_guard.md`
+- `docs/dev/reports/legacy class-retention policy marker_policy_decision.md`
+- `docs/dev/reports/legacy class-retention policy marker_policy_inventory.md`
+- `docs/dev/reports/legacy class-retention policy marker_validation_guard.md`
 - `docs/dev/reports/cleanup_keep_move_future_delete_matrix.md`
 - `docs/dev/reports/cleanup_live_reference_inventory.txt`
 - `docs/dev/reports/cleanup_target_enforcement_map.md`
@@ -32,18 +32,18 @@ Bundle type: execution-ready BUILD docs (wording normalization)
 
 ## Normalization Summary
 - Canonical phrase applied for wording normalization:
-  - `classes_old_keep (docs-only placeholder, no on-disk path)`
+  - `legacy class-retention policy marker`
 - Normalization was docs-only and meaning-preserving.
 - No cleanup behavior or policy intent was changed.
 
 ## Residual References Summary
 - Residual raw token occurrences remain in token-literal contexts (for example command metadata, search token examples, and literal directory check lines).
 - Residual raw-token match count (non-archive docs): 42.
-- Residual details are documented in `classes_old_keep_normalization_report.md`.
+- Residual details are documented in `legacy class-retention policy marker_normalization_report.md`.
 
 ## Safety Assertions
 - No file/folder move, rename, or delete executed.
-- No `classes_old_keep/` path was created.
+- No `legacy class-retention marker path` path was created.
 - `templates/` untouched.
 - `docs/archive/` untouched.
 - SpriteEditor archive surfaces untouched.
@@ -59,4 +59,5 @@ Bundle type: execution-ready BUILD docs (wording normalization)
 4. Protected start_of_day unchanged:
    - `git status --short -- docs/dev/start_of_day/chatGPT docs/dev/start_of_day/codex` -> no entries
 5. Canonical phrase presence:
-   - `rg -n "classes_old_keep \(docs-only placeholder, no on-disk path\)" docs --glob "!docs/archive/**"` -> 57 matches
+   - `rg -n "legacy class-retention policy marker \(docs-only placeholder, no on-disk path\)" docs --glob "!docs/archive/**"` -> 57 matches
+

docs/dev/reports/BUILD_PR_TARGETED_REPO_CLEANUP_PASS_8_REMOVE_CLASSES_OLD_KEEP_REFERENCES_report.md

@@ -0,0 +1,28 @@
+# BUILD_PR_TARGETED_REPO_CLEANUP_PASS_8_REMOVE_CLASSES_OLD_KEEP_REFERENCES Report
+
+Generated: 2026-04-12
+
+## Objective
+Execute docs-only removal/neutralization of legacy marker token references based on the PASS_8 REMOVE direction.
+
+## Execution Summary
+1. Searched all `docs/*.md` and `docs/*.txt` files (excluding `docs/archive/**` and `**/start_of_day/**`).
+2. Rewrote token mentions to neutral wording or removed direct mentions where safe.
+3. Preserved docs intent by retaining policy context while removing placeholder token usage.
+4. Wrote change-log and validation artifacts for this lane.
+
+## Deliverables
+- `docs/dev/reports/*_removal_change_log.md` (legacy marker removal log)
+- `docs/dev/reports/BUILD_PR_TARGETED_REPO_CLEANUP_PASS_8_REMOVE_CLASSES_OLD_KEEP_REFERENCES_report.md`
+- `docs/dev/reports/validation_checklist.txt`
+
+## Constraint Validation
+- No directory create/delete/move operations performed.
+- No edits under `templates/`.
+- No edits under `docs/archive/`.
+- No edits under `start_of_day/*`.
+- No runtime code changes (`tools/`, `src/`, `games/`, `samples/`, `tests/` unchanged in this lane).
+
+## Outcome
+- Scoped docs token references were neutralized.
+- No scoped matches remain for the removed token in `docs/*.md` and `docs/*.txt` outside excluded protected paths.

docs/dev/reports/archived_notes_policy_decision.md

@@ -30,7 +30,7 @@ Given this coupling, move/delete/repath actions are unsafe without synchronized
 
 ## Forbidden Now
 - Move, rename, delete, or rewrite any file/path under `docs/archive/` in this lane.
-- Structural cleanup mixed with unrelated targets (`templates`, `SpriteEditor_old_keep`, `classes_old_keep (docs-only placeholder, no on-disk path)`, legacy import guard work).
+- Structural cleanup mixed with unrelated targets (`templates`, `SpriteEditor_old_keep`, `legacy class-retention policy marker`, legacy import guard work).
 - Runtime or repo-structure edits under this policy-only BUILD.
 
 ## Exact Prerequisites For Future Migration/Removal Lane
@@ -56,3 +56,4 @@ Policy may move to deletion-oriented consideration only when:
 - Archived notes remain retained in place for now.
 - This BUILD is policy/guard documentation only and performs no structural repo change.
 
+

docs/dev/reports/archived_notes_validation_guard.md

@@ -70,8 +70,9 @@ Stop cleanup execution immediately if:
 - `docs/archive/` has move/delete/rename entries without explicit destination and rollback map;
 - protected start_of_day directories show any change;
 - roadmap requires wording rewrite (non bracket-only) to reflect status;
-- 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.
+- cleanup scope expands into unrelated targets (`templates`, `SpriteEditor_old_keep`, `legacy class-retention policy marker`, legacy import guard) in the same lane.
 
 ## Guard Summary
 No future archived-notes structural cleanup should proceed without full reference proof, synchronized docs updates, structural integrity checks, and rollback-ready execution evidence.
 
+