vimp-engine 0.14.3 → 0.15.0

package/src/client/style.css

@@ -46,7 +46,9 @@ body.hide-cursor {
   background-color: #fff;
   text-align: center;
   font-size: 14px;
-  line-height: 12px;
+  /* межстрочный не меньше кегля: при 12px глифы 14px вылезали за строку,
+     и ячейки панели держались только на её height/overflow */
+  line-height: 16px;
   position: absolute;
   top: 0;
   left: 0;
@@ -451,15 +453,19 @@ body.hide-cursor {
   z-index: 8;
 }
 
-#auth-error {
+/* общий стиль ошибок формы — #lobby-error и #auth-error (formBuilder.js
+   renderFormErrors рендерит в них один и тот же список <div>) */
+.form-error {
   color: #e00;
   font-size: 12px;
   line-height: 15px;
   margin: 0 5px;
 }
 
-#auth-error:empty {
-  margin: 0;
+/* пустой блок ошибок не должен занимать место: #lobby-error — флекс-элемент
+   #lobby-create с gap:8px, и одного обнуления margin там мало */
+.form-error:empty {
+  display: none;
 }
 
 #auth-enter {
@@ -619,14 +625,6 @@ body.hide-cursor {
   margin: 16px auto 0;
 }
 
-#lobby-error {
-  margin: 8px 0 0;
-  color: #d98282;
-  font-size: 12px;
-  line-height: 16px;
-  text-align: center;
-}
-
 #lobby-search {
   display: block;
   width: 100%;

package/src/client/views/gameShell.js

