dsh-telegram-multiagent 1.5.0 → 1.5.2

package/CHANGELOG.md

@@ -1,3 +1,48 @@
+## 1.5.2
+
+**Разбор брошенного — по ДВУМ ОСЯМ.** Три места (`Измеритель отказал`, `обход событий отказал`,
+`llm.resolveModelInfo отказал`) собирали причину как `String(e?.message ?? e)`.
+
+    ось 1 — пустое исключение   -> строка НЕ пуста      было 6 из 10
+    ось 2 — брошенный ПРИМИТИВ  -> текст СОХРАНЁН       было 4 из 5
+    стало: ось 1 — 10 из 10, ось 2 — 4 из 5 (не хуже)
+
+🔴 Первая предложенная редакция закрывала ось 1 и ЛОМАЛА ось 2: `throw 'сессия не открыта'`
+давала «string: (без пояснения)». Пустоту закрыли, потерю открыли рядом. Нашлось это не чтением
+кода, а второй осью — пробы одной оси слепы к дефекту другой, сколько бы их ни было.
+
+`code` добавлен сверх присланного замысла: у нас ДОКАЗАНО замером, что при пустом `message`
+причина живёт именно в нём (`ECONNRESET`, `ENOTFOUND` у fetch). Без него три случая из пяти
+дают «(без пояснения)» там, где код известен.
+
+⚠️ Остаток названный: `throw null` и `{}` текста не несут вовсе — там «(без пояснения)» честно,
+а не потеря.
+
+Две порчи, каждая ломает СВОЮ ось: первая редакция роняет ось 2, прежнее выражение — ось 1.
+Стенд 8 из 8.
+
+## 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.2",
   "description": "Канал Telegram для DeepSeek Harness: один модуль на машину обслуживает несколько агентов, у каждого свой бот, свой файл токена и свой список разрешённых. Метка отправителя неподделываема, память владельца и координатора слита в одну, режим доставки переключается файлом вне кода, цели управляются из канала.",
   "type": "module",
   "main": "src/index.js",

package/src/index.js

@@ -1364,10 +1364,57 @@ 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) -> ''.
+  // 🔴 РАЗБОР БРОШЕННОГО. Замысел соавтора от 04.09 плюс `code` — и то, и другое
+  // по замеру, а не по вкусу.
+  //
+  // ДВЕ ОСИ ПРИЁМКИ, и вторая нашла дефект, к которому первая слепа по построению:
+  //   ось 1 — пустое исключение   -> строка НЕ пуста
+  //   ось 2 — брошенный ПРИМИТИВ  -> текст СОХРАНЁН
+  // Прежнее `String(e?.message ?? e)` держало ось 2 (4 из 5) и проваливало ось 1
+  // (6 из 10): у Error с пустым message и у AggregateError печаталась пустота.
+  // Первая присланная редакция закрывала ось 1 и ЛОМАЛА ось 2: `throw 'сессия не
+  // открыта'` давала «string: (без пояснения)» — пустоту закрыли, потерю открыли рядом.
+  //
+  // Здесь: ось 1 — 10 из 10, ось 2 — 4 из 5 (не хуже прежнего).
+  // `code` добавлен сверх замысла соавтора, потому что у нас ДОКАЗАНО замером: при
+  // пустом message причина живёт именно в нём (ECONNRESET, ENOTFOUND у fetch). Без
+  // него три случая из пяти дают «(без пояснения)» там, где код известен.
+  // Остаток названный, а не замазанный: `throw null` и `{}` текста не несут вовсе —
+  // терять там нечего, и «(без пояснения)» тут ЧЕСТНО, а не потеря.
+  function prichina(e) {
+    const tekst = String(e?.message ?? (typeof e === 'object' ? '' : (e ?? ''))).trim();
+    const kod = e?.code ? String(e.code) : '';
+    const hvost = [kod, tekst].filter(Boolean).join(' ') || '(без пояснения)';
+    return ((e?.name ?? typeof e) + ': ' + hvost).slice(0, 90);
+  }
+
   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 +1444,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));
       }
     }
