@geminixiang/mikan 1.0.0-beta.3 → 1.0.0-beta.30

package/dist/adapters/slack/bot.js

@@ -1,22 +1,28 @@
 import { SocketModeClient } from "@slack/socket-mode";
-import { WebClient } from "@slack/web-api";
+import { WebAPIRateLimitedError, WebClient } from "@slack/web-api";
 import { existsSync, readFileSync } from "fs";
 import { readFile } from "fs/promises";
 import { basename, join } from "path";
+import { COMMAND_MANIFEST } from "../../commands/manifest.js";
 import { resolveConversationSettings } from "../../config.js";
+import { parseExtActionId } from "../../harness/extensions/blockkit.js";
 import * as log from "../../log.js";
-import { PRODUCT_NAME, formatForceStopped, formatNothingRunning } from "../../platform-messages.js";
-import { appendBotResponseLog, appendChannelLog, MessagingEventQueue, resolveOnlyScopedStopTarget, resolveStopTarget, withRetry, } from "../shared.js";
+import { isRecord, readTextFileIfExists } from "../../utils/file-guards.js";
+import { PRODUCT_NAME, formatForceStopped } from "../../platform-messages.js";
+import { appendBotResponseLog, appendChannelLog, MessagingEventQueue, withRetry, } from "../shared.js";
 import { processMessageIntake } from "../intake.js";
 import { createSlackAdapters } from "./context.js";
-import { hasMaterializedChatSession, registerThreadSession, } from "../../sessions/agent-memory-file-manager.js";
+import { hasMaterializedChatSession, registerThreadSession, } from "../../sessions/chat-history-sync.js";
+import { conversationIdOf } from "../../sessions/session-key.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.
+// web-api v8 throws WebAPIRateLimitedError; the duck-typed shapes cover
+// platform `data.error === "rate_limited"` / 429 responses and older callers.
 function slackIsRateLimited(err) {
+    if (err instanceof WebAPIRateLimitedError)
+        return true;
     if (err.code === "rate_limited")
         return true;
     const data = err.data;
@@ -152,6 +158,21 @@ export class SlackMessagingBot {
             return result.ts;
         });
     }
+    async addReaction(channel, messageTs, emoji) {
+        // Slack reaction names are colon-free short names; strip any wrapping colons.
+        const name = emoji.replace(/^:|:$/g, "");
+        await slackRetry(async () => {
+            try {
+                await this.webClient.reactions.add({ channel, timestamp: messageTs, name });
+            }
+            catch (err) {
+                // Re-reacting with the same emoji is not an error worth surfacing.
+                if (err?.data?.error === "already_reacted")
+                    return;
+                throw err;
+            }
+        });
+    }
     async postEphemeral(channel, user, text, threadTs) {
         return slackRetry(async () => {
             await this.webClient.chat.postEphemeral({
@@ -180,6 +201,11 @@ export class SlackMessagingBot {
             return result.ts;
         });
     }
+    async updateMessageBlocks(channel, ts, text, blocks) {
+        return slackRetry(async () => {
+            await this.webClient.chat.update({ channel, ts, text, blocks: blocks });
+        });
+    }
     async postPrivate(conversationId, userId, text) {
         await this.postEphemeral(conversationId, userId, text);
     }
@@ -294,10 +320,31 @@ export class SlackMessagingBot {
             ...(slackBlocks ? { slackBlocks } : {}),
         });
     }
+    ownsBlockKitMessage(channel, ts, threadTs) {
+        const content = readTextFileIfExists(join(this.workingDir, channel, "log.jsonl"));
+        if (content === undefined)
+            return false;
+        for (const line of content.trim().split("\n").toReversed()) {
+            try {
+                const entry = JSON.parse(line);
+                if (!isRecord(entry) || entry.ts !== ts)
+                    continue;
+                return (entry.isMessagingBot === true &&
+                    entry.platform === "slack" &&
+                    Array.isArray(entry.slackBlocks) &&
+                    entry.threadTs === threadTs);
+            }
+            catch {
+                continue;
+            }
+        }
+        return false;
+    }
     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).",
