@geminixiang/mikan 1.0.0-beta.7 → 1.0.0-beta.71

package/dist/adapters/slack/bot.js

@@ -1,22 +1,64 @@
 import { SocketModeClient } from "@slack/socket-mode";
-import { WebClient } from "@slack/web-api";
-import { existsSync, readFileSync } from "fs";
-import { readFile } from "fs/promises";
-import { basename, join } from "path";
-import { resolveConversationSettings } from "../../config.js";
+import { WebAPIRateLimitedError, WebClient } from "@slack/web-api";
+import { existsSync, readFileSync } from "node:fs";
+import { readFile, writeFile } from "node:fs/promises";
+import { basename, join } from "node:path";
+import { createConversationEvent, createConversationMessage, } from "../index.js";
+import { createOfficeAddress, listRegisteredOffices } from "../../office/index.js";
+import { COMMAND_MANIFEST } from "../commands/manifest.js";
+import { slackConversationAutoReplyMode, resolveConversationSettings, } from "../../settings/index.js";
+import { evaluateWithJev, JevNotConfiguredError } from "../../harness/index.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 { processMessageIntake } from "../intake.js";
+import { isRecord, readTextFileIfExists } from "../../file-guards.js";
+import { PRODUCT_NAME, formatForceStopped } from "../messages.js";
+import { appendBotResponseLog, appendChannelLog, MessagingEventQueue, MessagingIntakeTracker, saveIncomingAttachments, withRetry, } from "../shared.js";
+import { matchMagicWord, processMessageIntake } from "../intake.js";
+import { readPlatformChannelKind, recordPlatformChannelKind, } from "../../office/projection.js";
+import { AssistantThreadRegistry, handleAgentContextChanged, handleAgentDmOpened, handleAssistantThreadStarted, titleAssistantThread, } from "./assistant.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";
+import { reportUserFacingError } from "../../observability/index.js";
+import { recordSlackUpdate } from "./update-diagnostics.js";
+import { renderSlackBlocks, resolveSlackMentions } from "./blocks.js";
+import { querySlackTasks, formatTaskStatus, readTaskRoots, isTaskStatusQuestion, } from "./task-status.js";
+import { buildAutoReplyState, JEV_ADDRESSED_INSTRUCTIONS } from "./auto-reply-context.js";
+import { buildTaskIntentState, classifyTaskIntent } from "./task-intent.js";
+import { StreamStartLimiter } from "./stream-limits.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.
+function hasMessageContent(message, includeBlocks = true) {
+    const content = [message.text, message.files];
+    if (includeBlocks)
+        content.push(message.blocks, message.attachments);
+    return content.some((part) => !!part?.length);
+}
+function hasBotIdentity(message) {
+    return !!message.bot_id || message.subtype === "bot_message";
+}
+const USER_MESSAGE_SUBTYPES = new Set([undefined, "file_share"]);
+const BOT_MESSAGE_SUBTYPES = new Set([...USER_MESSAGE_SUBTYPES, "bot_message"]);
+/**
+ * Slack's per-call limit on `markdown_text`: "Limit this field to 12,000
+ * characters." mikan allows far more in a message, so streamed text is split
+ * to fit rather than truncated.
+ */
+const MAX_STREAM_TEXT_CHARS = 12_000;
+/** Slices `text` into pieces that each fit one streaming call. */
+export function chunkStreamText(text, limit = MAX_STREAM_TEXT_CHARS) {
+    if (text.length <= limit)
+        return [text];
+    const chunks = [];
+    for (let index = 0; index < text.length; index += limit) {
+        chunks.push(text.slice(index, index + limit));
+    }
+    return chunks;
+}
+// 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;
@@ -72,14 +114,110 @@ export function buildMrkdwnContextBlock(text) {
 // ============================================================================
 // SlackMessagingBot
 // ============================================================================
+class AttachmentDownloadHttpError extends Error {
+    constructor(message, status) {
+        super(message);
+        this.status = status;
+    }
+}
+function isRetryableAttachmentDownloadError(error) {
+    if (!(error instanceof AttachmentDownloadHttpError))
+        return true;
+    return error.status === 408 || error.status === 429 || error.status >= 500;
+}
 export class SlackMessagingBot {
+    /** Host office dir for a Slack conversation. */
+    conversationDir(channelId) {
+        return this.workspace.office(createOfficeAddress("slack", channelId)).dir;
+    }
+    resolveReplyMode(address) {
+        return (resolveConversationSettings(this.workspace.office(address)).slack?.replyMode ?? "top-level");
+    }
     createContext(event) {
-        return createSlackAdapters(event, this, {
-            replyMode: resolveConversationSettings(join(this.workingDir, event.conversationId)).slack?.replyMode ??
-                "top-level",
+        const context = createSlackAdapters(event, this, {
+            replyMode: this.resolveReplyMode(event.address),
         });
+        if (event.conversationKind === "direct") {
+            context.responder.getTaskStatus = (key) => querySlackTasks(this.conversationDir(event.channel), event.channel, this.handler
+                .getRunningSessions()
+                .filter((s) => s.address.platform === "slack" && s.address.conversationId === event.channel), key);
+        }
+        if (event.conversationKind === "direct" &&
+            event.thread_ts &&
+            this.isTaskThread(event.channel, event.thread_ts)) {
+            let notified = false;
+            context.responder.notifyCompletion = async () => {
+                if (notified)
+                    return;
+                const text = `<@${event.user}> 這一輪處理已結束，請查看上方結果。`;
+                const ts = await this.postInThread(event.channel, event.thread_ts, text);
+                notified = true;
+                this.logBotResponse(event.channel, text, ts, event.thread_ts);
+            };
+        }
+        if (event.conversationKind === "direct" && !event.thread_ts) {
+            context.responder.startTask = async (message, task) => {
+                if (this.stopped)
+                    throw new Error("Slack is shutting down; task was not started.");
+                await context.responder.deleteResponse();
+                const root = await this.postMessage(event.channel, message);
+                this.logToFile(event.channel, {
+                    date: new Date().toISOString(),
+                    ts: root,
+                    user: "bot",
+                    text: message,
+                    isMessagingBot: true,
+                    taskRoot: true,
+                });
+                const sessionKey = resolveSlackSessionKey(event.channel, root);
+                try {
+                    registerThreadSession({
+                        conversationDir: this.conversationDir(event.channel),
+                        sessionKey,
+                    });
+                    const child = {
+                        ...event,
+                        ts: `task:${root}`,
+                        thread_ts: root,
+                        sessionKey,
+                        text: task,
+                    };
+                    if (!this.getQueue(sessionKey).enqueue(async () => {
+                        try {
+                            await this.handler.handleEvent({
+                                ...child,
+                                attachments: child.attachments?.map((a) => ({
+                                    name: a.original,
+                                    localPath: a.localPath,
+                                })),
+                            }, this, this.createContext(child));
+                        }
+                        catch (error) {
+                            reportUserFacingError(error, {
+                                domain: "mikan",
+                                surface: "task_handoff",
+                                operation: "start_task_run",
+                                severity: "error",
+                                platform: "slack",
+                                context: { conversationId: event.channel, sessionKey, threadTs: root },
+                            });
+                            await this.postInThread(event.channel, root, "Task could not start. Please reply here to try again.");
+                            throw error;
+                        }
+                    }))
+                        throw new Error("Task queue is closed.");
+                }
+                catch (error) {
+                    await this.updateMessage(event.channel, root, `${message}\n\nTask could not start.`);
+                    throw error;
+                }
+                return sessionKey;
+            };
+        }
+        return context;
     }
     constructor(handler, config) {
+        this.statusUpdates = new Map();
         this.botUserId = null;
         this.botId = null;
         this.teamId = null;
@@ -87,11 +225,22 @@ export class SlackMessagingBot {
         this.startupTs = null; // Messages older than this are just logged, not processed
         this.users = new Map();
         this.channels = new Map();
+        /** Which threads belong to the assistant pane, and their last known context. */
+        this.assistantThreads = new AssistantThreadRegistry();
+        /**
+         * Native response streams opened recently. Per bot rather than per module:
+         * the rate tier applies per workspace, and this bot is the connection to
+         * one — which also means a test constructing a bot gets a fresh budget
+         * instead of inheriting whatever ran before it.
+         */
+        this.streamStarts = new StreamStartLimiter();
+        this.stopped = false;
         this.queues = new Map();
-        this.eventsWatcher = null;
+        this.intake = new MessagingIntakeTracker("Slack");
+        this.eventScheduler = null;
         this.handler = handler;
-        this.workingDir = config.workingDir;
-        this.store = config.store;
+        this.workspace = config.workspace;
+        this.botToken = config.botToken;
         this.socketClient = new SocketModeClient({
             appToken: config.appToken,
             // Default 5s is too tight: brief event-loop stalls (e.g. backfill, sync fs)
@@ -99,20 +248,29 @@ export class SlackMessagingBot {
             clientPingTimeout: 12_000,
         });
         this.webClient = new WebClient(config.botToken);
+        this.statusClient = new WebClient(config.botToken, {
+            timeout: 3000,
+            retryConfig: { retries: 0 },
+            rejectRateLimitedCalls: true,
+        });
     }
-    setEventsWatcher(watcher) {
-        this.eventsWatcher = watcher;
+    setEventScheduler(scheduler) {
+        this.eventScheduler = scheduler;
     }
     // ==========================================================================
     // Public API
     // ==========================================================================
     async start() {
+        this.stopped = false;
         const auth = await this.webClient.auth.test();
         this.botUserId = auth.user_id;
         this.botId = typeof auth.bot_id === "string" ? auth.bot_id : null;
         this.teamId = typeof auth.team_id === "string" ? auth.team_id : null;
         await Promise.all([this.fetchUsers(), this.fetchChannels()]);
+        if (this.stopped)
+            return;
         log.logInfo(`Loaded ${this.channels.size} channels, ${this.users.size} users`);
+        this.backfillChannelKinds();
         // Record startup time before opening the socket. Slack may replay older events;
         // those should be logged but not processed. Backfill runs in the background up
         // to this timestamp so startup is not blocked by one history call per channel.
@@ -124,6 +282,16 @@ export class SlackMessagingBot {
             log.logWarning("Slack backfill failed", String(error));
         });
     }
+    async stop() {
+        this.stopped = true;
+        try {
+            await this.socketClient.disconnect();
+        }
+        finally {
+            await this.intake.close();
+            await Promise.all([...this.queues.values()].map((queue) => queue.close()));
+        }
+    }
     getUser(userId) {
         return this.users.get(userId);
     }
@@ -145,9 +313,22 @@ export class SlackMessagingBot {
         }
         return source.replace(this.ownMentionRegex, "").trim();
     }
-    async postMessage(channel, text) {
+    /**
+     * Response-source `<@userName>` mentions become native `<@U…>` here — the
+     * one conversion every outgoing response path funnels through. Slack only
+     * links and notifies on raw user ids.
+     */
+    resolveMentions(text) {
+        return resolveSlackMentions(text, this.users.values());
+    }
+    async postMessage(channel, text, threadTs, identity) {
         return slackRetry(async () => {
-            const payload = { channel, ...renderSlackBlocks(text) };
+            const payload = {
+                channel,
+                ...(threadTs !== undefined ? { thread_ts: threadTs } : {}),
+                ...(identity ? { username: identity.username, icon_emoji: identity.iconEmoji } : {}),
+                ...renderSlackBlocks(this.resolveMentions(text)),
+            };
             const result = await this.webClient.chat.postMessage(payload);
             return result.ts;
         });
@@ -167,32 +348,35 @@ export class SlackMessagingBot {
             }
         });
     }
-    async postEphemeral(channel, user, text, threadTs) {
+    async postEphemeral(channel, user, text, threadTs, blocks) {
         return slackRetry(async () => {
             await this.webClient.chat.postEphemeral({
                 channel,
                 user,
                 text,
+                ...(blocks !== undefined ? { blocks: blocks } : {}),
                 ...(threadTs ? { thread_ts: threadTs } : {}),
             });
         });
     }
     async postEphemeralBlocks(channel, user, text, blocks, threadTs) {
+        return this.postEphemeral(channel, user, text, threadTs, blocks);
+    }
+    async postMessageBlocks(channel, text, blocks, threadTs) {
         return slackRetry(async () => {
-            await this.webClient.chat.postEphemeral({
+            const payload = {
                 channel,
-                user,
                 text,
                 blocks: blocks,
-                ...(threadTs ? { thread_ts: threadTs } : {}),
-            });
+                ...(threadTs !== undefined ? { thread_ts: threadTs } : {}),
+            };
+            const result = await this.webClient.chat.postMessage(payload);
+            return result.ts;
         });
     }
-    async postMessageBlocks(channel, text, blocks) {
+    async updateMessageBlocks(channel, ts, text, blocks) {
         return slackRetry(async () => {
-            const payload = { channel, text, blocks: blocks };
-            const result = await this.webClient.chat.postMessage(payload);
-            return result.ts;
+            await this.webClient.chat.update({ channel, ts, text, blocks: blocks });
         });
     }
     async postPrivate(conversationId, userId, text) {
@@ -205,7 +389,7 @@ export class SlackMessagingBot {
         }
         await this.postEphemeralBlocks(conversationId, userId, text, [buildMrkdwnContextBlock(text)]);
     }
-    async openDirectMessage(userId) {
+    async openDirectConversation(userId) {
         return slackRetry(async () => {
             const result = await this.webClient.conversations.open({ users: userId });
             const channelId = result.channel?.id;
@@ -215,33 +399,134 @@ export class SlackMessagingBot {
             return channelId;
         });
     }
+    /**
+     * Fetch recent messages from a channel, oldest first. Without `threadTs`
+     * this is one `conversations.history` page of top-level messages (thread
+     * replies are not expanded); with it, one `conversations.replies` page of
+     * that thread's replies, excluding the parent. Callers page forward by
+     * passing the last returned `ts` as `oldest`.
+     */
+    async fetchHistory(channel, options) {
+        const limit = Math.min(Math.max(options?.limit ?? 200, 1), 999);
+        const threadTs = options?.threadTs;
+        return slackRetry(async () => {
+            const result = threadTs
+                ? await this.webClient.conversations.replies({
+                    channel,
+                    ts: threadTs,
+                    ...(options?.oldest ? { oldest: options.oldest, inclusive: false } : {}),
+                    limit,
+                })
+                : await this.webClient.conversations.history({
+                    channel,
+                    ...(options?.oldest ? { oldest: options.oldest, inclusive: false } : {}),
+                    limit,
+                });
+            const messages = (result.messages ?? []);
+            const mapped = messages
+                .filter((msg) => !!msg.ts)
+                // conversations.replies leads with the thread parent; the caller asked
+                // for the replies to it, and including it would re-ingest the bot's
+                // own message on every poll.
+                .filter((msg) => msg.ts !== threadTs)
+                .map((msg) => {
+                const user = msg.user ? this.users.get(msg.user) : undefined;
+                const message = {
+                    ts: msg.ts,
+                    text: msg.text ?? "",
+                    isBot: !!msg.bot_id || msg.subtype === "bot_message" || user?.isBot === true,
+                };
+                if (msg.thread_ts && msg.thread_ts !== msg.ts)
+                    message.threadTs = msg.thread_ts;
+                if (msg.user)
+                    message.userId = msg.user;
+                if (user)
+                    message.userName = user.userName;
+                return message;
+            });
+            // history returns newest-first, replies oldest-first; the contract is
+            // oldest-first either way.
+            return threadTs ? mapped : mapped.toReversed();
+        });
+    }
+    /** Refresh and list the workspace's active users (`users.list`). */
+    async listUsers() {
+        await slackRetry(() => this.fetchUsers());
+        return this.getAllUsers().map((user) => ({
+            id: user.id,
+            userName: user.userName,
+            displayName: user.displayName,
+            isBot: user.isBot === true,
+        }));
+    }
     async updateMessage(channel, ts, text) {
         return slackRetry(async () => {
-            const payload = { channel, ts, ...renderSlackBlocks(text) };
-            await this.webClient.chat.update(payload);
+            const payload = { channel, ts, ...renderSlackBlocks(this.resolveMentions(text)) };
+            try {
+                await this.webClient.chat.update(payload);
+            }
+            catch (error) {
+                recordSlackUpdate(this, { channel, ts }, text, payload, { error });
+                throw error;
+            }
+            recordSlackUpdate(this, { channel, ts }, text, payload, { success: true });
         });
     }
+    /**
+     * Claim one native stream, or decline. Declining is not an error: the caller
+     * falls back to edit-based updates, which is a working answer rather than a
+     * degraded one, and costs no request against the tighter start tier.
+     */
+    tryReserveStreamStart() {
+        return this.streamStarts.tryReserve();
+    }
+    /**
+     * Open a stream, carrying text of any length.
+     *
+     * Slack caps `markdown_text` at 12,000 characters per call, and a long first
+     * render exceeds that easily — mikan allows 35,000 in a message. Rather than
+     * truncating (losing the answer) or failing (falling back to edit mode for
+     * the whole response), the opening call takes the first slice and the rest
+     * follows as appends before this resolves. The caller sees one operation
+     * that sent everything, so the renderer's idea of what has been streamed
+     * stays true.
+     *
+     * Splitting mid-word or mid-fence is harmless here: a stream concatenates,
+     * so Slack renders the reassembled text, not the pieces.
+     */
     async startMessageStream(channel, text, threadTs, recipientUserId) {
-        return slackRetry(async () => {
+        const [head, ...tail] = chunkStreamText(text);
+        const ts = await slackRetry(async () => {
             const result = await this.webClient.apiCall("chat.startStream", {
                 channel,
-                markdown_text: text,
+                markdown_text: this.resolveMentions(head ?? ""),
                 ...(threadTs ? { thread_ts: threadTs } : {}),
                 ...(this.teamId ? { recipient_team_id: this.teamId } : {}),
                 ...(recipientUserId ? { recipient_user_id: recipientUserId } : {}),
             });
-            const ts = result.ts;
-            if (!ts)
+            const streamTs = result.ts;
+            if (!streamTs)
                 throw new Error("Slack chat.startStream did not return ts");
-            return ts;
+            return streamTs;
         });
+        for (const part of tail)
+            await this.appendStreamChunk(channel, ts, part);
+        return ts;
     }
+    /** Append a delta of any length, split to the per-call cap. */
     async appendMessageStream(channel, ts, text) {
+        for (const part of chunkStreamText(text)) {
+            await this.appendStreamChunk(channel, ts, part);
+        }
+    }
+    async appendStreamChunk(channel, ts, text) {
         return slackRetry(async () => {
             await this.webClient.apiCall("chat.appendStream", {
                 channel,
                 ts,
-                markdown_text: text,
+                // A mention split across stream deltas stays unresolved here; the
+                // canonical final render (updateMessage) resolves the full text.
+                markdown_text: this.resolveMentions(text),
             });
         });
     }
@@ -260,33 +545,97 @@ export class SlackMessagingBot {
     // ==========================================================================
     /** Set the status for an assistant thread (shows "thinking" state) */
     async setAssistantStatus(channel, threadTs, status) {
-        return slackRetry(async () => {
-            await this.webClient.assistant.threads.setStatus({
+        const key = `${channel}:${threadTs}`;
+        const previous = this.statusUpdates.get(key) ?? Promise.resolve();
+        const update = previous
+            .catch(() => undefined)
+            .then(async () => {
+            await this.statusClient.assistant.threads.setStatus({
                 channel_id: channel,
                 thread_ts: threadTs,
                 status,
             });
         });
+        this.statusUpdates.set(key, update);
+        try {
+            await update;
+        }
+        finally {
+            if (this.statusUpdates.get(key) === update)
+                this.statusUpdates.delete(key);
+        }
     }
-    async postInThread(channel, threadTs, text) {
+    /** Offer starting points in a new assistant thread. */
+    async setAssistantSuggestedPrompts(channel, threadTs, prompts) {
         return slackRetry(async () => {
-            const payload = { channel, thread_ts: threadTs, ...renderSlackBlocks(text) };
-            const result = await this.webClient.chat.postMessage(payload);
-            return result.ts;
+            // Omitting thread_ts pins the prompts to the DM itself, which is what
+            // the agent surface wants: there is no thread yet when it opens.
+            await this.webClient.assistant.threads.setSuggestedPrompts({
+                channel_id: channel,
+                ...(threadTs ? { thread_ts: threadTs } : {}),
+                prompts,
+            });
         });
     }
-    async postInThreadBlocks(channel, threadTs, text, blocks) {
+    /** Name an assistant thread, which is what the sidebar conversation list shows. */
+    async setAssistantTitle(channel, threadTs, title) {
         return slackRetry(async () => {
-            const payload = {
-                channel,
+            await this.webClient.assistant.threads.setTitle({
+                channel_id: channel,
                 thread_ts: threadTs,
-                text, // fallback for notifications
-                blocks: blocks,
-            };
-            const result = await this.webClient.chat.postMessage(payload);
-            return result.ts;
+                title,
+            });
         });
     }
+    /** The assistant surface's view of this bot, handed to the lifecycle handlers. */
+    assistantOps() {
+        return {
+            postInThread: (channel, threadTs, text) => this.postInThread(channel, threadTs, text),
+            setSuggestedPrompts: (channel, threadTs, prompts) => this.setAssistantSuggestedPrompts(channel, threadTs, prompts),
+            setTitle: (channel, threadTs, title) => this.setAssistantTitle(channel, threadTs, title),
+            channelName: (channelId) => this.channels.get(channelId)?.name,
+        };
+    }
+    /**
+     * A new conversation opened in the assistant pane. Slack waits for the app
+     * to speak first here; staying silent is what makes the pane look broken.
+     */
+    handleAssistantThreadStarted({ event, ack }) {
+        ack();
+        const payload = event;
+        if (!payload.assistant_thread)
+            return;
+        void handleAssistantThreadStarted(this.assistantOps(), this.assistantThreads, payload.assistant_thread);
+    }
+    /**
+     * The person navigated to another channel while the pane is open. Slack
+     * sends this as `assistant_thread_context_changed` under `assistant_view`
+     * and `app_context_changed` under `agent_view`; the payload differs enough
+     * to normalize here rather than in the handler.
+     */
+    handleAgentContextChangedEvent({ event, ack, }) {
+        ack();
+        const payload = event;
+        if (payload.assistant_thread) {
+            handleAgentContextChanged(this.assistantThreads, payload.assistant_thread);
+            return;
+        }
+        const channelId = payload.channel_id ?? payload.channel;
+        if (channelId) {
+            handleAgentContextChanged(this.assistantThreads, {
+                channel_id: channelId,
+                context: payload.context,
+            });
+        }
+    }
+    async postInThread(channel, threadTs, text, identity) {
+        return identity
+            ? this.postMessage(channel, text, threadTs, identity)
+            : this.postMessage(channel, text, threadTs);
+    }
+    async postInThreadBlocks(channel, threadTs, text, blocks) {
+        return this.postMessageBlocks(channel, text, blocks, threadTs);
+    }
     async uploadFile(channel, filePath, title, threadTs) {
         return slackRetry(async () => {
             const fileName = title || basename(filePath);
@@ -301,18 +650,40 @@ export class SlackMessagingBot {
         });
     }
     logToFile(channel, entry) {
-        appendChannelLog(this.workingDir, channel, entry);
+        appendChannelLog(this.workspace.office(createOfficeAddress("slack", channel)), entry);
     }
     logBotResponse(channel, text, ts, threadTs, slackBlocks) {
-        appendBotResponseLog(this.workingDir, channel, text, ts, threadTs, {
+        appendBotResponseLog(this.workspace.office(createOfficeAddress("slack", channel)), text, ts, threadTs, {
             platform: "slack",
             ...(slackBlocks ? { slackBlocks } : {}),
         });
     }
+    ownsBlockKitMessage(channel, ts, threadTs) {
+        const content = readTextFileIfExists(join(this.conversationDir(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).",
+            ...(this.teamId ? { workspaceId: this.teamId } : {}),
+            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,
@@ -332,6 +703,8 @@ export class SlackMessagingBot {
      * Returns true if enqueued, false if queue is full (max 5).
      */
     enqueueEvent(event) {
+        if (this.stopped)
+            return false;
         const conversationId = event.conversationId;
         const queue = this.getQueue(conversationId);
         if (queue.size() >= 5) {
@@ -339,7 +712,7 @@ export class SlackMessagingBot {
             return false;
         }
         log.logInfo(`Enqueueing event for ${conversationId}: ${event.text.substring(0, 50)}`);
-        queue.enqueue(async () => {
+        return queue.enqueue(async () => {
             let anchorTs;
             if (!event.thread_ts) {
                 try {
@@ -367,15 +740,17 @@ export class SlackMessagingBot {
             const eventForRun = eventPlan.event;
             if (eventPlan.initialMessageTs && eventForRun.sessionKey) {
                 registerThreadSession({
-                    conversationDir: join(this.workingDir, conversationId),
+                    conversationDir: this.conversationDir(conversationId),
                     sessionKey: eventForRun.sessionKey,
                 });
             }
             const runQueueKey = planSlackAdapterSession(eventForRun, {
                 initialMessageTs: eventPlan.initialMessageTs,
             }).sessionKey;
-            this.getQueue(runQueueKey).enqueue(async () => {
-                const slackEvent = {
+            const run = async () => {
+                const slackEvent = createConversationEvent({
+                    platform: "slack",
+                    address: eventForRun.address,
                     type: eventForRun.type,
                     conversationId,
                     conversationKind: eventForRun.conversationKind,
@@ -384,21 +759,21 @@ export class SlackMessagingBot {
                     thread_ts: eventForRun.thread_ts,
                     user: eventForRun.user,
                     text: eventForRun.text,
-                    attachments: eventForRun.attachments?.map((attachment) => ({
-                        original: attachment.name,
-                        localPath: attachment.localPath,
-                    })),
+                    attachments: eventForRun.attachments,
                     sessionKey: eventForRun.sessionKey,
-                };
+                });
                 const context = createSlackAdapters(slackEvent, this, {
                     initialMessageTs: eventPlan.initialMessageTs,
-                    replyMode: resolveConversationSettings(join(this.workingDir, eventForRun.conversationId)).slack
-                        ?.replyMode ?? "top-level",
+                    replyMode: this.resolveReplyMode(eventForRun.address),
                 });
                 return this.handler.handleEvent(eventForRun, this, context);
-            });
+            };
+            const runQueue = this.getQueue(runQueueKey);
+            if (!runQueue.enqueue(run)) {
+                await runQueue.close();
+                await run();
+            }
         });
-        return true;
     }
     // ==========================================================================
     // Private - Event Handlers
@@ -407,6 +782,8 @@ export class SlackMessagingBot {
         let queue = this.queues.get(channelId);
         if (!queue) {
             queue = new MessagingEventQueue("Slack");
+            if (this.stopped)
+                void queue.close();
             this.queues.set(channelId, queue);
         }
         return queue;
@@ -414,21 +791,108 @@ export class SlackMessagingBot {
     resolveQueueKey(conversationId, sessionKey) {
         if (!isSlackThreadSessionKey(sessionKey))
             return sessionKey;
-        if (this.handler.isRunning(sessionKey))
+        if (this.handler.isRunning(createOfficeAddress("slack", conversationId), sessionKey)) {
             return sessionKey;
+        }
         return this.hasKnownThreadSession(conversationId, sessionKey) ? sessionKey : conversationId;
     }
+    isTaskThread(channel, root) {
+        return readTaskRoots(this.conversationDir(channel)).has(root);
+    }
+    hasRunningTaskThread(channel) {
+        const roots = readTaskRoots(this.conversationDir(channel));
+        if (!roots.size)
+            return false;
+        return this.handler
+            .getRunningSessions()
+            .some((s) => s.address.platform === "slack" &&
+            s.address.conversationId === channel &&
+            [...roots.keys()].some((root) => resolveSlackSessionKey(channel, root) === s.sessionKey));
+    }
     hasKnownThreadSession(conversationId, sessionKey) {
         return hasMaterializedChatSession({
-            conversationDir: join(this.workingDir, conversationId),
+            conversationDir: this.conversationDir(conversationId),
             sessionKey,
         });
     }
+    /**
+     * Slack's own conversation vocabulary for this channel, from the metadata
+     * fetchChannels() already loads. Externally shared channels report
+     * "external" regardless of privacy — platform-public within one workspace
+     * is the sharing condition, and a channel visible to another organization
+     * does not satisfy it. Unknown channels return undefined (never recorded,
+     * so the projection keeps its isolated default).
+     */
+    channelKindFor(channelId) {
+        if (channelId.startsWith("D"))
+            return "im";
+        const channel = this.channels.get(channelId);
+        if (!channel)
+            return undefined;
+        if (channel.name.startsWith("DM:"))
+            return "im";
+        if (channel.isExternallyShared)
+            return "external";
+        if (channel.isPrivate === true)
+            return "private_channel";
+        if (channel.isPrivate === false)
+            return "public_channel";
+        return undefined;
+    }
+    /**
+     * Office visibility follows the recorded channel kind, and an unrecorded
+     * kind fails closed to private (ADR 0008). Offices created before kinds
+     * were recorded would therefore lose public status until their next
+     * message; the channel list loaded at startup already knows every kind, so
+     * record it for all registered Slack offices now. Metadata only.
+     */
+    backfillChannelKinds() {
+        let recorded = 0;
+        for (const record of listRegisteredOffices(this.workspace.stateDir)) {
+            if (record.platform !== "slack")
+                continue;
+            const kind = this.channelKindFor(record.conversationId);
+            if (!kind)
+                continue;
+            try {
+                const office = this.workspace.office(record);
+                if (readPlatformChannelKind(office) === kind)
+                    continue;
+                recordPlatformChannelKind(office, kind);
+                recorded++;
+            }
+            catch (err) {
+                log.logWarning("Failed to backfill Slack channel kind", String(err));
+            }
+        }
+        if (recorded > 0)
+            log.logInfo(`Recorded channel kind for ${recorded} Slack offices`);
+    }
     processSlackMessageIntake(options) {
-        void processMessageIntake({
+        const kind = this.channelKindFor(options.event.conversationId);
+        if (kind) {
+            try {
+                recordPlatformChannelKind(this.workspace.office(options.event.address), kind);
+            }
+            catch (err) {
+                // A failed snapshot must not block the message; the projection just
+                // keeps its previous (or isolated default) posture.
+                log.logWarning("Failed to record Slack channel kind", String(err));
+            }
+        }
+        const absorbAttachmentFailure = () => {
+            void options.attachmentsPromise.catch((err) => {
+                log.logWarning("Failed to log Slack message", String(err));
+            });
+        };
+        return processMessageIntake({
             eventBase: options.event,
-            workingDir: this.workingDir,
-            isAutoReplyCandidate: options.isAutoReplyCandidate,
+            addressed: options.addressed,
+            magicWord: {
+                addressed: options.magicWordAddressed ?? options.addressed,
+                scopeFallback: "top-level",
+            },
+            busyPolicy: "queue",
             logEntryBase: {},
             processAttachments: () => options.attachmentsPromise,
             queueKey: options.queueKey,
@@ -436,26 +900,18 @@ 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() {
-        const blocks = [
-            {
-                type: "section",
-                text: {
-                    type: "mrkdwn",
-                    text: `*${PRODUCT_NAME}*\nStart a new task or check on running work.`,
-                },
-                accessory: {
-                    type: "image",
-                    image_url: "https://media1.tenor.com/m/lfDATg4Bhc0AAAAC/happy-cat.gif",
-                    alt_text: PRODUCT_NAME,
-                },
-            },
-        ];
-        // --- Running tasks ---
+    appendRunningTasks(blocks) {
         const runningSessions = this.handler.getRunningSessions();
         blocks.push({ type: "divider" }, {
             type: "header",
@@ -470,61 +926,56 @@ export class SlackMessagingBot {
                 type: "context",
                 elements: [{ type: "mrkdwn", text: "_No tasks running right now._" }],
             });
+            return;
         }
-        else {
-            // 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 channel = this.channels.get(channelId);
-                const channelName = channel ? `#${channel.name}` : channelId;
-                const elapsed = Math.floor((Date.now() - session.startedAt) / 60000);
-                const elapsedStr = elapsed < 1 ? "<1 min" : `${elapsed} min`;
-                // Check if task might be stuck
-                const lastActivity = session.lastActivityAt ? Date.now() - session.lastActivityAt : 0;
-                const isStuck = lastActivity > STUCK_THRESHOLD_MS;
-                const statusText = isStuck ? "_stuck_" : "_running_";
-                // Build status line: channel · status · time · step
-                let statusLine = `${statusText} · ${elapsedStr}`;
-                if (session.currentTool) {
-                    statusLine += ` · ${session.currentTool}`;
-                }
-                if (isStuck && lastActivity > 0) {
-                    const inactiveMin = Math.floor(lastActivity / 60000);
-                    statusLine += ` · idle ${inactiveMin}m`;
-                }
-                // Use context block for gray small text (like "No scheduled jobs.")
+        const stuckThresholdMs = 10 * 60 * 1000;
+        for (const session of runningSessions) {
+            const channelId = conversationIdOf(session.sessionKey);
+            const channelName = this.channels.get(channelId)?.name;
+            const elapsed = Math.floor((Date.now() - session.startedAt) / 60000);
+            const elapsedStr = elapsed < 1 ? "<1 min" : `${elapsed} min`;
+            const lastActivity = session.lastActivityAt ? Date.now() - session.lastActivityAt : 0;
+            const isStuck = lastActivity > stuckThresholdMs;
+            let statusLine = `${isStuck ? "_stuck_" : "_running_"} · ${elapsedStr}`;
+            if (session.currentTool)
+                statusLine += ` · ${session.currentTool}`;
+            if (isStuck)
+                statusLine += ` · idle ${Math.floor(lastActivity / 60000)}m`;
+            blocks.push({
+                type: "context",
+                elements: [
+                    {
+                        type: "mrkdwn",
+                        text: `*${channelName ? `#${channelName}` : channelId}* · ${statusLine}`,
+                    },
+                ],
+            });
+            if (isStuck) {
                 blocks.push({
                     type: "context",
                     elements: [
+                        { type: "mrkdwn", text: " " },
                         {
-                            type: "mrkdwn",
-                            text: `*${channelName}* · ${statusLine}`,
+                            type: "button",
+                            text: { type: "plain_text", text: "Force Stop", emoji: true },
+                            // `value` preserves the raw key; action_id is only a unique route.
+                            action_id: `force_stop_${session.sessionKey.replace(/:/g, "_")}`,
+                            value: session.sessionKey,
+                            style: "danger",
                         },
                     ],
                 });
-                // Add Force Stop button as separate element if stuck
-                if (isStuck) {
-                    blocks.push({
-                        type: "context",
-                        elements: [
-                            {
-                                type: "mrkdwn",
-                                text: " ",
-                            },
-                            {
-                                type: "button",
-                                text: { type: "plain_text", text: "Force Stop", emoji: true },
-                                action_id: `force_stop_${session.sessionKey.replace(/:/g, "_")}`,
-                                style: "danger",
-                            },
-                        ],
-                    });
-                }
             }
         }
-        // --- Cron jobs ---
-        const periodicEvents = this.eventsWatcher?.getPeriodicEvents() ?? [];
+    }
+    /**
+     * Scheduled jobs of the opener's own DM office only: the App Home is a
+     * personal surface, so other offices' schedules are not listed here.
+     */
+    appendScheduledJobs(blocks, dmChannelId) {
+        const periodicEvents = dmChannelId && this.eventScheduler
+            ? this.eventScheduler.periodicEvents(createOfficeAddress("slack", dmChannelId))
+            : [];
         blocks.push({ type: "divider" }, {
             type: "header",
             text: {
@@ -538,34 +989,46 @@ export class SlackMessagingBot {
                 type: "context",
                 elements: [{ type: "mrkdwn", text: "_No scheduled jobs._" }],
             });
+            return;
         }
-        else {
-            for (const ev of periodicEvents) {
-                const channelLabel = ev.platform === "slack"
-                    ? (() => {
-                        const channel = this.channels.get(ev.conversationId);
-                        const channelName = channel ? `#${channel.name}` : ev.conversationId;
-                        return `${ev.platform}:${channelName}`;
-                    })()
-                    : `${ev.platform}:${ev.conversationId}`;
-                const nextStr = ev.nextRun
-                    ? new Date(ev.nextRun).toLocaleString("en-US", {
-                        month: "short",
-                        day: "numeric",
-                        hour: "2-digit",
-                        minute: "2-digit",
-                    })
-                    : "—";
-                blocks.push({
-                    type: "section",
-                    text: {
-                        type: "mrkdwn",
-                        text: `*${ev.text}*\n└ \`${ev.schedule}\` · ${channelLabel} · Next: ${nextStr}`,
-                    },
-                });
-            }
+        for (const event of periodicEvents) {
+            const channel = event.platform === "slack" ? this.channels.get(event.conversationId) : undefined;
+            const channelId = channel ? `#${channel.name}` : event.conversationId;
+            const channelLabel = `${event.platform}:${channelId}`;
+            const nextRun = event.nextRun
+                ? new Date(event.nextRun).toLocaleString("en-US", {
+                    month: "short",
+                    day: "numeric",
+                    hour: "2-digit",
+                    minute: "2-digit",
+                })
+                : "—";
+            blocks.push({
+                type: "section",
+                text: {
+                    type: "mrkdwn",
+                    text: `*${event.text}*\n└ \`${event.schedule}\` · ${channelLabel} · Next: ${nextRun}`,
+                },
+            });
         }
-        // --- Footer ---
+    }
+    buildHomeView(dmChannelId) {
+        const blocks = [
+            {
+                type: "section",
+                text: {
+                    type: "mrkdwn",
+                    text: `*${PRODUCT_NAME}*\nStart a new task or check on running work.`,
+                },
+                accessory: {
+                    type: "image",
+                    image_url: "https://media1.tenor.com/m/lfDATg4Bhc0AAAAC/happy-cat.gif",
+                    alt_text: PRODUCT_NAME,
+                },
+            },
+        ];
+        this.appendRunningTasks(blocks);
+        this.appendScheduledJobs(blocks, dmChannelId);
         blocks.push({ type: "divider" }, {
             type: "context",
             elements: [
@@ -574,55 +1037,38 @@ 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 = {
+    createCommandAdapters(input) {
+        const { conversationId, userId, userName, text, ts, ...options } = input;
+        const message = createConversationMessage({
+            platform: "slack",
+            conversationId,
             id: ts,
-            sessionKey: conversationId,
+            // One session identity per dispatch: the caller's thread-resolved key.
+            // Defaulting to the top-level conversation here while the event carried
+            // a thread key made the same run answer to two different sessions.
+            sessionKey: options.sessionKey ?? conversationId,
             conversationKind: options.ephemeralChannelId ? "shared" : "direct",
             userId,
             userName,
             text,
             attachments: [],
-        };
-        const respond = async (responseText) => {
-            if (options.ephemeralChannelId) {
-                await this.postEphemeral(options.ephemeralChannelId, userId, responseText, options.threadTs);
-                return;
-            }
-            const messageTs = await this.postMessage(conversationId, responseText);
-            this.logBotResponse(conversationId, responseText, messageTs);
-        };
-        const respondMuted = async (responseText) => {
-            const blocks = [buildMrkdwnContextBlock(responseText)];
+        });
+        const respond = async (responseText, blocks) => {
             if (options.ephemeralChannelId) {
-                await this.postEphemeralBlocks(options.ephemeralChannelId, userId, responseText, blocks, options.threadTs);
+                await this.postEphemeral(options.ephemeralChannelId, userId, responseText, options.threadTs, blocks);
                 return;
             }
-            const messageTs = await this.postMessageBlocks(conversationId, responseText, blocks);
+            const messageTs = blocks
+                ? await this.postMessageBlocks(conversationId, responseText, blocks)
+                : await this.postMessage(conversationId, responseText);
             this.logBotResponse(conversationId, responseText, messageTs);
         };
         const responder = {
             respond,
-            replaceResponse: respond,
+            replaceResponse: (responseText) => respond(responseText),
             respondDiagnostic: async (responseText, responseOptions) => {
                 if (responseOptions?.style === "muted") {
-                    await respondMuted(responseText);
+                    await respond(responseText, [buildMrkdwnContextBlock(responseText)]);
                     return;
                 }
                 await respond(responseOptions?.style === "error" ? `_${responseText}_` : responseText);
@@ -639,6 +1085,7 @@ export class SlackMessagingBot {
             deleteResponse: async () => { },
         };
         return {
+            address: message.address,
             message,
             responder,
             platform: this.getMessagingInfo(),
@@ -666,9 +1113,10 @@ export class SlackMessagingBot {
             isMessagingBot: false,
             ...(threadTs ? { threadTs } : {}),
         });
-        const event = {
-            type: options.type ?? (isDirectMessage ? "dm" : "mention"),
+        const event = createConversationEvent({
+            platform: "slack",
             conversationId,
+            type: options.type ?? (isDirectMessage ? "dm" : "mention"),
             conversationKind: isDirectMessage ? "direct" : "shared",
             ts: eventTs,
             user: payload.user_id,
@@ -676,54 +1124,37 @@ export class SlackMessagingBot {
             attachments: [],
             ...(threadTs ? { thread_ts: threadTs } : {}),
             sessionKey,
-        };
-        const context = this.createCommandAdapters(conversationId, payload.user_id, userName, commandText, eventTs, isDirectMessage ? { threadTs } : { ephemeralChannelId: conversationId, threadTs });
-        return { event, context };
-    }
-    async routeSlashLoginCommand(payload) {
-        const { event, context } = this.buildSlashCommandEvent(payload, {
-            type: payload.channel_id.startsWith("D") ? "dm" : "private_command",
-            includeText: true,
         });
-        await this.handler.handleEvent(event, this, context);
-    }
-    async routeSlashNewCommand(payload) {
-        const conversationId = payload.channel_id;
-        if (!conversationId.startsWith("D")) {
-            await this.postEphemeral(conversationId, payload.user_id, `為了避免誤清除共享上下文，${payload.command} 目前只能在與 ${PRODUCT_NAME} 的私訊中使用。`);
-            return;
-        }
-        const createdAt = new Date();
-        const eventTs = (createdAt.getTime() / 1000).toFixed(6);
-        const userName = payload.user_name ?? this.getUser(payload.user_id)?.userName;
-        this.logToFile(conversationId, {
-            date: createdAt.toISOString(),
-            ts: eventTs,
-            user: payload.user_id,
+        const context = this.createCommandAdapters({
+            conversationId,
+            userId: payload.user_id,
             userName,
-            text: payload.command,
-            attachments: [],
-            isMessagingBot: false,
+            text: commandText,
+            ts: eventTs,
+            ...(isDirectMessage
+                ? { threadTs, sessionKey }
+                : { ephemeralChannelId: conversationId, threadTs, sessionKey }),
         });
-        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),
-            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);
+        return { event, context };
     }
-    async routeSlashAdminCommand(payload) {
-        const { event, context } = this.buildSlashCommandEvent(payload, { thread: true });
+    /**
+     * Generic slash-command route: synthesize the command text per the
+     * manifest's SlackSlashRoute data and dispatch to the handler. Every
+     * routed command — including `/pi-new` — reaches its CommandHandler
+     * through runtime dispatch; the adapter holds no per-command behavior.
+     */
+    async routeSlashCommand(route, payload) {
+        const { event, context } = this.buildSlashCommandEvent(payload, {
+            includeText: route.includeText,
+            thread: route.thread,
+            ...(route.privateCommand
+                ? {
+                    type: payload.channel_id.startsWith("D")
+                        ? "dm"
+                        : "private_command",
+                }
+                : {}),
+        });
         await this.handler.handleEvent(event, this, context);
     }
     setupEventHandlers() {
@@ -736,14 +1167,39 @@ export class SlackMessagingBot {
         this.socketClient.on("unable_to_socket_mode_start", (err) => {
             log.logWarning("Slack socket unable_to_start", err ? String(err) : "");
         });
-        this.socketClient.on("app_mention", (payload) => this.handleAppMention(payload));
-        this.socketClient.on("message", (payload) => this.handleMessageEvent(payload));
-        this.socketClient.on("slash_commands", (payload) => void this.handleSlashCommand(payload));
-        this.socketClient.on("app_home_opened", (payload) => this.handleAppHomeOpened(payload));
-        this.socketClient.on("block_actions", (payload) => void this.handleBlockAction(payload));
-        this.socketClient.on("interactive", (payload) => void this.handleBlockAction(payload));
-    }
-    handleAppMention({ event, ack }) {
+        this.socketClient.on("app_mention", (payload) => {
+            void this.intake.run(() => this.handleAppMention(payload));
+        });
+        this.socketClient.on("message", (payload) => {
+            void this.intake.run(() => this.handleMessageEvent(payload));
+        });
+        this.socketClient.on("slash_commands", (payload) => {
+            void this.intake.run(() => this.handleSlashCommand(payload));
+        });
+        this.socketClient.on("app_home_opened", (payload) => {
+            void this.intake.run(() => this.handleAppHomeOpened(payload));
+        });
+        // The assistant surface's lifecycle. Both events were already subscribed
+        // in the app manifest and then dropped on the floor, which is why the pane
+        // opened empty and the sidebar filled with untitled conversations.
+        this.socketClient.on("assistant_thread_started", (payload) => {
+            void this.intake.run(() => this.handleAssistantThreadStarted(payload));
+        });
+        this.socketClient.on("assistant_thread_context_changed", (payload) => {
+            void this.intake.run(() => this.handleAgentContextChangedEvent(payload));
+        });
+        // agent_view's spelling of the same signal.
+        this.socketClient.on("app_context_changed", (payload) => {
+            void this.intake.run(() => this.handleAgentContextChangedEvent(payload));
+        });
+        this.socketClient.on("block_actions", (payload) => {
+            void this.intake.run(() => this.handleBlockAction(payload));
+        });
+        this.socketClient.on("interactive", (payload) => {
+            void this.intake.run(() => this.handleBlockAction(payload));
+        });
+    }
+    async handleAppMention({ event, ack, }) {
         const e = event;
         // Skip DMs (handled by message event)
         if (e.channel.startsWith("D")) {
@@ -754,18 +1210,19 @@ export class SlackMessagingBot {
         // Thread replies get their own isolated session (channelId:thread_ts).
         const sessionKey = resolveSlackSessionKey(e.channel, e.thread_ts);
         const mentionText = this.stripOwnMention(e.text);
-        const slackEvent = {
-            type: "mention",
+        const slackEvent = createConversationEvent({
+            platform: "slack",
             conversationId: e.channel,
+            type: "mention",
             conversationKind: "shared",
-            channel: e.channel,
             ts: e.ts,
             thread_ts: e.thread_ts,
             user: e.user,
             text: mentionText || "Please respond to the recent conversation context.",
-            files: e.files,
             sessionKey,
-        };
+            channel: e.channel,
+            files: e.files,
+        });
         const attachmentsPromise = this.logUserMessage(slackEvent);
         // Only trigger processing for messages AFTER startup (not replayed old messages)
         if (this.startupTs && e.ts < this.startupTs) {
@@ -776,67 +1233,162 @@ 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({
+        const intake = this.processSlackMessageIntake({
             event: slackEvent,
             attachmentsPromise,
             queueKey: this.resolveQueueKey(e.channel, sessionKey),
-            isAutoReplyCandidate: false,
+            addressed: true,
         });
         ack();
+        await intake;
     }
-    handleMessageEvent({ event, ack }) {
-        const e = event;
-        const hasFiles = !!e.files && e.files.length > 0;
-        const hasSlackContent = !!e.text || hasFiles || !!e.blocks?.length || !!e.attachments?.length;
-        const isOwnMessagingBotMessage = (!!e.user && e.user === this.botUserId) || (!!this.botId && e.bot_id === this.botId);
-        if (isOwnMessagingBotMessage) {
+    admitHumanMessage(event, ack) {
+        const hasSlackContent = hasMessageContent(event);
+        const isOwnMessage = event.user === this.botUserId || (!!this.botId && event.bot_id === this.botId);
+        if (isOwnMessage) {
             ack();
-            return;
+            return false;
         }
-        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;
-            }
-            if (!hasSlackContent) {
-                ack();
-                return;
+        // User-token posts carry bot_id/app_id beside the human user. Only unknown
+        // authors and known bot users take the external-bot loop-protection path.
+        const authorIsKnownHuman = !!event.user && this.users.get(event.user)?.isBot === false;
+        const isExternalMessage = event.subtype === "bot_message" || (!!event.bot_id && !authorIsKnownHuman);
+        if (isExternalMessage) {
+            const supportedSubtype = BOT_MESSAGE_SUBTYPES.has(event.subtype);
+            if (supportedSubtype && hasSlackContent) {
+                void this.logExternalMessagingBotMessage(event).catch((err) => {
+                    log.logWarning("Failed to log Slack bot message", String(err));
+                });
             }
-            void this.logExternalMessagingBotMessage(e).catch((err) => {
-                log.logWarning("Failed to log Slack bot message", String(err));
-            });
             ack();
-            return;
+            return false;
         }
-        if (!e.user) {
-            ack();
-            return;
+        const isSupportedUserMessage = !!event.user && USER_MESSAGE_SUBTYPES.has(event.subtype) && hasSlackContent;
+        if (isSupportedUserMessage)
+            return true;
+        ack();
+        return false;
+    }
+    /**
+     * Handle a DM message that may be about a background task without spending
+     * a model turn: a pure status question is answered from observation, and a
+     * supplement is steered into the running task. Jev decides which (falling
+     * back to the regex status shortcut); anything else returns false so the
+     * message becomes an ordinary conversation turn.
+     */
+    async deliverTaskUpdate(event, attachmentsPromise) {
+        if (matchMagicWord(event.text) === "stop" || event.text.startsWith("/"))
+            return false;
+        event.attachments = await attachmentsPromise;
+        const context = this.createContext(event);
+        const post = (text) => event.thread_ts
+            ? this.postInThread(event.channel, event.thread_ts, text)
+            : this.postMessage(event.channel, text);
+        try {
+            const tasks = await context.responder.getTaskStatus(event.thread_ts ? event.sessionKey : undefined);
+            if (!tasks.length)
+                return false;
+            const active = tasks.filter((t) => t.status === "running" || t.status === "stopping");
+            const user = this.users.get(event.user);
+            // A message carrying files is never a pure status question; in a task
+            // thread it is a supplement, at top level an ordinary turn.
+            const intent = event.attachments?.length
+                ? event.thread_ts
+                    ? "steer"
+                    : "request"
+                : await classifyTaskIntent(buildTaskIntentState(this.conversationDir(event.channel), event, {
+                    speaker: user?.displayName ?? user?.userName,
+                    tasks,
+                    resolveName: (id) => this.users.get(id)?.displayName ?? this.users.get(id)?.userName,
+                    botUserId: this.botUserId,
+                }), event.text, { conversationId: event.channel, inTaskThread: !!event.thread_ts });
+            if (intent === "status") {
+                const selection = event.thread_ts ? tasks : active.length ? active : tasks.slice(0, 1);
+                const text = selection.length > 1
+                    ? "目前有多個任務在處理，請到你想查詢的任務對話串詢問，避免弄錯。"
+                    : formatTaskStatus(selection);
+                const ts = await post(text);
+                this.logBotResponse(event.channel, text, ts, event.thread_ts);
+                return true;
+            }
+            if (intent === "steer") {
+                // In a thread the message already targets that task; at top level it
+                // can only be steered when exactly one task is running.
+                const target = event.thread_ts
+                    ? context.message
+                    : active.length === 1
+                        ? {
+                            ...context.message,
+                            sessionKey: active[0].sessionKey,
+                            threadTs: active[0].threadTs,
+                        }
+                        : undefined;
+                if (target && (await this.handler.steer?.(target))) {
+                    const text = "收到補充，會在下一個處理步驟納入；目前的操作不會立即中斷。";
+                    const ts = await post(text);
+                    this.logBotResponse(event.channel, text, ts, event.thread_ts);
+                    return true;
+                }
+            }
         }
-        if (e.subtype !== undefined && e.subtype !== "file_share") {
-            ack();
-            return;
+        catch (error) {
+            await post(error instanceof Error ? error.message : "Could not deliver task update.");
+            return true;
         }
-        if (!hasSlackContent) {
-            ack();
-            return;
+        return false;
+    }
+    /**
+     * Ask Jev a single typed yes/no question about whether this unaddressed
+     * shared-channel message (top-level or thread reply) addresses mikan. The
+     * scored state carries the surrounding scope from log.jsonl, since a bare
+     * thread reply cannot be judged from its own text. Only called in `jev`
+     * auto-reply mode, and only awaited by the caller in that mode, so
+     * `on`/`off` stay synchronous (magic-word `stop` handling depends on that).
+     * Any failure (including a missing `OPENROUTER_API_KEY`) fails closed to
+     * "not addressed" so a misconfiguration cannot make the bot noisy in a
+     * shared channel.
+     */
+    async evaluateJevAddressed(event) {
+        if (!event.text.trim() || matchMagicWord(event.text) === "stop")
+            return false;
+        try {
+            const user = this.users.get(event.user);
+            const state = buildAutoReplyState(this.conversationDir(event.channel), event, {
+                speaker: user?.displayName ?? user?.userName,
+                resolveName: (id) => this.users.get(id)?.displayName ?? this.users.get(id)?.userName,
+                botUserId: this.botUserId,
+            });
+            const result = await evaluateWithJev(state, {
+                addressed: { type: "boolean", instructions: JEV_ADDRESSED_INSTRUCTIONS },
+            });
+            const probability = result.answers.addressed.probability;
+            log.logInfo(`[${event.channel}] jev auto-reply: probability=${probability.toFixed(2)} addressed=${probability > 0.5} text="${event.text.slice(0, 80)}"`);
+            return probability > 0.5;
+        }
+        catch (err) {
+            if (err instanceof JevNotConfiguredError) {
+                log.logWarning("Slack auto-reply jev mode requires OPENROUTER_API_KEY; treating message as unaddressed", String(err));
+                return false;
+            }
+            reportUserFacingError(err, {
+                domain: "chat_platform",
+                surface: "slack_auto_reply_jev",
+                operation: "evaluate",
+                severity: "warning",
+                context: { conversationId: event.conversationId },
+            });
+            return false;
         }
-        const isDM = e.channel_type === "im";
+    }
+    async handleMessageEvent({ event, ack, }) {
+        const e = event;
+        if (!this.admitHumanMessage(e, ack))
+            return;
+        // 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 classify a DM as an unaddressed
+        // channel message 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
@@ -844,20 +1396,26 @@ export class SlackMessagingBot {
             ack();
             return;
         }
-        const isThreadReply = !!e.thread_ts;
         const sessionKey = isDM ? resolveSlackSessionKey(e.channel, e.thread_ts) : undefined;
-        const slackEvent = {
-            type: isDM ? "dm" : "mention",
+        // Name the conversation from the first thing the person said. Guarded on
+        // the assistant registry, so classic DM threads are untouched, and it
+        // claims the title once per thread.
+        if (isDM && e.thread_ts && e.text) {
+            void titleAssistantThread(this.assistantOps(), this.assistantThreads, e.channel, e.thread_ts, this.stripOwnMention(e.text));
+        }
+        const slackEvent = createConversationEvent({
+            platform: "slack",
             conversationId: e.channel,
+            type: (isDM ? "dm" : "mention"),
             conversationKind,
-            channel: e.channel,
             ts: e.ts,
             thread_ts: e.thread_ts,
             user: e.user,
             text: this.stripOwnMention(e.text),
-            files: e.files,
             sessionKey,
-        };
+            channel: e.channel,
+            files: e.files,
+        });
         const attachmentsPromise = this.logUserMessage(slackEvent);
         // Only trigger processing for messages AFTER startup (not replayed old messages)
         if (this.startupTs && e.ts < this.startupTs) {
@@ -868,61 +1426,42 @@ 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"));
-            }
-            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();
+        const activeSessionKey = slackEvent.sessionKey ?? resolveSlackSessionKey(e.channel, e.thread_ts);
+        slackEvent.sessionKey = activeSessionKey;
+        // Task threads always go through the task gate. Top-level DMs do only
+        // while a task is running (so supplements can be steered) or when the
+        // regex already recognises a status question about a finished one, so
+        // idle DMs never pay a Jev round-trip.
+        const taskControl = isDM &&
+            (e.thread_ts
+                ? this.isTaskThread(e.channel, e.thread_ts)
+                : this.hasRunningTaskThread(e.channel) || isTaskStatusQuestion(slackEvent.text));
+        if (taskControl) {
             ack();
-            return;
+            if (await this.deliverTaskUpdate(slackEvent, attachmentsPromise))
+                return;
         }
-        const activeSessionKey = resolveSlackSessionKey(e.channel, e.thread_ts);
-        slackEvent.sessionKey = activeSessionKey;
-        this.processSlackMessageIntake({
+        // Shared-channel messages without a mention, top-level or in a thread,
+        // all go through the same auto-reply gate; jev mode sees the surrounding
+        // scope so thread follow-ups are judged in context rather than dropped.
+        const autoReplyMode = isDM
+            ? "off"
+            : slackConversationAutoReplyMode(this.workspace.office(slackEvent.address));
+        const autoReply = autoReplyMode === "on"
+            ? true
+            : autoReplyMode === "jev"
+                ? await this.evaluateJevAddressed(slackEvent)
+                : false;
+        const intake = this.processSlackMessageIntake({
             event: slackEvent,
             attachmentsPromise,
             queueKey: this.resolveQueueKey(e.channel, activeSessionKey),
-            isAutoReplyCandidate: true,
-            onNotTriggered: logOnly,
+            addressed: isDM || autoReply,
+            magicWordAddressed: isDM,
         });
-        ack();
+        if (!taskControl)
+            ack();
+        await intake;
     }
     async handleSlashCommand({ body, ack, }) {
         const payload = body;
@@ -931,73 +1470,44 @@ export class SlackMessagingBot {
             return;
         }
         const { command, text, channel_id, user_id, user_name, thread_ts } = payload;
-        let handlerPromise = null;
-        if (command === "/pi-login") {
-            handlerPromise = this.routeSlashLoginCommand({
+        const entry = COMMAND_MANIFEST.find((candidate) => candidate.slackCommand === command);
+        if (!entry)
+            return;
+        if (!entry.slackRoute)
+            return;
+        try {
+            await this.routeSlashCommand(entry.slackRoute, {
                 command,
                 text,
                 channel_id,
                 user_id,
                 user_name,
-            });
-        }
-        else if (command === "/pi-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({
-                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,
-            });
-        }
-        if (!handlerPromise) {
-            return;
-        }
-        handlerPromise.catch((err) => {
+        catch (err) {
             log.logWarning("Slack slash command error", err instanceof Error ? err.message : String(err));
-        });
+        }
     }
     handleAppHomeOpened({ event, ack }) {
         const e = event;
         ack();
+        // Under agent_view this is how Slack says "the person opened the app's
+        // DM" — the signal assistant_thread_started used to carry. It fires on
+        // every open, not once per conversation, so this path refreshes the
+        // pinned prompts and deliberately does not greet.
+        if (e.tab === "messages") {
+            if (e.channel) {
+                void handleAgentDmOpened(this.assistantOps(), this.assistantThreads, e.channel, e.context);
+            }
+            return;
+        }
         if (e.tab !== "home")
             return;
         this.webClient.views
             .publish({
             user_id: e.user,
-            view: this.buildHomeView(),
+            view: this.buildHomeView(e.channel),
         })
             .catch((err) => {
             log.logWarning(`Failed to publish App Home view`, String(err));
@@ -1015,21 +1525,24 @@ 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);
+        this.handler.forceStop(createOfficeAddress("slack", channelId), sessionKey);
         // Notify in channel
         await this.postMessage(channelId, formatForceStopped("slack", userId ?? "unknown"));
         // Refresh home tab
         if (userId) {
-            this.webClient.views
-                .publish({
+            this.openDirectConversation(userId)
+                .then((dmChannelId) => this.webClient.views.publish({
                 user_id: userId,
-                view: this.buildHomeView(),
-            })
+                view: this.buildHomeView(dmChannelId),
+            }))
                 .catch((err) => {
                 log.logWarning(`Failed to refresh App Home view`, String(err));
             });
@@ -1045,6 +1558,8 @@ export class SlackMessagingBot {
         const selectedOptions = Array.isArray(action.selected_options)
             ? action.selected_options
             : undefined;
+        const threadTs = container.thread_ts;
+        const sessionKey = resolveSlackSessionKey(channelId, threadTs);
         const selectedText = selectedOption?.text?.text ?? selectedOption?.value;
         const selectedTexts = selectedOptions?.map((option) => option.text?.text ?? option.value);
         const valueText = selectedTexts?.length
@@ -1052,8 +1567,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,
@@ -1080,9 +1593,10 @@ export class SlackMessagingBot {
                 messageTs: container.message_ts,
             },
         });
-        const event = {
-            type: "slack_action",
+        const event = createConversationEvent({
+            platform: "slack",
             conversationId: channelId,
+            type: "slack_action",
             conversationKind: channelId.startsWith("D") ? "direct" : "shared",
             ts,
             user: userId,
@@ -1090,23 +1604,71 @@ export class SlackMessagingBot {
             attachments: [],
             ...(threadTs ? { thread_ts: threadTs } : {}),
             sessionKey,
-        };
+        });
         this.getQueue(this.resolveQueueKey(channelId, sessionKey)).enqueue(async () => {
             const slackEvent = {
-                type: event.conversationKind === "direct" ? "dm" : "mention",
-                conversationId: channelId,
-                conversationKind: event.conversationKind,
+                ...createConversationEvent({
+                    platform: "slack",
+                    conversationId: channelId,
+                    type: (event.conversationKind === "direct" ? "dm" : "mention"),
+                    conversationKind: event.conversationKind,
+                    ts,
+                    thread_ts: threadTs,
+                    user: userId,
+                    text,
+                    attachments: [],
+                    sessionKey,
+                }),
                 channel: channelId,
-                ts,
-                thread_ts: threadTs,
-                user: userId,
-                text,
                 attachments: [],
-                sessionKey,
             };
             return this.handler.handleEvent(event, this, this.createContext(slackEvent));
         });
     }
+    /**
+     * Download Slack message files into the office's attachments directory.
+     * The Slack contract is all-or-error: any failed download throws after the
+     * successful ones are on disk, so callers still log the message text.
+     */
+    async processAttachments(channelId, files, timestamp) {
+        const items = [];
+        for (const file of files) {
+            const url = file.url_private_download || file.url_private;
+            if (!url)
+                continue;
+            if (!file.name) {
+                throw new Error(`Attachment missing name for URL: ${url}`);
+            }
+            items.push({
+                name: file.name,
+                // Slack timestamps are float seconds; the stored filename uses ms.
+                timestampMs: Math.floor(parseFloat(timestamp) * 1000),
+                download: (destPath) => this.downloadSlackFile(url, destPath),
+            });
+        }
+        const office = this.workspace.office(createOfficeAddress("slack", channelId));
+        const { saved, failed } = await saveIncomingAttachments(office, items);
+        const firstFailure = failed[0];
+        if (firstFailure) {
+            const { name, error } = firstFailure;
+            const errorMsg = error instanceof Error ? error.message : String(error);
+            throw new Error(`Failed to download attachment ${name}: ${errorMsg}`, { cause: error });
+        }
+        return saved;
+    }
+    /** Authorized download with retry; Slack file URLs require the bot token. */
+    async downloadSlackFile(url, destPath) {
+        await withRetry(async () => {
+            const response = await fetch(url, {
+                headers: { Authorization: `Bearer ${this.botToken}` },
+            });
+            if (!response.ok) {
+                throw new AttachmentDownloadHttpError(`HTTP ${response.status}: ${response.statusText}`, response.status);
+            }
+            const buffer = await response.arrayBuffer();
+            await writeFile(destPath, Buffer.from(buffer));
+        }, { maxAttempts: 3, baseDelayMs: 250, isRateLimited: isRetryableAttachmentDownloadError });
+    }
     /**
      * Log a user message to log.jsonl after attachments are ready.
      */
@@ -1116,7 +1678,7 @@ export class SlackMessagingBot {
         let attachmentError;
         if (event.files) {
             try {
-                attachments = await this.store.processAttachments(event.channel, event.files, event.ts);
+                attachments = await this.processAttachments(event.channel, event.files, event.ts);
             }
             catch (err) {
                 attachmentError = err;
@@ -1141,7 +1703,7 @@ export class SlackMessagingBot {
     }
     async logExternalMessagingBotMessage(event) {
         const attachments = event.files
-            ? await this.store.processAttachments(event.channel, event.files, event.ts)
+            ? await this.processAttachments(event.channel, event.files, event.ts)
             : [];
         const botName = event.username ?? event.bot_profile?.name ?? event.bot_profile?.real_name ?? event.bot_id;
         this.logToFile(event.channel, {
@@ -1164,7 +1726,7 @@ export class SlackMessagingBot {
     // Private - Backfill
     // ==========================================================================
     async getExistingTimestamps(channelId) {
-        const logPath = join(this.workingDir, channelId, "log.jsonl");
+        const logPath = join(this.conversationDir(channelId), "log.jsonl");
         const timestamps = new Set();
         if (!existsSync(logPath))
             return timestamps;
@@ -1172,7 +1734,7 @@ export class SlackMessagingBot {
         const lines = content.trim().split("\n").filter(Boolean);
         for (let i = 0; i < lines.length; i++) {
             try {
-                const entry = JSON.parse(lines[i]);
+                const entry = JSON.parse(lines[i] ?? "");
                 if (entry.ts)
                     timestamps.add(entry.ts);
             }
@@ -1182,6 +1744,19 @@ export class SlackMessagingBot {
         }
         return timestamps;
     }
+    isBackfillableMessage(message) {
+        if (message.user === this.botUserId)
+            return true;
+        if (hasBotIdentity(message)) {
+            if (this.botId && message.bot_id === this.botId)
+                return false;
+            return BOT_MESSAGE_SUBTYPES.has(message.subtype) && hasMessageContent(message);
+        }
+        // Unlike live intake, historical human posts need text or files, not just blocks.
+        return (!!message.user &&
+            USER_MESSAGE_SUBTYPES.has(message.subtype) &&
+            hasMessageContent(message, false));
+    }
     async backfillChannel(channelId, upperBoundTs) {
         const existingTs = await this.getExistingTimestamps(channelId);
         // Find the biggest ts in log.jsonl
@@ -1213,36 +1788,14 @@ export class SlackMessagingBot {
         const relevantMessages = allMessages.filter((msg) => {
             if (!msg.ts || existingTs.has(msg.ts))
                 return false; // Skip duplicates
-            if (msg.user === this.botUserId)
-                return true;
-            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 &&
-                    msg.subtype !== "bot_message" &&
-                    msg.subtype !== "file_share") {
-                    return false;
-                }
-                return (!!msg.text ||
-                    !!(msg.files && msg.files.length > 0) ||
-                    !!msg.blocks?.length ||
-                    !!msg.attachments?.length);
-            }
-            if (msg.subtype !== undefined && msg.subtype !== "file_share")
-                return false;
-            if (!msg.user)
-                return false;
-            if (!msg.text && (!msg.files || msg.files.length === 0))
-                return false;
-            return true;
+            return this.isBackfillableMessage(msg);
         });
         // Reverse to chronological order
         relevantMessages.reverse();
         // Log each message to log.jsonl
         for (const msg of relevantMessages) {
             const isMikanMessage = msg.user === this.botUserId;
-            const isExternalMessagingBotMessage = !isMikanMessage && (!!msg.bot_id || msg.subtype === "bot_message");
+            const isExternalMessagingBotMessage = !isMikanMessage && hasBotIdentity(msg);
             if (isExternalMessagingBotMessage) {
                 await this.logExternalMessagingBotMessage({ ...msg, channel: channelId, ts: msg.ts });
                 continue;
@@ -1250,7 +1803,7 @@ export class SlackMessagingBot {
             const user = this.users.get(msg.user);
             const text = this.stripOwnMention(msg.text);
             const attachments = msg.files
-                ? await this.store.processAttachments(channelId, msg.files, msg.ts)
+                ? await this.processAttachments(channelId, msg.files, msg.ts)
                 : [];
             this.logToFile(channelId, {
                 date: new Date(parseFloat(msg.ts) * 1000).toISOString(),
@@ -1271,7 +1824,7 @@ export class SlackMessagingBot {
         // Only backfill channels that already have a log.jsonl (mikan has interacted with them before)
         const channelsToBackfill = [];
         for (const [channelId, channel] of this.channels) {
-            const logPath = join(this.workingDir, channelId, "log.jsonl");
+            const logPath = join(this.conversationDir(channelId), "log.jsonl");
             if (existsSync(logPath)) {
                 channelsToBackfill.push([channelId, channel]);
             }
@@ -1289,7 +1842,7 @@ export class SlackMessagingBot {
                 log.logWarning(`Failed to backfill #${channel.name}`, String(error));
             }
             // Add delay between channels to avoid hitting Slack rate limits
-            if (channelId !== channelsToBackfill[channelsToBackfill.length - 1][0]) {
+            if (channelId !== channelsToBackfill[channelsToBackfill.length - 1]?.[0]) {
                 await new Promise((resolve) => setTimeout(resolve, 500));
             }
         }
@@ -1304,16 +1857,15 @@ export class SlackMessagingBot {
         do {
             const result = await this.webClient.users.list({ limit: 200, cursor });
             const members = result.members;
-            if (members) {
-                for (const u of members) {
-                    if (u.id && u.name && !u.deleted) {
-                        this.users.set(u.id, {
-                            id: u.id,
-                            userName: u.name,
-                            displayName: u.real_name || u.name,
-                        });
-                    }
-                }
+            for (const u of members ?? []) {
+                if (!u.id || !u.name || u.deleted)
+                    continue;
+                this.users.set(u.id, {
+                    id: u.id,
+                    userName: u.name,
+                    displayName: u.real_name || u.name,
+                    isBot: !!u.is_bot,
+                });
             }
             cursor = result.response_metadata?.next_cursor;
         } while (cursor);
@@ -1329,12 +1881,15 @@ export class SlackMessagingBot {
                 cursor,
             });
             const channels = result.channels;
-            if (channels) {
-                for (const c of channels) {
-                    if (c.id && c.name && c.is_member) {
-                        this.channels.set(c.id, { id: c.id, name: c.name });
-                    }
-                }
+            for (const c of channels ?? []) {
+                if (!c.id || !c.name || !c.is_member)
+                    continue;
+                this.channels.set(c.id, {
+                    id: c.id,
+                    name: c.name,
+                    ...(typeof c.is_private === "boolean" ? { isPrivate: c.is_private } : {}),
+                    ...(c.is_shared || c.is_ext_shared ? { isExternallyShared: true } : {}),
+                });
             }
             cursor = result.response_metadata?.next_cursor;
         } while (cursor);
@@ -1347,15 +1902,13 @@ export class SlackMessagingBot {
                 cursor,
             });
             const ims = result.channels;
-            if (ims) {
-                for (const im of ims) {
-                    if (im.id) {
-                        // Use user's name as channel name for DMs
-                        const user = im.user ? this.users.get(im.user) : undefined;
-                        const name = user ? `DM:${user.userName}` : `DM:${im.id}`;
-                        this.channels.set(im.id, { id: im.id, name });
-                    }
-                }
+            for (const im of ims ?? []) {
+                if (!im.id)
+                    continue;
+                // Use user's name as channel name for DMs
+                const user = im.user ? this.users.get(im.user) : undefined;
+                const name = user ? `DM:${user.userName}` : `DM:${im.id}`;
+                this.channels.set(im.id, { id: im.id, name });
             }
             cursor = result.response_metadata?.next_cursor;
         } while (cursor);