clawgram 2.21.1 → 2.23.0

package/dist/group-visible-reply-guard.js

@@ -5,10 +5,11 @@ exports.rememberVisibleGroupReply = rememberVisibleGroupReply;
 exports.rememberTurnSend = rememberTurnSend;
 exports.hadTurnSendJustNow = hadTurnSendJustNow;
 exports.resetVisibleGroupReplies = resetVisibleGroupReplies;
+const expiring_map_1 = require("./expiring-map");
 const helpers_1 = require("./helpers");
-/** When each visible reply went out, keyed by account + chat + incoming message. */
-const recentVisibleGroupReplies = new Map();
 const GROUP_VISIBLE_REPLY_TTL_MS = 10 * 60 * 1000;
+/** When each visible reply went out, keyed by account + chat + incoming message. */
+const recentVisibleGroupReplies = new expiring_map_1.ExpiringMap(GROUP_VISIBLE_REPLY_TTL_MS);
 /**
  * How long after the agent's own send core's delivery of the same turn's final
  * text still counts as an echo rather than as something new.
@@ -29,36 +30,19 @@ function buildVisibleGroupReplyKey(input) {
     }
     return [input.accountId ?? "", chatId, currentMessageId].join("\n");
 }
-function pruneExpiredEntries(now) {
-    for (const [key, expiresAt] of recentVisibleGroupReplies.entries()) {
-        if (expiresAt <= now) {
-            recentVisibleGroupReplies.delete(key);
-        }
-    }
-}
 function hasRecentVisibleGroupReply(input) {
-    const now = Date.now();
-    pruneExpiredEntries(now);
     const key = buildVisibleGroupReplyKey(input);
     if (!key) {
         return false;
     }
-    const expiresAt = recentVisibleGroupReplies.get(key);
-    if (!expiresAt) {
-        return false;
-    }
-    if (expiresAt <= now) {
-        recentVisibleGroupReplies.delete(key);
-        return false;
-    }
-    return true;
+    return recentVisibleGroupReplies.get(key) === true;
 }
 function rememberVisibleGroupReply(input, sentAt = Date.now()) {
     const key = buildVisibleGroupReplyKey(input);
     if (!key) {
         return;
     }
-    recentVisibleGroupReplies.set(key, sentAt + GROUP_VISIBLE_REPLY_TTL_MS);
+    recentVisibleGroupReplies.set(key, true, sentAt);
 }
 /**
  * When a turn last spoke for itself, keyed the same way.
@@ -68,14 +52,16 @@ function rememberVisibleGroupReply(input, sentAt = Date.now()) {
  * quietly make that rule stricter. This one answers a different question —
  * did core just echo the turn that has only now finished.
  */
-const lastTurnSends = new Map();
+// Та же машинерия, что у соседней карты: запись жила до чтения, а читают её
+// только если ядро прислало эхо этого хода. Хода без эха — большинство (A6-16).
+const lastTurnSends = new expiring_map_1.ExpiringMap(GROUP_TURN_ECHO_WINDOW_MS);
 /** Records that the agent itself put a message in the chat during this turn. */
 function rememberTurnSend(input, sentAt = Date.now()) {
     const key = buildVisibleGroupReplyKey(input);
     if (!key) {
         return;
     }
-    lastTurnSends.set(key, sentAt);
+    lastTurnSends.set(key, true, sentAt);
 }
 /**
  * True when this turn already put a visible message in this chat moments ago.
@@ -94,15 +80,9 @@ function hadTurnSendJustNow(input, now = Date.now()) {
     if (!key) {
         return false;
     }
-    const sentAt = lastTurnSends.get(key);
-    if (sentAt === undefined) {
-        return false;
-    }
-    if (now - sentAt > GROUP_TURN_ECHO_WINDOW_MS) {
-        lastTurnSends.delete(key);
-        return false;
-    }
-    return true;
+    // Окно эха и есть TTL записи, поэтому «свежесть» теперь спрашивается
+    // у карты: она же и вычистит просроченное.
+    return lastTurnSends.get(key, now) === true;
 }
 /** Test seam: module state must not leak between suites. */
 function resetVisibleGroupReplies() {

package/dist/helpers.js

@@ -46,6 +46,7 @@ exports.normalizeParseMode = normalizeParseMode;
 exports.resolveReplyParseMode = resolveReplyParseMode;
 exports.resolveOutboundParseMode = resolveOutboundParseMode;
 exports.resolveDryRun = resolveDryRun;
+exports.parseOptionalThreadId = parseOptionalThreadId;
 const node_fs_1 = require("node:fs");
 const node_path_1 = __importDefault(require("node:path"));
 const core_1 = require("openclaw/plugin-sdk/core");
@@ -135,6 +136,37 @@ function resolveTranscriptPathFromStoreEntry(input) {
         return undefined;
     }
 }
