@cecwxf/wtt 0.1.12 → 0.1.14

package/dist/channel.js

@@ -244,6 +244,27 @@ function isP2PTopicId(topicId) {
     const type = topicTypeCache.get(topicId.trim());
     return type === "p2p";
 }
+function isDiscussionTopicMessage(raw, topicId) {
+    const topicType = String(raw.topic_type ?? "").trim().toLowerCase();
+    if (topicType === "discussion")
+        return true;
+    if (topicType === "p2p" || topicType === "broadcast" || topicType === "collaborative")
+        return false;
+    const cached = topicTypeCache.get(topicId.trim());
+    if (cached === "discussion")
+        return true;
+    if (cached === "p2p" || cached === "broadcast" || cached === "collaborative")
+        return false;
+    const metadata = parseInboundMetadata(raw);
+    const metadataType = String(metadata?.topic_type ?? metadata?.topicType ?? "").trim().toLowerCase();
+    if (metadataType === "discussion")
+        return true;
+    if (metadataType === "p2p" || metadataType === "broadcast" || metadataType === "collaborative")
+        return false;
+    // Heuristic fallback: most non-p2p group threads are discussion-like and
+    // should keep a local topic-memory file for retrieval.
+    return Boolean(topicId && !isLikelyP2PMessage(raw));
+}
 function isP2PE2EEnabled(account) {
     const raw = account.config.p2pE2EEnabled;
     return raw === undefined ? DEFAULT_P2P_E2E_ENABLED : raw !== false;
@@ -1156,9 +1177,9 @@ function compactDiscussionContent(raw) {
  * Read and parse the local topic memory file into DiscussionHistoryMessage[].
  * Returns empty array if the file doesn't exist or is malformed.
  *
- * File format (one entry = 2 lines):
- *   - [timestamp] senderType:senderId(displayName) id=xxx [reply_to=xxx]
- *     compact content text
+ * Compatible formats:
+ * - Legacy 2-line entries
+ * - Extended multi-line entries with text/media/reply_excerpt blocks
  */
 async function readLocalTopicMemory(params) {
     if (!params.topicId)
@@ -1173,25 +1194,71 @@ async function readLocalTopicMemory(params) {
     }
     const messages = [];
     const lines = raw.split("\n");
-    // Pattern: - [timestamp] type:senderId(displayName) id=xxx [reply_to=xxx]
-    const entryRe = /^- \[([^\]]*)\]\s+([\w]+):(\S+?)(?:\(([^)]*)\))?\s+id=(\S+?)(?:\s+reply_to=(\S+))?$/;
-    for (let i = 0; i < lines.length; i++) {
+    // Pattern: - [timestamp] type:senderId(displayName) id=xxx [reply_to=xxx] [...]
+    const entryRe = /^- \[([^\]]*)\]\s+([\w]+):(\S+?)(?:\(([^)]*)\))?\s+id=(\S+?)(?:\s+reply_to=(\S+))?(?:\s+.*)?$/;
+    let i = 0;
+    while (i < lines.length) {
         const match = lines[i].match(entryRe);
-        if (!match)
+        if (!match) {
+            i += 1;
             continue;
+        }
         const [, createdAt, senderType, senderId, displayName, id, replyTo] = match;
-        // Next line (indented) is the content
-        const contentLine = (i + 1 < lines.length && lines[i + 1].startsWith("  "))
-            ? lines[i + 1].slice(2).trim()
-            : "";
+        i += 1;
+        const bodyLines = [];
+        while (i < lines.length && !entryRe.test(lines[i])) {
+            if (lines[i].startsWith("  "))
+                bodyLines.push(lines[i].slice(2));
+            i += 1;
+        }
+        let content = "";
+        const mediaPaths = [];
+        const mediaUrls = [];
+        let replyExcerpt;
+        for (const row of bodyLines) {
+            const line = row.trim();
+            if (!line)
+                continue;
+            if (line.startsWith("text:")) {
+                content = line.slice("text:".length).trim();
+                continue;
+            }
+            if (line.startsWith("media_paths:")) {
+                const payload = line.slice("media_paths:".length).trim();
+                for (const part of payload.split("|").map((item) => item.trim()).filter(Boolean)) {
+                    mediaPaths.push(part);
+                }
+                continue;
+            }
+            if (line.startsWith("media_urls:")) {
+                const payload = line.slice("media_urls:".length).trim();
+                for (const part of payload.split("|").map((item) => item.trim()).filter(Boolean)) {
+                    mediaUrls.push(part);
+                }
+                continue;
+            }
+            if (line.startsWith("reply_excerpt:")) {
+                replyExcerpt = line.slice("reply_excerpt:".length).trim() || undefined;
+                continue;
+            }
+            if (!content) {
+                content = line;
+            }
+            else {
+                content += ` ${line}`;
+            }
+        }
         messages.push({
             id,
             senderId,
             senderDisplayName: displayName || undefined,
             senderType: senderType || undefined,
-            content: contentLine,
+            content: content.trim(),
             createdAt: createdAt || undefined,
             replyTo: replyTo || undefined,
+            mediaPaths,
+            mediaUrls,
+            replyExcerpt,
         });
     }
     return messages;
