vimp-engine 0.17.0 → 0.18.1

package/package.json

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

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

@@ -1,6 +1,6 @@
 import Publisher from '../../../lib/Publisher.js';
-import { ENGINE_PACKAGE, ENGINE_VERSION } from '../../lib/engineVersion.js';
-import { renderPackageLink } from '../../lib/footerLink.js';
+import { ENGINE_PROJECT_URL, ENGINE_VERSION } from '../../lib/engineVersion.js';
+import { renderProjectLink } from '../../lib/footerLink.js';
 
 // Singleton LobbyView
 
@@ -44,7 +44,7 @@ export default class LobbyView {
       version.textContent = ENGINE_VERSION;
     }
 
-    renderPackageLink(document.getElementById(elems.linkId), ENGINE_PACKAGE);
+    renderProjectLink(document.getElementById(elems.linkId), ENGINE_PROJECT_URL);
 
     // заголовок игры для "<TITLE> TOP-N" (SVG-ориентир) — задаётся controller'ом
     // при выборе игры (сама модель не хранит title манифеста)

package/src/client/lib/engineVersion.js

@@ -7,13 +7,11 @@
 // бандлу (в npm-тарболле vimp-engine package.json есть всегда, поэтому
 // standalone SDK собирается у игры тем же импортом).
 import pkg from '../../../package.json' with { type: 'json' };
-import { homepageOf } from '../../lib/packageLink.js';
+import { resolveProjectUrl } from '../../lib/packageLink.js';
 
 export const ENGINE_VERSION = pkg.version;
 
-// метаданные для ссылки в футере лобби — та же форма, что мастер отдаёт про
-// пакет игры (GameCatalog), чтобы оба футера строил один resolvePackageLink
-export const ENGINE_PACKAGE = {
-  name: pkg.name,
-  homepage: homepageOf(pkg),
-};
+// адрес проекта для ссылки в футере лобби — то же поле, что мастер отдаёт
+// про пакет игры (GameCatalog.packageUrl), чтобы оба футера строил один
+// projectLink
+export const ENGINE_PROJECT_URL = resolveProjectUrl(pkg);

package/src/client/lib/footerLink.js

@@ -1,21 +1,21 @@
-import { resolvePackageLink } from '../../lib/packageLink.js';
+import { projectLink } from '../../lib/packageLink.js';
 
 // Ссылка в футере лобби и формы входа. Один рендер на оба экрана: лобби
-// подставляет пакет движка, форма входа — пакет игры из манифеста.
+// подставляет адрес проекта движка, форма входа — адрес игры из манифеста.
 //
-// Данных нет (standalone-манифест без метаданных пакета) — якорь гасится, а не
+// Адреса нет (пакет не объявил repository/homepage) — якорь гасится, а не
 // прячется: ячейка футера остаётся пустой, и раскладка space-between не едет.
 
 /**
  * @param {HTMLAnchorElement|null} anchor - якорь футера
- * @param {{name?: string, homepage?: string}|null} pkg - метаданные пакета
+ * @param {string|null} url - адрес проекта (resolveProjectUrl)
  */
