Normalize renderer vs rendering into one engine boundary

BUILD_PR_LEVEL_01_RENDERER_RENDERING_BOUNDARY_NORMALIZATION

Author: DavidQ

docs/dev/CODEX_COMMANDS.md

@@ -2,46 +2,25 @@ MODEL: GPT-5.4
 REASONING: high
 
 COMMAND:
-Create `BUILD_PR_LEVEL_01_RENDERING_VECTOR_BOUNDARY_AND_PHYSICS_CLOSEOUT`.
-
-Implement this as one combined section-1 closeout PR.
-
-Boundary decisions:
-1. `VectorDrawing` belongs in rendering.
-2. `VectorMath` belongs in shared math/utility.
-3. `src/engine/physics` must contain real reusable physics helpers, not just placeholder/barrel status.
-
-Required work:
-1. Move/normalize vector drawing:
-   - place `VectorDrawing` under the appropriate rendering-owned path
-   - update imports/exports accordingly
-
-2. Move/normalize vector math:
-   - place `VectorMath` under the appropriate shared math/utility path
-   - update imports/exports accordingly
-
-3. Close physics truthfully:
-   - extract the smallest valid set of reusable physics-domain helpers from existing games/samples into `src/engine/physics`
-   - examples may include gravity, friction, drag, acceleration/integration, bounce/impulse if already stable
-   - do NOT promote game-specific behavior prematurely
-   - keep the extraction surgical
-
-4. Validate:
+Create `BUILD_PR_LEVEL_01_RENDERER_RENDERING_BOUNDARY_NORMALIZATION`.
+
+Implement one clear rendering boundary decision:
+
+1. Do NOT keep both `renderer` and `rendering` as competing top-level engine concepts.
+2. Prefer `src/engine/rendering/` as the single domain boundary.
+3. Treat renderer as an implementation/class/module inside rendering, not as a separate parallel boundary.
+4. Normalize imports/exports to that one truth.
+5. Keep the structure simple:
+   - do NOT over-split into `vector/`, `sprite/`, `layer/` unless the repo already has enough stable clustered content to justify it
+   - prefer a flatter rendering boundary first
+6. Update roadmap/status only if needed and only by status markers or additive truth-safe structure clarification.
+7. Validate:
    - imports remain green
-   - post-move validation remains green
-   - rendering no longer owns generic vector math
-   - shared math no longer depends on rendering
-   - `src/engine/physics` now contains real reusable physics helpers
-
-5. Roadmap:
-   - update status markers only
-   - no roadmap text rewrite
-   - mark `src/engine/rendering` and `src/engine/physics` complete only if truthfully supported
-   - leave `src/engine/scene` open if it still is not done
+   - no duplicate renderer/rendering top-level ambiguity remains
+   - any subfolders added are truthfully justified, not speculative
 
-6. Final packaging step is REQUIRED:
-   - package ALL changed files into this exact repo-structured ZIP:
-     `<project folder>/tmp/BUILD_PR_LEVEL_01_RENDERING_VECTOR_BOUNDARY_AND_PHYSICS_CLOSEOUT.zip`
+Final packaging step is REQUIRED:
+`<project folder>/tmp/BUILD_PR_LEVEL_01_RENDERER_RENDERING_BOUNDARY_NORMALIZATION.zip`
 
 Hard rules:
 - implementation by Codex

docs/dev/COMMIT_COMMENT.txt

@@ -1,2 +1,2 @@
-Normalize vector rendering/math boundaries and close physics with real reusable helpers
-BUILD_PR_LEVEL_01_RENDERING_VECTOR_BOUNDARY_AND_PHYSICS_CLOSEOUT
+Normalize renderer vs rendering into one engine boundary
+BUILD_PR_LEVEL_01_RENDERER_RENDERING_BOUNDARY_NORMALIZATION

docs/dev/reports/change_summary.txt

