@zhin.js/adapter-slack 4.0.1 → 4.1.0

package/src/endpoint.ts

@@ -1,227 +1,126 @@
 /**
- * Slack Endpoint 实现
+ * Slack Endpoint — 双传输（Socket Mode / HTTP）
  */
-import { App as SlackApp, LogLevel } from "@slack/bolt";
-import { WebClient, type ChatPostMessageArguments } from "@slack/web-api";
-import { formatCompact, Endpoint, Message, MessageSegment, segment, SendContent, SendOptions, expandInteractiveSegmentsInContent,} from 'zhin.js';
-import type { SlackEndpointConfig, SlackMessageEvent } from "./types.js";
-import type { SlackAdapter } from "./adapter.js";
-import { normalizeSlackSenderForPermit } from "./platform-permit.js";
-import { fromCanonicalSegments, toCanonicalSegments } from './segment-mapper.js';
-
+import { WebClient } from '@slack/web-api';
+import { formatCompact, Endpoint, Message, segment, type SendOptions, type EditMessageOptions, expandInteractiveSegmentsInContent } from 'zhin.js';
+import type { SlackEndpointConfig, SlackMessageEvent } from './types.js';
+import type { SlackAdapter } from './adapter.js';
+import { toCanonicalSegments, fromCanonicalSegments } from './segment-mapper.js';
+import { parseSlackMessageToSegments, resolveSlackChannelType } from './slack-inbound.js';
+import { sendSlackContent, editSlackContent } from './slack-outbound.js';
+import { normalizeSlackReactionName } from './slack-reaction.js';
+import { formatSlackMessageRef, parseSlackMessageRef } from './slack-message-ref.js';
+import { SlackEventDispatcher } from './event-dispatcher.js';
+import { SlackSocketTransport } from './transport-socket.js';
+import { SlackHttpTransport, type RouterLike } from './transport-http.js';
 
 export class SlackEndpoint implements Endpoint<SlackEndpointConfig, SlackMessageEvent> {
   $connected: boolean;
-  /** 延迟到 `$connect`，避免子类 mock `$connect` 时仍在构造函数里创建 Bolt/WebClient（会在后台触发 Slack API） */
-  private app?: SlackApp;
-  private client?: WebClient;
+  /** Slack Bot User ID（auth.test.user_id），用于 @ 触发 AI */
+  $platformUserId?: string;
+  client?: WebClient;
+  /** message ts → channel id（Activity Feedback reaction / recall 定位频道） */
+  readonly messageChannelMap = new Map<string, string>();
+  senderPermitCache = new Map<string, { at: number; role?: string; permissions: string[] }>();
 
-  get logger() {
-    return this.adapter.plugin.logger;
-  }
+  private dispatcher?: SlackEventDispatcher;
+  private socketTransport?: SlackSocketTransport;
+  private httpTransport?: SlackHttpTransport;
 
-  get $id() {
-    return this.$config.name;
-  }
+  get logger() { return this.adapter.plugin.logger; }
+  get $id() { return this.$config.name; }
 
   constructor(public adapter: SlackAdapter, public $config: SlackEndpointConfig) {
     this.$connected = false;
   }
 
-  #ensureSlackRuntime(): void {
-    if (this.app && this.client) return;
-    const $config = this.$config;
-    if ($config.socketMode && $config.appToken) {
-      this.app = new SlackApp({
-        token: $config.token,
-        signingSecret: $config.signingSecret,
-        appToken: $config.appToken,
-        socketMode: true,
-        logLevel: $config.logLevel || LogLevel.INFO,
-      });
-    } else {
-      this.app = new SlackApp({
-        token: $config.token,
-        signingSecret: $config.signingSecret,
-        socketMode: false,
-        logLevel: $config.logLevel || LogLevel.INFO,
-      });
-    }
-    this.client = new WebClient($config.token);
-  }
-
-  async $connect(): Promise<void> {
-    this.#ensureSlackRuntime();
+  async $connect(router?: RouterLike): Promise<void> {
     try {
-      // Set up message event handler
-      this.app!.message(async ({ message, say }) => {
-        await this.handleSlackMessage(message as SlackMessageEvent);
-      });
-
-      // Set up app mention handler
-      this.app!.event("app_mention", async ({ event, say }) => {
-        await this.handleSlackMessage(event as any);
-      });
-
-      // Start the app
-      const port = this.$config.port || 3000;
+      this.client = new WebClient(this.$config.token);
+      this.dispatcher = new SlackEventDispatcher(this);
+
       if (this.$config.socketMode) {
-        await this.app!.start();
-      } else {
-        await this.app!.start(port);
+        this.socketTransport = new SlackSocketTransport(this.$config, this.dispatcher, this.logger);
+        await this.socketTransport.connect();
+      } else if (router) {
+        this.httpTransport = new SlackHttpTransport(this.$config, this.dispatcher, this.logger);
+        this.httpTransport.registerRoutes(router);
       }
 
       this.$connected = true;
 
-      // Get bot info
-      const authTest = await this.client!.auth.test();
-      this.logger.info(formatCompact({ endpoint: this.$config.name, user: authTest.user }));
-
-      if (!this.$config.socketMode) {
-        this.logger.info(formatCompact( { op: "listen", port }));
+      const authTest = await this.client.auth.test();
+      if (authTest.user_id) {
+        this.$platformUserId = String(authTest.user_id);
+      }
+      this.logger.debug(formatCompact({
+        endpoint: this.$config.name,
+        user: authTest.user,
+        platform_user_id: this.$platformUserId,
+      }));
+
+      if (!this.$config.socketMode && router) {
+        this.logger.debug(formatCompact({ op: 'listen', mode: 'http', path: '/slack/events' }));
       }
     } catch (error) {
-      this.logger.error("Failed to connect Slack bot:", error);
+      this.logger.error('Failed to connect Slack bot:', error);
       this.$connected = false;
       throw error;
     }
   }
 
   async $disconnect(): Promise<void> {
-    if (!this.app) {
-      this.$connected = false;
-      return;
-    }
     try {
-      await this.app!.stop();
+      if (this.socketTransport) {
+        await this.socketTransport.disconnect();
+        this.socketTransport = undefined;
+      }
+      this.httpTransport = undefined;
       this.$connected = false;
-      this.logger.info(formatCompact( { op: "disconnect", endpoint: this.$config.name }));
+      this.logger.debug(formatCompact({ op: 'disconnect', endpoint: this.$config.name }));
     } catch (error) {
-      this.logger.error("Error disconnecting Slack bot:", error);
+      this.logger.error('Error disconnecting Slack bot:', error);
       throw error;
     }
   }
 
-  private senderPermitCache = new Map<string, { at: number; role?: string; permissions: string[] }>();
-
-  private async enrichChannelSender(
-    message: Message<SlackMessageEvent>,
-    msg: SlackMessageEvent,
-  ): Promise<void> {
-    if (message.$channel.type !== "group" || !("user" in msg) || !msg.user) return;
-    const channelId = msg.channel;
-    const userId = msg.user;
-    const key = `${channelId}:${userId}`;
-    const now = Date.now();
-    const cached = this.senderPermitCache.get(key);
-    if (cached && now - cached.at < 60_000) {
-      message.$sender.role = cached.role;
-      message.$sender.permissions = cached.permissions;
-      return;
-    }
-    try {
-      const user = await this.getUserInfo(userId);
-      let isChannelManager = false;
-      try {
-        const channel = await this.getChannelInfo(channelId);
-        if (channel?.creator === userId) isChannelManager = true;
-      } catch {
-        // ignore
-      }
-      const normalized = normalizeSlackSenderForPermit({
-        isWorkspaceOwner: user.is_owner === true,
-        isWorkspaceAdmin: user.is_admin === true,
-        isChannelManager,
-      });
-      const entry = {
-        at: now,
-        role: normalized.role,
-        permissions: normalized.permissions ?? [],
-      };
-      this.senderPermitCache.set(key, entry);
-      message.$sender.role = entry.role;
-      message.$sender.permissions = entry.permissions;
-    } catch {
-      // 保守拒绝
-    }
-  }
-
-  private async handleSlackMessage(msg: SlackMessageEvent): Promise<void> {
-    // Ignore bot messages and message changes
-    if ("subtype" in msg && (msg.subtype === "bot_message" || msg.subtype === "message_changed")) {
-      return;
-    }
-
-    const message = this.$formatMessage(msg);
-    await this.enrichChannelSender(message, msg);
-    this.adapter.emit("message.receive", message);
-    this.logger.debug(
-      `${this.$config.name} recv  ${message.$channel.type}(${message.$channel.id}): ${segment.raw(message.$content)}`
-    );
-  }
-
   $formatMessage(msg: SlackMessageEvent): Message<SlackMessageEvent> {
-    // Determine channel type based on channel ID
-    const channelType = "channel_type" in msg && msg.channel_type === "im" ? "private" : "group";
+    const channelType = resolveSlackChannelType(msg);
     const channelId = msg.channel;
-
-    // Parse message content
-    const wire = this.parseMessageContent(msg);
+    this.trackMessageChannel(msg.ts, channelId);
+    const wire = parseSlackMessageToSegments(msg);
     const content = toCanonicalSegments(wire);
+    const userId = msg.user ?? '';
+    const userName = (msg as any).username ?? (userId || 'Unknown');
+    const messageText = msg.text ?? '';
 
-    // Extract user info safely
-    const userId = ("user" in msg ? msg.user : "") || "";
-    const userName = ("username" in msg ? msg.username : null) || userId || "Unknown";
-    const messageText = ("text" in msg ? msg.text : "") || "";
+    this.trackMessageChannel(msg.ts, channelId);
 
     const result = Message.from(msg, {
-      $id: msg.ts,
-      $adapter: "slack",
+      $id: formatSlackMessageRef(channelId, msg.ts),
+      $adapter: 'slack',
       $endpoint: this.$config.name,
-      $sender: {
-        id: userId,
-        name: userName,
-      },
-      $channel: {
-        id: channelId,
-        type: channelType,
-      },
+      $sender: { id: userId, name: userName },
+      $channel: { id: channelId, type: channelType },
       $content: content,
       $raw: messageText,
       $timestamp: parseFloat(msg.ts) * 1000,
+      $quote_id: msg.thread_ts && msg.thread_ts !== msg.ts ? msg.thread_ts : undefined,
       $recall: async () => {
-        try {
-          await this.client!.chat.delete({
-            channel: channelId,
-            ts: result.$id,
-          });
-        } catch (error) {
-          this.logger.error("Error recalling Slack message:", error);
-          throw error;
-        }
+        await this.client!.chat.delete({ channel: channelId, ts: msg.ts });
       },
-      $reply: async (
-        content: SendContent,
-        quote?: boolean | string
-      ): Promise<string> => {
-        if (!Array.isArray(content)) content = [content];
-
-        const sendOptions: Partial<ChatPostMessageArguments> = {
-          channel: channelId,
-        };
-
-        // Handle thread reply
-        if (quote) {
-          const threadTs = typeof quote === "boolean" ? result.$id : quote;
-          sendOptions.thread_ts = threadTs;
-        }
-
-        return  await this.adapter.sendMessage({
-          context: "slack",
+      $reply: async (replyContent, quote?) => {
+        if (!Array.isArray(replyContent)) replyContent = [replyContent];
+        const threadTs = quote
+          ? (typeof quote === 'boolean' ? msg.ts : quote)
+          : (msg.thread_ts ?? undefined);
+        return await this.adapter.sendMessage({
+          context: 'slack',
           endpoint: this.$config.name,
           id: channelId,
-          type: "channel",
-          content: content,
+          type: channelType,
+          content: replyContent,
+          ...(threadTs ? { threadId: threadTs } : {}),
         });
       },
     });
@@ -229,510 +128,213 @@ export class SlackEndpoint implements Endpoint<SlackEndpointConfig, SlackMessage
     return result;
   }
 
-  private parseMessageContent(msg: SlackMessageEvent): MessageSegment[] {
-    const segments: MessageSegment[] = [];
-
-    // Handle text
-    if ("text" in msg && msg.text) {
-      // Parse Slack formatting
-      segments.push(...this.parseSlackText(msg.text));
-    }
-
-    // Handle files
-    if ("files" in msg && msg.files) {
-      for (const file of msg.files) {
-        if (file.mimetype?.startsWith("image/")) {
-          segments.push({
-            type: "image",
-            data: {
-              id: file.id,
-              name: file.name,
-              url: file.url_private || file.permalink,
-              size: file.size,
-              mimetype: file.mimetype,
-            },
-          });
-        } else if (file.mimetype?.startsWith("video/")) {
-          segments.push({
-            type: "video",
-            data: {
-              id: file.id,
-              name: file.name,
-              url: file.url_private || file.permalink,
-              size: file.size,
-              mimetype: file.mimetype,
-            },
-          });
-        } else if (file.mimetype?.startsWith("audio/")) {
-          segments.push({
-            type: "audio",
-            data: {
-              id: file.id,
-              name: file.name,
-              url: file.url_private || file.permalink,
-              size: file.size,
-              mimetype: file.mimetype,
-            },
-          });
-        } else {
-          segments.push({
-            type: "file",
-            data: {
-              id: file.id,
-              name: file.name,
-              url: file.url_private || file.permalink,
-              size: file.size,
-              mimetype: file.mimetype,
-            },
-          });
-        }
-      }
-    }
-
-    // Handle attachments
-    if ("attachments" in msg && msg.attachments) {
-      for (const attachment of msg.attachments) {
-        if (attachment.image_url) {
-          segments.push({
-            type: "image",
-            data: {
-              url: attachment.image_url,
-              title: attachment.title,
-              text: attachment.text,
-            },
-          });
-        }
-      }
-    }
-
-    return segments.length > 0
-      ? segments
-      : [{ type: "text", data: { text: "" } }];
-  }
-
-  private parseSlackText(text: string): MessageSegment[] {
-    const segments: MessageSegment[] = [];
-    let lastIndex = 0;
-
-    // Match user mentions <@U12345678>
-    const userMentionRegex = /<@([UW][A-Z0-9]+)(?:\|([^>]+))?>/g;
-    // Match channel mentions <#C12345678|general>
-    const channelMentionRegex = /<#([C][A-Z0-9]+)(?:\|([^>]+))?>/g;
-    // Match links <http://example.com|Example>
-    const linkRegex = /<(https?:\/\/[^|>]+)(?:\|([^>]+))?>/g;
-
-    const allMatches: Array<{
-      match: RegExpExecArray;
-      type: "user" | "channel" | "link";
-    }> = [];
-
-    // Collect all matches
-    let match;
-    while ((match = userMentionRegex.exec(text)) !== null) {
-      allMatches.push({ match, type: "user" });
-    }
-    while ((match = channelMentionRegex.exec(text)) !== null) {
-      allMatches.push({ match, type: "channel" });
-    }
-    while ((match = linkRegex.exec(text)) !== null) {
-      allMatches.push({ match, type: "link" });
-    }
-
-    // Sort by position
-    allMatches.sort((a, b) => a.match.index! - b.match.index!);
-
-    // Process matches
-    for (const { match, type } of allMatches) {
-      const matchStart = match.index!;
-      const matchEnd = matchStart + match[0].length;
-
-      // Add text before match
-      if (matchStart > lastIndex) {
-        const beforeText = text.slice(lastIndex, matchStart);
-        if (beforeText.trim()) {
-          segments.push({ type: "text", data: { text: beforeText } });
-        }
-      }
-
-      // Add special segment
-      switch (type) {
-        case "user":
-          segments.push({
-            type: "at",
-            data: {
-              id: match[1],
-              name: match[2] || match[1],
-              text: match[0],
-            },
-          });
-          break;
-
-        case "channel":
-          segments.push({
-            type: "channel_mention",
-            data: {
-              id: match[1],
-              name: match[2] || match[1],
-              text: match[0],
-            },
-          });
-          break;
-
-        case "link":
-          segments.push({
-            type: "link",
-            data: {
-              url: match[1],
-              text: match[2] || match[1],
-            },
-          });
-          break;
-      }
-
-      lastIndex = matchEnd;
-    }
-
-    // Add remaining text
-    if (lastIndex < text.length) {
-      const remainingText = text.slice(lastIndex);
-      if (remainingText.trim()) {
-        segments.push({ type: "text", data: { text: remainingText } });
-      }
-    }
-
-    return segments.length > 0
-      ? segments
-      : [{ type: "text", data: { text } }];
-  }
-
   async $sendMessage(options: SendOptions): Promise<string> {
     try {
       const canonical = expandInteractiveSegmentsInContent(options.content);
       const wire = fromCanonicalSegments(canonical);
-      const result = await this.sendContentToChannel(
-        options.id,
-        wire
-      );
+      const result = await sendSlackContent(this.client!, wire, {
+        channel: options.id,
+        threadTs: options.threadId,
+      }, this.logger);
       this.logger.debug(
-        `${this.$config.name} send ${options.type}(${options.id}): ${segment.raw(options.content)}`
+        `${this.$config.name} send ${options.type}(${options.id}): ${segment.raw(options.content)}`,
       );
-      return result.ts || "";
+      this.trackMessageChannel(result.ts, options.id);
+      return formatSlackMessageRef(options.id, result.ts);
     } catch (error) {
-      this.logger.error("Failed to send Slack message:", error);
+      this.logger.error('Failed to send Slack message:', error);
       throw error;
     }
   }
 
-  private async sendContentToChannel(
-    channel: string,
-    content: SendContent,
-    extraOptions: Partial<ChatPostMessageArguments> = {}
-  ): Promise<any> {
-    if (!Array.isArray(content)) content = [content];
-
-    let textContent = "";
-    const attachments: any[] = [];
-
-    for (const segment of content) {
-      if (typeof segment === "string") {
-        textContent += segment;
-        continue;
-      }
+  async $editMessage(options: EditMessageOptions): Promise<void> {
+    const ref = parseSlackMessageRef(options.messageId);
+    const channel = ref?.channel ?? options.id;
+    const ts = ref?.ts ?? options.messageId;
+    const canonical = expandInteractiveSegmentsInContent(options.content);
+    const wire = fromCanonicalSegments(canonical);
+    await editSlackContent(this.client!, channel, ts, wire);
+  }
 
-      const { type, data } = segment;
-
-      switch (type) {
-        case "text":
-          textContent += data.text || "";
-          break;
-
-        case "at":
-          textContent += `<@${data.id}>`;
-          break;
-
-        case "channel_mention":
-          textContent += `<#${data.id}>`;
-          break;
-
-        case "link":
-          if (data.text && data.text !== data.url) {
-            textContent += `<${data.url}|${data.text}>`;
-          } else {
-            textContent += `<${data.url}>`;
-          }
-          break;
-
-        case "image":
-          if (data.url) {
-            attachments.push({
-              image_url: data.url,
-              title: data.name || data.title,
-            });
-          }
-          break;
-
-        case "file":
-          // Files need to be uploaded separately
-          if (data.file) {
-            try {
-              await this.client!.files.upload({
-                channels: channel,
-                file: data.file,
-                filename: data.name,
-              });
-            } catch (error) {
-              this.logger.error("Failed to upload file:", error);
-            }
-          }
-          break;
-
-        default:
-          textContent += data.text || `[${type}]`;
+  async $recallMessage(id: string): Promise<void> {
+    const ref = this.resolveMessageRef(id);
+    if (!ref) {
+      this.logger.warn(formatCompact({ op: 'recall_skip', reason: 'channel_unknown', message_id: id }));
+      return;
+    }
+    try {
+      await this.client!.chat.delete({ channel: ref.channel, ts: ref.ts });
+    } catch (error: unknown) {
+      const slackError = (error as { data?: { error?: string } })?.data?.error;
+      if (slackError === 'message_not_found') {
+        this.logger.debug(formatCompact({ op: 'recall_skip', reason: 'message_not_found', message_id: id }));
+        return;
       }
+      throw error;
     }
+  }
 
-    // Send message
-    const messageOptions: any = {
-      channel,
-      text: textContent.trim() || "Message",
-      ...extraOptions,
-    };
-
-    if (attachments.length > 0) {
-      messageOptions.attachments = attachments;
+  trackMessageChannel(ts: string, channel: string): void {
+    if (ts && channel) {
+      this.messageChannelMap.set(ts, channel);
     }
-
-    const result = await this.client!.chat.postMessage(messageOptions as ChatPostMessageArguments);
-    return result.message || {};
   }
 
-  async $recallMessage(id: string): Promise<void> {
-    // Slack requires both channel and ts (timestamp) to delete a message
-    // The Endpoint interface only provides message ID (ts), making recall impossible  
-    // Users should use message.$recall() instead, which has the full context
-    throw new Error(
-      "SlackEndpoint.$recallMessage: Message recall not supported without channel information. " +
-      "Use message.$recall() method instead, which contains the required context."
-    );
+  resolveMessageRef(messageId: string, channelHint?: string): { channel: string; ts: string } | null {
+    const parsed = parseSlackMessageRef(messageId);
+    if (parsed) {
+      this.trackMessageChannel(parsed.ts, parsed.channel);
+      return parsed;
+    }
+    if (channelHint) {
+      return { channel: channelHint, ts: messageId };
+    }
+    const channel = this.messageChannelMap.get(messageId);
+    if (channel) {
+      return { channel, ts: messageId };
+    }
+    return null;
   }
 
-  // ==================== 工作区管理 API ====================
-
   /**
-   * 邀请用户到频道
-   * @param channel 频道 ID
-   * @param users 用户 ID 列表
+   * Activity Feedback：在用户消息上添加表情回应
+   * @returns reaction name，供 $removeReaction 使用
    */
-  async inviteToChannel(channel: string, users: string[]): Promise<boolean> {
+  async $addReaction(
+    messageId: string,
+    emoji: string,
+    hint?: { sceneType?: 'private' | 'group' | 'channel'; channelId?: string },
+  ): Promise<string | null> {
+    const ref = this.resolveMessageRef(messageId, hint?.channelId);
+    if (!ref) {
+      this.logger.warn(formatCompact({ op: 'reaction_add_skip', reason: 'channel_unknown', message_id: messageId }));
+      return null;
+    }
+    const name = normalizeSlackReactionName(emoji);
     try {
-      await this.client!.conversations.invite({ channel, users: users.join(',') });
-      this.logger.debug(formatCompact( { op: "invite", endpoint: this.$id, channel, users: users.join(",") }));
-      return true;
+      await this.addReaction(ref.channel, ref.ts, name);
+      return name;
     } catch (error) {
-      this.logger.error(`Slack Endpoint ${this.$id} 邀请用户失败:`, error);
-      throw error;
+      this.logger.error('Failed to add Slack reaction:', error);
+      return null;
     }
   }
 
-  /**
-   * 从频道踢出用户
-   * @param channel 频道 ID
-   * @param user 用户 ID
-   */
-  async kickFromChannel(channel: string, user: string): Promise<boolean> {
+  /** Activity Feedback：移除本 Bot 在消息上的表情回应 */
+  async $removeReaction(messageId: string, reactionId: string, channelHint?: string): Promise<void> {
+    const ref = this.resolveMessageRef(messageId, channelHint);
+    if (!ref) {
+      this.logger.warn(formatCompact({ op: 'reaction_remove_skip', reason: 'channel_unknown', message_id: messageId }));
+      return;
+    }
+    const name = normalizeSlackReactionName(reactionId);
     try {
-      await this.client!.conversations.kick({ channel, user });
-      this.logger.debug(formatCompact( { op: "kick", endpoint: this.$id, channel, user }));
-      return true;
+      await this.removeReaction(ref.channel, ref.ts, name);
     } catch (error) {
-      this.logger.error(`Slack Endpoint ${this.$id} 踢出用户失败:`, error);
-      throw error;
+      const code = (error as { data?: { error?: string } })?.data?.error;
+      if (code === 'no_reaction') return;
+      this.logger.error('Failed to remove Slack reaction:', error);
     }
   }
 
-  /**
-   * 设置频道话题
-   * @param channel 频道 ID
-   * @param topic 话题
-   */
+  // ==================== 工作区管理 API ====================
+
+  async inviteToChannel(channel: string, users: string[]): Promise<boolean> {
+    await this.client!.conversations.invite({ channel, users: users.join(',') });
+    this.logger.debug(formatCompact({ op: 'invite', endpoint: this.$id, channel, users: users.join(',') }));
+    return true;
+  }
+
+  async kickFromChannel(channel: string, user: string): Promise<boolean> {
+    await this.client!.conversations.kick({ channel, user });
+    this.logger.debug(formatCompact({ op: 'kick', endpoint: this.$id, channel, user }));
+    return true;
+  }
+
   async setChannelTopic(channel: string, topic: string): Promise<boolean> {
-    try {
-      await this.client!.conversations.setTopic({ channel, topic });
-      this.logger.debug(formatCompact( { op: "set_topic", endpoint: this.$id, channel }));
-      return true;
-    } catch (error) {
-      this.logger.error(`Slack Endpoint ${this.$id} 设置话题失败:`, error);
-      throw error;
-    }
+    await this.client!.conversations.setTopic({ channel, topic });
+    this.logger.debug(formatCompact({ op: 'set_topic', endpoint: this.$id, channel }));
+    return true;
   }
 
-  /**
-   * 设置频道目的
-   * @param channel 频道 ID
-   * @param purpose 目的
-   */
   async setChannelPurpose(channel: string, purpose: string): Promise<boolean> {
-    try {
-      await this.client!.conversations.setPurpose({ channel, purpose });
-      this.logger.debug(formatCompact( { op: "set_purpose", endpoint: this.$id, channel }));
-      return true;
-    } catch (error) {
-      this.logger.error(`Slack Endpoint ${this.$id} 设置目的失败:`, error);
-      throw error;
-    }
+    await this.client!.conversations.setPurpose({ channel, purpose });
+    this.logger.debug(formatCompact({ op: 'set_purpose', endpoint: this.$id, channel }));
+    return true;
   }
 
-  /**
-   * 归档频道
-   * @param channel 频道 ID
-   */
   async archiveChannel(channel: string): Promise<boolean> {
-    try {
-      await this.client!.conversations.archive({ channel });
-      this.logger.debug(formatCompact( { op: "archive", endpoint: this.$id, channel }));
-      return true;
-    } catch (error) {
-      this.logger.error(`Slack Endpoint ${this.$id} 归档频道失败:`, error);
-      throw error;
-    }
+    await this.client!.conversations.archive({ channel });
+    this.logger.debug(formatCompact({ op: 'archive', endpoint: this.$id, channel }));
+    return true;
   }
 
-  /**
-   * 取消归档频道
-   * @param channel 频道 ID
-   */
   async unarchiveChannel(channel: string): Promise<boolean> {
-    try {
-      await this.client!.conversations.unarchive({ channel });
-      this.logger.debug(formatCompact( { op: "unarchive", endpoint: this.$id, channel }));
-      return true;
-    } catch (error) {
-      this.logger.error(`Slack Endpoint ${this.$id} 取消归档失败:`, error);
-      throw error;
-    }
+    await this.client!.conversations.unarchive({ channel });
+    this.logger.debug(formatCompact({ op: 'unarchive', endpoint: this.$id, channel }));
+    return true;
   }
 
-  /**
-   * 重命名频道
-   * @param channel 频道 ID
-   * @param name 新名称
-   */
   async renameChannel(channel: string, name: string): Promise<boolean> {
-    try {
-      await this.client!.conversations.rename({ channel, name });
-      this.logger.debug(formatCompact( { op: "rename", endpoint: this.$id, channel, name }));
-      return true;
-    } catch (error) {
-      this.logger.error(`Slack Endpoint ${this.$id} 重命名频道失败:`, error);
-      throw error;
-    }
+    await this.client!.conversations.rename({ channel, name });
+    this.logger.debug(formatCompact({ op: 'rename', endpoint: this.$id, channel, name }));
+    return true;
   }
 
-  /**
-   * 获取频道成员列表
-   * @param channel 频道 ID
-   */
   async getChannelMembers(channel: string): Promise<string[]> {
-    try {
-      const result = await this.client!.conversations.members({ channel });
-      return result.members || [];
-    } catch (error) {
-      this.logger.error(`Slack Endpoint ${this.$id} 获取成员列表失败:`, error);
-      throw error;
-    }
+    const result = await this.client!.conversations.members({ channel });
+    return result.members || [];
   }
 
-  /**
-   * 获取频道信息
-   * @param channel 频道 ID
-   */
   async getChannelInfo(channel: string): Promise<any> {
-    try {
-      const result = await this.client!.conversations.info({ channel });
-      return result.channel;
-    } catch (error) {
-      this.logger.error(`Slack Endpoint ${this.$id} 获取频道信息失败:`, error);
-      throw error;
-    }
+    const result = await this.client!.conversations.info({ channel });
+    return result.channel;
   }
 
-  /**
-   * 获取用户信息
-   * @param user 用户 ID
-   */
   async getUserInfo(user: string): Promise<any> {
-    try {
-      const result = await this.client!.users.info({ user });
-      return result.user;
-    } catch (error) {
-      this.logger.error(`Slack Endpoint ${this.$id} 获取用户信息失败:`, error);
-      throw error;
-    }
+    const result = await this.client!.users.info({ user });
+    return result.user;
   }
 
-  /**
-   * 添加消息反应
-   * @param channel 频道 ID
-   * @param timestamp 消息时间戳
-   * @param name 表情名称
-   */
   async addReaction(channel: string, timestamp: string, name: string): Promise<boolean> {
     try {
       await this.client!.reactions.add({ channel, timestamp, name });
-      this.logger.debug(formatCompact( { op: "reaction_add", endpoint: this.$id, name }));
+      this.logger.debug(formatCompact({ op: 'reaction_add', endpoint: this.$id, name }));
       return true;
     } catch (error) {
-      this.logger.error(`Slack Endpoint ${this.$id} 添加反应失败:`, error);
+      const code = (error as { data?: { error?: string } })?.data?.error;
+      if (code === 'already_reacted') {
+        this.logger.debug(formatCompact({ op: 'reaction_add', endpoint: this.$id, name, status: 'exists' }));
+        return true;
+      }
       throw error;
     }
   }
 
-  /**
-   * 移除消息反应
-   * @param channel 频道 ID
-   * @param timestamp 消息时间戳
-   * @param name 表情名称
-   */
   async removeReaction(channel: string, timestamp: string, name: string): Promise<boolean> {
     try {
       await this.client!.reactions.remove({ channel, timestamp, name });
-      this.logger.debug(formatCompact( { op: "reaction_remove", endpoint: this.$id, name }));
+      this.logger.debug(formatCompact({ op: 'reaction_remove', endpoint: this.$id, name }));
       return true;
     } catch (error) {
-      this.logger.error(`Slack Endpoint ${this.$id} 移除反应失败:`, error);
+      const code = (error as { data?: { error?: string } })?.data?.error;
+      if (code === 'no_reaction') {
+        this.logger.debug(formatCompact({ op: 'reaction_remove', endpoint: this.$id, name, status: 'gone' }));
+        return true;
+      }
       throw error;
     }
   }
 
-  /**
-   * 置顶消息
-   * @param channel 频道 ID
-   * @param timestamp 消息时间戳
-   */
   async pinMessage(channel: string, timestamp: string): Promise<boolean> {
-    try {
-      await this.client!.pins.add({ channel, timestamp });
-      this.logger.debug(formatCompact( { op: "pin", endpoint: this.$id, channel }));
-      return true;
-    } catch (error) {
-      this.logger.error(`Slack Endpoint ${this.$id} 置顶消息失败:`, error);
-      throw error;
-    }
+    await this.client!.pins.add({ channel, timestamp });
+    this.logger.debug(formatCompact({ op: 'pin', endpoint: this.$id, channel }));
+    return true;
   }
 
-  /**
-   * 取消置顶消息
-   * @param channel 频道 ID
-   * @param timestamp 消息时间戳
-   */
   async unpinMessage(channel: string, timestamp: string): Promise<boolean> {
-    try {
-      await this.client!.pins.remove({ channel, timestamp });
-      this.logger.debug(formatCompact( { op: "unpin", endpoint: this.$id, channel }));
-      return true;
-    } catch (error) {
-      this.logger.error(`Slack Endpoint ${this.$id} 取消置顶失败:`, error);
-      throw error;
-    }
+    await this.client!.pins.remove({ channel, timestamp });
+    this.logger.debug(formatCompact({ op: 'unpin', endpoint: this.$id, channel }));
+    return true;
   }
 }