-export function renderPackageLink(anchor, pkg) {
+export function renderProjectLink(anchor, url) {
   if (!anchor) {
     return;
   }
 
-  const link = resolvePackageLink(pkg);
+  const link = projectLink(url);
 
   if (!link) {
     anchor.textContent = '';

package/src/client/lib/formBuilder.js

@@ -97,7 +97,9 @@ function buildSelect(descriptor, ctx) {
       el.value = value;
     },
     onChange(cb) {
-      el.addEventListener('change', () => cb({ name: descriptor.name, value: el.value }));
+      el.addEventListener('change', () =>
+        cb({ name: descriptor.name, value: el.value }),
+      );
     },
   };
 }
@@ -110,6 +112,11 @@ function buildText(descriptor) {
   const numeric = isNumeric(descriptor);
 
   el.type = 'text';
+  // выпадашка прошлых значений/автозаполнения кроет форму, а подставить в
+  // игровое поле ей нечего. Только у текстового поля: у select такого списка
+  // нет, там autocomplete управлял бы лишь восстановлением значения при
+  // перезагрузке — а <option> создаёт JS, восстанавливать нечего
+  el.autocomplete = 'off';
   el.name = descriptor.name;
 
   // pattern/required — семантика контрола (a11y, автозаполнение), НЕ
@@ -157,7 +164,9 @@ function buildText(descriptor) {
       el.value = numeric ? toDisplay(descriptor, value) : (value ?? '');
     },
     onChange(cb) {
-      el.addEventListener('change', () => cb({ name: descriptor.name, value: getValue() }));
+      el.addEventListener('change', () =>
+        cb({ name: descriptor.name, value: getValue() }),
+      );
     },
   };
 }
@@ -182,7 +191,9 @@ function buildCheckbox(descriptor) {
       el.checked = Boolean(value);
     },
     onChange(cb) {
-      el.addEventListener('change', () => cb({ name: descriptor.name, value: el.checked }));
+      el.addEventListener('change', () =>
+        cb({ name: descriptor.name, value: el.checked }),
+      );
     },
   };
 }
@@ -239,7 +250,9 @@ function buildRadio(descriptor, ctx) {
     },
     onChange(cb) {
       inputs.forEach(input => {
-        input.addEventListener('change', () => cb({ name: descriptor.name, value: getValue() }));
+        input.addEventListener('change', () =>
+          cb({ name: descriptor.name, value: getValue() }),
+        );
       });
     },
   };
@@ -458,7 +471,9 @@ export function renderFormErrors(container, errors) {
     // для них, как и раньше, остаётся имя поля
     const title = String(label || name).toUpperCase();
 
-    line.textContent = error ? `${title}: ${error}` : `${title} is not correctly!`;
+    line.textContent = error
+      ? `${title}: ${error}`
+      : `${title} is not correctly!`;
 
     container.appendChild(line);
   });
@@ -556,7 +571,9 @@ export function buildForm(descriptors, container, ctx = {}, onChange) {
     } catch (e) {
       // одно поле с кривой/неизвестной схемой не должно ронять всю форму
       // (room-форма и особенно auth-форма — единственный путь к кнопке Start)
-      console.error(`formBuilder: skipping field "${descriptor.name}" — ${e.message}`);
+      console.error(
+        `formBuilder: skipping field "${descriptor.name}" — ${e.message}`,
+      );
       return;
     }
 
