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

package/README.md

@@ -44,6 +44,12 @@ import { fetchRetry, maskUrl } from "@andrey4emk/npm-app-back-b24/fetchRetry";
 
 Корневой импорт при этом остаётся безопасным: если `APP_B24_CLIENT_ID` и `APP_B24_CLIENT_SECRET` не заданы, пакет считает, что OAuth в проекте не используется, молча выставляет `$b24 = null` и пишет об этом только в `debug`.
 
+### Строгость типов
+
+Пакет публикуется исходниками на TypeScript и компилируется **в программе потребителя** — значит его код проверяется настройками потребителя, а не нашими. Поэтому сам пакет собирается с `strict` и `noUncheckedIndexedAccess`: на настройках Nuxt (где оба флага включены) штатный `nuxt typecheck` проходит по коду пакета без обходных скриптов и без `skipLibCheck`-заплаток.
+
+Практическое следствие: индексация массива внутри пакета всегда сопровождается проверкой на `undefined`. Если в своём проекте эти флаги включены — ошибок из `node_modules/@andrey4emk/npm-app-back-b24` быть не должно.
+
 ## Использование
 
 ```js
@@ -493,6 +499,10 @@ const response = await fetchRetry("https://api.example.com/data", { method: "GET
     );
     ```
 
+    Конструктор сразу разрешает тип мессенджера для `whatsApp` и `telegram` — берёт `messenger[0].type` и держит в полях, вместо того чтобы индексировать массив при каждой отправке. Если конфига нет или список `messenger` пуст, тип остаётся неразрешённым, и **методы этого мессенджера возвращают** `{ error: true, message: "ChatApp: не разрешён тип мессенджера whatsApp (нет конфига или пуст messenger)", data: null }` — вместо запроса на URL с `undefined` в пути. Конструктор при этом **не бросает**: экземпляр обычно создаётся на уровне модуля, и бросок уронил бы загрузку всего модуля отправки вместе с соседними каналами.
+
+    Отказ локальный: пустой `whatsApp` не мешает `telegram`. Но конфиг всё равно нужен для **обоих** мессенджеров, даже если проект пользуется одним — иначе второй канал молча перестанет работать, и узнать об этом можно будет только по `res.error` в логе.
+
     **Методы:**
 
     -   **`makeTokenChatApp()`** — получить токены доступа для ChatApp API.
