Skip to content

Commit 24cab0e

Browse files
author
DavidQ
committed
BUILD_PR_TARGETED_REPO_CLEANUP_PASS_5_CLASSES_OLD_KEEP_POLICY_PS_FIRST
Convert user-supplied PowerShell evidence into docs-only policy, inventory, and validation guard for classes_old_keep.
1 parent ab5cb52 commit 24cab0e

11 files changed

+462
-22
lines changed

‎docs/dev/CODEX_COMMANDS.md‎

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

44
COMMAND:
5-
Validate the completed move of:
6-
tools/SpriteEditor_old_keep/
7-
→ docs/archive/tools/SpriteEditor_old_keep/
5+
Execute BUILD_PR_TARGETED_REPO_CLEANUP_PASS_5_CLASSES_OLD_KEEP_POLICY_PS_FIRST.
86

9-
Steps:
10-
1. Confirm old path does not exist
11-
2. Confirm new archive path exists with identical structure
12-
3. Search repo for any remaining references to old path
13-
4. Confirm no runtime imports reference the old path
14-
5. Generate validation report:
15-
docs/dev/reports/spriteeditor_archive_move_validation.md
16-
6. If any broken references are found, list them (DO NOT FIX)
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
1711

18-
Package results to:
19-
<project folder>/tmp/APPLY_PR_SPRITEEDITOR_ARCHIVE_VALIDATE_v2.zip
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

‎docs/dev/COMMIT_COMMENT.txt‎

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,2 +1,3 @@
1-
APPLY_PR_SPRITEEDITOR_ARCHIVE_VALIDATE_v2
2-
Validate archive move.
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.

‎docs/dev/NEXT_COMMAND.txt‎

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

‎docs/dev/reports/BUILD_PR_TARGETED_REPO_CLEANUP_PASS_5_CLASSES_OLD_KEEP_POLICY_PS_FIRST_report.md‎

Lines changed: 51 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,51 @@
1+
# BUILD_PR_TARGETED_REPO_CLEANUP_PASS_5_CLASSES_OLD_KEEP_POLICY_PS_FIRST Report
2+
3+
Generated: 2026-04-12
4+
Bundle type: execution-ready BUILD docs (classes_old_keep policy, PS-first evidence)
5+
6+
## Exact Files Created
7+
- `docs/dev/reports/classes_old_keep_policy_inventory.md`
8+
- `docs/dev/reports/classes_old_keep_policy_decision.md`
9+
- `docs/dev/reports/classes_old_keep_validation_guard.md`
10+
- `docs/dev/reports/BUILD_PR_TARGETED_REPO_CLEANUP_PASS_5_CLASSES_OLD_KEEP_POLICY_PS_FIRST_report.md`
11+
12+
## Exact Files Changed
13+
- `docs/dev/reports/validation_checklist.txt`
14+
15+
## 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.
18+
- no active runtime/code references found in supplied scan output.
19+
20+
## Supporting Existing Cleanup Evidence
21+
- `docs/dev/reports/cleanup_live_reference_inventory.txt` (docs-reference-only, path missing)
22+
- `docs/dev/reports/cleanup_keep_move_future_delete_matrix.md` (`needs-manual-review` planning target)
23+
24+
## Chosen Policy Classification
25+
- `keep-as-doc-placeholder`
26+
- with future `needs-manual-review` gating for rename/removal decision lane.
27+
28+
## Roadmap Changes Applied
29+
- None.
30+
- Roadmap left unchanged in this lane.
31+
32+
## Unapplied Planned Delta
33+
- None.
34+
- Optional roadmap bracket progression was not required for this lane.
35+
36+
## Structural Safety Assertions
37+
- No `classes_old_keep/` directory was created, moved, renamed, or deleted.
38+
- `templates/` was untouched.
39+
- `docs/archive/` was untouched.
40+
- SpriteEditor archive surfaces were untouched.
41+
- No runtime code or repo-structure changes were made.
42+
43+
## Validation Results
44+
1. Verify no `classes_old_keep` directory exists on disk:
45+
- `Test-Path classes_old_keep` -> `False`
46+
2. Verify required report files exist:
47+
- all required files present in `docs/dev/reports/`
48+
3. Confirm protected start_of_day directories unchanged:
49+
- `git status --short -- docs/dev/start_of_day/chatGPT docs/dev/start_of_day/codex` -> no entries
50+
4. Roadmap diff check (if changed):
51+
- not required; roadmap unchanged in this lane.