@@ -602,7 +619,8 @@ export function buildForm(descriptors, container, ctx = {}, onChange) {
     const label = document.createElement(field.labelFor ? 'label' : 'span');
 
     label.className = 'form-label';
-    label.textContent = (descriptor.label || descriptor.name) + buildLabelSuffix(descriptor);
+    label.textContent =
+      (descriptor.label || descriptor.name) + buildLabelSuffix(descriptor);
 
     if (field.labelFor) {
       label.htmlFor = field.labelFor;

package/src/client/main.js

@@ -28,7 +28,7 @@ import VoteView from './components/view/Vote.js';
 import VoteCtrl from './components/controller/Vote.js';
 import { buildForm, mergeRoomDefaults, bindLiveErrors } from './lib/formBuilder.js';
 import { normalizeAuthParams } from './lib/authParams.js';
-import { renderPackageLink } from './lib/footerLink.js';
+import { renderProjectLink } from './lib/footerLink.js';
 import { createGameActivator } from './lib/gameActivator.js';
 import createAutostart from './lib/autostart.js';
 import { getBootConfig, resolveBootConfig } from './boot.js';
@@ -434,10 +434,10 @@ socketMethods[PS_AUTH_DATA] = data => {
     authVersion.textContent = activeGameManifest?.packageVersion ?? '';
   }
 
-  renderPackageLink(document.getElementById('auth-package-link'), {
-    name: activeGameManifest?.packageName,
-    homepage: activeGameManifest?.packageHomepage,
-  });
+  renderProjectLink(
+    document.getElementById('auth-package-link'),
+    activeGameManifest?.packageUrl,
+  );
 
   // память клиента (localStorage) + принудительное значение поля с
   // единственным вариантом. Здесь, а не в AuthView: solo-путь ниже отвечает

package/src/client/style.css

@@ -385,10 +385,11 @@ body.hide-cursor {
   background-color: #333;
   border: 1px solid #eee;
   padding: 10px;
-  margin: 10px auto;
+  margin: 10px auto 5px;
   text-shadow: 1px 1px 1px #000;
   font-size: 12px;
   line-height: 15px;
+  border-radius: 4px;
 }
 
 .card h2 {
@@ -396,17 +397,19 @@ body.hide-cursor {
   line-height: 30px;
   margin: 0 0 10px 0;
   text-align: center;
+  border-bottom: 1px solid #444;
+  padding: 5px 0 10px;
 }
 
-/* кнопки: провайдеры логина, Create server, Start, Load more, Sign out —
-   нативный вид input[type=button], без декоративных стилей, только размер */
+/* кнопки: провайдеры логина, Choose nick, Start, Load more — нативный вид
+   input[type=button], без декоративных стилей, только размер. Create server и
+   Sign out сюда не входят: они лежат во флекс-контейнерах с gap и
+   центрируются через align-self (секция lobby-auth ниже) */
 .lobby-auth-provider,
 #lobby-auth-nick-submit,
-#lobby-host,
-#lobby-user-logout,
 #lobby-more,
 #auth-enter {
-  margin: 4px;
+  margin: 15px auto;
 }
 
 /* один вертикальный ритм для всех строк формы (room-форма и auth-форма) */
@@ -415,7 +418,7 @@ body.hide-cursor {
   align-items: center;
   justify-content: space-between;
   gap: 8px;
-  margin: 8px 0;
+  margin: 0;
 }
 
 .form-row input[type='text'] {
@@ -423,6 +426,10 @@ body.hide-cursor {
   text-align: center;
 }
 
+#lobby-fields .form-row {
+  margin: 6px 0;
+}
+
 .form-label {
   color: #aaa;
 }
@@ -459,7 +466,7 @@ body.hide-cursor {
   color: #e00;
   font-size: 12px;
   line-height: 15px;
-  margin: 0 5px;
+  margin: 0 5px 10px;
 }
 
 /* пустой блок ошибок не должен занимать место: #lobby-error — флекс-элемент
@@ -470,7 +477,6 @@ body.hide-cursor {
 
 #auth-enter {
   display: block;
-  margin: 4px auto;
 }
 
 /* футеры лобби и формы входа: одна и та же полоса из трёх ячеек —
@@ -479,18 +485,35 @@ body.hide-cursor {
    разметкой) */
 #auth-link,
 #lobby-footer {
-  font-size: 9px;
   height: 15px;
-  margin: 0;
-  padding: 0 5px;
   text-shadow: none;
   display: flex;
-  justify-content: space-between;
 }
 
+#auth-link {
+  font-size: 9px;
+  margin: 0;
+  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));
+  font-size: 10px;
+  padding: 0;
+  margin: 0 var(--lobby-panel-pad) 0 0;
+}
+
+/* три равные ячейки: середину держит не space-between, а сама равность —
+   любая из крайних бывает пустой (у пакета игры может не быть метаданных),
+   и при space-between версия съезжала бы с центра */
 #auth-link p,
 #lobby-footer p {
   margin: 0;
+  flex: 1;
 }
 
 #auth-link p a,
@@ -498,9 +521,7 @@ body.hide-cursor {
   color: #ff0;
 }
 