@@ -1,4 +1,3 @@
-- Combined vector boundary normalization with physics closeout to reduce PR count
-- Rendering now owns vector drawing concerns
-- Shared math/utils now owns generic vector math
-- Physics must be closed by extracting real reusable helpers from games/samples
+- Added PR to collapse renderer/rendering ambiguity into one engine boundary
+- Recommends keeping rendering as the folder/domain and renderer as an implementation inside it
+- Avoids premature vector/sprite/layer over-structuring unless justified by real content

docs/dev/reports/validation_checklist.txt

@@ -1,8 +1,7 @@
-- VectorDrawing moved to rendering-owned path
-- VectorMath moved to shared math/utility path
+- only one top-level rendering boundary remains
+- renderer no longer competes with rendering as a parallel concept
 - imports/exports normalized and valid
-- src/engine/physics contains real reusable physics helpers
-- roadmap updated by status markers only
-- section 1 residue reduced or fully closed truthfully
+- no speculative subfolder explosion introduced
+- roadmap/status handling remains truth-safe
 - output ZIP created at:
-  <project folder>/tmp/BUILD_PR_LEVEL_01_RENDERING_VECTOR_BOUNDARY_AND_PHYSICS_CLOSEOUT.zip
+  <project folder>/tmp/BUILD_PR_LEVEL_01_RENDERER_RENDERING_BOUNDARY_NORMALIZATION.zip

docs/pr/BUILD_PR_LEVEL_01_RENDERER_RENDERING_BOUNDARY_NORMALIZATION.md

@@ -0,0 +1,43 @@
+# BUILD_PR_LEVEL_01_RENDERER_RENDERING_BOUNDARY_NORMALIZATION
+
+## Purpose
+Normalize engine rendering to one boundary: `src/engine/rendering`.
+
+## Applied Delta
+
+### 1) Single top-level rendering boundary enforced
+- Removed competing path `src/engine/render/` completely.
+- Kept `src/engine/rendering/` as the only render-domain boundary.
+
+### 2) Renderer remains implementation inside rendering
+- `CanvasRenderer`, `Renderer`, `ResolutionScaler`, `SpriteRenderSystem`, and `LayeredRenderSystem` now live directly in `src/engine/rendering/` as real implementations (not re-export wrappers).
+- `src/engine/core/Engine.js` now imports `CanvasRenderer` from `../rendering/index.js`.
+
+### 3) Import/export normalization
+- Updated sample/test imports from `/src/engine/render/index.js` and `../../src/engine/render/...` to rendering equivalents.
+- Updated render-domain contract labels in `tools/shared/renderPipelineContract.js` from `engine/render` to `engine/rendering`.
+- Updated sample metadata references from `engine/render/index/*` and `/src/engine/render/index.js` to rendering equivalents.
+
+### 4) Structure simplicity
+- No new `rendering/vector`, `rendering/sprite`, or `rendering/layer` subfolders were introduced.
+- Boundary remains flat with only one justified added module already in use: `src/engine/rendering/VectorDrawing.js`.
+
+## Roadmap Handling
+- No roadmap status change was required for this normalization slice.
+- No roadmap text rewrite was performed.
+
+## Validation
+- `node --check` on touched rendering/core/sample/test files.
+- `node tests/render/Renderer.test.mjs`
+- `node tests/production/ProductionReadiness.test.mjs`
+- `node --input-type=module -e "import { run } from './tests/final/PrecisionCollisionSystems.test.mjs'; run();"`
+- Boundary checks:
+  - no `src/engine/render/` imports remain in `src`, `games`, `samples`, `tools`, `tests`
+  - no `../render/` re-export dependencies remain
+
+## Packaging
+`<project folder>/tmp/BUILD_PR_LEVEL_01_RENDERER_RENDERING_BOUNDARY_NORMALIZATION.zip`
+
+## Scope guard
+- surgical implementation only
+- no unrelated repo changes

samples/metadata/samples.index.metadata.json

@@ -1111,7 +1111,7 @@
         "engine/debug/index/drawFrame",
         "engine/debug/index/drawPanel",
         "engine/input/index/InputService",
