Code-review feedback on commit 36975ef:
- Remove redundant SetCheckPos call in BuildGroundedTransition
(InitPath already set CheckPos to begin; the second call was a
no-op that misled readers into thinking it was load-bearing).
- Correct the class-level fixture-pattern attribution: pattern is
a blend of FindEnvCollisionsMultiCellTests (engine+DataCache
setup) and IndoorWalkablePlaneTests (sphere radius 0.48f +
BuildCellWithFloor pattern). Comment was misleading by naming
only the first.
Test still fails today with 'got 60. Finding 2 fix not complete.'
No functional change.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Test asserts 60 frames of indoor flat-floor walking should produce
≤5 ContactPlane writes. Fails today (broken code: ~60 writes).
Will pass after Task 4 + Task 5 strip the per-frame synthesis path.
Fixture: synthetic CellPhysics with flat floor (±10m XY, floorZ=0),
CellBSP=null so ResolveCellId keeps the indoor classification, BSP
bounding sphere centered at the global sphere center (worldPosZ +
sphereRadius = 0.43) so NodeIntersects passes in FindWalkableInternal.
worldPosZ = -0.05 places sphere bottom 0.05m below floor so
ValidateWalkable's below-surface branch fires (dist = -0.05 < -ε).
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
