vimp-engine 0.25.0 → 0.26.0

package/package.json

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

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

@@ -162,6 +162,12 @@ export default class GamesModel {
     }
 
     await this.loadAdmin();
+
+    // после loadAdmin, а не до: renderAdmin чистит строку отказа панели, и
+    // предупреждение, показанное раньше, стёрлось бы тем же тиком
+    if (json?.warning) {
+      this.publisher.emit('warning', { scope: 'admin', code: json.warning });
+    }
   }
 
   _emitAdmin() {

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

@@ -28,6 +28,13 @@ const ERROR_MESSAGES = {
   authServiceUnavailable: 'Registry service unavailable',
 };
 
+// предупреждения: решение принято, отказа не было, но состояние платформы
+// стоит назвать вслух
+const WARNING_MESSAGES = {
+  catalogEmpty:
+    'No published games left — the lobby cannot create rooms until one is published',
+};
+
 const STATUS_TITLES = {
   pending: 'in review',
   approved: 'published',
@@ -71,10 +78,20 @@ export default class GamesView {
     this._filters = document.getElementById(elems.filtersId);
 
     this._fields = new Map(
-      Object.entries(elems.fieldIds).map(([name, id]) => [
-        name,
-        document.getElementById(id),
-      ]),
+      Object.entries(elems.fieldIds).map(([name, id]) => {
+        const field = document.getElementById(id);
+
+        // карта строится по конфигу, а используется из обработчиков событий:
+        // разъехавшийся с games.pug id дал бы здесь null, а упал бы позже —
+        // безымянным TypeError внутри clearForm или _readForm. Остальная
+        // разметка панели проверяется тем же способом, только неявно: первое
+        // же обращение к отсутствующему элементу бросает в конструкторе
+        if (!field) {
+          throw new Error(`GamesView: no element "#${id}" for field "${name}"`);
+        }
+
+        return [name, field];
+      }),
     );
 
     this.publisher = new Publisher();
@@ -96,6 +113,7 @@ export default class GamesView {
     mp.on('mine-changed', 'renderMine', this);
     mp.on('admin-changed', 'renderAdmin', this);
     mp.on('error', 'renderError', this);
+    mp.on('warning', 'renderWarning', this);
   }
 
   // кнопку модерации показывает не сама панель, а роль вызывающего
@@ -180,6 +198,12 @@ export default class GamesView {
     });
   }
 
+  renderWarning({ scope, code }) {
+    const container = scope === 'admin' ? this._adminError : this._submitError;
+
+    container.textContent = WARNING_MESSAGES[code] ?? code;
+  }
+
   renderError({ scope, errors }) {
     const container = scope === 'admin' ? this._adminError : this._submitError;
 

package/src/client/lib/catalogState.js

@@ -0,0 +1,44 @@
+// Каталог игр платформы может быть ПУСТ, и это законное состояние лобби, а не
+// отказ загрузки: игры живут в реестре auth-сервиса, модератор вправе снять с
+// раздачи последнюю, а на первом развёртывании не одобрена ещё ни одна.
+//
+// Лобби в этом состоянии обязано жить целиком: вход, бейдж пользователя, «My
+// games» и «Moderation» от игры не зависят — и только через них каталог
+// возвращается к жизни. Раньше пустой каталог бросал из бутстрапа и стирал
+// разметку страницы: модератор, отключивший последнюю игру, запирал сам себя
+// и вернуть её мог только запросом мимо интерфейса.
+//
+// Вынесено из main.js (бутстрап, в happy-dom не поднимается) отдельным
+// модулем, чтобы обе ветки проверялись юнит-тестом.
+
+/**
+ * Приводит лобби в соответствие с активной игрой.
+ * @param {Object} [manifest] - Манифест активной игры; `undefined` — каталог
+ *   платформы пуст.
+ * @param {Object} deps - Проводка лобби.
+ * @param {HTMLElement} [deps.hostBtn] - Кнопка «Create server».
+ * @param {string} deps.emptyText - Строка отказа для пустого каталога.
+ * @param {Function} deps.bindGame - Форма комнаты и Leaderboard активной игры.
+ * @param {Function} deps.showError - Показать строку отказа лобби.
+ * @param {Function} deps.clearError - Снять строку отказа лобби.
+ * @returns {void}
+ */
+export function applyCatalogState(
+  manifest,
+  { hostBtn, emptyText, bindGame, showError, clearError },
+) {
+  if (manifest) {
+    bindGame(manifest);
+    clearError();
+  } else {
+    // форму комнаты и Leaderboard не трогаем вовсе: без игры у первой нет
+    // схемы полей, а второй — игры, за рейтингом которой идти
+    showError(emptyText);
+  }
+
+  if (hostBtn) {
+    hostBtn.disabled = !manifest;
+  }
+}
+
+export default applyCatalogState;

package/src/client/main.js

@@ -35,8 +35,13 @@ import { normalizeAuthParams } from './lib/authParams.js';
 import { renderProjectLink } from './lib/footerLink.js';
 import { createGameActivator } from './lib/gameActivator.js';
 import createAutostart from './lib/autostart.js';
+import { applyCatalogState } from './lib/catalogState.js';
 import { getBootConfig, resolveBootConfig } from './boot.js';
-import { ensureGameShell, ensureCanvas } from './views/gameShell.js';
+import {
+  ensureGameShell,
+  ensureCanvas,
+  showBootFailure,
+} from './views/gameShell.js';
 import { createContextTracker } from './lib/contextTracker.js';
 import { createLocalPlayer } from './lib/localPlayer.js';
 import { createAccolades } from './lib/accolades.js';
@@ -102,6 +107,10 @@ let activeGameManifest;
 let clientPlugin;
 let gamesManifest;
 
+// почему активной игры нет, если каталог непуст: строка отказа лобби вместо
+// общего «игр пока не опубликовано» (см. pickActiveGame)
+let catalogProblem = null;
+
 // узел игрового CSS: при переключении игры его текст заменяется, иначе стили
 // двух игр жили бы в head одновременно и конфликтовали селекторами
 let gameStyleNode = null;
@@ -157,20 +166,40 @@ try {
       throw gamesManifest;
     }
 
-    // недоступная игра активной быть не может (lib/pickActiveGame.js)
-    activeGameManifest = pickActiveGame(gamesManifest, boot.gameId);
+    try {
+      // недоступная игра активной быть не может (lib/pickActiveGame.js)
+      activeGameManifest = pickActiveGame(gamesManifest, boot.gameId);
+    } catch (e) {
+      // «каталог непустой, но играбельного в нём нет» (движок обновили, все
+      // опубликованные игры просят возможность, которой в нём уже нет) — для
+      // лобби это то же самое, что пустой каталог: играть не во что, но
+      // модерация и заявка на месте, и только они это чинят. Терять их из-за
+      // такого нельзя, поэтому причина едет строкой отказа лобби, а не
+      // терминальным оверлеем. В solo/dedicated отказ остаётся отказом
+      if (!isLobbyMode) {
+        throw e;
+      }
+
+      catalogProblem = e.message;
+    }
   }
 
-  if (!activeGameManifest) {
+  // пустой каталог — состояние лобби, а не отказ загрузки: модератор вправе
+  // отключить последнюю игру, реестр вправе ещё ничего не одобрить, и вернуть
+  // каталог к жизни можно только из панели, которая живёт в этой же вкладке.
+  // В solo/dedicated игры нет — это отказ: там весь смысл вкладки в матче
+  if (!activeGameManifest && !isLobbyMode) {
     throw new Error('master has no games in its catalog');
   }
 
-  bindActiveGame(
-    activeGameManifest,
-    boot.clientPlugin ?? (await loadClientPlugin(activeGameManifest)),
-  );
+  if (activeGameManifest) {
+    bindActiveGame(
+      activeGameManifest,
+      boot.clientPlugin ?? (await loadClientPlugin(activeGameManifest)),
+    );
+  }
 } catch (e) {
-  document.body.textContent = `Failed to load the game: ${e.message}`;
+  showBootFailure(`Failed to load the game: ${e.message}`, gameContainer);
   throw e;
 }
 
@@ -1990,7 +2019,12 @@ async function selectActiveGame(gameId) {
   // сверка идёт с самим манифестом, а не с его id: каталог вкладки может
   // пополниться застейдженной версией той же игры (registerGameManifest), и
   // сравнение по id оставило бы активной одобренную версию
-  if (gameId === activeGameManifest.id && gamesById.get(gameId) === activeGameManifest) {
+  // activeGameManifest?. — каталог платформы может быть пуст: активной игры
+  // у вкладки тогда нет вовсе, и первая же активация обязана состояться
+  if (
+    gameId === activeGameManifest?.id &&
+    gamesById.get(gameId) === activeGameManifest
+  ) {
     return true;
   }
 
@@ -2011,6 +2045,9 @@ async function selectActiveGame(gameId) {
   }
 }
 
+// очередь активации черновиков (см. подписку на 'staged' в initLobby)
+let stagedActivation = Promise.resolve();
+
 // id игр, чей манифест в каталоге вкладки подменён застейдженной версией
 // (master-game-registry, этап 4): в селекторе такая игра помечается, чтобы
 // админ видел, что поднимет комнату НЕ на одобренной версии
@@ -2070,8 +2107,29 @@ function populateGameSelect() {
     gameSelect.appendChild(option);
   });
 
-  gameSelect.value =
-    selected && gamesById.has(selected) ? selected : activeGameManifest.id;
+  if (selected && gamesById.has(selected)) {
+    gameSelect.value = selected;
+  } else if (activeGameManifest) {
+    gameSelect.value = activeGameManifest.id;
+  }
+  // без активной игры выбор не трогаем: каталог был пуст, и селектор только
+  // что получил первую строку — она и стоит выбранной по умолчанию.
+  // Присваивание `''` вместо этого сняло бы выбор совсем (selectedIndex = -1)
+}
+
+// форма комнаты, Leaderboard и «Create server» по активной игре; пустой
+// каталог — вторая законная ветка (client/lib/catalogState.js)
+function syncCatalogState() {
+  applyCatalogState(activeGameManifest, {
+    hostBtn: document.getElementById(lobbyConfig.elems.hostBtnId),
+    emptyText: catalogProblem ?? lobbyConfig.create.emptyCatalogText,
+    bindGame: manifest => {
+      populateRoomForm(manifest);
+      lobby.gameChanged(manifest.id, manifest.title);
+    },
+    showError: showLobbyError,
+    clearError: clearLobbyError,
+  });
 }
 
 // поднимает лобби после welcome от мастера (iceServers уже получены);
