@mikitasazan/notify 1.4.0 → 1.4.1

package/dist/cli.js

@@ -26,13 +26,25 @@ import { render } from "./render.js";
 import { notify } from "./send.js";
 import { setupTopic } from "./setup.js";
 const log = (msg) => console.error(`[notify] ${msg}`);
+/**
+ * Anything taken from the caller and echoed into a message goes through this.
+ *
+ * `sent`, `failed` and `skipped` are a CONTRACT: `notify-fail.sh` greps
+ * `^\[notify\] sent$`, the server-side silence watchdog matches `*sent*`, and
+ * `action.yml` warns on `*failed*|*skipped*`. An input value carrying one of
+ * those words puts it on stderr — `notify sent --project=x` printed
+ * `unknown event type: sent`, and the watchdog read a card that was never
+ * delivered as delivered, then stopped repeating the alarm. Found by Codex,
+ * 25.08.2026.
+ */
+const safe = (v) => String(v ?? '').replace(/\b(sent|failed|skipped)\b/gi, (w) => `${w[0]}·${w.slice(1)}`);
 const args = process.argv.slice(2);
 const command = args[0];
 if (command === 'setup') {
     const [, chatId, projectKey] = args;
     if (!chatId || !projectKey) {
-        log('использование: notify setup <chat_id форума> <ключ-проекта>');
-        log('  сначала создай группу, включи в ней «Темы» и добавь бота админом');
+        log('usage: notify setup <forum chat_id> <project key>');
+        log('  create the group first, turn Topics on in it, add the bot as admin');
         process.exit(0);
     }
     await setupTopic(chatId, projectKey);
@@ -45,7 +57,7 @@ const BOOLEAN_FLAGS = new Set(['json', 'recovered', 'dry-run']);
 for (let i = 1; i < args.length; i++) {
     const arg = args[i];
     if (!arg.startsWith('--')) {
-        parseErrors.push(`stray argument with no flag: "${arg}"`);
+        parseErrors.push(`stray argument with no flag: "${safe(arg)}"`);
         continue;
     }
     // Форма `--key=value` обязательна для значений, начинающихся с `--`
@@ -67,7 +79,7 @@ for (let i = 1; i < args.length; i++) {
     // ТЕРЯЛОСЬ ВСЁ сообщение, а `--status` без значения рисовал 🔴 на
     // успешном деплое. Теперь это явная ошибка разбора.
     if (next === undefined || next.startsWith('--')) {
-        parseErrors.push(`flag --${key} with no value`);
+        parseErrors.push(`flag --${safe(key)} with no value`);
         continue;
     }
     i++;
@@ -76,7 +88,7 @@ for (let i = 1; i < args.length; i++) {
 const one = (key) => flags.get(key)?.[0];
 for (const key of flags.keys()) {
     if (!KNOWN_FLAGS.has(key)) {
-        parseErrors.push(`unknown flag --${key}`);
+        parseErrors.push(`unknown flag --${safe(key)}`);
     }
 }
 // Число с явной ошибкой разбора, иначе рендер рисовал «PR #NaN».
@@ -84,7 +96,7 @@ const num = (key) => {
     const raw = one(key);
     const n = Number(raw);
     if (raw === undefined || Number.isNaN(n)) {
-        parseErrors.push(`--${key}: ожидается число, получено «${raw ?? ''}»`);
+        parseErrors.push(`--${safe(key)}: expected a number, got "${safe(raw)}"`);
         return 0;
     }
     return n;
@@ -121,7 +133,7 @@ const ISSUE_ALIASES = {
 const prAction = (raw) => {
     const hit = PR_ALIASES[(raw ?? '').toLowerCase()];
     if (!hit) {
-        parseErrors.push(`--action: unknown PR action "${raw ?? ''}" (${Object.keys(PR_ALIASES).join(', ')})`);
+        parseErrors.push(`--action: unknown PR action "${safe(raw)}" (${Object.keys(PR_ALIASES).join(', ')})`);
         return 'opened';
     }
     return hit;
@@ -166,7 +178,7 @@ if (flags.has('json')) {
     catch (err) {
         // Тоже в parseErrors: обе аналитики зовут CLI через `|| true`, и молчащий
         // разбор JSON означал бы зелёный крон без дневного отчёта.
-        parseErrors.push(`--json from stdin did not parse: ${err instanceof Error ? err.message : String(err)}`);
+        parseErrors.push(`--json from stdin did not parse: ${safe(err instanceof Error ? err.message : err)}`);
     }
 }
 else {
@@ -192,7 +204,7 @@ else {
             event = {
                 type: 'job',
                 project: project(),
-                job: one('job') ?? '(без имени)',
+                job: one('job') ?? '(no name)',
                 status: jobStatus(),
                 stats: pairs('stat'),
                 items: items(),
@@ -206,7 +218,7 @@ else {
             event = {
                 type: 'report',
                 project: project(),
-                title: one('title') ?? '(без заголовка)',
+                title: one('title') ?? '(no title)',
                 period: one('period'),
                 lines: pairs('line'),
                 items: items(),
@@ -236,7 +248,7 @@ else {
                 project: project(),
                 action: prAction(one('action')),
                 number: num('number'),
-                title: one('title') ?? '(без заголовка)',
+                title: one('title') ?? '(no title)',
                 body: one('body'),
                 author: one('author'),
                 reviewer: one('reviewer'),
@@ -249,7 +261,7 @@ else {
                 project: project(),
                 action: issueAction(one('action')),
                 number: num('number'),
-                title: one('title') ?? '(без заголовка)',
+                title: one('title') ?? '(no title)',
                 body: one('body'),
                 author: one('author'),
                 assignee: one('assignee'),
@@ -260,7 +272,7 @@ else {
             event = {
                 type: 'incident',
                 project: project(),
-                title: one('title') ?? '(без заголовка)',
+                title: one('title') ?? '(no title)',
                 detail: one('detail'),
                 logs: one('logs'),
                 url: one('url')
@@ -270,7 +282,7 @@ else {
             event = {
                 type: 'heartbeat_miss',
                 project: project(),
-                job: one('job') ?? '(без имени)',
+                job: one('job') ?? '(no name)',
                 lastSeen: one('last-seen'),
                 expected: one('expected'),
                 recovered: flags.has('recovered'),
@@ -280,12 +292,12 @@ else {
         case 'file': {
             const path = one('path');
             if (!path) {
-                parseErrors.push('--path: обязателен для file');
+                parseErrors.push('--path: required for a file event');
             }
             event = {
                 type: 'file',
                 project: project(),
-                title: one('title') ?? '(без заголовка)',
+                title: one('title') ?? '(no title)',
                 path: path ?? '',
                 filename: one('filename'),
                 note: one('note')
@@ -296,7 +308,7 @@ else {
             // В parseErrors, а не просто в лог: иначе неизвестный тип уходил в
             // тишину — событие не собиралось, ошибок разбора не было, и CLI выходил
             // нулём, ничего не отправив и ничего об этом не сказав.
-            parseErrors.push(`unknown event type: ${command ?? '(none given)'}`);
+            parseErrors.push(`unknown event type: ${safe(command ?? '(none given)')}`);
     }
 }
 // --key применим к любому типу — одна точка вместо строки в каждом case
@@ -343,7 +355,7 @@ if (event) {
     catch (err) {
         // Слово `failed` — контракт, тот же, что у ошибки разбора выше. Без него
         // исключение при отправке читалось сторожами как «ничего не случилось».
-        log(`failed: ${err instanceof Error ? err.message : String(err)}`);
+        log(`failed: ${safe(err instanceof Error ? err.message : err)}`);
     }
 }
 process.exit(0);

package/dist/events.d.ts

@@ -49,12 +49,12 @@ export type NotifyEvent = Keyed & (
     commit?: string;
     /** Ссылка на коммит — строка «коммит» становится кликабельной. */
     commitUrl?: string;
-    /** Заголовок коммита — рендерится рядом с телом в цитате. */
+    /** Заголовок коммита — рендерится полем `Title:`, тело идёт цитатой ниже. */
     commitTitle?: string;
     /** Тело коммита, если есть — та же цитата, что и заголовок. */
     commitBody?: string;
     workflowUrl?: string;
-    /** Название прогона для видимого текста ссылки (по умолчанию — просто "run"). */
+    /** Название прогона для видимого текста ссылки (по умолчанию — `open`). */
     workflowName?: string;
     url?: string;
     /**
@@ -85,7 +85,7 @@ export type NotifyEvent = Keyed & (
     items?: Item[];
     note?: string;
     workflowUrl?: string;
-    /** Название прогона для видимого текста ссылки (по умолчанию — просто "run"). */
+    /** Название прогона для видимого текста ссылки (по умолчанию — `open`). */
     workflowName?: string;
     /**
      * Запасное имя для ссылки на прогон: половина отправителей шлёт её как
@@ -139,9 +139,9 @@ export type NotifyEvent = Keyed & (
      * schedule, a manual press. Renders as `Reason:`, same as on deploy.
      */
     note?: string;
-    /** Ссылка на прогон (workflow run) — отдельно от `url`, который у CI не используется. */
+    /** Ссылка на прогон (workflow run) — отдельно от `url` — запасной для `workflowUrl`. */
     workflowUrl?: string;
-    /** Название прогона для видимого текста ссылки (по умолчанию — просто "run"). */
+    /** Название прогона для видимого текста ссылки (по умолчанию — `open`). */
     workflowName?: string;
     url?: string;
 }

package/dist/render.js

@@ -46,12 +46,27 @@ export const clampMessage = (text, limit = 4000) => {
     // Telegram отвечает на такое 400 — то есть карточка пропадала целиком, а
     // отправитель с `|| true` этого не замечал. Нашёл Codex; воспроизводится
     // отчётом с именем группы в 5000 знаков.
-    const tail = ['b', 'a', 'i', 'u', 'code', 'blockquote']
-        .filter((t) => {
-        const opened = (body.match(new RegExp(`<${t}[ >]`, 'g')) ?? []).length;
-        const closed = (body.match(new RegExp(`</${t}>`, 'g')) ?? []).length;
-        return opened > closed;
-    })
+    //
+    // Порядок закрытия — обратный порядку ОТКРЫТИЯ, а не фиксированный список.
+    // Фиксированный список закрывал `<i><u>` как `</i></u>`: тегов поровну,
+    // счётчик сходится, вложенность нарушена, и Telegram отвечает тем же 400.
+    // Второй заход того же бага (25.08.2026), поэтому теперь порядок берётся из
+    // самого текста: последний открытый закрывается первым.
+    const open = [];
+    const tagRe = /<(\/?)(b|a|i|u|code|blockquote)[ >]/g;
+    for (let m = tagRe.exec(body); m !== null; m = tagRe.exec(body)) {
+        if (m[1] === '/') {
+            const at = open.lastIndexOf(m[2]);
+            if (at !== -1) {
+                open.splice(at, 1);
+            }
+        }
+        else {
+            open.push(m[2]);
+        }
+    }
+    const tail = open
+        .reverse()
         .map((t) => `</${t}>`)
         .join('');
     return `${body}${tail}\n…`;
@@ -409,7 +424,7 @@ export const render = (e) => {
     // строка, и неизвестное значение роняло процесс через `renderer is not a
     // function`. Падать из-за уведомления нельзя.
     if (typeof renderer !== 'function') {
-        throw new Error(`неизвестный тип события: ${String(e.type)}`);
+        throw new Error(`unknown event type: ${String(e.type)}`);
     }
     const tags = tagsLine(e);
     // clampMessage может выйти за переданный limit на хвост закрывающих тегов и

package/dist/send.js

@@ -89,7 +89,7 @@ const attempt = async (token, target, text) => {
         // почему уведомления пропали, невозможно — а разбираться будет не
         // разработчик, а владелец.
         const detail = (await res.json().catch(() => null));
-        log(`HTTP ${res.status}: ${detail?.description ?? 'без описания'} — не повторяем, ошибка постоянная`);
+        log(`HTTP ${res.status}: ${detail?.description ?? 'no description'} — permanent error, not retried`);
         return { outcome: 'fail' };
     }
     catch (err) {
@@ -99,7 +99,7 @@ const attempt = async (token, target, text) => {
         // на таймауте останавливаемся и честно пишем 'failed': лишняя копия аварии
         // хуже, чем пропущенная строка в логе, а сообщение, скорее всего, ушло.
         if (err instanceof Error && err.name === 'TimeoutError') {
-            log('таймаут ответа — не повторяем: сообщение могло уже уйти');
+            log('answer timed out — not retried: the message may already be out');
             return { outcome: 'fail' };
         }
         // Сюда попадают отказы соединения (DNS, TLS, сеть недоступна) — запрос
@@ -109,7 +109,7 @@ const attempt = async (token, target, text) => {
         // отсутствие дублей невозможно без idempotency-key у Bot API (его нет).
         // Логика прежняя: дубль на редком reset — меньшее зло, чем потеря
         // сообщения на частом сетевом сбое.
-        log('fetch не прошёл, пробуем curl…');
+        log('fetch did not go through, trying curl…');
         const curl = sendViaCurl(token, target, text);
         return curl === 'retry' ? { outcome: 'retry', waitMs: 1000 } : { outcome: curl };
     }
@@ -130,14 +130,14 @@ const sendOne = async (token, target, text) => {
         }
         waitMs = result.waitMs;
     }
-    log('исчерпаны попытки отправки');
+    log('out of send attempts');
     return 'failed';
 };
 /** Общий хвост для `notify` и `sendReport`: токен, цели, последовательная отправка. */
 const deliver = async (where, text) => {
     const token = process.env.OPS_BOT_TOKEN?.trim();
     if (!token) {
-        log('нет OPS_BOT_TOKEN — сообщение не отправлено');
+        log('skipped: no OPS_BOT_TOKEN, the message was not sent');
         return 'skipped';
     }
     // Токен interpolируется в URL и в curl-конфиг (`url = "...bot${token}..."`).
@@ -145,7 +145,7 @@ const deliver = async (where, text) => {
     // строки/`?` сломало бы разбор (инъекция директивы curl или query-хвост).
     // Это требует покорёженного секрета, но проверка копеечная.
     if (!/^\d+:[A-Za-z0-9_-]+$/.test(token)) {
-        log('OPS_BOT_TOKEN не похож на токен Telegram — отправка отменена');
+        log('failed: OPS_BOT_TOKEN does not look like a Telegram token, send cancelled');
         return 'skipped';
     }
     if (where.length === 0) {
@@ -196,27 +196,27 @@ const sendFileOnce = async (token, target, e, caption) => {
             return { outcome: 'retry', waitMs: 1000 };
         }
         const detail = (await res.json().catch(() => null));
-        log(`HTTP ${res.status}: ${detail?.description ?? 'без описания'} — не повторяем, ошибка постоянная`);
+        log(`HTTP ${res.status}: ${detail?.description ?? 'no description'} — permanent error, not retried`);
         return { outcome: 'fail' };
     }
     catch (err) {
         if (err instanceof Error && err.name === 'TimeoutError') {
-            log('таймаут ответа — не повторяем: файл мог уже уйти');
+            log('answer timed out — not retried: the file may already be out');
             return { outcome: 'fail' };
         }
         // Файл не читается (нет на диске, нет прав) — постоянная ошибка.
         if (err instanceof Error && 'code' in err) {
-            log(`файл не отправлен: ${err.message}`);
+            log(`failed to send the file: ${err.message}`);
             return { outcome: 'fail' };
         }
-        log(`сеть не пустила файл: ${err instanceof Error ? err.message : String(err)}`);
+        log(`the network refused the file: ${err instanceof Error ? err.message : String(err)}`);
         return { outcome: 'retry', waitMs: 1000 };
     }
 };
 const sendFile = async (e) => {
     const token = process.env.OPS_BOT_TOKEN?.trim();
     if (!token || !/^\d+:[A-Za-z0-9_-]+$/.test(token)) {
-        log('нет валидного OPS_BOT_TOKEN — файл не отправлен');
+        log('skipped: no valid OPS_BOT_TOKEN, the file was not sent');
         return 'skipped';
     }
     const where = targets(e);
@@ -261,13 +261,13 @@ const sendFile = async (e) => {
 const reportLostProject = async (project, kind) => {
     // Локальный лог называет и допустимые написания — это единственная
     // диагностика, доступная на машине, где случилась опечатка.
-    log(`неизвестный проект «${String(project)}» — известны: ${Object.keys(ROUTES).join(', ')}`);
+    log(`unknown project "${String(project)}" — known: ${Object.keys(ROUTES).join(', ')}`);
     const lost = {
         type: 'job',
         project: 'mac-config',
-        job: 'notify: событие потеряно',
+        job: 'notify: an event was lost',
         status: 'fail',
-        note: `проект «${String(project)}» не в ROUTES — событие «${kind}» никуда не доставлено`,
+        note: `project "${String(project)}" is not in ROUTES — event "${kind}" went nowhere`,
         key: 'notify-unknown-project'
     };
     await deliver(targets(lost), render(lost)).catch(() => undefined);

package/package.json

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