Sprite sync hardening, fix double-centering, extract Bumper outward-force to testable helper
## Codex review on 6bb1c81
### #1 Critical (corrected): visible ball not syncing
Codex заявил что ball.syncSprite() нигде не вызывается. Это неверно —
syncSprite вызывается каждый update tick на MatchScene.ts:216 (grep
подтверждает). Но описанный симптом (ball визуально не двигается) —
реальная проблема и требует hardening.
Hardening:
- Ball.setPosition() теперь immediate-sync sprite вместе с physics body.
Раньше при teleport (stuck nudge, hard respawn) sprite оставался на
старой позиции до следующего update tick — теоретическая 1-frame
рассинхронизация устранена.
- Ball sprite получил setDepth(30) — выше fixed/setup bumper'ов чтобы
не быть случайно перекрытым в render order.
### #2 High: double-centering canvas
Корень: #game-container использовал display:flex justify-content:center,
И Phaser Scale.CENTER_BOTH добавлял margin-left на canvas. Оба
центрирования складывались → canvas сдвинут вправо.
Fix: убрал flex из #game-container, оставил только Phaser CENTER_BOTH
(он надёжно центрирует через computed margin'ы).
### #3 Medium: physics fix не тестируется
Extract Bumper outward-force logic в pure helper src/game/bumperPhysics.ts
(computeOutwardForce). Возвращает new velocity или null если apply не
нужен.
src/game/bumperPhysics.test.ts — 10 unit-тестов:
- happy paths: fast outward, fast diagonal outward → null
- force apply: slow outward (still apply min speed), fast inward,
zero velocity, fast tangential (radial<threshold)
- direction correctness: ball over bumper → outward up; ball diagonal
→ outward по диагонали 45°
- degenerate: ball точно в центре bumper → fallback вверх
- REAL stuck regression: low speed + radial nearly zero → force outward
Bumper.handleBallHit рефактор: вызывает computeOutwardForce, applies
если non-null.
117/117 tests (было 107). typecheck/lint/build ✅.
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
This commit is contained in:
@@ -20,12 +20,11 @@
|
||||
user-select: none;
|
||||
-webkit-user-select: none;
|
||||
}
|
||||
/* НЕ используем flex centering — Phaser Scale.CENTER_BOTH центрирует canvas
|
||||
через margin'ы; flex поверх этого создаёт double-centering и сдвиг вправо. */
|
||||
#game-container {
|
||||
width: 100vw;
|
||||
height: 100vh;
|
||||
display: flex;
|
||||
align-items: center;
|
||||
justify-content: center;
|
||||
}
|
||||
#loading {
|
||||
color: #ff006e;
|
||||
|
||||
Reference in New Issue
Block a user