@andrey4emk/npm-app-back-b24 3.3.0 → 3.4.0

package/README.md

@@ -35,6 +35,7 @@ import {
     refreshAndSaveTokens,
     reinitializeB24,
     stopProactiveRefresh,
+    getResultData,
     errorB24,
     event,
     Event,
@@ -118,14 +119,20 @@ const response = await fetchRetry("https://api.example.com/data", { method: "GET
 
     **Retry при сетевых ошибках:**
 
-    Экземпляр `$b24` обёрнут Proxy, который автоматически повторяет запросы при сетевых ошибках. Повторные попытки применяются к методам `callMethod`, `callListMethod`, `callBatch` и `fetchListMethod`, а также к обновлению токена (`refreshAuth`).
+    Экземпляр `$b24` обёрнут Proxy, который автоматически повторяет запросы при сетевых ошибках. Повторные попытки применяются к методам `callMethod`, `callListMethod`, `callBatch`, а также к обновлению токена (`refreshAuth`). Метод `fetchListMethod` из ретраев исключён — это async-генератор, оборачивать его в async-функцию нельзя; постраничные запросы SDK ретраит сам.
 
     | Параметр       | Значение                                  |
     | -------------- | ----------------------------------------- |
     | Попыток        | 5                                         |
-    | Задержка       | 500 мс (экспоненциально для refreshAuth)  |
+    | Задержка       | 500 мс (линейно растущая для refreshAuth) |
 
-    Retry срабатывает только при сетевых проблемах (`ECONNRESET`, `ETIMEDOUT`, `ERR_NETWORK` и т.д.). HTTP-ошибки (400, 500) и ошибки бизнес-логики Bitrix24 **не** вызывают повторных попыток. Каждая неудачная попытка логируется через `logs.add()` с уровнем `error`.
+    Retry срабатывает только при сетевых проблемах (`ECONNRESET`, `ETIMEDOUT`, `ERR_NETWORK` и т.д.). HTTP-ошибки (400, 500) и ошибки бизнес-логики Bitrix24 **не** вызывают повторных попыток. Каждая неудачная попытка логируется через `logs.add()` с уровнем `error` — в сообщение попадает код ошибки SDK.
+
+    У самого SDK есть собственный слой ретраев (`maxRetries: 3`), но он работает только при rate limit (HTTP 429, 503) и при распознанных `NETWORK_ERROR` / `REQUEST_TIMEOUT`. Замаскированные транспортные сбои приходят с кодом `JSSDK_UNKNOWN_ERROR`, который входит в `BUILT_IN_HARD_ERROR_CODES` — на таких ошибках SDK делает одну попытку, и повторяет только Proxy.
+
+    **«Мягкие» ошибки Bitrix24:**
+
+    Часть ошибок (`ERROR_ENTITY_NOT_FOUND`, `BITRIX_REST_V3_EXCEPTION_*`) SDK не бросает исключением, а возвращает как `AjaxResult` с ошибкой: `isSuccess === false`, и тогда `response.getData()` вернёт `undefined`. Отдельный случай — успешный ответ, у которого сам `result` пустой (`null`/`undefined`). Чтобы не падать на `undefined`, разбирайте ответы через `getResultData()` — он покрывает обе ситуации и бросает понятную ошибку с именем метода.
 
     **Проактивное обновление токена:**
 
@@ -193,6 +200,20 @@ const response = await fetchRetry("https://api.example.com/data", { method: "GET
     });
     ```
 
+-   **`getResultData(response, methodName)`** — достаёт `result` из ответа SDK. Если Bitrix24 вернул «мягкую» ошибку (`AjaxResult` с ошибкой вместо исключения) или пустой ответ, бросает `Error` с понятным текстом вместо падения на `undefined`.
+
+    -   **Параметры:**
+        -   `response` (AjaxResult) — результат `$b24.callMethod()`.
+        -   `methodName` (string) — имя метода B24, попадёт в текст ошибки.
+    -   **Возвращает:** содержимое `result`. Тип задаётся дженериком.
+
+    ```js
+    import { $b24, getResultData } from "@andrey4emk/npm-app-back-b24";
+
+    const response = await $b24.callMethod("crm.deal.get", { id: 123 });
+    const deal = getResultData(response, "crm.deal.get");
+    ```
+
 ### Задачи ошибок
 
 -   **`errorB24(dataTask)`** — создаёт служебную задачу в Bitrix24 при ошибках/событиях. Использует глобальный `$b24`.

package/bitrix24/b24.ts

@@ -1,5 +1,5 @@
-import { B24OAuth, AjaxError } from "@bitrix24/b24jssdk";
-import type { B24OAuthParams, B24OAuthSecret, AuthData } from "@bitrix24/b24jssdk";
+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 Conf from "conf";
@@ -34,12 +34,15 @@ const RETRY_COUNT = 5;
 /** Задержка между попытками (мс) */
 const RETRY_DELAY_MS = 500;
 
-/** Методы B24OAuth, оборачиваемые retry-логикой */
-const RETRYABLE_METHODS = new Set(["callMethod", "callListMethod", "callBatch", "fetchListMethod"]);
+/** Методы B24OAuth, оборачиваемые retry-логикой (fetchListMethod — async-генератор, его оборачивать нельзя) */
+const RETRYABLE_METHODS = new Set(["callMethod", "callListMethod", "callBatch"]);
 
 /** Коды AxiosError, означающие проблему с сетью */
 const AXIOS_NETWORK_CODES = new Set(["ERR_NETWORK", "ECONNABORTED"]);
 
+/** Коды ошибок SDK (AjaxError / RefreshTokenError), означающие транспортный сбой */
+const SDK_NETWORK_CODES = new Set(["NETWORK_ERROR", "REQUEST_TIMEOUT", "ERR_NETWORK", "ECONNABORTED"]);
+
 const confAuthB24 = new Conf({
     cwd: path.resolve(CONFIG_DIR),
     configName: "authB24",
@@ -55,6 +58,31 @@ function cleanDomain(domain: string): string {
 /** Задержка на указанное количество миллисекунд */
 const delay = (ms: number) => new Promise<void>((resolve) => setTimeout(resolve, ms));
 
+/**
+ * Достаёт result из ответа SDK.
+ *
+ * SDK при «мягких» ошибках (ERROR_ENTITY_NOT_FOUND, BITRIX_REST_V3_EXCEPTION_*)
+ * не бросает исключение, а возвращает AjaxResult с ошибкой.
+ * Функция превращает такой ответ в понятную ошибку вместо падения на undefined.
+ * Также отсекает случай, когда запрос успешен, но result пустой (null/undefined).
+ *
+ * @param response — результат $b24.callMethod()
+ * @param methodName — имя метода B24, попадёт в текст ошибки
+ */
+export function getResultData<T = any>(response: AjaxResult, methodName: string): T {
+    if (!response.isSuccess) {
+        const messages = response.getErrorMessages().join("; ") || "неизвестная ошибка";
+        throw new Error(`${methodName}: Bitrix24 вернул ошибку — ${messages}`);
+    }
+
+    const data = response.getData();
+    if (data?.result == null) {
+        throw new Error(`${methodName}: Bitrix24 вернул пустой ответ`);
+    }
+
+    return data.result as T;
+}
+
 // ==================== Создание экземпляра B24OAuth ====================
 
 function createB24Instance(): B24OAuth | null {
@@ -91,7 +119,14 @@ function createB24Instance(): B24OAuth | null {
 
     const secret: B24OAuthSecret = { clientId: CLIENT_ID, clientSecret: CLIENT_SECRET };
 
-    return new B24OAuth(authParams, secret);
+    const b24 = new B24OAuth(authParams, secret);
+
+    // Логгер без хендлеров: SDK 2.x на каждый вызов callMethod/callBatch пишет
+    // предупреждение об устаревании напрямую в console.warn, если логгер — NullLogger.
+    // Любой другой логгер получает запись через свой интерфейс, поэтому консоль остаётся чистой.
+    b24.setLogger(Logger.create("npm-app-back-b24"));
+
+    return b24;
 }
 
 // ==================== Работа с токенами ====================
@@ -188,8 +223,9 @@ async function refreshAuthWithMutex(): Promise<AuthData> {
                     lastError = error;
                     if (!isB24NetworkError(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: попытка ${attempt}/${RETRY_COUNT} не удалась (${msg}), повтор через ${delayMs}мс`, "error");
+                    logs.add(`refreshAuth${code}: попытка ${attempt}/${RETRY_COUNT} не удалась (${msg}), повтор через ${delayMs}мс`, "error");
                     await delay(delayMs);
                 }
             }
