@andrey4emk/npm-app-back-b24 3.7.1 → 3.8.1

package/sendMessage/smsgold.ts

@@ -1,5 +1,6 @@
 import { getResultData, type B24MethodCaller } from "../bitrix24/b24.ts";
-import { fetchRetry } from "../utils/fetchRetry.ts";
+import { fetchRetry, fetchWithTimeout, isTimeoutError, FETCH_TIMEOUTS } from "../utils/fetchRetry.ts";
+import { logs } from "../logs/logs.ts";
 
 // ==================== Типы ====================
 
@@ -17,6 +18,15 @@ interface SmsMessageData {
     fileName?: string;
 }
 
+/**
+ * Папка на диске Bitrix24, куда кладётся вложение перед отправкой SMS.
+ * Значение — папка «contract_company» проекта «Документы и договора» портала
+ * repacheb.bitrix24.ru. Для опубликованной библиотеки это чужая константа,
+ * поэтому она вынесена в параметр конструктора; здесь она осталась дефолтом,
+ * иначе смена была бы ломающей для всех текущих потребителей.
+ */
+const DEFAULT_UPLOAD_FOLDER_ID = 2792881;
+
 /** Результат отправки SMS */
 interface SmsResult {
     error: boolean;
@@ -27,6 +37,12 @@ interface SmsResult {
         data?: string;
         errorText?: string;
     } | null;
+    /**
+     * Выставляется только при таймауте ожидания ответа SMSGold: запрос ушёл, ответ не пришёл.
+     * Означает «сообщение могло уйти» — не «не отправлено». Повторять отправку нельзя,
+     * решение принимает человек.
+     */
+    unknownDelivery?: boolean;
 }
 
 // ==================== Класс ====================