-/* позициями заведует space-between, а не nth-child: любая из ячеек бывает
-   пустой (у пакета игры может не быть метаданных), и порядковые правила
-   тогда съезжали бы */
+/* позиция текста внутри ячейки; сами ячейки уже разложены flex: 1 выше */
 #auth-link p:first-child,
 #lobby-footer p:first-child {
   text-align: left;
@@ -516,7 +537,11 @@ body.hide-cursor {
   text-align: right;
 }
 
-.auth-inform {
+/* коробка контента: блок помощи формы входа и группа полей комнаты.
+   .lobby-card сюда не входит — ей нужен фон темнее, иначе он совпадёт с
+   её собственным hover'ом и наведение перестанет быть видно */
+.auth-inform,
+#lobby-fields {
   background-color: #444;
   padding: 7px 13px;
   margin: 10px 0;
@@ -592,6 +617,13 @@ body.hide-cursor {
 /*lobby*/
 
 #lobby {
+  /* ширины панелей и их внутренние поля: по ним же считается ширина футера
+     (секция auth выше), поэтому числа живут в одном месте — иначе смена
+     ширины любой панели молча разъезжает вёрстку */
+  --lobby-setup-width: 410px;
+  --lobby-browser-width: 430px;
+  --lobby-panel-pad: 10px;
+
   background-color: #222;
   position: absolute;
   width: 100%;
@@ -602,16 +634,20 @@ body.hide-cursor {
   display: flex;
   align-items: flex-start;
   justify-content: center;
-  padding: 10px 0 0;
+  padding: 0;
   box-sizing: border-box;
 }
 
+#lobby-user,
+.lobby-section-title,
+#lobby-create {
+  padding: 8px;
+}
+
 #lobby-create {
   display: flex;
   flex-direction: column;
-  gap: 8px;
-  margin-bottom: 12px;
-  padding: 8px;
+  gap: 10px;
 }
 
 /* поле имени сервера — строка формы, а не узкий числовой инпут
@@ -638,13 +674,10 @@ body.hide-cursor {
   margin: 0;
 }
 
-#lobby-host {
-  margin: 16px auto 0;
-}
-
 #lobby-search {
   display: block;
   width: 100%;
+  margin-bottom: 10px;
 }
 
 #lobby-list {
@@ -662,9 +695,10 @@ body.hide-cursor {
   column-gap: 8px;
   row-gap: 2px;
   padding: 10px 12px;
-  margin-bottom: 6px;
+  margin: 0 0 6px;
   background-color: #363636;
-  border: 1px solid #444;
+  border: 1px solid #999;
+  border-radius: 4px;
   cursor: pointer;
   align-items: center;
 }
@@ -735,6 +769,7 @@ body.hide-cursor {
 .lobby-column {
   display: flex;
   flex-direction: column;
+  align-items: flex-end;
 }
 
 /* два столбца лобби (lobby-page-plan): настройки/создание слева, вкладки
@@ -744,12 +779,12 @@ body.hide-cursor {
    высоты более высокой панели */
 .lobby-grid {
   display: flex;
-  gap: 20px;
+  gap: 10px;
   justify-content: center;
 }
 
 #lobby-setup-panel {
-  width: 410px;
+  width: var(--lobby-setup-width);
 }
 
 /* высоту правой .card задаёт форма слева: обе панели тянутся до высоты
@@ -758,7 +793,7 @@ body.hide-cursor {
    вынесен в абсолютный слой .lobby-browser-inner (его высота не участвует в
    расчёте высоты строки) */
 #lobby-browser-panel {
-  width: 430px;
+  width: var(--lobby-browser-width);
   position: relative;
   padding: 0;
 }
@@ -768,7 +803,7 @@ body.hide-cursor {
   inset: 0;
   display: flex;
   flex-direction: column;
-  padding: 10px;
+  padding: var(--lobby-panel-pad);
   box-sizing: border-box;
 }
 
@@ -793,23 +828,23 @@ body.hide-cursor {
   flex-shrink: 0;
 }
 
