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

package/logs/logs.ts

@@ -39,6 +39,17 @@ const B24_TEXT_LIMIT = 4000;
  */
 const B24_CHAT_TIMEOUT_MS = 5_000;
 
+/** Сколько символов тела неуспешного ответа чата показывать в `console.error` */
+const B24_CHAT_BODY_PREVIEW = 300;
+
+/**
+ * Ошибка — обрыв по бюджету `AbortSignal.timeout`. Проверка локальная, без импорта
+ * `isTimeoutError` из `utils/fetchRetry.ts` — тот модуль импортирует `logs`, был бы цикл
+ */
+function isTimeout(error: unknown): boolean {
+    return (error as { name?: unknown } | null)?.name === "TimeoutError";
+}
+
 // ==================== Настройки ====================
 
 const configDir: string = process.env.CONFIG_DIR || "../config";
@@ -198,31 +209,70 @@ class LogsAPI {
             text = this.stripBbCodes(text);
         }
 
-        fetch(url, {
-            method: "POST",
-            headers: { "Content-Type": "application/json" },
-            body: JSON.stringify({
-                BOT_ID: B24_BOT_ID,
-                CLIENT_ID: B24_CLIENT_ID,
-                DIALOG_ID: this.b24ChatId,
-                MESSAGE: text,
-            }),
-            signal: AbortSignal.timeout(B24_CHAT_TIMEOUT_MS),
-        })
-            .then((response) => {
-                if (!response.ok) {
-                    console.error(`Logs B24 Chat: HTTP ${response.status}`);
-                }
-            })
-            .catch((error: unknown) => {
-                // Проверка локальная, без импорта isTimeoutError — тот модуль импортирует logs
-                if ((error as { name?: unknown })?.name === "TimeoutError") {
-                    console.error(`Logs B24 Chat: не ответил за ${B24_CHAT_TIMEOUT_MS / 1000}с, сообщение не доставлено`);
-                    return;
-                }
-                const errMsg = error instanceof Error ? error.message : String(error);
-                console.error(`Logs B24 Chat: ошибка отправки — ${errMsg}`);
+        void this.postToB24Chat(url, text);
+    }
+
+    /**
+     * Сам запрос в чат и разбор ответа. Никогда не бросает: все ошибки — в `console.error`,
+     * не через `logs`, иначе отказ канала порождал бы новый error и зацикливался.
+     *
+     * Сигнал `AbortSignal.timeout` отдан в `fetch`, а он действует и на чтение тела
+     * ответа: зависшее тело обрывается тем же бюджетом с `TimeoutError`. До общего `catch`
+     * такая ошибка не доходит — у каждого чтения тела свой `.catch`, поэтому таймаут
+     * разбирается прямо там: при `!ok` к статусу дописывается пометка «тело не дочитано»,
+     * при 200 пишется отдельная строка «доставка не подтверждена».
+     */
+    private async postToB24Chat(url: string, text: string): Promise<void> {
+        try {
+            const response = await fetch(url, {
+                method: "POST",
+                headers: { "Content-Type": "application/json" },
+                body: JSON.stringify({
+                    BOT_ID: B24_BOT_ID,
+                    CLIENT_ID: B24_CLIENT_ID,
+                    DIALOG_ID: this.b24ChatId,
+                    MESSAGE: text,
+                }),
+                signal: AbortSignal.timeout(B24_CHAT_TIMEOUT_MS),
+            });
+
+            if (!response.ok) {
+                // Тело — подсказка к причине отказа, его чтение отправку уронить не должно
+                const body = await response.text().catch((error: unknown) =>
+                    isTimeout(error) ? `[тело не дочитано за ${B24_CHAT_TIMEOUT_MS / 1000}с]` : "",
+                );
+                const bodyPart = body ? ` — ${body.slice(0, B24_CHAT_BODY_PREVIEW)}` : "";
+                console.error(`Logs B24 Chat: HTTP ${response.status}, сообщение не доставлено${bodyPart}`);
+                return;
+            }
+
+            // B24 умеет отвечать 200 с телом {error, error_description} — сообщения при этом нет.
+            // Нечитаемый JSON — молча null (портал ответил 200, ошибки в теле не видно),
+            // а вот зависшее тело — отказ канала: доставку подтвердить нечем, молчать нельзя
+            let bodyTimedOut = false;
+            const data: unknown = await response.json().catch((error: unknown) => {
+                bodyTimedOut = isTimeout(error);
+                return null;
             });
+            if (bodyTimedOut) {
+                console.error(
+                    `Logs B24 Chat: HTTP ${response.status}, но ответ не дочитан за ${B24_CHAT_TIMEOUT_MS / 1000}с — доставка не подтверждена`,
+                );
+                return;
+            }
+            if (data && typeof data === "object" && "error" in data) {
+                const { error, error_description: description } = data as { error?: unknown; error_description?: unknown };
+                const reason = description ? String(description) : String(error);
+                console.error(`Logs B24 Chat: портал отклонил сообщение — ${reason}`);
+            }
+        } catch (error: unknown) {
+            if (isTimeout(error)) {
+                console.error(`Logs B24 Chat: не ответил за ${B24_CHAT_TIMEOUT_MS / 1000}с, сообщение не доставлено`);
+                return;
+            }
+            const errMsg = error instanceof Error ? error.message : String(error);
+            console.error(`Logs B24 Chat: ошибка отправки — ${errMsg}`);
+        }
     }
 }
 

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@andrey4emk/npm-app-back-b24",
-    "version": "4.0.1",
+    "version": "4.0.2",
     "description": "Bitrix24 OAuth helpers for Node.js projects",
     "main": "index.ts",
     "type": "module",