@@ -629,6 +639,8 @@ const response = await fetchRetry("https://api.example.com/data", { method: "GET
     -   **Особенности:**
         -   SMTP-сервер: `smtp.yandex.ru:465` (SMTPS).
         -   Копия письма автоматически отправляется на адрес отправителя.
+        -   Транспорт — `nodemailer` `^9` (диапазон, а не точная версия: иначе патч безопасности нельзя было бы применить на стороне потребителя).
+        -   Вложения передаются готовым `Buffer` — файл по `fileUrl` пакет скачивает сам. Поля `path` и `href` намеренно не поддерживаются: nodemailer не должен ходить по URL.
 
 ### Wappi
 

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@andrey4emk/npm-app-back-b24",
-    "version": "3.7.0",
+    "version": "3.7.1",
     "description": "Bitrix24 OAuth helpers for Node.js projects",
     "main": "index.ts",
     "type": "module",
@@ -39,12 +39,12 @@
     "dependencies": {
         "@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"
+        "@types/luxon": "3.7.4",
+        "@types/node": "25.9.5",
+        "@types/nodemailer": "8.0.1",
+        "conf": "15.1.0",
+        "dotenv": "17.4.2",
+        "luxon": "3.7.2",
+        "nodemailer": "^9.0.5"
     }
 }

package/sendMessage/chatApp.ts

@@ -57,12 +57,32 @@ export class ChatApp {
     private auth: ChatAppAuthParam;
     private type: ChatAppTypeParam;
     private updateToken: boolean;
+    private readonly whatsAppType: string | null;
+    private readonly telegramType: string | null;
+
+    /**
+     * Тип мессенджера из конфига. Массив приходит из внешнего источника
+     * (config/chatApp.json обновляется данными API ChatApp), поэтому проверяем:
+     * пустой messenger дал бы URL с undefined и «отправку в никуда».
+     * Не бросаем: экземпляр создаётся на уровне модуля, и бросок уронил бы
+     * импорт всего модуля отправки вместе с остальными каналами. Отказ локальный —
+     * его возвращают методы того мессенджера, который не разрешился.
+     */
+    private static resolveMessengerType(config: MessengerConfig | undefined): string | null {
+        const first = config?.messenger?.[0];
+        if (typeof first?.type !== "string" || !first.type.trim()) {
+            return null;
+        }
+        return first.type;
+    }
 
     constructor(makeParam: ChatAppMakeParam, authParam: ChatAppAuthParam, typeParam: ChatAppTypeParam) {
         this.make = makeParam;
         this.auth = authParam;
         this.type = typeParam;
         this.updateToken = false;
+        this.whatsAppType = ChatApp.resolveMessengerType(typeParam?.whatsApp);
+        this.telegramType = ChatApp.resolveMessengerType(typeParam?.telegram);
     }
 
     async sendMessageChatApp(messangerType: MessengerType, messageData: MessageData): Promise<ChatAppResult> {
@@ -73,6 +93,11 @@ export class ChatApp {
         if (!messageData.phone || !messageData.message) {
             return { error: true, message: "Отсутствует phone или message", data: null };
         }
+        // Тип мессенджера не разрешился из конфига — отправлять некуда
+        const resolvedType = messangerType === "whatsApp" ? this.whatsAppType : this.telegramType;
+        if (!resolvedType) {
+            return { error: true, message: `ChatApp: не разрешён тип мессенджера ${messangerType} (нет конфига или пуст messenger)`, data: null };
+        }
 
         const check = await this.checkTokenChatApp();
         if (check.error) {
@@ -84,10 +109,10 @@ export class ChatApp {
 
         let url: string | undefined;
         if (messangerType === "whatsApp") {
-            url = `https://api.chatapp.online/v1/licenses/${this.type.whatsApp.licenseId}/messengers/${this.type.whatsApp.messenger[0].type}/chats/${phone}/messages/text`;
+            url = `https://api.chatapp.online/v1/licenses/${this.type.whatsApp.licenseId}/messengers/${resolvedType}/chats/${phone}/messages/text`;
         }
         if (messangerType === "telegram") {
-            url = `https://api.chatapp.online/v1/licenses/${this.type.telegram.licenseId}/messengers/${this.type.telegram.messenger[0].type}/chats/${phone}/messages/text`;
+            url = `https://api.chatapp.online/v1/licenses/${this.type.telegram.licenseId}/messengers/${resolvedType}/chats/${phone}/messages/text`;
         }
 
         const res = await fetch(url!, {
@@ -123,6 +148,11 @@ export class ChatApp {
         if (!messageData.phone || !messageData.message) {
             return { error: true, message: "Отсутствует phone или message", data: null };
         }
+        // Тип мессенджера не разрешился из конфига — отправлять некуда
+        const resolvedType = messangerType === "whatsApp" ? this.whatsAppType : this.telegramType;
+        if (!resolvedType) {
+            return { error: true, message: `ChatApp: не разрешён тип мессенджера ${messangerType} (нет конфига или пуст messenger)`, data: null };
+        }
 
         const check = await this.checkTokenChatApp();
         if (check.error) {
@@ -136,10 +166,10 @@ export class ChatApp {
 
         let url: string | undefined;
         if (messangerType === "whatsApp") {
-            url = `https://api.chatapp.online/v1/licenses/${this.type.whatsApp.licenseId}/messengers/${this.type.whatsApp.messenger[0].type}/chats/${phone}/messages/file`;
+            url = `https://api.chatapp.online/v1/licenses/${this.type.whatsApp.licenseId}/messengers/${resolvedType}/chats/${phone}/messages/file`;
         }
         if (messangerType === "telegram") {
-            url = `https://api.chatapp.online/v1/licenses/${this.type.telegram.licenseId}/messengers/${this.type.telegram.messenger[0].type}/chats/${phone}/messages/file`;
+            url = `https://api.chatapp.online/v1/licenses/${this.type.telegram.licenseId}/messengers/${resolvedType}/chats/${phone}/messages/file`;
         }
 
         const res = await fetch(url!, {
@@ -171,6 +201,11 @@ export class ChatApp {
         if (!["whatsApp", "telegram"].includes(messangerType)) {
             return { error: true, message: "Неверный тип мессенджера", data: null };
         }
+        // Тип мессенджера не разрешился из конфига — проверять телефон негде
+        const resolvedType = messangerType === "whatsApp" ? this.whatsAppType : this.telegramType;
+        if (!resolvedType) {
+            return { error: true, message: `ChatApp: не разрешён тип мессенджера ${messangerType} (нет конфига или пуст messenger)`, data: null };
+        }
 
         const check = await this.checkTokenChatApp();
         if (check.error) {
@@ -179,10 +214,10 @@ export class ChatApp {
 
         let url: string | undefined;
         if (messangerType === "whatsApp") {
-            url = `https://api.chatapp.online/v1/licenses/${this.type.whatsApp.licenseId}/messengers/${this.type.whatsApp.messenger[0].type}/phones/${phone}/check`;
+            url = `https://api.chatapp.online/v1/licenses/${this.type.whatsApp.licenseId}/messengers/${resolvedType}/phones/${phone}/check`;
         }
         if (messangerType === "telegram") {
-            url = `https://api.chatapp.online/v1/licenses/${this.type.telegram.licenseId}/messengers/${this.type.telegram.messenger[0].type}/phones/${phone}/check`;
+            url = `https://api.chatapp.online/v1/licenses/${this.type.telegram.licenseId}/messengers/${resolvedType}/phones/${phone}/check`;
         }
 
         const res = await fetch(url!, {