@ykurilov/agent-chat-core 0.80.1 → 0.81.0

package/CHANGELOG.md

@@ -1,5 +1,84 @@
 # @ykurilov/agent-chat-core
 
+## 0.81.0
+
+### Minor Changes
+
+- 58204d2: **Ломает приложения со своим хранилищем мессенджеров или сценариев.** Хранилища из пакетов (память, SQLite,
+  Postgres) уже готовы — если вы передаёте их, менять ничего не нужно.
+  
+  - `MessengerStore`: новый обязательный метод `saveLinkIfFree` и признак `guarded: true`. Привязка аккаунта по коду
+    идёт только через `saveLinkIfFree`: два кода разных людей на один аккаунт мессенджера больше не переписывают друг
+    друга.
+  - `FlowStore`: новый обязательный метод `swapDelivery` и признак `guarded: true`.
+  - Без `guarded: true` `createMessengers` и `createAgentFlow` сразу бросают `TypeError`. Признак — обещание, что
+    хранилище выполняет условия держателя и аренды (список — в шаге 3 ниже).
+  
+  Новое для своей доставки сценария (`attach({ deliver })`):
+  
+  - Исход `{ status: "await_repair" }` — канал сломан и ждёт починки (например, отозван токен бота). Ответ ждёт,
+    попытка не тратится, новых ходов и касаний нет. Починили — `flow.resumeDelivery({ ownerId, chatId, messageId })`:
+    тот же `messageId`, текст и части уходят ближайшим обходом таймеров. Ответ: `resumed`, `pending`, `busy`,
+    `not_found` или `dropped` (человек уже написал после ответа — ответ снят и помечен недоставленным). Ожидание
+    переживает перезапуск. В `reportDelivery` этот исход — `TypeError`.
+  - Пока идёт `deliver`, аренда чата продлевается: вторая копия сервера ту же попытку не начинает. Копия упала посреди
+    отправки — та же попытка повторяется через её паузу. Итог устаревшей копии не пишется.
+  - Перед самым `deliver`, уже под резервом, SDK ещё раз сверяет: разговор не закончен после хода (`flow.end`), ответ
+    есть в ветке, после починки человек не написал после него. Иначе `deliver` не зовётся, ответ помечен недоставленным.
+  - Повтор доставки прощания у законченного разговора теперь доходит — без нового хода и без открытия разговора.
+  - «Ровно один раз» снаружи SDK не обещает: после падения копии тот же ответ может уйти ещё раз с тем же `messageId`.
+    Двойную отправку отсекает приложение по `messageId`.
+  
+  Остальные исправления аудита работают без правок в приложении:
+  
+  - Ход группы мессенджера остаётся групповым и тогда, когда агент пишет первым, когда человек правит сообщение
+    группы, шлёт поправку к ходу или нажимает кнопку-подсказку: без памяти и личных блоков владельца, только
+    разрешённые группе инструменты, с пределом сообщений участника.
+  - Удаление чата ждёт хода на любой копии сервера: идёт ход — `ChatBusyError`, чат цел.
+  - Не удалось отметить вложения сообщения прикреплёнными — ошибка доходит до вызывающего, сообщение не записывается,
+    место в счёте сообщений отдаётся назад; повтор того же сообщения проходит.
+  - Сценарий, который не сходится с подтверждениями приложения (`confirmations`, `exitGuards`), проверяется до записи
+    версии и закрепления чата: чат идёт без сценария, перевод чатов на такую версию — отказ.
+  - Копия сервера, у которой забрали аренду таймера сценария или обновления мессенджера, больше не шлёт касание и не
+    переписывает срок и итог другой копии.
+  
+  ### Как обновить приложение
+  
+  Нужно только если приложение передаёт в `createMessengers` или `createAgentFlow` своё хранилище (не
+  `createMemoryMessengerStore`, `createSqliteMessengerStore`, `createPostgresMessengerStore`, `memoryFlowStore`,
+  `sqliteFlowStore`, `postgresFlowStore`).
+  
+  1. Найдите свою реализацию `MessengerStore` (поиск по `MessengerStore` или `saveLink(`).
+  2. Добавьте метод — одной атомарной операцией базы:
+  
+     ```ts
+     async saveLinkIfFree(link) {
+       // Аккаунт свободен или уже за link.userId — записать (имя и время обновить) и вернуть "saved".
+       // Аккаунт за другим пользователем — ничего не менять и вернуть "taken".
+     }
+     ```
+  
+     В SQL это `insert … on conflict (messenger, bot_id, external_user_id) do update set … where user_id = $userId
+     returning …`: строка вернулась — `"saved"`, нет — `"taken"`. Отдельные «найти, потом записать» не годятся: между
+     ними успевает второй код.
+  3. Добавьте признак `guarded: true` в объект хранилища. Он обещает:
+     - `finishUpdate(key, status, now, holder)`, `renewUpdates` и `releaseUpdate` пишут, только пока обновление за этим
+       держателем; `acquireLease` не отдаёт чужую живую аренду;
+     - `saveLinkIfFree` — одна атомарная операция.
+  4. Своё `FlowStore` (если есть): метод `swapDelivery(chatId, { expected, next, due, claim, freeAt, endedAt })` одной
+     транзакцией. Заменить запись доставки, только если сейчас там `expected` (совпали `messageId`, `attempt` и `token`;
+     `null` — записи нет). С `claim` — только если аренда чата свободна, уже у `claim.owner` или у `claim.from`, и взять
+     её до `now + leaseMs`. С `freeAt` — только если живой аренды нет. С `endedAt` — только если конец разговора всё
+     ещё этот (`null` — не закончен). `due` записать вместе с записью. Вернуть `true`,
+     если заменил этот вызов. Плюс `guarded: true`, который обещает:
+     - `setDue(…, { leaseOwner })` пишет, только пока аренда у этой копии; `{ unlessDelivery: true }` — только если
+       доставки не ждёт;
+     - `leaseDue` не берёт живую аренду, а законченные разговоры берёт только со сроком `FLOW_DELIVERY_EXIT`;
+     - `endChat` при ждущей доставке не снимает срок и аренду.
+     Образец — `postgresFlowStore` и `sqliteFlowStore` в `@ykurilov/agent-chat-flows`.
+  5. Проверка: `pnpm typecheck` проходит. Без признака или без `saveLinkIfFree` запуск сразу бросает `TypeError` с
+     подсказкой, без `swapDelivery` — `attach({ deliver })`.
+
 ## 0.80.1
 
 ### Patch Changes