-/* полосу футера рисуют общие с #auth-link правила в секции auth; здесь —
-   только отступ от панелей */
-#lobby-footer {
-  margin: 6px 0 10px;
-}
-
 @media (max-width: 800px) {
   .lobby-grid {
-    flex-direction: column;
-    align-items: stretch;
-    gap: unset;
+    display: block;
   }
 
   /* в столбик обёртка тянется на всю ширину: иначе она сжалась бы по
      содержимому и увела бы панели из центра */
   .lobby-column {
     width: 100%;
+    align-items: center;
+  }
+
+  /* правое поле, прижимавшее футер к контенту правой панели, в одну
+     колонку не нужно: с ним align-items: center увёл бы его влево на
+     половину поля */
+  #lobby-footer {
+    margin: 0 auto;
   }
 
   /* в столбик высоту строкой никто не задаёт — возвращаем слой в поток,

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

@@ -2,4 +2,4 @@ div#chat
   div#chat-box
   //- maxlength — лишь UX-подсказка; авторитетный лимит на сервере
   //- (chatMaxLength в src/config/hostDefaults.js). Значения держать синхронными.
-  input#cmd(type='text', maxlength='60' autocomplete="off")
+  input#cmd(type='text', maxlength='60', autocomplete='off')

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

@@ -21,11 +21,11 @@ div#lobby(style='display:none')
           //- селектор игры (lobby-page-plan): рабочий с первой же игрой в
           //- каталоге мастера, а не только когда их несколько
           div.form-row
-            label.form-label(for='lobby-game')= 'Select Game'
+            label.form-label(for='lobby-game')= 'Select game'
             select#lobby-game.field-select
           div.form-row
             label.form-label(for='lobby-name')= 'Server name'
-            input#lobby-name.field-text(type='text', maxlength='30')
+            input#lobby-name.field-text(type='text', maxlength='30', autocomplete='off')
           //- поля настроек комнаты генерируются по явной схеме roomForm
           //- манифеста активной игры (docs/en/plugin-api.md "Form schema")
           div#lobby-fields
@@ -45,7 +45,7 @@ div#lobby(style='display:none')
             input#btn-show-servers.lobby-tab-btn.active(type='button', value='Active Servers')
             input#btn-show-leaderboard.lobby-tab-btn(type='button', value='Leaderboard')
           div#lobby-servers-content
-            input#lobby-search.field-text(type='text', placeholder='Search servers…')
+            input#lobby-search.field-text(type='text', placeholder='Search servers…', autocomplete='off')
             ul#lobby-list
             p#lobby-empty= 'No servers available'
             input#lobby-more(type='button', value='Load more')

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

@@ -14,6 +14,6 @@ div#lobby-auth
 
   div#lobby-auth-nick.card(style='display:none')
     h2= 'Choose your nick'
-    input#lobby-auth-nick-input.field-text(type='text', maxlength='14', placeholder='YourNick')
+    input#lobby-auth-nick-input.field-text(type='text', maxlength='14', placeholder='YourNick', autocomplete='off')
     div#lobby-auth-nick-error
     input#lobby-auth-nick-submit(type='button', value='Continue')

package/src/devtools/contract/rules/a7-package-repository.js

@@ -0,0 +1,29 @@
+import { WARN, skip, verdict } from '../result.js';
+import { resolveProjectUrl } from '../../../lib/packageLink.js';
+
+// Игра без объявленного репозитория собирается и играется — это не отказ.
+// Но движок показывает ссылку на проект в футере формы входа, и берёт он её
+// только отсюда (фолбэка нет, см. lib/packageLink.js): без поля ячейка
+// футера у игроков останется пустой, а заметить это без подсказки негде.
+// Отсюда warn — под --strict он становится ошибкой.
+export default {
+  id: 'A7',
+  name: 'packageRepository',
+  level: WARN,
+  title: 'package.json declares repository (the #auth footer links to it)',
+
+  check(ctx) {
+    if (!ctx.pkg) {
+      return skip('no package.json');
+    }
+
+    if (resolveProjectUrl(ctx.pkg)) {
+      return verdict([]);
+    }
+
+    return verdict([
+      'neither "repository" nor "homepage" resolves to a project URL — ' +
+        "the game's entry form will show no project link",
+    ]);
+  },
+};

package/src/devtools/contract/rules/index.js

@@ -4,6 +4,7 @@ import a3 from './a3-entry-paths.js';
 import a4 from './a4-vite-config.js';
 import a5 from './a5-cargo-core.js';
 import a6 from './a6-manifest.js';
+import a7 from './a7-package-repository.js';
 import b1 from './b1-host-shape.js';
 import b2 from './b2-engine-api.js';
 import b3 from './b3-game-config-shape.js';
@@ -34,7 +35,7 @@ import e3 from './e3-sound-registry.js';
 // Порядок групп — порядок отчёта: A пакет и сборка, B host, C client,
 // D снапшот, E ассеты (plan/create-vimp-game/stage_1.md).
 export const rules = [
-  a1, a2, a3, a4, a5, a6,
+  a1, a2, a3, a4, a5, a6, a7,
   b1, b2, b3, b4, b5, b6, b7, b8, b9, b10,
   c1, c2, c3, c4, c5, c6, c7, c8, c9, c10,
   d1, d2, d3,

package/src/lib/packageLink.js

@@ -1,67 +1,73 @@
-// Ссылка на пакет для футеров лобби и формы входа: движок показывает рядом
-// npm-версию, и клик по соседней ячейке должен вести на страницу этого пакета.
+// Ссылка на проект для футеров лобби и формы входа: движок показывает рядом
+// npm-версию, и клик по соседней ячейке ведёт в репозиторий этого пакета.
 //
-// Источник — `homepage`/`repository` его package.json, но объявлены они не у
-// всех (у @vimp-games/snakes нет ни того, ни другого), а footer без ссылки
-// выглядит поломкой. Поэтому фолбэк — страница пакета на npm: она выводится из
-// одного лишь имени, то есть существует всегда, и сама ведёт на репозиторий,
-// если он объявлен.
+// Источник — `repository` его package.json (иначе `homepage`). Фолбэка нет:
+// пакет, не объявивший ни того, ни другого, ссылки не получает, и ячейка
+// футера остаётся пустой. Это сознательно — прежний фолбэк на страницу пакета
+// в npm давал разнобой («GitHub» у одной игры, «npm» у другой) и молча
+// маскировал недостающие метаданные вместо того, чтобы показать их отсутствие.
+// О пропущенном поле предупреждает правило контракта A7.
 //
-// Модуль изоморфный: мастер зовёт homepageOf для пакета игры (GameCatalog),
-// клиент — для собственного package.json движка (client/lib/engineVersion.js).
-
-const NPM_PACKAGE_BASE = 'https://www.npmjs.com/package/';
+// Хостинг не ограничен github.com: подпись выводится из хоста, поэтому чужой
+// плагин на GitLab получает честную ссылку, а не пустую ячейку.
+//
+// Модуль изоморфный: мастер зовёт resolveProjectUrl для пакета игры
+// (GameCatalog), клиент — для собственного package.json движка
+// (client/lib/engineVersion.js).
 
 /**
- * Сырой адрес проекта из package.json: homepage, иначе repository.
+ * Адрес проекта из package.json, приведённый к https: repository, иначе
+ * homepage.
  * @param {Object} pkg - разобранный package.json
- * @returns {string|null} адрес как объявлен (нормализует resolvePackageLink)
+ * @returns {string|null} https-URL либо null, если поля нет или это не адрес
  */
