@geminixiang/mikan 0.5.0-beta.0 → 0.5.1

package/dist/adapters/slack/bot.js

@@ -6,12 +6,13 @@ import { basename, join } from "path";
 import { resolveConversationSettings } from "../../config.js";
 import * as log from "../../log.js";
 import { PRODUCT_NAME, formatForceStopped, formatNothingRunning } from "../../platform-messages.js";
-import { appendBotResponseLog, appendChannelLog, ChannelQueue, resolveOnlyScopedStopTarget, resolveStopTarget, withRetry, } from "../shared.js";
+import { appendBotResponseLog, appendChannelLog, MessagingEventQueue, resolveOnlyScopedStopTarget, resolveStopTarget, withRetry, } from "../shared.js";
 import { processMessageIntake } from "../intake.js";
 import { createSlackAdapters } from "./context.js";
-import { hasMaterializedChatSession, registerThreadSession, } from "../../sessions/chat-session-manager.js";
+import { hasMaterializedChatSession, registerThreadSession, } from "../../sessions/agent-memory-file-manager.js";
 import { isSlackThreadSessionKey, planSlackAdapterSession, planSlackEventAnchorRun, resolveSlackSessionKey, } from "./session.js";
 import { reportUserFacingError } from "../../observability/sentry.js";
+import { renderSlackBlocks } from "./blocks.js";
 const SLACK_EVENT_ANCHOR_TEXT = "Working on it...";
 // Slack WebClient errors carry either `code: "rate_limited"` (retry-after) or
 // the legacy `data.error === "rate_limited"` / 429 status shape.
@@ -69,10 +70,10 @@ export function buildMrkdwnContextBlock(text) {
     return { type: "context", elements: [{ type: "mrkdwn", text: blockText }] };
 }
 // ============================================================================
-// SlackBot
+// SlackMessagingBot
 // ============================================================================
