Fix unexpected shared extraction guard violations by normalizing shared imports and consolidating duplicated numeric helpers across affected samples and tools while preserving current test behavior.

&
Fix test drift in AsteroidsValidation and VectorNativeTemplate
- export run from AsteroidsValidation test
- ensure #game canvas exists before bootAsteroidsNew
- update VectorNativeTemplate expected path to tools/templates
&
Fix node test-runner drift in AsteroidsValidation and VectorNativeTemplate
- restore named run export contract for AsteroidsValidation
- replace invalid browser-only DOM setup with a minimal node-safe test shim
- update VectorNativeTemplate path expectation to current tools/templates location

Author: DavidQ

docs/dev/CODEX_COMMANDS.md

@@ -1,13 +1,41 @@
-MODEL: GPT-5.3-codex
-REASONING: low
+MODEL: GPT-5.4-codex
+REASONING: high
 
 COMMAND:
-Execute BUILD_PR_ENGINE_IMPORT_ENFORCE_MINIMAL.
+Create BUILD_PR_TEST_RUNNER_DRIFT_FIX as a docs-following, test-only patch.
 
-Rules:
-- Fix max 2 files
-- Replace engine/ → src/engine/
-- Do NOT touch anything else
+Inspect these files first:
+- tests/run-tests.mjs
+- tests/games/AsteroidsValidation.test.mjs
+- games/Asteroids/index.js
+- tests/tools/VectorNativeTemplate.test.mjs
 
-Create report + checklist
-Package output zip
+Then apply the minimum safe changes needed to restore `npm test --ignore-scripts`:
+
+1. Re-align `tests/games/AsteroidsValidation.test.mjs` with the runner contract:
+   - export a named `run`
+   - remove any self-executing `run().catch(...)` block if present
+   - preserve existing assertions and structure
+
+2. Fix the Asteroids validation environment using a Node-safe local shim:
+   - do NOT use `document.body.innerHTML`
+   - inspect the boot path and mock only the minimal globals actually touched
+   - likely candidates: `globalThis.document`, `document.getElementById('game')`, a minimal fake canvas/context, and possibly `globalThis.window`
+   - restore original globals after the test
+
+3. Update `tests/tools/VectorNativeTemplate.test.mjs` so the expected template path matches the current repo output:
+   - `tools/templates/vector-native-arcade/`
+
+Constraints:
+- no broad rewrites
+- no stub file replacement
+- no runtime behavior changes in game code unless absolutely required by an already-existing contract mismatch
+- no new heavy dependencies
+
+Validate with:
+- npm test --ignore-scripts
+
+If there is another failing test afterward, report it separately instead of rolling unrelated fixes into this PR.
+
+NEXT COMMAND:
+npm test --ignore-scripts

docs/dev/COMMIT_COMMENT.txt

@@ -1 +1,5 @@
-Enforce minimal /src/engine import consistency (surgical fixes only)
+Fix node test-runner drift in AsteroidsValidation and VectorNativeTemplate
+
+- restore named run export contract for AsteroidsValidation
+- replace invalid browser-only DOM setup with a minimal node-safe test shim
+- update VectorNativeTemplate path expectation to current tools/templates location

docs/dev/reports/change_summary.txt

@@ -1,4 +1,12 @@
-- low-token engine import baseline lane
-- no import rewrites
-- no structural moves
-- records current /src/engine usage as contract candidate
+PR intent:
+- repair current `npm test --ignore-scripts` failures without changing runtime behavior
+- keep scope limited to test harness drift and assertion drift
+
+Primary targets:
+- tests/games/AsteroidsValidation.test.mjs
+- tests/tools/VectorNativeTemplate.test.mjs
+
+Known symptoms from user run:
+- missing named `run` export in AsteroidsValidation after a bad overwrite attempt
+- `document is not defined` when trying a browser-style patch in a plain Node test runner
+- template path expectation still points at `templates/...` while actual output is `tools/templates/...`

docs/dev/reports/file_tree.txt

@@ -1,12 +1,11 @@
-.
-./docs
-./docs/dev
-./docs/dev/codex_commands.md
-./docs/dev/commit_comment.txt
-./docs/dev/next_command.txt
-./docs/dev/reports
-./docs/dev/reports/change_summary.txt
-./docs/dev/reports/file_tree.txt
-./docs/dev/reports/validation_checklist.txt
-./docs/pr
-./docs/pr/BUILD_PR_SAMPLES_INDEX_FIXES_AND_TARGETED_PREVIEW_RETRY.md
+docs/
+  pr/
+    PLAN_PR_TEST_RUNNER_DRIFT_FIX.md
+    BUILD_PR_TEST_RUNNER_DRIFT_FIX.md
+  dev/
+    codex_commands.md
+    commit_comment.txt
+    reports/
+      change_summary.txt
+      validation_checklist.txt
+      file_tree.txt

docs/dev/reports/launch_smoke_report.md

@@ -1,20 +1,8 @@
-BUILD_PR_ENGINE_IMPORT_ENFORCE_MINIMAL
-Validation Checklist
-Generated: 2026-04-12
-
-[PASS] enforcement limited to max 2 files
-Evidence:
-- `scripts/normalize-samples-presentation.mjs`
-- `samples/phase11/1104/DistributionPackagingScene.js`
-
-[PASS] replacement rule applied (`engine/` -> `src/engine/`) in changed files
-Evidence:
-- path literals updated in both files
-
-[PASS] no additional runtime/source files changed by enforcement step
-Evidence:
-- `git status --short -- scripts/normalize-samples-presentation.mjs samples/phase11/1104/DistributionPackagingScene.js`
-
-[PASS] report generated
-Evidence:
-- `docs/dev/reports/BUILD_PR_ENGINE_IMPORT_ENFORCE_MINIMAL_report.md`
+[ ] Inspect current source before editing; do not patch from memory
+[ ] `tests/games/AsteroidsValidation.test.mjs` exports named `run`
+[ ] No standalone `run().catch(...)` remains in that file
+[ ] AsteroidsValidation uses a node-safe local DOM/canvas shim rather than `document.body.innerHTML`
+[ ] Original globals are restored after the test
+[ ] `tests/tools/VectorNativeTemplate.test.mjs` expects `tools/templates/vector-native-arcade/`
+[ ] `npm test --ignore-scripts` rerun after edits
+[ ] Any remaining failures documented separately