package/dist/{chunk-PORXDIWX.js → chunk-YGHGURNF.js}

@@ -1700,9 +1700,13 @@ function channelOfMessage(message) {
 }
 var isNoticeMessage = (message) => message.parts.some((part) => part.type === AGENT_NOTICE_PART_TYPE);
 var isHumanMessage = (message) => message.role === "user" && !isNoticeMessage(message) && !isStateMessage(message);
+var hasChannelPart = (message) => message.parts.some((part) => part.type === CHANNEL_PART_TYPE);
 function turnChannelOf(branch) {
-  const human = branch.findLast(isHumanMessage);
-  return human ? channelOfMessage(human) : { channel: "web" };
+  const source = branch.findLast((message) => isHumanMessage(message) || message.role === "user" && isNoticeMessage(message) && hasChannelPart(message));
+  return source ? channelOfMessage(source) : { channel: "web" };
+}
+function inheritedGroupPart(scope) {
+  return scope.group ? channelPart(scope.channel, scope.group) : void 0;
 }
 function turnOriginOf(branch) {
   const parent = branch.at(-2);
@@ -8639,7 +8643,8 @@ function shortOutput(value) {
 var MAX_SERVER_TURNS_IN_ROW = 5;
 function noticeMessageParts(input) {
   const data = { notices: input.notices, deliveryKey: input.deliveryKey, wakeAgent: input.wakeAgent };
-  return [{ type: AGENT_NOTICE_PART_TYPE, data }];
+  const channel = channelPart(input.channel ?? "web", input.group);
+  return [{ type: AGENT_NOTICE_PART_TYPE, data }, ...channel ? [channel] : []];
 }
 function noticeDataOf(message) {
   if (message.role !== "user") return void 0;
@@ -13188,9 +13193,9 @@ function createChatService(input) {
       throw error;
     }
   }
-  async function confirmFiles(messageId, fileParts) {
+  async function confirmFiles(fileParts) {
     if (fileParts.length === 0) return;
-    await options2.attachments.confirmUserParts(fileParts).catch((error) => logError(`\u041D\u0435 \u0443\u0434\u0430\u043B\u043E\u0441\u044C \u043E\u0442\u043C\u0435\u0442\u0438\u0442\u044C \u0432\u043B\u043E\u0436\u0435\u043D\u0438\u044F \u0441\u043E\u043E\u0431\u0449\u0435\u043D\u0438\u044F ${logValue(messageId)}: ${String(error)}`));
+    await options2.attachments.confirmUserParts(fileParts);
   }
   async function ownChat(userId, chatId) {
     const chat = await storage.getChat(chatId);
@@ -13907,7 +13912,7 @@ function createChatService(input) {
         // Окно тишины пачки считается от этого времени: по часам службы.
         ...burst && { createdAt: writtenAt.toISOString() }
       });
-      await confirmFiles(question3.id, fileParts);
+      await confirmFiles(fileParts);
       await storage.insertMessage(question3);
       return question3;
     });
@@ -14143,14 +14148,20 @@ function createChatService(input) {
       return await writingCountedMessage(userId, chatId, async () => {
         const queued = await steering.accept(chatId, async (parentId) => {
           const fileParts = await acceptFiles(userId, chatId, files);
+          const groupPart = inheritedGroupPart(turnChannelOf(await currentBranch(chatId)));
           const correction = newStoredMessage({
             id: message.id,
             chatId,
             parentId,
             role: "user",
-            parts: [...await filePartsWithNumbers(chatId, fileParts), ...text.trim().length > 0 ? [{ type: "text", text }] : [], steerPart()]
+            parts: [
+              ...await filePartsWithNumbers(chatId, fileParts),
+              ...text.trim().length > 0 ? [{ type: "text", text }] : [],
+              steerPart(),
+              ...groupPart ? [groupPart] : []
+            ]
           });
-          await confirmFiles(correction.id, fileParts);
+          await confirmFiles(fileParts);
           await storage.insertMessage(correction);
           return correction;
         }, page);
@@ -14438,7 +14449,7 @@ function createChatService(input) {
         await compaction.cancel(chatId);
         await options2.attachments?.removeChatFiles?.(chatId);
         await storage.deleteChat(chatId);
-      }).finally(() => deleting.delete(chatId));
+      }, { lease: "required" }).finally(() => deleting.delete(chatId));
       await options2.background?.chatDeleted(userId, chatId).catch((error) => logError(`\u0424\u043E\u043D: \u043D\u0435 \u0443\u0434\u0430\u043B\u043E\u0441\u044C \u0443\u0431\u0440\u0430\u0442\u044C \u0437\u0430\u0434\u0430\u0447\u0438 \u0443\u0434\u0430\u043B\u0451\u043D\u043D\u043E\u0433\u043E \u0447\u0430\u0442\u0430 ${chatId}: ${String(error)}`));
       await options2.limits?.chatDeleted?.({ userId, chatId }).catch((error) => logError(`\u041B\u0438\u043C\u0438\u0442\u044B: \u043D\u0435 \u0443\u0434\u0430\u043B\u043E\u0441\u044C \u0441\u0442\u0435\u0440\u0435\u0442\u044C \u043E\u043A\u043D\u0430 \u0443\u0434\u0430\u043B\u0451\u043D\u043D\u043E\u0433\u043E \u0447\u0430\u0442\u0430 ${chatId}: ${String(error)}`));
     },
