vimp-engine 0.35.1 → 0.35.3

package/package.json

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

package/src/client/SoundManager.js

@@ -716,8 +716,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 +790,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 +799,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/diagnostics.js

@@ -417,6 +417,7 @@ export function createDiagnostics({
    * @returns {Function} снятие перехватчиков
    */
   function install(target = window) {
+    const doc = target.document;
     // ошибки загрузки <img>/<script> приходят обычным Event на элементе —
     // это сбои ресурсов, а не исключения
     const onError = event => {
@@ -428,7 +429,14 @@ export function createDiagnostics({
       capture(event.reason, { kind: 'rejection' });
     };
     const onPageHide = () => flush();
-    const doc = target.document;
+    // фоновую вкладку мобильный браузер убивает и без pagehide: скрытие —
+    // последний надёжный момент отправить накопленное (по таймеру отчёты
+    // ждут до minIntervalMs)
+    const onVisibility = () => {
+      if (doc?.visibilityState === 'hidden') {
+        flush();
+      }
+    };
 
     try {
       target.addEventListener('error', onError);
@@ -436,6 +444,7 @@ export function createDiagnostics({
       target.addEventListener('pagehide', onPageHide);
       // событие всплывает к документу
       doc?.addEventListener('securitypolicyviolation', reportCsp);
+      doc?.addEventListener('visibilitychange', onVisibility);
     } catch {
       // нет цели — просто не ловим
     }
@@ -446,6 +455,7 @@ export function createDiagnostics({
         target.removeEventListener('unhandledrejection', onRejection);
         target.removeEventListener('pagehide', onPageHide);
         doc?.removeEventListener('securitypolicyviolation', reportCsp);
+        doc?.removeEventListener('visibilitychange', onVisibility);
       } catch {
         // снятие не должно ронять вызывающего
       }

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();