vimp-engine 0.14.4 → 0.16.0

package/core/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "vimp-engine-core"
-version = "0.8.3"
+version = "0.8.4"
 edition = "2024"
 description = "VIMP — движковый каркас симуляции (физика, карта, снапшот-фрейминг, интерполяция/предикт/raycast-примитивы, нав-утилиты). Без wasm-bindgen — обёртки для WASM-ABI собирает game-crate."
 license = "MIT"

package/package.json

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

package/src/client/components/model/Auth.js

@@ -31,21 +31,14 @@ export default class AuthModel {
   }
 
   // обновление данных
-  // если value невалиден, возвращается текущий value
+  // формат поля проверяет formBuilder.collectFormErrors перед сабмитом —
+  // единственный контур валидации формы (раньше здесь была вторая,
+  // параллельная проверка по options.regExp, которая молча зануляла
+  // значение и падала на строке из JSON: у неё нет .test)
   update(data) {
-    const { name } = data;
-    const { regExp } = this._options[name] ?? {};
-    let { value } = data;
-
-    if (regExp) {
-      if (regExp.test(value)) {
-        this._data[name] = value;
-      } else {
-        this._data[name] = value = '';
-      }
-    } else {
-      this._data[name] = value;
-    }
+    const { name, value } = data;
+
+    this._data[name] = value;
 
     this.publisher.emit('form', { name, value });
   }

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

@@ -1,5 +1,5 @@
 import Publisher from '../../../lib/Publisher.js';
-import { buildForm, reportFormValidity } from '../../lib/formBuilder.js';
+import { bindLiveErrors, buildForm, renderFormErrors } from '../../lib/formBuilder.js';
 
 // Singleton AuthView
 
@@ -24,19 +24,29 @@ export default class AuthView {
     this._enter = document.getElementById(elems.enterId);
     this._fieldsContainer = document.getElementById(elems.fieldsId);
     this._fields = new Map();
+    this._descriptors = [];
 
     this.publisher = new Publisher();
 
     this._renderTexts(elems, texts);
     this._renderFields(elems, params);
 
-    // форма заполнена; нативная проверка (pattern/required) — сервер всё
-    // равно валидирует своими validators (renderError), это лишь UX-фильтр
-    // до отправки
+    // ошибки видны по ходу правки (см. bindLiveErrors), а не по клику:
+    // до сабмита — по тронутым полям, после — по всей форме
+    this._liveErrors = bindLiveErrors(this._fieldsContainer, this._error, () => ({
+      descriptors: this._descriptors,
+      fields: this._fields,
+    }));
+
+    // форма заполнена; клиентская проверка (required/regExp/min/max) —
+    // сервер всё равно валидирует своими validators (renderError), это лишь
+    // UX-фильтр до отправки
     this._enter.onclick = () => {
-      if (reportFormValidity(this._fieldsContainer)) {
-        authView.publisher.emit('enter');
+      if (this._liveErrors.arm().length) {
+        return;
       }
+
+      authView.publisher.emit('enter');
     };
 
     this._mPublic.on('form', 'renderData', this);
@@ -65,6 +75,11 @@ export default class AuthView {
       ...descriptorRest,
     }));
 
+    this._descriptors = descriptors;
+    // params сюда приходят чужие (PS_AUTH_DATA, тот же массив main.js
+    // отдаёт следом в AuthCtrl.init) и остаются нетронутыми: значение поля
+    // с единственным вариантом закрепляет main.js (resolveForcedValue) до
+    // разветвления на solo-путь, а не эта форма
     this._fields = buildForm(descriptors, container, {}, ({ name, value }) => {
       this.publisher.emit('input', { name, value });
     });
