@chat-adapter/slack 4.19.0 → 4.20.1

package/README.md

@@ -213,6 +213,7 @@ SLACK_ENCRYPTION_KEY=...             # Optional, for token encryption
 | Delete message | Yes |
 | File uploads | Yes |
 | Streaming | Native API |
+| Scheduled messages | Yes (native, with cancel) |
 
 ### Rich content
 

package/dist/index.d.ts

@@ -107,6 +107,18 @@ interface SlackThreadId {
 }
 /** Slack event payload (raw message format) */
 interface SlackEvent {
+    /** Rich text blocks containing structured elements (links, mentions, etc.) */
+    blocks?: Array<{
+        type: string;
+        elements?: Array<{
+            type: string;
+            elements?: Array<{
+                type: string;
+                url?: string;
+                text?: string;
+            }>;
+        }>;
+    }>;
     bot_id?: string;
     channel?: string;
     /** Channel type: "channel", "group", "mpim", or "im" (DM) */
@@ -162,6 +174,7 @@ declare class SlackAdapter implements Adapter<SlackThreadId, unknown> {
     private _botId;
     private readonly formatConverter;
     private static USER_CACHE_TTL_MS;
+    private static CHANNEL_CACHE_TTL_MS;
     private readonly clientId;
     private readonly clientSecret;
     private readonly encryptionKey;
@@ -222,6 +235,11 @@ declare class SlackAdapter implements Adapter<SlackThreadId, unknown> {
      * Returns display name and real name, or falls back to user ID.
      */
     private lookupUser;
+    /**
+     * Look up channel name from Slack API with caching via state adapter.
+     * Returns channel name, or falls back to channel ID.
+     */
+    private lookupChannel;
     handleWebhook(request: Request, options?: WebhookOptions): Promise<Response>;
     /** Extract and dispatch events from a validated payload */
     private processEventPayload;
@@ -307,6 +325,17 @@ declare class SlackAdapter implements Adapter<SlackThreadId, unknown> {
      *   detection doesn't apply.
      */
     private resolveInlineMentions;
+    /**
+     * Extract link URLs from a Slack event.
+     * Uses the `blocks` field (rich_text blocks with link elements) when available,
+     * falling back to parsing `<url>` patterns from the text field.
+     */
+    private extractLinks;
+    /**
+     * Create a LinkPreview for a URL. If the URL points to a Slack message,
+     * includes a `fetchMessage` callback that fetches and parses the linked message.
+     */
+    private createLinkPreview;
     private parseSlackMessage;
     /**
      * Create an Attachment object from a Slack file.

package/dist/index.js

@@ -708,6 +708,18 @@ function radioSelectToBlock(radioSelect) {
 
 // src/index.ts
 var SLACK_USER_ID_PATTERN = /^[A-Z0-9_]+$/;
+function findNextMention(text) {
+  const atIdx = text.indexOf("<@");
+  const hashIdx = text.indexOf("<#");
+  if (atIdx === -1) {
+    return hashIdx;
+  }
+  if (hashIdx === -1) {
+    return atIdx;
+  }
+  return Math.min(atIdx, hashIdx);
+}
+var SLACK_MESSAGE_URL_PATTERN = /^https?:\/\/[^/]+\.slack\.com\/archives\/([A-Z0-9]+)\/p(\d+)(?:\?.*)?$/;
 var SlackAdapter = class _SlackAdapter {
   name = "slack";
   userName;
@@ -722,6 +734,8 @@ var SlackAdapter = class _SlackAdapter {
   formatConverter = new SlackFormatConverter();
   static USER_CACHE_TTL_MS = 60 * 60 * 1e3;
   // 1 hour
+  static CHANNEL_CACHE_TTL_MS = 60 * 60 * 1e3;
+  // 1 hour
   // Multi-workspace support
   clientId;
   clientSecret;
@@ -1004,6 +1018,37 @@ var SlackAdapter = class _SlackAdapter {
       return { displayName: userId, realName: userId };
     }
   }
+  /**
+   * Look up channel name from Slack API with caching via state adapter.
+   * Returns channel name, or falls back to channel ID.
+   */
+  async lookupChannel(channelId) {
+    const cacheKey = `slack:channel:${channelId}`;
+    if (this.chat) {
+      const cached = await this.chat.getState().get(cacheKey);
+      if (cached) {
+        return cached.name;
+      }
+    }
+    try {
+      const result = await this.client.conversations.info(
+        this.withToken({ channel: channelId })
+      );
+      const name = result.channel?.name || channelId;
+      if (this.chat) {
+        await this.chat.getState().set(
+          cacheKey,
+          { name },
+          _SlackAdapter.CHANNEL_CACHE_TTL_MS
+        );
+      }
+      this.logger.debug("Fetched channel info", { channelId, name });
+      return name;
+    } catch (error) {
+      this.logger.warn("Could not fetch channel info", { channelId, error });
+      return channelId;
+    }
+  }
   async handleWebhook(request, options) {
     const body = await request.text();
     this.logger.debug("Slack webhook raw body", { body });
@@ -1406,7 +1451,7 @@ var SlackAdapter = class _SlackAdapter {
       channel: event.channel,
       threadTs
     });
-    const isMention = isDM || event.type === "app_mention";
+    const isMention = event.type === "app_mention";
     const factory = async () => {
       const msg = await this.parseSlackMessage(event, threadId);
       if (isMention) {
@@ -1668,41 +1713,56 @@ var SlackAdapter = class _SlackAdapter {
    */
   async resolveInlineMentions(text, skipSelfMention) {
     const userIds = /* @__PURE__ */ new Set();
+    const channelIds = /* @__PURE__ */ new Set();
     for (const segment of text.split("<")) {
       const end = segment.indexOf(">");
       if (end === -1) {
         continue;
       }
       const inner = segment.slice(0, end);
-      if (!inner.startsWith("@")) {
-        continue;
-      }
-      const rest = inner.slice(1);
-      const pipeIdx = rest.indexOf("|");
-      const uid = pipeIdx >= 0 ? rest.slice(0, pipeIdx) : rest;
-      if (SLACK_USER_ID_PATTERN.test(uid)) {
-        userIds.add(uid);
+      if (inner.startsWith("@")) {
+        const rest = inner.slice(1);
+        const pipeIdx = rest.indexOf("|");
+        const uid = pipeIdx >= 0 ? rest.slice(0, pipeIdx) : rest;
+        if (SLACK_USER_ID_PATTERN.test(uid)) {
+          userIds.add(uid);
+        }
+      } else if (inner.startsWith("#")) {
+        const rest = inner.slice(1);
+        const pipeIdx = rest.indexOf("|");
+        if (pipeIdx === -1 && SLACK_USER_ID_PATTERN.test(rest)) {
+          channelIds.add(rest);
+        }
       }
     }
-    if (userIds.size === 0) {
+    if (userIds.size === 0 && channelIds.size === 0) {
       return text;
     }
     if (skipSelfMention && this._botUserId) {
       userIds.delete(this._botUserId);
     }
-    if (userIds.size === 0) {
+    if (userIds.size === 0 && channelIds.size === 0) {
       return text;
     }
-    const lookups = await Promise.all(
-      [...userIds].map(async (uid) => {
-        const info = await this.lookupUser(uid);
-        return [uid, info.displayName];
-      })
-    );
-    const nameMap = new Map(lookups);
+    const [userLookups, channelLookups] = await Promise.all([
+      Promise.all(
+        [...userIds].map(async (uid) => {
+          const info = await this.lookupUser(uid);
+          return [uid, info.displayName];
+        })
+      ),
+      Promise.all(
+        [...channelIds].map(async (cid) => {
+          const name = await this.lookupChannel(cid);
+          return [cid, name];
+        })
+      )
+    ]);
+    const userNameMap = new Map(userLookups);
+    const channelNameMap = new Map(channelLookups);
     let result = "";
     let remaining = text;
-    let startIdx = remaining.indexOf("<@");
+    let startIdx = findNextMention(remaining);
     while (startIdx !== -1) {
       result += remaining.slice(0, startIdx);
       remaining = remaining.slice(startIdx);
@@ -1710,20 +1770,89 @@ var SlackAdapter = class _SlackAdapter {
       if (endIdx === -1) {
         break;
       }
+      const prefix = remaining[1];
       const inner = remaining.slice(2, endIdx);
       const pipeIdx = inner.indexOf("|");
-      const uid = pipeIdx >= 0 ? inner.slice(0, pipeIdx) : inner;
-      if (SLACK_USER_ID_PATTERN.test(uid)) {
-        const name = nameMap.get(uid);
-        result += name ? `<@${uid}|${name}>` : `<@${uid}>`;
+      const id = pipeIdx >= 0 ? inner.slice(0, pipeIdx) : inner;
+      if (prefix === "@" && SLACK_USER_ID_PATTERN.test(id)) {
+        const name = userNameMap.get(id);
+        result += name ? `<@${id}|${name}>` : `<@${id}>`;
+      } else if (prefix === "#" && pipeIdx === -1 && channelNameMap.has(id)) {
+        const name = channelNameMap.get(id);
+        result += `<#${id}|${name}>`;
       } else {
         result += remaining.slice(0, endIdx + 1);
       }
       remaining = remaining.slice(endIdx + 1);
-      startIdx = remaining.indexOf("<@");
+      startIdx = findNextMention(remaining);
     }
     return result + remaining;
   }
+  /**
+   * Extract link URLs from a Slack event.
+   * Uses the `blocks` field (rich_text blocks with link elements) when available,
+   * falling back to parsing `<url>` patterns from the text field.
+   */
+  extractLinks(event) {
+    const urls = /* @__PURE__ */ new Set();
+    if (event.blocks) {
+      for (const block of event.blocks) {
+        if (block.type === "rich_text" && block.elements) {
+          for (const section of block.elements) {
+            if (section.elements) {
+              for (const element of section.elements) {
+                if (element.type === "link" && element.url) {
+                  urls.add(element.url);
+                }
+              }
+            }
+          }
+        }
+      }
+    }
+    if (urls.size === 0 && event.text) {
+      const urlPattern = /<(https?:\/\/[^>]+)>/g;
+      for (const match of event.text.matchAll(urlPattern)) {
+        const raw = match[1];
+        const pipeIdx = raw.indexOf("|");
+        urls.add(pipeIdx >= 0 ? raw.slice(0, pipeIdx) : raw);
+      }
+    }
+    return [...urls].map((url) => this.createLinkPreview(url));
+  }
+  /**
+   * Create a LinkPreview for a URL. If the URL points to a Slack message,
+   * includes a `fetchMessage` callback that fetches and parses the linked message.
+   */
+  createLinkPreview(url) {
+    const match = SLACK_MESSAGE_URL_PATTERN.exec(url);
+    if (!match) {
+      return { url };
+    }
+    const channel = match[1];
+    const rawTs = match[2];
+    const ts = `${rawTs.slice(0, rawTs.length - 6)}.${rawTs.slice(rawTs.length - 6)}`;
+    const threadId = this.encodeThreadId({ channel, threadTs: ts });
+    return {
+      url,
+      fetchMessage: async () => {
+        const result = await this.client.conversations.history(
+          this.withToken({
+            channel,
+            latest: ts,
+            inclusive: true,
+            limit: 1
+          })
+        );
+        const messages = result.messages || [];
+        const target = messages.find((msg) => msg.ts === ts);
+        if (!target) {
+          throw new Error(`Message not found: ${url}`);
+        }
+        return this.parseSlackMessage(target, threadId);
+      }
+    };
+  }
   async parseSlackMessage(event, threadId, options) {
     const isMe = this.isMessageFromSelf(event);
     const skipSelfMention = options?.skipSelfMention ?? true;
@@ -1756,7 +1885,8 @@ var SlackAdapter = class _SlackAdapter {
       },
       attachments: (event.files || []).map(
         (file) => this.createAttachment(file)
-      )
+      ),
+      links: this.extractLinks(event)
     });
   }
   /**
@@ -1794,6 +1924,13 @@ var SlackAdapter = class _SlackAdapter {
             `Failed to fetch file: ${response.status} ${response.statusText}`
           );
         }
+        const contentType = response.headers.get("content-type") ?? "";
+        if (contentType.includes("text/html")) {
+          throw new NetworkError(
+            "slack",
+            `Failed to download file from Slack: received HTML login page instead of file data. Ensure your Slack app has the "files:read" OAuth scope. URL: ${url}`
+          );
+        }
         const arrayBuffer = await response.arrayBuffer();
         return Buffer.from(arrayBuffer);
       } : void 0
@@ -2769,7 +2906,8 @@ var SlackAdapter = class _SlackAdapter {
       },
       attachments: (event.files || []).map(
         (file) => this.createAttachment(file)
-      )
+      ),
+      links: this.extractLinks(event)
     });
   }
   // =========================================================================