@andrey4emk/npm-app-back-b24 5.2.0 → 5.2.1

package/README.md

@@ -242,7 +242,7 @@ const sent = await fetchWithTimeout("https://api.example.com/send", { method: "P
 
     **Уровни логов.** Промежуточные попытки пишутся на `warn`, окончательный отказ — на `error`: исчерпание всех попыток и отмена повтора гейтом. Так один упавший вызов даёт одно сообщение в Телеграм-группу через ретранслятор вместо пяти, но при этом не теряется совсем. Логировать провал обязан сам Proxy: `errorB24()`, `Event` и `Smsgold` ошибку только возвращают вызывающему коду и в лог не пишут.
 
-    **Создающие вызовы не повторяются Proxy-слоем.** Если в вызове есть метод, создающий сущность или запускающий действие (`*.add`, `*.create`, `*.start`, `*.send`, `*.uploadfile`, `*.import`, `*.register` — в том числе среди команд `callBatch`), retry отменяется, а в лог идёт `повтор отменён`. Причина: транспортная ошибка не говорит, дошёл ли запрос до портала. Обрыв приходит и когда соединение не состоялось, и когда оно случилось после того, как портал уже принял и выполнил запрос — а повтор во втором случае создаёт вторую задачу, второй файл, второй экземпляр бизнес-процесса. Идемпотентные записи (`*.update`, `*.delete`, `*.set`) повторяются как прежде: повторное применение даёт то же состояние.
+    **Создающие вызовы не повторяются Proxy-слоем.** Если в вызове есть метод, создающий сущность или запускающий действие (`*.add`, `*.create`, `*.start`, `*.send`, `*.upload`, `*.uploadfile`, `*.import`, `*.register` — в том числе среди команд `callBatch`), retry отменяется, а в лог идёт `повтор отменён`. Причина: транспортная ошибка не говорит, дошёл ли запрос до портала. Обрыв приходит и когда соединение не состоялось, и когда оно случилось после того, как портал уже принял и выполнил запрос — а повтор во втором случае создаёт вторую задачу, второй файл, второй экземпляр бизнес-процесса. Идемпотентные записи (`*.update`, `*.delete`, `*.set`) повторяются как прежде: повторное применение даёт то же состояние. Загрузка файла в чат (`imbot.v2.File.upload`, `im.v2.File.upload`) под защитой с выпуска после 5.2.0: раньше обрыв связи повторял её до пяти раз, и в чат приходили одинаковые файлы.
 
     Ключевое слово ищется **в любой позиции** имени, а не только в конце: `imconnector.send.messages` и `imconnector.send.status.delivery` тоже под гейтом. После ключевого слова обязателен разделитель (`.`, `?` или конец строки), поэтому `crm.deal.addcustom` и `landing.landing.addbytemplate` повторяются как прежде.
 

package/bitrix24/b24/retry.ts

@@ -39,7 +39,7 @@ export const NETWORK_ERROR_CODES = new Set(["NETWORK_ERROR", "REQUEST_TIMEOUT",
 
 /**
  * REST-методы B24, повтор которых создаёт дубликат сущности или запускает
- * повторное действие: add, create, start, send, uploadfile, import, register.
+ * повторное действие: add, create, start, send, upload, uploadfile, import, register.
  *
  * Транспортная ошибка не говорит, дошёл ли запрос до портала: обрыв приходит
  * и когда соединение не состоялось, и когда оно оборвалось после того, как портал уже
@@ -66,8 +66,14 @@ export const NETWORK_ERROR_CODES = new Set(["NETWORK_ERROR", "REQUEST_TIMEOUT",
  * перестанет, и это осознанная плата за правило без списка исключений: потеря отметки
  * о доставке косметическая, переписку она не ломает, а ручной список молча устаревал бы
  * при появлении нового метода портала — ровно тот дефект, которым T-48 и был.
+ *
+ * Слово `upload` добавлено задачей T-73: `imbot.v2.File.upload` и `im.v2.File.upload`
+ * загружают файл в чат, а в перечне было только `uploadfile`, и обрыв связи давал до пяти
+ * одинаковых файлов в чате. Разделитель после слова по-прежнему обязателен, поэтому
+ * `upload` не срабатывает внутри `uploadfile` (тот ловится своим словом) и внутри
+ * имён вида `crm.deal.uploadcustom`.
  */
-export const NON_IDEMPOTENT_METHOD_RE = /\.(add|create|start|send|uploadfile|import|register)(\.|\?|$)/i;
+export const NON_IDEMPOTENT_METHOD_RE = /\.(add|create|start|send|upload|uploadfile|import|register)(\.|\?|$)/i;
 
 /**
  * Методы, повтор которых теряет данные, хотя сущностей не создаёт.
@@ -450,9 +456,13 @@ export async function runWithRetry<T>(fn: () => Promise<T>, label: string, block
             // потеряться уже после того, как портал его выполнил. Различить нельзя,
             // поэтому вызовы под ограничением не повторяем — лучше вернуть ошибку,
             // чем создать дубликат или потерять зарезервированный пакет
-            // Отмена только резервирующего вызова (event.offline.get) — warn: приложение и так
-            // пишет свою строку error о том же сбое, а две строки дали бы два сообщения в
-            // Телеграме. Отмена создающего остаётся error: сущность могла не создаться
+            // Отмена только резервирующего вызова (event.offline.get) — warn: повтор потерю не
+            // исправит, а зависшие пакеты подбирает само приложение (b24_conector#T-97). Строки
+            // приложений о транспортном обрыве (socket hang up, ECONNRESET, таймаут) несут сетевой
+            // признак и уходят только сводкой, поэтому одиночный обрыв виден лишь в Loki — это
+            // принято сознательно. Строка об ответе без JSON («портал вернул не JSON: HTTP 200,
+            // text/html…») сетевого признака не несёт и уходит в Телеграм сразу.
+            // Отмена создающего остаётся error: сущность могла не создаться
             if (blockReason && !proven) {
                 const level = blockReason.includes(CREATING_REASON_HEAD) ? "error" : "warn";
                 logs.add(`${label}${code}: повтор отменён, ${blockReason} — ${msg}`, level);

package/logs/logs.ts

@@ -78,9 +78,16 @@ const NETWORK_FAILURE_RE =
 
 /**
  * Строки с сетевым признаком, которые уходят сразу и по одной: отменённое создание
- * сущности, провал обмена токена, неизвестный статус доставки — их разбирает человек
+ * сущности, провал обмена токена, неизвестный статус доставки, потерянное после повторов
+ * сообщение («[tg_1] Сообщение оператора не доставлено после 5 повторных попыток…: fetch failed»
+ * из b24_conector) — их разбирает человек.
+ * Фраза «не доставлено» выводит из сводки и вторую строку b24_conector:
+ * «[<ключ>] Сообщение не доставлено в Битрикс24: соединение не состоялось (id: N): getaddrinfo ENOTFOUND …»
+ * (reportSendFailure в server/api/telegram/b24MessageBuffer.ts, уровень error). Это тоже настоящая
+ * потеря сообщения, так задумано. Следствие: если портал долго недоступен, в Телеграм придёт
+ * по одному сообщению на каждое входящее сообщение клиента
  */
-const NETWORK_KEEP_RE = /вызов создаёт сущности|обновление токена не удалось|статус неизвестен/;
+const NETWORK_KEEP_RE = /вызов создаёт сущности|обновление токена не удалось|статус неизвестен|не доставлено/;
 
 /** Окно, за которое считаются сетевые сбои, и пауза между сводками, мс */
 const NETWORK_WINDOW_MS = 15 * 60_000;

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@andrey4emk/npm-app-back-b24",
-    "version": "5.2.0",
+    "version": "5.2.1",
     "description": "Bitrix24 OAuth helpers for Node.js projects",
     "main": "index.ts",
     "type": "module",