No commit tonight; leave all PR_072 through PR_074 changes local for review - Classify PR 072 debug barrel validation failure before continuing - PR_26140_072-followup - Fix Phase 17 runtime test browser-root import resolution - PR_26140_072b-fix-phase17-browser-root-test-resolution

Author: DavidQ

docs/dev/reports/engine_debug_index_barrel_removal_report.md

@@ -0,0 +1,47 @@
+# PR_26140_072 Engine Debug Index Barrel Removal
+
+## Summary
+- Removed active imports from the targeted debug index barrels:
+  - `src/engine/debug/index.js`
+  - `src/engine/debug/standard/threeD/index.js`
+  - `src/engine/debug/network/index.js`
+  - `src/engine/debug/inspectors/index.js`
+- Replaced debug barrel imports with direct canonical imports to the owning debug files.
+- Deleted the four targeted debug index barrel files after active imports were removed.
+- Kept changes import-only. No schema files, sample JSON, sample/game entry `index.js` files, Git commits, pushes, or GitHub PRs were created.
+
+## Direct Import Mapping
+- Debug panels:
+  - `drawSceneFrame`, `drawFrame`, `drawPanel` -> `src/engine/debug/DebugPanel.js`
+  - stacked debug panel helpers -> `src/engine/debug/DebugOverlayLayout.js`
+  - combat/debug overlay helpers -> their owning files under `src/engine/debug/`
+- Standard 3D debug:
+  - provider registry and provider constants/helpers -> `src/engine/debug/standard/threeD/providers/*`
+  - panel registry and panel constants/helpers -> `src/engine/debug/standard/threeD/panels/*`
+  - presets/bootstrap helpers -> `src/engine/debug/standard/threeD/presets/*` and `bootstrap/*`
+- Network debug:
+  - providers, panels, command packs, dashboard helpers, and diagnostics now import from their owning files under `src/engine/debug/network/`
+- Advanced inspectors:
+  - registry, host, commands, bootstrap, presets, and view models now import from owning files under `src/engine/debug/inspectors/`
+
+## Validation
+- PASS: debug barrel scan reports `NO_DEBUG_BARREL_IMPORTS`.
+- PASS: target deletion scan confirms all four targeted debug `index.js` files no longer exist.
+- PASS: no JSON files changed.
+- PASS: `node --check` passed for 226 changed existing JS/MJS files.
+- PASS: local import target validation passed for 226 changed existing JS/MJS files.
+- PASS: `npm run test:workspace-v2` passed 59/59 tests.
+- PASS: focused debug Node tests passed:
+  - `tests/final/DebugObservabilityMaturity.test.mjs`
+  - `tests/final/NetworkDebugAndServerDashboardCloseout.test.mjs`
+  - `tests/tools/CameraDebugPanel.test.mjs`
+  - `tests/tools/CollisionOverlaysDebugPanel.test.mjs`
+  - `tests/tools/RenderPipelineStagesDebugPanel.test.mjs`
+  - `tests/tools/SceneGraphInspectorDebugPanel.test.mjs`
+  - `tests/tools/TransformInspectorDebugPanel.test.mjs`
+- FAIL: `tests/runtime/Phase17DebugOverlayBottomRightPosition.test.mjs`
+  - Failure: Node could not resolve browser-root import `/src/engine/scene/Scene.js` from `samples/phase-17/1701/RaycastDemoScene.js`.
+  - Error path: `C:\src\engine\scene\Scene.js`.
+  - This surfaced during extra focused debug validation and appears related to existing Node execution of browser-root sample imports, not to debug barrel ownership.
+- STOPPED: per unattended workflow instruction, PR_26140_073 and PR_26140_074 were not started after this validation failure.
+- SKIPPED: full samples smoke test, per instruction.

docs/dev/reports/phase17_browser_root_test_resolution_report.md