@@ -146,22 +161,17 @@ export default class AuthView {
   renderData(data) {
     const { name, value } = data;
 
-    this._error.textContent = '';
+    // блок ошибок ведёт bindLiveErrors — он перевалидирует форму по 'input',
+    // то есть по ходу правки. Чистка, стоявшая здесь, висела на событии
+    // модели, а оно приходит из 'change' (у text-инпута — blur): игрок,
+    // перейдя из починенного поля в следующее, терял ошибки полей, которых
+    // ещё не касался
     this._fields.get(name)?.setValue(value);
   }
 
-  // отображает ошибки
+  // отображает серверные ошибки (PS_AUTH_DATA/validators) — тот же рендер,
+  // что и у клиентской collectFormErrors (formBuilder.renderFormErrors)
   renderError(data) {
-    this._error.textContent = '';
-
-    data.forEach(item => {
-      const name = item.name.toUpperCase();
-      const err = item.error;
-      const line = document.createElement('div');
-
-      line.textContent = err ? `${name}: ${err}` : `${name} is not correctly!`;
-
-      this._error.appendChild(line);
-    });
+    renderFormErrors(this._error, data);
   }
 }

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

@@ -1,4 +1,5 @@
 import Publisher from '../../../lib/Publisher.js';
+import { ENGINE_VERSION } from '../../lib/engineVersion.js';
 
 // Singleton LobbyView
 
@@ -34,6 +35,14 @@ export default class LobbyView {
     this._leaderboardTotal = document.getElementById(elems.leaderboardTotalId);
     this._myPlacement = document.getElementById(elems.myPlacementId);
 
+    // футер лобби: строка статична на всё время жизни вкладки, поэтому
+    // пишется здесь, а не через модель и publisher
+    const version = document.getElementById(elems.versionId);
+
+    if (version) {
+      version.textContent = `vimp-engine ${ENGINE_VERSION}`;
+    }
+
     // заголовок игры для "<TITLE> TOP-N" (SVG-ориентир) — задаётся controller'ом
     // при выборе игры (сама модель не хранит title манифеста)
     this._gameTitle = '';

package/src/client/lib/authParams.js

@@ -0,0 +1,41 @@
+import { resolveForcedValue } from './formBuilder.js';
+
+// Значения полей auth-формы до её постройки (PS_AUTH_DATA.params).
+// Вынесено из socketMethods[PS_AUTH_DATA] (main.js), чтобы этот код был
+// накрыт тестом: solo-путь (boot.autoAuth) отвечает хосту вообще без формы и
+// берёт значения ровно отсюда, то есть ошибка здесь не видна ни в одном
+// тесте формы.
+
+/**
+ * Правит `param.value` на месте: сначала память клиента
+ * (`localStorage[storage]`), затем схема. Порядок важен — поле с
+ * единственным вариантом форма не показывает и править не даёт, поэтому
+ * устаревший `localStorage` от версии игры, где вариантов было больше,
+ * обязан быть перекрыт: иначе он уедет на хост и получит отказ от
+ * игровых validators, а поправить его игроку нечем.
+ *
+ * Мутация, а не копия: тот же массив main.js отдаёт следом в
+ * `AuthCtrl.init` (оттуда он попадает в `AuthModel._data`) и в
+ * `defaultsFrom(params)` на solo-пути.
+ *
+ * @param {Array<Object>} params - PS_AUTH_DATA.params.
+ * @param {Object} [storage] - Хранилище значений, по умолчанию localStorage.
+ * @returns {Array<Object>} Тот же массив.
+ */
+export function normalizeAuthParams(params, storage = localStorage) {
+  params.forEach(param => {
+    const descriptor = param.options ?? {};
+
+    if (descriptor.storage) {
+      param.value = storage[descriptor.storage] || param.value || '';
+    }
+
+    const forced = resolveForcedValue(descriptor);
+
+    if (forced !== undefined) {
+      param.value = forced;
+    }
+  });
+
+  return params;
+}

package/src/client/lib/engineVersion.js

@@ -0,0 +1,11 @@
+// Версия npm-пакета движка, запечённая в бандл на сборке: сети для неё нет
+// (у мастера нет version-эндпоинта), а лобби обязано показывать версию того
+// кода, который сейчас исполняется, а не того, что развёрнут на сервере.
+//
+// Модуль лежит в client/lib, а не в src/config: config/* импортирует мастер
+// под Node, а сюда тянется package.json пакета — он нужен только браузерному
+// бандлу (в npm-тарболле vimp-engine package.json есть всегда, поэтому
+// standalone SDK собирается у игры тем же импортом).
+import pkg from '../../../package.json' with { type: 'json' };
+
+export const ENGINE_VERSION = pkg.version;