Deduplicate test commands, standardize on run-node-tests

BUILD_PR_LEVEL_19_14_TEST_COMMAND_DEDUP

Author: DavidQ

docs/dev/CODEX_COMMANDS.md

@@ -1,52 +1,10 @@
 MODEL: GPT-5.4-codex
-REASONING: high
+REASONING: low
 
 COMMAND:
-1. Open the roadmap at:
-   docs/dev/roadmaps/MASTER_ROADMAP_HIGH_LEVEL.md
-
-2. Execute the smallest real PR needed to complete Level 19 Track B:
-   - validate boot -> run -> shutdown lifecycle
-   - validate hot reload / reset flows
-   - validate error handling paths
-   - validate long-running stability
-
-3. Search only where needed to find the existing lifecycle/runtime validation surfaces:
-   - tests/
-   - scripts/
-   - launch smoke / runtime smoke helpers
-   - engine runtime boot/reset/shutdown entry points actually used by samples/games/tools
-
-4. Implement the minimum validation-backed changes required:
-   - add/fix focused tests or validation harnesses
-   - add tiny runtime lifecycle fixes only if required to make validation deterministic
-   - do not create new features
-   - do not perform broad repo cleanup
-
-5. Produce reports:
-   - docs/dev/reports/BUILD_PR_LEVEL_19_13_RUNTIME_LIFECYCLE_VALIDATION_summary.md
-   - docs/dev/reports/BUILD_PR_LEVEL_19_13_RUNTIME_LIFECYCLE_VALIDATION_coverage.md
-   - docs/dev/reports/BUILD_PR_LEVEL_19_13_RUNTIME_LIFECYCLE_VALIDATION_results.md
-
-6. In the reports, explicitly record:
-   - which lifecycle paths were exercised
-   - which commands were run
-   - pass/fail results
-   - any remaining bounded caveats
-
-7. Update the roadmap only for Track B items proven by this PR:
-   - [x] validate boot -> run -> shutdown lifecycle
-   - [x] validate hot reload / reset flows
-   - [x] validate error handling paths
-   - [x] validate long-running stability
-   Only set to [x] if actually validated by execution in this PR.
-
-8. Package the repo-structured ZIP to:
-   <project folder>/tmp/BUILD_PR_LEVEL_19_13_RUNTIME_LIFECYCLE_VALIDATION.zip
-
-CONSTRAINTS:
-- one PR purpose only
-- smallest scoped valid change
-- no vague wording
-- no repo-wide scanning unless required
-- no roadmap-only promotion without executed validation
+1. Inspect npm test vs run-node-tests.mjs
+2. Confirm overlap
+3. Update scripts/docs to use ONLY:
+   node ./scripts/run-node-tests.mjs
+4. Remove redundant references to npm test where duplicate
+5. Package ZIP

docs/dev/COMMIT_COMMENT.txt

@@ -1,8 +1,3 @@
-Execute runtime lifecycle validation for Level 19 Track B.
+Deduplicate test commands, standardize on run-node-tests
 
-BUILD_PR_LEVEL_19_13_RUNTIME_LIFECYCLE_VALIDATION
-- validate boot/run/shutdown lifecycle
-- validate hot reload/reset flows
-- validate error handling paths
-- validate long-running stability
-- update roadmap only for execution-backed results
+BUILD_PR_LEVEL_19_14_TEST_COMMAND_DEDUP

docs/dev/reports/BUILD_PR_LEVEL_19_14_TEST_COMMAND_DEDUP_summary.md

@@ -0,0 +1,11 @@
+# Test Command Deduplication
+
+## Result
+Removed redundant test execution path.
+
+## Standard
+node ./scripts/run-node-tests.mjs is now canonical.
+
+## Overlap Confirmed
+- `npm test` resolves to `node ./scripts/run-node-tests.mjs` via `package.json`.
+- Canonical guidance was normalized in active Level 19 BUILD/PLAN docs to avoid duplicate command instructions.

docs/pr/BUILD_PR_LEVEL_19_11_FULL_VALIDATION_AND_PROMOTION.md

@@ -15,10 +15,8 @@ Promote roadmap items to [x] if validation is clean.
 
 ## Steps
 1. Run:
-   npm test
-2. Run:
    node ./scripts/run-node-tests.mjs
-3. Run:
+2. Run:
    npm run test:launch-smoke
 
 4. If ALL PASS:

docs/pr/BUILD_PR_LEVEL_19_12_REVALIDATE_AND_PROMOTE.md

@@ -9,9 +9,8 @@ Re-run full validation after Level 18 completion and prior fixes; promote remain
 - No feature work
 
 ## Steps
-1. npm test
-2. node ./scripts/run-node-tests.mjs
-3. npm run test:launch-smoke
+1. node ./scripts/run-node-tests.mjs
+2. npm run test:launch-smoke
 
 ## Acceptance
 - If ALL PASS → set all remaining Level 19 items to [x]

docs/pr/BUILD_PR_LEVEL_19_13_RUNTIME_LIFECYCLE_VALIDATION.md

@@ -43,7 +43,6 @@ Avoid:
 
 ## Validation
 Run as applicable:
-- `npm test`
 - `node ./scripts/run-node-tests.mjs`
 - `npm run test:launch-smoke`
 

docs/pr/BUILD_PR_LEVEL_19_14_TEST_COMMAND_DEDUP.md

@@ -0,0 +1,28 @@
+# BUILD_PR_LEVEL_19_14_TEST_COMMAND_DEDUP
+
+## Purpose
+Remove duplicate test execution paths to streamline validation.
+
+## Scope
+- Single purpose PR
+- Deduplicate test commands
+- No functional changes
+
+## Change
+Standardize on:
+- node ./scripts/run-node-tests.mjs
+
+Remove redundant use of:
+- npm test (when it executes same suite)
+
+## Overlap Confirmation
+- `package.json` defines `"test": "node ./scripts/run-node-tests.mjs"`.
+- Therefore `npm test` and `node ./scripts/run-node-tests.mjs` execute the same node test suite path, with `npm test` adding npm lifecycle wrapper behavior.
+
+## Validation
+- Ensure selected command covers full suite
+- Ensure no loss in coverage
+
+## Acceptance
+- One canonical test command
+- No duplicate execution paths

docs/pr/PLAN_PR_LEVEL_19_12_COMPLETE_REMAINING_18_19.md

@@ -34,7 +34,6 @@ Close the active Level 18 overlay hardening lane by finishing the last in-scope
 - Smallest scoped valid tests only.
 
 ## Validation Targets (for BUILD)
-- `npm test`
 - `node ./scripts/run-node-tests.mjs`
 - `npm run test:launch-smoke`
 - Any focused overlay/runtime tests touched by the Level 18 closure item