@@ -43,7 +43,7 @@ const SHELL = [
         children: [
           // заголовок и help-секции заполняет AuthView из authSchema.texts
           { tag: 'h2', id: 'auth-title' },
-          { tag: 'div', id: 'auth-error' },
+          { tag: 'div', id: 'auth-error', attrs: { class: 'form-error' } },
           { tag: 'div', id: 'auth-fields' },
           {
             tag: 'input',

package/src/client/views/includes/auth.pug

@@ -6,7 +6,7 @@ div#auth
 
     h2#auth-title
 
-    div#auth-error
+    div#auth-error.form-error
 
     div#auth-fields
 

package/src/client/views/includes/lobby.pug

@@ -26,12 +26,13 @@ div#lobby(style='display:none')
         //- поля настроек комнаты генерируются по явной схеме roomForm
         //- манифеста активной игры (docs/en/plugin-api.md "Form schema")
         div#lobby-fields
-        input#lobby-host(type='button', value='Create server')
-        //- отказ загрузки плагина выбранной игры: восстановим (другая игра,
-        //- повторный клик), поэтому строкой в форме, а не #tech-informer —
+        //- ошибки формы (formBuilder.renderFormErrors) и отказ загрузки
+        //- плагина выбранной игры — строкой в форме, а не в #tech-informer:
         //- тот кроет вкладку непрозрачным слоем и рассчитан на терминальные
-        //- причины
-        p#lobby-error(style='display:none')
+        //- причины, а эти восстановимы (поправить поле, другая игра,
+        //- повторный клик). Над кнопкой — рядом с полями, к которым относятся
+        div#lobby-error.form-error
+        input#lobby-host(type='button', value='Create server')
     //- правая панель: вкладки Active Servers / Leaderboard (lobby-page-plan)
     div#lobby-browser-panel.card
       //- абсолютный слой: высоту .card задаёт форма слева (stretch), а

package/src/devtools/contract/index.js

@@ -1,6 +1,6 @@
 import { loadContext } from './loadContext.js';
 import { rules } from './rules/index.js';
-import { ERROR, FAIL, summarize } from './result.js';
+import { ERROR, FAIL, WARN, summarize } from './result.js';
 
 // Статическая проверка контракта «движок ↔ игра»: то же намерение, что у
 // headless-раннера (молчаливое нарушение обязано становиться строкой),
@@ -35,20 +35,43 @@ export async function checkContract(gameDir) {
   };
 }
 
+// Вердикт может нести собственный уровень (см. verdict) — но только один из
+// двух известных: неизвестное значение молча перестало бы блокировать прогон
+// (hasBlockingFailure сравнивает с ERROR), то есть опечатка в правиле
+// отключала бы его без единой строки. Поэтому она громкая, а уровень берётся
+// объявленный правилом.
+function resolveLevel(level, rule) {
+  if (level === undefined) {
+    return rule.level;
+  }
+
+  if (level !== ERROR && level !== WARN) {
+    console.error(
+      `contract: rule ${rule.id} returned unknown level "${level}" — ` +
+        `falling back to "${rule.level}"`,
+    );
+
+    return rule.level;
+  }
+
+  return level;
+}
+
 /**
  * Прогон правил по готовому контексту (тесты собирают его вручную).
  * @param {Object} ctx
+ * @param {Array<Object>} [ruleList] - Набор правил; по умолчанию все.
  * @returns {Array<Object>} Результаты правил.
  */
-export function runRules(ctx) {
-  return rules.map(rule => {
-    const { status, violations, note } = rule.check(ctx);
+export function runRules(ctx, ruleList = rules) {
+  return ruleList.map(rule => {
+    const { status, violations, note, level } = rule.check(ctx);
 
     return {
       id: rule.id,
       name: rule.name,
       title: rule.title,
-      level: rule.level,
+      level: resolveLevel(level, rule),
       status,
       violations,
       note,

package/src/devtools/contract/result.js

@@ -24,15 +24,21 @@ export function skip(note) {
 
 /**
  * Вердикт по списку нарушений: пустой список — pass.
+ * `level` перекрывает уровень правила на этом прогоне: у правила он один на
+ * все случаи, а бывает, что нарушение доказуемо только частично (C4 не может
+ * перечислить игровые сервисы, не запустив игровой код) — такое обязано
+ * попасть в отчёт, но не обязано валить прогон.
  * @param {Array<string>} violations
  * @param {string} [note]
- * @returns {Object} { status, violations, note }.
+ * @param {string} [level] - ERROR или WARN; по умолчанию — уровень правила.
+ * @returns {Object} { status, violations, note, level }.
  */
-export function verdict(violations, note) {
+export function verdict(violations, note, level) {
   return {
     status: violations.length ? FAIL : PASS,
     violations,
     note,
+    level,
   };
 }
 

package/src/devtools/contract/rules/b5-room-form.js

@@ -1,9 +1,11 @@
 import { ERROR, skip, verdict } from '../result.js';
+import { anchorPattern } from '../../../lib/formPattern.js';
 
 // roomForm. Поле вне белого списка форма показывает, лобби отправляет, а
 // хост молча выбрасывает — правило игры, построенное на своей настройке
 // комнаты, не работает и ничего об этом не сообщает. Контролов в v3 ровно
-// четыре: неизвестный пропускается с console.error.
+// четыре: неизвестный пропускается с console.error. `regExp` едет в манифесте
+// строкой и компилируется уже у игрока — некомпилируемая ловится здесь.
 const HONOURED = ['maps', 'maxPlayers', 'map', 'roundTime', 'mapTime', 'friendlyFire'];
 const CONTROLS = ['text', 'select', 'checkbox', 'radio'];
 
@@ -36,6 +38,20 @@ export default {
             `not exist (${CONTROLS.join(', ')})`,
         );
       }
+
+      if (field.regExp !== undefined) {
+        try {
+          // ровно та форма, что компилирует клиент (общий anchorPattern):
+          // иначе «компилируется» у чекера и у игрока разъедется молча
+          anchorPattern(field.regExp);
+        } catch (e) {
+          violations.push(
+            `roomForm field "${field.name}": regExp "${field.regExp}" does ` +
+              `not compile (${e.message}) — the client drops the check with ` +
+              'a console.error, so the field ends up with no pattern at all',
+          );
+        }
+      }
     }
 
     return verdict(violations);

package/src/devtools/contract/rules/c10-auth-schema.js

@@ -1,17 +1,21 @@
 import { ERROR, skip, verdict } from '../result.js';
+import { resolveValidator } from '../../../lib/validators.js';
 
-// authSchema. Три ошибки, каждая из которых уже случалась:
+// authSchema. Четыре ошибки, каждая из которых уже случалась:
 // formId вместо fieldsId (контейнер резолвится в null и экран авторизации
 // умирает TypeError на первом рендере), поле ника (личность приходит из
 // JWT лобби) и поле выбора модели под своим именем — движок читает
-// params.model, всё остальное до Participant не доезжает.
+// params.model, всё остальное до Participant не доезжает, и имя валидатора,
+// которого нет в authSchema.validators (поле не проверяется никем).
 const NICKNAME = /^(name|nick|nickname|player|playername|login|username)$/i;
 
 export default {
   id: 'C10',
   name: 'authSchema',
   level: ERROR,
-  title: 'authSchema: fieldsId, no nickname field, the model field',
+  title:
+    'authSchema: fieldsId, no nickname field, the model field, resolvable ' +
+    'validators',
 
   check(ctx) {
     if (!ctx.authSchema) {
@@ -34,6 +38,8 @@ export default {
       );
     }
 
+    const validators = ctx.authSchema.validators;
+
     for (const field of params) {
       if (NICKNAME.test(field.name)) {
         violations.push(
@@ -41,6 +47,23 @@ export default {
             'identity comes from the lobby JWT',
         );
       }
+
+      const validatorName = field.options?.validator;
+
+      // опечатка в имени (как и не-функция под верным именем) = поле не
+      // проверяется никем: validateAuth пропускает нерезолвнутый валидатор
+      // молча. Резолвер — тот же, которым зовёт хост (lib/validators.js),
+      // чтобы правило не обещало того, чего не проверяет
+      if (
+        validatorName !== undefined &&
+        !resolveValidator(validatorName, validators)
+      ) {
+        violations.push(
+          `authSchema param "${field.name}" names validator ` +
+            `"${validatorName}", which authSchema.validators does not ` +
+            'provide as a function — the host skips the check silently',
+        );
+      }
     }
 
     if (!params.some(field => field.name === 'model')) {

package/src/devtools/contract/rules/c4-component-dependencies.js

@@ -1,8 +1,12 @@
-import { ERROR, skip, verdict } from '../result.js';
+import { ERROR, WARN, skip, verdict } from '../result.js';
 
-// Пул сервисов клиента ровно из четырёх имён (client/main.js). Незнакомое имя
-// не ошибка для движка: part получает undefined и рисует пустоту — карта
-// без assetsBase выглядит как чистый холст без единой строки в консоли.
+// Движковых сервисов ровно четыре (client/main.js), но пул ими не
+// исчерпывается: игра доливает туда свои через ClientPlugin.hooks.services(core)
+// — например геометрию предсказанной динамики карты
+// (docs/en/plugin-api.md «hooks.services»). Незнакомое имя не ошибка для
+// движка: part получает undefined и рисует пустоту — карта без assetsBase
+// выглядит как чистый холст без единой строки в консоли, ради этого правило
+// и существует.
 const SERVICES = ['renderer', 'soundManager', 'assetsBase', 'localPlayer'];
 
 export default {
@@ -19,15 +23,42 @@ export default {
     }
 
     // раскладка «сервис → парты, которым он нужен» (client.js игры)
-    const violations = Object.keys(deps)
-      .filter(service => !SERVICES.includes(service))
-      .map(
+    const unknown = Object.keys(deps).filter(
+      service => !SERVICES.includes(service),
+    );
+
+    if (unknown.length === 0) {
+      return verdict([]);
+    }
+
+    const pool = SERVICES.join(', ');
+    // hooks.services(core) требует живой инстанс ядра, а игровой код чекер не
+    // запускает — статически перечислить игровые сервисы нечем. Поэтому при
+    // объявленном хуке имя вне движкового пула остаётся строкой отчёта, но
+    // прогон не валит: иначе правило краснеет на штатном механизме движка
+    if (typeof ctx.clientPlugin?.hooks?.services === 'function') {
+      return verdict(
+        unknown.map(
+          service =>
+            `componentDependencies declares service "${service}" — not one of ` +
+            `the engine's (${pool}), so ClientPlugin.hooks.services() has to ` +
+            'return it; verifying that needs a live core, which this check ' +
+            'has no way to build (a service nothing provides is silently ' +
+            'undefined in the part)',
+        ),
+        'the plugin declares hooks.services(), so the pool is not statically known',
+        WARN,
+      );
+    }
+
+    return verdict(
+      unknown.map(
         service =>
           `componentDependencies declares service "${service}" — the engine ` +
-          `provides only ${SERVICES.join(', ')} (an unknown one is silently ` +
+          `provides only ${pool}, and this plugin declares no ` +
+          'hooks.services() to add its own (an unknown one is silently ' +
           'undefined in the part)',
-      );
-
-    return verdict(violations);
+      ),
+    );
   },
 };

package/src/devtools/contract/rules/c6-stat-columns.js

@@ -1,13 +1,77 @@
 import { WARN, skip, verdict } from '../result.js';
 
-// Движок пишет ровно пять имён (name, status, score, deaths, latency), и
-// его CSS свёрстана под пять колонок. Другое число — не отказ: игра вправе
-// привезти свои стили в ClientPlugin.styles. Поэтому warn, а не error.
+// Сколько в статистике колонок — решение игры: их объявляет её схема, а
+// движковые записи в необъявленную колонку просто игнорируются
+// (host/meta/modules/Stat.js, Д7). Своя у движка только вёрстка — style.css
+// раздаёт ширины пяти колонкам (#stat …:nth-child(1)…(5)), и шестая без
+// собственных стилей схлопывается в ноль. Поэтому правило проверяет не число
+// колонок, а покрыта ли раскладка: игра со своими ClientPlugin.styles вправе
+// объявить сколько угодно. Ширина — соглашение, а не отказ, отсюда warn.
+const ENGINE_COLUMNS = 5;
+
+// Правило разбирает CSS плагина эвристикой, а не парсером: «колонку кроет
+// объявление ширины на селекторе с #stat и ячейкой».
+// Пара «селектор + тело»: [^{}] не перешагивает вложенную скобку, поэтому
+// matchAll ловит именно внутреннее правило, в том числе завёрнутое в
+// @media/@supports (разбор по split('}') видел там обёртку и терял селектор).
+const RULE = /([^{}]+)\{([^{}]*)\}/g;
+// ячейка колонки: движковая раскладка адресует их как `#stat … td|span`
+const CELL = /(?:^|[\s.#>+~])(?:td|th|span)(?=[\s.:#>+~,[]|$)/;
+// правило про раскладку, а не про цвет: колонку кроет только объявление
+// ширины или её флекс-эквивалент
+const WIDTH =
+  /(?:^|[\s;])(?:width|min-width|max-width|flex|flex-basis|flex-grow)\s*:/;
+// трек-лист грида задаёт ширины всем колонкам разом и стоит на контейнере
+// (`#stat table`, `#stat .stat-head`), который CELL по определению не
+// проходит, — поэтому отдельной веткой, а не пунктом в WIDTH
+const GRID = /(?:^|[\s;])grid-template-columns\s*:/;
+
+// Индексы колонок, которым стили плагина задают ширину. Селектор ячеек без
+// nth-child ('#stat table td') правит всю таблицу разом.
+function styledColumns(styles, total) {
+  const covered = new Set();
+
+  for (const [, selector, body] of String(styles || '').matchAll(RULE)) {
+    if (!selector.includes('#stat')) {
+      continue;
+    }
+
+    // грид на контейнере перечисляет все треки сразу — колонки покрыты все
+    if (GRID.test(body)) {
+      for (let index = 1; index <= total; index += 1) {
+        covered.add(index);
+      }
+
+      continue;
+    }
+
+    if (!CELL.test(selector) || !WIDTH.test(body)) {
+      continue;
+    }
+
+    const indexes = [...selector.matchAll(/nth-child\((\d+)\)/g)];
+
+    if (indexes.length === 0) {
+      for (let index = 1; index <= total; index += 1) {
+        covered.add(index);
+      }
+
+      continue;
+    }
+
+    for (const [, index] of indexes) {
+      covered.add(Number(index));
+    }
+  }
+
+  return covered;
+}
+
 export default {
   id: 'C6',
   name: 'statColumns',
   level: WARN,
-  title: 'stat declares five columns (the engine CSS assumes five)',
+  title: 'stat columns past the engine layout are styled by the plugin',
 
   check(ctx) {
     const columns = ctx.clientConfig?.modules?.stat?.params?.columns;
@@ -16,21 +80,32 @@ export default {
       return skip('no client stat columns');
     }
 
-    if (columns.length === 5) {
+    // меньше пяти — движковых ширин просто хватает с запасом
+    if (columns.length <= ENGINE_COLUMNS) {
       return verdict([]);
     }
 
-    const note = ctx.clientPlugin?.styles
-      ? 'the plugin ships its own styles'
-      : undefined;
+    const covered = styledColumns(ctx.clientPlugin?.styles, columns.length);
+    const missing = [];
+
+    for (let index = ENGINE_COLUMNS + 1; index <= columns.length; index += 1) {
+      if (!covered.has(index)) {
+        missing.push(index);
+      }
+    }
+
+    if (missing.length === 0) {
+      return verdict(
+        [],
+        `${columns.length} columns, laid out by the plugin's own styles`,
+      );
+    }
 
-    return verdict(
-      [
-        `stat declares ${columns.length} column(s): the engine populates ` +
-          'exactly name, status, score, deaths, latency and its CSS is laid ' +
-          'out for five',
-      ],
-      note,
-    );
+    return verdict([
+      `stat declares ${columns.length} columns, but ClientPlugin.styles ` +
+        `gives no width to column(s) ${missing.join(', ')}: the engine CSS ` +
+        `lays out ${ENGINE_COLUMNS} (#stat …:nth-child(1)…(${ENGINE_COLUMNS})), ` +
+        'so the rest are rendered with no width of their own',
+    ]);
   },
 };

package/src/host/PortMachine.js

@@ -7,7 +7,7 @@
 
 import wsports from '../config/wsports.js';
 import closeCodes from '../config/closeCodes.js';
-import { validateAuth } from '../lib/validators.js';
+import { resolveValidator, validateAuth } from '../lib/validators.js';
 
 // PC (client ports): порты получения данных от клиента
 const PC_CONFIG_READY = wsports.client.CONFIG_READY;
@@ -49,6 +49,22 @@ export default class PortMachine {
       ...(authSchema.params ?? []),
     ];
 
+    // Правило C10 говорит это статически, но контракт-чекер запускают не
+    // все: нерезолвнутое имя валидатора означает поле, которое не проверяет
+    // никто (validateAuth пропускает его молча — для клиента это норма)
+    for (const { name, options } of this._authParams) {
+      if (
+        options?.validator &&
+        !resolveValidator(options.validator, authSchema.validators)
+      ) {
+        console.error(
+          `PortMachine: authSchema param "${name}" names validator ` +
+            `"${options.validator}", which authSchema.validators does not ` +
+            'provide — the field is checked by nobody',
+        );
+      }
+    }
+
     // состояние подключений: socketId → { gameId, methods, enabled }
     this._clients = new Map();
   }

package/src/lib/formOptions.js

@@ -0,0 +1,16 @@
+// Разбор списка вариантов дескриптора поля — общий для формы
+// (client/lib/formBuilder.js) и для авторитетной проверки
+// (lib/validators.js): вариант пишут либо строкой, либо { value, label }.
+// Живёт в lib по той же причине, что и formPattern.js: хосту нельзя
+// импортировать слой клиента, а разъехаться эти два разбора не должны —
+// форма отдаёт ровно то, что хост потом сверяет со списком.
+
+/**
+ * @param {Array} [list] - Список вариантов из дескриптора (options/source).
+ * @returns {Array<{value: *, label: string}>} Нормализованные варианты.
+ */
+export function normalizeOptions(list) {
+  return (list || []).map(opt =>
+    opt !== null && typeof opt === 'object' ? opt : { value: opt, label: String(opt) },
+  );
+}

package/src/lib/formPattern.js

@@ -0,0 +1,16 @@
+// Якорная форма паттерна поля формы — общая для клиента и контракт-чекера.
+//
+// Браузер применяет атрибут `pattern` ко ВСЕЙ строке, оборачивая паттерн в
+// `^(?:…)$` сам. Движок повторяет это за ним (formBuilder.validateField), а
+// правило B5 обязано компилировать ровно ту же форму: иначе «компилируется»
+// у чекера и у клиента разъедется молча, и правило начнёт обещать то, чего
+// не проверяет.
+
+/**
+ * @param {string} regExp - Паттерн из дескриптора поля (строка, не RegExp).
+ * @returns {RegExp} Скомпилированный паттерн, привязанный ко всей строке.
+ * @throws {SyntaxError} Если паттерн не компилируется.
+ */
+export function anchorPattern(regExp) {
+  return new RegExp(`^(?:${regExp})$`);
+}