-        "engine/render/index/renderSpriteReadyEntities",
+        "engine/rendering/index/renderSpriteReadyEntities",
         "engine/scenes/index/Scene"
       ],
       "thumbnail": "/samples/phase-02/0213/assets/images/preview.svg",
@@ -1138,7 +1138,7 @@
         "engine/debug/index/drawFrame",
         "engine/debug/index/drawPanel",
         "engine/input/index/InputService",
-        "engine/render/index/renderSpriteReadyEntities",
+        "engine/rendering/index/renderSpriteReadyEntities",
         "engine/scenes/index/Scene",
         "engine/theme/index/Theme",
         "engine/theme/index/ThemeTokens",
@@ -1165,7 +1165,7 @@
         "engine/debug/index/drawFrame",
         "engine/debug/index/drawPanel",
         "engine/input/index/InputService",
-        "engine/render/index/renderByLayers",
+        "engine/rendering/index/renderByLayers",
         "engine/scenes/index/Scene",
         "engine/theme/index/Theme",
         "engine/theme/index/ThemeTokens"
@@ -1199,7 +1199,7 @@
         "engine/prefabs/index/createPickupPrefab",
         "engine/prefabs/index/createPlayerPrefab",
         "engine/prefabs/index/createProjectilePrefab",
-        "engine/render/index/renderSpriteReadyEntities",
+        "engine/rendering/index/renderSpriteReadyEntities",
         "engine/scenes/index/Scene",
         "engine/theme/index/Theme",
         "engine/theme/index/ThemeTokens"
@@ -1267,8 +1267,8 @@
         "engine/interaction/index/resolveInteraction",
         "engine/prefabs/index/createEnemyPrefab",
         "engine/prefabs/index/createPlayerPrefab",
-        "engine/render/index/renderByLayers",
-        "engine/render/index/renderSpriteReadyEntities",
+        "engine/rendering/index/renderByLayers",
+        "engine/rendering/index/renderSpriteReadyEntities",
         "engine/scenes/index/Scene",
         "engine/systems/index/spawnProjectile",
         "engine/systems/index/updateProjectiles",
@@ -1301,7 +1301,7 @@
         "engine/debug/index/drawFrame",
         "engine/debug/index/drawPanel",
         "engine/input/index/InputService",
-        "engine/render/index/renderSpriteReadyEntities",
+        "engine/rendering/index/renderSpriteReadyEntities",
         "engine/scenes/index/Scene",
         "engine/theme/index/Theme",
         "engine/theme/index/ThemeTokens"
@@ -1465,7 +1465,7 @@
         "engine/input/index/ActionInputService",
         "engine/persistence/index/deserializeWorldState",
         "engine/persistence/index/serializeWorldState",
-        "engine/render/index/renderSpriteReadyEntities",
+        "engine/rendering/index/renderSpriteReadyEntities",
         "engine/scenes/index/Scene",
         "engine/theme/index/Theme",
         "engine/theme/index/ThemeTokens",
@@ -1509,7 +1509,7 @@
         "engine/debug/index/drawPanel",
         "engine/input/index/ActionInputMap",
         "engine/input/index/ActionInputService",
-        "engine/render/index/renderSpriteReadyEntities",
+        "engine/rendering/index/renderSpriteReadyEntities",
         "engine/scenes/index/Scene",
         "engine/systems/index/moveRectWithTilemapCollision",
         "engine/theme/index/Theme",
@@ -1555,7 +1555,7 @@
         "engine/debug/index/drawPanel",
         "engine/input/index/ActionInputMap",
         "engine/input/index/ActionInputService",
-        "engine/render/index/renderSpriteReadyEntities",
+        "engine/rendering/index/renderSpriteReadyEntities",
         "engine/scenes/index/Scene",
         "engine/systems/index/moveRectWithTilemapCollision",
         "engine/theme/index/Theme",
@@ -1601,7 +1601,7 @@
         "engine/debug/index/drawPanel",
         "engine/input/index/ActionInputMap",
         "engine/input/index/ActionInputService",