-export function homepageOf(pkg) {
+export function resolveProjectUrl(pkg) {
   if (!pkg || typeof pkg !== 'object') {
     return null;
   }
 
   const { homepage, repository } = pkg;
 
-  if (typeof homepage === 'string' && homepage.trim()) {
-    return homepage;
-  }
-
   // repository бывает и строкой ("github:user/repo"), и объектом {type, url}
-  if (typeof repository === 'string') {
-    return repository;
-  }
-
-  if (repository && typeof repository.url === 'string') {
-    return repository.url;
-  }
+  const declared =
+    typeof repository === 'string' ? repository : repository?.url;
 
-  return null;
+  // объявленный, но не приводящийся к http(s) repository (пустая строка,
+  // внутренний git-хост) не повод терять валидный homepage: поле проверяется
+  // по результату, а не по факту наличия
+  return normalizeRepository(declared) ?? normalize(homepage);
 }
 
 /**
- * Ссылка футера по метаданным пакета.
- * @param {{name?: string, homepage?: string}} [pkg] - имя пакета и сырой homepage
- * @returns {{url: string, label: string}|null} null, если нет даже имени
+ * Ссылка футера по готовому адресу проекта.
+ * @param {string|null} [url] - результат resolveProjectUrl
+ * @returns {{url: string, label: string}|null} null, если адреса нет
  */
-export function resolvePackageLink(pkg) {
-  const { name, homepage } = pkg ?? {};
-  const declared = normalize(homepage);
+export function projectLink(url) {
+  // адрес перепроверяется, хотя свои вызовы приходят уже из
+  // resolveProjectUrl: функция — публичный экспорт ("./lib/*"), а результат
+  // уходит прямо в href якоря (client/lib/footerLink.js)
+  const value = typeof url === 'string' ? url.trim() : '';
 
-  if (declared) {
-    return { url: declared, label: labelFor(declared) };
+  if (!/^https?:\/\//.test(value)) {
+    return null;
   }
 
-  if (typeof name === 'string' && name.trim()) {
-    return { url: `${NPM_PACKAGE_BASE}${name.trim()}`, label: 'npm' };
-  }
+  return { url: value, label: labelFor(value) };
+}
+
+// шорткат "user/repo" — семантика поля repository (так его читает npm), в
+// homepage такая строка означала бы относительный путь, а не адрес на GitHub
+function normalizeRepository(raw) {
+  const value = typeof raw === 'string' ? raw.trim() : '';
 
-  return null;
+  return /^[\w.-]+\/[\w.-]+$/.test(value)
+    ? `https://github.com/${value}`
+    : normalize(raw);
 }
 
 // приводит объявленный адрес к https-виду; всё, что после нормализации не
-// http(s) (git@ без известной формы, file:, мусор), — не ссылка, а фолбэк
+// http(s) (git@ без известной формы, file:, мусор), адресом не считается
 function normalize(raw) {
   if (typeof raw !== 'string') {
     return null;
@@ -106,6 +112,9 @@ function normalize(raw) {
 
   return (
     url
+      // логин (и тем более пароль) из ssh-формы адреса — не часть проекта, а
+      // в href он уехал бы в DOM и историю браузера
+      .replace(/^(https?:\/\/)[^/@]*@/, '$1')
       .replace(/\.git(?=$|[#?])/, '')
       // npm сам дописывает "#readme" в homepage при генерации package.json
       .replace(/#readme$/, '')
@@ -113,14 +122,16 @@ function normalize(raw) {
   );
 }
 
+// подпись ячейки: у наших пакетов это всегда GitHub, у чужого хостинга —
+// его хост, чтобы подпись не врала об источнике
 function labelFor(url) {
-  if (/^https?:\/\/(www\.)?github\.com\//.test(url)) {
-    return 'GitHub';
-  }
+  let host;
 
-  if (url.startsWith(NPM_PACKAGE_BASE)) {
-    return 'npm';
+  try {
+    host = new URL(url).hostname.replace(/^www\./, '');
+  } catch {
+    return 'Website';
   }
 
-  return 'Website';
+  return host === 'github.com' ? 'GitHub' : host;
 }