dsh-telegram-multiagent 1.5.0 → 1.5.1

package/CHANGELOG.md

@@ -1,3 +1,25 @@
+## 1.5.1
+
+🔴 **`errDetail` печатал пустоту после слова «причина».** Собирал причину через `??`, а нулевое
+слияние реагирует только на `null` и `undefined` — пустая строка для него «найденное значение»,
+и до `code` дело не доходило. У `AggregateError`, которую кладёт `fetch`, перебрав адреса,
+собственный `message` как раз пустой.
+
+В живом журнале это дало **125 строк за три часа** вида `причина: ` с пустотой после двоеточия.
+Хуже отсутствия поля: «причина:» с пустотой читается как «причину узнали, она пустая», а её
+просто не достали из объекта.
+
+Теперь причина собирается перечнем: класс ошибки (AggregateError и Error лечатся по-разному, а
+по тексту неотличимы), `code`, `message`, и внутренние ошибки `AggregateError.errors`, где у неё
+и лежат настоящие причины. Пустой перечень даёт «объект есть, полей с описанием нет» — это НЕ
+пустая причина, а другое состояние, и слова для него другие.
+
+Строка отказа теперь называет, ЧТО опрашивалось: в цикле три опроса подряд (a2a, ящик, telegram),
+и без имени отказ читается как «всё сломалось», хотя стоять может одно из трёх.
+
+Стенд `stend-prichiny-otkaza.mjs`, 5 проб, вырезает функцию ИЗ ПРЕДМЕТА. Порча возвратом `??`
+роняет три пробы из пяти.
+
 ## 1.5.0
 
 🔴 **`/compact-status` больше не выдаёт устаревшую опору за свежую.** Строка «Опора замера:

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-telegram-multiagent",
-  "version": "1.5.0",
+  "version": "1.5.1",
   "description": "Канал Telegram для DeepSeek Harness: один модуль на машину обслуживает несколько агентов, у каждого свой бот, свой файл токена и свой список разрешённых. Метка отправителя неподделываема, память владельца и координатора слита в одну, режим доставки переключается файлом вне кода, цели управляются из канала.",
   "type": "module",
   "main": "src/index.js",

package/src/index.js

@@ -1364,10 +1364,33 @@ export function apply(ctx, config = {}) {
   // «причины нет», и это та же слепота, что и раньше).
   // Перенесено из второй живой копии модуля 01.09.2026 ДОСЛОВНО: своя редакция
   // того же лечения развела бы копии по логике при сверке.
+  // 🔴 `??` ПРОПУСКАЕТ ПУСТУЮ СТРОКУ КАК ЗАДАННОЕ ЗНАЧЕНИЕ (замер 04.09.2026).
+  // Было `c?.message ?? c?.code ?? String(c)`. Нулевое слияние реагирует только на
+  // null и undefined, а `message` у AggregateError (её кладёт fetch, перебрав адреса)
+  // бывает ПУСТОЙ СТРОКОЙ — и она «найдена», до `code` дело не доходит.
+  // В журнале это дало 125 строк за три часа вида «причина: » с пустотой после
+  // двоеточия. Хуже отсутствия поля: «причина:» с пустотой читается как «причину
+  // узнали, она пустая», а её просто не достали.
+  // Воспроизведено: errDetail(cause с message='') -> '', errDetail(AggregateError) -> ''.
   function errDetail(e) {
     const c = e?.cause;
     if (!c) return 'не указана';
-    return c?.message ?? c?.code ?? String(c);
+    const chasti = [];
+    // Класс ошибки называем всегда: AggregateError и Error лечатся по-разному, а по
+    // тексту они неотличимы.
+    if (c?.constructor?.name) chasti.push(c.constructor.name);
+    if (c?.code) chasti.push(String(c.code));
+    if (c?.message) chasti.push(String(c.message));
+    // У AggregateError настоящие причины лежат в errors, а собственный message пуст.
+    if (Array.isArray(c?.errors) && c.errors.length) {
+      const vnutri = c.errors.slice(0, 3)
+        .map((x) => [x?.code, x?.message].filter(Boolean).join(' ') || String(x))
+        .filter(Boolean);
+      if (vnutri.length) chasti.push('внутри: ' + vnutri.join('; '));
+    }
+    // Пустой перечень — это НЕ пустая причина, а «объект есть, а сказать про него
+    // нечего». Разные состояния, разные слова.
+    return chasti.length ? chasti.join(' ') : `объект ${typeof c}, полей с описанием нет`;
   }
 
   async function loop(run) {
@@ -1397,7 +1420,10 @@ export function apply(ctx, config = {}) {
         }
       } catch (e) {
         // Сеть моргнула или Telegram ответил ошибкой — ждём и продолжаем.
-        log(`опрос не удался: ${e?.message ?? e} | причина: ${errDetail(e)}`);
+        // ЧТО именно опрашивалось — называем: в цикле три опроса подряд (a2a, ящик,
+        // телеграм), и без имени отказ читается как «всё сломалось», хотя стоять может
+        // одно из трёх.
+        log(`опрос не удался (a2a+ящик+telegram): ${e?.message ?? e} | причина: ${errDetail(e)}`);
         await new Promise((r) => setTimeout(r, 3000));
       }
     }

