vimp-engine 0.19.0 → 0.20.0

package/package.json

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

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

@@ -27,6 +27,7 @@ export default class LobbyCtrl {
     this._model = model;
     this._view = view;
     this._clock = clock;
+    this._periods = [];
 
     this.publisher = new Publisher();
 
@@ -37,6 +38,12 @@ export default class LobbyCtrl {
     vp.on('visible', 'pingHost', this);
     vp.on('join', 'join', this);
     vp.on('show-tab', 'showTab', this);
+    vp.on('show-period', 'showPeriod', this);
+
+    // активная игра и открытый срез рейтинга (rank-periods): запрос за
+    // лидербордом всегда несёт обе координаты, а меняется каждый раз одна
+    this._gameId = null;
+    this._period = null;
   }
 
   // показывает лобби и запрашивает первую страницу
@@ -69,13 +76,47 @@ export default class LobbyCtrl {
     this._view.showTab(tab);
   }
 
+  // срез рейтинга: подсветить кнопку и перезапросить список. Данных за день
+  // и за месяц у клиента нет — это другой ответ сервера, а не другая
+  // сортировка того же (rank-periods)
+  showPeriod(period) {
+    if (period === this._period) {
+      return;
+    }
+
+    this._period = period;
+    this._view.setPeriod(period, this._titleOf(period));
+
+    if (this._gameId !== null) {
+      this.publisher.emit('leaderboard-needed', {
+        gameId: this._gameId,
+        period,
+      });
+    }
+  }
+
+  // подпись среза для заголовка списка; неизвестный id остаётся без подписи
+  _titleOf(period) {
+    return this._periods.find(item => item.id === period)?.title ?? '';
+  }
+
   // вызывается main.js при смене #lobby-game (селектор игры вне этого MVC),
   // а также один раз при открытии лобби для активной по умолчанию игры —
   // Leaderboard всегда готов заранее, до того как пользователь откроет
   // вкладку (см. класс-комментарий выше)
   gameChanged(gameId, title) {
+    this._gameId = gameId;
     this._view.setGameTitle(title);
 
-    this.publisher.emit('leaderboard-needed', gameId);
+    this.publisher.emit('leaderboard-needed', { gameId, period: this._period });
+  }
+
+  // срезы рейтинга и открытый по умолчанию (main.js передаёт их из
+  // lobbyConfig): контроллер не знает их наизусть, чтобы список срезов
+  // правился в конфиге, а не здесь
+  setPeriods(periods, defaultPeriod) {
+    this._periods = periods || [];
+    this._period = defaultPeriod;
+    this._view.setPeriod(defaultPeriod, this._titleOf(defaultPeriod));
   }
 }

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

