@andrey4emk/npm-app-back-b24 3.8.2 → 4.0.0

package/sendMessage/chatApp.ts

@@ -45,6 +45,63 @@ interface FileMessageData extends MessageData {
     fileName: string;
 }
 
+/**
+ * Ответ ChatApp в той части, которую разбирает класс.
+ *
+ * Описан по фактически читаемым полям, а не по всему API: остальное уезжает
+ * потребителю в `data` как есть и типом не сужается
+ */
+interface ChatAppResponse {
+    success?: boolean;
+    /** Полезная нагрузка ответа. Форму задаёт сам ChatApp, разбираем на месте */
+    data?: unknown;
+    /** Дописывается классом в phoneCheckChatApp. В ответе ChatApp этого поля нет */
+    check?: unknown;
+}
+
+/**
+ * Комплект токенов ChatApp — то, что сервис обещает в ответе на выдачу и обновление.
+ *
+ * Поля необязательные намеренно: обязательность в чужом JSON ничем не подкреплена,
+ * а усечённый ответ клал бы `undefined` под типом `string`. Комплект проверяется
+ * на месте разбора
+ */
+interface ChatAppTokens {
+    accessToken?: string;
+    accessTokenEndTime?: string;
+    refreshToken?: string;
+    refreshTokenEndTime?: string;
+}
+
+/** Ответ на выдачу или обновление токена */
+interface ChatAppTokenResponse {
+    success?: boolean;
+    data?: ChatAppTokens;
+}
+
+/**
+ * Проверяет, что в ответе пришёл весь комплект токенов.
+ *
+ * Частичный ответ до этой проверки клал `undefined` в `this.auth` под типом `string`:
+ * метод отчитывался успехом, а следующее обновление уходило с пустым refresh-токеном
+ * и не восстанавливалось до перезапуска процесса
+ */
+function isFullTokenSet(tokens: ChatAppTokens | undefined): tokens is ChatAppAuthParam {
+    return !!(
+        tokens &&
+        typeof tokens.accessToken === "string" && tokens.accessToken.length > 0 &&
+        typeof tokens.accessTokenEndTime === "string" &&
+        typeof tokens.refreshToken === "string" && tokens.refreshToken.length > 0 &&
+        typeof tokens.refreshTokenEndTime === "string"
+    );
+}
+
+/** Ответ на запрос списка лицензий */
+interface ChatAppLicensesResponse {
+    success?: boolean;
+    licenses?: unknown;
+}
+
 /** Стандартный результат операции */
 interface ChatAppResult {
     error: boolean;
@@ -136,7 +193,7 @@ export class ChatApp {
 
         const url = `${baseUrl}/chats/${phone}/messages/text`;
 
-        let data: any;
+        let data: ChatAppResponse;
         try {
             const res = await fetchWithTimeout(
                 url,
@@ -152,7 +209,7 @@ export class ChatApp {
                 },
                 FETCH_TIMEOUTS.send
             );
-            data = await res.json();
+            data = (await res.json()) as ChatAppResponse;
         } catch (error: unknown) {
             // Метод исторически без try/catch — перехватываем только таймаут, который сами
             // и вводим. Остальные ошибки летят наружу как раньше
@@ -210,7 +267,7 @@ export class ChatApp {
 
         const url = `${baseUrl}/chats/${phone}/messages/file`;
 
-        let data: any;
+        let data: ChatAppResponse;
         try {
             // Файл по ссылке забирает сам ChatApp, поэтому ответа можно ждать дольше — бюджет transfer
             const res = await fetchWithTimeout(
@@ -227,7 +284,7 @@ export class ChatApp {
                 },
                 FETCH_TIMEOUTS.transfer
             );
-            data = await res.json();
+            data = (await res.json()) as ChatAppResponse;
         } catch (error: unknown) {
             // Перехватываем только таймаут, остальные ошибки ведут себя как раньше
             if (!isTimeoutError(error)) throw error;
@@ -280,7 +337,7 @@ export class ChatApp {
 
         const url = `${baseUrl}/phones/${phone}/check`;
 
-        let data: any;
+        let data: ChatAppResponse;
         try {
             const res = await fetchWithTimeout(
                 url,
@@ -294,7 +351,7 @@ export class ChatApp {
                 },
                 FETCH_TIMEOUTS.api
             );
-            data = await res.json();
+            data = (await res.json()) as ChatAppResponse;
         } catch (error: unknown) {
             // Перехватываем только таймаут, остальные ошибки ведут себя как раньше
             if (!isTimeoutError(error)) throw error;
@@ -315,13 +372,14 @@ export class ChatApp {
             newAuth = this.auth;
             this.updateToken = false;
         }
-        // Ответ разбирается как any — noUncheckedIndexedAccess здесь ничего не ловит,
-        // а обращение к data.data.exist без проверки давало TypeError мимо всякого try/catch
-        if (typeof data.data !== "object" || data.data === null) {
+        // Полезная нагрузка ответа объявлена unknown: обращение к data.data.exist
+        // без проверки давало TypeError мимо всякого try/catch
+        const payload = data.data;
+        if (typeof payload !== "object" || payload === null) {
             return { error: true, message: `Ответ ChatApp без поля data при проверке телефона через ${messangerType}`, data, auth: newAuth };
         }
 
-        data.check = data.data.exist;
+        data.check = (payload as Record<string, unknown>)["exist"];
         return { error: false, message: `Телефон успешно проверен в ChatApp через ${messangerType}`, data, auth: newAuth };
     }
 
@@ -340,9 +398,14 @@ export class ChatApp {
                 FETCH_TIMEOUTS.quick
             );
 
-            const data: any = await response.json();
+            const data = (await response.json()) as ChatAppResponse;
             if (!data.success) {
-                await this.refreshTokenChatApp();
+                // Результат обновления не проглатываем: раньше «Токен обновлен» уходило
+                // и когда обновление провалилось, а мёртвый токен доезжал до отправки
+                const refreshResult = await this.refreshTokenChatApp();
+                if (refreshResult.error) {
+                    return { error: true, message: `Не удалось обновить токен ChatApp: ${refreshResult.message ?? ""}`, data: null };
+                }
                 return { error: false, message: "Токен обновлен" };
             }
             return { error: false, message: "Токен действителен" };
@@ -353,7 +416,6 @@ export class ChatApp {
     }
 
     async makeTokenChatApp(): Promise<{ error: boolean; message?: string; data?: unknown }> {
-        console.warn("makeTokenChatApp called");
         try {
             const response = await fetchWithTimeout(
                 "https://api.chatapp.online/v1/tokens",
@@ -373,15 +435,24 @@ export class ChatApp {
                 FETCH_TIMEOUTS.quick
             );
 
-            const data: any = await response.json();
+            const data = (await response.json()) as ChatAppTokenResponse;
             if (!data.success) {
                 return { error: true, message: "Не удалось получить токен ChatApp", data };
             }
 
-            this.auth.accessToken = data.data.accessToken;
-            this.auth.accessTokenEndTime = data.data.accessTokenEndTime;
-            this.auth.refreshToken = data.data.refreshToken;
-            this.auth.refreshTokenEndTime = data.data.refreshTokenEndTime;
+            // Успех без комплекта токенов раньше давал TypeError на undefined и уходил
+            // в catch общим сообщением. Отказ явный, флаг ошибки тот же.
+            // Проверяем весь комплект, а не наличие контейнера: усечённый ответ клал бы
+            // undefined в this.auth, и обновление токена не восстановилось бы до перезапуска
+            const tokens = data.data;
+            if (!isFullTokenSet(tokens)) {
+                return { error: true, message: "ChatApp: в ответе нет данных токена", data };
+            }
+
+            this.auth.accessToken = tokens.accessToken;
+            this.auth.accessTokenEndTime = tokens.accessTokenEndTime;
+            this.auth.refreshToken = tokens.refreshToken;
+            this.auth.refreshTokenEndTime = tokens.refreshTokenEndTime;
             this.updateToken = true;
             return { error: false };
         } catch (error: unknown) {
@@ -406,7 +477,7 @@ export class ChatApp {
                 FETCH_TIMEOUTS.quick
             );
 
-            const data: any = await response.json();
+            const data = (await response.json()) as ChatAppTokenResponse;
             if (!data.success) {
                 const resMakeToken = await this.makeTokenChatApp();
                 if (resMakeToken.error) {
@@ -415,10 +486,16 @@ export class ChatApp {
                 return { error: false };
             }
 
-            this.auth.accessToken = data.data.accessToken;
-            this.auth.accessTokenEndTime = data.data.accessTokenEndTime;
-            this.auth.refreshToken = data.data.refreshToken;
-            this.auth.refreshTokenEndTime = data.data.refreshTokenEndTime;
+            // Тот же явный отказ, что и в makeTokenChatApp: успех без комплекта токенов
+            const tokens = data.data;
+            if (!isFullTokenSet(tokens)) {
+                return { error: true, message: "ChatApp: в ответе нет данных токена", data };
+            }
+
+            this.auth.accessToken = tokens.accessToken;
+            this.auth.accessTokenEndTime = tokens.accessTokenEndTime;
+            this.auth.refreshToken = tokens.refreshToken;
+            this.auth.refreshTokenEndTime = tokens.refreshTokenEndTime;
             this.updateToken = true;
             return { error: false };
         } catch (error: unknown) {
@@ -441,7 +518,7 @@ export class ChatApp {
                 },
                 FETCH_TIMEOUTS.api
             );
-            const data: any = await response.json();
+            const data = (await response.json()) as ChatAppLicensesResponse;
 
             if (!data.success) {
                 return { error: true, message: "Не удалось получить лицензии ChatApp", data: null };

package/sendMessage/email.ts

@@ -1,4 +1,5 @@
-import nodemailer from "nodemailer";
+import nodemailer, { type SendMailOptions } from "nodemailer";
+import { logs } from "../logs/logs.ts";
 import { fetchWithTimeout, FETCH_TIMEOUTS } from "../utils/fetchRetry.ts";
 
 // ==================== Типы ====================
@@ -9,6 +10,12 @@ interface EmailAuthParam {
     pass: string;
 }
 
+/** Вложение письма */
+interface EmailAttachment {
+    filename: string;
+    content: Buffer;
+}
+
 /** Данные для отправки письма */
 interface EmailData {
     to: string;
@@ -17,7 +24,7 @@ interface EmailData {
     html?: string;
     fileUrl?: string;
     fileName?: string;
-    attachments?: { filename: string; content: Buffer }[] | null;
+    attachments?: EmailAttachment[] | null;
 }
 
 /** Результат отправки письма */
@@ -27,66 +34,260 @@ interface EmailResult {
     info?: unknown;
 }
 
+/**
+ * Ответ транспорта в той части, которую разбирает класс.
+ *
+ * Собственный узкий тип, а не `SentMessageInfo` из `@types/nodemailer`: там он
+ * объявлен как `any`, и в сигнатуру пакета это тянуть нельзя.
+ */
+export interface SendMailInfo {
+    accepted?: unknown;
+    rejected?: unknown;
+    response?: unknown;
+}
+
+/**
+ * Минимальный контракт транспорта — ровно то, что использует класс.
+ *
+ * Второй аргумент конструктора существует ради тестов и подмены транспорта
+ * у потребителя; по умолчанию класс поднимает SMTP Яндекса сам.
+ */
+export interface MailTransport {
+    sendMail(mail: SendMailOptions): Promise<SendMailInfo>;
+}
+
+// ==================== Утилиты ====================
+
+/**
+ * Грубая проверка адреса: непустая строка без пробелов, одна собака,
+ * непустые части слева и справа.
+ *
+ * Полная валидация адресов регуляркой — известная ловушка, и она здесь не нужна:
+ * цель — отсечь пустую строку и явный мусор, отказ сервера всё равно проверяется
+ * по ответу. Точку в домене не требуем: внутрикорпоративные адреса без точки существуют.
+ */
+function isLikelyEmail(value: unknown): value is string {
+    if (typeof value !== "string") return false;
+    if (/\s/.test(value)) return false;
+
+    const parts = value.split("@");
+    if (parts.length !== 2) return false;
+
+    const [local, domain] = parts;
+    return Boolean(local) && Boolean(domain);
+}
+
+/**
+ * Приводит адрес к той форме, в которой его возвращает почтовый сервер.
+ *
+ * Сравнивать «как передали» нельзя: конверт письма строит nodemailer, и в `accepted`
+ * приходит адрес из конверта — домен в punycode, угловые скобки сняты. Живой адрес
+ * `ivan@пример.рф` возвращался как `ivan@xn--e1afmkfd.xn--p1ai`, с исходным не совпадал,
+ * и доставленное письмо объявлялось недоставленным.
+ *
+ * Применяется к обеим сторонам сравнения. Локальная часть приводится к нижнему регистру
+ * намеренно: формально она регистрозависима, но здесь адрес сверяется сам с собой,
+ * а не с чужим.
+ */
+function normalizeAddress(value: string): string {
+    let address = value.trim();
+
+    if (address.startsWith("<") && address.endsWith(">")) {
+        address = address.slice(1, -1).trim();
+    }
+
+    // Разделяем по последней собаке: в локальной части она допустима в кавычках
+    const separator = address.lastIndexOf("@");
+    if (separator <= 0) return address.toLowerCase();
+
+    const local = address.slice(0, separator);
+    const domain = address.slice(separator + 1);
+
+    let host: string;
+    try {
+        // Доступный в Node способ получить punycode-форму домена: разбор хоста в URL
+        // делает то же преобразование, что и nodemailer при построении конверта
+        host = new URL(`http://${domain}`).hostname;
+    } catch {
+        // Домен как хост не разбирается (пустой, с недопустимым символом) — сверяем как есть
+        host = domain.toLowerCase();
+    }
+
+    return `${local.toLowerCase()}@${host}`;
+}
+
+/**
+ * Приводит `accepted`/`rejected` к списку нормализованных адресов.
+ *
+ * Элементы приходят строкой либо объектом `{ name, address }` — форма зависит
+ * от того, как адрес был передан в `sendMail`.
+ */
+function toAddressList(value: unknown): string[] {
+    if (!Array.isArray(value)) return [];
+
+    const addresses: string[] = [];
+
+    for (const item of value as unknown[]) {
+        if (typeof item === "string") {
+            addresses.push(item);
+            continue;
+        }
+
+        if (item && typeof item === "object") {
+            const address = (item as { address?: unknown }).address;
+            if (typeof address === "string") addresses.push(address);
+        }
+    }
+
+    return addresses.map((address) => normalizeAddress(address)).filter((address) => address !== "");
+}
+
+/**
+ * Имя для скачанного вложения: явно переданное `fileName`, иначе последний
+ * сегмент пути из URL (без query), иначе запасное `attachment`.
+ *
+ * Прежде здесь стояло `dataMail.fileName!`, и письмо с `fileUrl` без имени
+ * уезжало с вложением без имени файла.
+ */
+function resolveAttachmentName(fileUrl: string, fileName?: string): string {
+    const explicit = fileName?.trim();
+    if (explicit) return explicit;
+
+    try {
+        const segment = new URL(fileUrl).pathname.split("/").pop();
+        const decoded = segment ? decodeURIComponent(segment) : "";
+        if (decoded) return decoded;
+    } catch {
+        // Адрес может быть относительным или битым — тогда берём запасное имя
+    }
+
+    return "attachment";
+}
+
 // ==================== Класс ====================
 
 export class Email {
     private auth: EmailAuthParam;
-    private transporter: ReturnType<typeof nodemailer.createTransport>;
+    private transporter: MailTransport;
 
-    constructor(auth: EmailAuthParam) {
+    constructor(auth: EmailAuthParam, transporter?: MailTransport) {
         this.auth = auth;
-        this.transporter = nodemailer.createTransport({
-            auth: {
-                user: this.auth.user,
-                pass: this.auth.pass,
-            },
-            host: "smtp.yandex.ru",
-            port: 465, // 465 = SMTPS, 587 = STARTTLS
-            secure: true, // true для 465
-            // Дефолты nodemailer (соединение 2 мин, приветствие 30с, сокет 10 мин) слишком щедры:
-            // письмо с вложением уходит на Яндекс за секунды, а 10 минут молчания сокета
-            // блокируют вызывающую очередь
-            connectionTimeout: 15_000,
-            greetingTimeout: 10_000,
-            socketTimeout: 120_000,
-        });
+        this.transporter =
+            transporter ??
+            nodemailer.createTransport({
+                auth: {
+                    user: this.auth.user,
+                    pass: this.auth.pass,
+                },
+                host: "smtp.yandex.ru",
+                port: 465, // 465 = SMTPS, 587 = STARTTLS
+                secure: true, // true для 465
+                // Дефолты nodemailer (соединение 2 мин, приветствие 30с, сокет 10 мин) слишком щедры:
+                // письмо с вложением уходит на Яндекс за секунды, а 10 минут молчания сокета
+                // блокируют вызывающую очередь
+                connectionTimeout: 15_000,
+                greetingTimeout: 10_000,
+                socketTimeout: 120_000,
+            });
     }
 
+    /**
+     * Отправляет письмо получателю, копию — на адрес отправителя.
+     *
+     * Адрес получателя проверяется до вызова транспорта: пустой или мусорный `to`
+     * раньше давал «успешную» отправку копии себе и ни одной жалобы в логе.
+     *
+     * Вложения вызывающего (`attachments`) сохраняются и дополняются файлом,
+     * скачанным по `fileUrl`. Входной объект не мутируется.
+     *
+     * @param dataMail — адресат, тема, тело письма, вложения
+     * @returns `{ error, message?, info? }`; `error: true` приходит и тогда, когда
+     *          сервер принял копию отправителю, но отверг получателя
+     */
     async send(dataMail: EmailData): Promise<EmailResult> {
         try {
-            // Если прислали ссылку на файл, то скачиваем его и добавляем в attachments
+            const to = typeof dataMail.to === "string" ? dataMail.to.trim() : "";
+            if (!isLikelyEmail(to)) {
+                return { error: true, message: `Email: адрес получателя некорректен ('${String(dataMail.to)}') — письмо не отправлено` };
+            }
+
+            // Вложения вызывающего не затираем и входной объект не мутируем:
+            // `attachments` и пара `fileUrl`/`fileName` объявлены независимыми полями,
+            // ни одно не документировано как «вместо». Тот, кто заполнил оба, хочет оба;
+            // замена молча теряла данные. Порядок: сначала переданные, затем скачанное
+            const attachments: EmailAttachment[] = [...(dataMail.attachments ?? [])];
+
             if (dataMail.fileUrl) {
                 const resDownloadFile = await this.downloadFileToUrl(dataMail.fileUrl);
-                if (resDownloadFile.error) {
+                if (resDownloadFile.error || !resDownloadFile.buffer) {
                     // Подстановка вместо возможного undefined: при exactOptionalPropertyTypes
                     // у потребителя явный undefined не подходит полю message?: string
                     return { error: true, message: resDownloadFile.message ?? "Не удалось скачать файл по fileUrl" };
                 }
-                dataMail.attachments = [
-                    {
-                        filename: dataMail.fileName!,
-                        content: resDownloadFile.buffer!,
-                    },
-                ];
-            } else {
-                dataMail.attachments = null;
+
+                attachments.push({
+                    filename: resolveAttachmentName(dataMail.fileUrl, dataMail.fileName),
+                    content: resDownloadFile.buffer,
+                });
             }
 
             const info = await this.transporter.sendMail({
                 from: `Натяжные потолки Репа. <${this.auth.user}>`,
-                to: [dataMail.to, this.auth.user],
+                to: [to, this.auth.user],
                 subject: dataMail.subject,
                 text: dataMail.text,
                 html: dataMail.html,
-                attachments: dataMail.attachments ?? undefined,
+                attachments: attachments.length > 0 ? attachments : undefined,
             });
-            return { error: false, info };
+
+            return this.checkDelivery(to, info);
         } catch (error: unknown) {
             const message = error instanceof Error ? error.message : String(error);
             return { error: true, message };
         }
     }
 
+    /**
+     * Проверяет по ответу транспорта, принял ли сервер основного адресата.
+     *
+     * В `to` всегда уходит пара «получатель + отправитель», а `sendMail` резолвится,
+     * если принят хотя бы один адрес. Без этой проверки отказ по получателю
+     * возвращался вызывающему как `error: false`.
+     *
+     * Отказ логируется не здесь: результат уходит вызывающему, а уровень `error`
+     * дублировался бы в чат B24. От таймаутов отправки в `Smsgold`/`Wappi` случай
+     * отличается тем, что статус известен — сервер прямо сказал «не принял».
+     */
+    private checkDelivery(to: string, info: SendMailInfo): EmailResult {
+        const hasReport = Array.isArray(info?.accepted) || Array.isArray(info?.rejected);
+
+        // Транспорт вообще не сообщил, какие адреса приняты. Отказ здесь не выдумываем:
+        // у SMTP-транспорта эти поля есть всегда, а объявить успешно доставленное письмо
+        // ошибкой — регресс хуже исходного дефекта. Успех плюс строка в лог
+        if (!hasReport) {
+            logs.add("Email: транспорт не сообщил, какие адреса приняты — проверить доставку нечем", "warn");
+            return { error: false, info };
+        }
+
+        // Обе стороны сравнения нормализуем одинаково: сервер отдаёт адреса из конверта,
+        // а не в том виде, в каком их передали
+        const target = normalizeAddress(to);
+        const accepted = toAddressList(info.accepted);
+        const rejected = toAddressList(info.rejected);
+
+        if (rejected.includes(target) || !accepted.includes(target)) {
+            const response = typeof info.response === "string" && info.response.trim() !== "" ? ` Ответ сервера: ${info.response}` : "";
+
+            return {
+                error: true,
+                message: `Email: почтовый сервер не принял адрес получателя ${to}, письмо ему не ушло (копия отправителю могла уйти).${response}`,
+                info,
+            };
+        }
+
+        return { error: false, info };
+    }
+
     private async downloadFileToUrl(fileUrl: string): Promise<{ error: boolean; message?: string; buffer?: Buffer }> {
         try {
             const response = await fetchWithTimeout(fileUrl, undefined, FETCH_TIMEOUTS.transfer);

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);