@ykurilov/agent-chat-attachments 0.80.1 → 0.81.0

package/CHANGELOG.md

@@ -1,5 +1,22 @@
 # @ykurilov/agent-chat-attachments
 
+## 0.81.0
+
+### Minor Changes
+
+- 038b6e2: Исправления аудита клиентской зоны. `useAgentChat` без `key` при смене чата больше не переносит в новый чат дела прежнего: «Стоп», ответ на подтверждение или виджет, передачу поправок и сорвавшуюся голосовую замену; новый чат не остаётся «занятым», повторное открытие того же чата начинается с чистого состояния. Песочница не зовёт хуки приложения, которые пишут данные (`onStateChanged`, `onTurnFinished`, `onModelChosen`), ни в песочном чате, ни в копии чата человека; `newChatGreeting` и `modelMenu` решаются для настоящего админа. Вложения: удаление человеком и чистка брошенных больше не убирают файл, который отправляемое сообщение уже отметило; `confirmUserParts` отказывает `AttachmentRejectedError` (`attachment_not_found` — файла нет или отметка не записана, `attachment_unavailable` — сбой хранилища), а не отмечает молча.
+  
+  ### Как обновить приложение
+  
+  Обновите пакеты SDK одной версией. Со встроенными хранилищами вложений (`createDiskAttachmentStorage`, `createMemoryAttachmentStorage`) ничего менять не нужно. Своё хранилище (`AttachmentStorage`, например S3) должно получить метод `removeUnattached(id): Promise<boolean>` — удалить файл, только если он не прикреплён, атомарно с `markAttached` того же файла; `removeUnattachedOlderThan` должен следовать тому же правилу. Без метода `createAttachments` бросает ошибку при запуске. Проверьте хранилище обновлённым `describeAttachmentStorageContract`. Если песочница полагалась на `onStateChanged`, `onTurnFinished` или `onModelChosen` в песочных чатах — в песочнице они больше не зовутся. [Подробности](https://github.com/ykurilov/agent-chat-sdk/blob/main/docs/audit-client.md).
+
+### Patch Changes
+
+- Updated dependencies [038b6e2]
+- Updated dependencies [58204d2]
+  - @ykurilov/agent-chat-react@0.81.0
+  - @ykurilov/agent-chat-core@0.81.0
+
 ## 0.80.1
 
 ### Patch Changes

package/README.md

@@ -88,6 +88,9 @@ import { attachmentsClient } from "@ykurilov/agent-chat-attachments/react";
   Сбор текста останавливается, как только набран потолок символов. Текстовые логи с цветовыми кодами терминала принимаются как текст.
 - **Хранилище файлов** заменяемое (`AttachmentStorage`): из коробки `createDiskAttachmentStorage` и `createMemoryAttachmentStorage`.
   Своё (например, S3) проверяется общим набором `describeAttachmentStorageContract` из `@ykurilov/agent-chat-attachments/storage-contract`.
+  Неотправленный файл хранилище убирает только условно: `removeUnattached(id)` и `removeUnattachedOlderThan(before)` не трогают файл,
+  отметка которого (`markAttached`) уже записана, — проверка и удаление атомарны относительно отметки. Без `removeUnattached`
+  `createAttachments` не создаётся. Хранилище на диске даёт эту атомарность в пределах одного процесса сервера.
   Схема базы чата не меняется, миграции не нужны. Метод `removeAllOfChat(chatId)` удаляет все файлы чата: его вызывает удаление чата из списка чатов.
 - **Безопасность**: файл привязан к чату и владельцу, чужой и несуществующий файл отвечают одинаково (404). Адрес в хранилище — серверный UUID,
   имя файла в путь не попадает. Файлы отдаются с `nosniff` и `Content-Security-Policy: sandbox`. Ключи провайдеров в браузер не уходят.

package/dist/attachment-storage-contract.js

@@ -15399,6 +15399,67 @@ function describeAttachmentStorageContract(createStorage) {
       await storage.remove(saved.id);
       await storage.remove(crypto.randomUUID());
     });
