BUILD_PR_TARGETED_REPO_CLEANUP_PASS_2

Adds legacy import guard and report.

Author: DavidQ

docs/dev/CODEX_COMMANDS.md

@@ -2,12 +2,4 @@ MODEL: GPT-5.3-codex
 REASONING: high
 
 COMMAND:
-Execute BUILD_PR_TARGETED_REPO_CLEANUP_PASS_1.
-
-- Create `docs/dev/reports/cleanup_target_enforcement_map.md`, `docs/dev/reports/cleanup_execution_guard.md`, `docs/dev/reports/cleanup_target_normalization_report.md`, `docs/dev/reports/BUILD_PR_TARGETED_REPO_CLEANUP_PASS_1_report.md`, and `docs/dev/reports/validation_checklist.txt` exactly as specified in `docs/pr/BUILD_PR_TARGETED_REPO_CLEANUP_PASS_1.md`.
-- Use only the existing cleanup inventory, cleanup matrix, prior cleanup BUILD report, templates inventory, templates policy, templates validation guard, templates BUILD report, repo cleanup targets list, and master roadmap as source evidence.
-- Do not modify `templates/`, runtime code, test logic, repo structure, or protected start_of_day directories.
-- Do not introduce new cleanup targets.
-- If roadmap alignment is attempted, it must be bracket-only and only if the exact existing text already supports it.
-- Package output to:
-  <project folder>/tmp/BUILD_PR_TARGETED_REPO_CLEANUP_PASS_1.zip
+Scan repo for legacy engine import paths and generate guard report.

docs/dev/COMMIT_COMMENT.txt

@@ -1 +1,3 @@
-BUILD_PR_TARGETED_REPO_CLEANUP_PASS_1 - targeted cleanup enforcement/normalization docs lane
+BUILD_PR_TARGETED_REPO_CLEANUP_PASS_2
+
+Adds legacy import guard and report.

docs/dev/NEXT_COMMAND.txt

@@ -1 +1 @@
-After Codex completes this BUILD and the output files validate, the next testable lane should be a single-target cleanup BUILD such as a legacy-import-guard lane or an archived-notes-policy lane, chosen strictly from the approved cleanup target set.
+Proceed to APPLY_PR_TARGETED_REPO_CLEANUP_PASS_2 if validation passes.

docs/dev/reports/change_summary.txt

@@ -1,14 +1,2 @@
-Bundle type: execution-ready BUILD docs only
-PR purpose: targeted cleanup enforcement/normalization pass
-Scope:
-- create cleanup target enforcement map
-- create cleanup execution guard
-- create cleanup target normalization report
-- create BUILD report
-- create validation checklist
-Constraints:
-- non-destructive
-- templates untouched
-- no runtime/test changes
-- no start_of_day changes
-- no new cleanup targets
+- Added legacy import guard report
+- No structural repo changes

docs/dev/reports/legacy_import_guard_report.md

@@ -0,0 +1,31 @@
+# Legacy Engine Import Path Guard Report
+
+Generated: 2026-04-12 15:57:09 -04:00
+Scope: tools/src/games/samples/tests (excluding node_modules).
+
+Result
+- Guard status: PASS
+- Total legacy signature matches: 0
+
+Pattern Findings
+- Pattern: '/engine/ | Matches: 0
+- Pattern: "/engine/ | Matches: 0
+- Pattern: '../engine/ | Matches: 0
+- Pattern: "../engine/ | Matches: 0
+- Pattern: './engine/ | Matches: 0
+- Pattern: "./engine/ | Matches: 0
+- Pattern: require('/engine/ | Matches: 0
+- Pattern: require("/engine/ | Matches: 0
+- Pattern: require('../engine/ | Matches: 0
+- Pattern: require("../engine/ | Matches: 0
+- Pattern: import('/engine/ | Matches: 0
+- Pattern: import("/engine/ | Matches: 0
+- Pattern: import('../engine/ | Matches: 0
+- Pattern: import("../engine/ | Matches: 0
+
+Command Pattern
+- rg -n -F --glob '!**/node_modules/**' <pattern> tools src games samples tests
+
+Notes
+- This guard targets legacy signatures (/engine/, ../engine/, ./engine/) and excludes valid src/engine imports.
+- If future matches appear, block cleanup/apply lanes until paths are normalized to src/engine.

docs/dev/reports/validation_checklist.txt

@@ -1,33 +1,4 @@
-BUILD_PR_TARGETED_REPO_CLEANUP_PASS_1
-Validation Checklist
-Generated: 2026-04-12
-
-[PASS] no deletion/move/rename executed
-Evidence: `git diff --name-status` + structural guard => `STRUCTURAL_CHECK: PASS` (no D/R/C entries)
-
-[PASS] `templates/` untouched
-Evidence: `git diff --name-status -- templates` => no entries (`TEMPLATES_TOUCH_CHECK: PASS`)
-
-[PASS] no runtime code changes
-Evidence: `git diff --name-only -- tools src games samples tests` => no entries
-
-[PASS] no test logic changes
-Evidence: `git diff --name-only -- tools src games samples tests` => no entries
-
-[PASS] enforcement map created
-Evidence: `docs/dev/reports/cleanup_target_enforcement_map.md`
-
-[PASS] cleanup execution guard created
-Evidence: `docs/dev/reports/cleanup_execution_guard.md`
-
-[PASS] normalization report created
-Evidence: `docs/dev/reports/cleanup_target_normalization_report.md`
-
-[PASS] no new cleanup targets introduced
-Evidence: enforcement map contains only approved six targets from PR spec
-
-[PASS] roadmap unchanged or bracket-only if changed
-Evidence: `git diff --unified=0 -- docs/dev/roadmaps/MASTER_ROADMAP_HIGH_LEVEL.md` => no diff in this lane
-
-[PASS] protected start_of_day directories untouched
-Evidence: `git status --short -- docs/dev/start_of_day/chatGPT docs/dev/start_of_day/codex` => no entries
+[PASS] no deletion/move/rename
+[PASS] templates untouched
+[PASS] no runtime changes
+[PASS] guard report created

docs/pr/BUILD_PR_TARGETED_REPO_CLEANUP_PASS_2.md

@@ -0,0 +1,20 @@
+# BUILD_PR_TARGETED_REPO_CLEANUP_PASS_2
+
+## Purpose
+Introduce enforceable guard for legacy engine import paths.
+
+## Scope
+- Detect and prevent usage of:
+  - /engine/
+  - ../engine/
+  - ./engine/
+
+## Non-Goals
+- No code rewrites
+- No deletions
+- No moves
+- No templates changes
+
+## Acceptance Criteria
+- Guard report created
+- Zero matches confirmed OR matches listed explicitly