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

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,11 @@ 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 **не** вызывают повторных попыток. Каждая неудачная попытка логируется через `logs.add()` с уровнем `warn` — в сообщение попадает код ошибки SDK. Уровень `warn` выбран намеренно: `error` уходит в чат B24, и пять попыток одного упавшего вызова превращались в пять сообщений. Финальный провал приходит вызывающему коду исключением и логируется им один раз.
+
+    **Создающие вызовы не повторяются.** Если в вызове есть метод, создающий сущность (`*.add`, `*.uploadfile`, `*.import`, `*.register` — в том числе внутри `callBatch`), retry отменяется, а в лог идёт `повтор отменён`. Причина: при ответе портала «200 без поля `result`» (заглушка прокси или WAF) SDK падает в собственной ветке логирования и отдаёт ошибку со `status: 0`, неотличимую от транспортного сбоя — хотя запрос уже выполнен. Повтор в такой ситуации создавал до пяти дубликатов задачи или файла. Идемпотентные записи (`*.update`, `*.delete`, `*.set`) повторяются как прежде: повторное применение даёт то же состояние.
+
+    **Обмен refresh-токена** повторяется по более строгому правилу — только если соединение заведомо не состоялось (`ENOTFOUND`, `ECONNREFUSED`, `EAI_AGAIN`, `ENETUNREACH`, см. `isPreConnectionError()`). При таймауте или обрыве сервер мог уже провести ротацию, а ответ потеряться: повтор вернул бы `invalid_grant` и оставил в конфиге мёртвый `refresh_token`.
 
     У самого SDK есть собственный слой ретраев (`maxRetries: 3`), но он работает только при rate limit (HTTP 429, 503) и при распознанных `NETWORK_ERROR` / `REQUEST_TIMEOUT`. Замаскированные транспортные сбои приходят с кодом `JSSDK_UNKNOWN_ERROR`, который входит в `BUILT_IN_HARD_ERROR_CODES` — на таких ошибках SDK делает одну попытку, и повторяет только Proxy.
 
