@mikitasazan/notify 1.1.1 → 1.1.3

package/dist/cli.js

@@ -163,8 +163,7 @@ else {
                 url: one('url'),
                 target: one('target'),
                 via: one('via'),
-                note: one('note'),
-                key: one('key')
+                note: one('note')
             };
             break;
         case 'job':
@@ -176,8 +175,7 @@ else {
                 stats: pairs('stat'),
                 items: items(),
                 note: one('note'),
-                url: one('url'),
-                key: one('key')
+                url: one('url')
             };
             break;
         case 'report':
@@ -188,8 +186,7 @@ else {
                 period: one('period'),
                 lines: pairs('line'),
                 items: items(),
-                url: one('url'),
-                key: one('key')
+                url: one('url')
             };
             break;
         case 'ci':
@@ -200,8 +197,7 @@ else {
                 branch: one('branch'),
                 commit: one('commit'),
                 actor: one('actor'),
-                url: one('url'),
-                key: one('key')
+                url: one('url')
             };
             break;
         case 'pr':
@@ -213,8 +209,7 @@ else {
                 title: one('title') ?? '(без заголовка)',
                 author: one('author'),
                 reviewer: one('reviewer'),
-                url: one('url'),
-                key: one('key')
+                url: one('url')
             };
             break;
         case 'issue':
@@ -226,8 +221,7 @@ else {
                 title: one('title') ?? '(без заголовка)',
                 author: one('author'),
                 assignee: one('assignee'),
-                url: one('url'),
-                key: one('key')
+                url: one('url')
             };
             break;
         case 'incident':
@@ -236,8 +230,7 @@ else {
                 project: project(),
                 title: one('title') ?? '(без заголовка)',
                 detail: one('detail'),
-                url: one('url'),
-                key: one('key')
+                url: one('url')
             };
             break;
         case 'heartbeat_miss':
@@ -246,8 +239,7 @@ else {
                 project: project(),
                 job: one('job') ?? '(без имени)',
                 lastSeen: one('last-seen'),
-                expected: one('expected'),
-                key: one('key')
+                expected: one('expected')
             };
             break;
         case 'file': {
@@ -261,8 +253,7 @@ else {
                 title: one('title') ?? '(без заголовка)',
                 path: path ?? '',
                 filename: one('filename'),
-                note: one('note'),
-                key: one('key')
+                note: one('note')
             };
             break;
         }
@@ -270,6 +261,12 @@ else {
             log(`неизвестный тип события: ${command ?? '(не указан)'}`);
     }
 }