@@ -14620,11 +14631,13 @@ function createChatService(input) {
                 )
               ] : [],
               // Правленое сообщение пачки остаётся в своей пачке: отметка с прежним временем записи.
-              ...edited2.parts.filter((part) => part.type === BURST_PART_TYPE)
+              ...edited2.parts.filter((part) => part.type === BURST_PART_TYPE),
+              // Правленое сообщение группы остаётся в границах группы: отметка — из записанного сообщения, не из запроса.
+              ...[inheritedGroupPart(channelOfMessage(edited2))].filter((part) => part !== void 0)
             ],
             createdAt: nextSiblingTime(messages, selections, edited2.parentId)
           });
-          await confirmFiles(question3.id, fileParts);
+          await confirmFiles(fileParts);
           await storage.insertMessage(question3);
           return { branch: branch2, index: index2, edited: edited2, question: question3 };
         });
@@ -15228,6 +15241,7 @@ export {
   THOUGHT_TITLE_METADATA_KEY,
   thoughtTitlesOfParts,
   CHANNEL_PART_TYPE,
+  isHumanMessage,
   turnOriginOf,
   GREETING_LEAD,
   STEER_PART_TYPE,
@@ -15352,4 +15366,4 @@ export {
   InvalidWidgetAnswerError,
   createChatService
 };
-//# sourceMappingURL=chunk-PORXDIWX.js.map
+//# sourceMappingURL=chunk-YGHGURNF.js.map