@@ -662,6 +666,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,22 @@ 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, повтор которых создаёт дубликат сущности.
+ *
+ * SDK 2.x при ответе портала «200 без поля result» (заглушка прокси или WAF) падает
+ * внутри собственной логирующей ветки и отдаёт ошибку со status 0 — неотличимую от
+ * транспортного сбоя. Запрос при этом уже выполнен, поэтому retry создаст вторую
+ * задачу, второй файл и т.д. Идемпотентные записи (update, delete, set) повторять
+ * безопасно: повторное применение даёт то же состояние.
+ */
+const NON_IDEMPOTENT_METHOD_RE = /\.(add|uploadfile|import|register)(\?|$)/i;
 
 const confAuthB24 = new Conf({
     cwd: path.resolve(CONFIG_DIR),
@@ -230,11 +241,17 @@ async function refreshAuthWithMutex(): Promise<AuthData> {
                     return await $b24.auth.refreshAuth();
                 } catch (error) {
                     lastError = error;
-                    if (!isB24NetworkError(error) || attempt === RETRY_COUNT) throw error;
+
+                    // Повторяем только когда соединение заведомо не состоялось: при таймауте
+                    // или обрыве сервер мог уже провести ротацию, ответ потеряться, и повтор
+                    // вернёт invalid_grant — refresh-токен окажется сожжён, потребуется
+                    // ручная переавторизация
+                    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 +338,48 @@ 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-методов B24 из аргументов вызова SDK */
+function extractB24Methods(sdkMethod: string, args: any[]): string[] {
+    const first = args[0];
+
+    // callMethod(method, params) и callListMethod(method, params, ...)
+    if (sdkMethod === "callMethod" || sdkMethod === "callListMethod") {
+        return typeof first === "string" ? [first] : [];
+    }
+
+    // callBatch(calls, ...) — команды приходят массивом либо объектом-словарём
+    if (sdkMethod === "callBatch") {
+        const commands: unknown[] = Array.isArray(first) ? first : first && typeof first === "object" ? Object.values(first) : [];
+
+        return commands
+            .map((cmd) => (typeof cmd === "string" ? cmd : (cmd as { method?: unknown })?.method))
+            .filter((method): method is string => typeof method === "string");
+    }
+
+    return [];
+}
+
+/** Возвращает методы вызова, повтор которых создал бы дубликаты сущностей */
+function getNonIdempotentMethods(sdkMethod: string, args: any[]): string[] {
+    return extractB24Methods(sdkMethod, args).filter((method) => NON_IDEMPOTENT_METHOD_RE.test(method));
+}
+
 /** Оборачивает async-функцию retry-логикой при сетевых ошибках */
 function withRetry<T extends (...args: any[]) => Promise<any>>(fn: T, context: any, methodName: string): T {
     return (async (...args: any[]) => {
+        // Состав вызова между попытками не меняется — разбираем один раз
+        const nonIdempotent = getNonIdempotentMethods(methodName, args);
         let lastError: unknown;
 
         for (let attempt = 1; attempt <= RETRY_COUNT; attempt++) {
@@ -348,7 +394,21 @@ function withRetry<T extends (...args: any[]) => Promise<any>>(fn: T, context: a
 
                 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");
+
+                // Ошибка со status 0 может означать и «запрос не ушёл», и «запрос выполнен,
+                // а ответ не разобрался»: различить их нельзя, поэтому создающие вызовы
+                // не повторяем — лучше вернуть ошибку, чем создать дубликат
+                if (nonIdempotent.length > 0) {
+                    logs.add(
+                        `$b24.${methodName}${code}: повтор отменён, вызов создаёт сущности (${nonIdempotent.join(", ")}) — ${msg}`,
+                        "warn"
+                    );
+                    throw error;
+                }
+
+                // Уровень warn: промежуточные попытки не должны улетать в чат B24,
+                // финальный провал прилетит вызывающему коду исключением
+                logs.add(`$b24.${methodName}${code}: попытка ${attempt}/${RETRY_COUNT} не удалась (${msg}), повтор через ${RETRY_DELAY_MS}мс`, "warn");
 
                 await delay(RETRY_DELAY_MS);
             }
@@ -435,5 +495,12 @@ export async function reinitializeB24(): Promise<SaveResult> {
 
 if (_b24Raw) {
     setupRefreshCallback(_b24Raw);
-    refreshAndSaveTokens().then(() => startProactiveRefresh());
+
+    // Намеренно не ждём результат: импорт модуля не должен блокироваться сетевым
+    // запросом к oauth.bitrix.info. refreshAndSaveTokens() исключений не бросает —
+    // ошибку возвращает в SaveResult и пишет в лог сам
+    void (async () => {
+        await refreshAndSaveTokens();
+        startProactiveRefresh();
+    })();
 }

package/bitrix24/eventB24.ts

@@ -154,10 +154,13 @@ export class Event {
 
                 if (allEvents.length > 0) {
                     data.processId = arrOfflineEvents.process_id;
-                    data.message = allEvents.map((event) => {
-                        const messageData = event.EVENT_DATA.MESSAGES[0];
+                    // Событие может принести несколько сообщений. Раньше бралось только
+                    // первое, а остальные терялись безвозвратно — событие после обработки
+                    // удаляется из очереди B24
+                    data.message = allEvents.flatMap((event) => {
+                        const messages = event.EVENT_DATA.MESSAGES ?? [];
 
-                        return {
+                        return messages.map((messageData: any) => ({
                             connectorId: event.EVENT_DATA.CONNECTOR,
                             lineId: event.EVENT_DATA.LINE,
                             chatId: messageData.chat.id,
@@ -165,7 +168,7 @@ export class Event {
                             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.0",
     "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", "ECONNREFUSED", "ENETUNREACH"];
+
+/**
+ * Проверяет, что соединение не состоялось — запрос точно не был выполнен сервером.
+ *
+ * Отличается от `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 перед записью в лог.
  *