dsh-telegram-multiagent 1.5.1 → 1.5.3

package/CHANGELOG.md

@@ -1,3 +1,41 @@
+## 1.5.3
+
+🔴 **Починка стенда, который я сломала правкой 1.5.2 — и выпустила, не прочитав его вывод.**
+`stend-svezhesti-opory` вырезает блок свежести из предмета и собирает через `new Function`.
+В 1.5.2 блок стал звать `prichina(e)`, объявленную рядом, но ВНЕ блока — вырезанное перестало
+быть самодостаточным, стенд дал «prichina is not defined» и покраснел на исправном коде.
+
+Стенд теперь вырезает блок ВМЕСТЕ С ЗАВИСИМОСТЬЮ; отсутствие `prichina` в предмете даёт
+СЛЕПОТУ с названной причиной, а не расхождение.
+
+⚠️ Мой промах в порядке, а не в коде: я прогнала стенды перед выпуском, увидела вывод и не
+прочла в нём «расхождений 1». Обёртка публикации стенды не проверяет — это её граница, и
+1.5.2 ушёл в реестр с красным стендом. Получатель прогнал бы `npm test` и увидел красное на
+работающем пакете.
+
+## 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` печатал пустоту после слова «причина».** Собирал причину через `??`, а нулевое

package/package.json

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

package/src/index.js

@@ -1372,6 +1372,30 @@ export function apply(ctx, config = {}) {
   // двоеточия. Хуже отсутствия поля: «причина:» с пустотой читается как «причину
   // узнали, она пустая», а её просто не достали.
   // Воспроизведено: 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 'не указана';
@@ -1495,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'
@@ -1536,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 };
@@ -1597,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

@@ -69,5 +69,47 @@ t('cause без единого поля → сказано, что описан
   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)

package/test/stend-svezhesti-opory.mjs

@@ -36,9 +36,21 @@ if (!m) {
   process.exit(2)
 }
 // Оборачиваем вырезанное в функцию от sess: внутри блок обращается только к `sess`.
+// 🔴 ВЫРЕЗАЕМ ВМЕСТЕ С ЗАВИСИМОСТЬЮ. Блок свежести зовёт `prichina(e)` — функцию,
+// объявленную рядом, но ВНЕ блока. Вырезав один блок, стенд получал «prichina is not
+// defined» и краснел на исправном предмете.
+// Это не оговорка к приёму «вырезать из предмета», а его цена: вырезанное должно быть
+// самодостаточным, и когда предмет теряет самодостаточность, стенд обязан сказать об
+// этом — он и сказал. Правка предмета сломала способ его проверки, и это верное красное.
+const mPr = text.match(/function prichina\(e\) \{[\s\S]*?\n  \}/)
+if (!mPr) {
+  console.log('СЛЕПОТА: блок свежести зовёт prichina(), а её объявления в предмете нет —')
+  console.log('  вырезанное не соберётся. Проверить руками, не переименована ли она.')
+  process.exit(2)
+}
 let svezhestOt
 try {
-  svezhestOt = new Function('sess', m[0].replace('const svezhest = ', 'return ') .replace(/;$/, '') + ';')
+  svezhestOt = new Function('sess', mPr[0] + '\n' + m[0].replace('const svezhest = ', 'return ').replace(/;$/, '') + ';')
 } catch (e) {
   console.log(`СЛЕПОТА: вырезанный блок не собирается: ${e?.message ?? e}`); process.exit(2)
 }