@@ -1469,7 +1519,7 @@ export function apply(ctx, config = {}) {
               if (!sess) return { kind: 'error', text: 'Сессия недоступна.' };
               let m;
               try { m = schetchik.measure(sess); }
-              catch (e) { return { kind: 'error', text: `Измеритель отказал: ${String(e?.message ?? e).slice(0, 160)}` }; }
+              catch (e) { return { kind: 'error', text: `Измеритель отказал: ${prichina(e)}` }; }
               // 🔴 СВЕЖЕСТЬ ОПОРЫ. baseline.kind различает ПРОИСХОЖДЕНИЕ опоры и НЕ различает
               // её СВЕЖЕСТЬ: при surface-replace (compaction/summary|prune) поверхность
               // пересчитывается, а якорь остаётся прежним и тащит свой kind='usage'
@@ -1510,7 +1560,7 @@ export function apply(ctx, config = {}) {
                     return { kind: 'ne-znayu', pochemu: 'у сессии нет ни events, ни eventAt+seq' };
                   }
                 } catch (e) {
-                  return { kind: 'ne-znayu', pochemu: `обход событий отказал: ${String(e?.message ?? e).slice(0, 90)}` };
+                  return { kind: 'ne-znayu', pochemu: `обход событий отказал: ${prichina(e)}` };
                 }
                 if (poslednijKompakt < 0) return { kind: 'svezhaya', kompaktov: 0 };
                 if (poslednijYakor > poslednijKompakt) return { kind: 'svezhaya', kompaktov: 1 };
@@ -1571,7 +1621,7 @@ export function apply(ctx, config = {}) {
                       okno = undefined;
                     }
                   } catch (e) {
-                    pochemuNet = `llm.resolveModelInfo отказал: ${String(e?.message ?? e).slice(0, 120)}`;
+                    pochemuNet = `llm.resolveModelInfo отказал: ${prichina(e)}`;
                   }
                 }
               }

package/test/stend-prichiny-otkaza.mjs

@@ -0,0 +1,115 @@
+// Стенд разбора причины отказа опроса.
+//
+// Коды: 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, 'пустой объект')
+})
+
+// ── ДВЕ ОСИ ПРИЁМКИ ДЛЯ prichina(). ────────────────────────────────────────
+// 🔴 ОСИ, А НЕ «ЕЩЁ ПРОБЫ». Ось — это другой ВОПРОС к тому же предмету, и пробы одной
+// оси слепы к дефекту другой, сколько бы их ни было.
+//   ось 1 — пустое исключение   -> строка НЕ пуста
+//   ось 2 — брошенный ПРИМИТИВ  -> текст СОХРАНЁН
+// Первая присланная редакция закрывала ось 1 и ломала ось 2 (`throw 'текст'` давала
+// «string: (без пояснения)»): пустоту закрыли, потерю открыли рядом. Нашлось это не
+// чтением кода, а второй осью.
+const mP = text.match(/function prichina\(e\) \{[\s\S]*?\n  \}/)
+if (!mP) { console.log('СЛЕПОТА: функция prichina в предмете не найдена'); process.exit(2) }
+let prichina
+try { prichina = new Function(mP[0] + '; return prichina;')() }
+catch (e) { console.log(`СЛЕПОТА: prichina не собирается: ${e?.message ?? e}`); process.exit(2) }
+
+t('ОСЬ 1: пустые исключения дают НЕпустую строку', () => {
+  const bed = []
+  for (const [imya, v] of [['Error с пустым message', new Error('')],
+                           ['AggregateError пустой', new AggregateError([], '')],
+                           ['throw null', null], ['throw undefined', undefined],
+                           ['объект пустой', {}]]) {
+    const r = prichina(v)
+    if (typeof r !== 'string' || r.trim() === '' || /^(undefined|null):?\s*$/.test(r.trim()))
+      bed.push(`${imya}: «${r}»`)
+  }
+  if (bed.length) throw new Error(bed.join(' | '))
+})
+
+t('ОСЬ 2: брошенный примитив — текст СОХРАНЁН, не заменён пояснением', () => {
+  const bed = []
+  for (const [v, chto] of [['сессия не открыта', 'сессия не открыта'], [42, '42'],
+                           [{ message: 'своё' }, 'своё'], [new Error('сеть'), 'сеть']]) {
+    const r = prichina(v)
+    if (!r.includes(chto)) bed.push(`потерян текст «${chto}»: «${r}»`)
+  }
+  if (bed.length) throw new Error(bed.join(' | '))
+})
+
+t('код ошибки попадает в причину, когда message пуст', () => {
+  const r = prichina(Object.assign(new Error(''), { code: 'ECONNRESET' }))
+  if (!/ECONNRESET/.test(r)) throw new Error(`код потерян: «${r}»`)
+})
+
+console.log(`итог: ${ok} из ${ok + bed}`)
+process.exit(bed === 0 ? 0 : 1)