+/**
+ * Хвост стенограммы, а не вся она.
+ *
+ * Читался весь файл целиком и синхронно — на каждом ходе, где ядро ничего не
+ * доставило. Стенограмма живой сессии растёт неограниченно, а нужна ровно
+ * последняя запись ассистента: всё, что дальше пары сотен килобайт назад, по
+ * определению не «только что» и проверку свежести всё равно не прошло бы
+ * (находка A6-15).
+ *
+ * Первая строка куска отбрасывается: чтение с произвольного смещения почти
+ * наверняка попадает в середину строки, а заодно — в середину UTF-8-символа.
+ * Целая строка перед ней нам не нужна, потому что ищем мы с конца.
+ */
+const TRANSCRIPT_TAIL_BYTES = 256 * 1024;
+function readTranscriptTail(sessionFile) {
+    const fd = (0, node_fs_1.openSync)(sessionFile, "r");
+    try {
+        const size = (0, node_fs_1.fstatSync)(fd).size;
+        if (size <= TRANSCRIPT_TAIL_BYTES) {
+            return (0, node_fs_1.readFileSync)(sessionFile, "utf8");
+        }
+        const buffer = Buffer.allocUnsafe(TRANSCRIPT_TAIL_BYTES);
+        const read = (0, node_fs_1.readSync)(fd, buffer, 0, TRANSCRIPT_TAIL_BYTES, size - TRANSCRIPT_TAIL_BYTES);
+        const text = buffer.subarray(0, read).toString("utf8");
+        const firstBreak = text.indexOf("\n");
+        return firstBreak === -1 ? "" : text.slice(firstBreak + 1);
+    }
+    finally {
+        (0, node_fs_1.closeSync)(fd);
+    }
+}
 /**
  * Salvages a reply that reached the transcript but not stdout.
  *
@@ -160,7 +192,7 @@ function readLatestAssistantFallbackFromTranscript(sessionKey, storePath, notBef
         if (!sessionFile) {
             return undefined;
         }
-        const lines = (0, node_fs_1.readFileSync)(sessionFile, "utf8")
+        const lines = readTranscriptTail(sessionFile)
             .split("\n")
             .map((line) => line.trim())
             .filter(Boolean);
@@ -268,15 +300,29 @@ function resolveActionTarget(params, toolContext) {
     }
     throw new Error("clawgram: message target is required");
 }
+/**
+ * Ответ на сообщение — во всех чатах, а не только в группах.
+ *
+ * Прежде `replyToId` молча отбрасывался, если тип цели не группа и не канал, а
+ * `inferOutboundTargetKind` возвращает `undefined` и для `@username`, и для
+ * положительного числового id — то есть для любой лички. Инструмент отвечал
+ * `ok: true`, человек получал сообщение вне ветки, и ничего об этом не
+ * сообщало. Приходящий путь лички при этом ветку проставлял, так что два пути
+ * расходились между собой (находка A6-05).
+ *
+ * Telegram поддерживает `reply_to` и в приватных чатах, поэтому чинится это
+ * не отказом, а тем, что ветка ставится везде. Нечисловой `replyToId` —
+ * ошибка вызова, а не повод молча отправить вне ветки.
+ */
 function resolveReplyToMessageIdForTarget(rawTarget, replyToId) {
     if (replyToId === null || replyToId === undefined || replyToId === "") {
         return undefined;
     }
-    const targetKind = inferOutboundTargetKind(rawTarget);
-    if (targetKind === "group" || targetKind === "channel") {
-        return Number(replyToId);
+    const id = Number(replyToId);
+    if (!Number.isFinite(id) || id <= 0) {
+        throw new Error(`clawgram: replyToId must be a message id, got ${JSON.stringify(replyToId)}`);
     }
-    return undefined;
+    return id;
 }
 /**
  * Разметка синтеза речи, которая не должна доехать до человека.
@@ -838,3 +884,23 @@ function resolveOutboundParseMode(params, cfg, accountId) {
         ? resolveReplyParseMode(cfg, accountId)
         : normalizeParseMode(raw);
 }
+/**
+ * `messageThreadId` из параметров: число, строка из цифр — или ничего.
+ *
+ * Живёт здесь, а не в channel.ts: им пользуется и диспетчер, и исходящий
+ * контур, а импорт из channel.ts в outbound.ts замкнул бы круг.
+ */
+function parseOptionalThreadId(value) {
+    if (typeof value === "number") {
+        return Number.isFinite(value) ? Math.trunc(value) : undefined;
+    }
+    if (typeof value !== "string") {
+        return undefined;
+    }
+    const trimmed = value.trim();
+    if (!trimmed || !/^\d+$/.test(trimmed)) {
+        return undefined;
+    }
+    const parsed = Number.parseInt(trimmed, 10);
+    return Number.isFinite(parsed) ? parsed : undefined;
+}

package/dist/history.js

@@ -20,6 +20,7 @@ exports.normalizeParticipants = normalizeParticipants;
 exports.parseListParticipantsParams = parseListParticipantsParams;
 exports.buildHistoryQuery = buildHistoryQuery;
 exports.normalizeChatKey = normalizeChatKey;
+exports.chatKeyCandidates = chatKeyCandidates;
 exports.isChatReadable = isChatReadable;
 exports.isWithinWindow = isWithinWindow;
 exports.normalizeHistoryMessage = normalizeHistoryMessage;
@@ -29,6 +30,7 @@ exports.HISTORY_MAX_LIMIT = 500;
 const media_1 = require("./media");
 const helpers_1 = require("./helpers");
 const constants_1 = require("./constants");
+const normalize_1 = require("./normalize");
 /**
  * Matches `normalize.ts` and `gramjs-client.ts` deliberately.
  *
@@ -41,17 +43,6 @@ const constants_1 = require("./constants");
  * Peer instead of the id inside it, which would otherwise produce a plausible
  * looking string.
  */
-function toStringId(value) {
-    if (value === null || value === undefined)
-        return undefined;
-    try {
-        const text = String(value);
-        return text && text !== "[object Object]" ? text : undefined;
-    }
-    catch {
-        return undefined;
-    }
-}
 /**
  * Accepts Unix seconds or anything `Date` can parse (ISO 8601 in practice).
  *
@@ -103,7 +94,11 @@ function parseMessageId(value, field) {
         return undefined;
     const numeric = typeof value === "number" ? value : Number(value);
     if (!Number.isFinite(numeric) || !Number.isInteger(numeric) || numeric < 1) {
-        throw new Error(`clawgram: ${field} must be a positive message id`);
+        // Значение в тексте — из копии, которая жила в reactions.ts: без него
+        // «must be a positive message id» не говорит, что именно пришло, а
+        // приходит туда обычно чужой объект. Две копии расходились ещё и
+        // проверками (`Number.isFinite` была только здесь) — A12-05.
+        throw new Error(`clawgram: ${field} must be a positive message id, got ${JSON.stringify(value)}`);
     }
     return numeric;
 }
@@ -153,7 +148,7 @@ function normalizeParticipants(raw, options) {
         if (!entry || typeof entry !== "object")
             continue;
         const candidate = entry;
-        const userId = toStringId(candidate.id);
+        const userId = (0, normalize_1.toStringId)(candidate.id);
         if (!userId)
             continue;
         const member = {
@@ -239,6 +234,29 @@ function buildHistoryQuery(args) {
 function normalizeChatKey(value) {
     return String(value ?? "").trim().replace(/^@/, "").toLowerCase();
 }
+/**
+ * Все написания одной цели, по которым её ищут в списке доступа.
+ *
+ * Ворота сравнивали сырое написание, а адрес приходит в нескольких формах:
+ * `-1001234`, `clawgram:-1001234` (префикс канала от ядра), `-1001234:topic:5`
+ * (тема форума). Разбор этих форм живёт в `gramjs-client`, то есть ПОСЛЕ
+ * ворот, — и чат, честно перечисленный в `readChats`, получал отказ, стоило
+ * ядру адресовать его с префиксом (находка A5-17).
+ *
+ * Возвращается и полное написание, и «только чат»: запись списка вида
+ * `-1001234:topic:5` сегодня работает как область в одну тему, и сведение
+ * всего к чату молча расширило бы её на весь чат.
+ */
+function chatKeyCandidates(target) {
+    const raw = String(target ?? "").trim();
+    const withoutChannel = raw.replace(/^(?:clawgram|tguserbot|telegram|tg):/i, "");
+    const withoutKind = withoutChannel.replace(/^(?:user|channel|group|conversation|room|dm):/i, "");
+    const chatOnly = withoutKind.replace(/:topic:\d+$/i, "");
+    const candidates = [raw, withoutKind, chatOnly]
+        .map(normalizeChatKey)
+        .filter(Boolean);
+    return [...new Set(candidates)];
+}
 /**
  * Read scope for the account, checked before any history call.
  *
@@ -259,7 +277,8 @@ function isChatReadable(target, readChats) {
     // writes) or held `*`. The deny is unconditional on purpose: no deployment
     // has a reason to let the agent read its own login codes, and a config entry
     // that enabled it would be an account-takeover switch.
-    if (normalizeChatKey(target) === constants_1.TELEGRAM_SERVICE_CHAT_ID)
+    const candidates = chatKeyCandidates(target);
+    if (candidates.includes(constants_1.TELEGRAM_SERVICE_CHAT_ID))
         return false;
     if (readChats === undefined || readChats === null)
         return true;
@@ -272,7 +291,7 @@ function isChatReadable(target, readChats) {
         return false;
     if (entries.includes("*"))
         return true;
-    return entries.includes(normalizeChatKey(target));
+    return candidates.some((candidate) => entries.includes(candidate));
 }
 function isWithinWindow(timestamp, since, until) {
     // A message without a date cannot be placed in the window. Keeping it only
@@ -289,13 +308,26 @@ function readSender(msg, key) {
     const value = msg?.sender?.[key] ?? msg?._sender?.[key];
     return typeof value === "string" && value.length > 0 ? value : undefined;
 }
+/**
+ * Хэндл отправителя — через `resolveActiveUsername`, а не с сырого поля.
+ *
+ * Telegram оставляет `username` пустым, как только у аккаунта больше одного
+ * хэндла: они переезжают в `usernames[]`. Разбор участников в этом же файле
+ * так и делает и объясняет почему, а разбор истории читал сырое поле — то есть
+ * терял хэндл ровно у владельца, у которого хэндлов несколько. Сводки
+ * подписывали его сообщения голым id, пока у всех остальных стоял `@`, а
+ * `allowFrom`, пересобранный из `read`, терял того же человека (A6-04).
+ */
+function senderHandle(msg) {
+    return (0, helpers_1.resolveActiveUsername)(msg?.sender ?? msg?._sender);
+}
 function resolveSenderDisplay(msg) {
     const first = readSender(msg, "firstName");
     const last = readSender(msg, "lastName");
     const joined = [first, last].filter(Boolean).join(" ").trim();
     if (joined)
         return joined;
-    return readSender(msg, "title") ?? readSender(msg, "username");
+    return readSender(msg, "title") ?? senderHandle(msg);
 }
 /**
  * Same rule as the inbound path (`normalize.ts`): the text is the evidence,
@@ -312,7 +344,7 @@ function resolveHistoryReplyQuote(msg) {
  * into a prompt.
  */
 function normalizeHistoryMessage(msg, fallbackChatId) {
-    const messageId = toStringId(msg?.id);
+    const messageId = (0, normalize_1.toStringId)(msg?.id);
     if (!messageId)
         return null;
     const text = typeof msg?.message === "string" ? msg.message :
@@ -328,16 +360,16 @@ function normalizeHistoryMessage(msg, fallbackChatId) {
     }
     return {
         messageId,
-        chatId: toStringId(msg?.chatId) ?? fallbackChatId,
-        senderId: toStringId(msg?.senderId) ?? toStringId(msg?.fromId?.userId) ?? toStringId(msg?.fromId?.channelId),
-        senderUsername: readSender(msg, "username"),
+        chatId: (0, normalize_1.toStringId)(msg?.chatId) ?? fallbackChatId,
+        senderId: (0, normalize_1.toStringId)(msg?.senderId) ?? (0, normalize_1.toStringId)(msg?.fromId?.userId) ?? (0, normalize_1.toStringId)(msg?.fromId?.channelId),
+        senderUsername: senderHandle(msg),
         senderDisplay: resolveSenderDisplay(msg),
         text,
         timestamp,
         sentAt: timestamp === undefined ? undefined : new Date(timestamp * 1000).toISOString(),
-        replyToMessageId: toStringId(msg?.replyTo?.replyToMsgId) ?? toStringId(msg?.replyToMsgId),
+        replyToMessageId: (0, normalize_1.toStringId)(msg?.replyTo?.replyToMsgId) ?? (0, normalize_1.toStringId)(msg?.replyToMsgId),
         replyQuoteText: resolveHistoryReplyQuote(msg),
-        messageThreadId: toStringId(msg?.replyTo?.replyToTopId) ?? toStringId(msg?.replyToTopId),
+        messageThreadId: (0, normalize_1.toStringId)(msg?.replyTo?.replyToTopId) ?? (0, normalize_1.toStringId)(msg?.replyToTopId),
         media: (0, media_1.describeMedia)(msg?.media),
         isOutgoing: msg?.out === true,
     };

package/dist/html-render.js

@@ -33,6 +33,7 @@
  * `<a href="…">` working.
  */
 Object.defineProperty(exports, "__esModule", { value: true });
+exports.indexOfClosingTag = indexOfClosingTag;
 exports.renderTelegramHtml = renderTelegramHtml;
 /** Tags GramJS's HTML parser turns into entities, plus aliases it does not
  * know mapped onto ones it does. `canonical` is what gets emitted; `attrs`
@@ -73,6 +74,23 @@ const ATTR_RE = /([a-zA-Z-]+)(?:\s*=\s*(?:"([^"]*)"|'([^']*)'|([^\s>]+)))?/g;
 /** A character reference the parser will decode; everything else is a bare `&`. */
 const ENTITY_RE = /^&(?:#\d{1,7}|#[xX][0-9a-fA-F]{1,6}|[a-zA-Z][a-zA-Z0-9]{1,31});/;
 const WORD_RE = /[\p{L}\p{N}_]/u;
+/**
+ * Позиция закрывающего тега в ИСХОДНОЙ строке, без учёта регистра.
+ *
+ * `indexOf` по `s.toLowerCase()` возвращает смещение в другой строке.
+ * Сравниваем посимвольно в оригинале — тогда позиция всегда его собственная.
+ */
+function indexOfClosingTag(s, name, from) {
+    const tag = `</${name}>`;
+    const len = tag.length;
+    for (let k = from; k + len <= s.length; k += 1) {
+        if (s[k] !== "<")
+            continue;
+        if (s.slice(k, k + len).toLowerCase() === tag)
+            return k;
+    }
+    return -1;
+}
 /** GFM backslash-escapable punctuation, so `\*` means a literal asterisk. */
 const ESCAPABLE = new Set([..."\\`*_{}[]()#+-.!|~<>"]);
 const EMPHASIS = {
@@ -269,6 +287,8 @@ function tryEmphasis(s, i) {
 function renderInline(s) {
     let out = "";
     let i = 0;
+    // Сколько брошенных `tg-emoji` ждут своего закрывающего тега.
+    let droppedEmojiDepth = 0;
     while (i < s.length) {
         const c = s[i];
         if (c === "\\" && i + 1 < s.length && ESCAPABLE.has(s[i + 1])) {
@@ -291,14 +311,37 @@ function renderInline(s) {
                 const spec = TELEGRAM_TAGS[name];
                 if (spec) {
                     if (closing) {
+                        // Закрывающий тег брошенного `tg-emoji` тоже не выпускаем: иначе
+                        // в тексте останется `</tg-emoji>` без пары.
+                        if (spec.canonical === "tg-emoji" && droppedEmojiDepth > 0) {
+                            droppedEmojiDepth -= 1;
+                            i += m[0].length;
+                            continue;
+                        }
                         out += `</${spec.canonical}>`;
                         i += m[0].length;
                         continue;
                     }
+                    // `tg-emoji` без пригодного `emoji-id` выбрасывается целиком, а
+                    // текст внутри остаётся. Проверено на самом парсере GramJS: голый
+                    // `<tg-emoji>` не падает при разборе, а даёт сущность
+                    // `MessageEntityCustomEmoji` с `documentId: undefined` и нулевой
+                    // длиной — то есть ломается всё сообщение, а не один значок
+                    // (находка A5-18).
+                    if (spec.canonical === "tg-emoji" && !/^\d+$/.test(parseAttrs(m[3]).get("emoji-id") ?? "")) {
+                        droppedEmojiDepth += 1;
+                        i += m[0].length;
+                        continue;
+                    }
                     if (spec.canonical === "code" || spec.canonical === "pre") {
                         // Code bodies pass through untouched by markdown: `**` inside
                         // <code> is content, not emphasis.
-                        const close = s.toLowerCase().indexOf(`</${name}>`, i + m[0].length);
+                        // Искать в `s.toLowerCase()`, а применять смещение к `s` нельзя:
+                        // понижение регистра не сохраняет длину. `İ` (U+0130) даёт два
+                        // кода, поэтому каждая такая буква перед блоком сдвигала позицию
+                        // на единицу: тело кода обрезалось, а продолжение начиналось
+                        // внутри `</code>` и выпускало наружу `<` и `/` (находка A6-13).
+                        const close = indexOfClosingTag(s, name, i + m[0].length);
                         if (close !== -1) {
                             out += buildOpenTag(spec.canonical, spec.attrs, parseAttrs(m[3]))
                                 + escapeKeepEntities(s.slice(i + m[0].length, close))

package/dist/joins.js

@@ -1,6 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.JOINS_JOURNAL_MAX_RECORDS = exports.JOINS_MAX_LIMIT = exports.JOINS_DEFAULT_LIMIT = void 0;
+exports.JOINS_MAX_LIMIT = exports.JOINS_DEFAULT_LIMIT = void 0;
 exports.parseJoinEvent = parseJoinEvent;
 exports.resolveJoinsJournalPath = resolveJoinsJournalPath;
 exports.readJoinRecords = readJoinRecords;
@@ -17,13 +17,15 @@ exports.parseJoinsParams = parseJoinsParams;
  * business and would turn the journal into surveillance.
  */
 const node_fs_1 = require("node:fs");
-const node_os_1 = require("node:os");
+const state_dir_1 = require("./state-dir");
 const node_path_1 = require("node:path");
 const normalize_js_1 = require("./normalize.js");
 exports.JOINS_DEFAULT_LIMIT = 50;
 exports.JOINS_MAX_LIMIT = 500;
 /** The journal answers "recently", not "since the beginning of time". */
-exports.JOINS_JOURNAL_MAX_RECORDS = 2000;
+// Модульная константа: снаружи её никто не читает, а `export` обещает
+// публичную поверхность, которой нет (A6-21).
+const JOINS_JOURNAL_MAX_RECORDS = 2000;
 function chatIdOf(rawMessage) {
     return (0, normalize_js_1.toStringId)(rawMessage?.chatId) ??
         (0, normalize_js_1.toPeerChannelId)(rawMessage?.peerId?.channelId) ??
@@ -89,7 +91,7 @@ function resolveJoinsJournalPath(accountCfg, accountId) {
         return configured.trim();
     }
     const safeAccount = accountId.replace(/[^A-Za-z0-9._-]/g, "_");
-    return (0, node_path_1.join)((0, node_os_1.homedir)(), ".openclaw", "state", "clawgram", `joins-${safeAccount}.jsonl`);
+    return (0, node_path_1.join)((0, state_dir_1.resolveStateDir)(), "state", "clawgram", `joins-${safeAccount}.jsonl`);
 }
 function readJoinRecords(path) {
     if (!(0, node_fs_1.existsSync)(path))
@@ -115,8 +117,8 @@ function appendJoinRecord(path, event) {
     (0, node_fs_1.mkdirSync)((0, node_path_1.dirname)(path), { recursive: true });
     (0, node_fs_1.appendFileSync)(path, JSON.stringify(event) + "\n");
     const records = readJoinRecords(path);
-    if (records.length > exports.JOINS_JOURNAL_MAX_RECORDS) {
-        const kept = records.slice(records.length - exports.JOINS_JOURNAL_MAX_RECORDS);
+    if (records.length > JOINS_JOURNAL_MAX_RECORDS) {
+        const kept = records.slice(records.length - JOINS_JOURNAL_MAX_RECORDS);
         (0, node_fs_1.writeFileSync)(path, kept.map((entry) => JSON.stringify(entry)).join("\n") + "\n");
     }
 }

package/dist/manage.js

@@ -33,23 +33,17 @@ exports.summarizeMissingInvitees = summarizeMissingInvitees;
 exports.readInviteLink = readInviteLink;
 const history_1 = require("./history");
 const helpers_1 = require("./helpers");
-function readString(value) {
-    if (typeof value !== "string") {
-        return undefined;
-    }
-    const trimmed = value.trim();
-    return trimmed === "" ? undefined : trimmed;
-}
+const util_1 = require("./util");
 /** A user reference: `@username` or a numeric id, as a trimmed string. */
 function readUserRef(value) {
     if (typeof value === "number" && Number.isFinite(value)) {
         return String(Math.trunc(value));
     }
-    return readString(value);
+    return (0, util_1.readString)(value);
 }
 function readTarget(params, toolContext, action) {
     const raw = (0, helpers_1.readChatTargetParam)(params, toolContext);
-    const target = readString(raw);
+    const target = (0, util_1.readString)(raw);
     if (!target) {
         throw new Error(`clawgram: ${action} requires a chatId`);
     }
@@ -74,13 +68,13 @@ function readFlag(value) {
     return value === true || value === "true";
 }
 function parseCreateGroupParams(params) {
-    const title = readString(params.title ?? params.name);
+    const title = (0, util_1.readString)(params.title ?? params.name);
     if (!title) {
         throw new Error("clawgram: createGroup requires a title");
     }
     return {
         title,
-        about: readString(params.about ?? params.description),
+        about: (0, util_1.readString)(params.about ?? params.description),
         users: readUserList(params),
     };
 }
@@ -150,7 +144,7 @@ function parsePromoteAdminParams(params, toolContext) {
         user,
         isAdmin: true,
         rights,
-        rank: readString(params.rank ?? params.title),
+        rank: (0, util_1.readString)(params.rank ?? params.title),
     };
 }
 function parseDemoteAdminParams(params, toolContext) {
@@ -203,7 +197,7 @@ function parseInviteLinkParams(params, toolContext) {
         target: readTarget(params, toolContext, "inviteLink"),
         expireDate: parseExpireDate(params.expireDate ?? params.expiresAt),
         usageLimit: parseUsageLimit(params.usageLimit ?? params.memberLimit),
-        title: readString(params.title ?? params.label),
+        title: (0, util_1.readString)(params.title ?? params.label),
         requestNeeded: readFlag(params.requestNeeded ?? params.requireApproval),
     };
 }
@@ -231,7 +225,7 @@ function isChatManageable(target, manageChats) {
     if (entries.includes("*")) {
         return true;
     }
-    return entries.includes((0, history_1.normalizeChatKey)(target));
+    return (0, history_1.chatKeyCandidates)(target).some((candidate) => entries.includes(candidate));
 }
 /**
  * The id of the supergroup a `channels.CreateChannel` call just created, read
@@ -284,5 +278,5 @@ function summarizeMissingInvitees(result) {
 }
 /** The link out of a `TypeExportedChatInvite`, or nothing when there is none. */
 function readInviteLink(result) {
-    return readString(result?.link);
+    return (0, util_1.readString)(result?.link);
 }

package/dist/media.js

@@ -24,21 +24,12 @@ exports.isLocalMediaPath = isLocalMediaPath;
 exports.assertLocalMediaWithinRoots = assertLocalMediaWithinRoots;
 const node_fs_1 = require("node:fs");
 const node_path_1 = __importDefault(require("node:path"));
+const util_1 = require("./util");
 /**
  * GramJS carries numbers as `big-integer` objects as often as native numbers —
  * the same shape that once made `senderId` silently undefined. Anything that
  * stringifies to digits is accepted.
  */
-function readNumber(value) {
-    if (typeof value === "number") {
-        return Number.isFinite(value) ? value : undefined;
-    }
-    if (value === undefined || value === null) {
-        return undefined;
-    }
-    const parsed = Number(String(value));
-    return Number.isFinite(parsed) ? parsed : undefined;
-}
 function readAttributes(document) {
     const attributes = document?.attributes;
     return Array.isArray(attributes) ? attributes : [];
@@ -60,7 +51,7 @@ const SIMPLE_KINDS = {
  */
 function describeDocument(document) {
     const mimeType = typeof document?.mimeType === "string" ? document.mimeType : undefined;
-    const size = readNumber(document?.size);
+    const size = (0, util_1.readNumber)(document?.size);
     const fileName = findAttribute(document, "DocumentAttributeFilename")?.fileName;
     const base = {
         kind: "document",
@@ -74,7 +65,7 @@ function describeDocument(document) {
     }
     const audio = findAttribute(document, "DocumentAttributeAudio");
     if (audio) {
-        const duration = readNumber(audio.duration);
+        const duration = (0, util_1.readNumber)(audio.duration);
         return {
             ...base,
             kind: audio.voice === true ? "voice" : "audio",
@@ -83,7 +74,7 @@ function describeDocument(document) {
     }
     const video = findAttribute(document, "DocumentAttributeVideo");
     if (video) {
-        const duration = readNumber(video.duration);
+        const duration = (0, util_1.readNumber)(video.duration);
         return {
             ...base,
             kind: "video",
@@ -186,12 +177,20 @@ async function downloadMessageMediaToFile(params) {
     if (!buffer || !(buffer instanceof Buffer) || buffer.length === 0) {
         return undefined;
     }
-    const { mkdir, writeFile } = await import("node:fs/promises");
+    const { mkdir, writeFile, chmod } = await import("node:fs/promises");
     const { join } = await import("node:path");
-    await mkdir(params.dir, { recursive: true });
+    // Личная переписка на диске: каталог и файл принадлежат только агенту.
+    // По умолчанию (umask 022) выходило 0755/0644, то есть вложения из личных
+    // чатов читал любой локальный пользователь — на этом же хосте живёт
+    // gitlab-runner (A5-13). `mode` у mkdir и writeFile маскируется umask,
+    // поэтому права выставляются отдельным chmod, как это уже делается для
+    // конфига в update-config.ts.
+    await mkdir(params.dir, { recursive: true, mode: 0o700 });
+    await chmod(params.dir, 0o700).catch(() => undefined);
     const extension = extensionFor(described, understanding);
     const path = join(params.dir, params.fileNameFor({ media: described, extension }));
-    await writeFile(path, buffer);
+    await writeFile(path, buffer, { mode: 0o600 });
+    await chmod(path, 0o600).catch(() => undefined);
     return { path, mimeType: described.mimeType, understanding, media: described };
 }
 /**

package/dist/normalize.js

@@ -9,7 +9,12 @@ function toStringId(value) {
     if (value === null || value === undefined)
         return undefined;
     try {
-        return String(value);
+        const text = String(value);
+        // `[object Object]` — это не id, а целый Peer, переданный вместо поля
+        // внутри него: строка выглядит правдоподобно и уезжает дальше как ключ.
+        // Guard жил только в history.ts, поэтому один и тот же peer у одного пути
+        // был «найден», а у другого «неизвестен» (находка A12-05).
+        return text && text !== "[object Object]" ? text : undefined;
     }
     catch {
         return undefined;