@mikitasazan/notify 1.0.0 → 1.0.2

package/README.md

@@ -1,7 +1,7 @@
 # @mikitasazan/notify
 
 Единая типизированная отправка Telegram-уведомлений: **форум на проект**,
-внутри вкладки «⚙️ Ops» (роботы) и «💬 Dev» (люди), один бот, семь типов
+внутри вкладки «⚙️ Ops» (роботы) и «💬 Dev» (люди), один бот, восемь типов
 событий. Заменяет 18 разных имён переменных и 4+ независимых реализации
 `jq | curl`, что были в playhub, arvent, game-publisher до 26.07.2026.
 
@@ -22,9 +22,11 @@
 проект значит дописать одну строку в `src/routes.ts`, больше нигде ничего
 заводить не нужно.
 
-**Ноль рантайм-зависимостей.** Node 22.18+ / 24+ исполняет `.ts` напрямую
-(проверено на macOS и на проде playhub, Node 22.22.2) — сборки нет,
-в GitHub Actions ставить нечего.
+**Ноль рантайм-зависимостей.** Composite-action и консьюмеры не собирают ничего:
+Node 22.18+ / 24+ исполняет `.ts` напрямую (проверено на macOS и на проде
+playhub, Node 22.22.2), в GitHub Actions ставить нечего. Релиз пакета в npm
+**собирается** в `dist` (`prepare` → `tsc`) — поэтому npm-консьюмеры получают
+скомпилированный JS и типы.
 
 ## Установка
 

package/dist/cli.js

@@ -70,6 +70,16 @@ for (let i = 1; i < args.length; i++) {
     flags.set(key, [...(flags.get(key) ?? []), next]);
 }
 const one = (key) => flags.get(key)?.[0];