@@ -36,6 +36,15 @@ export default class LobbyView {
     this._leaderboardTotal = document.getElementById(elems.leaderboardTotalId);
     this._myPlacement = document.getElementById(elems.myPlacementId);
 
+    // кнопки срезов рейтинга (rank-periods): id периода -> элемент. Кнопки
+    // объявлены конфигом, а не захардкожены здесь, чтобы срез можно было
+    // убрать или добавить в одном месте
+    this._periodBtns = new Map(
+      Object.entries(elems.periodBtnIds || {})
+        .map(([period, id]) => [period, document.getElementById(id)])
+        .filter(([, el]) => el),
+    );
+
     // футер лобби: содержимое статично на всё время жизни вкладки, поэтому
     // пишется здесь, а не через модель и publisher
     const version = document.getElementById(elems.versionId);
@@ -56,6 +65,10 @@ export default class LobbyView {
     // игроков, чем лимит)
     this._leaderboardLimit = 0;
 
+    // заголовок открытого среза ("<TITLE> TOP-N — TODAY"): подпись, а не
+    // источник правды, срез хранит модель
+    this._periodTitle = '';
+
     // ник вызывающего (lobby-page-plan, code review M4-остаток) — задаётся
     // main.js один раз при открытии лобби; используется, чтобы решить,
     // виден ли вызывающий уже в отрисованном топе (по нику, не по числу
@@ -86,6 +99,10 @@ export default class LobbyView {
     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);
+    }
+
     this._mPublic = model.publisher;
     this._mPublic.on('list', 'renderList', this);
     this._mPublic.on('ping-update', 'updatePing', this);
@@ -118,6 +135,17 @@ export default class LobbyView {
     this._selfNick = nick || '';
   }
 
+  // подсвечивает открытый срез рейтинга и запоминает его подпись
+  // (rank-periods). Рисовать список здесь нечем: данные среза приезжают
+  // отдельным запросом, и до ответа список чистится моделью
+  setPeriod(period, title) {
+    this._periodTitle = title || '';
+
+    for (const [id, btn] of this._periodBtns) {
+      btn.classList.toggle('active', id === period);
+    }
+  }
+
   // переключает вкладки Active Servers / Leaderboard (lobby-page-plan)
   showTab(tab) {
     const showLeaderboard = tab === 'leaderboard';
@@ -130,7 +158,10 @@ export default class LobbyView {
 
   // рендер топ-N + позиции вызывающего (lobby-page-plan)
   renderLeaderboard({ leaderboard, total, myPlacement, loaded = true }) {
-    this._leaderboardTitle.textContent = `${this._gameTitle.toUpperCase()} TOP-${this._leaderboardLimit}`;
+    const period = this._periodTitle ? ` — ${this._periodTitle}` : '';
+
+    this._leaderboardTitle.textContent =
+      `${this._gameTitle.toUpperCase()} TOP-${this._leaderboardLimit}${period}`;
     this._leaderboardTotal.textContent = `Total: ${total} players`;
 
     this._leaderboardList.textContent = '';

package/src/client/main.js

@@ -1790,10 +1790,13 @@ async function fetchServers({ offset, limit, search }) {
 
 // топ-N рейтинга игры (lobby-page-plan) — публичный эндпоинт, доступен и до
 // логина, поэтому без Authorization
-async function fetchLeaderboard(gameId) {
+async function fetchLeaderboard(gameId, period) {
   const params = new URLSearchParams({
     game: gameId,
     limit: lobbyConfig.leaderboardLimit,
+    // rank-periods: срез времени. Мастер отвечает 400 на незнакомый —
+    // значение всегда из lobbyConfig.leaderboardPeriods
+    period,
   });
 
   try {
@@ -1807,7 +1810,7 @@ async function fetchLeaderboard(gameId) {
 
 // позиция вызывающего в рейтинге игры (lobby-page-plan) — требует identity-
 // токена, как и остальные /auth/* запросы игрока (rank/state)
-async function fetchPlacement(gameId) {
+async function fetchPlacement(gameId, period) {
   const token = lobbyAuthModel.getToken();
 
   if (!token) {
@@ -1816,7 +1819,9 @@ async function fetchPlacement(gameId) {
 
   try {
     const res = await fetch(
-      `${lobbyConfig.placementUrl}?${new URLSearchParams({ game: gameId })}`,
+      // тот же срез, что и у списка рядом: плашка позиции, посчитанная за
+      // всё время под заголовком «сегодня», противоречила бы списку
+      `${lobbyConfig.placementUrl}?${new URLSearchParams({ game: gameId, period })}`,
       { headers: { authorization: `Bearer ${token}` } },
     );
 
@@ -1954,6 +1959,13 @@ function initLobby() {
 
   lobby = new LobbyCtrl(lobbyModel, lobbyView);
 
+  // срезы рейтинга (rank-periods) — сразу после создания контроллера и до
+  // первого gameChanged: тот уже несёт открытый срез в запросе
+  lobby.setPeriods(
+    lobbyConfig.leaderboardPeriods,
+    lobbyConfig.defaultLeaderboardPeriod,
+  );
+
   // список серверов — REST-запросом к мастеру
   lobbyModel.publisher.on('fetch', async query => {
     const list = await fetchServers(query);
@@ -1989,17 +2001,17 @@ function initLobby() {
   // под заголовком новой, пока не пришёл ответ (и остаются навсегда при
   // сетевом сбое); reqId отбрасывает ответ устаревшего запроса, если игру
   // переключили быстрее, чем пришёл ответ (latest-wins)
-  lobby.publisher.on('leaderboard-needed', async gameId => {
+  lobby.publisher.on('leaderboard-needed', async ({ gameId, period }) => {
     lobbyModel.clearLeaderboard();
 
     const reqId = ++leaderboardReqId;
     const [leaderboard, placement] = await Promise.all([
-      fetchLeaderboard(gameId),
-      fetchPlacement(gameId),
+      fetchLeaderboard(gameId, period),
+      fetchPlacement(gameId, period),
     ]);
 
     if (reqId !== leaderboardReqId) {
-      return; // игру уже переключили ещё раз — этот ответ устарел
+      return; // игру или срез уже переключили ещё раз — ответ устарел
     }
 
     if (leaderboard) {

package/src/client/style.css

@@ -908,6 +908,32 @@ body.hide-cursor {
   border-bottom: 2px solid #fff;
 }
 
+/* срезы рейтинга (rank-periods): день/месяц/всё время. Ряд под вкладками
+   правой панели — это выбор ДАННЫХ, а не вкладка, поэтому кнопки выглядят
+   как переключатель, а не как заголовок раздела */
+.lobby-period-tabs {
+  display: flex;
+  gap: 4px;
+  margin-bottom: 10px;
+}
+
+.lobby-period-btn {
+  flex: 1 1 0;
+  background: rgb(255 255 255 / 6%);
+  border: none;
+  color: #888;
+  font-size: 12px;
+  font-weight: bold;
+  padding: 5px 4px;
+  margin: 0;
+  text-shadow: none;
+}
+
+.lobby-period-btn.active {
+  background: rgb(255 255 255 / 18%);
+  color: #fff;
+}
+
 .lobby-leaderboard-header {
   display: flex;
   justify-content: space-between;

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

@@ -50,6 +50,12 @@ div#lobby(style='display:none')
             p#lobby-empty= 'No servers available'
             input#lobby-more(type='button', value='Load more')
           div#lobby-leaderboard-content(style='display:none')
+            //- срез рейтинга (rank-periods): день/месяц считаются из леджера
+            //- дельт, all-time — из кэша ratings
+            div.lobby-period-tabs
+              input#btn-period-day.lobby-period-btn(type='button', value='Daily')
+              input#btn-period-month.lobby-period-btn(type='button', value='Monthly')
+              input#btn-period-all.lobby-period-btn.active(type='button', value='All-Time')
             div.lobby-leaderboard-header
               span#leaderboard-title
               span#leaderboard-total

package/src/config/lobby.js

@@ -50,6 +50,17 @@ export default {
   placementUrl: '/auth/placement',
   leaderboardLimit: 10,
 
+  // rank-periods: срезы рейтинга и тот, что открыт по умолчанию. Порядок
+  // здесь — порядок кнопок; `id` едет в auth как ?period=, `title` идёт в
+  // заголовок списка. Значения должны совпадать с RANK_PERIODS auth-сервиса:
+  // на всё прочее он отвечает 400
+  leaderboardPeriods: [
+    { id: 'day', title: 'TODAY' },
+    { id: 'month', title: 'THIS MONTH' },
+    { id: 'all', title: 'ALL-TIME' },
+  ],
+  defaultLeaderboardPeriod: 'all',
+
   // переподключение сигнального WS хоста (комната без него выпадает из
   // выдачи мастера): экспоненциальный бэкофф от baseDelay до maxDelay (мс)
   reconnect: {
@@ -93,6 +104,12 @@ export default {
     serversContentId: 'lobby-servers-content',
     leaderboardContentId: 'lobby-leaderboard-content',
     leaderboardListId: 'lobby-leaderboard-list',
+    // кнопки срезов (rank-periods): id периода -> id элемента
+    periodBtnIds: {
+      day: 'btn-period-day',
+      month: 'btn-period-month',
+      all: 'btn-period-all',
+    },
     leaderboardTitleId: 'leaderboard-title',
     leaderboardTotalId: 'leaderboard-total',
     myPlacementId: 'lobby-my-placement',

package/src/host/HostGame.js

@@ -1063,6 +1063,19 @@ export default class HostGame {
     this._playerDataSync.addRank(gameId, delta);
   }
 
+  // синхронизация профилей (rank/state) всех участников на мастер прямо
+  // сейчас — для игр, у которых нет ни конца раунда, ни смены карты
+  // (`endlessRound` + карта, пересобираемая через overrideMapData): обе
+  // штатные границы flushAll живут в RoundManager, и такая игра не проходит
+  // ни через одну из них. Без этого накопленный за матч ранг уезжает в auth
+  // только на выходе участника, а закрытая вкладка хоста теряет его вовсе.
+  //
+  // Best-effort, как и весь PlayerDataSync: промис не отвергается, сбой
+  // логируется в `[playerData]` и повторится следующим flush'ем.
+  flushPlayerData() {
+    return this._playerDataSync.flushAll();
+  }
+
   // hostId + per-room секрет комнаты, подтверждённые мастером при
   // register_host (кодревью №1) — не известны при создании HostGame (Worker
   // стартует раньше ответа мастера); нужны PlayerDataSync для атрибуции