+// --key применим к любому типу — одна точка вместо строки в каждом case
+// (девять копий уже потеряли бы десятую). `??`: путь --json может нести
+// key в самом объекте, отсутствие флага не должно его затирать.
+if (event) {
+    event.key = one('key') ?? event.key;
+}
 // Ошибки разбора — до отправки: лучше внятно сказать, что не так с командой,
 // чем прислать сообщение с «true» вместо ссылки или 🔴 на успешном деплое.
 if (parseErrors.length > 0) {

package/dist/events.d.ts

@@ -16,8 +16,9 @@ export type Project = 'playhub' | 'one-q' | 'arvent' | 'game-publisher' | 'vault
  * поздней карточкой того же ключа?» без сравнения человеческих формулировок,
  * которые меняются. Необязателен: без него ключ выводится из типа и заголовка
  * (см. `render.ts`), но выведенный наследует хрупкость формулировки — наши
- * регулярные отправители передают его явно. В stdout CLI ключ не попадает
- * никогда: сторож на VPS разбирает stdout по словам `sent|failed|skipped`.
+ * регулярные отправители передают его явно. В вывод CLI (stderr, и его же
+ * сторож на VPS читает объединённым потоком по словам `sent|failed|skipped`)
+ * ключ не попадает никогда — новое слово там ослепило бы сторожа.
  */
 type Keyed = {
     key?: string;

package/dist/render.js

@@ -175,7 +175,11 @@ const RENDERERS = {
 const slug = (raw) => raw
     .toLowerCase()
     .replace(/[^\p{L}\p{N}]+/gu, '-')
-    .replace(/^-+|-+$/g, '');
+    .replace(/^-+|-+$/g, '')
+    // Ключ — идентификатор, не пересказ: без среза тег из длинного заголовка
+    // съедал бюджет caption до отрицательного, и slice с минусом возвращал
+    // почти весь текст — Telegram отвечал постоянным 400, файл терялся.
+    .slice(0, 60);
 export const eventKey = (e) => {
     const fallback = () => {
         switch (e.type) {
@@ -215,6 +219,6 @@ export const render = (e) => {
     // clampMessage может выйти за переданный limit на хвост закрывающих тегов и
     // многоточие — минус 40 оставляет ему этот запас. У сообщений свой запас уже
     // есть (4000 против 4096 у Telegram), у caption лимит 1024 настоящий.
-    const budget = e.type === 'file' ? 1024 - tag.length - 40 : 4000 - tag.length - 1;
+    const budget = Math.max(64, e.type === 'file' ? 1024 - tag.length - 40 : 4000 - tag.length - 1);
     return `${clampMessage(renderer(e), budget)}\n${tag}`;
 };

package/dist/send.d.ts

@@ -1,16 +1,5 @@
 import type { NotifyEvent, Project } from './events.ts';
 export type SendResult = 'sent' | 'skipped' | 'failed';
-/**
- * Отправляет событие во все его цели (тема проекта + при необходимости
- * `incidents` + чат команды). Цели идут последовательно; провал одной не
- * отменяет остальные. `'sent'`, если хотя бы одна цель получила сообщение.
- *
- * Неизвестный проект по-прежнему НЕ роняет вызвавший крон (код возврата не
- * меняется) — но и не исчезает молча: в mac-config Ops уходит красная
- * карточка. Дважды этот класс провала жил незамеченным неделями: «vault» и
- * «mac-config» до 04.08, отчёты Alitools — до 18.08. Рекурсия невозможна:
- * карточка-ошибка адресована mac-config, который в ROUTES есть всегда.
- */
 export declare const notify: (e: NotifyEvent) => Promise<SendResult>;
 /**
  * Готовый HTML во вкладку «Ops» проекта — ТОЛЬКО для дневных отчётов.
@@ -29,4 +18,4 @@ export declare const notify: (e: NotifyEvent) => Promise<SendResult>;
  *
  * Всегда беззвучно: отчёт читают утром, а не по звонку.
  */
-export declare const sendReport: (project: Project, html: string) => Promise<SendResult>;
+export declare const sendReport: (project: Project, html: string, key?: string) => Promise<SendResult>;

package/dist/send.js

@@ -224,24 +224,28 @@ const sendFile = async (e) => {
         return 'skipped';
     }
     const caption = render(e);
+    const results = [];
+    // Контракт тот же, что у deliver: провал одной цели не отменяет остальные.
     for (const target of where) {
         let waitMs = 0;
+        let got = 'failed';
         for (let i = 0; i < MAX_ATTEMPTS; i++) {
             if (waitMs > 0) {
                 await sleep(waitMs);
             }
             const result = await sendFileOnce(token, target, e, caption);
             if (result.outcome === 'ok') {
-                return 'sent';
+                got = 'sent';
+                break;
             }
             if (result.outcome === 'fail') {
-                return 'failed';
+                break;
             }
             waitMs = result.waitMs;
         }
+        results.push(got);
     }
-    log('исчерпаны попытки отправки файла');
-    return 'failed';
+    return results.includes('sent') ? 'sent' : 'failed';
 };
 /**
  * Отправляет событие во все его цели (тема проекта + при необходимости
@@ -254,17 +258,25 @@ const sendFile = async (e) => {
  * «mac-config» до 04.08, отчёты Alitools — до 18.08. Рекурсия невозможна:
  * карточка-ошибка адресована mac-config, который в ROUTES есть всегда.
  */
+const reportLostProject = async (project, kind) => {
+    // Локальный лог называет и допустимые написания — это единственная
+    // диагностика, доступная на машине, где случилась опечатка.
+    log(`неизвестный проект «${String(project)}» — известны: ${Object.keys(ROUTES).join(', ')}`);
+    const lost = {
+        type: 'job',
+        project: 'mac-config',
+        job: 'notify: событие потеряно',
+        status: 'fail',
+        note: `проект «${String(project)}» не в ROUTES — событие «${kind}» никуда не доставлено`,
+        key: 'notify-unknown-project'
+    };
+    await deliver(targets(lost), render(lost)).catch(() => undefined);
+};
 export const notify = async (e) => {
-    if (!(e.project in ROUTES)) {
-        const lost = {
-            type: 'job',
-            project: 'mac-config',
-            job: 'notify: событие потеряно',
-            status: 'fail',
-            note: `проект «${String(e.project)}» не в ROUTES — событие «${String(e.type)}» никуда не доставлено`,
-            key: 'notify-unknown-project'
-        };
-        await deliver(targets(lost), render(lost)).catch(() => undefined);
+    // Object.hasOwn, не `in`: `in` ходит по цепочке прототипов, и --project
+    // toString/constructor проходил бы гвард, терял событие И карточку о потере.
+    if (!Object.hasOwn(ROUTES, e.project)) {
+        await reportLostProject(e.project, String(e.type));
         return 'skipped';
     }
     if (e.type === 'file') {
@@ -289,11 +301,14 @@ export const notify = async (e) => {
  *
  * Всегда беззвучно: отчёт читают утром, а не по звонку.
  */
-export const sendReport = async (project, html) => {
-    const forum = ROUTES[project];
-    if (!forum) {
-        log(`неизвестный проект «${project}» — отчёт не отправлен`);
+export const sendReport = async (project, html, key) => {
+    if (!Object.hasOwn(ROUTES, project)) {
+        // Та же красная карточка, что у notify(): дверь свободного HTML — не
+        // лазейка для молчаливой потери (этот класс уже жил неделями дважды).
+        await reportLostProject(project, 'report');
         return 'skipped';
     }
-    return deliver([{ chat: forum.chat, thread: forum.ops, silent: true }], clampMessage(html));
+    const forum = ROUTES[project];
+    const tag = key ? `\n<code>#${project}/${key}</code>` : '';
+    return deliver([{ chat: forum.chat, thread: forum.ops, silent: true }], clampMessage(html) + tag);
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mikitasazan/notify",
-  "version": "1.1.1",
+  "version": "1.1.3",
   "description": "Единая типизированная отправка Telegram-уведомлений (форум-темы, маршрутизация, ретраи) для всех проектов",
   "type": "module",
   "license": "MIT",