‎docs/dev/reports/change_summary.txt‎

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,2 +1,4 @@
1-
- SpriteEditor_old_keep successfully moved to archive
2-
- No runtime or template impact
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.

‎docs/dev/reports/classes_old_keep_policy_decision.md‎

Lines changed: 56 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,56 @@
1+
# classes_old_keep Policy Decision
2+
3+
Generated: 2026-04-12
4+
Scope: policy classification for `classes_old_keep` using user-supplied PowerShell evidence first.
5+
6+
## Classification Options Considered
7+
- `keep-as-doc-placeholder`
8+
- `archive-doc-references-later`
9+
- `needs-manual-review`
10+
11+
## Chosen Classification (This PR)
12+
- `keep-as-doc-placeholder`
13+
- secondary flag: `needs-manual-review` for future retirement/rename decision lane
14+
15+
## Rationale (Evidence-Grounded)
16+
Primary (user-supplied PowerShell evidence):
17+
- `classes_old_keep` appears only in docs/planning/generated-doc files.
18+
- no `classes_old_keep` directory exists on disk.
19+
- no active runtime/code references were found.
20+
21+
Supporting cleanup artifacts:
22+
- cleanup inventory marks `classes_old_keep` as docs-reference-only and non-existent on disk.
23+
- cleanup matrix marks `classes_old_keep` as `needs-manual-review`.
24+
25+
Together, this indicates `classes_old_keep` is a planning placeholder, not an active runtime surface.
26+
27+
## Allowed Now
28+
- Maintain docs-only placeholder tracking.
29+
- Document policy/validation guard for future cleanup sequencing.
30+
- Keep references explicit until a dedicated rename/removal docs lane is approved.
31+
32+
## Forbidden Now
33+
- Creating a `classes_old_keep/` directory.
34+
- Moving, renaming, or deleting paths in a structural cleanup lane from this PR.
35+
- Mixing `classes_old_keep` policy work with runtime changes, templates changes, archive changes, or SpriteEditor archive changes.
36+
37+
## Prerequisites For Future Rename/Removal/Archive Lane
38+
1. Reconfirm docs-only reference surface at execution time.
39+
2. Confirm no runtime/code references exist at execution time.
40+
3. Prepare synchronized updates for roadmap + cleanup targets + cleanup reports.
41+
4. Provide clear replacement wording if placeholder is renamed/retired.
42+
5. Run guard checks and capture before/after validation report in same lane.
43+
44+
## Reclassification Signals
45+
Move from `keep-as-doc-placeholder` to `archive-doc-references-later` or full removal only when:
46+
- docs references are reduced to a clear, bounded set,
47+
- target intent is resolved (historical placeholder vs active policy target),
48+
- synchronized docs updates are ready and validated.
49+
50+
Escalate to `needs-manual-review` when:
51+
- contradictory references appear (for example runtime/code usage), or
52+
- policy intent cannot be resolved from cleanup docs alone.
53+
54+
## Decision Summary
55+
- `classes_old_keep` remains docs-only placeholder in this PR.
56+
- No structural path action is authorized or performed.

‎docs/dev/reports/classes_old_keep_policy_inventory.md‎

