vimp-engine 0.35.6 → 0.36.0

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

@@ -2,6 +2,10 @@ import Publisher from '../../../lib/Publisher.js';
 
 // Singleton VoteModel
 
+// имя с '@' — голосование движка (host-migration этап 10: '@changeHost'):
+// его ведёт мастер, а не хост, и игра таких имён не объявляет
+const isEngineVote = name => typeof name === 'string' && name.startsWith('@');
+
 let voteModel;
 
 export default class VoteModel {
@@ -13,6 +17,7 @@ export default class VoteModel {
     voteModel = this;
 
     this._formatMessage = data.formatMessage;
+    this._now = data.now ?? (() => Date.now());
 
     // игра может не объявлять голосований вовсе (noSpectators): меню по 'm'
     // тогда пустое, а не падение на длине undefined
@@ -28,6 +33,12 @@ export default class VoteModel {
     this._timeOff = false; // флаг отключения времени жизни голосования
 
     this._voteName = ''; // название голосования
+    this._deadline = null; // конец голосования движка (окно — на остаток)
+    this._showing = false; // окно (голосование или меню) открыто
+
+    // голосования, ждущие закрытия открытого окна: голосование движка не
+    // затирает чужое и не затирается им (этап 10)
+    this._queue = [];
 
     this._title = null; // заголовок голосования
     this._values = []; // все значения голосования
@@ -45,7 +56,18 @@ export default class VoteModel {
     this.publisher.emit('mode', { name: 'vote', status: 'opened' });
   }
 
+  // голосование хоста по шаблону; false — встало в очередь за открытым
+  // голосованием движка
   createWithTemplate({ name, params, values }) {
+    if (
+      this._showing &&
+      this._type === 'vote' &&
+      isEngineVote(this._voteName)
+    ) {
+      this._queue.push({ template: { name, params, values } });
+      return false;
+    }
+
     const templateArr = this._templates[name];
 
     if (templateArr) {
@@ -59,14 +81,63 @@ export default class VoteModel {
 
       this.createVote(name, title, values, timeOff);
     }
+
+    return true;
+  }
+
+  /**
+   * Голосование движка (имя с '@'): окно живёт до deadline; открыто другое
+   * окно — ждёт в очереди.
+   * @param {Object} vote - { name, title, values, deadline }.
+   * @returns {boolean} окно открыто сразу.
+   */
+  createEngineVote(vote) {
+    if (this._showing || this._waitingValues) {
+      this._queue.push({ engine: { ...vote } });
+      return false;
+    }
+
+    this._startEngineVote(vote);
+
+    return true;
+  }
+
+  /**
+   * Голосование движка завершилось (итог от мастера): окно закрывается или
+   * снимается из очереди.
+   * @param {string} name
+   */
+  closeEngineVote(name) {
+    this._queue = this._queue.filter(item => item.engine?.name !== name);
+
+    if (this._showing && this._type === 'vote' && this._voteName === name) {
+      this.complete();
+    }
+  }
+
+  // голосования хоста не пережили переподключение: открытое закрывается,
+  // ждущие отбрасываются; голосование движка (его ведёт мастер) остаётся
+  removeHostVotes() {
+    this._queue = this._queue.filter(item => item.engine);
+
+    if (!(this._type === 'vote' && isEngineVote(this._voteName))) {
+      this.complete();
+    }
+  }
+
+  _startEngineVote({ name, title, values, deadline }) {
+    this.createVote(name, title, values, false, deadline);
+    this.open();
   }
 
-  // создает голосование
-  createVote(name, title, values, timeOff) {
+  // создает голосование; deadline — только у голосований движка
+  createVote(name, title, values, timeOff, deadline = null) {
     if (this._waitingValues) {
       return;
     }
 
+    this._deadline = deadline;
+
     this._type = 'vote';
     this._back = false;
     this._more = false;
@@ -91,7 +162,24 @@ export default class VoteModel {
       return;
     }
 
+    // меню открывается поверх голосования движка — оно вернётся следом
+    if (
+      this._showing &&
+      this._type === 'vote' &&
+      isEngineVote(this._voteName)
+    ) {
+      this._queue.unshift({
+        engine: {
+          name: this._voteName,
+          title: this._title,
+          values: this._values,
+          deadline: this._deadline,
+        },
+      });
+    }
+
     this._timeOff = false;
+    this._deadline = null;
 
     this._type = 'menu';
     this._back = false;
@@ -204,6 +292,7 @@ export default class VoteModel {
       currentValues = this._currentValues;
     }
 
+    this._showing = true;
     this.publisher.emit('clear', this._timerId);
 
     this.publisher.emit('vote', {
@@ -211,15 +300,44 @@ export default class VoteModel {
       list: currentValues,
       back: this._back,
       more: this._more,
-      time: this._timeOff === true ? null : this._time,
+      time: this._timeOff === true ? null : this._windowTime(),
     });
   }
 
-  // завершает голосование
+  // время окна: у голосования движка — остаток до его конца
+  _windowTime() {
+    return this._deadline === null
+      ? this._time
+      : Math.max(0, this._deadline - this._now());
+  }
+
+  // завершает голосование; следом открывается ждущее в очереди
   complete() {
     this._waitingValues = false;
+    this._showing = false;
+    this._deadline = null;
     this.publisher.emit('clear', this._timerId);
     this.publisher.emit('mode', { name: 'vote', status: 'closed' });
+    this._openNext();
+  }
+
+  // первое ждущее голосование; голосование движка, чьё время вышло, —
+  // пропускается
+  _openNext() {
+    while (this._queue.length > 0) {
+      const { engine, template } = this._queue.shift();
+
+      if (engine) {
+        if (engine.deadline - this._now() > 0) {
+          this._startEngineVote(engine);
+          return;
+        }
+      } else {
+        this.createWithTemplate(template);
+        this.open();
+        return;
+      }
+    }
   }
 
   // добавляет id таймера голосования

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

@@ -1,5 +1,9 @@
 import Publisher from '../../../lib/Publisher.js';
-import { bindLiveErrors, buildForm, renderFormErrors } from '../../lib/formBuilder.js';
+import {
+  bindLiveErrors,
+  buildForm,
+  renderFormErrors,
+} from '../../lib/formBuilder.js';
 
 // Singleton AuthView
 
@@ -33,10 +37,14 @@ export default class AuthView {
 
     // ошибки видны по ходу правки (см. bindLiveErrors), а не по клику:
     // до сабмита — по тронутым полям, после — по всей форме
-    this._liveErrors = bindLiveErrors(this._fieldsContainer, this._error, () => ({
-      descriptors: this._descriptors,
-      fields: this._fields,
-    }));
+    this._liveErrors = bindLiveErrors(
+      this._fieldsContainer,
+      this._error,
+      () => ({
+        descriptors: this._descriptors,
+        fields: this._fields,
+      }),
+    );
 
     // форма заполнена; клиентская проверка (required/regExp/min/max) —
     // сервер всё равно валидирует своими validators (renderError), это лишь
@@ -69,11 +77,13 @@ export default class AuthView {
 
     // param.options — дескриптор-хвост (control/label/min/max/... —
     // "options" здесь ключ протокола PS_AUTH_DATA, не список выбора select
-    const descriptors = params.map(({ name, value, options: descriptorRest }) => ({
-      name,
-      default: value,
-      ...descriptorRest,
-    }));
+    const descriptors = params.map(
+      ({ name, value, options: descriptorRest }) => ({
+        name,
+        default: value,
+        ...descriptorRest,
+      }),
+    );
 
     this._descriptors = descriptors;
     // params сюда приходят чужие (PS_AUTH_DATA, тот же массив main.js

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

@@ -169,12 +169,17 @@ export default class ClientReportsView {
       [timeAgo(report.lastSeen), 'reports-when'],
       [`×${report.count}`, 'reports-count'],
       [`${report.kind}/${report.source}`, 'reports-kind'],
-      [truncate(String(report.code ?? report.message ?? ''), MESSAGE_PREVIEW), 'reports-message'],
+      [
+        truncate(String(report.code ?? report.message ?? ''), MESSAGE_PREVIEW),
+        'reports-message',
+      ],
       [report.gameId ? `${report.gameId}@${report.gameVersion ?? '—'}` : '—'],
       [`engine ${report.engineVersion ?? '—'}`],
       [report.box ?? '—'],
       [report.status, `reports-status reports-status-${report.status}`],
-    ].forEach(([text, className]) => summary.appendChild(this._span(text, className)));
+    ].forEach(([text, className]) =>
+      summary.appendChild(this._span(text, className)),
+    );
 
     details.style.display = this._expanded.has(report.id) ? '' : 'none';
     summary.onclick = () => {
@@ -199,7 +204,9 @@ export default class ClientReportsView {
     const details = document.createElement('div');
 
     details.className = 'reports-details';
-    details.appendChild(this._line(report.message ?? '', 'reports-full-message'));
+    details.appendChild(
+      this._line(report.message ?? '', 'reports-full-message'),
+    );
 
     if (report.stack) {
       details.appendChild(this._pre(report.stack));
@@ -210,7 +217,9 @@ export default class ClientReportsView {
     }
 
     details.appendChild(this._line(`User agent: ${report.userAgent ?? '—'}`));
-    details.appendChild(this._line(`Role: ${report.role ?? '—'}; page: ${report.page ?? '—'}`));
+    details.appendChild(
+      this._line(`Role: ${report.role ?? '—'}; page: ${report.page ?? '—'}`),
+    );
     details.appendChild(
       this._line(
         `First seen: ${report.firstSeen ?? '—'}; last seen: ${report.lastSeen ?? '—'}` +
@@ -218,7 +227,9 @@ export default class ClientReportsView {
       ),
     );
     details.appendChild(
-      this._line(`Fingerprint: ${String(report.fingerprint ?? '').slice(0, 12)}`),
+      this._line(
+        `Fingerprint: ${String(report.fingerprint ?? '').slice(0, 12)}`,
+      ),
     );
 
     if (report.statusAt) {

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

@@ -209,7 +209,10 @@ export default class GamesView {
     this._submitError.textContent = '';
 
     this._preview.appendChild(
-      this._line(`${title ? `${title} · ` : ''}${id ?? '—'}`, 'games-item-title'),
+      this._line(
+        `${title ? `${title} · ` : ''}${id ?? '—'}`,
+        'games-item-title',
+      ),
     );
     this._preview.appendChild(this._line(`Version: ${version ?? '—'}`));
     this._preview.appendChild(this._line(`Engine API: ${engineApi ?? '—'}`));
@@ -233,9 +236,9 @@ export default class GamesView {
     // разрешённая версия в поле меняет `${packageName}@${version}`, и
     // следующий blur прошёл бы как «другой пакет»: запрет повтора обновляется
     // фактическим ответом мастера
-    this._lastLookup = `${this._fields.get('packageName').value.trim()}@${
-      this._fields.get('version').value.trim()
-    }`;
+    this._lastLookup = `${this._fields.get('packageName').value.trim()}@${this._fields
+      .get('version')
+      .value.trim()}`;
 
     const problems = errors || [];
 
@@ -283,7 +286,10 @@ export default class GamesView {
       send.type = 'button';
       send.value = 'Update version';
       send.onclick = () =>
-        this.publisher.emit('update-version', { id: game.id, version: version.value.trim() });
+        this.publisher.emit('update-version', {
+          id: game.id,
+          version: version.value.trim(),
+        });
 
       item.appendChild(version);
       item.appendChild(send);
@@ -344,11 +350,15 @@ export default class GamesView {
     // решение по ней приняли. Из всей карточки ей нужны срок полного
     // удаления и возврат — остальные кнопки правили бы то, чего не видно
     if (game.deletedAt) {
-      item.appendChild(this._line(`Author: ${game.authorNick ?? game.authorUserId ?? '—'}`));
+      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 })),
+        this._button('Restore', () =>
+          this.publisher.emit('restore', { id: game.id }),
+        ),
       );
 
       return item;
@@ -356,7 +366,9 @@ export default class GamesView {
 
     // у игр, засеянных миграцией, автора нет вовсе: без запасного прочерка
     // в строке печаталось бы литеральное "null"
-    item.appendChild(this._line(`Author: ${game.authorNick ?? game.authorUserId ?? '—'}`));
+    item.appendChild(
+      this._line(`Author: ${game.authorNick ?? game.authorUserId ?? '—'}`),
+    );
 
     // переназначение автора: игры платформы засеяны без него, и «My games»
     // у их автора пуст, пока админ не проставит ник здесь. Пустое поле —
@@ -371,7 +383,10 @@ export default class GamesView {
     item.appendChild(author);
     item.appendChild(
       this._button('Set author', () =>
-        this.publisher.emit('set-author', { id: game.id, nick: author.value.trim() }),
+        this.publisher.emit('set-author', {
+          id: game.id,
+          nick: author.value.trim(),
+        }),
       ),
     );
     this._appendRepo(item, game.repoUrl);
@@ -387,7 +402,9 @@ export default class GamesView {
       item.appendChild(
         this._line(
           `On this master: ${game.local.downloaded ? 'downloaded' : 'not downloaded'}` +
-            (game.local.stagedVersion ? `; staged ${game.local.stagedVersion}` : '') +
+            (game.local.stagedVersion
+              ? `; staged ${game.local.stagedVersion}`
+              : '') +
             (game.local.lastError ? `; error: ${game.local.lastError}` : ''),
         ),
       );
@@ -409,7 +426,9 @@ export default class GamesView {
       ),
     );
     item.appendChild(
-      this._button('Approve', () => this.publisher.emit('approve', { id: game.id })),
+      this._button('Approve', () =>
+        this.publisher.emit('approve', { id: game.id }),
+      ),
     );
     item.appendChild(
       this._button('Reject', () =>
@@ -417,7 +436,9 @@ export default class GamesView {
       ),
     );
     item.appendChild(
-      this._button('Disable', () => this.publisher.emit('disable', { id: game.id })),
+      this._button('Disable', () =>
+        this.publisher.emit('disable', { id: game.id }),
+      ),
     );
     item.appendChild(
       this._button('npm versions', () =>
@@ -457,14 +478,18 @@ export default class GamesView {
 
   _statusLine(game) {
     const status = STATUS_TITLES[game.status] ?? game.status;
-    const date = game.createdAt ? new Date(game.createdAt).toLocaleDateString() : '';
+    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() : '—';
+      const purge = game.purgeAt
+        ? new Date(game.purgeAt).toLocaleDateString()
+        : '—';
 
       return `Deleted ${deleted} (was ${status}); removed for good on ${purge}`;
     }

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

@@ -1,6 +1,7 @@
 import Publisher from '../../../lib/Publisher.js';
 import { ENGINE_PROJECT_URL, ENGINE_VERSION } from '../../lib/engineVersion.js';
 import { renderProjectLink } from '../../lib/footerLink.js';
+import { absoluteLink, formatRoomLink } from '../../lib/roomLink.js';
 
 // Singleton LobbyView
 
@@ -9,9 +10,16 @@ let lobbyView;
 // неизвестная задержка сортируется в конец списка
 const UNKNOWN_LATENCY = Infinity;
 
+// 'unknown' мастер ставит, когда заголовка региона нет: в карточке это шум
+const knownRegion = region =>
+  typeof region === 'string' && region !== '' && region !== 'unknown'
+    ? region
+    : null;
+
 // Представление лобби: рендер списка серверов, поиск, «Загрузить ещё» и
 // умный пинг через IntersectionObserver (пинг шлётся только для карточек,
 // попавших в видимую область). Observer инъектируется ради тестируемости.
+
 export default class LobbyView {
   constructor(model, elems, observerFactory) {
     if (lobbyView) {
@@ -28,9 +36,13 @@ export default class LobbyView {
 
     // вкладки правой панели (lobby-page-plan)
     this._tabServersBtn = document.getElementById(elems.tabServersBtnId);
-    this._tabLeaderboardBtn = document.getElementById(elems.tabLeaderboardBtnId);
+    this._tabLeaderboardBtn = document.getElementById(
+      elems.tabLeaderboardBtnId,
+    );
     this._serversContent = document.getElementById(elems.serversContentId);
-    this._leaderboardContent = document.getElementById(elems.leaderboardContentId);
+    this._leaderboardContent = document.getElementById(
+      elems.leaderboardContentId,
+    );
     this._leaderboardList = document.getElementById(elems.leaderboardListId);
     this._leaderboardTitle = document.getElementById(elems.leaderboardTitleId);
     this._leaderboardTotal = document.getElementById(elems.leaderboardTotalId);
@@ -53,7 +65,10 @@ export default class LobbyView {
       version.textContent = ENGINE_VERSION;
     }
 
-    renderProjectLink(document.getElementById(elems.linkId), ENGINE_PROJECT_URL);
+    renderProjectLink(
+      document.getElementById(elems.linkId),
+      ENGINE_PROJECT_URL,
+    );
 
     // заголовок игры для "<TITLE> TOP-N" (SVG-ориентир) — задаётся controller'ом
     // при выборе игры (сама модель не хранит title манифеста)
@@ -76,7 +91,7 @@ export default class LobbyView {
     // границе LIMIT, см. комментарий в renderLeaderboard)
     this._selfNick = '';
 
-    this._cards = new Map(); // hostId -> { card, latencyEl, latency }
+    this._cards = new Map(); // roomId -> { card, latencyEl, latency }
 
     this.publisher = new Publisher();
 
@@ -84,20 +99,23 @@ export default class LobbyView {
       observerFactory ||
       (cb => new IntersectionObserver(cb, { root: this._list }));
 
-    // видимая карточка → запрос пинга (hostId в data-атрибуте)
+    // видимая карточка → запрос пинга (roomId в data-атрибуте)
     this._observer = makeObserver(entries => {
       entries.forEach(entry => {
         if (entry.isIntersecting) {
-          this.publisher.emit('visible', entry.target.dataset.hostId);
+          this.publisher.emit('visible', entry.target.dataset.roomId);
         }
       });
     });
 
-    this._search.oninput = () => this.publisher.emit('search', this._search.value);
+    this._search.oninput = () =>
+      this.publisher.emit('search', this._search.value);
     this._more.onclick = () => this.publisher.emit('more');
 
-    this._tabServersBtn.onclick = () => this.publisher.emit('show-tab', 'servers');
-    this._tabLeaderboardBtn.onclick = () => this.publisher.emit('show-tab', 'leaderboard');
+    this._tabServersBtn.onclick = () =>
+      this.publisher.emit('show-tab', 'servers');
+    this._tabLeaderboardBtn.onclick = () =>
+      this.publisher.emit('show-tab', 'leaderboard');
 
     for (const [period, btn] of this._periodBtns) {
       btn.onclick = () => this.publisher.emit('show-period', period);
@@ -160,8 +178,7 @@ export default class LobbyView {
   renderLeaderboard({ leaderboard, total, myPlacement, loaded = true }) {
     const period = this._periodTitle ? ` — ${this._periodTitle}` : '';
 
-    this._leaderboardTitle.textContent =
-      `${this._gameTitle.toUpperCase()} TOP-${this._leaderboardLimit}${period}`;
+    this._leaderboardTitle.textContent = `${this._gameTitle.toUpperCase()} TOP-${this._leaderboardLimit}${period}`;
     this._leaderboardTotal.textContent = `Total: ${total} players`;
 
     this._leaderboardList.textContent = '';
@@ -217,7 +234,8 @@ export default class LobbyView {
     // списке, ни в плашке. Ники уникальны (users_nick_lower_unique_idx),
     // так что membership по нику однозначен
     const inTop =
-      this._selfNick !== '' && leaderboard.some(entry => entry.nick === this._selfNick);
+      this._selfNick !== '' &&
+      leaderboard.some(entry => entry.nick === this._selfNick);
 
     if (inTop) {
       this._myPlacement.textContent = '';
@@ -252,8 +270,8 @@ export default class LobbyView {
   }
 
   // обновляет задержку карточки и переставляет её по возрастанию latency
-  updatePing({ hostId, latency }) {
-    const entry = this._cards.get(hostId);
+  updatePing({ roomId, latency }) {
+    const entry = this._cards.get(roomId);
 
     if (!entry) {
       return;
@@ -262,35 +280,34 @@ export default class LobbyView {
     entry.latency = latency;
     entry.latencyEl.textContent = `${latency} ms`;
 
-    this._reorderCard(hostId);
+    this._reorderCard(roomId);
   }
 
   _appendCard(server) {
     const card = document.createElement('li');
 
     card.className = 'lobby-card';
-    card.dataset.hostId = server.hostId;
+    card.dataset.roomId = server.roomId;
 
     const name = document.createElement('span');
 
     name.className = 'lobby-card-name';
-    // формат "gameId/name" (lobby-page-plan): готовит серверный поиск
-    // "gameId/name" к тому, чтобы искать по тому же, что видно на карточке;
-    // gameId nullable для хостов до Этапа 6.4 (code review L6)
-    name.textContent = `${server.gameId ?? '?'}/${server.name}`;
+    // формат "gameId/roomId": серверный поиск понимает ровно то, что видно
+    // на карточке; gameId nullable для хостов до Этапа 6.4 (code review L6)
+    name.textContent = `${server.gameId ?? '?'}/${server.roomId}`;
 
     const info = document.createElement('span');
 
     info.className = 'lobby-card-info';
-    info.textContent = `${server.mapName} · ${server.currentPlayers}/${server.maxPlayers} · ${server.region}`;
-
-    // рейтинг хостера (server-rating этап 3) — сервер уже отдаёт кэшированное
-    // значение в GET /servers, диапазон известен из конфига движка
-    const ratingEl = document.createElement('span');
-
-    ratingEl.className = 'lobby-card-rating';
-    ratingEl.classList.toggle('is-negative', server.rating < 0);
-    ratingEl.textContent = server.rating > 0 ? `+${server.rating}` : `${server.rating}`;
+    // пустые сегменты не выводятся: строку карточки игра может не задавать
+    // (gameConfig.lobbyInfo), а регион мастер не знает без заголовка CDN
+    info.textContent = [
+      server.info,
+      `${server.currentPlayers}/${server.maxPlayers}`,
+      knownRegion(server.region),
+    ]
+      .filter(Boolean)
+      .join(' · ');
 
     const latencyEl = document.createElement('span');
 
@@ -299,27 +316,61 @@ export default class LobbyView {
       server.latency === null ? '…' : `${server.latency} ms`;
 
     card.appendChild(name);
+
+    // прямая ссылка на комнату (host-migration, этап 3); без gameId (хост до
+    // Этапа 6.4) ссылку не собрать
+    if (server.gameId) {
+      card.appendChild(this._copyLinkButton(server));
+    }
+
     card.appendChild(info);
-    card.appendChild(ratingEl);
     card.appendChild(latencyEl);
 
-    card.onclick = () => this.publisher.emit('join', server.hostId);
+    card.onclick = () => this.publisher.emit('join', server.roomId);
 
     this._list.appendChild(card);
     this._observer.observe(card);
 
-    this._cards.set(server.hostId, {
+    this._cards.set(server.roomId, {
       card,
       latencyEl,
       latency: server.latency === null ? UNKNOWN_LATENCY : server.latency,
-      rating: server.rating,
     });
   }
 
-  // вставляет карточку перед первым соседом с большей задержкой; при равном
-  // пинге тай-брейк по rating (убыв.) — lobby-page-plan
-  _reorderCard(hostId) {
-    const entry = this._cards.get(hostId);
+  // вставляет карточку перед первым соседом с большей задержкой
+  // «Copy link» карточки: клик не должен всплыть до карточки (вход в комнату)
+  _copyLinkButton(server) {
+    const btn = document.createElement('input');
+    const label = 'Copy link';
+
+    btn.type = 'button';
+    btn.className = 'lobby-card-copy';
+    btn.value = label;
+    btn.onclick = event => {
+      event.stopPropagation();
+
+      const link = absoluteLink(formatRoomLink(server.gameId, server.roomId));
+      const feedback = text => {
+        btn.value = text;
+        setTimeout(() => {
+          btn.value = label;
+        }, 1500);
+      };
+
+      Promise.resolve()
+        .then(() => navigator.clipboard.writeText(link))
+        .then(
+          () => feedback('Copied'),
+          () => feedback('Copy failed'),
+        );
+    };
+
+    return btn;
+  }
+
+  _reorderCard(roomId) {
+    const entry = this._cards.get(roomId);
     const { card } = entry;
 
     let before = null;
@@ -329,13 +380,9 @@ export default class LobbyView {
         continue;
       }
 
-      const other = this._cards.get(sibling.dataset.hostId);
+      const other = this._cards.get(sibling.dataset.roomId);
 
-      if (
-        other &&
-        (other.latency > entry.latency ||
-          (other.latency === entry.latency && other.rating < entry.rating))
-      ) {
+      if (other && other.latency > entry.latency) {
         before = sibling;
         break;
       }