vimp-engine 0.27.0 → 0.28.0

package/package.json

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

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

@@ -29,6 +29,7 @@ export default class GamesCtrl {
     vp.on('submit', 'submit', this);
     vp.on('update-version', 'updateVersion', this);
     vp.on('delete', 'remove', this);
+    vp.on('restore', 'restore', this);
     vp.on('filter', 'filter', this);
     vp.on('stage', 'stage', this);
     vp.on('approve', 'approve', this);
@@ -81,6 +82,10 @@ export default class GamesCtrl {
     this._model.remove(id, scope);
   }
 
+  restore({ id }) {
+    this._model.restore(id);
+  }
+
   filter(status) {
     this._model.setFilter(status);
   }

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

@@ -156,7 +156,11 @@ export default class GamesModel {
   }
 
   // удаление игры. scope решает, чей список перечитать: карточка автора и
-  // очередь модерации показывают разные проекции одной строки
+  // очередь модерации показывают разные проекции одной строки.
+  //
+  // Удаление обратимо: игра уходит в графу Deleted очереди модерации и
+  // возвращается оттуда `restore` — подтверждения второй кнопкой оно
+  // поэтому не требует
   async remove(id, scope = 'mine') {
     const { ok, json } = await this._request(this._config.urls.remove(id), {
       method: 'DELETE',
@@ -169,12 +173,34 @@ export default class GamesModel {
 
     if (scope === 'admin') {
       await this.loadAdmin();
+
+      // после loadAdmin, а не до: renderAdmin чистит строку панели (см.
+      // moderate). Снятая с раздачи последняя игра оставляет лобби без
+      // каталога, и сказать об этом обязан тот же тик
+      if (json?.warning) {
+        this.publisher.emit('warning', { scope: 'admin', code: json.warning });
+      }
+
       return;
     }
 
     await this.loadMine();
   }
 
+  // возврат мягко удалённой игры из графы Deleted (только админ)
+  async restore(id) {
+    const { ok, json } = await this._request(this._config.urls.restore(id), {
+      method: 'POST',
+    });
+
+    if (!ok) {
+      this._fail('admin', json);
+      return;
+    }
+
+    await this.loadAdmin();
+  }
+
   // «Test»: мастер качает версию и кладёт её в каталог не раздаваемой.
   // Манифест едет наружу — по нему вкладка админа поднимает комнату
   async stage(id, version) {
@@ -218,9 +244,17 @@ export default class GamesModel {
     await this.moderate(id, { authorNick: nick === '' ? null : nick });
   }
 
+  // графа, в которой стоит игра. Мягко удалённая уходит в свою графу
+  // ЦЕЛИКОМ, сохраняя прежний статус: восстановление возвращает её туда,
+  // где она была, а показывать её ещё и в Pending/Published значило бы
+  // предлагать модерировать то, чего у игроков уже нет
+  _bucketOf(game) {
+    return game.deletedAt ? 'deleted' : game.status;
+  }
+
   _emitAdmin() {
     this.publisher.emit('admin-changed', {
-      games: this._all.filter(game => game.status === this._filter),
+      games: this._all.filter(game => this._bucketOf(game) === this._filter),
       filter: this._filter,
       versions: this._versions,
     });

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

@@ -288,8 +288,10 @@ export default class GamesView {
       item.appendChild(version);
       item.appendChild(send);
       item.appendChild(
-        this._deleteButton(() =>
-          this.publisher.emit('delete', { id: game.id, scope: 'mine' }),
+        this._button(
+          'Delete',
+          () => this.publisher.emit('delete', { id: game.id, scope: 'mine' }),
+          'games-delete-btn',
         ),
       );
       this._mineList.appendChild(item);
@@ -337,6 +339,21 @@ export default class GamesView {
         'games-item-title',
       ),
     );
+
+    // Мягко удалённая игра модерации не подлежит: у игроков её уже нет, а
+    // решение по ней приняли. Из всей карточки ей нужны срок полного
+    // удаления и возврат — остальные кнопки правили бы то, чего не видно
+    if (game.deletedAt) {
+      item.appendChild(this._line(`Author: ${game.authorNick ?? game.authorUserId ?? '—'}`));
+      this._appendRepo(item, game.repoUrl);
+      item.appendChild(this._line(this._statusLine(game), 'games-purge-line'));
+      item.appendChild(
+        this._button('Restore', () => this.publisher.emit('restore', { id: game.id })),
+      );
+
+      return item;
+    }
+
     // у игр, засеянных миграцией, автора нет вовсе: без запасного прочерка
     // в строке печаталось бы литеральное "null"
     item.appendChild(this._line(`Author: ${game.authorNick ?? game.authorUserId ?? '—'}`));
@@ -408,7 +425,11 @@ export default class GamesView {
       ),
     );
     item.appendChild(
-      this._deleteButton(() => this.publisher.emit('delete', { id: game.id, scope: 'admin' })),
+      this._button(
+        'Delete',
+        () => this.publisher.emit('delete', { id: game.id, scope: 'admin' }),
+        'games-delete-btn',
+      ),
     );
 
     return item;
@@ -438,6 +459,16 @@ export default class GamesView {
     const status = STATUS_TITLES[game.status] ?? game.status;
     const date = game.createdAt ? new Date(game.createdAt).toLocaleDateString() : '';
 
+    // у удалённой игры прежний статус — след того, куда её вернёт Restore, —
+    // а вести отсчёт человеку нужно от срока полного удаления: только он
+    // ограничен во времени
+    if (game.deletedAt) {
+      const deleted = new Date(game.deletedAt).toLocaleDateString();
+      const purge = game.purgeAt ? new Date(game.purgeAt).toLocaleDateString() : '—';
+
+      return `Deleted ${deleted} (was ${status}); removed for good on ${purge}`;
+    }
+
     return date ? `Status: ${status} (submitted ${date})` : `Status: ${status}`;
   }
 
@@ -459,34 +490,17 @@ export default class GamesView {
     });
   }
 
-  // Удаление необратимо и уносит рейтинги игры, поэтому кнопка требует
-  // второго нажатия: первое переводит её в «Confirm delete», второе публикует
-  // событие. Повторная отрисовка списка возвращает её в исходное состояние —
-  // незавершённое подтверждение не переживает перерисовку
-  _deleteButton(onConfirm) {
-    const btn = this._button('Delete', () => {
-      if (btn.dataset.armed !== '1') {
-        btn.dataset.armed = '1';
-        btn.value = 'Confirm delete';
-        btn.classList.add('games-delete-armed');
-        return;
-      }
-
-      onConfirm();
-    });
-
-    btn.className = 'games-delete-btn';
-
-    return btn;
-  }
-
-  _button(value, onclick) {
+  _button(value, onclick, className) {
     const btn = document.createElement('input');
 
     btn.type = 'button';
     btn.value = value;
     btn.onclick = onclick;
 
+    if (className) {
+      btn.className = className;
+    }
+
     return btn;
   }
 

package/src/client/style.css

@@ -217,9 +217,9 @@ body.hide-cursor {
   background: rgba(25, 25, 25, 0.9);
   border-radius: 5px;
   border: 3px solid #fff;
-  overflow: hidden;
+  overflow: auto;
   width: 90%;
-  max-width: 550px;
+  max-width: 500px;
   font-size: 10px;
   line-height: 12px;
   color: #fff;
@@ -438,7 +438,7 @@ body.hide-cursor {
   background-color: #333;
   border: 1px solid #eee;
   padding: 10px;
-  margin: 10px auto 5px;
+  margin: 5px auto 0;
   text-shadow: 1px 1px 1px #000;
   font-size: 12px;
   line-height: 15px;
@@ -549,15 +549,11 @@ body.hide-cursor {
   padding: 0 10px;
 }
 
-/* ширина — контентная область правой панели лобби (её ширина минус
-   собственные поля .lobby-browser-inner), под которой футер и стоит;
-   max-width, а не width: в одну колонку он обязан ужиматься по экрану */
 #lobby-footer {
-  width: 100%;
-  max-width: calc(var(--lobby-browser-width) - 2 * var(--lobby-panel-pad));
+  width: calc(var(--lobby-width) + var(--lobby-panel-pad));
   font-size: 10px;
   padding: 0;
-  margin: 0 var(--lobby-panel-pad) 0 0;
+  margin: 0 calc(var(--lobby-panel-pad) / 2) 0 0;
 }
 
 /* три равные ячейки: середину держит не space-between, а сама равность —
@@ -670,11 +666,7 @@ body.hide-cursor {
 /*lobby*/
 
 #lobby {
-  /* ширины панелей и их внутренние поля: по ним же считается ширина футера
-     (секция auth выше), поэтому числа живут в одном месте — иначе смена
-     ширины любой панели молча разъезжает вёрстку */
-  --lobby-setup-width: 410px;
-  --lobby-browser-width: 430px;
+  --lobby-width: 370px;
   --lobby-panel-pad: 10px;
 
   background-color: #222;
@@ -832,12 +824,12 @@ body.hide-cursor {
    высоты более высокой панели */
 .lobby-grid {
   display: flex;
-  gap: 10px;
+  gap: 5px;
   justify-content: center;
 }
 
 #lobby-setup-panel {
-  width: var(--lobby-setup-width);
+  width: var(--lobby-width);
 }
 
 /* высоту правой .card задаёт форма слева: обе панели тянутся до высоты
@@ -846,9 +838,8 @@ body.hide-cursor {
    вынесен в абсолютный слой .lobby-browser-inner (его высота не участвует в
    расчёте высоты строки) */
 #lobby-browser-panel {
-  width: var(--lobby-browser-width);
+  width: var(--lobby-width);
   position: relative;
-  padding: 0;
 }
 
 .lobby-browser-inner {
@@ -1147,16 +1138,16 @@ body.hide-cursor {
   color: #eee;
 }
 
-/* удаление необратимо: взведённое подтверждение обязано отличаться от
-   первого нажатия не только подписью */
+/* удаление обратимо (игра уходит в графу Deleted на 30 суток), но выделяется
+   среди соседних кнопок: рядом стоит правка версии, а не действие того же веса */
 .games-delete-btn {
   color: #d66;
 }
 
-.games-delete-armed {
-  background: #802020;
-  border-color: #d66;
-  color: #fee;
+/* срок полного удаления — единственное, что в карточке удалённой игры
+   ограничено во времени */
+.games-purge-line {
+  color: #d66;
 }
 
 /* кнопки входа в панель стоят рядом с Sign out в бейдже пользователя */

package/src/config/lobby.js

@@ -218,19 +218,26 @@ export default {
       staged: '/admin/games/manifest.json',
       // «Test»: скачать версию и положить её в каталог не раздаваемой
       stage: id => `/admin/games/${encodeURIComponent(id)}/stage`,
+      // возврат мягко удалённой игры из графы Deleted (только админ)
+      restore: id => `/admin/games/${encodeURIComponent(id)}/restore`,
       // решение модератора
       moderate: id => `/admin/games/${encodeURIComponent(id)}`,
       // что опубликовано в npm — индикатор «есть версия новее»
       versions: id => `/admin/games/${encodeURIComponent(id)}/versions`,
     },
 
-    // фильтры очереди модерации: id статуса реестра -> подпись кнопки.
-    // Значения обязаны совпадать со статусами auth-сервиса
+    // фильтры («графы») очереди модерации: id -> подпись кнопки. Первые
+    // четыре — статусы реестра, и значения обязаны совпадать со статусами
+    // auth-сервиса. Пятая графа стоит особняком: 'deleted' статусом НЕ
+    // является (мягко удалённая игра сохраняет свой прежний статус, иначе
+    // восстанавливать было бы не во что) — под неё попадает всё с
+    // непустым deletedAt, см. GamesModel._bucketOf
     statuses: [
       { id: 'pending', title: 'Pending' },
       { id: 'approved', title: 'Published' },
       { id: 'rejected', title: 'Rejected' },
       { id: 'disabled', title: 'Disabled' },
+      { id: 'deleted', title: 'Deleted' },
     ],
     defaultStatus: 'pending',