+            trustModel: "membership",
+            formattingGuide: "## Slack Formatting\nWrite standard Markdown/GFM: **bold**, _italic_, ~~strike~~, `code`, fenced code blocks, [links](url), lists, and pipe tables (rendered as native Slack tables).\nDo NOT use Slack mrkdwn syntax like *single-asterisk bold* or <url|label> links.",
             channels: this.getAllChannels().map((c) => ({ id: c.id, name: c.name })),
             users: this.getAllUsers().map((u) => ({
                 id: u.id,
@@ -410,10 +457,17 @@ export class SlackMessagingBot {
         });
     }
     processSlackMessageIntake(options) {
-        void processMessageIntake({
+        const absorbAttachmentFailure = () => {
+            void options.attachmentsPromise.catch((err) => {
+                log.logWarning("Failed to log Slack message", String(err));
+            });
+        };
+        processMessageIntake({
             eventBase: options.event,
             workingDir: this.workingDir,
             isAutoReplyCandidate: options.isAutoReplyCandidate,
+            magicWord: { addressed: options.addressed, scopeFallback: "top-level" },
+            busyPolicy: "queue",
             logEntryBase: {},
             processAttachments: () => options.attachmentsPromise,
             queueKey: options.queueKey,
@@ -421,8 +475,15 @@ export class SlackMessagingBot {
             handler: this.handler,
             bot: this,
             createContext: (event) => this.createContext(event),
-            onNotTriggered: options.onNotTriggered,
             deferAttachmentsUntilRun: true,
+        }).then((outcome) => {
+            // Slack logs eagerly via logUserMessage; when intake does not enqueue,
+            // nothing awaits the attachment download, so absorb its failure here.
+            if (outcome !== "enqueued")
+                absorbAttachmentFailure();
+        }, (err) => {
+            log.logWarning("Slack message intake failed", String(err));
+            absorbAttachmentFailure();
         });
     }
     buildHomeView() {
@@ -460,7 +521,7 @@ export class SlackMessagingBot {
             // Threshold for "stuck" detection (10 minutes)
             const STUCK_THRESHOLD_MS = 10 * 60 * 1000;
             for (const session of runningSessions) {
-                const channelId = session.sessionKey.split(":")[0];
+                const channelId = conversationIdOf(session.sessionKey);
                 const channel = this.channels.get(channelId);
                 const channelName = channel ? `#${channel.name}` : channelId;
                 const elapsed = Math.floor((Date.now() - session.startedAt) / 60000);
@@ -500,7 +561,14 @@ export class SlackMessagingBot {
                             {
                                 type: "button",
                                 text: { type: "plain_text", text: "Force Stop", emoji: true },
+                                // The raw session key travels in `value` (Slack returns it
+                                // verbatim); the action_id only routes and must be unique per
+                                // view, so it carries a sanitized copy of the key. Never
+                                // decode the key from the action_id — the `:`↔`_` rewrite is
+                                // irreversible for conversation ids that contain `_` (GitHub's
+                                // GH_owner_repo_number do, by design).
                                 action_id: `force_stop_${session.sessionKey.replace(/:/g, "_")}`,
+                                value: session.sessionKey,
                                 style: "danger",
                             },
                         ],
@@ -559,22 +627,6 @@ export class SlackMessagingBot {
         });
         return { type: "home", blocks: blocks };
     }
-    resolveStopTarget(channelId, threadTs) {
-        const directTarget = resolveStopTarget({
-            handler: this.handler,
-            conversationId: channelId,
-            sessionKey: resolveSlackSessionKey(channelId, threadTs),
-        });
-        if (directTarget)
-            return directTarget;
-        if (threadTs)
-            return null;
-        return resolveOnlyScopedStopTarget(this.handler, channelId);
-    }
-    isStopText(text) {
-        const normalized = text.trim().toLowerCase();
-        return normalized === "stop" || normalized === "/stop";
-    }
     createCommandAdapters(conversationId, userId, userName, text, ts, options = {}) {
         const message = {
             id: ts,
@@ -665,10 +717,23 @@ export class SlackMessagingBot {
         const context = this.createCommandAdapters(conversationId, payload.user_id, userName, commandText, eventTs, isDirectMessage ? { threadTs } : { ephemeralChannelId: conversationId, threadTs });
         return { event, context };
     }
-    async routeSlashLoginCommand(payload) {
+    /**
+     * Generic slash-command route: synthesize the command text per the
+     * manifest's SlackSlashRoute data and dispatch to the handler. `/pi-new`
+     * stays bespoke (routeSlashNewCommand) because it replies in 中文 and calls
+     * handleNewCommand directly.
+     */
+    async routeSlashCommand(route, payload) {
         const { event, context } = this.buildSlashCommandEvent(payload, {
-            type: payload.channel_id.startsWith("D") ? "dm" : "private_command",
-            includeText: true,
+            includeText: route.includeText,
+            thread: route.thread,
+            ...(route.privateCommand
+                ? {
+                    type: payload.channel_id.startsWith("D")
+                        ? "dm"
+                        : "private_command",
+                }
+                : {}),
         });
         await this.handler.handleEvent(event, this, context);
     }
@@ -697,19 +762,19 @@ export class SlackMessagingBot {
             enqueueEvent: (event) => this.enqueueEvent(event),
             getMessagingInfo: () => this.getMessagingInfo(),
         };
-        await this.handler.handleNewCommand(conversationId, conversationId, commandMessagingBot);
-    }
-    async routeSlashModelCommand(payload) {
-        const { event, context } = this.buildSlashCommandEvent(payload, { includeText: true });
-        await this.handler.handleEvent(event, this, context);
-    }
-    async routeSlashSessionCommand(payload) {
-        const { event, context } = this.buildSlashCommandEvent(payload, { thread: true });
-        await this.handler.handleEvent(event, this, context);
-    }
-    async routeSlashAdminCommand(payload) {
-        const { event, context } = this.buildSlashCommandEvent(payload, { thread: true });
-        await this.handler.handleEvent(event, this, context);
+        const event = {
+            type: "dm",
+            conversationId,
+            conversationKind: "direct",
+            channel: conversationId,
+            ts: eventTs,
+            user: payload.user_id,
+            text: payload.command,
+            attachments: [],
+            sessionKey: conversationId,
+        };
+        const context = createSlackAdapters(event, this);
+        await this.handler.handleNewCommand(conversationId, conversationId, commandMessagingBot, context.message, context.responder, context.platform);
     }
     setupEventHandlers() {
         this.socketClient.on("disconnect", (err) => {
@@ -761,26 +826,12 @@ export class SlackMessagingBot {
             ack();
             return;
         }
-        // Check for stop command - execute immediately, don't queue!
-        if (this.isStopText(slackEvent.text)) {
-            const stopTarget = this.resolveStopTarget(e.channel, e.thread_ts);
-            if (stopTarget) {
-                this.handler.handleStop(stopTarget, e.channel, this);
-            }
-            else {
-                this.postMessage(e.channel, formatNothingRunning("slack"));
-            }
-            void attachmentsPromise.catch((err) => {
-                log.logWarning("Failed to log Slack message", String(err));
-            });
-            ack();
-            return;
-        }
         this.processSlackMessageIntake({
             event: slackEvent,
             attachmentsPromise,
             queueKey: this.resolveQueueKey(e.channel, sessionKey),
             isAutoReplyCandidate: false,
+            addressed: true,
         });
         ack();
     }
@@ -793,7 +844,13 @@ export class SlackMessagingBot {
             ack();
             return;
         }
-        const isExternalMessagingBotMessage = !!e.bot_id || e.subtype === "bot_message";
+        // API posts made with a user token carry the posting app's bot_id/app_id
+        // alongside the human `user` — bot_id alone does not make the author a
+        // bot. Only treat the message as bot-authored when the author is not a
+        // known human user (bot users have is_bot=true; unknown authors stay on
+        // the conservative bot path so loop protection holds).
+        const authorIsKnownHuman = !!e.user && this.users.get(e.user)?.isBot === false;
+        const isExternalMessagingBotMessage = e.subtype === "bot_message" || (!!e.bot_id && !authorIsKnownHuman);
         if (isExternalMessagingBotMessage) {
             if (e.subtype !== undefined && e.subtype !== "bot_message" && e.subtype !== "file_share") {
                 ack();
@@ -821,7 +878,11 @@ export class SlackMessagingBot {
             ack();
             return;
         }
-        const isDM = e.channel_type === "im";
+        // message.im normally carries channel_type "im", but fall back to the
+        // D-prefix convention (used by handleAppMention and session keys) so a
+        // missing channel_type cannot silently demote a DM to an auto-reply
+        // candidate that never triggers.
+        const isDM = e.channel_type === "im" || e.channel.startsWith("D");
         const conversationKind = isDM ? "direct" : "shared";
         const isMessagingBotMention = e.text?.includes(`<@${this.botUserId}>`);
         // Skip channel @mentions - already handled by app_mention event
@@ -853,59 +914,26 @@ export class SlackMessagingBot {
             ack();
             return;
         }
-        // Stop command for DM only (app_mention handles shared-channel "@mikan stop").
-        if (isDM && this.isStopText(slackEvent.text)) {
-            const stopTarget = this.resolveStopTarget(e.channel, e.thread_ts);
-            if (stopTarget) {
-                this.handler.handleStop(stopTarget, e.channel, this);
-            }
-            else {
-                this.postMessage(e.channel, formatNothingRunning("slack"));
-            }
+        if (!isDM && isThreadReply) {
             void attachmentsPromise.catch((err) => {
                 log.logWarning("Failed to log Slack message", String(err));
             });
             ack();
             return;
         }
-        const enqueueTriggered = () => {
-            const activeSessionKey = slackEvent.sessionKey ?? resolveSlackSessionKey(e.channel, e.thread_ts);
-            // Auto-reply top-level channel messages start with no sessionKey because
-            // they are only candidates until the policy allows them. Once triggered,
-            // persist the resolved key on the event; otherwise the runtime fallback
-            // treats the message ts as a thread session (`channel:ts`) instead of the
-            // persistent top-level channel session.
-            slackEvent.sessionKey = activeSessionKey;
-            this.processSlackMessageIntake({
-                event: slackEvent,
-                attachmentsPromise,
-                queueKey: this.resolveQueueKey(e.channel, activeSessionKey),
-                isAutoReplyCandidate: false,
-            });
-        };
-        const logOnly = () => {
-            void attachmentsPromise.catch((err) => {
-                log.logWarning("Failed to log Slack message", String(err));
-            });
-        };
-        if (isDM) {
-            enqueueTriggered();
-            ack();
-            return;
-        }
-        if (isThreadReply) {
-            logOnly();
-            ack();
-            return;
-        }
-        const activeSessionKey = resolveSlackSessionKey(e.channel, e.thread_ts);
+        const activeSessionKey = slackEvent.sessionKey ?? resolveSlackSessionKey(e.channel, e.thread_ts);
+        // Auto-reply top-level channel messages start with no sessionKey because
+        // they are only candidates until the policy allows them. Persist the
+        // resolved key on the event; otherwise the runtime fallback treats the
+        // message ts as a thread session (`channel:ts`) instead of the persistent
+        // top-level channel session.
         slackEvent.sessionKey = activeSessionKey;
         this.processSlackMessageIntake({
             event: slackEvent,
             attachmentsPromise,
             queueKey: this.resolveQueueKey(e.channel, activeSessionKey),
-            isAutoReplyCandidate: true,
-            onNotTriggered: logOnly,
+            isAutoReplyCandidate: !isDM,
+            addressed: isDM,
         });
         ack();
     }
@@ -916,54 +944,20 @@ export class SlackMessagingBot {
             return;
         }
         const { command, text, channel_id, user_id, user_name, thread_ts } = payload;
+        const entry = COMMAND_MANIFEST.find((candidate) => candidate.slackCommand === command);
+        if (!entry)
+            return;
         let handlerPromise = null;
-        if (command === "/pi-login") {
-            handlerPromise = this.routeSlashLoginCommand({
-                command,
-                text,
-                channel_id,
-                user_id,
-                user_name,
-            });
-        }
-        else if (command === "/pi-new") {
+        if (entry.name === "new") {
             handlerPromise = this.routeSlashNewCommand({ command, channel_id, user_id, user_name });
         }
-        else if (command === "/pi-session") {
-            handlerPromise = this.routeSlashSessionCommand({
-                command,
-                channel_id,
-                user_id,
-                user_name,
-                thread_ts,
-            });
-        }
-        else if (command === "/pi-model") {
-            handlerPromise = this.routeSlashModelCommand({
+        else if (entry.slackRoute) {
+            handlerPromise = this.routeSlashCommand(entry.slackRoute, {
                 command,
                 text,
                 channel_id,
                 user_id,
                 user_name,
-            });
-        }
-        else if (command === "/pi-sandbox" ||
-            command === "/pi-auto-reply" ||
-            command === "/pi-extensions") {
-            handlerPromise = this.routeSlashModelCommand({
-                command,
-                text,
-                channel_id,
-                user_id,
-                user_name,
-            });
-        }
-        else if (command === "/pi-admin") {
-            handlerPromise = this.routeSlashAdminCommand({
-                command,
-                channel_id,
-                user_id,
-                user_name,
                 thread_ts,
             });
         }
@@ -1000,9 +994,12 @@ export class SlackMessagingBot {
             return;
         }
         ack();
-        const sessionKey = action.action_id.replace("force_stop_", "").replace(/_/g, ":");
+        // Prefer the verbatim key from `value`; the action_id fallback only
+        // serves buttons rendered before `value` existed and misdecodes session
+        // keys whose conversation id contains "_".
+        const sessionKey = action.value ?? action.action_id.replace("force_stop_", "").replace(/_/g, ":");
         const userId = body.user?.id;
-        const channelId = body.container?.channel_id || sessionKey.split(":")[0];
+        const channelId = body.container?.channel_id || conversationIdOf(sessionKey);
         log.logInfo(`[Force Stop] User ${userId} requested force stop for ${sessionKey}`);
         // Use handler's forceStop method
         this.handler.forceStop(sessionKey);
@@ -1030,6 +1027,38 @@ export class SlackMessagingBot {
         const selectedOptions = Array.isArray(action.selected_options)
             ? action.selected_options
             : undefined;
+        const threadTs = container.thread_ts;
+        const sessionKey = resolveSlackSessionKey(channelId, threadTs);
+        // Extension-owned actions (ext:<slug>: namespaced by api.blockkit.post)
+        // dispatch exclusively to the extension's onAction handler — no agent
+        // run, no chat-history entry. They never fall through to the model: an
+        // unconsumed namespaced action means the extension is gone, and its
+        // routing id would only confuse an agent run.
+        const extAction = parseExtActionId(action.action_id ?? "");
+        if (extAction) {
+            this.getQueue(this.resolveQueueKey(channelId, sessionKey)).enqueue(async () => {
+                const consumed = await this.handler.handleExtensionAction({
+                    conversationId: channelId,
+                    sessionKey,
+                    conversationKind: channelId.startsWith("D") ? "direct" : "shared",
+                    slug: extAction.slug,
+                    action: {
+                        actionId: extAction.actionId,
+                        value: action.value ?? selectedOption?.value ?? selectedOption?.text?.text,
+                        selectedValues: selectedOptions?.map((option) => option.value ?? option.text?.text ?? ""),
+                        userId,
+                        userName: body.user?.username ?? body.user?.name,
+                        conversationId: channelId,
+                        messageTs: container.message_ts,
+                        threadTs,
+                    },
+                });
+                if (!consumed) {
+                    log.logWarning(`[${channelId}] Dropped extension action ${action.action_id}`, `no handler registered by extension "${extAction.slug}"`);
+                }
+            });
+            return;
+        }
         const selectedText = selectedOption?.text?.text ?? selectedOption?.value;
         const selectedTexts = selectedOptions?.map((option) => option.text?.text ?? option.value);
         const valueText = selectedTexts?.length
@@ -1037,8 +1066,6 @@ export class SlackMessagingBot {
             : (selectedText ?? action.value ?? action.action_id);
         const text = `[Slack action] ${action.action_id}: ${valueText}`;
         const ts = `action:${Date.now()}`;
-        const threadTs = container.thread_ts;
-        const sessionKey = resolveSlackSessionKey(channelId, threadTs);
         this.logToFile(channelId, {
             date: new Date().toISOString(),
             ts,
@@ -1296,6 +1323,7 @@ export class SlackMessagingBot {
                             id: u.id,
                             userName: u.name,
                             displayName: u.real_name || u.name,
+                            isBot: !!u.is_bot,
                         });
                     }
                 }