@andrey4emk/npm-app-back-b24 3.9.0 → 4.0.1

package/sendMessage/smsgold.ts

@@ -27,6 +27,15 @@ interface SmsMessageData {
  */
 const DEFAULT_UPLOAD_FOLDER_ID = 2792881;
 
+/**
+ * Файл, загруженный на диск Bitrix24 — то, что классу нужно от ответа
+ * `disk.folder.uploadfile`. Идентификатор портал отдаёт то строкой, то числом,
+ * и класс лишь передаёт его дальше в `disk.file.getExternalLink`
+ */
+interface UploadedFile {
+    ID: string | number;
+}
+
 /** Результат отправки SMS */
 interface SmsResult {
     error: boolean;
@@ -175,7 +184,7 @@ export class Smsgold {
     }
 
     /** Загружает файл в папку Битрикс24 через метод disk.folder.uploadfile */
-    private async uploadFileInFolderB24(fileBuffer: Buffer, fileName: string): Promise<{ error: boolean; message?: string; data?: any }> {
+    private async uploadFileInFolderB24(fileBuffer: Buffer, fileName: string): Promise<{ error: boolean; message?: string; data?: UploadedFile }> {
         try {
             const base64File = fileBuffer.toString("base64");
 
@@ -185,7 +194,7 @@ export class Smsgold {
                 data: { NAME: fileName },
                 fileContent: [fileName, base64File],
             });
-            const data = getResultData(res, "disk.folder.uploadfile");
+            const data = getResultData<UploadedFile>(res, "disk.folder.uploadfile");
             return { error: false, data };
         } catch (error: unknown) {
             const errMsg = error instanceof Error ? error.message : String(error);

package/sendMessage/wappi.ts

@@ -25,11 +25,31 @@ interface WappiFileMessageData extends WappiMessageData {
     fileName: string;
 }
 
+/**
+ * Ответ Wappi в той части, которую разбирает класс.
+ *
+ * Описан по фактически читаемым полям, а не по всему API. Поля `check` и `haveContact`
+ * в ответе Wappi отсутствуют — их дописывает сам класс в объект ответа, поэтому
+ * в интерфейсе они обязаны быть, иначе присваивание не пройдёт тайпчек
+ */
+interface WappiResponse {
+    status?: string;
+    detail?: string;
+    on_max?: boolean;
+    on_whatsapp?: boolean;
+    contact?: unknown;
+    /** Дописывается классом: номер пригоден для отправки */
+    check?: boolean;
+    /** Дописывается классом: контакт найден или создан */
+    haveContact?: boolean;
+}
+
 /** Стандартный результат операции */
 interface WappiResult {
     error: boolean;
     message: string;
-    data: any;
+    /** Ответ Wappi как есть — форму задаёт сервис, потребитель приводит сам */
+    data: unknown;
     /**
      * Ответа не дождались. Отличает «проверка сказала нет» от «проверки не было»:
      * без этого признака вызывающий код принимает таймаут за отрицательный ответ.
@@ -93,7 +113,7 @@ export class Wappi {
             url = `https://wappi.pro/maxapi/sync/message/send?${sendOpenLine}profile_id=${profile_id}`;
         }
 
-        let data: any;
+        let data: WappiResponse;
         try {
             const res = await fetchWithTimeout(
                 url!,
@@ -110,7 +130,7 @@ export class Wappi {
                 },
                 FETCH_TIMEOUTS.send
             );
-            data = await res.json();
+            data = (await res.json()) as WappiResponse;
         } catch (error: unknown) {
             // Метод исторически без try/catch — перехватываем только таймаут, который сами
             // и вводим, чтобы не появился новый путь исключений там, где его сегодня нет.
@@ -202,7 +222,7 @@ export class Wappi {
             url = `https://wappi.pro/maxapi/async/message/file/url/send?${sendOpenLine}profile_id=${profile_id}`;
         }
 
-        let data: any;
+        let data: WappiResponse;
         try {
             // В теле уходит base64 файла — бюджет transfer, а не send
             const res = await fetchWithTimeout(
@@ -223,7 +243,7 @@ export class Wappi {
                 },
                 FETCH_TIMEOUTS.transfer
             );
-            data = await res.json();
+            data = (await res.json()) as WappiResponse;
         } catch (error: unknown) {
             // Перехватываем только таймаут, остальные ошибки ведут себя как раньше
             if (!isTimeoutError(error)) throw error;
@@ -265,9 +285,14 @@ export class Wappi {
             const contactCheck = await this.getContactTelegramWappi(phone);
 
             // 2 Если существует, то номер в телеге есть, можем отправлять сообщение
-            if (!contactCheck.error && contactCheck.data.haveContact) {
-                contactCheck.data.check = true;
-                return { error: false, message: `Номер ${phone} проверен для мессенджера ${messangerType}`, data: contactCheck.data };
+            // Приведение, а не проверка: `data` объявлен unknown только в публичном типе,
+            // внутри класса это всегда разобранный ответ Wappi. На ошибке там null,
+            // но до чтения поля не доходит — первым проверяется `!contactCheck.error`,
+            // порядок вычисления тот же, что и до 4.0.0
+            const contactData = contactCheck.data as WappiResponse;
+            if (!contactCheck.error && contactData.haveContact) {
+                contactData.check = true;
+                return { error: false, message: `Номер ${phone} проверен для мессенджера ${messangerType}`, data: contactData };
             }
 
             // Ответа не дождались — это не «контакта нет». Создавать контакт вслепую нельзя:
@@ -280,14 +305,15 @@ export class Wappi {
 
             // 3 Если не существует, то номера в телеге нет, пробуем создать
             const addContactResult = await this.addContactTelegramWappi(phone);
+            const addedData = addContactResult.data as WappiResponse;
 
             if (!addContactResult.error) {
                 // Если получается создать, отправляем информацию
-                addContactResult.data.check = true;
-                return { error: false, message: `Номер ${phone} успешно добавлен и проверен для мессенджера ${messangerType}`, data: addContactResult.data };
+                addedData.check = true;
+                return { error: false, message: `Номер ${phone} успешно добавлен и проверен для мессенджера ${messangerType}`, data: addedData };
             }
             // Если не получается, то отправляем информацию об ошибке
-            return { error: true, message: `Не удалось проверить и добавить номер ${phone} в Telegram: ${addContactResult.message}`, data: addContactResult.data };
+            return { error: true, message: `Не удалось проверить и добавить номер ${phone} в Telegram: ${addContactResult.message}`, data: addedData };
         }
         if (messangerType === "max") {
             token = this.maxAuth.token;
@@ -295,7 +321,7 @@ export class Wappi {
             url = `https://wappi.pro/maxapi/sync/contact/check?profile_id=${profile_id}&phone=${phone}`;
         }
 
-        let data: any;
+        let data: WappiResponse;
         try {
             const res = await fetchWithTimeout(
                 url!,
@@ -308,7 +334,7 @@ export class Wappi {
                 },
                 FETCH_TIMEOUTS.api
             );
-            data = await res.json();
+            data = (await res.json()) as WappiResponse;
         } catch (error: unknown) {
             // Перехватываем только таймаут, остальные ошибки ведут себя как раньше
             if (!isTimeoutError(error)) throw error;
@@ -353,7 +379,7 @@ export class Wappi {
                 FETCH_TIMEOUTS.api
             );
 
-            const data: any = await res.json();
+            const data = (await res.json()) as WappiResponse;
 
             if (data.status !== "done") {
                 return { error: true, message: `Ошибка получения контакта ${phone} из Telegram`, data };
@@ -405,7 +431,7 @@ export class Wappi {
                 FETCH_TIMEOUTS.send
             );
 
-            const data: any = await res.json();
+            const data = (await res.json()) as WappiResponse;
             if (data.status !== "done") {
                 return { error: true, message: `Ошибка создания контакта ${phone} в Telegram`, data };
             }