aevgarik cff8656f0b 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>

2026-05-24 13:24:47 +03:00

1.9 KiB

Raw Blame History

View Raw