+// Число с явной ошибкой разбора, иначе рендер рисовал «PR #NaN».
+const num = (key) => {
+    const raw = one(key);
+    const n = Number(raw);
+    if (raw === undefined || Number.isNaN(n)) {
+        parseErrors.push(`--${key}: ожидается число, получено «${raw ?? ''}»`);
+        return 0;
+    }
+    return n;
+};
 // --item "текст" или --item "текст|https://ссылка"
 const items = () => (flags.get('item') ?? []).map((raw) => {
     const idx = raw.lastIndexOf('|');
@@ -85,7 +95,6 @@ const PR_ALIASES = {
     reopened: 'opened',
     ready_for_review: 'ready_for_review',
     review_requested: 'review_requested',
-    review_request_removed: 'review_requested',
     approved: 'approved',
     changes_requested: 'changes_requested',
     merged: 'merged',
@@ -132,7 +141,9 @@ let event;
 if (flags.has('json')) {
     try {
         const payload = JSON.parse(readFileSync(0, 'utf-8'));
-        event = { type: command, ...payload };
+        // type — за командой, не за payload: иначе --pr <объект с type:deploy>
+        // отправил бы событие другого типа.
+        event = { ...payload, type: command };
     }
     catch (err) {
         log(`не удалось разобрать --json со stdin: ${err instanceof Error ? err.message : String(err)}`);
@@ -149,7 +160,8 @@ else {
                 commitUrl: one('commit-url'),
                 url: one('url'),
                 target: one('target'),
-                via: one('via')
+                via: one('via'),
+                note: one('note')
             };
             break;
         case 'job':
@@ -191,7 +203,7 @@ else {
                 type: 'pr',
                 project: project(),
                 action: prAction(one('action')),
-                number: Number(one('number')),
+                number: num('number'),
                 title: one('title') ?? '(без заголовка)',
                 author: one('author'),
                 reviewer: one('reviewer'),
@@ -203,7 +215,7 @@ else {
                 type: 'issue',
                 project: project(),
                 action: issueAction(one('action')),
-                number: Number(one('number')),
+                number: num('number'),
                 title: one('title') ?? '(без заголовка)',
                 author: one('author'),
                 assignee: one('assignee'),

package/dist/events.d.ts

@@ -41,6 +41,8 @@ export type NotifyEvent =
      * какой сработал.
      */
     via?: string;
+    /** Пояснение: почему отменён/пропущен прогон после деплоя. */
+    note?: string;
 }
 /** Регулярная задача по расписанию: импорт игр, бэкап БД, валидатор. */
  | {
@@ -122,5 +124,5 @@ export type NotifyEvent =
     expected?: string;
 };
 export type EventType = NotifyEvent['type'];
-/** Красное = со звуком и с дублем в тему инцидентов. Всё остальное — тихо. */
+/** Красное = со звуком. Всё остальное — тихо. (Отдельной темы «инциденты» больше нет — авария видна в ленте проекта.) */
 export declare const severity: (e: NotifyEvent) => "info" | "error";

package/dist/events.js

@@ -9,7 +9,7 @@
  *   - обязательные поля не добавляются никогда — только новый тип события.
  * Тогда старый вызывающий код и новый пакет совместимы в обе стороны.
  */
-/** Красное = со звуком и с дублем в тему инцидентов. Всё остальное — тихо. */
+/** Красное = со звуком. Всё остальное — тихо. (Отдельной темы «инциденты» больше нет — авария видна в ленте проекта.) */
 export const severity = (e) => {
     if (e.type === 'incident' || e.type === 'heartbeat_miss') {
         return 'error';

package/dist/render.js

@@ -70,6 +70,7 @@ const renderDeploy = (e) => {
         commitLine,
         kv('откуда', e.via),
         kv('куда', e.target),
+        kv('примечание', e.note),
         link(e.url, 'Открыть логи')
     ]);
 };

package/dist/send.js

@@ -100,8 +100,13 @@ const attempt = async (token, target, text) => {
             log('таймаут ответа — не повторяем: сообщение могло уже уйти');
             return { outcome: 'fail' };
         }
-        // Сюда попадают отказы соединения (DNS, TLS, сеть недоступна) — запрос не
-        // ушёл, дубля быть не может, фолбэк безопасен.
+        // Сюда попадают отказы соединения (DNS, TLS, сеть недоступна) — запрос
+        // почти наверняка не ушёл, и curl-фолбэк безопасен. НЕ абсолютно: обрыв
+        // (reset/truncation) ПОСЛЕ того, как Telegram принял POST, тоже кидает
+        // исключение — тогда повтор даст дубль. Это редкий случай; полное
+        // отсутствие дублей невозможно без idempotency-key у Bot API (его нет).
+        // Логика прежняя: дубль на редком reset — меньшее зло, чем потеря
+        // сообщения на частом сетевом сбое.
         log('fetch не прошёл, пробуем curl…');
         const curl = sendViaCurl(token, target, text);
         return curl === 'retry' ? { outcome: 'retry', waitMs: 1000 } : { outcome: curl };
@@ -133,6 +138,14 @@ const deliver = async (where, text) => {
         log('нет OPS_BOT_TOKEN — сообщение не отправлено');
         return 'skipped';
     }
+    // Токен interpolируется в URL и в curl-конфиг (`url = "...bot${token}..."`).
+    // Валидный токен Telegram — это `\d+:[\w-]+`; что-либо с кавычкой/переводом
+    // строки/`?` сломало бы разбор (инъекция директивы curl или query-хвост).
+    // Это требует покорёженного секрета, но проверка копеечная.
+    if (!/^\d+:[A-Za-z0-9_-]+$/.test(token)) {
+        log('OPS_BOT_TOKEN не похож на токен Telegram — отправка отменена');
+        return 'skipped';
+    }
     if (where.length === 0) {
         return 'skipped';
     }

package/dist/setup.js

@@ -14,18 +14,25 @@
  */
 const log = (msg) => console.error(`[notify] ${msg}`);
 const createTopic = async (token, chat, name, color) => {
-    const res = await fetch(`https://api.telegram.org/bot${token}/createForumTopic`, {
-        method: 'POST',
-        headers: { 'Content-Type': 'application/json' },
-        body: JSON.stringify({ chat_id: chat, name, icon_color: color }),
-        signal: AbortSignal.timeout(10_000)
-    });
-    const body = (await res.json());
-    if (!body.ok || !body.result) {
-        log(`не удалось создать «${name}»: ${body.description ?? `HTTP ${res.status}`}`);
+    try {
+        const res = await fetch(`https://api.telegram.org/bot${token}/createForumTopic`, {
+            method: 'POST',
+            headers: { 'Content-Type': 'application/json' },
+            body: JSON.stringify({ chat_id: chat, name, icon_color: color }),
+            signal: AbortSignal.timeout(10_000)
+        });
+        const body = (await res.json());
+        if (!body.ok || !body.result) {
+            log(`не удалось создать «${name}»: ${body.description ?? `HTTP ${res.status}`}`);
+            return null;
+        }
+        return body.result.message_thread_id;
+    }
+    catch (err) {
+        // Сеть/таймаут/нечитаемый ответ — не роняем CLI (его контракт: всегда exit 0).
+        log(`не удалось создать «${name}»: ${err instanceof Error ? err.message : String(err)}`);
         return null;
     }
-    return body.result.message_thread_id;
 };
 export const setupTopic = async (chatId, projectKey) => {
     const token = process.env.OPS_BOT_TOKEN?.trim();
@@ -35,8 +42,16 @@ export const setupTopic = async (chatId, projectKey) => {
     }
     const ops = await createTopic(token, chatId, '⚙️ Ops', 9367192);
     const dev = await createTopic(token, chatId, '💬 Dev', 7322096);
-    if (ops === null || dev === null) {
-        log('проверь: бот админ группы с правом «Управление темами», а темы в группе включены?');
+    // Частичный успех: если создался только Ops — печатаем его, иначе повторный
+    // запуск создал бы ДРУГУЮ тему Ops, а старый id потерялся бы.
+    if (ops === null) {
+        log('Ops не создан — проверь: бот админ группы с правом «Управление темами», темы включены?');
+        return;
+    }
+    if (dev === null) {
+        log(`Ops создан (id=${ops}), Dev нет — добавь Dev вручную или повтори, и возьми ops=${ops}`);
+        log('добавь в src/routes.ts (dev подставь после):');
+        log(`  ${JSON.stringify(projectKey)}: { chat: '${chatId}', ops: ${ops}, dev: <вставь> },`);
         return;
     }
     log(`вкладки созданы: Ops=${ops}, Dev=${dev}`);

package/package.json

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