-        "engine/render/index/renderSpriteReadyEntities",
+        "engine/rendering/index/renderSpriteReadyEntities",
         "engine/scenes/index/Scene",
         "engine/systems/index/stepArcadeBody",
         "engine/theme/index/Theme",
@@ -1648,7 +1648,7 @@
         "engine/debug/index/drawPanel",
         "engine/input/index/ActionInputMap",
         "engine/input/index/ActionInputService",
-        "engine/render/index/renderSpriteReadyEntities",
+        "engine/rendering/index/renderSpriteReadyEntities",
         "engine/scenes/index/Scene",
         "engine/systems/index/moveRectWithTilemapCollision",
         "engine/systems/index/stepArcadeBody",
@@ -1696,7 +1696,7 @@
         "engine/debug/index/drawPanel",
         "engine/input/index/ActionInputMap",
         "engine/input/index/ActionInputService",
-        "engine/render/index/renderSpriteReadyEntities",
+        "engine/rendering/index/renderSpriteReadyEntities",
         "engine/scenes/index/Scene",
         "engine/systems/index/moveRectWithTilemapCollision",
         "engine/systems/index/stepArcadeBody",
@@ -1744,7 +1744,7 @@
         "engine/debug/index/drawPanel",
         "engine/input/index/ActionInputMap",
         "engine/input/index/ActionInputService",
-        "engine/render/index/renderSpriteReadyEntities",
+        "engine/rendering/index/renderSpriteReadyEntities",
         "engine/scenes/index/Scene",
         "engine/systems/index/moveRectWithTilemapCollision",
         "engine/systems/index/stepArcadeBody",
@@ -3340,7 +3340,7 @@
         "engine/debug/index/drawPanel",
         "engine/input/index/ActionInputMap",
         "engine/input/index/ActionInputService",
-        "engine/render/index/ResolutionScaler",
+        "engine/rendering/index/ResolutionScaler",
         "engine/scenes/index/Scene",
         "engine/theme/index/Theme",
         "engine/theme/index/ThemeTokens"

samples/metadata/samples.shared.boundaries.report.json

@@ -374,7 +374,7 @@
         "boundaryStatus": "allowed-public"
       },
       {
-        "specifier": "/src/engine/render/index.js",
+        "specifier": "/src/engine/rendering/index.js",
         "importCount": 14,
         "importerExamples": [
           "phase-02/0213/SpriteRenderLayerScene.js",

samples/phase-02/0213/SpriteRenderLayerScene.js

@@ -7,7 +7,7 @@ SpriteRenderLayerScene.js
 import { Scene } from '/src/engine/scenes/index.js';
 import { Theme, ThemeTokens } from '/src/engine/theme/index.js';
 import { drawFrame, drawPanel } from '/src/engine/debug/index.js';
-import { renderSpriteReadyEntities } from '/src/engine/render/index.js';
+import { renderSpriteReadyEntities } from '/src/engine/rendering/index.js';
 
 const theme = new Theme(ThemeTokens);
 

samples/phase-02/0214/AnimationSpriteBindingScene.js

@@ -9,7 +9,7 @@ import { Theme, ThemeTokens } from '/src/engine/theme/index.js';
 import { clamp } from '/src/engine/utils/index.js';
 import { drawFrame, drawPanel } from '/src/engine/debug/index.js';
 import { AnimationController } from '/src/engine/animation/index.js';
-import { renderSpriteReadyEntities } from '/src/engine/render/index.js';
+import { renderSpriteReadyEntities } from '/src/engine/rendering/index.js';
 
 const theme = new Theme(ThemeTokens);
 

samples/phase-02/0215/ZLayerOrderingScene.js

@@ -7,7 +7,7 @@ ZLayerOrderingScene.js
 import { Scene } from '/src/engine/scenes/index.js';
 import { Theme, ThemeTokens } from '/src/engine/theme/index.js';
 import { drawFrame, drawPanel } from '/src/engine/debug/index.js';
-import { renderByLayers } from '/src/engine/render/index.js';
+import { renderByLayers } from '/src/engine/rendering/index.js';
 
 const theme = new Theme(ThemeTokens);