dsh-telegram-multiagent 1.2.1 → 1.2.3

package/README.md

@@ -50,7 +50,7 @@ see "Where to put the row" below:
 | `a2aSession` | no | Session id used for that channel. Default `a2a`. |
 | `transcribeCommand` | no | External command for voice messages: `<cmd> <audio-file> auto` → transcript on stdout. Omit and voice is politely refused. |
 
-## Four things that cost us a day
+## Five things that cost us a day
 
 Each of these fails **while looking like success**. They are commented inline in the source; this is
 the short version.
@@ -74,6 +74,17 @@ session id makes the persistence layer abort *every* turn with an id-collision e
 "accepted the message and went quiet" — the turn honestly starts and dies in milliseconds. It works
 until the first restart, which is what makes it nasty. Use `resume()` when the session exists.
 
+**5. `ctx.get()` returns nothing *quietly* while a service is still starting.** cordis hands back
+`undefined` without throwing until the provider's fiber is active, so "this service is not in the
+build" and "this service is thirty milliseconds away" arrive as the same value. Code written as
+`const x = ctx.get('x'); if (x) {...}` then takes the "feature absent" branch and says nothing — the
+agent is assembled, answers, and never mentions what it lost. This bit us three times in three days:
+the agent factory (3), session persistence (4, the whole resume block was skipped), and the agent
+preset — where a neighbouring agent's toolset dropped from 33 tools to 3 with no error and no log
+line. Wait for the service with a stated deadline, and when the deadline passes, say so *and name
+the consequence*. One helper for all of them: a fix applied to the instance instead of the class
+guarantees a relapse, and the relapse looks like a new illness.
+
 ## Where to put the row
 
 In the `web` profile the common-plane tools are **disabled on purpose**; the toolset comes from the

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-telegram-multiagent",
-  "version": "1.2.1",
+  "version": "1.2.3",
   "description": "Telegram channel for DeepSeek Harness — one shared module serving several agents, with unforgeable sender marks, merged owner+coordinator memory, and a delivery mode you switch in a file outside the code.",
   "type": "module",
   "main": "src/index.js",

package/src/index.js

