@chat-adapter/slack 4.20.1 → 4.20.2

package/dist/index.d.ts

@@ -175,6 +175,7 @@ declare class SlackAdapter implements Adapter<SlackThreadId, unknown> {
     private readonly formatConverter;
     private static USER_CACHE_TTL_MS;
     private static CHANNEL_CACHE_TTL_MS;
+    private static REVERSE_INDEX_TTL_MS;
     private readonly clientId;
     private readonly clientSecret;
     private readonly encryptionKey;
@@ -291,6 +292,7 @@ declare class SlackAdapter implements Adapter<SlackThreadId, unknown> {
      * Fires when a user (including the bot) joins a channel.
      */
     private handleMemberJoinedChannel;
+    private handleUserChange;
     /**
      * Publish a Home tab view for a user.
      * Slack API: views.publish
@@ -342,14 +344,24 @@ declare class SlackAdapter implements Adapter<SlackThreadId, unknown> {
      * Includes a fetchData method that uses the bot token for auth.
      */
     private createAttachment;
+    /**
+     * Resolve @name mentions in text to Slack <@USER_ID> format using the
+     * reverse user cache. When multiple users share a display name, prefers
+     * the one who is a participant in the given thread.
+     */
+    private resolveOutgoingMentions;
+    /**
+     * Pre-process an outgoing message to resolve @name mentions before rendering.
+     */
+    private resolveMessageMentions;
     /**
      * Try to render a message using native Slack table blocks.
      * Returns blocks + fallback text if the message contains tables, null otherwise.
      */
     private renderWithTableBlocks;
-    postMessage(threadId: string, message: AdapterPostableMessage): Promise<RawMessage<unknown>>;
-    postEphemeral(threadId: string, userId: string, message: AdapterPostableMessage): Promise<EphemeralMessage>;
-    scheduleMessage(threadId: string, message: AdapterPostableMessage, options: {
+    postMessage(threadId: string, _message: AdapterPostableMessage): Promise<RawMessage<unknown>>;
+    postEphemeral(threadId: string, userId: string, _message: AdapterPostableMessage): Promise<EphemeralMessage>;
+    scheduleMessage(threadId: string, _message: AdapterPostableMessage, options: {
         postAt: Date;
     }): Promise<ScheduledMessage>;
     openModal(triggerId: string, modal: ModalElement, contextId?: string): Promise<{
@@ -363,7 +375,7 @@ declare class SlackAdapter implements Adapter<SlackThreadId, unknown> {
      * Returns the file IDs of uploaded files.
      */
     private uploadFiles;
-    editMessage(threadId: string, messageId: string, message: AdapterPostableMessage): Promise<RawMessage<unknown>>;
+    editMessage(threadId: string, messageId: string, _message: AdapterPostableMessage): Promise<RawMessage<unknown>>;
     deleteMessage(threadId: string, messageId: string): Promise<void>;
     addReaction(threadId: string, messageId: string, emoji: EmojiValue | string): Promise<void>;
     removeReaction(threadId: string, messageId: string, emoji: EmojiValue | string): Promise<void>;

package/dist/index.js

@@ -708,6 +708,7 @@ function radioSelectToBlock(radioSelect) {
 
 // src/index.ts
 var SLACK_USER_ID_PATTERN = /^[A-Z0-9_]+$/;
+var SLACK_USER_ID_EXACT_PATTERN = /^U[A-Z0-9]+$/;
 function findNextMention(text) {
   const atIdx = text.indexOf("<@");
   const hashIdx = text.indexOf("<#");
@@ -732,10 +733,12 @@ var SlackAdapter = class _SlackAdapter {
   _botId = null;
   // Bot app ID (B_xxx) - different from user ID
   formatConverter = new SlackFormatConverter();
-  static USER_CACHE_TTL_MS = 60 * 60 * 1e3;
-  // 1 hour
-  static CHANNEL_CACHE_TTL_MS = 60 * 60 * 1e3;
-  // 1 hour
+  static USER_CACHE_TTL_MS = 8 * 24 * 60 * 60 * 1e3;
+  // 8 days
+  static CHANNEL_CACHE_TTL_MS = 8 * 24 * 60 * 60 * 1e3;
+  // 8 days
+  static REVERSE_INDEX_TTL_MS = 8 * 24 * 60 * 60 * 1e3;
+  // 8 days
   // Multi-workspace support
   clientId;
   clientSecret;
@@ -1006,6 +1009,15 @@ var SlackAdapter = class _SlackAdapter {
           { displayName, realName },
           _SlackAdapter.USER_CACHE_TTL_MS
         );
+        const normalizedName = displayName.toLowerCase();
+        const reverseKey = `slack:user-by-name:${normalizedName}`;
+        const existing = await this.chat.getState().getList(reverseKey);
+        if (!existing.includes(userId)) {
+          await this.chat.getState().appendToList(reverseKey, userId, {
+            maxLength: 50,
+            ttlMs: _SlackAdapter.REVERSE_INDEX_TTL_MS
+          });
+        }
       }
       this.logger.debug("Fetched user info", {
         userId,
@@ -1144,6 +1156,8 @@ var SlackAdapter = class _SlackAdapter {
           event,
           options
         );
+      } else if (event.type === "user_change") {
+        this.handleUserChange(event);
       }
     }
   }
@@ -1650,6 +1664,19 @@ var SlackAdapter = class _SlackAdapter {
       options
     );
   }
+  async handleUserChange(event) {
+    if (!this.chat) {
+      return;
+    }
+    try {
+      await this.chat.getState().delete(`slack:user:${event.user.id}`);
+    } catch (error) {
+      this.logger.warn("Failed to invalidate user cache", {
+        userId: event.user.id,
+        error
+      });
+    }
+  }
   /**
    * Publish a Home tab view for a user.
    * Slack API: views.publish
@@ -1864,6 +1891,24 @@ var SlackAdapter = class _SlackAdapter {
       userName = userInfo.displayName;
       fullName = userInfo.realName;
     }
+    if (event.user && this.chat) {
+      try {
+        const participantKey = `slack:thread-participants:${threadId}`;
+        const participants = await this.chat.getState().getList(participantKey);
+        if (!participants.includes(event.user)) {
+          await this.chat.getState().appendToList(participantKey, event.user, {
+            maxLength: 100,
+            ttlMs: _SlackAdapter.REVERSE_INDEX_TTL_MS
+          });
+        }
+      } catch (error) {
+        this.logger.warn("Failed to track thread participant", {
+          threadId,
+          userId: event.user,
+          error
+        });
+      }
+    }
     const text = await this.resolveInlineMentions(rawText, skipSelfMention);
     return new Message({
       id: event.ts || "",
@@ -1936,6 +1981,107 @@ var SlackAdapter = class _SlackAdapter {
       } : void 0
     };
   }
+  /**
+   * Resolve @name mentions in text to Slack <@USER_ID> format using the
+   * reverse user cache. When multiple users share a display name, prefers
+   * the one who is a participant in the given thread.
+   */
+  async resolveOutgoingMentions(text, threadId) {
+    if (!this.chat) {
+      return text;
+    }
+    const state = this.chat.getState();
+    const mentionPattern = /@(\w+)/g;
+    const mentions = /* @__PURE__ */ new Map();
+    for (const match of text.matchAll(mentionPattern)) {
+      const name = match[1];
+      if (SLACK_USER_ID_EXACT_PATTERN.test(name)) {
+        continue;
+      }
+      const idx = match.index;
+      if (idx > 0 && text[idx - 1] === "<") {
+        continue;
+      }
+      if (!mentions.has(name.toLowerCase())) {
+        mentions.set(name.toLowerCase(), []);
+      }
+    }
+    if (mentions.size === 0) {
+      return text;
+    }
+    for (const name of mentions.keys()) {
+      const userIds = await state.getList(`slack:user-by-name:${name}`);
+      const unique = [...new Set(userIds)];
+      mentions.set(name, unique);
+    }
+    let participants = null;
+    const needsParticipants = [...mentions.values()].some(
+      (ids) => ids.length > 1
+    );
+    if (needsParticipants) {
+      const participantList = await state.getList(
+        `slack:thread-participants:${threadId}`
+      );
+      participants = new Set(participantList);
+    }
+    return text.replace(
+      mentionPattern,
+      (match, name, offset) => {
+        if (offset > 0 && text[offset - 1] === "<") {
+          return match;
+        }
+        if (SLACK_USER_ID_EXACT_PATTERN.test(name)) {
+          return match;
+        }
+        const userIds = mentions.get(name.toLowerCase());
+        if (!userIds || userIds.length === 0) {
+          return match;
+        }
+        if (userIds.length === 1) {
+          return `<@${userIds[0]}>`;
+        }
+        if (participants) {
+          const inThread = userIds.filter((id) => participants.has(id));
+          if (inThread.length === 1) {
+            return `<@${inThread[0]}>`;
+          }
+        }
+        return match;
+      }
+    );
+  }
+  /**
+   * Pre-process an outgoing message to resolve @name mentions before rendering.
+   */
+  async resolveMessageMentions(message, threadId) {
+    if (!this.chat) {
+      return message;
+    }
+    if (typeof message === "string") {
+      return this.resolveOutgoingMentions(message, threadId);
+    }
+    if (typeof message === "object" && message !== null) {
+      if ("raw" in message && typeof message.raw === "string") {
+        return {
+          ...message,
+          raw: await this.resolveOutgoingMentions(
+            message.raw,
+            threadId
+          )
+        };
+      }
+      if ("markdown" in message && typeof message.markdown === "string") {
+        return {
+          ...message,
+          markdown: await this.resolveOutgoingMentions(
+            message.markdown,
+            threadId
+          )
+        };
+      }
+    }
+    return message;
+  }
   /**
    * Try to render a message using native Slack table blocks.
    * Returns blocks + fallback text if the message contains tables, null otherwise.
@@ -1962,7 +2108,8 @@ var SlackAdapter = class _SlackAdapter {
     );
     return { text: fallbackText, blocks };
   }
-  async postMessage(threadId, message) {
+  async postMessage(threadId, _message) {
+    const message = await this.resolveMessageMentions(_message, threadId);
     const { channel, threadTs } = this.decodeThreadId(threadId);
     try {
       const files = extractFiles(message);
@@ -2066,7 +2213,8 @@ var SlackAdapter = class _SlackAdapter {
       this.handleSlackError(error);
     }
   }
-  async postEphemeral(threadId, userId, message) {
+  async postEphemeral(threadId, userId, _message) {
+    const message = await this.resolveMessageMentions(_message, threadId);
     const { channel, threadTs } = this.decodeThreadId(threadId);
     try {
       const card = extractCard(message);
@@ -2159,7 +2307,8 @@ var SlackAdapter = class _SlackAdapter {
       this.handleSlackError(error);
     }
   }
-  async scheduleMessage(threadId, message, options) {
+  async scheduleMessage(threadId, _message, options) {
+    const message = await this.resolveMessageMentions(_message, threadId);
     const { channel, threadTs } = this.decodeThreadId(threadId);
     const postAtUnix = Math.floor(options.postAt.getTime() / 1e3);
     if (postAtUnix <= Math.floor(Date.now() / 1e3)) {
@@ -2345,7 +2494,8 @@ var SlackAdapter = class _SlackAdapter {
     }
     return fileIds;
   }
-  async editMessage(threadId, messageId, message) {
+  async editMessage(threadId, messageId, _message) {
+    const message = await this.resolveMessageMentions(_message, threadId);
     const ephemeral = this.decodeEphemeralMessageId(messageId);
     if (ephemeral) {
       const { threadTs } = this.decodeThreadId(threadId);