@@ -0,0 +1,45 @@
+# PR_26140_072b Phase17 Browser-Root Test Resolution
+
+## Scope
+- Fixed `tests/runtime/Phase17DebugOverlayBottomRightPosition.test.mjs` only.
+- Did not change sample runtime code.
+- Did not change `src/engine` runtime code.
+- Did not change schemas.
+- Did not touch sample JSON.
+- Did not run PR_26140_073.
+
+## Change Summary
+- Added a Node `registerHooks` resolver in the Phase17 test for browser-root imports:
+  - `/src/`
+  - `/games/`
+  - `/tools/`
+  - `/samples/`
+- Converted the Phase17 sample imports from static imports to dynamic imports after the resolver is registered.
+- Added direct-run execution so `node tests/runtime/Phase17DebugOverlayBottomRightPosition.test.mjs` runs the assertions instead of only checking module load.
+
+## Why
+The PR_26140_072 classification showed the test failed because Node resolved browser-root sample imports such as `/src/engine/scene/Scene.js` as filesystem-root paths like `C:\src\engine\scene\Scene.js`.
+
+The main node test runner already registers equivalent browser-root aliases in `scripts/run-node-tests.mjs`. This PR applies that same import-resolution pattern locally to the focused Phase17 test so it can run directly under Node.
+
+## PR_26140_072 Debug Barrel Cleanup Confirmation
+- Active-code scan found no remaining imports of:
+  - `src/engine/debug/index.js`
+  - `src/engine/debug/inspectors/index.js`
+  - `src/engine/debug/network/index.js`
+  - `src/engine/debug/standard/threeD/index.js`
+- The four targeted debug index barrel files remain deleted.
+
+## Validation
+- PASS: `node tests/runtime/Phase17DebugOverlayBottomRightPosition.test.mjs`
+- PASS: `node --check tests/runtime/Phase17DebugOverlayBottomRightPosition.test.mjs`
+- PASS: `npm run test:workspace-v2`
+  - 59 passed.
+- PASS: PR_26140_072 debug barrel cleanup confirmation.
+
+## Not Run
+- PR_26140_073 was not run.
+- Full samples smoke test was not run.
+
+## Delta ZIP
+- `tmp/PR_26140_072b-fix-phase17-browser-root-test-resolution_delta.zip`

docs/dev/reports/pr_072_failure_classification_report.md

@@ -0,0 +1,49 @@
+# PR_26140_072 Failure Classification
+
+## Scope
+- Reviewed PR_26140_072 only.
+- Did not continue to PR_26140_073.
+- Did not change schemas.
+- Did not touch sample JSON.
+- Did not change runtime behavior.
+
+## Classification
+`tests/runtime/Phase17DebugOverlayBottomRightPosition.test.mjs` is failing because Node does not resolve browser-root `/src/...` imports to the repository root.
+
+This is unrelated validation debt exposed by the extra focused validation, not a PR_26140_072 debug import regression.
+
+## Evidence
+- The test imports `samples/phase-17/1701/RaycastDemoScene.js`.
+- `samples/phase-17/1701/RaycastDemoScene.js` imports `Scene` with a browser-root path:
+  - `import Scene from '/src/engine/scene/Scene.js';`
+- Running the test directly fails before any assertion:
+  - `Error [ERR_MODULE_NOT_FOUND]: Cannot find module 'C:\src\engine\scene\Scene.js' imported from ...\samples\phase-17\1701\RaycastDemoScene.js`
+  - Node resolves `/src/engine/scene/Scene.js` as `file:///C:/src/engine/scene/Scene.js`.
+- The PR_26140_072 diff for `samples/phase-17/1701/RaycastDemoScene.js` only replaced the debug barrel import:
+  - from `/src/engine/debug/index.js`
+  - to `/src/engine/debug/DebugOverlayLayout.js` and `/src/engine/debug/DebugPanel.js`
+- The thrown module-not-found error is for `/src/engine/scene/Scene.js`, not a removed debug barrel.
+
+## Decision
+No import-only PR_26140_072 fix was applied.
+
+Changing this test to support browser-root imports under Node would be a separate validation-harness concern. It would affect sample import resolution more broadly than the debug barrel removal and is outside the scoped PR_26140_072 cleanup.
+
+## Validation
+- `npm run test:workspace-v2`
+  - PASS: 59 passed.
+- Focused PR_26140_072 debug tests:
+  - PASS: `node tests/final/DebugObservabilityMaturity.test.mjs`
+  - PASS: `node tests/final/NetworkDebugAndServerDashboardCloseout.test.mjs`
+  - PASS: `node tests/tools/CameraDebugPanel.test.mjs`
+  - PASS: `node tests/tools/CollisionOverlaysDebugPanel.test.mjs`
+  - PASS: `node tests/tools/RenderPipelineStagesDebugPanel.test.mjs`
+  - PASS: `node tests/tools/SceneGraphInspectorDebugPanel.test.mjs`
+  - PASS: `node tests/tools/TransformInspectorDebugPanel.test.mjs`
+- Focused Phase17 placement test:
+  - FAIL: `node tests/runtime/Phase17DebugOverlayBottomRightPosition.test.mjs`
+  - Reason: unrelated Node/browser-root import resolution for `/src/engine/scene/Scene.js`.
+
+## Not Run
+- PR_26140_073 was not run.
+- Full samples smoke test was not run.