Lines changed: 65 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,65 @@
1+
# classes_old_keep Policy Inventory
2+
3+
Generated: 2026-04-12
4+
Scope: docs-only policy inventory using user-supplied PowerShell scan evidence as primary source of truth.
5+
6+
## Primary Evidence Input (User-Supplied PowerShell Scan)
7+
The user-provided scan evidence establishes:
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.
11+
12+
This lane treats the above as authoritative baseline and avoids broad rediscovery scans.
13+
14+
## Supporting Existing Cleanup Evidence
15+
- `docs/dev/reports/cleanup_live_reference_inventory.txt`
16+
- Target 3: `classes_old_keep` policy locations/references
17+
- Exact path: `classes_old_keep/`
18+
- Exists: no
19+
- Inbound references: roadmap + cleanup targets + build spec docs
20+
- Assessment: docs-reference-only
21+
- `docs/dev/reports/cleanup_keep_move_future_delete_matrix.md`
22+
- Target row: `classes_old_keep` policy target
23+
- Proposed classification: `needs-manual-review`
24+
- Evidence summary: planning references only; no on-disk path
25+
26+
## Evidence File Matches By Type
27+
28+
### Cleanup Report
29+
- `docs/dev/reports/cleanup_live_reference_inventory.txt`
30+
31+
### Cleanup Matrix / Normalization / Enforcement Reports
32+
- `docs/dev/reports/cleanup_keep_move_future_delete_matrix.md`
33+
- `docs/dev/reports/cleanup_target_normalization_report.md` (tracking context)
34+
- `docs/dev/reports/cleanup_target_enforcement_map.md` (tracking context)
35+
36+
### Roadmap
37+
- `docs/dev/roadmaps/MASTER_ROADMAP_HIGH_LEVEL.md` (`classes_old_keep` policy tracking item)
38+
39+
### PR / BUILD Spec
40+
- `docs/pr/BUILD_PR_REPO_CLEANUP_AND_ROADMAP_UPDATE.md` (historical cleanup evidence lane)
41+
- `docs/pr/BUILD_PR_TARGETED_REPO_CLEANUP_PASS_5_CLASSES_OLD_KEEP_POLICY_PS_FIRST.md` (this lane)
42+
43+
### Generated Command Metadata (From User-Supplied Scan Context)
44+
- `docs/dev/CODEX_COMMANDS.md`
45+
- `docs/dev/NEXT_COMMAND.txt`
46+
- generated report metadata surfaces under `docs/dev/reports/`
47+
48+
## Separation Required By This Lane
49+
50+
### Docs/Planning References
51+
- Present (policy/roadmap/build docs).
52+
53+
### Generated BUILD Metadata References
54+
- Present (command/report metadata surfaces).
55+
56+
### On-Disk Existence Result
57+
- `classes_old_keep/` directory exists on disk: **no**.
58+
59+
## Runtime/Code Surface Statement
60+
- Based on the user-supplied PowerShell scan evidence, no active runtime/code references were found for `classes_old_keep`.
61+
- No contradictory runtime evidence was required for this lane.
62+
63+
## Inventory Conclusion
64+
- `classes_old_keep` is currently a docs-only planning placeholder with no on-disk implementation path.
65+
- Structural actions (create/move/delete path) are out of scope for this docs-only policy lane.

‎docs/dev/reports/classes_old_keep_validation_guard.md‎

