vimp-engine 0.35.2 → 0.35.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "vimp-engine",
-  "version": "0.35.2",
+  "version": "0.35.4",
   "description": "VIMP — движок-приложение (мастер, P2P-транспорт, Worker-хост, мета, MVC-каркас клиента)",
   "license": "MIT",
   "repository": {

package/src/client/SoundManager.js

@@ -514,8 +514,13 @@ export default class SoundManager {
       // rate только на изменение: Howler на каждый вызов делает два seek(),
       // переписывает _rateSeek/_playStart и пересоздаёт таймер конца петли
       // — на 60 Гц это лишняя нагрузка и лишние события 'end' на каждом
-      // обороте
-      if (typeof rate === 'number' && rate !== activeInstance.rate) {
+      // обороте.
+      // И только конечный: громкость Howler проверяет сам, а rate — нет,
+      // NaN/Infinity уходят прямо в playbackRate.setValueAtTime, тот
+      // бросает TypeError, и тикер Pixi останавливается (как с позицией в
+      // _writePos). К тому же NaN !== NaN — вызов повторялся бы каждый кадр.
+      // До первого конечного значения звучит прежний rate
+      if (Number.isFinite(rate) && rate !== activeInstance.rate) {
         sound.rate(rate, soundId);
         activeInstance.rate = rate;
       }
@@ -716,8 +721,11 @@ export default class SoundManager {
 
     // отсечка в мировых координатах: она обязана совпадать с maxDistance
     // самого PannerNode, поэтому зумом НЕ масштабируется — иначе движок
-    // считал бы источник слышимым там, где узел уже отдал тишину
-    if (Math.hypot(dx, dy) >= this._spatial.maxDistance) {
+    // считал бы источник слышимым там, где узел уже отдал тишину.
+    // «не ближе maxDistance», а не «дальше»: NaN-дистанция (координата
+    // источника или слушателя не число) иначе считалась бы слышимой на
+    // полной громкости
+    if (!(Math.hypot(dx, dy) < this._spatial.maxDistance)) {
       sound.volume(0, soundId);
 
       return false;
@@ -787,7 +795,8 @@ export default class SoundManager {
    * @private Пишет позицию в паннер, если она изменилась заметнее
    * POSITION_EPSILON. Единственная точка записи: у неподвижного источника
    * позиция уже в узле, и повторная запись — только лишняя автоматизация,
-   * а именно её поток WebKit и не переносит.
+   * а именно её поток WebKit и не переносит. Неконечная позиция
+   * пропускается: не пишется и не запоминается.
    * @param {Howl} sound - Экземпляр Howl.
    * @param {number} soundId - ID конкретного проигрываемого экземпляра.
    * @param {number} px - Координата X в осях Web Audio.
@@ -795,6 +804,17 @@ export default class SoundManager {
    * @param {number} pz - Координата Z в осях Web Audio.
    */
   _writePos(sound, soundId, px, py, pz) {
+    // Неконечная координата в узел не пишется: Web Audio бросает на ней
+    // TypeError из setValueAtTime, исключение уходит из рендер-тика в тикер
+    // Pixi, и тот больше не запрашивает кадр — игра замирает целиком.
+    // Позиции источников задаёт и игра (registerSound/updateSoundData),
+    // поэтому одного фильтра камеры (lib/applyCamera.js) здесь мало.
+    // Запомненная позиция не трогается: следующую конечную сравним с
+    // настоящей
+    if (!Number.isFinite(px) || !Number.isFinite(py) || !Number.isFinite(pz)) {
+      return;
+    }
+
     const written = this._pannerPos.get(soundId);
 
     if (written !== undefined) {

package/src/client/lib/applyCamera.js

@@ -14,12 +14,31 @@
  * @param {object} soundManager - Экземпляр SoundManager.
  * @param {Array | number} camera - `[x, y, cameraReset, shakeData]` из
  * горячего буфера ядра; `0` или пустое значение — кадра камеры нет.
+ * @param {Function} [onInvalid] - Зовётся с кадром, у которого x или y — не
+ * конечное число (NaN, Infinity, undefined, null); такой кадр не применяется.
  */
-export default function applyCamera(canvasManager, soundManager, camera) {
+export default function applyCamera(
+  canvasManager,
+  soundManager,
+  camera,
+  onInvalid,
+) {
   if (!camera || camera === 0) {
     return;
   }
 
+  // Неконечная координата не применяется ни к полотну, ни к слушателю.
+  // Полотну один NaN вредит надолго: CanvasManagerModel сглаживает камеру
+  // через lerp, NaN оседает в его накопителях до cameraReset — масштаб
+  // сцены NaN, картинки нет. Слушателю — сразу: NaN уходит в Howl.pos(),
+  // тот бросает, и тикер Pixi останавливается. null отсекается тоже: serde
+  // пишет неконечное число JSON-кадра как null, а в арифметике null молча
+  // стал бы нулём — прыжок камеры в начало координат
+  if (!Number.isFinite(camera[0]) || !Number.isFinite(camera[1])) {
+    onInvalid?.(camera);
+    return;
+  }
+
   canvasManager.updateCoords(camera);
   soundManager.setListenerPosition(
     camera[0],

package/src/client/lib/hotTick.js

@@ -0,0 +1,66 @@
+import { HOT_FLAGS } from '../../config/opcodes.js';
+import { reconstructHot } from '../../lib/reconstructHot.js';
+
+// Рендер-тик клиентского ядра: сэмпл, событийные кадры, горячие данные,
+// камера. Вынесено из main.js ради одной вещи — порядка ЧТЕНИЯ, который
+// ничем больше не защищён: main.js целиком в тесте не поднять.
+//
+// Hot-буфер читается view поверх памяти WASM (zero-copy), а view живёт
+// ровно до следующего роста этой памяти: `memory.grow` отцепляет старый
+// ArrayBuffer, длина view становится 0, hot[i] — undefined. Растить память
+// может любой аллоцирующий вызов ядра: take_frames() собирает строку JSON
+// в куче WASM, парты при разборе кадров сами зовут ядро. Поэтому всё нужное
+// из буфера снимается копией СРАЗУ, а применяется в прежнем порядке.
+// Раньше камера читалась после take_frames(): undefined в ней становился
+// NaN, NaN навсегда оседал в сглаживании камеры (картинка пропадала до
+// cameraReset), а слушатель звука с NaN ронял Howl.pos() — и тикер Pixi.
+
+/**
+ * Один рендер-тик клиентского ядра.
+ * @param {Object} deps
+ * @param {Object} deps.core - ClientCore: sample, hot_ptr, take_frames.
+ * @param {WebAssembly.Memory} deps.memory - Память WASM клиентского ядра.
+ * @param {Object} deps.snapshotKeysById - Результат buildSnapshotKeysById.
+ * @param {number} deps.now - Время рендера (performance.now()).
+ * @param {Function} deps.applyShot - (game, camera): событийный кадр.
+ * @param {Function} deps.applyGameData - (game): горячие данные сущностей.
+ * @param {Function} deps.applyCamera - ([x, y]): камера тика.
+ */
+export default function runHotTick({
+  core,
+  memory,
+  snapshotKeysById,
+  now,
+  applyShot,
+  applyGameData,
+  applyCamera,
+}) {
+  const len = core.sample(now);
+  // указатель — раньше memory.buffer: аргументы `new Float32Array(...)`
+  // вычисляются слева направо, и buffer, взятый до вызова ядра, мог бы уже
+  // оказаться отцепленным
+  const ptr = core.hot_ptr();
+  const hot = new Float32Array(memory.buffer, ptr, len);
+  const flags = hot[0];
+  // reconstructHot копирует поля (Array.from): после разбора view не нужен
+  const game =
+    flags & (HOT_FLAGS.GAME | HOT_FLAGS.PREDICTED)
+      ? reconstructHot(hot, snapshotKeysById)
+      : null;
+  const camera = flags & HOT_FLAGS.CAMERA ? [hot[1], hot[2]] : null;
+
+  if (flags & HOT_FLAGS.FRAMES) {
+    JSON.parse(core.take_frames()).forEach(frame => {
+      applyShot(frame.game, frame.camera);
+    });
+  }
+
+  if (game) {
+    applyGameData(game);
+  }
+
+  // камера уже разрешена ядром: предсказанная позиция либо интерполированная
+  if (camera) {
+    applyCamera(camera);
+  }
+}

package/src/client/main.js

@@ -52,10 +52,7 @@ import { readCoreAbi, dispatchCoreOp, ABI_UNKNOWN } from '../lib/coreAbi.js';
 import { ABI_OP_DEBUG_JSON } from '../config/abiOps.js';
 import { createDebugApi, debugLog, DEBUG_PREFIX } from './debug.js';
 import { buildClientCoreConfig } from '../lib/clientCoreConfig.js';
-import {
-  buildSnapshotKeysById,
-  reconstructHot,
-} from '../lib/reconstructHot.js';
+import { buildSnapshotKeysById } from '../lib/reconstructHot.js';
 import Factory from '../lib/factory.js';
 import { formatMessage } from '../lib/formatters.js';
 import { sanitizeMessage } from '../lib/sanitizers.js';
@@ -87,7 +84,6 @@ import ClientReportsView from './components/view/ClientReports.js';
 import ClientReportsCtrl from './components/controller/ClientReports.js';
 import BakingProvider from './providers/BakingProvider.js';
 import DependencyProvider from './providers/DependencyProvider.js';
-import { HOT_FLAGS } from '../config/opcodes.js';
 import wsports from '../config/wsports.js';
 import GAME_CODES from '../config/gameCodes.js';
 import {
@@ -99,6 +95,7 @@ import lobbyConfig from '../config/lobby.js';
 import authClientConfig from '../config/authClient.js';
 import clientDefaults from '../config/clientDefaults.js';
 import applyCamera from './lib/applyCamera.js';
+import runHotTick from './lib/hotTick.js';
 
 // Динамическая загрузка игры по каталогу мастера (Этап 6.3): ClientPlugin
 // (parts, bakers, игровой CSS, хуки ядра) грузится по entries.client манифеста
@@ -951,15 +948,28 @@ function applyGameData(game) {
   });
 }
 
+// Камеру с NaN/undefined applyCamera не применяет (lib/applyCamera.js), но
+// молчать о ней нельзя: это симптом сбоя выше по течению (ядро, чтение
+// hot-буфера), и журнал — единственный способ узнать о нём с прода.
+// String(): JSON.stringify превратил бы NaN в null, а undefined потерял бы
+function reportBadCamera(camera) {
+  diagnostics.warn(
+    'engine.camera.non-finite',
+    { x: String(camera[0]), y: String(camera[1]) },
+    { source: 'client' },
+  );
+}
+
 // применяет кадр целиком (первый кадр и дискретные кадры интерполяции)
 function applyShot(game, camera) {
   applyGameData(game);
-  applyCamera(modules.canvasManager, soundManager, camera);
+  applyCamera(modules.canvasManager, soundManager, camera, reportBadCamera);
 }
 
 // рендер-тик: ядро выдаёт пересечённые кадры (события, создания/удаления)
 // JSON-очередью, а горячие позиции (танки/динамика/камера + предсказанный
-// свой танк) — плоским Float32-буфером zero-copy из памяти WASM
+// свой танк) — плоским Float32-буфером zero-copy из памяти WASM; порядок
+// чтения буфера — в lib/hotTick.js
 function renderTick() {
   if (!clientCore) {
     return;
@@ -974,26 +984,16 @@ function renderTick() {
     task();
   }
 
-  const len = clientCore.sample(performance.now());
-
-  // view пересоздаётся каждый тик: рост памяти WASM детачит buffer
-  const hot = new Float32Array(wasm.memory.buffer, clientCore.hot_ptr(), len);
-  const flags = hot[0];
-
-  if (flags & HOT_FLAGS.FRAMES) {
-    JSON.parse(clientCore.take_frames()).forEach(frame => {
-      applyShot(frame.game, frame.camera);
-    });
-  }
-
-  if (flags & (HOT_FLAGS.GAME | HOT_FLAGS.PREDICTED)) {
-    applyGameData(reconstructHot(hot, snapshotKeysById));
-  }
-
-  if (flags & HOT_FLAGS.CAMERA) {
-    // камера уже разрешена ядром: предсказанная позиция либо интерполированная
-    applyCamera(modules.canvasManager, soundManager, [hot[1], hot[2]]);
-  }
+  runHotTick({
+    core: clientCore,
+    memory: wasm.memory,
+    snapshotKeysById,
+    now: performance.now(),
+    applyShot,
+    applyGameData,
+    applyCamera: camera =>
+      applyCamera(modules.canvasManager, soundManager, camera, reportBadCamera),
+  });
 
   soundManager.processAudibility();
   soundManager.updateActiveSounds();

package/src/devtools/VirtualClient.js

@@ -8,9 +8,12 @@ import { HOT_FLAGS } from '../config/opcodes.js';
 // процессе — без этого отказы тира 1–2 (сущность не спавнится, поля схемы
 // перепутаны, панель не обновляется) не видны нигде, кроме чёрного холста.
 //
-// Рендер-конвейер повторяет client/main.js:renderTick, но hot читается через
-// hot_values() (копия), а не hot_ptr()+память WASM: в Node нет смысла в
-// zero-copy, а копия не детачится при росте памяти ядра.
+// Рендер-конвейер повторяет client/lib/hotTick.js (его зовёт renderTick в
+// client/main.js): порядок применения тот же — кадры → горячие данные →
+// камера. Hot читается через hot_values() (копия), а не hot_ptr()+память
+// WASM: в Node нет смысла в zero-copy, а копия не детачится при росте
+// памяти ядра — поэтому порядок ЧТЕНИЯ, ради которого hotTick.js вынесен
+// из main.js, здесь не важен.
 
 // предел числа сохраняемых однотипных нарушений (дальше только счётчик)
 const NOTE_LIMIT = 20;
@@ -216,7 +219,7 @@ class VirtualClient {
   }
 
   /**
-   * Рендер-тик: зеркало client/main.js:renderTick без PixiJS.
+   * Рендер-тик: зеркало client/lib/hotTick.js без PixiJS.
    * @param {number} localNow - Локальное время клиента (мс).
    */
   render(localNow) {