@@ -123,7 +123,7 @@ export function apply(ctx, config = {}) {
   // улетит в чат владельца. Ключ — сессия, значение — 'a2a' или 'tg'.
   const lastOrigin = new Map();
 
-  // 🔴 СЛИЯНИЕ КАНАЛОВ В ОДНУ ПАМЯТЬ (20.08.2026, решение Александра).
+  // 🔴 СЛИЯНИЕ КАНАЛОВ В ОДНУ ПАМЯТЬ (20.08.2026, решение владельца).
   // mergeChatIntoA2A: <идентификатор чата> — сообщения этого чата попадают НЕ в
   // свою сессию `telegram-<чат>`, а в служебную. Тогда владелец и координатор
   // говорят с ОДНИМ агентом и одной памятью разговора, а различает он их по
@@ -134,14 +134,14 @@ export function apply(ctx, config = {}) {
   // Куда отвечать в мессенджер, если сессия общая: sessionId → числовой чат.
   const tgChatFor = new Map();
 
-  // 🔴 КОМУ АДРЕСОВАН ОТВЕТ (21.08.2026, задача Александра).
+  // 🔴 КОМУ АДРЕСОВАН ОТВЕТ (21.08.2026, задача владельца).
   // Пометка на ВХОДЕ решает «кто спросил», но не решает «кому отвечено»: когда в
   // одной памяти сидят двое, ответ без адреса читается обоими как свой. И хуже:
   // маршрут «отвечаем последнему спросившему» ломается, если второй вопрос
   // пришёл, пока первый ещё считается, — ответ уедет не тому.
   // Поэтому происхождение привязано к ХОДУ, а не к последнему сообщению: на
   // входе кладём в очередь, на turn/start снимаем, ответ метим тем, что сняли.
-  // ── НАСТРОЙКА ДОСТАВКИ, ОТДЕЛЬНЫМ ФАЙЛОМ (21.08.2026, задача Александра)
+  // ── НАСТРОЙКА ДОСТАВКИ, ОТДЕЛЬНЫМ ФАЙЛОМ (21.08.2026, задача владельца)
   //
   // Файл живёт ВНЕ модуля и переживает его обновление: код можно переписать,
   // перевыпустить, поставить заново — настройка останется. Читается на лету,
@@ -264,6 +264,46 @@ export function apply(ctx, config = {}) {
     throw new Error(`фабрика агентов не появилась за ${tries} с`);
   }
 
+  /**
+   * Дождаться сервиса платформы, не заперев канал навсегда.
+   *
+   * 🔴 ctx.get() МОЛЧИТ, КОГДА СЕРВИСА ЕЩЁ НЕТ, и это опаснее исключения.
+   * cordis возвращает undefined БЕЗ ошибки, пока волокно поставщика не активно
+   * (ReflectService._getImpl: `if (strict && impl.fiber.state !== 2) return;`).
+   * Значит «сервиса нет в сборке» и «сервис ещё поднимается» приходят одним и
+   * тем же значением, и код, написанный как `const x = ctx.get('x'); if (x)`,
+   * тихо уходит по ветке «этой возможности просто нет». Снаружи это выглядит
+   * штатной работой: агент собирается, отвечает и молчит о том, чего лишился.
+   *
+   * Поймано трижды за три дня, каждый раз дороже предыдущего:
+   *   19.08.2026 — фабрика агентов (см. withFactoryRetry выше);
+   *   22.08.2026 — sessionPersistence: блок продолжения истории пропускался
+   *                целиком, агент создавался поверх существующего журнала, и
+   *                КАЖДЫЙ ход умирал с «id collision» за считаные миллисекунды;
+   *   22.08.2026 — agentPresets: набор инструментов у модели упал с 33 до 3,
+   *                без ошибки и без строки в журнале.
+   * Общий вывод, ради которого функция ОДНА, а не три: лечение, применённое к
+   * экземпляру, а не ко всем местам класса, гарантирует рецидив.
+   *
+   * ГРАНИЦА ОЖИДАНИЯ — назвать обязательно, иначе защита сама станет отказом:
+   * ждём не дольше `tries` секунд и только при заведении агента. Не дождались —
+   * говорим ГРОМКО, называем последствие и идём дальше. Канал не запирается там,
+   * где сервиса в сборке нет вовсе (это законная сборка, а не поломка).
+   */
+  async function waitService(name, tag, consequence, tries = 30) {
+    for (let i = 0; i < tries; i++) {
+      const found = ctx.get(name);
+      if (found) {
+        if (i > 0) log(`${tag} сервис ${name} появился через ${i} с ожидания`);
+        return found;
+      }
+      if (i === 0) log(`${tag} сервис ${name} ещё не активен — жду платформу`);
+      await new Promise((r) => setTimeout(r, 1000));
+    }
+    log(`${tag} 🔴 сервис ${name} не появился за ${tries} с — ${consequence}`);
+    return undefined;
+  }
+
   async function agentFor(chatId) {
     const key = String(chatId);
     const found = chats.get(key);
@@ -275,9 +315,11 @@ export function apply(ctx, config = {}) {
     // в запрос к модели не попадает поле tools вообще, и агент физически не
     // может вызвать ни один инструмент, хотя они смонтированы (проверено
     // 2026-08-18 по событию request/header в журнале сессии).
-    const presets = ctx.get('agentPresets');
+    const presets = await waitService('agentPresets', `[${key}]`,
+      'агент будет собран БЕЗ ИНСТРУМЕНТОВ');
     const presetId = presets ? (await presets.resolve(config.preset)).id : undefined;
-    const defaultModel = ctx.get('agentDefaultModel');
+    const defaultModel = await waitService('agentDefaultModel', `[${key}]`,
+      'модель берётся из настроек плагина, а не из выбора платформы');
     const selection = defaultModel?.currentSelection?.() ?? {
       provider: config.provider,
       model: config.model,
@@ -299,16 +341,36 @@ export function apply(ctx, config = {}) {
     // события turn/end (раньше плагин печатал только тип события).
     // Правильный путь — ctx.agents.resume(): «загрузить сохранённую сессию и
     // продолжить агента на ней» (dsh-agent/lib/index.js, resume()).
-    const persistence = ctx.get('sessionPersistence');
+    const persistence = await waitService('sessionPersistence', `[${key}]`,
+      'сохранённую сессию продолжить нечем, история будет потеряна');
     let handle;
     if (persistence) {
-      let onDisk = false;
+      let known;
       try {
-        onDisk = (await persistence.list()).some((h) => h.id === sessionId);
+        known = (await persistence.list()).map((h) => h.id);
       } catch (e) {
-        log(`не смогла перечислить сохранённые сессии: ${e?.message ?? e}`);
+        log(`🔴 [${key}] не смог перечислить сохранённые сессии: ${e?.message ?? e}`);
+      }
+      // 🔴 УХОДИТЬ В create() МОЛЧА НЕЛЬЗЯ — ЭТО ВТОРОЙ ТИХИЙ ПУТЬ В ЭТОЙ ЖЕ
+      // ФУНКЦИИ. Снаружи он неотличим от честного первого запуска, а на деле
+      // означает create() поверх существующего журнала и «id collision» на
+      // каждом ходе. Поэтому список сообщаем ЧИСЛОМ и именами.
+      if (known !== undefined && !known.includes(sessionId)) {
+        // Пустой список — честный первый запуск, тревожить нечем. Тревога там,
+        // где сохранённые сессии ЕСТЬ, а нашей среди них нет.
+        if (known.length === 0) log(`[${key}] сохранённых сессий на диске нет — ${sessionId} заводится впервые`);
+        else {
+          // ⚠️, а не 🔴, СОЗНАТЕЛЬНО: по одному списку два исхода неразличимы —
+          // либо это первый разговор в новом чате (законно и обычно), либо
+          // журнал сессии потерян (беда). Отличить их отсюда нечем, поэтому
+          // строка называет ФАКТЫ и оба прочтения, а не ставит диагноз.
+          // Кричать 🔴 на законном событии — это защита, которая приучает
+          // не читать журнал.
+          const spisok = known.slice(0, 12).join(', ') + (known.length > 12 ? `, …и ещё ${known.length - 12}` : '');
+          log(`⚠️ [${key}] на диске ${known.length} сохранённых сессий, ${sessionId} среди них НЕТ: либо это первый разговор в чате, либо история потеряна. Продолжать нечего, создаю заново. Найдено: ${spisok}`);
+        }
       }
-      if (onDisk) {
+      if (known !== undefined && known.includes(sessionId)) {
         try {
           handle = await ctx.agents.resume({
             resumeSessionId: sessionId,
@@ -602,14 +664,43 @@ export function apply(ctx, config = {}) {
   const A2A_OUT = A2A_DIR ? path.join(A2A_DIR, 'out') : null;
   const A2A_CHAT = config.a2aSession || 'a2a';  // отдельная сессия, не смешивается с Telegram
 
+  // Берём простые текстовые форматы. Всё остальное НЕ берём — но и не
+  // проглатываем: имя отвергнутого файла называем в журнале. Поймано
+  // 21.08.2026: письмо с расширением .md пролежало во входящих 26 минут,
+  // и снаружи это выглядело как «агент не отвечает».
+  const A2A_EXT = ['.txt', '.md'];
+  // ГРАНИЦА СТОРОЖА: о каждом имени сообщаем ОДИН раз — опрос идёт
+  // каждые ~25 секунд, и повтор превратил бы защиту в шум. Файл убрали —
+  // имя забываем, положат снова — сообщим снова.
+  const a2aReported = new Set();
+
   async function pollA2A() {
     if (!A2A_DIR) return;
-    let files = [];
+    let all = [];
     try {
       fs.mkdirSync(A2A_IN, { recursive: true });
       fs.mkdirSync(A2A_OUT, { recursive: true });
-      files = fs.readdirSync(A2A_IN).filter((f) => f.endsWith('.txt')).sort();
-    } catch { return; }
+      all = fs.readdirSync(A2A_IN).sort();
+    } catch (e) {
+      // Раньше здесь стояло `catch { return; }`: каталог мог быть недоступен,
+      // а канал молча не работал вовсе. Одна строка на каждую причину.
+      const key = `DIR:${e?.code ?? 'ERR'}`;
+      if (!a2aReported.has(key)) {
+        a2aReported.add(key);
+        log(`[a2a] \u{1F534} каталог входящих недоступен (${A2A_IN}): ${e?.message ?? e}`);
+      }
+      return;
+    }
+    const files = all.filter((f) => A2A_EXT.some((x) => f.endsWith(x)));
+    for (const f of all) {
+      if (files.includes(f) || a2aReported.has(f)) continue;
+      a2aReported.add(f);
+      log(`[a2a] \u{1F534} НЕ ВЗЯТ файл ${f}: беру только ${A2A_EXT.join(', ')}. `
+        + `Он так и будет лежать во входящих — переименуй отправителю`);
+    }
+    for (const name of [...a2aReported]) {
+      if (!name.startsWith('DIR:') && !all.includes(name)) a2aReported.delete(name);
+    }
     for (const f of files) {
       const full = path.join(A2A_IN, f);
       let text = '';