@andrey4emk/npm-app-back-b24 3.5.0 → 3.6.1

package/README.md

@@ -32,7 +32,7 @@ npm install @andrey4emk/npm-app-back-b24
 | --------------------------------------------- | ----------------------------- | ---------------------- |
 | `@andrey4emk/npm-app-back-b24`                | всё                           | да                     |
 | `@andrey4emk/npm-app-back-b24/logs`           | `logs`                        | нет                    |
-| `@andrey4emk/npm-app-back-b24/fetchRetry`     | `fetchRetry`, `isNetworkError`, `maskUrl` | нет        |
+| `@andrey4emk/npm-app-back-b24/fetchRetry`     | `fetchRetry`, `isNetworkError`, `isPreConnectionError`, `maskUrl` | нет |
 
 Корневой импорт — barrel: он тянет модуль OAuth, который при загрузке читает `authB24.json` и, если приложение авторизовано, запускает таймер проактивного обновления токена. Проектам, которые ходят в B24 по **входящему вебхуку** или берут из пакета только логгер и `fetchRetry`, удобнее импортировать по подпутям — тогда OAuth-модуль не загружается вообще.
 
@@ -146,7 +146,17 @@ const response = await fetchRetry("https://api.example.com/data", { method: "GET
     | Попыток        | 5                                         |
     | Задержка       | 500 мс (линейно растущая для refreshAuth) |
 
-    Retry срабатывает только при сетевых проблемах (`ECONNRESET`, `ETIMEDOUT`, `ERR_NETWORK` и т.д.). HTTP-ошибки (400, 500) и ошибки бизнес-логики Bitrix24 **не** вызывают повторных попыток. Каждая неудачная попытка логируется через `logs.add()` с уровнем `error` — в сообщение попадает код ошибки SDK.
+    Retry срабатывает только при сетевых проблемах (`ECONNRESET`, `ETIMEDOUT`, `ERR_NETWORK` и т.д.). HTTP-ошибки (400, 500) и ошибки бизнес-логики Bitrix24 **не** вызывают повторных попыток.
+
+    **Уровни логов.** Промежуточные попытки пишутся на `warn`, окончательный отказ — на `error`: исчерпание всех попыток и отмена повтора гейтом. Так один упавший вызов даёт одно сообщение в чат B24 вместо пяти, но при этом не теряется совсем. Логировать провал обязан сам Proxy: `errorB24()`, `Event` и `Smsgold` ошибку только возвращают вызывающему коду и в лог не пишут.
+
+    **Создающие вызовы не повторяются Proxy-слоем.** Если в вызове есть метод, создающий сущность или запускающий действие (`*.add`, `*.create`, `*.start`, `*.send`, `*.uploadfile`, `*.import`, `*.register` — в том числе среди команд `callBatch`), retry отменяется, а в лог идёт `повтор отменён`. Причина: при ответе портала «200 без поля `result`» (заглушка прокси или WAF) SDK падает в собственной ветке логирования и отдаёт ошибку со `status: 0`, неотличимую от транспортного сбоя — хотя запрос уже выполнен. Повтор в такой ситуации создавал до пяти дубликатов задачи или файла. Идемпотентные записи (`*.update`, `*.delete`, `*.set`) повторяются как прежде: повторное применение даёт то же состояние.
+
+    Команды `callBatch` разбираются в обеих формах — кортеж `["crm.deal.add", {...}]` (основная в SDK 2.x) и объект `{ method, params }`. Если форму команды разобрать не удалось, вызов считается создающим и не повторяется: для защиты от дубликатов безопаснее ошибиться в сторону осторожности.
+
+    Важно: **у SDK есть собственный слой ретраев**, который гейт не отменяет. При настоящем разрыве соединения (`ECONNRESET` и подобные) SDK сходит на портал до 3 раз независимо от нашей защиты. Гейт закрывает конкретный случай — `JSSDK_UNKNOWN_ERROR`, который входит в `BUILT_IN_HARD_ERROR_CODES` и самим SDK не повторяется.
+
+    **Обмен refresh-токена** повторяется по более строгому правилу — только если соединение заведомо не состоялось (см. `isPreConnectionError()`). При таймауте или обрыве сервер мог уже провести ротацию; тогда локальный `refresh_token` мёртв независимо от наших действий, и повтор не помогает, а лишь маскирует проблему серией одинаковых отказов. Следующая попытка всё равно будет: проактивный таймер повторяет через 2 минуты при времени жизни токена около часа.
 
     У самого SDK есть собственный слой ретраев (`maxRetries: 3`), но он работает только при rate limit (HTTP 429, 503) и при распознанных `NETWORK_ERROR` / `REQUEST_TIMEOUT`. Замаскированные транспортные сбои приходят с кодом `JSSDK_UNKNOWN_ERROR`, который входит в `BUILT_IN_HARD_ERROR_CODES` — на таких ошибках SDK делает одну попытку, и повторяет только Proxy.
 
@@ -662,6 +672,20 @@ const response = await fetchRetry("https://api.example.com/data", { method: "GET
     // https://portal.bitrix24.ru/rest/1/***/crm.deal.get.json
     ```
 
+-   **`isPreConnectionError(error)`** — строгая проверка: соединение не состоялось, значит запрос **точно** не был выполнен сервером (`ENOTFOUND`, `ECONNREFUSED`, `EAI_AGAIN`, `ENETUNREACH`; код ищется и во вложенных `originalError` / `cause`). Отличие от `isNetworkError()` в том, что таймаут и обрыв соединения тоже сетевые, но при них ответ мог потеряться уже после обработки запроса. Используйте эту проверку там, где повтор небезопасен сам по себе: создание сущностей, отправка сообщений, обмен токена.
+
+    ```js
+    import { isPreConnectionError } from "@andrey4emk/npm-app-back-b24";
+
+    try {
+        await sendSms(phone, text);
+    } catch (error) {
+        // Повторяем только если сообщение заведомо не ушло
+        if (isPreConnectionError(error)) await sendSms(phone, text);
+        else throw error;
+    }
+    ```
+
 -   **`isNetworkError(error)`** — проверяет, является ли ошибка сетевой (стоит повторить запрос). Используется внутри `fetchRetry` и retry-обёртки `$b24`. Можно использовать в своём коде для аналогичных проверок.
 
     ```js

package/bitrix24/b24.ts

@@ -1,7 +1,7 @@
 import { B24OAuth, AjaxError, SdkError, Logger } from "@bitrix24/b24jssdk";
 import type { B24OAuthParams, B24OAuthSecret, AuthData, AjaxResult } from "@bitrix24/b24jssdk";
 import { logs } from "../logs/logs.ts";
-import { isNetworkError } from "../utils/fetchRetry.ts";
+import { isNetworkError, isPreConnectionError } from "../utils/fetchRetry.ts";
 import Conf from "conf";
 import path from "path";
 import type { Request, Response } from "express";
@@ -37,11 +37,23 @@ const RETRY_DELAY_MS = 500;
 /** Методы B24OAuth, оборачиваемые retry-логикой (fetchListMethod — async-генератор, его оборачивать нельзя) */
 const RETRYABLE_METHODS = new Set(["callMethod", "callListMethod", "callBatch"]);
 
-/** Коды AxiosError, означающие проблему с сетью */
-const AXIOS_NETWORK_CODES = new Set(["ERR_NETWORK", "ECONNABORTED"]);
+/**
+ * Коды транспортного сбоя. Один набор и для SdkError (AjaxError, RefreshTokenError),
+ * и для вложенного AxiosError — списки совпадали, держать их раздельно смысла нет.
+ */
+const NETWORK_ERROR_CODES = new Set(["NETWORK_ERROR", "REQUEST_TIMEOUT", "ERR_NETWORK", "ECONNABORTED"]);
 
-/** Коды ошибок SDK (AjaxError / RefreshTokenError), означающие транспортный сбой */
-const SDK_NETWORK_CODES = new Set(["NETWORK_ERROR", "REQUEST_TIMEOUT", "ERR_NETWORK", "ECONNABORTED"]);
+/**
+ * REST-методы B24, повтор которых создаёт дубликат сущности или запускает
+ * повторное действие: add, create, start, send, uploadfile, import, register.
+ *
+ * SDK 2.x при ответе портала «200 без поля result» (заглушка прокси или WAF) падает
+ * внутри собственной логирующей ветки и отдаёт ошибку со status 0 — неотличимую от
+ * транспортного сбоя. Запрос при этом уже выполнен, поэтому retry создаст вторую
+ * задачу, второй файл, второй экземпляр бизнес-процесса. Идемпотентные записи
+ * (update, delete, set) повторять безопасно: повторное применение даёт то же состояние.
+ */
+const NON_IDEMPOTENT_METHOD_RE = /\.(add|create|start|send|uploadfile|import|register)(\.json)?(\?|$)/i;
 
 const confAuthB24 = new Conf({
     cwd: path.resolve(CONFIG_DIR),
@@ -230,11 +242,19 @@ async function refreshAuthWithMutex(): Promise<AuthData> {
                     return await $b24.auth.refreshAuth();
                 } catch (error) {
                     lastError = error;
-                    if (!isB24NetworkError(error) || attempt === RETRY_COUNT) throw error;
+
+                    // Повторяем только когда соединение заведомо не состоялось.
+                    // При таймауте или обрыве сервер мог уже провести ротацию: тогда
+                    // локальный refresh-токен мёртв независимо от наших действий, и повтор
+                    // не помогает, а лишь маскирует проблему серией одинаковых отказов.
+                    // Следующая попытка всё равно будет — проактивный таймер повторит
+                    // через 2 минуты при времени жизни токена около часа
+                    if (!isPreConnectionError(error) || attempt === RETRY_COUNT) throw error;
+
                     const msg = error instanceof Error ? error.message : String(error);
                     const code = error instanceof SdkError ? ` [${error.code}]` : "";
                     const delayMs = RETRY_DELAY_MS * attempt;
-                    logs.add(`refreshAuth${code}: попытка ${attempt}/${RETRY_COUNT} не удалась (${msg}), повтор через ${delayMs}мс`, "error");
+                    logs.add(`refreshAuth${code}: попытка ${attempt}/${RETRY_COUNT} не удалась (${msg}), повтор через ${delayMs}мс`, "warn");
                     await delay(delayMs);
                 }
             }
@@ -321,19 +341,85 @@ function isB24NetworkError(error: unknown): boolean {
 
     if (error instanceof SdkError) {
         const { code, originalError, status } = error;
+        const originalCode = (originalError as { code?: string } | undefined)?.code;
 
-        if (SDK_NETWORK_CODES.has(code)) return true;
+        if (NETWORK_ERROR_CODES.has(code)) return true;
         if (originalError && isNetworkError(originalError)) return true;
-        if (AXIOS_NETWORK_CODES.has((originalError as any)?.code)) return true;
+        if (originalCode && NETWORK_ERROR_CODES.has(originalCode)) return true;
         if (status === 0) return true;
     }
 
     return isNetworkError(error);
 }
 
+/**
+ * Достаёт имя REST-метода из одной команды batch.
+ * Возвращает null, если форма команды не распознана.
+ */
+function extractBatchCommandMethod(cmd: unknown): string | null {
+    // Кортеж ["crm.deal.add", { ... }] — основная форма в SDK 2.x
+    if (Array.isArray(cmd)) {
+        return typeof cmd[0] === "string" ? cmd[0] : null;
+    }
+
+    // Объект { method, params }
+    if (cmd && typeof cmd === "object") {
+        const method = (cmd as { method?: unknown }).method;
+        return typeof method === "string" ? method : null;
+    }
+
+    // Строка "crm.deal.add?ID=1" — в SDK 2.x эта форма уже не поддерживается
+    // (ParseRow бросает JSSDK_INTERACTION_BATCH_ROW_FAIL), разбираем на случай,
+    // если потребитель остался на ней со старой версии
+    if (typeof cmd === "string") {
+        return cmd;
+    }
+
+    return null;
+}
+
+/**
+ * Возвращает причину, по которой вызов запрещено повторять, либо null.
+ *
+ * Для callBatch действует правило fail-closed: если хоть одну команду разобрать
+ * не удалось, вызов считается создающим. Ошибиться в сторону лишней осторожности
+ * дешевле — потребитель получит ошибку вместо тихого дубликата.
+ */
+function getRetryBlockReason(sdkMethod: string, args: any[]): string | null {
+    const first = args[0];
+
+    // callMethod(method, params) и callListMethod(method, params, ...)
+    if (sdkMethod === "callMethod" || sdkMethod === "callListMethod") {
+        if (typeof first !== "string") return null;
+        const method = first.trim();
+        return NON_IDEMPOTENT_METHOD_RE.test(method) ? method : null;
+    }
+
+    if (sdkMethod !== "callBatch") return null;
+
+    // callBatch(calls, ...) — команды приходят массивом либо объектом-словарём
+    if (!first || typeof first !== "object") {
+        return "аргументы batch не разобраны";
+    }
+
+    const commands: unknown[] = Array.isArray(first) ? first : Object.values(first);
+    const blocking: string[] = [];
+
+    for (const cmd of commands) {
+        const method = extractBatchCommandMethod(cmd);
+
+        if (method === null) return "команда batch не разобрана";
+        if (NON_IDEMPOTENT_METHOD_RE.test(method.trim())) blocking.push(method.trim());
+    }
+
+    return blocking.length > 0 ? blocking.join(", ") : null;
+}
+
 /** Оборачивает async-функцию retry-логикой при сетевых ошибках */
 function withRetry<T extends (...args: any[]) => Promise<any>>(fn: T, context: any, methodName: string): T {
     return (async (...args: any[]) => {
+        // Состав вызова между попытками не меняется — разбираем один раз
+        const blockReason = getRetryBlockReason(methodName, args);
         let lastError: unknown;
 
         for (let attempt = 1; attempt <= RETRY_COUNT; attempt++) {
@@ -342,13 +428,33 @@ function withRetry<T extends (...args: any[]) => Promise<any>>(fn: T, context: a
             } catch (error: unknown) {
                 lastError = error;
 
-                if (!isB24NetworkError(error) || attempt === RETRY_COUNT) {
-                    throw error;
-                }
+                // Ошибка не сетевая (бизнес-логика B24, неверные параметры) — отдаём
+                // вызывающему коду как есть, он решает, что с ней делать
+                if (!isB24NetworkError(error)) throw error;
 
                 const msg = error instanceof Error ? error.message : String(error);
                 const code = error instanceof SdkError ? ` [${error.code}]` : "";
-                logs.add(`$b24.${methodName}${code}: попытка ${attempt}/${RETRY_COUNT} не удалась (${msg}), повтор через ${RETRY_DELAY_MS}мс`, "error");
+
+                // Окончательные отказы логируем на error: собственные модули пакета
+                // (errorB24, Event, Smsgold) ошибку только возвращают вызывающему коду,
+                // но не пишут в лог — без этих строк сбой $b24 не виден нигде
+
+                if (attempt === RETRY_COUNT) {
+                    logs.add(`$b24.${methodName}${code}: исчерпаны все ${RETRY_COUNT} попыток — ${msg}`, "error");
+                    throw error;
+                }
+
+                // Ошибка со status 0 может означать и «запрос не ушёл», и «запрос выполнен,
+                // а ответ не разобрался»: различить их нельзя, поэтому создающие вызовы
+                // не повторяем — лучше вернуть ошибку, чем создать дубликат
+                if (blockReason) {
+                    logs.add(`$b24.${methodName}${code}: повтор отменён, вызов создаёт сущности (${blockReason}) — ${msg}`, "error");
+                    throw error;
+                }
+
+                // Промежуточные попытки — warn: в чат B24 уходит только уровень error,
+                // иначе один упавший вызов дал бы пять сообщений
+                logs.add(`$b24.${methodName}${code}: попытка ${attempt}/${RETRY_COUNT} не удалась (${msg}), повтор через ${RETRY_DELAY_MS}мс`, "warn");
 
                 await delay(RETRY_DELAY_MS);
             }
@@ -435,5 +541,19 @@ export async function reinitializeB24(): Promise<SaveResult> {
 
 if (_b24Raw) {
     setupRefreshCallback(_b24Raw);
-    refreshAndSaveTokens().then(() => startProactiveRefresh());
+
+    // Намеренно не ждём результат: импорт модуля не должен блокироваться сетевым
+    // запросом к oauth.bitrix.info. try/finally обязателен — если обновление всё же
+    // бросит (например, битый log.json уронит логгер), без него таймер не стартует
+    // никогда и процесс проживёт без обновления токена до перезапуска
+    void (async () => {
+        try {
+            await refreshAndSaveTokens();
+        } catch (error: unknown) {
+            const msg = error instanceof Error ? error.message : String(error);
+            logs.add(`Начальное обновление токенов упало: ${msg}`, "error");
+        } finally {
+            startProactiveRefresh();
+        }
+    })();
 }

package/bitrix24/eventB24.ts

@@ -62,6 +62,21 @@ interface OfflineEventsResponse {
 
 const SYSTEM_USER_ID = "138"; // ID системного пользователя для фильтрации
 
+// ==================== Утилиты ====================
+
+/**
+ * Приводит MESSAGES из события к массиву.
+ *
+ * B24 отдаёт коллекции то массивом, то словарём с числовыми ключами (так PHP
+ * сериализует разреженный массив), поэтому опираться на одну форму нельзя.
+ * Неизвестная форма даёт пустой массив, а не исключение.
+ */
+function normalizeMessages(messages: unknown): any[] {
+    if (Array.isArray(messages)) return messages;
+    if (messages && typeof messages === "object") return Object.values(messages);
+    return [];
+}
+
 // ==================== Класс ====================
 
 /**
@@ -154,18 +169,28 @@ export class Event {
 
                 if (allEvents.length > 0) {
                     data.processId = arrOfflineEvents.process_id;
-                    data.message = allEvents.map((event) => {
-                        const messageData = event.EVENT_DATA.MESSAGES[0];
-
-                        return {
-                            connectorId: event.EVENT_DATA.CONNECTOR,
-                            lineId: event.EVENT_DATA.LINE,
-                            chatId: messageData.chat.id,
-                            text: messageData.message.text || null,
-                            file: this.convertB24FilesToTelegramFormat(messageData.message.files),
-                            attachments: messageData.message.attachments || null,
-                            im: messageData.im || null,
-                        };
+                    // Событие может принести несколько сообщений. Раньше бралось только
+                    // первое, а остальные терялись безвозвратно — событие после обработки
+                    // удаляется из очереди B24.
+                    //
+                    // Разбор намеренно устойчив к форме и к битым элементам: исключение
+                    // здесь означало бы, что get() вернёт ошибку, потребитель не вызовет
+                    // clear(), и те же события придут следующим опросом — очередь встала бы
+                    // навсегда на одном кривом сообщении
+                    data.message = allEvents.flatMap((event) => {
+                        const eventData = event.EVENT_DATA ?? {};
+
+                        return normalizeMessages(eventData.MESSAGES)
+                            .filter((messageData) => messageData?.chat && messageData?.message)
+                            .map((messageData) => ({
+                                connectorId: eventData.CONNECTOR,
+                                lineId: eventData.LINE,
+                                chatId: messageData.chat.id,
+                                text: messageData.message.text || null,
+                                file: this.convertB24FilesToTelegramFormat(messageData.message.files),
+                                attachments: messageData.message.attachments || null,
+                                im: messageData.im || null,
+                            }));
                     });
                 }
 

package/package.json

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

package/sendMessage/smsgold.ts

@@ -1,5 +1,6 @@
 import type { B24OAuth } from "@bitrix24/b24jssdk";
 import { getResultData } from "../bitrix24/b24.ts";
+import { fetchRetry } from "../utils/fetchRetry.ts";
 
 // ==================== Типы ====================
 
@@ -150,7 +151,10 @@ export class Smsgold {
 
     private async downloadFileToUrl(fileUrl: string): Promise<{ error: boolean; message?: string; buffer?: Buffer }> {
         try {
-            const response = await fetch(fileUrl);
+            // Скачивание файла идемпотентно — повтор при сетевом сбое безопасен.
+            // Отправка SMS (fetch выше по коду) намеренно остаётся без retry:
+            // при потерянном ответе повтор отправил бы сообщение второй раз
+            const response = await fetchRetry(fileUrl);
             if (!response.ok) {
                 throw new Error(`HTTP error! status: ${response.status}`);
             }

package/utils/fetchRetry.ts

@@ -32,6 +32,35 @@ export function isNetworkError(error: unknown): boolean {
     return false;
 }
 
+/**
+ * Коды, доказывающие, что соединение не было установлено: DNS не разрешился либо
+ * хост отверг подключение. Запрос при таких ошибках гарантированно не дошёл до сервера.
+ */
+const PRE_CONNECTION_ERROR_CODES = ["ENOTFOUND", "EAI_AGAIN", "EAI_NODATA", "ECONNREFUSED", "ENETUNREACH", "EHOSTUNREACH", "ENETDOWN"];
+
+/**
+ * Проверяет, что соединение не состоялось — запрос точно не был выполнен сервером.
+ *
+ * Отличается от `isNetworkError()` строгостью: таймаут и обрыв соединения тоже сетевые,
+ * но при них ответ мог потеряться уже после того, как сервер обработал запрос.
+ * Используйте эту проверку вместо `isNetworkError()` там, где повтор небезопасен сам
+ * по себе: обмен refresh-токена, создание сущностей, отправка сообщений.
+ *
+ * @param error — пойманная ошибка
+ * @returns true, если повторить запрос заведомо безопасно
+ */
+export function isPreConnectionError(error: unknown): boolean {
+    if (!error || typeof error !== "object") return false;
+
+    const codes: unknown[] = [
+        (error as { code?: unknown }).code,
+        (error as { originalError?: { code?: unknown } }).originalError?.code,
+        (error as { cause?: { code?: unknown } }).cause?.code,
+    ];
+
+    return codes.some((code) => typeof code === "string" && PRE_CONNECTION_ERROR_CODES.includes(code));
+}
+
 /**
  * Маскирует секреты в URL перед записью в лог.
  *
@@ -50,8 +79,9 @@ export function maskUrl(url: string | URL | Request): string {
         raw
             // Секрет входящего вебхука: /rest/<id пользователя>/<секрет>/
             .replace(/(\/rest\/\d+\/)[^/?#]+/gi, "$1***")
-            // Токены в query-параметрах
-            .replace(/([?&](?:auth|access_token|refresh_token|token)=)[^&#]+/gi, "$1***")
+            // Токены и подписи в query-параметрах. signature и sessid нужны отдельно:
+            // ссылка на файл B24-диска несёт секрет именно в signature
+            .replace(/([?&](?:auth|access_token|refresh_token|token|signature|sessid)=)[^&#]+/gi, "$1***")
     );
 }