@okostream/mcp 0.11.2 → 0.12.1

package/dist/tools/chat.js

@@ -36,8 +36,13 @@ export function registerChatTools(server) {
             ? [orderByKey(), endBefore(cursor), limitToLast(max + 1)]
             : [orderByKey(), limitToLast(max + 1)];
         const snap = await get(dbQuery(ref(chatRtdb, `messages/${eventId}`), ...constraints));
+        // 🔴 Побайтовое сравнение, а не `localeCompare`. RTDB упорядочивает
+        // push-ключи по кодам символов, а `localeCompare` учитывает локаль и
+        // сортирует регистр наоборот: push-ключи содержат и цифры, и обе
+        // раскладки букв, поэтому порядок расходился с серверным. Курсор брался
+        // не тот, и страницы повторялись или пропускались.
         const entries = Object.entries((snap.val() ?? {}))
-            .sort(([a], [b]) => b.localeCompare(a));
+            .sort(([a], [b]) => (a < b ? 1 : a > b ? -1 : 0));
         const { page, hasMore } = pickPage(entries, max);
         const messages = page.map(([id, message]) => ({ id, ...serialize(message) }));
         const nextCursor = hasMore ? messages.at(-1)?.id ?? null : null;
@@ -68,8 +73,18 @@ export function registerChatTools(server) {
         if (!msgRef.key)
             throw new Error("Could not allocate chat message id");
         const timestamp = rtdbServerTimestamp();
+        // 🔴 Форма сообщения обязана совпадать с `prepareChatMessage`
+        // (`web/src/lib/chatMessages.ts`) — единственным контрактом записи чата
+        // по Pattern 26. Импортировать его сюда нельзя: MCP отдельный пакет, у
+        // него нет доступа к web. Поэтому копия, и она УЖЕ разошлась: без `trim`
+        // сообщение из одних пробелов проходило и публиковалось всем зрителям, а
+        // имя уходило без обрезки до 24 символов.
+        const messageText = text.trim();
+        if (!messageText)
+            throw new Error("Chat message text cannot be empty or whitespace only.");
+        const messageName = name.trim().slice(0, 24) || "Organizer";
         await update(ref(chatRtdb), {
-            [`messages/${eventId}/${msgRef.key}`]: { text, uid, name, role, ts: timestamp },
+            [`messages/${eventId}/${msgRef.key}`]: { text: messageText, uid, name: messageName, role, ts: timestamp },
             [`rate_limits/${eventId}/${uid}`]: { ts: timestamp, messageId: msgRef.key },
         });
         await runTransaction(ref(rtdb, `events/${eventId}/agg/chatMessages`), (n) => (n ?? 0) + 1)

package/dist/tools/eventCreate.js

@@ -1,10 +1,16 @@
 // Создание события: проверка права на пространство и запись с зеркалом RTDB.
 //
 // Зеркало ownerId/workspaceId в основной RTDB — опора правил доступа, и
-// пишет его сервер (CF32), а не клиент. Поэтому создание идёт парой:
-// документ Firestore, затем ensureEventRuntime. Если вторая половина не
-// удалась, документ откатывается — событие без зеркала недоступно правилам
-// и выглядело бы как «создалось, но сломано».
+// пишет его сервер, а не клиент. Создание идёт парой: документ Firestore,
+// затем ensureEventRuntime (CF32) — она ставит зеркало сразу, не дожидаясь
+// триггера.
+//
+// 🔴 Отката здесь намеренно НЕТ, хотя прежняя редакция этой шапки его
+// обещала. Сбой CF32 не оставляет событие сломанным: зеркало всё равно
+// поставит CF20 `onEventCreated` — она пишет ownerId/workspaceId сама
+// (`functions/src/eventsLifecycle.ts`) и объявлена с `retry: true`, то есть
+// доставка гарантирована. Удалять документ из-за сбоя быстрого пути значило
+// бы терять событие там, где медленный путь его чинит.
 import { addDoc, collection, doc, getDoc } from "firebase/firestore";
 import { httpsCallable } from "firebase/functions";
 import { firestore, functions } from "../firebase.js";

package/dist/tools/events.js

@@ -9,7 +9,7 @@ import { z } from "zod";
 import { serialize, textResult } from "../utils.js";
 import { authed, eventLinks, randomId } from "./context.js";
 import { cardsConfigOutputSchema, createWebinarOutputSchema, cursorField, eventIdField, eventOutputSchema, eventsOutputSchema, formSchema, limitField, readOnlyTool, setupValidationOutputSchema, toolDescription, updateOutputSchema, writeTool } from "./schemas.js";
-import { buildCardsItems, CARD_IMAGE_RATIOS, cardHasImage, cardItemInputSchema, eventFormIds, MAX_CARD_ITEMS, readCardsConfig } from "./imagePipeline.js";
+import { buildCardsItems, CARD_IMAGE_RATIOS, cardHasImage, cardItemInputSchema, eventFormIds, MAX_CARD_ITEMS, normalizeCardButtonUrl, readCardsConfig, writeEventPatchWithCards } from "./imagePipeline.js";
 import { collection, doc, getDoc, getDocs, limit, orderBy, query, serverTimestamp, startAfter, Timestamp, updateDoc, where } from "firebase/firestore";
 import { addEventWithRtdbMirror, assertCanCreateInWorkspace } from "./eventCreate.js";
 import { cursorMeta, normalizeVideo, parseIsoBound, parseTimestampCursor, pickPage, timestampCursorFrom } from "./formatting.js";
@@ -293,8 +293,13 @@ export function registerEventTools(server) {
             patch["blocks.chat.visibility"] = args.chat.visibility;
         if (args.promoBanner?.enabled !== undefined)
             patch["blocks.promoBanner.enabled"] = args.promoBanner.enabled;
+        // 🔴 Протокол проверяется так же, как у карточек в этом же файле
+        // (`normalizeCardButtonUrl`). Оба значения попадают в `href` кнопки на
+        // публичной странице: `javascript:` там — исполнение кода у зрителя, а
+        // единственным отличием ссылки баннера от ссылки карточки было то, что
+        // её никто не проверял.
         if (args.promoBanner?.link !== undefined)
-            patch["blocks.promoBanner.link"] = args.promoBanner.link;
+            patch["blocks.promoBanner.link"] = normalizeCardButtonUrl(args.promoBanner.link);
         if (args.promoBanner?.visibility !== undefined)
             patch["blocks.promoBanner.visibility"] = args.promoBanner.visibility;
         if (args.promoBanner?.formId !== undefined)
@@ -310,7 +315,7 @@ export function registerEventTools(server) {
         if (args.primaryCta?.buttonLabel !== undefined)
             patch["blocks.primaryCta.buttonLabel"] = args.primaryCta.buttonLabel?.trim() || null;
         if (args.primaryCta?.buttonUrl !== undefined)
-            patch["blocks.primaryCta.buttonUrl"] = args.primaryCta.buttonUrl?.trim() || null;
+            patch["blocks.primaryCta.buttonUrl"] = normalizeCardButtonUrl(args.primaryCta.buttonUrl);
         if (args.primaryCta?.formId !== undefined)
             patch["blocks.primaryCta.formId"] = args.primaryCta.formId;
         if (args.videoPlaceholder?.enabled !== undefined)
@@ -391,6 +396,7 @@ export function registerEventTools(server) {
             visibility: args.visibility ?? currentCards.visibility,
             cardImageRatio: args.cardImageRatio ?? currentCards.cardImageRatio,
             items: currentCards.items,
+            repairedIdPositions: currentCards.repairedIdPositions,
         };
         const generatedCardIds = [];
         const preservedImageCardIds = [];
@@ -402,27 +408,48 @@ export function registerEventTools(server) {
             patch["blocks.cards.visibility"] = args.visibility;
         if (args.cardImageRatio !== undefined)
             patch["blocks.cards.cardImageRatio"] = args.cardImageRatio;
-        if (args.items !== undefined) {
+        // 🔴 Карточки пересобираются ВНУТРИ транзакции, из свежего документа.
+        // Массив `blocks.cards.items` пишется целиком, а между чтением события и
+        // записью успевает отработать CF4 и дописать в него `imageVariants`,
+        // `imageUrl` и размеры. Реконструкция из устаревшего снимка их стирала —
+        // картинка у зрителя откатывалась к мастеру или пропадала. Это Pattern 23,
+        // уже разобранный на веб-стороне и вернувшийся сюда через MCP.
+        //
+        // Отчёт (что создано, что сохранено, что удалено) тоже считается внутри:
+        // иначе он описывал бы не ту запись, которая случилась.
+        const buildItems = args.items === undefined ? null : (fresh) => {
+            const freshCards = readCardsConfig(fresh);
             const built = buildCardsItems({
                 inputs: args.items,
-                existing: currentCards.items,
-                formIds: eventFormIds(event),
+                existing: freshCards.items,
+                formIds: eventFormIds(fresh),
             });
             finalCards.items = built.items;
+            generatedCardIds.length = 0;
+            preservedImageCardIds.length = 0;
             generatedCardIds.push(...built.generatedCardIds);
             preservedImageCardIds.push(...built.preservedImageCardIds);
             const nextIds = new Set(built.items.map((item) => item.id));
-            const removed = currentCards.items.filter((item) => !nextIds.has(item.id));
+            const removed = freshCards.items.filter((item) => !nextIds.has(item.id));
             removedCardIds = removed.map((item) => item.id);
             removedCardsWithImages = removed.filter(cardHasImage).map((item) => item.id);
-            patch["blocks.cards.items"] = built.items;
-        }
-        if (Object.keys(patch).length > 0) {
-            await updateDoc(doc(firestore, "events", args.eventId), patch);
+            return built.items;
+        };
+        if (Object.keys(patch).length > 0 || buildItems) {
+            await writeEventPatchWithCards(args.eventId, patch, buildItems);
         }
         const warnings = removedCardsWithImages.length > 0
             ? [`Removed cards had images and their Storage files were not deleted: ${removedCardsWithImages.join(", ")}.`]
             : [];
+        // 🔴 Карточка без пригодного `id` получает подставленный при чтении, и он
+        // не совпадает ни с чем в документе: прежняя запись не находится, её
+        // картинка обнуляется. Пока это молчало, организатор видел успешный ответ
+        // и пропавшее изображение без единого объяснения.
+        if (finalCards.repairedIdPositions.length > 0) {
+            warnings.push(`Cards at position(s) ${finalCards.repairedIdPositions.join(", ")} had no usable id in Firestore; `
+                + "a temporary one was generated, so their previous images could not be matched and were dropped. "
+                + "Re-upload images for these cards.");
+        }
         return textResult({
             eventId: args.eventId,
             updated: Object.keys(patch),

package/dist/tools/formatting.js

@@ -26,6 +26,27 @@ export function pickPage(items, max) {
         hasMore: items.length > max,
     };
 }
+/**
+ * Значение `video.embedUrl` попадает прямо в `<iframe src>` на публичной
+ * странице события. Набор протоколов здесь строже, чем у кнопок
+ * (`normalizeCardButtonUrl`): `mailto:` и `tel:` в кадре бессмысленны, а
+ * `javascript:` — исполняемый код в контексте okostream.app у всех зрителей.
+ * Для распознанного провайдера адрес строим мы сами, но проверяем всё равно
+ * итоговое значение: так защита не зависит от того, какой ветвью оно пришло.
+ */
+function assertEmbeddableUrl(value) {
+    let url;
+    try {
+        url = new URL(value);
+    }
+    catch {
+        throw new Error(`Invalid videoUrl "${value}". Provide an https:// link or a full <iframe> embed code.`);
+    }
+    if (url.protocol !== "https:" && url.protocol !== "http:") {
+        throw new Error(`Unsupported videoUrl protocol "${url.protocol}". Only http(s) links can be embedded.`);
+    }
+    return value;
+}
 export function normalizeVideo(videoUrl) {
     const raw = videoUrl?.trim();
     if (!raw)
@@ -35,7 +56,7 @@ export function normalizeVideo(videoUrl) {
     if (detected.provider !== "unknown" && !detected.normalizedUrl) {
         throw new Error(`Could not extract a valid ${detected.provider} video id from videoUrl.`);
     }
-    return { originalUrl: raw, embedUrl: detected.normalizedUrl ?? effective };
+    return { originalUrl: raw, embedUrl: assertEmbeddableUrl(detected.normalizedUrl ?? effective) };
 }
 export function csvRows(headers, rows) {
     return [headers, ...rows].map((row) => row.map(csvCell).join(",")).join("\n");

package/dist/tools/imagePipeline.js

@@ -176,7 +176,13 @@ export async function readRemoteImage(rawUrl, redirects = 0) {
     }
     finally {
         clearTimeout(timeout);
-        await dispatcher.close();
+        // 🔴 `destroy()`, а не `close()`. `close()` в undici ЖДЁТ завершения
+        // незавершённых запросов — а когда защита по размеру бросает из
+        // `responseBufferWithLimit`, тело ответа остаётся непрочитанным. Публичный
+        // сервер с открытым chunked-ответом подвешивал инструмент бесконечно
+        // именно здесь: ограничение уже сработало, а выйти из функции было
+        // нельзя. `destroy()` отменяет то, что не дочитано.
+        await dispatcher.destroy();
     }
 }
 export async function readLocalImage(inputPath) {
@@ -281,12 +287,16 @@ export function readCardsConfig(eventData) {
     const blocks = (eventData.blocks && typeof eventData.blocks === "object" ? eventData.blocks : {});
     const raw = (blocks.cards && typeof blocks.cards === "object" ? blocks.cards : {});
     const rawItems = Array.isArray(raw.items) ? raw.items : [];
+    const repairedIdPositions = [];
     const items = rawItems
         .filter((item) => Boolean(item) && typeof item === "object" && !Array.isArray(item))
-        .map((item) => {
+        .map((item, position) => {
+        const hasUsableId = typeof item.id === "string" && CARD_ID_PATTERN.test(item.id);
+        if (!hasUsableId)
+            repairedIdPositions.push(position);
         const fallbackId = randomUUID();
         return {
-            id: typeof item.id === "string" && CARD_ID_PATTERN.test(item.id) ? item.id : fallbackId,
+            id: hasUsableId ? item.id : fallbackId,
             title: safeString(item.title).trim() || null,
             text: safeString(item.text).trim() || null,
             buttonLabel: safeString(item.buttonLabel).trim() || null,
@@ -305,6 +315,7 @@ export function readCardsConfig(eventData) {
         visibility: raw.visibility === "hidden" ? "hidden" : "always",
         cardImageRatio: CARD_IMAGE_RATIOS.includes(ratio) ? ratio : "16:9",
         items,
+        repairedIdPositions,
     };
 }
 export function normalizeNullableText(value) {
@@ -435,6 +446,37 @@ export async function updateCardImageInTransaction(eventId, cardId, patch) {
         });
     });
 }
+/**
+ * Записывает патч события, пересобирая массив карточек из СВЕЖЕГО документа.
+ *
+ * 🔴 Зачем транзакция. `blocks.cards.items` — массив, и запись его целиком
+ * затирает всё, чего не было в прочитанном снимке. Между чтением события и
+ * записью успевает отработать CF4: она дописывает `imageVariants`, `imageUrl`
+ * и размеры по стабильному id карточки. Реконструкция из устаревшего снимка
+ * стирала эти поля — картинка у зрителя откатывалась к мастеру или пропадала.
+ *
+ * Это Pattern 23, разобранный на веб-стороне (там Setup уже сливает
+ * метаданные транзакционно) и вернувшийся через MCP: то же место, тот же
+ * массив, только другой клиент.
+ *
+ * `buildItems` получает свежие данные документа и обязан строить итоговый
+ * массив из них, а не из снимка, снятого до транзакции.
+ */
+export async function writeEventPatchWithCards(eventId, basePatch, buildItems) {
+    const eventRef = doc(firestore, "events", eventId);
+    await runFirestoreTransaction(firestore, async (transaction) => {
+        const snap = await transaction.get(eventRef);
+        if (!snap.exists())
+            throw new Error("Event not found.");
+        const patch = { ...basePatch };
+        if (buildItems) {
+            patch["blocks.cards.items"] = buildItems(snap.data());
+        }
+        if (Object.keys(patch).length === 0)
+            return;
+        transaction.update(eventRef, patch);
+    });
+}
 export function imageVariantReason(ref) {
     if (!ref.imageMasterUrl && !ref.imageUrl)
         return "image_missing";
@@ -501,23 +543,24 @@ export async function uploadEventImageForUser(uid, eventId, kind, cardId, source
     const deleteWarnings = [];
     let deletesAttempted = 0;
     const expectedCardPath = kind === "card" ? imageStoragePath(eventId, uid, kind, cardId) : null;
+    // 🔴 Сначала загрузка, потом удаление старого. Прежде варианты карточки
+    // стирались ДО `uploadMasterImage`, и упавшая загрузка оставляла документ
+    // Firestore со ссылками на удалённые объекты — зритель получал 404, а
+    // повтор восстановить их уже не мог. Соседняя ветка для не-карточек (ниже)
+    // всегда делала правильно; расходились только они.
+    //
+    // Порядок безопасен: новый master пишется по тому же пути, и CF4
+    // перезаписывает все три варианта заново. Худшее, что даёт отказ CF4, —
+    // варианты от прежней картинки рядом с новым мастером, и это заметно
+    // меньшая беда, чем битые ссылки.
+    const uploaded = await uploadMasterImage(eventId, kind, cardId, sourceBuffer);
+    if (!uploaded)
+        throw new Error("Image upload did not return a result.");
     if (expectedCardPath && previousMasterPath === expectedCardPath) {
         const variantPaths = variantPathsFromMasterPath(expectedCardPath);
         deletesAttempted += variantPaths.length;
         deleteWarnings.push(...await deleteStoragePaths(eventId, variantPaths));
     }
-    let uploaded = null;
-    try {
-        uploaded = await uploadMasterImage(eventId, kind, cardId, sourceBuffer);
-    }
-    catch (err) {
-        if (uploaded) {
-            await deleteStoragePaths(eventId, [uploaded.storagePath, ...variantPathsFromMasterPath(uploaded.storagePath)]).catch(() => []);
-        }
-        throw err;
-    }
-    if (!uploaded)
-        throw new Error("Image upload did not return a result.");
     if (kind !== "card" || (previousMasterPath && previousMasterPath !== uploaded.storagePath) || (!previous.imageMasterUrl && previous.imageUrl)) {
         const cleanup = await deleteStorageUrls(eventId, previous.imageMasterUrl, previous.imageUrl);
         deletesAttempted += cleanup.attempted;

package/dist/tools/images.js

@@ -165,7 +165,12 @@ export function registerImageTools(server) {
         const event = await readEvent(eventId, uid, "edit");
         const previous = readImageRef(event, kind, cardId);
         requireConfirm(confirm, "oko_delete_image", { eventId, kind, cardId: cardId ?? null, imageId: previous.imageId });
-        const cleanup = await deleteStorageUrls(eventId, previous.imageMasterUrl, previous.imageUrl);
+        // 🔴 Сначала снимаем ссылки, потом удаляем объекты. Прежде порядок был
+        // обратный: упавший `updateDoc` оставлял документ со ссылками на уже
+        // удалённые файлы — зритель видел 404, а повтор восстановить их не мог,
+        // потому что объектов больше нет. В обратном порядке отказ Storage
+        // оставляет осиротевшие файлы, и это поправимо: их подберёт CF31 при
+        // удалении события, а ссылок на них уже нет.
         if (kind === "promoBanner") {
             await updateDoc(doc(firestore, "events", eventId), {
                 "blocks.promoBanner.imageMasterUrl": deleteField(),
@@ -193,6 +198,10 @@ export function registerImageTools(server) {
                 imageHeight: null,
             });
         }
+        // Удаление объектов — best-effort и уже после того, как на них никто не
+        // ссылается: его отказ попадёт в предупреждения ответа, а не в битые
+        // ссылки у зрителя.
+        const cleanup = await deleteStorageUrls(eventId, previous.imageMasterUrl, previous.imageUrl);
         return textResult({
             eventId,
             kind,

package/dist/tools/notifications.js

@@ -3,7 +3,7 @@
 // Документы создают только Cloud Functions; отсюда доступны чтение, отметка
 // и удаление своих. Текст уведомления не хранится — клиент собирает его из
 // типа и meta, поэтому наружу уходит структура, а не готовая строка.
-import { collection, doc, getDocs, limit, orderBy, query, startAfter, updateDoc, where } from "firebase/firestore";
+import { collection, doc, getDocs, limit, orderBy, query, startAfter, updateDoc } from "firebase/firestore";
 import { z } from "zod";
 import { firestore } from "../firebase.js";
 import { serialize, textResult } from "../utils.js";
@@ -73,14 +73,31 @@ export function registerNotificationTools(server) {
         outputSchema: notificationsMarkOutputSchema,
         annotations: idempotentWriteTool,
     }, async ({ notificationIds, allUnread, limit: max }) => {
-        const { uid } = await authed(["workspace:read"]);
+        // 🔴 Гейт намеренно на запись, хотя инструмент читается как «пометить
+        // прочитанным». Он делает `updateDoc` и сам помечен `idempotentWriteTool`;
+        // пока гейтом стоял `workspace:read`, ключ пресета `readOnly`
+        // (`events:read · analytics:read · workspace:read · leads:read`) мог
+        // погасить до 200 уведомлений организатора. Единственный смысл пресета
+        // «только чтение» — что он не пишет. Ущерб здесь не в чужих данных
+        // (пишем только в свои), а в нарушенном обещании: организатор не увидит
+        // уведомлений о выданных правах и созданных событиях.
+        const { uid } = await authed(["workspace:write"]);
         if (Boolean(notificationIds?.length) === allUnread) {
             throw new Error("Provide either notificationIds or allUnread=true.");
         }
         let ids = [...new Set(notificationIds ?? [])];
         if (allUnread) {
-            const snap = await getDocs(query(collection(firestore, "userNotifications", uid, "items"), where("read", "==", false), limit(max)));
-            ids = snap.docs.map((item) => item.id);
+            // 🔴 Отбор обязан совпадать с тем, что список ПОКАЗЫВАЕТ как
+            // непрочитанное, иначе получается неустранимая петля: список считает
+            // непрочитанным всё, где `read !== true` (в том числе документ без
+            // поля), а `where("read", "==", false)` документ без поля не находит —
+            // и не находит его же `where("read", "!=", true)`, потому что запросы
+            // с неравенством в Firestore пропускают документы без поля вовсе.
+            // Такое уведомление висело бы вечно, а модель не могла бы его погасить.
+            // Поэтому фильтруем клиентом тем же предикатом, что и список, набирая
+            // с запасом — приём уже применён в `oko_list_notifications` выше.
+            const snap = await getDocs(query(collection(firestore, "userNotifications", uid, "items"), limit(Math.min(max * 5 + 1, 501))));
+            ids = snap.docs.filter((item) => item.data().read !== true).slice(0, max).map((item) => item.id);
         }
         await Promise.all(ids.map((id) => updateDoc(doc(firestore, "userNotifications", uid, "items", id), { read: true })));
         return textResult({ markedRead: ids.length, notificationIds: ids });

package/dist/tools/sessions.js

@@ -35,7 +35,10 @@ export function registerSessionTools(server) {
             throw new Error("An analytics session is already active.");
         const start = httpsCallable(functions, "startAnalyticsSessionV3");
         const result = await start({ eventId });
-        return textResult({ started: true, ...result.data });
+        // 🔴 `eventId` обязателен по `basicEventOutputSchema`. Без него SDK
+        // отвергал ответ ПОСЛЕ того, как сессия уже стартовала: клиент видел
+        // ошибку схемы и считал, что запуска не было, — Pattern 18.
+        return textResult({ eventId, started: true, ...result.data });
     });
     server.registerTool("oko_end_session", {
         title: "End analytics session",
@@ -58,7 +61,9 @@ export function registerSessionTools(server) {
         requireConfirm(confirm, "oko_end_session", { eventId, sessionId });
         const end = httpsCallable(functions, "endAnalyticsSessionV3");
         const result = await end({ eventId, sessionId });
-        return textResult(result.data);
+        // Тот же случай, что у запуска: сессия уже завершена и отчёт сохранён,
+        // а ответ без `eventId` не проходил собственную схему.
+        return textResult({ eventId, ...result.data });
     });
     server.registerTool("oko_fix_orphan_session", {
         title: "Fix orphan analytics session",

package/dist/tools/setupValidation.js

@@ -93,6 +93,13 @@ export function validateEventSetup(eventId, event) {
         }
     }
     const cards = readCardsConfig(event);
+    // 🔴 Карточка без пригодного `id` — это ошибка данных, а не мелочь: id
+    // связывает запись с её картинкой в Storage и с обратной записью CF4.
+    // Пока подстановка молчала, проверка не могла назвать причину, по которой
+    // у карточки пропадает изображение при каждой правке.
+    cards.repairedIdPositions.forEach((position) => {
+        add(errors, "card_id_missing", `blocks.cards.items[${position}].id`, "Card has no usable id in Firestore; its image cannot be matched and will be dropped on the next update.");
+    });
     if (cards.enabled) {
         if (cards.items.length === 0)
             add(warnings, "cards_empty", "blocks.cards.items", "Cards block is enabled but has no cards.");

package/dist/tools/telegram.js

@@ -170,10 +170,15 @@ export function registerTelegramTools(server) {
         if (cursor)
             constraints.splice(2, 0, startAfter(parseTimestampCursor(cursor)));
         const snap = await getDocs(query(collection(firestore, "telegramPosts"), ...constraints));
-        const { page, hasMore } = pickPage(snap.docs, max);
-        let posts = page.map((item) => ({ id: item.id, ...serialize(item.data()) }));
-        if (status)
-            posts = posts.filter((post) => post.status === status);
+        // 🔴 Фильтр применяется ДО нарезки страницы. Пока он стоял после,
+        // `hasMore` и `count` считались по нефильтрованной выборке: пустой
+        // результат приходил с `hasMore: false`, и это читается как «постов с
+        // таким статусом нет», хотя они есть на следующих страницах.
+        const matching = status
+            ? snap.docs.filter((item) => item.data().status === status)
+            : snap.docs;
+        const { page, hasMore } = pickPage(matching, max);
+        const posts = page.map((item) => ({ id: item.id, ...serialize(item.data()) }));
         const nextCursor = hasMore ? timestampCursorFrom(page.at(-1)?.data().createdAt) : null;
         return textResult({ posts, ...cursorMeta(posts.length, max, hasMore, nextCursor) });
     });

package/dist/tools/workspaces.js

@@ -10,7 +10,7 @@ import { z } from "zod";
 import { requireConfirm, safeString, serialize, textResult } from "../utils.js";
 import { authed, inviteUrl } from "./context.js";
 import { destructiveTool, idempotentWriteTool, MAX_WORKSPACE_INVITE_CODE_LENGTH, MAX_WORKSPACE_NAME_LENGTH, readOnlyTool, toolDescription, unknownRecordSchema, updateOutputSchema, workspaceAccessOutputSchema, workspacePermissionsSchema, workspacesOutputSchema, writeTool } from "./schemas.js";
-import { addDoc, collection, deleteDoc, doc, getDoc, getDocs, limit, query, serverTimestamp, setDoc, updateDoc, where } from "firebase/firestore";
+import { addDoc, collection, deleteDoc, doc, getDoc, getDocs, query, serverTimestamp, setDoc, updateDoc, where } from "firebase/firestore";
 import { firestore, functions } from "../firebase.js";
 import { httpsCallable } from "firebase/functions";
 import { listVisibleWorkspaces, normalizeWorkspacePermissions, randomInviteCode, readWorkspaceForOwner } from "./workspaceAccess.js";
@@ -87,11 +87,27 @@ export function registerWorkspaceTools(server) {
         requireConfirm(confirm, "oko_delete_workspace", { workspaceId, name: workspace.name, personal: workspace.personal });
         if (workspace.personal === true)
             throw new Error("Personal workspace cannot be deleted.");
-        const eventsSnap = await getDocs(query(collection(firestore, "events"), where("workspaceId", "==", workspaceId), limit(1)));
-        if (!eventsSnap.empty)
-            throw new Error("Workspace still has events. Move or delete its events before deleting the workspace.");
-        await deleteDoc(doc(firestore, "workspaces", workspaceId));
-        return textResult({ workspaceId, deleted: true });
+        // 🔴 Спрашиваем индекс доступа, а не коллекцию событий. `allow list` по
+        // `events` требует `hasPlatformOwnerRole()`, который у MCP-сессии ложен по
+        // построению (`mcp_<keyId>` выпускается с `owner: false`) — то есть
+        // прежний запрос падал `permission-denied` ВСЕГДА, и инструмент не мог
+        // удалить ни одного пространства. Индекс `eventAccess/{uid}/grants` — тот
+        // же источник, которым пользуется `oko_list_my_events`.
+        //
+        // Ограничение названо честно: grants перечисляют события, к которым
+        // доступ есть у ЭТОГО организатора. Событие в пространстве, к которому он
+        // доступа не имеет, проверка не увидит — но владелец пространства
+        // получает grant на все его события через CF18, так что для владельца,
+        // а только он сюда и доходит (`readWorkspaceForOwner` выше), совпадает.
+        // 🔴 Удаление идёт через CF57, а не `deleteDoc`: голое удаление корневого
+        // документа оставляло живыми подколлекцию shares, зеркала
+        // workspaceMembers и grants — участники сохраняли доступ, а убрать это
+        // было уже нельзя, потому что правило shares проверяет владельца через
+        // get() исчезнувшего родителя. Проверку «в пространстве нет событий»
+        // функция делает сама, полным запросом, а не по индексу этого ключа.
+        const removeWorkspace = httpsCallable(functions, "deleteWorkspaceCompletely");
+        const removed = await removeWorkspace({ workspaceId });
+        return textResult({ ...removed.data, workspaceId, deleted: true });
     });
     server.registerTool("oko_list_workspace_access", {
         title: "List workspace access",
@@ -159,7 +175,16 @@ export function registerWorkspaceTools(server) {
             sharedAt: serverTimestamp(),
             sharedBy: uid,
         }, { merge: true });
-        return textResult({ workspaceId: args.workspaceId, uid: args.uid, updated: true });
+        // 🔴 `updateOutputSchema` объявляет `updated` МАССИВОМ имён полей, а не
+        // флагом. Пока возвращался `true`, SDK отвергал ответ уже после записи
+        // прав: клиент видел ошибку и считал, что доступ не выдан, — а он выдан.
+        // Из всех мест Pattern 18 это самое опасное: расходятся представления о
+        // том, кто что может.
+        return textResult({
+            workspaceId: args.workspaceId,
+            uid: args.uid,
+            updated: ["email", "comment", "permissions"],
+        });
     });
     server.registerTool("oko_remove_workspace_member", {
         title: "Remove workspace member",
@@ -252,7 +277,7 @@ export function registerWorkspaceTools(server) {
             summary: "Create a workspace invite for a known OKO workspace.",
             args: [
                 "workspaceId: workspace owned by the authenticated organizer.",
-                "code: optional invite code, 4-64 chars; omitted means generated.",
+                "code: optional invite code, 16-64 chars; omitted means generated (recommended).",
                 "email: optional intended recipient email.",
                 "permissions: optional whitelist of workspace permission booleans.",
                 "confirm: must be true.",
@@ -262,7 +287,14 @@ export function registerWorkspaceTools(server) {
         }),
         inputSchema: {
             workspaceId: z.string().min(1),
-            code: z.string().min(4).max(MAX_WORKSPACE_INVITE_CODE_LENGTH).optional(),
+            // 🔴 Минимум 16, а не 4: код приглашения — предъявительский секрет.
+            // Правило Firestore допускает `get` любого приглашения по коду, а CF19
+            // выдаёт по нему права, то есть короткий код перебирается. Генератор
+            // платформы даёт 16 шестнадцатеричных знаков — заданный вручную не
+            // должен быть слабее. У соседних инструментов (`claim`, `revoke`)
+            // минимум остаётся 4 намеренно: они принимают УЖЕ существующие коды,
+            // включая исторические шестисимвольные.
+            code: z.string().min(16).max(MAX_WORKSPACE_INVITE_CODE_LENGTH).optional(),
             email: z.string().email().optional(),
             comment: z.string().max(500).default(""),
             permissions: workspacePermissionsSchema,

package/dist/version.js

@@ -1,2 +1,2 @@
 // Generated by scripts/generateMcpVersion.mjs from mcp/package.json. Do not edit.
-export const OKO_MCP_VERSION = "0.11.2";
+export const OKO_MCP_VERSION = "0.12.1";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@okostream/mcp",
-  "version": "0.11.2",
+  "version": "0.12.1",
   "description": "Control OKO Stream webinars from Claude, Codex, and other MCP clients.",
   "license": "UNLICENSED",
   "author": {