Lines changed: 51 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,51 @@
1+
# classes_old_keep Validation Guard
2+
3+
Generated: 2026-04-12
4+
Purpose: define checks required before any future rename/removal/archive lane for `classes_old_keep` docs references.
5+
6+
## A) Windows PowerShell-Safe Search Commands (Text-Like Files)
7+
Run from repo root.
8+
9+
1. Docs/planning/reference scan (text-like extensions only):
10+
```powershell
11+
Get-ChildItem -Recurse -File docs -Include *.md,*.txt,*.json,*.yml,*.yaml |
12+
Select-String -Pattern 'classes_old_keep' -SimpleMatch
13+
```
14+
15+
2. Optional scoped scan for command/report metadata:
16+
```powershell
17+
Get-ChildItem -Recurse -File docs\dev -Include *.md,*.txt,*.json |
18+
Select-String -Pattern 'classes_old_keep' -SimpleMatch
19+
```
20+
21+
3. Runtime/code contradiction check (only for contradiction validation):
22+
```powershell
23+
Get-ChildItem -Recurse -File tools,src,games,samples,tests -Include *.js,*.mjs,*.cjs,*.ts,*.tsx |
24+
Select-String -Pattern 'classes_old_keep' -SimpleMatch
25+
```
26+
27+
## B) On-Disk Existence Check
28+
```powershell
29+
Test-Path classes_old_keep
30+
```
31+
Expected for current policy: `False`.
32+
33+
## C) Required Docs Synchronization Surfaces (If References Are Renamed/Removed)
34+
- `docs/dev/roadmaps/MASTER_ROADMAP_HIGH_LEVEL.md`
35+
- `docs/dev/reports/repo_cleanup_targets.txt`
36+
- `docs/dev/reports/cleanup_live_reference_inventory.txt`
37+
- `docs/dev/reports/cleanup_keep_move_future_delete_matrix.md`
38+
- `docs/dev/reports/cleanup_target_enforcement_map.md`
39+
- `docs/dev/reports/cleanup_target_normalization_report.md`
40+
- active `docs/pr/*` specs referencing `classes_old_keep`
41+
42+
## D) Blocking Failure Conditions
43+
Stop future cleanup execution if any of the following occur:
44+
- `classes_old_keep/` path is created implicitly without explicit lane approval.
45+
- runtime/code references are detected.
46+
- roadmap/docs synchronization set is incomplete.
47+
- changes expand into unrelated targets (`templates`, `docs/archive`, SpriteEditor archive surfaces).
48+
- roadmap requires wording rewrite (non bracket-only) during a status-only lane.
49+
50+
## Guard Summary
51+
`classes_old_keep` remains a docs-only placeholder until a dedicated, synchronized docs cleanup lane resolves naming/retirement intent.

‎docs/dev/reports/validation_checklist.txt‎

Lines changed: 31 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,31 @@
1-
[ ] old path removed
2-
[ ] new path exists
3-
[ ] no broken refs
4-
[ ] no runtime impact
1+
BUILD_PR_TARGETED_REPO_CLEANUP_PASS_5_CLASSES_OLD_KEEP_POLICY_PS_FIRST
2+
Validation Checklist
3+
Generated: 2026-04-12
4+
5+
[PASS] protected start_of_day directories untouched
6+
Evidence: `git status --short -- docs/dev/start_of_day/chatGPT docs/dev/start_of_day/codex` -> no entries
7+
8+
[PASS] no `classes_old_keep/` directory created/moved/deleted
9+
Evidence: `Test-Path classes_old_keep` -> False
10+
11+
[PASS] `templates/` untouched
12+
Evidence: no files changed under `templates/` in this lane
13+
14+
[PASS] `docs/archive/` untouched
15+
Evidence: no files changed under `docs/archive/` in this lane
16+
17+
[PASS] classes_old_keep evidence reports created
18+
Evidence:
19+
- `docs/dev/reports/classes_old_keep_policy_inventory.md`
20+
- `docs/dev/reports/classes_old_keep_policy_decision.md`
21+
- `docs/dev/reports/classes_old_keep_validation_guard.md`
22+
- `docs/dev/reports/BUILD_PR_TARGETED_REPO_CLEANUP_PASS_5_CLASSES_OLD_KEEP_POLICY_PS_FIRST_report.md`
23+
24+
[PASS] no unrelated cleanup targets changed
25+
Evidence: lane output limited to classes_old_keep policy docs and validation checklist
26+
27+
[PASS] roadmap wording preserved if roadmap changed
28+
Evidence: roadmap unchanged in this lane
29+
30+
[PASS] roadmap changes bracket-only if roadmap changed
31+
Evidence: N/A (roadmap unchanged)

‎docs/pr/BUILD_PR_TARGETED_REPO_CLEANUP_PASS_5_CLASSES_OLD_KEEP_POLICY.md‎

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,14 @@
1+
# BUILD_PR_TARGETED_REPO_CLEANUP_PASS_5_CLASSES_OLD_KEEP_POLICY
2+
3+
## Purpose
4+
Create policy + inventory for classes_old_keep.
5+
6+
## Scope
7+
Docs-only, no structural changes.
8+
9+
## Deliverables
10+
- classes_old_keep_policy_inventory.md
11+
- classes_old_keep_policy_decision.md
12+
- classes_old_keep_validation_guard.md
13+
- BUILD report
14+
- validation checklist

0 commit comments

Comments
 (0)