@@ -1257,7 +1324,9 @@ async function persistDiscussionTopicMemory(params) {
         const path = joinPath(dir, `topic_id_${params.topicId}.md`);
         const lines = [];
         lines.push(`# topic_id_${params.topicId}`);
-        lines.push("");
+        if (params.topicName?.trim()) {
+            lines.push(`topic_name: ${params.topicName.trim()}`);
+        }
         lines.push(`updated_at: ${new Date().toISOString()}`);
         lines.push("");
         for (const msg of params.messages) {
@@ -1265,9 +1334,17 @@ async function persistDiscussionTopicMemory(params) {
             const nameTag = msg.senderDisplayName ? `(${msg.senderDisplayName})` : "";
             const who = `${msg.senderType || "unknown"}:${msg.senderId}${nameTag}`;
             const content = compactDiscussionContent(msg.content).replace(/\n/g, " ").trim();
-            lines.push(`- [${ts}] ${who} id=${msg.id}${msg.replyTo ? ` reply_to=${msg.replyTo}` : ""}`);
-            if (content) {
-                lines.push(`  ${content}`);
+            const mediaCount = (msg.mediaPaths?.length ?? 0) + (msg.mediaUrls?.length ?? 0);
+            lines.push(`- [${ts}] ${who} id=${msg.id}${msg.replyTo ? ` reply_to=${msg.replyTo}` : ""}${mediaCount > 0 ? ` media_count=${mediaCount}` : ""}`);
+            lines.push(`  text: ${content}`);
+            if (msg.mediaPaths && msg.mediaPaths.length > 0) {
+                lines.push(`  media_paths: ${msg.mediaPaths.join(" | ")}`);
+            }
+            if (msg.mediaUrls && msg.mediaUrls.length > 0) {
+                lines.push(`  media_urls: ${msg.mediaUrls.join(" | ")}`);
+            }
+            if (msg.replyExcerpt) {
+                lines.push(`  reply_excerpt: ${msg.replyExcerpt}`);
             }
         }
         await writeFile(path, `${lines.join("\n")}\n`, "utf8");
@@ -1299,22 +1376,45 @@ async function appendDiscussionTopicMemory(params) {
         const idMarker = `id=${params.msg.id}`;
         if (existing.includes(idMarker))
             return;
+        const history = existing ? await readLocalTopicMemory({ topicId: params.topicId }) : [];
+        const replyTarget = params.msg.replyTo
+            ? history.find((item) => item.id === params.msg.replyTo)
+            : undefined;
+        const replyExcerpt = replyTarget
+            ? compactDiscussionContent(replyTarget.content).slice(0, 220)
+            : undefined;
         const ts = params.msg.createdAt ?? new Date().toISOString();
         const nameTag = params.msg.senderDisplayName ? `(${params.msg.senderDisplayName})` : "";
         const who = `${params.msg.senderType || "unknown"}:${params.msg.senderId}${nameTag}`;
         const content = compactDiscussionContent(params.msg.content).replace(/\n/g, " ").trim();
-        let line = `- [${ts}] ${who} ${idMarker}${params.msg.replyTo ? ` reply_to=${params.msg.replyTo}` : ""}`;
-        if (content) {
-            line += `\n  ${content}`;
+        const mediaPaths = Array.from(new Set((params.msg.mediaPaths ?? []).map((v) => String(v || "").trim()).filter(Boolean)));
+        const mediaUrls = Array.from(new Set((params.msg.mediaUrls ?? []).map((v) => String(v || "").trim()).filter(Boolean)));
+        const mediaCount = mediaPaths.length + mediaUrls.length;
+        const lines = [];
+        lines.push(`- [${ts}] ${who} ${idMarker}${params.msg.replyTo ? ` reply_to=${params.msg.replyTo}` : ""}${mediaCount > 0 ? ` media_count=${mediaCount}` : ""}`);
+        lines.push(`  text: ${content}`);
+        if (mediaPaths.length > 0) {
+            lines.push(`  media_paths: ${mediaPaths.join(" | ")}`);
+        }
+        if (mediaUrls.length > 0) {
+            lines.push(`  media_urls: ${mediaUrls.join(" | ")}`);
+        }
+        if (replyExcerpt) {
+            lines.push(`  reply_excerpt: ${replyExcerpt}`);
         }
         if (!existing) {
             // Create file with header
-            const header = `# topic_id_${params.topicId}\n\nupdated_at: ${new Date().toISOString()}\n\n`;
-            await writeFile(path, header + line + "\n", "utf8");
+            const headerLines = [`# topic_id_${params.topicId}`];
+            if (params.topicName?.trim()) {
+                headerLines.push(`topic_name: ${params.topicName.trim()}`);
+            }
+            headerLines.push(`updated_at: ${new Date().toISOString()}`);
+            headerLines.push("");
+            await writeFile(path, `${headerLines.join("\n")}\n${lines.join("\n")}\n`, "utf8");
         }
         else {
             // Append to existing file
-            await writeFile(path, existing.trimEnd() + "\n" + line + "\n", "utf8");
+            await writeFile(path, `${existing.trimEnd()}\n${lines.join("\n")}\n`, "utf8");
         }
     }
     catch (err) {
@@ -1336,15 +1436,26 @@ function buildDiscussionContextBlock(params) {
             const targetName = target.senderDisplayName ? `(${target.senderDisplayName})` : "";
             lines.push(`[被回复消息] id=${target.id} sender=${target.senderId}${targetName}`);
             lines.push(compactDiscussionContent(target.content).slice(0, 1200));
+            if (target.mediaPaths && target.mediaPaths.length > 0) {
+                lines.push(`media_paths: ${target.mediaPaths.join(" | ")}`);
+            }
+            if (target.mediaUrls && target.mediaUrls.length > 0) {
+                lines.push(`media_urls: ${target.mediaUrls.join(" | ")}`);
+            }
             lines.push("");
         }
     }
     for (const msg of focus) {
         const compact = compactDiscussionContent(msg.content);
-        if (!compact)
+        const hasMedia = Boolean((msg.mediaPaths && msg.mediaPaths.length > 0) || (msg.mediaUrls && msg.mediaUrls.length > 0));
+        if (!compact && !hasMedia)
             continue;
         const nameTag = msg.senderDisplayName ? `(${msg.senderDisplayName})` : "";
-        lines.push(`- id=${msg.id} sender=${msg.senderId}${nameTag}${msg.replyTo ? ` reply_to=${msg.replyTo}` : ""}: ${compact}`);
+        const mediaTag = (msg.mediaPaths && msg.mediaPaths.length > 0)
+            ? ` media_paths=${msg.mediaPaths.join("|")}`
+            : ((msg.mediaUrls && msg.mediaUrls.length > 0) ? ` media_urls=${msg.mediaUrls.join("|")}` : "");
+        const replyExcerptTag = msg.replyExcerpt ? ` reply_excerpt=${msg.replyExcerpt}` : "";
+        lines.push(`- id=${msg.id} sender=${msg.senderId}${nameTag}${msg.replyTo ? ` reply_to=${msg.replyTo}` : ""}${mediaTag}${replyExcerptTag}: ${compact || "[media_message]"}`);
     }
     let block = lines.join("\n").trim();
     if (block.length > params.maxChars) {
@@ -1886,11 +1997,13 @@ export async function routeInboundWsMessage(params) {
     }
     const typingTopicId = normalized.topicId?.trim() || "";
     const mentionMatch = resolveMentionMatch(String(rawMsg.content ?? ""), params.account.agentId, params.account.name);
+    const originalMediaUrls = normalized.mediaUrls.slice();
+    const originalMediaTypes = normalized.mediaTypes.slice();
     // Remember recent media from this sender/topic even when the message itself
     // does not trigger inference (for example: image first, @mention second).
-    if (typingTopicId && normalized.mediaUrls.length > 0) {
-        rememberRecentTopicMedia(typingTopicId, normalized.senderId, normalized.mediaUrls, normalized.mediaTypes);
-        params.log?.("info", `[${params.accountId}] inbound media captured topic=${typingTopicId} sender=${normalized.senderId} count=${normalized.mediaUrls.length}`);
+    if (typingTopicId && originalMediaUrls.length > 0) {
+        rememberRecentTopicMedia(typingTopicId, normalized.senderId, originalMediaUrls, originalMediaTypes);
+        params.log?.("info", `[${params.accountId}] inbound media captured topic=${typingTopicId} sender=${normalized.senderId} count=${originalMediaUrls.length}`);
     }
     // If current @mention message carries no media, try to reuse sender's latest
     // media in the same topic so "image + @mention" split messages still work.
@@ -1915,24 +2028,6 @@ export async function routeInboundWsMessage(params) {
             params.log?.("info", `[${params.accountId}] inbound media cache_miss topic=${typingTopicId} sender=${normalized.senderId} raw_keys=${rawKeys} metadata_keys=${metadataKeys} content_preview=${contentPreview}`);
         }
     }
-    // Incrementally record every discussion message to the topic memory file.
-    // This runs BEFORE inference gating, so even messages that don't trigger
-    // inference (no @mention) are persisted for later context retrieval.
-    if (inferredTopicType === "discussion" && typingTopicId && normalized.text.trim()) {
-        appendDiscussionTopicMemory({
-            topicId: typingTopicId,
-            msg: {
-                id: normalized.messageId,
-                senderId: normalized.senderId,
-                senderDisplayName: normalized.senderName ?? toOptionalString(rawMsg.sender_display_name),
-                senderType: String(rawMsg.sender_type ?? "unknown"),
-                content: normalized.text,
-                createdAt: normalized.timestamp,
-                replyTo: toOptionalString(rawMsg.reply_to),
-            },
-            log: params.log,
-        }).catch(() => { });
-    }
     const emitTypingSignal = async (state) => {
         if (!typingTopicId || !params.typingSignal)
             return;
@@ -1995,12 +2090,43 @@ export async function routeInboundWsMessage(params) {
     const slashBypassMentionGate = params.account.config.slashBypassMentionGate ?? DEFAULT_SLASH_BYPASS_MENTION_GATE;
     const topicType = String(rawMsg.topic_type ?? "").toLowerCase();
     const topicName = String(rawMsg.topic_name ?? "");
+    const isDiscussionTopic = isDiscussionTopicMessage(rawMsg, typingTopicId);
     const isTaskLinkedTopic = Boolean(inboundTaskId || topicName.startsWith("TASK-"));
     const isSlashLike = /^\/\S+/.test((normalized.text || "").trim());
     const inboundMetadata = parseInboundMetadata(rawMsg);
-    const mentionTargetedDiscussion = topicType === "discussion" && mentionMatch.matchesAgent;
+    const mentionTargetedDiscussion = isDiscussionTopic && mentionMatch.matchesAgent;
+    // For discussion topics, keep local topic memory complete even when inference
+    // is not triggered. Download image media and index local paths/urls.
+    let indexedTopicMedia = { mediaPaths: [], mediaTypes: [] };
+    if (isDiscussionTopic && typingTopicId && (normalized.text.trim() || originalMediaUrls.length > 0)) {
+        if (originalMediaUrls.length > 0) {
+            indexedTopicMedia = await materializeInboundMediaForContext({
+                mediaUrls: originalMediaUrls,
+                mediaTypes: originalMediaTypes,
+                account: params.account,
+                accountId: params.accountId,
+                log: params.log,
+            });
+        }
+        await appendDiscussionTopicMemory({
+            topicId: typingTopicId,
+            topicName: normalized.topicName ?? topicName,
+            msg: {
+                id: normalized.messageId,
+                senderId: normalized.senderId,
+                senderDisplayName: normalized.senderName ?? toOptionalString(rawMsg.sender_display_name),
+                senderType: String(rawMsg.sender_type ?? "unknown"),
+                content: normalized.text,
+                createdAt: normalized.timestamp,
+                replyTo: toOptionalString(rawMsg.reply_to),
+                mediaPaths: indexedTopicMedia.mediaPaths,
+                mediaUrls: originalMediaUrls,
+            },
+            log: params.log,
+        });
+    }
     if (slashCompatEnabled) {
-        if (topicType === "discussion" && !isTaskLinkedTopic && isSlashLike) {
+        if (isDiscussionTopic && !isTaskLinkedTopic && isSlashLike) {
             const targetAgentId = toOptionalString(inboundMetadata?.command_target_agent_id)
                 ?? toOptionalString(inboundMetadata?.commandTargetAgentId);
             // For non-task discuss slash commands: execute only on explicitly targeted agent.
@@ -2054,13 +2180,16 @@ export async function routeInboundWsMessage(params) {
     }
     // Telegram-style media handling: download inbound media first, then pass
     // local file paths into MediaPath/MediaPaths for stable vision pipeline input.
-    const downloadedInboundMedia = await materializeInboundMediaForContext({
-        mediaUrls: normalized.mediaUrls,
-        mediaTypes: normalized.mediaTypes,
-        account: params.account,
-        accountId: params.accountId,
-        log: params.log,
-    });
+    // Reuse files already downloaded for topic-memory indexing when available.
+    const downloadedInboundMedia = indexedTopicMedia.mediaPaths.length > 0
+        ? indexedTopicMedia
+        : await materializeInboundMediaForContext({
+            mediaUrls: normalized.mediaUrls,
+            mediaTypes: normalized.mediaTypes,
+            account: params.account,
+            accountId: params.accountId,
+            log: params.log,
+        });
     const contextMediaPaths = downloadedInboundMedia.mediaPaths.length > 0
         ? downloadedInboundMedia.mediaPaths
         : normalized.mediaUrls;
@@ -2099,12 +2228,12 @@ export async function routeInboundWsMessage(params) {
         ?? toOptionalString(rawMsg.taskTitle)
         ?? toOptionalString(rawMsg.title)
         ?? (normalized.topicName && !normalized.topicName.startsWith("TASK-") ? normalized.topicName : undefined);
-    const mentionDirective = (topicType === "discussion" && mentionTargetedDiscussion)
+    const mentionDirective = (isDiscussionTopic && mentionTargetedDiscussion)
         ? "注意：这是讨论话题中明确 @ 你的消息，必须直接回应用户问题，禁止输出 NO_REPLY。\n\n"
         : "";
     const replyToId = toOptionalString(rawMsg.reply_to);
     let discussionContextBlock = "";
-    if (topicType === "discussion" && normalized.topicId && (mentionTargetedDiscussion || Boolean(replyToId))) {
+    if (isDiscussionTopic && normalized.topicId && (mentionTargetedDiscussion || Boolean(replyToId))) {
         const contextWindow = toPositiveInt(params.account.config.discussionContextWindow, DEFAULT_DISCUSSION_CONTEXT_WINDOW);
         const contextMaxChars = toPositiveInt(params.account.config.discussionContextMaxChars, DEFAULT_DISCUSSION_CONTEXT_MAX_CHARS);
         // Local-first: read from local memory file (kept up-to-date by incremental append).
@@ -2126,6 +2255,7 @@ export async function routeInboundWsMessage(params) {
             if (discussionMessages.length > 0) {
                 await persistDiscussionTopicMemory({
                     topicId: normalized.topicId,
+                    topicName: normalized.topicName,
                     messages: discussionMessages,
                     log: params.log,
                 });
@@ -2281,7 +2411,7 @@ export async function routeInboundWsMessage(params) {
     finally {
         await emitTypingSignal("stop");
     }
-    const shouldForceMentionAck = topicType === "discussion"
+    const shouldForceMentionAck = isDiscussionTopic
         && Boolean(inferDecision?.trigger)
         && (mentionTargetedDiscussion || inferDecision?.reason === "discussion_runner_match");
     if (!dispatchProducedOutput && shouldForceMentionAck) {