-export class SlackBot {
-    createAdapters(event) {
+export class SlackMessagingBot {
+    createContext(event) {
         return createSlackAdapters(event, this, {
             replyMode: resolveConversationSettings(join(this.workingDir, event.conversationId)).slack?.replyMode ??
                 "top-level",
@@ -146,7 +147,8 @@ export class SlackBot {
     }
     async postMessage(channel, text) {
         return slackRetry(async () => {
-            const result = await this.webClient.chat.postMessage({ channel, text });
+            const payload = { channel, ...renderSlackBlocks(text) };
+            const result = await this.webClient.chat.postMessage(payload);
             return result.ts;
         });
     }
@@ -173,11 +175,8 @@ export class SlackBot {
     }
     async postMessageBlocks(channel, text, blocks) {
         return slackRetry(async () => {
-            const result = await this.webClient.chat.postMessage({
-                channel,
-                text,
-                blocks: blocks,
-            });
+            const payload = { channel, text, blocks: blocks };
+            const result = await this.webClient.chat.postMessage(payload);
             return result.ts;
         });
     }
@@ -203,7 +202,8 @@ export class SlackBot {
     }
     async updateMessage(channel, ts, text) {
         return slackRetry(async () => {
-            await this.webClient.chat.update({ channel, ts, text });
+            const payload = { channel, ts, ...renderSlackBlocks(text) };
+            await this.webClient.chat.update(payload);
         });
     }
     async startMessageStream(channel, text, threadTs, recipientUserId) {
@@ -255,42 +255,20 @@ export class SlackBot {
     }
     async postInThread(channel, threadTs, text) {
         return slackRetry(async () => {
-            // Use Block Kit section for long messages to trigger Slack's "Show more" collapsing (~700 chars)
-            const SECTION_TEXT_LIMIT = 3000;
-            if (text.length > 500) {
-                const blockText = text.length > SECTION_TEXT_LIMIT
-                    ? text.substring(0, SECTION_TEXT_LIMIT - 20) + "\n_(truncated)_"
-                    : text;
-                const result = await this.webClient.chat.postMessage({
-                    channel,
-                    thread_ts: threadTs,
-                    text, // full text as notification fallback
-                    blocks: [{ type: "section", text: { type: "mrkdwn", text: blockText } }],
-                });
-                return result.ts;
-            }
-            const result = await this.webClient.chat.postMessage({ channel, thread_ts: threadTs, text });
+            const payload = { channel, thread_ts: threadTs, ...renderSlackBlocks(text) };
+            const result = await this.webClient.chat.postMessage(payload);
             return result.ts;
         });
     }
     async postInThreadBlocks(channel, threadTs, text, blocks) {
         return slackRetry(async () => {
-            const result = await this.webClient.chat.postMessage({
+            const payload = {
                 channel,
                 thread_ts: threadTs,
                 text, // fallback for notifications
                 blocks: blocks,
-            });
-            return result.ts;
-        });
-    }
-    async postBlocks(channel, text, blocks) {
-        return slackRetry(async () => {
-            const result = await this.webClient.chat.postMessage({
-                channel,
-                text,
-                blocks: blocks,
-            });
+            };
+            const result = await this.webClient.chat.postMessage(payload);
             return result.ts;
         });
     }
@@ -316,7 +294,7 @@ export class SlackBot {
             ...(slackBlocks ? { slackBlocks } : {}),
         });
     }
-    getPlatformInfo() {
+    getMessagingInfo() {
         return {
             name: "slack",
             formattingGuide: "## Slack Formatting (mrkdwn, NOT Markdown)\nBold: *text*, Italic: _text_, Code: `code`, Block: ```code```, Links: <url|text>\nDo NOT use **double asterisks** or [markdown](links).",
@@ -397,12 +375,12 @@ export class SlackBot {
                     })),
                     sessionKey: eventForRun.sessionKey,
                 };
-                const adapters = createSlackAdapters(slackEvent, this, {
+                const context = createSlackAdapters(slackEvent, this, {
                     initialMessageTs: eventPlan.initialMessageTs,
                     replyMode: resolveConversationSettings(join(this.workingDir, eventForRun.conversationId)).slack
                         ?.replyMode ?? "top-level",
                 });
-                return this.handler.handleEvent(eventForRun, this, adapters);
+                return this.handler.handleEvent(eventForRun, this, context);
             });
         });
         return true;
@@ -413,7 +391,7 @@ export class SlackBot {
     getQueue(channelId) {
         let queue = this.queues.get(channelId);
         if (!queue) {
-            queue = new ChannelQueue("Slack");
+            queue = new MessagingEventQueue("Slack");
             this.queues.set(channelId, queue);
         }
         return queue;
@@ -442,7 +420,7 @@ export class SlackBot {
             enqueue: (queueKey, work) => this.getQueue(queueKey).enqueue(work),
             handler: this.handler,
             bot: this,
-            createAdapters: (event) => this.createAdapters(event),
+            createContext: (event) => this.createContext(event),
             onNotTriggered: options.onNotTriggered,
             deferAttachmentsUntilRun: true,
         });
@@ -624,7 +602,7 @@ export class SlackBot {
             const messageTs = await this.postMessageBlocks(conversationId, responseText, blocks);
             this.logBotResponse(conversationId, responseText, messageTs);
         };
-        const responseCtx = {
+        const responder = {
             respond,
             replaceResponse: respond,
             respondDiagnostic: async (responseText, responseOptions) => {
@@ -647,8 +625,8 @@ export class SlackBot {
         };
         return {
             message,
-            responseCtx,
-            platform: this.getPlatformInfo(),
+            responder,
+            platform: this.getMessagingInfo(),
         };
     }
     buildSlashCommandEvent(payload, options = {}) {
@@ -670,7 +648,7 @@ export class SlackBot {
             userName,
             text: commandText,
             attachments: [],
-            isBot: false,
+            isMessagingBot: false,
             ...(threadTs ? { threadTs } : {}),
         });
         const event = {
@@ -684,15 +662,15 @@ export class SlackBot {
             ...(threadTs ? { thread_ts: threadTs } : {}),
             sessionKey,
         };
-        const adapters = this.createCommandAdapters(conversationId, payload.user_id, userName, commandText, eventTs, isDirectMessage ? { threadTs } : { ephemeralChannelId: conversationId, threadTs });
-        return { event, adapters };
+        const context = this.createCommandAdapters(conversationId, payload.user_id, userName, commandText, eventTs, isDirectMessage ? { threadTs } : { ephemeralChannelId: conversationId, threadTs });
+        return { event, context };
     }
     async routeSlashLoginCommand(payload) {
-        const { event, adapters } = this.buildSlashCommandEvent(payload, {
+        const { event, context } = this.buildSlashCommandEvent(payload, {
             type: payload.channel_id.startsWith("D") ? "dm" : "private_command",
             includeText: true,
         });
-        await this.handler.handleEvent(event, this, adapters);
+        await this.handler.handleEvent(event, this, context);
     }
     async routeSlashNewCommand(payload) {
         const conversationId = payload.channel_id;
@@ -710,28 +688,28 @@ export class SlackBot {
             userName,
             text: payload.command,
             attachments: [],
-            isBot: false,
+            isMessagingBot: false,
         });
-        const commandBot = {
+        const commandMessagingBot = {
             start: async () => { },
             postMessage: async (_channel, text) => this.postMessage(conversationId, text),
             updateMessage: async (channel, ts, text) => this.updateMessage(channel, ts, text),
             enqueueEvent: (event) => this.enqueueEvent(event),
-            getPlatformInfo: () => this.getPlatformInfo(),
+            getMessagingInfo: () => this.getMessagingInfo(),
         };
-        await this.handler.handleNewCommand(conversationId, conversationId, commandBot);
+        await this.handler.handleNewCommand(conversationId, conversationId, commandMessagingBot);
     }
     async routeSlashModelCommand(payload) {
-        const { event, adapters } = this.buildSlashCommandEvent(payload, { includeText: true });
-        await this.handler.handleEvent(event, this, adapters);
+        const { event, context } = this.buildSlashCommandEvent(payload, { includeText: true });
+        await this.handler.handleEvent(event, this, context);
     }
     async routeSlashSessionCommand(payload) {
-        const { event, adapters } = this.buildSlashCommandEvent(payload, { thread: true });
-        await this.handler.handleEvent(event, this, adapters);
+        const { event, context } = this.buildSlashCommandEvent(payload, { thread: true });
+        await this.handler.handleEvent(event, this, context);
     }
     async routeSlashAdminCommand(payload) {
-        const { event, adapters } = this.buildSlashCommandEvent(payload, { thread: true });
-        await this.handler.handleEvent(event, this, adapters);
+        const { event, context } = this.buildSlashCommandEvent(payload, { thread: true });
+        await this.handler.handleEvent(event, this, context);
     }
     setupEventHandlers() {
         this.socketClient.on("disconnect", (err) => {
@@ -810,13 +788,13 @@ export class SlackBot {
         const e = event;
         const hasFiles = !!e.files && e.files.length > 0;
         const hasSlackContent = !!e.text || hasFiles || !!e.blocks?.length || !!e.attachments?.length;
-        const isOwnBotMessage = (!!e.user && e.user === this.botUserId) || (!!this.botId && e.bot_id === this.botId);
-        if (isOwnBotMessage) {
+        const isOwnMessagingBotMessage = (!!e.user && e.user === this.botUserId) || (!!this.botId && e.bot_id === this.botId);
+        if (isOwnMessagingBotMessage) {
             ack();
             return;
         }
-        const isExternalBotMessage = !!e.bot_id || e.subtype === "bot_message";
-        if (isExternalBotMessage) {
+        const isExternalMessagingBotMessage = !!e.bot_id || e.subtype === "bot_message";
+        if (isExternalMessagingBotMessage) {
             if (e.subtype !== undefined && e.subtype !== "bot_message" && e.subtype !== "file_share") {
                 ack();
                 return;
@@ -825,7 +803,7 @@ export class SlackBot {
                 ack();
                 return;
             }
-            void this.logExternalBotMessage(e).catch((err) => {
+            void this.logExternalMessagingBotMessage(e).catch((err) => {
                 log.logWarning("Failed to log Slack bot message", String(err));
             });
             ack();
@@ -845,9 +823,9 @@ export class SlackBot {
         }
         const isDM = e.channel_type === "im";
         const conversationKind = isDM ? "direct" : "shared";
-        const isBotMention = e.text?.includes(`<@${this.botUserId}>`);
+        const isMessagingBotMention = e.text?.includes(`<@${this.botUserId}>`);
         // Skip channel @mentions - already handled by app_mention event
-        if (!isDM && isBotMention) {
+        if (!isDM && isMessagingBotMention) {
             ack();
             return;
         }
@@ -1067,7 +1045,7 @@ export class SlackBot {
             userName: body.user?.username ?? body.user?.name,
             text,
             attachments: [],
-            isBot: false,
+            isMessagingBot: false,
             platform: "slack",
             slackInteraction: {
                 type: "block_actions",
@@ -1109,7 +1087,7 @@ export class SlackBot {
                 attachments: [],
                 sessionKey,
             };
-            return this.handler.handleEvent(event, this, this.createAdapters(slackEvent));
+            return this.handler.handleEvent(event, this, this.createContext(slackEvent));
         });
     }
     /**
@@ -1138,13 +1116,13 @@ export class SlackBot {
             displayName: user?.displayName,
             text: event.text,
             attachments,
-            isBot: false,
+            isMessagingBot: false,
         });
         if (attachmentError)
             throw attachmentError;
         return attachments;
     }
-    async logExternalBotMessage(event) {
+    async logExternalMessagingBotMessage(event) {
         const attachments = event.files
             ? await this.store.processAttachments(event.channel, event.files, event.ts)
             : [];
@@ -1158,7 +1136,7 @@ export class SlackBot {
             displayName: botName,
             text: buildSlackAppMessageText(event),
             attachments,
-            isBot: true,
+            isMessagingBot: true,
             botId: event.bot_id,
             appId: event.app_id ?? event.bot_profile?.app_id,
             subtype: event.subtype,
@@ -1220,8 +1198,8 @@ export class SlackBot {
                 return false; // Skip duplicates
             if (msg.user === this.botUserId)
                 return true;
-            const isExternalBotMessage = !!msg.bot_id || msg.subtype === "bot_message";
-            if (isExternalBotMessage) {
+            const isExternalMessagingBotMessage = !!msg.bot_id || msg.subtype === "bot_message";
+            if (isExternalMessagingBotMessage) {
                 if (this.botId && msg.bot_id === this.botId)
                     return false;
                 if (msg.subtype !== undefined &&
@@ -1247,9 +1225,9 @@ export class SlackBot {
         // Log each message to log.jsonl
         for (const msg of relevantMessages) {
             const isMikanMessage = msg.user === this.botUserId;
-            const isExternalBotMessage = !isMikanMessage && (!!msg.bot_id || msg.subtype === "bot_message");
-            if (isExternalBotMessage) {
-                await this.logExternalBotMessage({ ...msg, channel: channelId, ts: msg.ts });
+            const isExternalMessagingBotMessage = !isMikanMessage && (!!msg.bot_id || msg.subtype === "bot_message");
+            if (isExternalMessagingBotMessage) {
+                await this.logExternalMessagingBotMessage({ ...msg, channel: channelId, ts: msg.ts });
                 continue;
             }
             const user = this.users.get(msg.user);
@@ -1266,7 +1244,7 @@ export class SlackBot {
                 displayName: isMikanMessage ? undefined : user?.displayName,
                 text,
                 attachments,
-                isBot: isMikanMessage,
+                isMessagingBot: isMikanMessage,
             });
         }
         return relevantMessages.length;