games/GravityWell/game/GravityWellScene.js

@@ -7,7 +7,7 @@ GravityWellScene.js
 import Scene from '/src/engine/scene/Scene.js';
 import { Theme } from '/src/engine/theme/Theme.js';
 import { ThemeTokens } from '/src/engine/theme/ThemeTokens.js';
-import { drawPanel } from '/src/engine/debug/index.js';
+import { drawPanel } from '/src/engine/debug/DebugPanel.js';
 import { drawVectorShape } from '/src/engine/rendering/VectorDrawing.js';
 import ReplaySystem from '/src/engine/replay/ReplaySystem.js';
 import GravityWellWorld from './GravityWellWorld.js';

samples/phase-01/0115/ECSFoundationScene.js

@@ -8,7 +8,7 @@ import Scene from '/src/engine/scene/Scene.js';
 import { Theme } from '/src/engine/theme/Theme.js';
 import { ThemeTokens } from '/src/engine/theme/ThemeTokens.js';
 import World from '/src/engine/ecs/World.js';
-import { drawSceneFrame } from '/src/engine/debug/index.js';
+import { drawSceneFrame } from '/src/engine/debug/DebugPanel.js';
 
 const theme = new Theme(ThemeTokens);
 

samples/phase-01/0116/ECSMovementSystemScene.js

@@ -9,7 +9,7 @@ import { Theme } from '/src/engine/theme/Theme.js';
 import { ThemeTokens } from '/src/engine/theme/ThemeTokens.js';
 import World from '/src/engine/ecs/World.js';
 import { createTransform, createSize, createVelocity, createRenderable } from '/src/engine/components/index.js';
-import { drawSceneFrame } from '/src/engine/debug/index.js';
+import { drawSceneFrame } from '/src/engine/debug/DebugPanel.js';
 import { moveEntities, bounceEntitiesHorizontallyInBounds, renderRectEntities } from '/src/engine/systems/index.js';
 
 const theme = new Theme(ThemeTokens);

samples/phase-01/0117/ECSInputSystemScene.js

@@ -16,7 +16,7 @@ import {
   createInputControlled,
   createRenderable,
 } from '/src/engine/components/index.js';
-import { drawSceneFrame } from '/src/engine/debug/index.js';
+import { drawSceneFrame } from '/src/engine/debug/DebugPanel.js';
 import { applyInputControl, moveEntities, clampEntitiesToBounds, renderRectEntities } from '/src/engine/systems/index.js';
 
 const theme = new Theme(ThemeTokens);

samples/phase-01/0118/ECSCollisionSystemScene.js

@@ -18,7 +18,7 @@ import {
   createSolid,
   createRenderable,
 } from '/src/engine/components/index.js';
-import { drawSceneFrame } from '/src/engine/debug/index.js';
+import { drawSceneFrame } from '/src/engine/debug/DebugPanel.js';
 import {
   applyInputControl,
   moveEntities,

samples/phase-01/0119/ECSRenderSystemScene.js

@@ -8,7 +8,7 @@ import Scene from '/src/engine/scene/Scene.js';
 import { Theme } from '/src/engine/theme/Theme.js';
 import { ThemeTokens } from '/src/engine/theme/ThemeTokens.js';
 import World from '/src/engine/ecs/World.js';
-import { drawSceneFrame } from '/src/engine/debug/index.js';
+import { drawSceneFrame } from '/src/engine/debug/DebugPanel.js';
 import { renderRectEntities } from '/src/engine/systems/index.js';
 
 const theme = new Theme(ThemeTokens);

samples/phase-01/0120/ECSSceneWorldScene.js

@@ -18,7 +18,8 @@ import {
   createSolid,
   createRenderable,
 } from '/src/engine/components/index.js';
-import { drawSceneFrame, drawValidationPanel, validateWorldEntities } from '/src/engine/debug/index.js';
+import { drawSceneFrame } from '/src/engine/debug/DebugPanel.js';
+import { drawValidationPanel, validateWorldEntities } from '/src/engine/debug/Validation.js';
 import {
   applyInputControl,
   moveEntities,