@@ -261,15 +297,23 @@ export function stopProactiveRefresh(): void {
 
 /**
  * Проверяет, является ли ошибка из SDK сетевой.
- * AjaxError хранит AxiosError в поле originalError.
+ *
+ * SDK 2.x маскирует транспортные сбои: в логирующей ветке падает собственный
+ * TypeError, и наружу приходит AjaxError с кодом JSSDK_UNKNOWN_ERROR и status 0.
+ * Поэтому проверяем и код, и originalError, и status.
+ * RefreshTokenError приходит как SdkError с кодом вроде ENOTFOUND и без originalError.
  */
 function isB24NetworkError(error: unknown): boolean {
     if (error instanceof AjaxError) {
-        // SDK уже сам retry'ил и исчерпал все попытки — не повторять
+        // Практически недостижимо (SDK бросает реальную ошибку раньше),
+        // но если код всё же пришёл — SDK уже исчерпал свои попытки
         if (error.code === "JSSDK_CALL_ALL_ATTEMPTS_EXHAUSTED") return false;
+    }
 
-        const { originalError, status } = error;
+    if (error instanceof SdkError) {
+        const { code, originalError, status } = error;
 
+        if (SDK_NETWORK_CODES.has(code)) return true;
         if (originalError && isNetworkError(originalError)) return true;
         if (AXIOS_NETWORK_CODES.has((originalError as any)?.code)) return true;
         if (status === 0) return true;
@@ -294,7 +338,8 @@ function withRetry<T extends (...args: any[]) => Promise<any>>(fn: T, context: a
                 }
 
                 const msg = error instanceof Error ? error.message : String(error);
-                logs.add(`$b24.${methodName}: попытка ${attempt}/${RETRY_COUNT} не удалась (${msg}), повтор через ${RETRY_DELAY_MS}мс`, "error");
+                const code = error instanceof SdkError ? ` [${error.code}]` : "";
+                logs.add(`$b24.${methodName}${code}: попытка ${attempt}/${RETRY_COUNT} не удалась (${msg}), повтор через ${RETRY_DELAY_MS}мс`, "error");
 
                 await delay(RETRY_DELAY_MS);
             }
@@ -306,32 +351,36 @@ function withRetry<T extends (...args: any[]) => Promise<any>>(fn: T, context: a
 }
 
 /**
- * Proxy-обёртка вокруг B24OAuth: автоматически оборачивает
- * callMethod, callListMethod, callBatch и fetchListMethod retry-логикой.
- * Обёрнутые методы кешируются — обёртка создаётся один раз на метод.
+ * Proxy-обёртка вокруг B24OAuth.
+ *
+ * Методы из RETRYABLE_METHODS оборачиваются retry-логикой, все остальные
+ * привязываются к оригинальному объекту через bind: класс B24OAuth использует
+ * приватные поля (#authOAuthManager), и при вызове метода с this === Proxy
+ * движок бросает "Cannot read private member".
+ *
+ * Обёртки кешируются — по одной на метод, чтобы не ломать сравнение по ссылке.
  */
 function wrapB24WithRetry(b24: B24OAuth): B24OAuth {
     const methodCache = new Map<string, Function>();
 
     return new Proxy(b24, {
-        get(target, prop, receiver) {
-            // Передаём target вместо receiver — this внутри геттеров должен указывать
-            // на оригинальный target, иначе приватные поля (#authOAuthManager и др.)
-            // недоступны через Proxy
-
-            // Для методов из RETRYABLE_METHODS — оборачиваем retry-логикой
-            if (typeof prop === "string" && RETRYABLE_METHODS.has(prop)) {
-                const value = Reflect.get(target, prop, target);
-                if (typeof value === "function") {
-                    if (!methodCache.has(prop)) {
-                        methodCache.set(prop, withRetry(value, target, prop));
-                    }
-                    return methodCache.get(prop);
-                }
+        get(target, prop) {
+            // Передаём target третьим аргументом: геттеры (например, auth)
+            // тоже должны исполняться с this === target
+            const value = Reflect.get(target, prop, target);
+
+            if (typeof prop !== "string" || typeof value !== "function") {
+                return value;
+            }
+
+            if (!methodCache.has(prop)) {
+                const wrapped = RETRYABLE_METHODS.has(prop)
+                    ? withRetry(value as (...args: any[]) => Promise<any>, target, prop)
+                    : value.bind(target);
+                methodCache.set(prop, wrapped);
             }
 
-            // Для остальных — передаём target чтобы приватные поля работали
-            return Reflect.get(target, prop, target);
+            return methodCache.get(prop);
         },
     });
 }

package/bitrix24/errTaskB24.ts

@@ -1,5 +1,5 @@
 import { DateTime } from "luxon";
-import { $b24 } from "./b24.ts";
+import { $b24, getResultData } from "./b24.ts";
 import { logs } from "../logs/logs.ts";
 import dotEnv from "dotenv";
 
@@ -54,8 +54,7 @@ export async function checkB24Scope(): Promise<void> {
 
     try {
         const scopeResponse = await $b24.callMethod("scope");
-        const scopeData = scopeResponse.getData() as any;
-        const scopes: string[] = scopeData.result ?? [];
+        const scopes = getResultData<string[]>(scopeResponse, "scope") ?? [];
 
         logs.add(`Права приложения Bitrix24: ${scopes.join(", ")}`, "info");
 
@@ -104,8 +103,7 @@ export async function errorB24(dataTask: ErrorTaskData): Promise<ErrorB24Result>
             select: ["ID", "STATUS"],
         });
 
-        const data = response.getData() as any;
-        const tasks: TaskItem[] = data.result?.tasks ?? [];
+        const tasks: TaskItem[] = getResultData<{ tasks?: TaskItem[] }>(response, "tasks.task.list").tasks ?? [];
 
         if (tasks.length >= maxTasks) {
             return {
@@ -128,12 +126,12 @@ export async function errorB24(dataTask: ErrorTaskData): Promise<ErrorB24Result>
             },
         });
 
-        const createData = createResponse.getData() as any;
+        const createdTask = getResultData(createResponse, "tasks.task.add");
 
         return {
             error: false,
             message: "Задача создана в Битрикс24.",
-            data: createData.result,
+            data: createdTask,
         };
     } catch (error: any) {
         return {

package/bitrix24/eventB24.ts

@@ -1,5 +1,5 @@
 import type { B24OAuth } from "@bitrix24/b24jssdk";
-import { $b24 } from "./b24.ts";
+import { $b24, getResultData } from "./b24.ts";
 
 // ==================== Типы ====================
 
@@ -141,7 +141,9 @@ export class Event {
                 filter: { EVENT_NAME: eventName },
             });
 
-            const arrOfflineEvents = response.getData() as OfflineEventsResponse;
+            const arrOfflineEvents = getResultData<OfflineEventsResponse["result"]>(response, "event.offline.get");
+            // B24 для пустой коллекции может вернуть result: [] — тогда events отсутствует
+            const allEvents = arrOfflineEvents?.events ?? [];
 
             // Обработка событий коннектора
             if (eventName === "ONIMCONNECTORMESSAGEADD") {
@@ -150,9 +152,9 @@ export class Event {
                     message: [],
                 };
 
-                if (arrOfflineEvents.result.events.length > 0) {
-                    data.processId = arrOfflineEvents.result.process_id;
-                    data.message = arrOfflineEvents.result.events.map((event) => {
+                if (allEvents.length > 0) {
+                    data.processId = arrOfflineEvents.process_id;
+                    data.message = allEvents.map((event) => {
                         const messageData = event.EVENT_DATA.MESSAGES[0];
 
                         return {
@@ -178,23 +180,23 @@ export class Event {
             };
 
             // Очищаем события от системного пользователя
-            const arrEventsToClear = arrOfflineEvents.result.events.filter((event) => event.EVENT_ADDITIONAL.user_id === SYSTEM_USER_ID);
+            const arrEventsToClear = allEvents.filter((event) => event.EVENT_ADDITIONAL.user_id === SYSTEM_USER_ID);
 
             if (arrEventsToClear.length > 0) {
                 const arrMessageIdToClear = arrEventsToClear.map((event) => event.MESSAGE_ID);
                 await this.b24.callMethod("event.offline.clear", {
-                    process_id: arrOfflineEvents.result.process_id,
+                    process_id: arrOfflineEvents.process_id,
                     message_id: arrMessageIdToClear,
                 });
-
-                // Удаляем эти события из основного массива
-                arrOfflineEvents.result.events = arrOfflineEvents.result.events.filter((event) => event.EVENT_ADDITIONAL.user_id !== SYSTEM_USER_ID);
             }
 
-            if (arrOfflineEvents.result.events.length > 0) {
-                data.processId = arrOfflineEvents.result.process_id;
-                data.entitysId = arrOfflineEvents.result.events.map((event) => event.EVENT_DATA.FIELDS.ID);
-                data.arrMessageIdAndEntityId = arrOfflineEvents.result.events.map((event) => ({
+            // Работаем только с событиями, не относящимися к системному пользователю
+            const activeEvents = allEvents.filter((event) => event.EVENT_ADDITIONAL.user_id !== SYSTEM_USER_ID);
+
+            if (activeEvents.length > 0) {
+                data.processId = arrOfflineEvents.process_id;
+                data.entitysId = activeEvents.map((event) => event.EVENT_DATA.FIELDS.ID);
+                data.arrMessageIdAndEntityId = activeEvents.map((event) => ({
                     messageId: event.MESSAGE_ID,
                     entityId: event.EVENT_DATA.FIELDS.ID,
                 }));

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@andrey4emk/npm-app-back-b24",
-    "version": "3.3.0",
+    "version": "3.4.0",
     "description": "Bitrix24 OAuth helpers for Node.js projects",
     "main": "index.ts",
     "type": "module",
@@ -35,15 +35,14 @@
         "utils/fetchRetry.ts"
     ],
     "dependencies": {
-        "@bitrix24/b24jssdk": "^1.0.4",
-        "@types/express": "^5.0.6",
-        "@types/luxon": "^3.7.1",
-        "@types/node": "^25.0.10",
-        "conf": "^15.0.2",
-        "dotenv": "^17.2.3",
-        "luxon": "^3.4.4",
-        "nodemailer": "^7.0.6",
-        "@types/nodemailer": "^7.0.9"
-    },
-    "devDependencies": {}
+        "@bitrix24/b24jssdk": "2.0.0",
+        "@types/express": "5.0.6",
+        "@types/luxon": "3.7.1",
+        "@types/node": "25.0.10",
+        "@types/nodemailer": "7.0.9",
+        "conf": "15.0.2",
+        "dotenv": "17.2.3",
+        "luxon": "3.4.4",
+        "nodemailer": "7.0.6"
+    }
 }

package/sendMessage/smsgold.ts

@@ -1,4 +1,5 @@
 import type { B24OAuth } from "@bitrix24/b24jssdk";
+import { getResultData } from "../bitrix24/b24.ts";
 
 // ==================== Типы ====================
 
@@ -74,10 +75,10 @@ export class Smsgold {
 
             // Получаем публичную ссылку на файл для вставки в сообщение
             try {
-                let resLink = await this.b24!.callMethod("disk.file.getExternalLink", {
+                const resLink = await this.b24!.callMethod("disk.file.getExternalLink", {
                     id: idFileContract,
                 });
-                publicLink = (resLink as any).getData().result;
+                publicLink = getResultData<string>(resLink, "disk.file.getExternalLink");
             } catch (error: unknown) {
                 const errMsg = error instanceof Error ? error.message : String(error);
                 return { error: true, message: `Ошибка при получении публичной ссылки на файл: ${errMsg}` };
@@ -139,7 +140,7 @@ export class Smsgold {
                 data: { NAME: fileName },
                 fileContent: [fileName, base64File],
             });
-            const data = (res as any).getData().result;
+            const data = getResultData(res, "disk.folder.uploadfile");
             return { error: false, data };
         } catch (error: unknown) {
             const errMsg = error instanceof Error ? error.message : String(error);