vimp-engine 0.35.0 → 0.35.1

package/package.json

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

package/src/client/components/view/ClientReports.js

@@ -210,6 +210,7 @@ export default class ClientReportsView {
     }
 
     details.appendChild(this._line(`User agent: ${report.userAgent ?? '—'}`));
+    details.appendChild(this._line(`Role: ${report.role ?? '—'}; page: ${report.page ?? '—'}`));
     details.appendChild(
       this._line(
         `First seen: ${report.firstSeen ?? '—'}; last seen: ${report.lastSeen ?? '—'}` +

package/src/client/lib/diagnostics.js

@@ -6,8 +6,9 @@
 // Репортёр не должен стать новым источником падений и шума: каждая
 // публичная функция глушит свои исключения, повтор известной ошибки — только
 // счётчик, разных ошибок за сессию не больше maxKeysPerSession, отправка —
-// пачками с дебаунсом и дельтами счётчиков. console.* не перехватывается
-// осознанно: там шум и случайные данные в аргументах.
+// пачками с дельтами счётчиков: первая через 2 с, следующие не чаще раза в
+// 10 с. console.* не перехватывается осознанно: там шум и случайные данные в
+// аргументах.
 
 // обрезки — те же числа, что у приёма на боксе (stage_2.md, контракт)
 const MAX_MESSAGE = 500;
@@ -125,10 +126,13 @@ function isExtensionUrl(value) {
 // очередь переполнена — fetch с keepalive. Ответ не читается
 function defaultSend(url, json) {
   if (typeof navigator !== 'undefined' && navigator.sendBeacon) {
-    const blob = new Blob([json], { type: 'application/json' });
-
-    if (navigator.sendBeacon(url, blob)) {
-      return;
+    try {
+      if (navigator.sendBeacon(url, new Blob([json], { type: 'application/json' }))) {
+        return;
+      }
+    } catch {
+      // старые Chromium бросали SecurityError на Blob с application/json
+      // (crbug.com/490015) — тогда запасной fetch
     }
   }
 
@@ -151,7 +155,9 @@ function noop() {}
  * @param {Function} [opts.send] - (url, json) => void; по умолчанию beacon/fetch
  * @param {Function} [opts.now]
  * @param {number} [opts.maxKeysPerSession=50]
- * @param {number} [opts.flushDelayMs=2000]
+ * @param {number} [opts.flushDelayMs=2000] - задержка первой отправки
+ * @param {number} [opts.minIntervalMs=10000] - интервал между отправками по
+ *   таймеру; прямой flush() его не ждёт
  * @returns {{ capture, warn, flush, setContext, install }}
  */
 export function createDiagnostics({
@@ -161,6 +167,7 @@ export function createDiagnostics({
   now = Date.now,
   maxKeysPerSession = 50,
   flushDelayMs = 2000,
+  minIntervalMs = 10000,
 } = {}) {
   if (!url) {
     return {
@@ -181,6 +188,7 @@ export function createDiagnostics({
 
   let capWarned = false;
   let timer = null;
+  let lastSentAt = null;
 
   // ошибка внутри самого репортёра не должна порождать новый отчёт
   let busy = false;
@@ -220,9 +228,16 @@ export function createDiagnostics({
   }
 
   function schedule() {
-    if (timer === null) {
-      timer = setTimeout(flush, flushDelayMs);
+    if (timer !== null) {
+      return;
     }
+
+    // первая отправка — через flushDelayMs, следующие — не чаще
+    // minIntervalMs: бокс режет приём лимитом запросов с адреса, и 429
+    // молча съедал бы приросты счётчиков непрерывно повторяющейся ошибки
+    const sinceLast = lastSentAt === null ? Infinity : now() - lastSentAt;
+
+    timer = setTimeout(flush, Math.max(flushDelayMs, minIntervalMs - sinceLast));
   }
 
   function guarded(fn) {
@@ -371,14 +386,25 @@ export function createDiagnostics({
         }
       }
 
+      if (items.length > 0) {
+        lastSentAt = now();
+      }
+
       for (let i = 0; i < items.length; i += MAX_BATCH_ITEMS) {
         sendBatch(items.slice(i, i + MAX_BATCH_ITEMS));
       }
     });
   }
 
+  // смена контекста (игра, роль): накопленное уходит СО СВОИМ контекстом —
+  // иначе отчёт игры A, отправленный уже под игрой B, получил бы на боксе
+  // отпечаток и source maps игры B
   function setContext(patch) {
     try {
+      if (Object.keys(patch).some(key => ctx[key] !== patch[key])) {
+        flush();
+      }
+
       Object.assign(ctx, patch);
     } catch {
       // контекст — вспомогательный, его сбой отчёты не останавливает

package/src/client/network/HostController.js

@@ -20,6 +20,24 @@ const SWAP_QUEUE_LIMIT = 2000;
 // предел ожидания ответа Worker'а на отладочный запрос (dump/запись)
 const DEBUG_TIMEOUT = 5000;
 
+// ErrorEvent Worker'а в главном потоке: `error` там null (исключение между
+// потоками не клонируется) — есть только message и filename/lineno/colno.
+// Кадр собирается из них: без него у отчёта нет верхнего кадра, то есть ни
+// различимого отпечатка, ни расшифровки по source map. Пустой message
+// (Worker не загрузился) бокс отбросил бы — отсюда запасной текст
+function workerErrorReport(event) {
+  if (typeof event?.error?.stack === 'string') {
+    return event.error;
+  }
+
+  const message = event?.message || 'Worker error';
+  const stack = event?.filename
+    ? `${message}\n    at ${event.filename}:${event.lineno ?? 0}:${event.colno ?? 0}`
+    : null;
+
+  return { message, stack };
+}
+
 export default class HostController {
   /**
    * @param {Object} room - настройки комнаты (имя/карта/лимит/таймеры).
@@ -85,7 +103,7 @@ export default class HostController {
   // браузера остаётся как есть
   _watchWorkerErrors(worker) {
     worker.onerror = event =>
-      this._diagnostics?.capture(event.error ?? event.message, {
+      this._diagnostics?.capture(workerErrorReport(event), {
         source: 'host-worker',
         kind: 'worker',
       });
@@ -342,27 +360,20 @@ export default class HostController {
     if (msg.type === 'ready') {
       this._finishSwap();
     } else if (msg.type === 'error') {
-      this._reportWorkerError(msg);
+      this._reportWorkerMessage(msg, 'error');
       this._abortSwap(msg.message);
     } else if (msg.type === 'diagnostic') {
-      this._reportWorkerDiagnostic(msg);
+      this._reportWorkerMessage(msg, msg.kind);
     }
   }
 
-  // сбой init Worker'а — в журнал клиентских ошибок (plan/client-reports)
-  _reportWorkerError(msg) {
-    this._diagnostics?.capture(new Error(msg.message), {
-      source: 'host-worker',
-      kind: 'error',
-    });
-  }
-
-  // необработанный reject внутри Worker'а (host.worker.js) — только в
-  // журнал: откат эстафеты завязан на 'error', не на него
-  _reportWorkerDiagnostic(msg) {
+  // сбой Worker'а, присланный сообщением ('error' — провал init,
+  // 'diagnostic' — необработанный reject): стек — самого Worker'а, а не
+  // главного потока
+  _reportWorkerMessage(msg, kind) {
     this._diagnostics?.capture(
-      { message: msg.message, stack: msg.stack },
-      { source: 'host-worker', kind: msg.kind },
+      { message: msg.message, stack: msg.stack ?? null },
+      { source: 'host-worker', kind },
     );
   }
 
@@ -426,12 +437,12 @@ export default class HostController {
         break;
 
       case 'error':
-        this._reportWorkerError(msg);
+        this._reportWorkerMessage(msg, 'error');
         this._onError?.(msg);
         break;
 
       case 'diagnostic':
-        this._reportWorkerDiagnostic(msg);
+        this._reportWorkerMessage(msg, msg.kind);
         break;
 
       case 'map_changed':

package/src/config/master.js

@@ -175,7 +175,9 @@ export default {
     // известных отпечатков бюджет не тратят
     newFingerprintsPerMinute: 60,
     // ключ — адрес, для IPv6 — подсеть /64 (lib/clientIp.js → rateLimitKey)
-    rateLimit: { limit: 10, windowMs: 60000 },
+    // клиент шлёт не чаще раза в 10 с — запас на несколько вкладок за одним
+    // адресом (NAT); главная защита — бюджет новых отпечатков
+    rateLimit: { limit: 30, windowMs: 60000 },
     bodyLimit: '16kb',
     maxItemsPerRequest: 10,
   },

package/src/host/host.worker.js

@@ -178,6 +178,9 @@ self.onmessage = async event => {
         self.postMessage({
           type: 'error',
           message: e && e.message ? e.message : String(e),
+          // стек Worker'а — для журнала клиентских ошибок: в главном потоке его
+          // уже не восстановить
+          stack: e && typeof e.stack === 'string' ? e.stack : null,
         });
       }
       break;