+    it("removeUnattached \u0443\u0431\u0438\u0440\u0430\u0435\u0442 \u0442\u043E\u043B\u044C\u043A\u043E \u043D\u0435\u043F\u0440\u0438\u043A\u0440\u0435\u043F\u043B\u0451\u043D\u043D\u044B\u0439 \u0444\u0430\u0439\u043B \u0438 \u0433\u043E\u0432\u043E\u0440\u0438\u0442, \u0443\u0431\u0440\u0430\u043B \u043B\u0438; \u043D\u0435\u0438\u0437\u0432\u0435\u0441\u0442\u043D\u044B\u0439 \u0438 \u043D\u0435-UUID \u2014 false", async () => {
+      const storage = await createStorage();
+      const free = record();
+      const sent = record();
+      for (const item of [free, sent]) await storage.save(item, bytes());
+      await storage.markAttached(sent.id);
+      globalExpect(await storage.removeUnattached(free.id)).toBe(true);
+      globalExpect(await storage.get(free.id)).toBeUndefined();
+      globalExpect(await storage.readBytes(free.id)).toBeUndefined();
+      globalExpect(await storage.removeUnattached(sent.id)).toBe(false);
+      globalExpect((await storage.get(sent.id))?.attached).toBe(true);
+      globalExpect([...await storage.readBytes(sent.id)]).toEqual([1, 2, 3]);
+      globalExpect(await storage.removeUnattached(free.id)).toBe(false);
+      globalExpect(await storage.removeUnattached("../../etc/passwd")).toBe(false);
+    });
+    async function markedThenRead(storage, id) {
+      await storage.markAttached(id);
+      return (await storage.get(id))?.attached === true;
+    }
+    async function expectIntactIfMarked(storage, id, marked) {
+      if (marked) {
+        globalExpect((await storage.get(id))?.attached).toBe(true);
+        globalExpect([...await storage.readBytes(id) ?? []]).toEqual([1, 2, 3]);
+      } else {
+        globalExpect(await storage.get(id)).toBeUndefined();
+        globalExpect(await storage.readBytes(id)).toBeUndefined();
+      }
+    }
+    it("removeUnattached \u043E\u0434\u043D\u043E\u0432\u0440\u0435\u043C\u0435\u043D\u043D\u043E \u0441 \u043E\u0442\u043C\u0435\u0442\u043A\u043E\u0439: \u043E\u0442\u043C\u0435\u0447\u0435\u043D\u043D\u044B\u0439 \u0444\u0430\u0439\u043B \u0446\u0435\u043B, \u0438\u043D\u0430\u0447\u0435 \u0443\u0431\u0440\u0430\u043D \u0446\u0435\u043B\u0438\u043A\u043E\u043C", async () => {
+      const storage = await createStorage();
+      const items = Array.from({ length: 20 }, () => record());
+      for (const item of items) await storage.save(item, bytes());
+      const outcomes = await Promise.all(items.map(async (item) => {
+        const [marked, removed] = await Promise.all([markedThenRead(storage, item.id), storage.removeUnattached(item.id)]);
+        return { item, marked, removed };
+      }));
+      for (const { item, marked, removed } of outcomes) {
+        globalExpect(removed).toBe(!marked);
+        await expectIntactIfMarked(storage, item.id, marked);
+      }
+    });
+    it("removeUnattachedOlderThan \u043E\u0434\u043D\u043E\u0432\u0440\u0435\u043C\u0435\u043D\u043D\u043E \u0441 \u043E\u0442\u043C\u0435\u0442\u043A\u043E\u0439: \u043E\u0442\u043C\u0435\u0447\u0435\u043D\u043D\u044B\u0439 \u0444\u0430\u0439\u043B \u0446\u0435\u043B, \u0438\u043D\u0430\u0447\u0435 \u0443\u0431\u0440\u0430\u043D \u0446\u0435\u043B\u0438\u043A\u043E\u043C", async () => {
+      const storage = await createStorage();
+      const items = Array.from({ length: 20 }, () => record());
+      for (const item of items) await storage.save(item, bytes());
+      const [marks] = await Promise.all([
+        Promise.all(items.map((item) => markedThenRead(storage, item.id))),
+        storage.removeUnattachedOlderThan(/* @__PURE__ */ new Date("2027-01-01T00:00:00.000Z"))
+      ]);
+      for (const [index, item] of items.entries()) await expectIntactIfMarked(storage, item.id, marks[index]);
+    });
+    it("remove \u043E\u0434\u043D\u043E\u0432\u0440\u0435\u043C\u0435\u043D\u043D\u043E \u0441 \u043E\u0442\u043C\u0435\u0442\u043A\u043E\u0439: \u0437\u0430\u043F\u0438\u0441\u0438 \u0431\u0435\u0437 \u0431\u0430\u0439\u0442\u043E\u0432 \u043D\u0435 \u043E\u0441\u0442\u0430\u0451\u0442\u0441\u044F", async () => {
+      const storage = await createStorage();
+      const items = Array.from({ length: 20 }, () => record());
+      for (const item of items) await storage.save(item, bytes());
+      await Promise.all(items.flatMap((item) => [storage.markAttached(item.id), storage.remove(item.id)]));
+      for (const item of items) {
+        const left = await storage.get(item.id);
+        globalExpect(left === void 0 ? void 0 : [...await storage.readBytes(item.id) ?? []]).toEqual(left && [1, 2, 3]);
+      }
+    });
     it("removeAllOfChat \u0443\u0431\u0438\u0440\u0430\u0435\u0442 \u0432\u0441\u0435 \u0444\u0430\u0439\u043B\u044B \u044D\u0442\u043E\u0433\u043E \u0447\u0430\u0442\u0430, \u0438 \u043F\u0440\u0438\u043A\u0440\u0435\u043F\u043B\u0451\u043D\u043D\u044B\u0435 \u0442\u043E\u0436\u0435; \u0447\u0443\u0436\u0438\u0435 \u0447\u0430\u0442\u044B \u0446\u0435\u043B\u044B; \u0433\u043E\u0432\u043E\u0440\u0438\u0442, \u0441\u043A\u043E\u043B\u044C\u043A\u043E \u0443\u0431\u0440\u0430\u043B", async () => {
       const storage = await createStorage();
       const free = record({ chatId: "chat-1" });
@@ -15448,6 +15509,7 @@ function describeAttachmentStorageContract(createStorage) {
         globalExpect(await storage.readBytes(id)).toBeUndefined();
         await storage.remove(id);
         await storage.markAttached(id);
+        globalExpect(await storage.removeUnattached(id)).toBe(false);
       }
     });
     it("\u043A\u0430\u0440\u0442\u0438\u043D\u043A\u0430 \u0431\u0435\u0437 \u0438\u0437\u0432\u043B\u0435\u0447\u0451\u043D\u043D\u043E\u0433\u043E \u0442\u0435\u043A\u0441\u0442\u0430 \u0445\u0440\u0430\u043D\u0438\u0442\u0441\u044F \u0431\u0435\u0437 \u044D\u0442\u0438\u0445 \u043F\u043E\u043B\u0435\u0439, \u0438\u0441\u0445\u043E\u0434\u043D\u044B\u0439 \u0442\u0438\u043F \u0441\u043E\u0445\u0440\u0430\u043D\u044F\u0435\u0442\u0441\u044F", async () => {