@@ -37,11 +53,14 @@ export class Smsgold {
     // Структурный тип, а не B24OAuth: объявляет callMethod сам, поэтому переживёт
     // удаление метода из SDK. Сигнатуру конструктора не сужает — она стала шире
     private b24: B24MethodCaller | null;
+    // ID папки на диске B24 для вложений SMS — параметр, а не константа модуля
+    private uploadFolderId: number;
 
-    constructor(authParam: SmsgoldAuthParam, b24: B24MethodCaller | null = null) {
+    constructor(authParam: SmsgoldAuthParam, b24: B24MethodCaller | null = null, uploadFolderId: number = DEFAULT_UPLOAD_FOLDER_ID) {
         this.user = authParam.user;
         this.pass = authParam.pass;
         this.b24 = b24;
+        this.uploadFolderId = uploadFolderId;
     }
 
     async sendSms(messageData: SmsMessageData): Promise<SmsResult> {
@@ -97,13 +116,20 @@ export class Smsgold {
                 sender: "potolkiRepa",
             }).toString();
 
-            const result = await fetch("https://web.smsgold.ru/http2/", {
-                method: "POST",
-                headers: {
-                    "Content-Type": "application/x-www-form-urlencoded",
+            // Отправка неидемпотентна — повторов нет, единственная защита от зависшего
+            // соединения — бюджет времени. 30с взяты с запасом, чтобы «неизвестный статус»
+            // случался как можно реже
+            const result = await fetchWithTimeout(
+                "https://web.smsgold.ru/http2/",
+                {
+                    method: "POST",
+                    headers: {
+                        "Content-Type": "application/x-www-form-urlencoded",
+                    },
+                    body,
                 },
-                body,
-            });
+                FETCH_TIMEOUTS.send
+            );
 
             if (!result.ok) {
                 const errorText = await result.text();
@@ -126,6 +152,23 @@ export class Smsgold {
                 },
             };
         } catch (error: unknown) {
+            // Ответа не дождались: запрос уже ушёл, сообщение могло быть доставлено.
+            // Это не «ошибка отправки» — статус неизвестен, и решить это кодом нельзя.
+            // Единственное место в классе, где пишем в лог: уровень error уходит в чат B24,
+            // потому что разбираться должен человек
+            if (isTimeoutError(error)) {
+                logs.add(
+                    `[${phone}] SMS: таймаут ожидания ответа SMSGold ${FETCH_TIMEOUTS.send}мс — статус неизвестен, повторная отправка запрещена`,
+                    "error"
+                );
+                return {
+                    error: true,
+                    message: `SMSGold не ответил за ${FETCH_TIMEOUTS.send / 1000}с — статус отправки неизвестен: сообщение могло уйти, повторять нельзя`,
+                    result: null,
+                    unknownDelivery: true,
+                };
+            }
+
             const errMsg = error instanceof Error ? error.message : String(error);
             return { error: true, message: errMsg, result: null };
         }
@@ -138,7 +181,7 @@ export class Smsgold {
 
             // Используем метод disk.folder.uploadfile
             const res = await this.b24!.callMethod("disk.folder.uploadfile", {
-                id: 2792881, // Id папки "contract_company" из проекта "Документы и договора"
+                id: this.uploadFolderId,
                 data: { NAME: fileName },
                 fileContent: [fileName, base64File],
             });
@@ -153,9 +196,11 @@ export class Smsgold {
     private async downloadFileToUrl(fileUrl: string): Promise<{ error: boolean; message?: string; buffer?: Buffer }> {
         try {
             // Скачивание файла идемпотентно — повтор при сетевом сбое безопасен.
-            // Отправка SMS (fetch выше по коду) намеренно остаётся без retry:
-            // при потерянном ответе повтор отправил бы сообщение второй раз
-            const response = await fetchRetry(fileUrl);
+            // Отправка SMS (fetchWithTimeout выше по коду) намеренно остаётся без retry:
+            // при потерянном ответе повтор отправил бы сообщение второй раз.
+            // Попыток 3, а не 5: при бюджете 60с на попытку пять попыток дали бы до пяти
+            // минут ожидания перед самой отправкой SMS
+            const response = await fetchRetry(fileUrl, undefined, 3, 500, FETCH_TIMEOUTS.transfer);
             if (!response.ok) {
                 throw new Error(`HTTP error! status: ${response.status}`);
             }

package/sendMessage/wappi.ts

@@ -1,3 +1,6 @@
+import { fetchWithTimeout, isTimeoutError, FETCH_TIMEOUTS } from "../utils/fetchRetry.ts";
+import { logs } from "../logs/logs.ts";
+
 // ==================== Типы ====================
 
 /** Параметры авторизации Wappi-мессенджера */
@@ -27,6 +30,11 @@ interface WappiResult {
     error: boolean;
     message: string;
     data: any;
+    /**
+     * Ответа не дождались. Отличает «проверка сказала нет» от «проверки не было»:
+     * без этого признака вызывающий код принимает таймаут за отрицательный ответ.
+     */
+    timeout?: boolean;
 }
 
 // ==================== Класс ====================
@@ -85,18 +93,42 @@ export class Wappi {
             url = `https://wappi.pro/maxapi/sync/message/send?${sendOpenLine}profile_id=${profile_id}`;
         }
 
-        const res = await fetch(url!, {
-            method: "POST",
-            headers: {
-                "Content-Type": "application/json",
-                Authorization: token!,
-            },
-            body: JSON.stringify({
-                recipient: phone,
-                body: message,
-            }),
-        });
-        const data: any = await res.json();
+        let data: any;
+        try {
+            const res = await fetchWithTimeout(
+                url!,
+                {
+                    method: "POST",
+                    headers: {
+                        "Content-Type": "application/json",
+                        Authorization: token!,
+                    },
+                    body: JSON.stringify({
+                        recipient: phone,
+                        body: message,
+                    }),
+                },
+                FETCH_TIMEOUTS.send
+            );
+            data = await res.json();
+        } catch (error: unknown) {
+            // Метод исторически без try/catch — перехватываем только таймаут, который сами
+            // и вводим, чтобы не появился новый путь исключений там, где его сегодня нет.
+            // Остальные ошибки летят наружу как раньше
+            if (!isTimeoutError(error)) throw error;
+
+            // Уровень error — единственный, который уходит в чат B24. До появления
+            // таймаута зависание долетало исключением до потребителя и там порождало
+            // задачу через errorB24(); перехватив ошибку здесь, мы обязаны сами оставить
+            // след, иначе правка сделала бы отказ отправки тише, чем он был
+            logs.add(`[${phone}] Wappi: таймаут ${FETCH_TIMEOUTS.send}мс при отправке сообщения через ${messangerType} — статус неизвестен, повторять нельзя`, "error");
+
+            return {
+                error: true,
+                message: `Wappi: таймаут ${FETCH_TIMEOUTS.send}мс при отправке сообщения через ${messangerType} — статус неизвестен`,
+                data: null,
+            };
+        }
 
         if (data.status !== "done") {
             return { error: true, message: `Ошибка при отправке сообщения в ChatApp через ${messangerType}`, data };
@@ -170,21 +202,40 @@ export class Wappi {
             url = `https://wappi.pro/maxapi/async/message/file/url/send?${sendOpenLine}profile_id=${profile_id}`;
         }
 
-        const res = await fetch(url!, {
-            method: "POST",
-            headers: {
-                "Content-Type": "application/json",
-                Authorization: token!,
-            },
-            body: JSON.stringify({
-                recipient: phone,
-                url: fileUrl,
-                file_name: fileName,
-                caption: message,
-                b64_file: base64,
-            }),
-        });
-        const data: any = await res.json();
+        let data: any;
+        try {
+            // В теле уходит base64 файла — бюджет transfer, а не send
+            const res = await fetchWithTimeout(
+                url!,
+                {
+                    method: "POST",
+                    headers: {
+                        "Content-Type": "application/json",
+                        Authorization: token!,
+                    },
+                    body: JSON.stringify({
+                        recipient: phone,
+                        url: fileUrl,
+                        file_name: fileName,
+                        caption: message,
+                        b64_file: base64,
+                    }),
+                },
+                FETCH_TIMEOUTS.transfer
+            );
+            data = await res.json();
+        } catch (error: unknown) {
+            // Перехватываем только таймаут, остальные ошибки ведут себя как раньше
+            if (!isTimeoutError(error)) throw error;
+
+            logs.add(`[${phone}] Wappi: таймаут ${FETCH_TIMEOUTS.transfer}мс при отправке файла через ${messangerType} — статус неизвестен, повторять нельзя`, "error");
+
+            return {
+                error: true,
+                message: `Wappi: таймаут ${FETCH_TIMEOUTS.transfer}мс при отправке файла через ${messangerType} — статус неизвестен`,
+                data: null,
+            };
+        }
 
         if (data.status !== "done") {
             return { error: true, message: `Ошибка при отправке файла в ChatApp через ${messangerType}`, data };
@@ -219,6 +270,14 @@ export class Wappi {
                 return { error: false, message: `Номер ${phone} проверен для мессенджера ${messangerType}`, data: contactCheck.data };
             }
 
+            // Ответа не дождались — это не «контакта нет». Создавать контакт вслепую нельзя:
+            // он мог уже быть в адресной книге, а результат объявил бы номер проверенным,
+            // хотя проверки не было
+            if (contactCheck.timeout) {
+                logs.add(`[${phone}] Wappi: таймаут ${FETCH_TIMEOUTS.api}мс при проверке контакта в Telegram — контакт не создаём`, "warn");
+                return { error: true, message: `Wappi: таймаут ${FETCH_TIMEOUTS.api}мс при проверке контакта ${phone} в Telegram`, data: null };
+            }
+
             // 3 Если не существует, то номера в телеге нет, пробуем создать
             const addContactResult = await this.addContactTelegramWappi(phone);
 
@@ -236,14 +295,29 @@ export class Wappi {
             url = `https://wappi.pro/maxapi/sync/contact/check?profile_id=${profile_id}&phone=${phone}`;
         }
 
-        const res = await fetch(url!, {
-            method: "GET",
-            headers: {
-                "Content-Type": "application/json",
-                Authorization: token!,
-            },
-        });
-        const data: any = await res.json();
+        let data: any;
+        try {
+            const res = await fetchWithTimeout(
+                url!,
+                {
+                    method: "GET",
+                    headers: {
+                        "Content-Type": "application/json",
+                        Authorization: token!,
+                    },
+                },
+                FETCH_TIMEOUTS.api
+            );
+            data = await res.json();
+        } catch (error: unknown) {
+            // Перехватываем только таймаут, остальные ошибки ведут себя как раньше
+            if (!isTimeoutError(error)) throw error;
+
+            // warn, а не error: проверка номера ничего не меняет, повторить её безопасно
+            logs.add(`[${phone}] Wappi: таймаут ${FETCH_TIMEOUTS.api}мс при проверке номера в ${messangerType}`, "warn");
+
+            return { error: true, message: `Wappi: таймаут ${FETCH_TIMEOUTS.api}мс при проверке номера ${phone}`, data: null };
+        }
 
         if (data.status !== "done") {
             return { error: true, message: `Ошибка проверки номера ${phone} ${messangerType}`, data };
@@ -267,13 +341,17 @@ export class Wappi {
         const url = `https://wappi.pro/tapi/sync/contact/get?profile_id=${profile_id}&phone=${phone}`;
 
         try {
-            const res = await fetch(url, {
-                method: "GET",
-                headers: {
-                    "Content-Type": "application/json",
-                    Authorization: token,
+            const res = await fetchWithTimeout(
+                url,
+                {
+                    method: "GET",
+                    headers: {
+                        "Content-Type": "application/json",
+                        Authorization: token,
+                    },
                 },
-            });
+                FETCH_TIMEOUTS.api
+            );
 
             const data: any = await res.json();
 
@@ -286,6 +364,13 @@ export class Wappi {
             return { error: false, message: `Контакт ${phone} получен из Telegram`, data };
         } catch (error: unknown) {
             const errMsg = error instanceof Error ? error.message : String(error);
+
+            // Отмечаем таймаут отдельно: вызывающий phoneCheckWappi иначе примет
+            // «не дождались ответа» за «контакта нет» и полезет создавать контакт
+            if (isTimeoutError(error)) {
+                return { error: true, message: `Ошибка при запросе контакта: ${errMsg}`, data: null, timeout: true };
+            }
+
             return { error: true, message: `Ошибка при запросе контакта: ${errMsg}`, data: null };
         }
     }
@@ -304,17 +389,21 @@ export class Wappi {
         const url = `https://wappi.pro/tapi/sync/contact/add?profile_id=${profile_id}`;
 
         try {
-            const res = await fetch(url, {
-                method: "POST",
-                headers: {
-                    "Content-Type": "application/json",
-                    Authorization: token,
+            const res = await fetchWithTimeout(
+                url,
+                {
+                    method: "POST",
+                    headers: {
+                        "Content-Type": "application/json",
+                        Authorization: token,
+                    },
+                    body: JSON.stringify({
+                        recipient: phone,
+                        name: contactName,
+                    }),
                 },
-                body: JSON.stringify({
-                    recipient: phone,
-                    name: contactName,
-                }),
-            });
+                FETCH_TIMEOUTS.send
+            );
 
             const data: any = await res.json();
             if (data.status !== "done") {
@@ -325,14 +414,16 @@ export class Wappi {
             return { error: false, message: `Контакт ${phone} успешно создан в Telegram`, data };
         } catch (error: unknown) {
             const errMsg = error instanceof Error ? error.message : String(error);
-            return { error: true, message: `Ошибка при создании контакта: ${errMsg}`, data: null };
+            // Создание контакта неидемпотентно: при таймауте ответ потерян, а контакт мог создаться
+            const unknownStatus = isTimeoutError(error) ? " — контакт мог быть создан" : "";
+            return { error: true, message: `Ошибка при создании контакта: ${errMsg}${unknownStatus}`, data: null };
         }
     }
 
     /** Скачиваем файл по ссылке и конвертируем в base64 */
     private async convertToBase64(fileUrl: string): Promise<{ error: boolean; message?: string; data?: string }> {
         try {
-            const response = await fetch(fileUrl);
+            const response = await fetchWithTimeout(fileUrl, undefined, FETCH_TIMEOUTS.transfer);
             if (!response.ok) {
                 throw new Error(`HTTP error! status: ${response.status}`);
             }