vimp-engine 0.24.1 → 0.24.2

package/package.json

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

package/src/devtools/contract/rules/b2-engine-api.js

@@ -123,7 +123,7 @@ export default {
 // исключена: пакет, собранный до появления поля, обязан оставаться
 // валидным (И1/И2)
 function halfViolations(ctx, manifestRequires) {
-  const wanted = new Set(manifestRequires);
+  const wanted = new Set(resolveNames(manifestRequires));
   const violations = [];
   const halves = [
     ['HostPlugin', ctx.hostPlugin?.requires],
@@ -139,7 +139,7 @@ function halfViolations(ctx, manifestRequires) {
       continue;
     }
 
-    const extra = value.filter(name => !wanted.has(name));
+    const extra = resolveNames(value).filter(name => !wanted.has(name));
 
     if (extra.length) {
       violations.push(
@@ -158,10 +158,12 @@ function halfViolations(ctx, manifestRequires) {
     return violations;
   }
 
-  const declared = halves
-    .map(([, value]) => value)
-    .filter(Array.isArray)
-    .flat();
+  const declared = resolveNames(
+    halves
+      .map(([, value]) => value)
+      .filter(Array.isArray)
+      .flat(),
+  );
 
   const missing = [...wanted].filter(name => !declared.includes(name));
 
@@ -176,6 +178,19 @@ function halfViolations(ctx, manifestRequires) {
   return violations;
 }
 
+// Имена сравниваются РЕЗОЛВНУТЫМИ: реестр возможностей append-only, и
+// выведенное алиасом имя движок принимает вечно наравне с активным.
+// Манифест с новым именем и половина, оставшаяся на выведенном, просят одно
+// и то же — считать это расхождением значило бы ругаться на игру за то, что
+// движок переименовал возможность (тот самый отказ по возрасту, который план
+// снимал). Неизвестное имя резолва не имеет и сравнивается как есть: о нём
+// говорит проверка по реестру выше
+function resolveNames(names) {
+  return (Array.isArray(names) ? names : []).map(
+    name => ENGINE_CAPABILITIES.resolve(name) ?? name,
+  );
+}
+
 // то же, что делают B5 и C10: выведенное имя — не отказ, а предупреждение.
 // Отвергать за него значило бы отвергать игру за возраст (И1)
 function finish(violations, retired) {

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

@@ -2,6 +2,7 @@ import { ERROR, WARN, skip, verdict } from '../result.js';
 import { resolveValidator } from '../../../lib/validators.js';
 import {
   formControls,
+  resolveDescriptor,
   ACTIVE_FORM_CONTROLS,
 } from '../../../lib/formControls.js';
 
@@ -13,6 +14,10 @@ import {
 // которого нет в authSchema.validators (поле не проверяется никем).
 const NICKNAME = /^(name|nick|nickname|player|playername|login|username)$/i;
 
+// те же имена, что у validateAuth и билдера формы: список вариантов есть
+// только у них, и только им движок резолвит `source`
+const OPTION_CONTROLS = ['select', 'radio'];
+
 export default {
   id: 'C10',
   name: 'authSchema',
@@ -76,8 +81,18 @@ export default {
       // значит список вариантов у такого поля пуст всегда: игрок видит
       // 'no options available' и войти не может, а хост отвергает любое
       // значение как не-вариант. Это только для authSchema; в roomForm
-      // (правило B5) source штатный — там движок отдаёт каталог карт
-      if (field.options?.source !== undefined) {
+      // (правило B5) source штатный — там движок отдаёт каталог карт.
+      //
+      // Только для контролов со списком: у text/checkbox `source` не читает
+      // никто (formBuilder.resolveOptions зовут лишь buildSelect/buildRadio),
+      // и такое поле работает — отвергать за лишний ключ значило бы отвергать
+      // игру, которая логинит игроков как ни в чём не бывало
+      // по резолвнутому контролу, а не по написанному: 'segmented' — та же
+      // группа вариантов, что radio, и source в ней так же не резолвится
+      if (
+        OPTION_CONTROLS.includes(resolveDescriptor(field.options)?.control) &&
+        field.options?.source !== undefined
+      ) {
         violations.push(
           `authSchema param "${field.name}" declares options.source ` +
             `"${field.options.source}" — the auth form is built without ` +

package/src/lib/loadGamePackage.js

@@ -2,6 +2,7 @@ import { access, readFile } from 'node:fs/promises';
 import path from 'node:path';
 import { pathToFileURL } from 'node:url';
 import { checkPluginCompatibility } from './gamePlugin.js';
+import { ENGINE_CAPABILITIES } from './capabilities.js';
 
 // Загрузка пакета игры в Node по его собранному dist/ (Этап 4 плана
 // standalone-sdk). В браузере плагин грузится по URL из GameManifest мастера;
@@ -107,25 +108,28 @@ function assertPluginMatchesManifest(manifest, plugins) {
     }
   }
 
-  assertRequiresMatchManifest(manifest, plugins);
+  warnOnRequiresMismatch(manifest, plugins);
 }
 
 // `requires` пишут ТРИ независимых места одного пакета игры: скрипт сборки
 // манифеста и обе половины плагина (последние — ради standalone SDK, у
 // которого манифеста нет вовсе). Разъехавшись, они дают игру, которая в
-// лобби честно отвергается, а в solo-режиме тихо недоигрывает — то есть
-// именно тот молчаливый режим, ради отказа от которого возможности и
-// заводились. Проверка — того же класса, что сверка engineApi выше: это
-// рассинхрон сборки ВНУТРИ пакета, а не отказ по возрасту (И4).
+// лобби честно отвергается, а в solo-режиме тихо недоигрывает.
+//
+// Здесь это ПРЕДУПРЕЖДЕНИЕ, а не отказ, и разница принципиальная. Node-путь
+// (dedicated, vimp-sim, inline host) читает авторитетный манифест, и его
+// совместимость уже проверена выше (checkPluginCompatibility): рассинхрон с
+// половинами ничего здесь не ломает. Бросить значило бы отвергнуть пакет,
+// который грузился раньше, — прямое нарушение И4 плана
+// plugin-forward-compat ради дефекта упаковки, который движок переживает.
+// Место отказа — правило контракта B2: чекер запускает автор игры, и он
+// узнаёт о расхождении до публикации, а не игрок вместо матча.
 //
 // Половина, вовсе НЕ объявившая поле, из сверки исключена: старый пакет,
-// собранный до его появления, обязан грузиться (И1/И2). Объявленное пустым
-// (`requires: []`) — уже утверждение «игре ничего не нужно», и оно
-// расходится с манифестом наравне с неполным списком.
-function assertRequiresMatchManifest(manifest, plugins) {
-  const wanted = new Set(
-    Array.isArray(manifest.requires) ? manifest.requires : [],
-  );
+// собранный до его появления, ничего не утверждает (И1/И2). Объявленное
+// пустым (`requires: []`) — уже утверждение «игре ничего не нужно».
+function warnOnRequiresMismatch(manifest, plugins) {
+  const wanted = new Set(resolveNames(manifest.requires));
 
   for (const [half, plugin] of Object.entries(plugins)) {
     if (plugin.requires === undefined || plugin.requires === null) {
@@ -133,16 +137,19 @@ function assertRequiresMatchManifest(manifest, plugins) {
     }
 
     if (!Array.isArray(plugin.requires)) {
-      throw new Error(
+      console.warn(
         `game "${manifest.id}": ${half} plugin requires must be an array ` +
-          'of capability names',
+          'of capability names — the standalone SDK reads it',
       );
+      continue;
     }
 
-    const extra = plugin.requires.filter(name => !wanted.has(name));
+    const extra = resolveNames(plugin.requires).filter(
+      name => !wanted.has(name),
+    );
 
     if (extra.length) {
-      throw new Error(
+      console.warn(
         `game "${manifest.id}": ${half} plugin requires ` +
           `${extra.join(', ')}, which manifest.requires does not list — ` +
           'stale dist/ (the manifest is what the lobby master reads)',
@@ -156,19 +163,18 @@ function assertRequiresMatchManifest(manifest, plugins) {
   // (`requires: []`) — это утверждение «ничего не нужно», и оно расходится
   // с манифестом наравне с неполным списком
   const halves = Object.values(plugins).filter(
-    plugin => plugin.requires !== undefined && plugin.requires !== null,
+    plugin => Array.isArray(plugin.requires),
   );
 
   if (halves.length === 0) {
     return;
   }
 
-  const declared = halves.flatMap(plugin => plugin.requires);
-
+  const declared = resolveNames(halves.flatMap(plugin => plugin.requires));
   const missing = [...wanted].filter(name => !declared.includes(name));
 
   if (missing.length) {
-    throw new Error(
+    console.warn(
       `game "${manifest.id}": manifest.requires names ${missing.join(', ')}, ` +
         'which neither plugin half declares — stale dist/ (the standalone ' +
         'SDK reads the halves, there is no manifest in solo mode)',
@@ -176,6 +182,19 @@ function assertRequiresMatchManifest(manifest, plugins) {
   }
 }
 
+// Сравнивать имена возможностей буквально нельзя: реестр append-only, и
+// выведенное алиасом имя движок принимает вечно наравне с активным
+// (ENGINE_CAPABILITIES.has в checkPluginCompatibility). Манифест, назвавший
+// новое имя, и половина, оставшаяся на выведенном, просят ОДНО И ТО ЖЕ —
+// объявить это расхождением значило бы ругаться на игру за то, что движок
+// переименовал возможность. Неизвестное имя резолва не имеет и сравнивается
+// как есть: о нём говорит checkPluginCompatibility, а не эта сверка
+function resolveNames(names) {
+  return (Array.isArray(names) ? names : []).map(
+    name => ENGINE_CAPABILITIES.resolve(name) ?? name,
+  );
+}
+
 function importDefault(baseDir, entry, assetsBase) {
   const file = path.resolve(baseDir, stripBase(entry, assetsBase));
 

package/src/lib/validators.js

@@ -56,15 +56,16 @@ const numericError = (options, value) => {
     return `must be <= ${options.max}`;
   }
 
-  // regExp — не дубль диапазона: сгенерированный сборкой паттерн (вроде
-  // "^([1-8])$") ловит то, что диапазон пропускает — дробное и ведущий
-  // ноль. Клиент проверяет числовое поле и тем, и другим (formBuilder.
-  // validateField), и сверяет паттерн с СЫРОЙ строкой поля, то есть со
-  // значением в единице отображения — здесь то же самое
-  if (options.regExp && !matchesPattern(options.regExp, String(shown))) {
-    return 'invalid format';
-  }
-
+  // regExp числового поля хост НЕ проверяет, и это осознанно. Клиент
+  // сверяет паттерн с сырой строкой, которую набрал игрок
+  // (formBuilder.validateField: field.getRaw()); до хоста доезжает уже
+  // число, и восстановить ту строку нечем — `String(значение)` даёт
+  // каноническую запись, а не набранную. Паттерн, принимающий
+  // неканоническую («^\d+\.\d{2}$» и ввод «1.50»), совпал бы с формой и
+  // разошёлся бы с хостом: игрок видит поле валидным, а вход не пускает, и
+  // набрать строку, годную обеим сторонам, нельзя вовсе. Блокировка входа
+  // хуже той щели, которую паттерн закрывает: диапазон уже проверен, а
+  // ведущий ноль до хоста и не доезжает (Number('08') === 8)
   return null;
 };