package/test/stend-prichiny-otkaza.mjs

@@ -0,0 +1,73 @@
+// Стенд разбора причины отказа опроса.
+//
+// Коды: 0 сошлось | 1 расхождение | 2 слепота.
+//
+// ЧТО ЛЕЧИТ ПРЕДМЕТ. `errDetail` собирал причину через `??`. Нулевое слияние реагирует
+// только на null и undefined, а `message` у AggregateError (её кладёт fetch, перебрав
+// адреса) бывает ПУСТОЙ СТРОКОЙ — она «найдена», и до `code` дело не доходит.
+// В живом журнале это дало 125 строк за три часа вида «причина: » с пустотой.
+// 🔴 Хуже отсутствия поля: «причина:» с пустотой читается как «причину узнали, она
+// пустая», а её просто не достали из объекта.
+//
+// 🔴 ВЫРЕЗАЕМ ИЗ ПРЕДМЕТА, а не переписываем: копия стареет молча.
+//
+// ГДЕ НЕ ПРИМЕНЯЕТСЯ: стенд проверяет только РАЗБОР объекта ошибки. Он не проверяет,
+// что отказ вообще случается, и не проверяет доставку строки в журнал.
+
+import { readFileSync } from 'node:fs'
+
+const SRC = process.env.SVYAZ_SRC || new URL('../src/index.js', import.meta.url).pathname
+let text
+try { text = readFileSync(SRC, 'utf8') } catch (e) {
+  console.log(`СЛЕПОТА: предмет ${SRC} не читается: ${e?.message ?? e}`); process.exit(2)
+}
+const m = text.match(/function errDetail\(e\) \{[\s\S]*?\n  \}/)
+if (!m) { console.log('СЛЕПОТА: функция errDetail в предмете не найдена — вырезать нечего'); process.exit(2) }
+let errDetail
+try { errDetail = new Function(m[0] + '; return errDetail;')() }
+catch (e) { console.log(`СЛЕПОТА: вырезанное не собирается: ${e?.message ?? e}`); process.exit(2) }
+
+let ok = 0, bed = 0
+const t = (imya, delo) => { try { delo(); ok++; console.log(`  ok   ${imya}`) }
+  catch (e) { bed++; console.log(`  FAIL ${imya} — ${e?.message ?? e}`) } }
+const nePusto = (v, chto) => {
+  if (typeof v !== 'string' || v.trim() === '')
+    throw new Error(`${chto}: причина пуста — «${v}»`)
+}
+
+t('cause отсутствует → «не указана», а не пустота', () => {
+  const r = errDetail(new Error('fetch failed'))
+  nePusto(r, 'нет cause'); if (!/не указана/.test(r)) throw new Error(`ждали «не указана», получили «${r}»`)
+})
+
+// 🔴 ГЛАВНАЯ: ровно тот случай, что дал 125 пустых строк в бою.
+t('cause.message ПУСТАЯ СТРОКА → причина всё равно названа', () => {
+  const c = Object.assign(new Error(''), { code: 'ECONNRESET' })
+  const r = errDetail(Object.assign(new Error('fetch failed'), { cause: c }))
+  nePusto(r, 'пустой message')
+  if (!/ECONNRESET/.test(r)) throw new Error(`код не попал в причину: «${r}»`)
+})
+
+t('AggregateError с пустым message → названа и она сама, и её внутренние', () => {
+  const vnutri = [Object.assign(new Error('connect ECONNREFUSED'), { code: 'ECONNREFUSED' })]
+  const c = new AggregateError(vnutri, '')
+  const r = errDetail(Object.assign(new Error('fetch failed'), { cause: c }))
+  nePusto(r, 'AggregateError')
+  if (!/AggregateError/.test(r)) throw new Error(`класс ошибки не назван: «${r}»`)
+  if (!/ECONNREFUSED/.test(r)) throw new Error(`внутренние причины потеряны: «${r}»`)
+})
+
+t('обычная причина с message → она и печатается', () => {
+  const c = Object.assign(new Error('getaddrinfo ENOTFOUND api.telegram.org'), { code: 'ENOTFOUND' })
+  const r = errDetail(Object.assign(new Error('fetch failed'), { cause: c }))
+  if (!/ENOTFOUND/.test(r)) throw new Error(`потеряно: «${r}»`)
+})
+
+// Отличать «объект есть, описания нет» от «cause отсутствует» — разные состояния.
+t('cause без единого поля → сказано, что описания нет, а не пустота', () => {
+  const r = errDetail(Object.assign(new Error('fetch failed'), { cause: Object.create(null) }))
+  nePusto(r, 'пустой объект')
+})
+
+console.log(`итог: ${ok} из ${ok + bed}`)
+process.exit(bed === 0 ? 0 : 1)