@@ -2117,7 +2175,15 @@ function initLobby() {
   // хостов старше 6.4 — тогда заходим на активной игре, как раньше: неизвестная
   // версия не повод запретить вход
   lobbyModel.publisher.on('join', async ({ hostId, gameId }) => {
-    if (gameId && gamesById.has(gameId) && !(await selectActiveGame(gameId))) {
+    if (gameId && gamesById.has(gameId)) {
+      if (!(await selectActiveGame(gameId))) {
+        return;
+      }
+    } else if (!activeGameManifest) {
+      // комната пережила свою игру: её сняли с раздачи, пока комната шла, и
+      // каталог вкладки пуст — заходить не на чем
+      showLobbyError('This room runs a game that is no longer published');
+
       return;
     }
 
@@ -2181,12 +2247,39 @@ function initLobby() {
   games.publisher.on('staged', ({ manifest }) => {
     registerGameManifest(manifest);
     gamesView.hide();
+
+    // каталог был пуст (первое развёртывание платформы либо снятая с раздачи
+    // последняя игра): черновик — единственная игра вкладки, и создание
+    // комнаты включается только после того, как её плагин привязан
+    if (activeGameManifest) {
+      return;
+    }
+
+    // строго по очереди: loadStaged() восстанавливает все черновики этого
+    // мастера синхронным циклом, и без очереди два черновика при пустом
+    // каталоге ушли бы активироваться одновременно — активной осталась бы
+    // одна игра, а форма комнаты собралась бы по другой
+    stagedActivation = stagedActivation.then(async () => {
+      if (activeGameManifest) {
+        return; // предыдущий черновик уже стал активным
+      }
+
+      // только на успехе: отказ активации уже написал в ту же строку СВОЮ
+      // причину («Failed to load X: …»), и общее «игр пока не опубликовано»
+      // затёрло бы её — админ увидел бы не то, что случилось
+      if (await selectActiveGame(manifest.id)) {
+        syncCatalogState();
+      }
+    }).catch(e => {
+      // отказ не должен заклинить очередь: следующий черновик обязан
+      // получить свой шанс стать активной игрой
+      console.error('[game] staged activation failed:', e);
+    });
   });
 
   // создание комнаты в этой же вкладке (хост-игрок через loopback)
   populateGameSelect();
-  populateRoomForm(activeGameManifest);
-  lobby.gameChanged(activeGameManifest.id, activeGameManifest.title);
+  syncCatalogState();
 
   // селектор игр: меняет форму создания комнаты и Leaderboard сразу
   // (синхронно, без сети); сама игра активируется уже по клику

package/src/client/views/gameShell.js

@@ -135,6 +135,37 @@ export function ensureCanvas(canvasId, size, container = document.body) {
   return canvas;
 }
 
+/**
+ * Терминальный отказ загрузки — поверх страницы, а не вместо неё.
+ *
+ * Раньше здесь стояло `document.body.textContent = …`: разметка страницы
+ * стиралась целиком, вместе с формой входа и панелью реестра игр, — то есть
+ * любая ошибка загрузки игры уносила с собой и путь к её исправлению.
+ * `#tech-informer` для того и есть: полноэкранный слой поверх (z-index 9),
+ * под которым разметка остаётся целой и рабочей после перезагрузки вкладки.
+ *
+ * Слой ТЕРМИНАЛЬНЫЙ и снять его нечем — он кроет и `#lobby-auth`, и
+ * `#games-panel` (обе z-index 8). Поэтому сюда попадает только то, что и
+ * правда терминально: отсутствие каталога само по себе не терминально, и в
+ * lobby-режиме оно идёт строкой отказа лобби (client/lib/catalogState.js).
+ * @param {string} text - Сообщение игроку.
+ * @param {HTMLElement} [container] - Точка монтирования каркаса.
+ * @returns {void}
+ */
+export function showBootFailure(text, container = document.body) {
+  let elem = document.getElementById('tech-informer');
+
+  if (!elem) {
+    // каркаса нет (отказ случился до ensureGameShell): добавляем узел ПЕРЕД
+    // содержимым, а не вместо него
+    elem = createNode({ tag: 'div', id: 'tech-informer' });
+    container.prepend(elem);
+  }
+
+  elem.textContent = text;
+  elem.style.display = 'block';
+}
+
 /**
  * Набор id, которые собирает каркас. Нужен тесту паритета с pug.
  * @returns {Set<string>}

package/src/config/env.js

@@ -26,12 +26,6 @@ export function applyMasterEnv(config, env = process.env) {
     config.set('master:security:authServiceUrl', env.VIMP_AUTH_SERVICE_URL);
   }
 
-  // список игр-плагинов мастера (Этап A2), по образцу остальных *_MATRIX
-  // env-переопределений — JSON-массив {id, package}
-  if (env.GAMES_MATRIX) {
-    config.set('master:games', JSON.parse(env.GAMES_MATRIX));
-  }
-
   // корень хранилища игровых пакетов (направление master-game-registry) —
   // в проде это смонтированный том, переживающий пересоздание контейнера
   if (env.VIMP_GAMES_DIR) {

package/src/config/lobby.js

@@ -269,6 +269,12 @@ export default {
   create: {
     defaultName: 'My Server',
 
+    // каталог платформы пуст: реестр ещё ничего не одобрил либо модератор
+    // снял с раздачи последнюю игру. Комнату создавать не на чем, но лобби
+    // живо — и текст называет то единственное, что выводит его из этого
+    // состояния
+    emptyCatalogText: 'No games are published yet — see “My games”',
+
     // период heartbeat/актуализации комнаты у мастера (мс); должен быть
     // меньше master.host.heartbeatTimeout (30 c), иначе комнату выметет
     heartbeatInterval: 10000,

package/src/config/master.js

@@ -23,11 +23,11 @@ export default {
   // auth-сервиса, откуда мастер скачивает одобренные пакеты сам
   // (master/GameSync.js). Движок игр не знает и зависимостями их не тянет.
   //
-  // Массив и переменная окружения GAMES_MATRIX (JSON) остаются
-  // переопределением для двух случаев: локальная разработка (каталог вдобавок
-  // дополняется собранными пакетами @vimp-games/* из node_modules,
-  // master/localGames.js — прилинкованная игра важнее реестра) и self-hosted
-  // мастер без реестра. `package` — имя npm-пакета игры, резолвится через
+  // Массив остаётся двум потребителям, и env-переопределения у него нет:
+  // локальной разработке (каталог наполняют собранные пакеты @vimp-games/*
+  // из node_modules, master/localGames.js — прилинкованная игра важнее
+  // реестра) и dedicated-серверу, который берёт отсюда игру, названную
+  // VIMP_DEDICATED_GAME. `package` — имя npm-пакета игры, резолвится через
   // node_modules; версию задаёт сама установленная зависимость.
   // maxGameScore (snakes-v3 этап 3.3, необязательное поле рядом с
   // id/package) — потолок результата ОДНОЙ игры для этой игры; мастер