Skip to content

Commit 60a3075

Browse files
author
DavidQ
committed
Collision boundary normalization
1 parent 30ec6c1 commit 60a3075

9 files changed

Lines changed: 30 additions & 57 deletions

docs/dev/CODEX_COMMANDS.md

Lines changed: 5 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -2,30 +2,8 @@ MODEL: GPT-5.4
22
REASONING: high
33

44
COMMAND:
5-
Create `BUILD_PR_LEVEL_01_FINAL_RESIDUE_ONLY` as the final residue-only PR for roadmap section 1.
6-
7-
Only close these remaining items:
8-
- implementation PRs executed
9-
- src/engine/rendering
10-
- src/engine/physics
11-
- src/engine/scene
12-
13-
Required work:
14-
1. Make the smallest valid repo changes needed to truthfully close those four items.
15-
2. Ensure imports/paths remain valid.
16-
3. Ensure validation remains green.
17-
4. Update roadmap status markers only.
18-
5. Do NOT expand into section 2 or unrelated structure work.
19-
6. Do NOT reopen or rework already-complete section-1 items.
20-
21-
Success target:
22-
- section 1 becomes fully complete after this PR.
23-
24-
Final packaging step is REQUIRED:
25-
`<project folder>/tmp/BUILD_PR_LEVEL_01_FINAL_RESIDUE_ONLY.zip`
26-
27-
Hard rules:
28-
- residue-only closeout
29-
- smallest valid change set
30-
- no unrelated repo changes
31-
- no missing ZIP
5+
- Promote collision to its own boundary
6+
- Remove physics proxy
7+
- Normalize imports
8+
- Update roadmap status only
9+
- Output zip to <project>/tmp

docs/dev/COMMIT_COMMENT.txt

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,2 +1 @@
1-
Close the final section-1 residue for rendering, physics, scene, and implementation status
2-
BUILD_PR_LEVEL_01_FINAL_RESIDUE_ONLY
1+
Collision boundary normalization

docs/dev/NEXT_COMMAND.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
BUILD_PR_LEVEL_02_ENGINE_CORE_BASELINE_AND_BOUNDARY_PASS
1+
BUILD_PR_LEVEL_01_SECTION_1_CLOSEOUT_RECHECK
Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1 @@
1-
- Added final residue-only PR for the four remaining section-1 items
2-
- Limits work to rendering, physics, scene, and implementation-complete truth
3-
- Intended to fully close roadmap section 1 in one last pass
1+
Repack bundle
Lines changed: 1 addition & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1 @@
1-
- implementation PRs executed can be truthfully marked complete
2-
- src/engine/rendering exists and is valid
3-
- src/engine/physics exists and is valid
4-
- src/engine/scene exists and is valid
5-
- imports/paths remain valid
6-
- validation remains green
7-
- roadmap updated by status markers only
8-
- output ZIP created at:
9-
<project folder>/tmp/BUILD_PR_LEVEL_01_FINAL_RESIDUE_ONLY.zip
1+
Ensure collision != physics

docs/dev/roadmaps/MASTER_ROADMAP_HIGH_LEVEL.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -106,7 +106,7 @@
106106
- [x] duplicate-helper migration map defined
107107
- [x] ambiguous-name rename map defined
108108
- [x] legacy migration map defined
109-
- [x] implementation PRs executed
109+
- [.] implementation PRs executed
110110
- [x] imports normalized after moves
111111
- [x] post-move validation complete
112112

@@ -115,7 +115,7 @@
115115
- [x] `src/engine/state`
116116
- [x] `src/engine/rendering`
117117
- [x] `src/engine/input`
118-
- [x] `src/engine/physics`
118+
- [ ] `src/engine/physics`
119119
- [x] `src/engine/audio`
120120
- [x] `src/engine/scene`
121121
- [x] `src/shared/utils`
Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,18 @@
1+
# BUILD_PR_LEVEL_01_COLLISION_BOUNDARY_DEFINITION_AND_NORMALIZATION
2+
3+
## Purpose
4+
Promote collision as the direct engine boundary and remove the temporary physics proxy surface.
5+
6+
## Applied Delta
7+
- Removed proxy file: `src/engine/physics/index.js`
8+
- Normalized focused residue test import:
9+
- from `src/engine/physics/index.js`
10+
- to `src/engine/collision/index.js`
11+
- Updated roadmap status markers only:
12+
- `implementation PRs executed` -> `[.]`
13+
- `src/engine/physics` -> `[ ]`
14+
15+
## Validation
16+
- `node --check tests/core/Section1FinalResidueStructure.test.mjs`
17+
- `node tests/core/Section1FinalResidueStructure.test.mjs`
18+
- `node tests/final/PrecisionCollisionSystems.test.mjs`

src/engine/physics/index.js

Lines changed: 0 additions & 11 deletions
This file was deleted.

tests/core/Section1FinalResidueStructure.test.mjs

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ import {
2020
AttractModeController,
2121
DEFAULT_ATTRACT_CONFIG,
2222
} from '../../src/engine/scene/index.js';
23-
import { isColliding } from '../../src/engine/physics/index.js';
23+
import { isColliding } from '../../src/engine/collision/index.js';
2424

2525
assert.equal(typeof CanvasRenderer, 'function');
2626
assert.equal(typeof ResolutionScaler, 'function');
@@ -51,4 +51,3 @@ assert.equal(
5151
);
5252

5353
console.log('Section1FinalResidueStructure.test.mjs passed');
54-

0 commit comments

Comments
 (0)