@chat-adapter/slack 4.17.0 → 4.19.0

package/dist/index.js

@@ -3,6 +3,7 @@ import { AsyncLocalStorage } from "async_hooks";
 import { createHmac, timingSafeEqual } from "crypto";
 import {
   AdapterRateLimitError,
+  AuthenticationError,
   extractCard,
   extractFiles,
   NetworkError,
@@ -11,12 +12,12 @@ import {
 } from "@chat-adapter/shared";
 import { WebClient } from "@slack/web-api";
 import {
-  ChatError,
   ConsoleLogger,
   convertEmojiPlaceholders,
   defaultEmojiResolver,
   isJSX,
   Message,
+  parseMarkdown as parseMarkdown2,
   StreamingMarkdownRenderer,
   toModalElement
 } from "chat";
@@ -59,13 +60,14 @@ function cardToBlockKit(card) {
       alt_text: card.title || "Card image"
     });
   }
+  const state = { usedNativeTable: false };
   for (const child of card.children) {
-    const childBlocks = convertChildToBlocks(child);
+    const childBlocks = convertChildToBlocks(child, state);
     blocks.push(...childBlocks);
   }
   return blocks;
 }
-function convertChildToBlocks(child) {
+function convertChildToBlocks(child, state) {
   switch (child.type) {
     case "text":
       return [convertTextToBlock(child)];
@@ -76,13 +78,13 @@ function convertChildToBlocks(child) {
     case "actions":
       return [convertActionsToBlock(child)];
     case "section":
-      return convertSectionToBlocks(child);
+      return convertSectionToBlocks(child, state);
     case "fields":
       return [convertFieldsToBlock(child)];
     case "link":
       return [convertLinkToBlock(child)];
     case "table":
-      return convertTableToBlocks(child);
+      return convertTableToBlocks(child, state);
     default: {
       const text = cardChildToFallbackText(child);
       if (text) {
@@ -250,10 +252,10 @@ function convertRadioSelectToElement(radioSelect) {
   }
   return element;
 }
-function convertTableToBlocks(element) {
+function convertTableToBlocks(element, state) {
   const MAX_ROWS = 100;
   const MAX_COLS = 20;
-  if (element.rows.length > MAX_ROWS || element.headers.length > MAX_COLS) {
+  if (state.usedNativeTable || element.rows.length > MAX_ROWS || element.headers.length > MAX_COLS) {
     return [
       {
         type: "section",
@@ -266,31 +268,28 @@ ${tableElementToAscii(element.headers, element.rows)}
       }
     ];
   }
-  const columns = element.headers.map((header, i) => ({
-    id: `col_${i}`,
-    header: {
-      type: "plain_text",
-      text: convertEmoji(header)
-    }
+  state.usedNativeTable = true;
+  const headerRow = element.headers.map((header) => ({
+    type: "raw_text",
+    text: convertEmoji(header)
   }));
-  const rows = element.rows.map((row) => ({
-    cells: row.map((cell) => ({
-      type: "plain_text",
+  const dataRows = element.rows.map(
+    (row) => row.map((cell) => ({
+      type: "raw_text",
       text: convertEmoji(cell)
     }))
-  }));
+  );
   return [
     {
       type: "table",
-      columns,
-      rows
+      rows: [headerRow, ...dataRows]
     }
   ];
 }
-function convertSectionToBlocks(element) {
+function convertSectionToBlocks(element, state) {
   const blocks = [];
   for (const child of element.children) {
-    blocks.push(...convertChildToBlocks(child));
+    blocks.push(...convertChildToBlocks(child, state));
   }
   return blocks;
 }
@@ -428,16 +427,68 @@ var SlackFormatConverter = class extends BaseFormatConverter {
    */
   toAst(mrkdwn) {
     let markdown = mrkdwn;
-    markdown = markdown.replace(/<@([^|>]+)\|([^>]+)>/g, "@$2");
-    markdown = markdown.replace(/<@([^>]+)>/g, "@$1");
-    markdown = markdown.replace(/<#[^|>]+\|([^>]+)>/g, "#$1");
-    markdown = markdown.replace(/<#([^>]+)>/g, "#$1");
+    markdown = markdown.replace(/<@([A-Z0-9_]+)\|([^>]+)>/g, "@$2");
+    markdown = markdown.replace(/<@([A-Z0-9_]+)>/g, "@$1");
+    markdown = markdown.replace(/<#[A-Z0-9_]+\|([^>]+)>/g, "#$1");
+    markdown = markdown.replace(/<#([A-Z0-9_]+)>/g, "#$1");
     markdown = markdown.replace(/<(https?:\/\/[^|>]+)\|([^>]+)>/g, "[$2]($1)");
     markdown = markdown.replace(/<(https?:\/\/[^>]+)>/g, "$1");
     markdown = markdown.replace(/(?<![_*\\])\*([^*\n]+)\*(?![_*])/g, "**$1**");
     markdown = markdown.replace(/(?<!~)~([^~\n]+)~(?!~)/g, "~~$1~~");
     return parseMarkdown(markdown);
   }
+  /**
+   * Convert AST to Slack blocks, using a native table block for the first table.
+   * Returns null if the AST contains no tables (caller should use regular text).
+   * Slack allows at most one table block per message; additional tables use ASCII.
+   */
+  toBlocksWithTable(ast) {
+    const hasTable = ast.children.some((node) => isTableNode(node));
+    if (!hasTable) {
+      return null;
+    }
+    const blocks = [];
+    let usedNativeTable = false;
+    let textBuffer = [];
+    const flushText = () => {
+      if (textBuffer.length > 0) {
+        const text = textBuffer.join("\n\n");
+        if (text.trim()) {
+          blocks.push({
+            type: "section",
+            text: { type: "mrkdwn", text }
+          });
+        }
+        textBuffer = [];
+      }
+    };
+    for (const child of ast.children) {
+      const node = child;
+      if (isTableNode(node)) {
+        flushText();
+        if (usedNativeTable) {
+          blocks.push({
+            type: "section",
+            text: {
+              type: "mrkdwn",
+              text: `\`\`\`
+${tableToAscii(node)}
+\`\`\``
+            }
+          });
+        } else {
+          blocks.push(
+            mdastTableToSlackBlock(node, this.nodeToMrkdwn.bind(this))
+          );
+          usedNativeTable = true;
+        }
+      } else {
+        textBuffer.push(this.nodeToMrkdwn(node));
+      }
+    }
+    flushText();
+    return blocks;
+  }
   nodeToMrkdwn(node) {
     if (isParagraphNode(node)) {
       return getNodeChildren(node).map((child) => this.nodeToMrkdwn(child)).join("");
@@ -489,6 +540,26 @@ ${tableToAscii(node)}
     return this.defaultNodeToText(node, (child) => this.nodeToMrkdwn(child));
   }
 };
+function mdastTableToSlackBlock(node, cellConverter) {
+  const rows = [];
+  for (const row of node.children) {
+    const cells = getNodeChildren(row).map((cell) => ({
+      type: "raw_text",
+      text: getNodeChildren(cell).map(cellConverter).join("")
+    }));
+    rows.push(cells);
+  }
+  const block = { type: "table", rows };
+  if (node.align) {
+    const columnSettings = node.align.map(
+      (a) => ({
+        align: a || "left"
+      })
+    );
+    block.column_settings = columnSettings;
+  }
+  return block;
+}
 
 // src/modals.ts
 function encodeModalMetadata(meta) {
@@ -636,6 +707,7 @@ function radioSelectToBlock(radioSelect) {
 }
 
 // src/index.ts
+var SLACK_USER_ID_PATTERN = /^[A-Z0-9_]+$/;
 var SlackAdapter = class _SlackAdapter {
   name = "slack";
   userName;
@@ -700,9 +772,9 @@ var SlackAdapter = class _SlackAdapter {
     if (this.defaultBotToken) {
       return this.defaultBotToken;
     }
-    throw new ChatError(
-      "No bot token available. In multi-workspace mode, ensure the webhook is being processed.",
-      "MISSING_BOT_TOKEN"
+    throw new AuthenticationError(
+      "slack",
+      "No bot token available. In multi-workspace mode, ensure the webhook is being processed."
     );
   }
   /**
@@ -747,9 +819,9 @@ var SlackAdapter = class _SlackAdapter {
    */
   async setInstallation(teamId, installation) {
     if (!this.chat) {
-      throw new ChatError(
-        "Adapter not initialized. Ensure chat.initialize() has been called first.",
-        "NOT_INITIALIZED"
+      throw new ValidationError(
+        "slack",
+        "Adapter not initialized. Ensure chat.initialize() has been called first."
       );
     }
     const state = this.chat.getState();
@@ -769,9 +841,9 @@ var SlackAdapter = class _SlackAdapter {
    */
   async getInstallation(teamId) {
     if (!this.chat) {
-      throw new ChatError(
-        "Adapter not initialized. Ensure chat.initialize() has been called first.",
-        "NOT_INITIALIZED"
+      throw new ValidationError(
+        "slack",
+        "Adapter not initialized. Ensure chat.initialize() has been called first."
       );
     }
     const state = this.chat.getState();
@@ -798,17 +870,17 @@ var SlackAdapter = class _SlackAdapter {
    */
   async handleOAuthCallback(request) {
     if (!(this.clientId && this.clientSecret)) {
-      throw new ChatError(
-        "clientId and clientSecret are required for OAuth. Pass them in createSlackAdapter().",
-        "MISSING_OAUTH_CONFIG"
+      throw new ValidationError(
+        "slack",
+        "clientId and clientSecret are required for OAuth. Pass them in createSlackAdapter()."
       );
     }
     const url = new URL(request.url);
     const code = url.searchParams.get("code");
     if (!code) {
-      throw new ChatError(
-        "Missing 'code' query parameter in OAuth callback request.",
-        "MISSING_OAUTH_CODE"
+      throw new ValidationError(
+        "slack",
+        "Missing 'code' query parameter in OAuth callback request."
       );
     }
     const redirectUri = url.searchParams.get("redirect_uri") ?? void 0;
@@ -819,9 +891,9 @@ var SlackAdapter = class _SlackAdapter {
       redirect_uri: redirectUri
     });
     if (!(result.ok && result.access_token && result.team?.id)) {
-      throw new ChatError(
-        `Slack OAuth failed: ${result.error || "missing access_token or team.id"}`,
-        "OAUTH_FAILED"
+      throw new AuthenticationError(
+        "slack",
+        `Slack OAuth failed: ${result.error || "missing access_token or team.id"}`
       );
     }
     const teamId = result.team.id;
@@ -838,9 +910,9 @@ var SlackAdapter = class _SlackAdapter {
    */
   async deleteInstallation(teamId) {
     if (!this.chat) {
-      throw new ChatError(
-        "Adapter not initialized. Ensure chat.initialize() has been called first.",
-        "NOT_INITIALIZED"
+      throw new ValidationError(
+        "slack",
+        "Adapter not initialized. Ensure chat.initialize() has been called first."
       );
     }
     const state = this.chat.getState();
@@ -1257,7 +1329,10 @@ var SlackAdapter = class _SlackAdapter {
     if (isJSX(modal)) {
       const converted = toModalElement(modal);
       if (!converted) {
-        throw new Error("Invalid JSX element: must be a Modal element");
+        throw new ValidationError(
+          "slack",
+          "Invalid JSX element: must be a Modal element"
+        );
       }
       return converted;
     }
@@ -1344,7 +1419,7 @@ var SlackAdapter = class _SlackAdapter {
   /**
    * Handle reaction events from Slack (reaction_added, reaction_removed).
    */
-  handleReactionEvent(event, options) {
+  async handleReactionEvent(event, options) {
     if (!this.chat) {
       this.logger.warn("Chat instance not initialized, ignoring reaction");
       return;
@@ -1355,9 +1430,32 @@ var SlackAdapter = class _SlackAdapter {
       });
       return;
     }
+    let parentTs = event.item.ts;
+    try {
+      const result = await this.client.conversations.replies(
+        this.withToken({
+          channel: event.item.channel,
+          ts: event.item.ts,
+          limit: 1
+        })
+      );
+      const firstMessage = result.messages?.[0];
+      if (firstMessage?.thread_ts) {
+        parentTs = firstMessage.thread_ts;
+      }
+    } catch (error) {
+      this.logger.warn(
+        "Failed to resolve parent thread for reaction, using message ts",
+        {
+          error: String(error),
+          channel: event.item.channel,
+          ts: event.item.ts
+        }
+      );
+    }
     const threadId = this.encodeThreadId({
       channel: event.item.channel,
-      threadTs: event.item.ts
+      threadTs: parentTs
     });
     const messageId = event.item.ts;
     const rawEmoji = event.reaction;
@@ -1569,12 +1667,22 @@ var SlackAdapter = class _SlackAdapter {
    *   detection doesn't apply.
    */
   async resolveInlineMentions(text, skipSelfMention) {
-    const mentionPattern = /<@([A-Z0-9]+)(?:\|[^>]*)?>/g;
     const userIds = /* @__PURE__ */ new Set();
-    let match = mentionPattern.exec(text);
-    while (match) {
-      userIds.add(match[1]);
-      match = mentionPattern.exec(text);
+    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 (userIds.size === 0) {
       return text;
@@ -1592,10 +1700,29 @@ var SlackAdapter = class _SlackAdapter {
       })
     );
     const nameMap = new Map(lookups);
-    return text.replace(/<@([A-Z0-9]+)(?:\|[^>]*)?>/g, (_m, uid) => {
-      const name = nameMap.get(uid);
-      return name ? `<@${uid}|${name}>` : `<@${uid}>`;
-    });
+    let result = "";
+    let remaining = text;
+    let startIdx = remaining.indexOf("<@");
+    while (startIdx !== -1) {
+      result += remaining.slice(0, startIdx);
+      remaining = remaining.slice(startIdx);
+      const endIdx = remaining.indexOf(">");
+      if (endIdx === -1) {
+        break;
+      }
+      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}>`;
+      } else {
+        result += remaining.slice(0, endIdx + 1);
+      }
+      remaining = remaining.slice(endIdx + 1);
+      startIdx = remaining.indexOf("<@");
+    }
+    return result + remaining;
   }
   async parseSlackMessage(event, threadId, options) {
     const isMe = this.isMessageFromSelf(event);
@@ -1672,6 +1799,32 @@ var SlackAdapter = class _SlackAdapter {
       } : void 0
     };
   }
+  /**
+   * Try to render a message using native Slack table blocks.
+   * Returns blocks + fallback text if the message contains tables, null otherwise.
+   */
+  renderWithTableBlocks(message) {
+    let ast = null;
+    if (typeof message === "object" && message !== null) {
+      if ("ast" in message) {
+        ast = message.ast;
+      } else if ("markdown" in message) {
+        ast = parseMarkdown2(message.markdown);
+      }
+    }
+    if (!ast) {
+      return null;
+    }
+    const blocks = this.formatConverter.toBlocksWithTable(ast);
+    if (!blocks) {
+      return null;
+    }
+    const fallbackText = convertEmojiPlaceholders(
+      this.formatConverter.renderPostable(message),
+      "slack"
+    );
+    return { text: fallbackText, blocks };
+  }
   async postMessage(threadId, message) {
     const { channel, threadTs } = this.decodeThreadId(threadId);
     try {
@@ -1718,6 +1871,33 @@ var SlackAdapter = class _SlackAdapter {
           raw: result2
         };
       }
+      const tableResult = this.renderWithTableBlocks(message);
+      if (tableResult) {
+        this.logger.debug("Slack API: chat.postMessage (table blocks)", {
+          channel,
+          threadTs,
+          blockCount: tableResult.blocks.length
+        });
+        const result2 = await this.client.chat.postMessage(
+          this.withToken({
+            channel,
+            thread_ts: threadTs,
+            text: tableResult.text,
+            blocks: tableResult.blocks,
+            unfurl_links: false,
+            unfurl_media: false
+          })
+        );
+        this.logger.debug("Slack API: chat.postMessage response", {
+          messageId: result2.ts,
+          ok: result2.ok
+        });
+        return {
+          id: result2.ts,
+          threadId,
+          raw: result2
+        };
+      }
       const text = convertEmojiPlaceholders(
         this.formatConverter.renderPostable(message),
         "slack"
@@ -1782,6 +1962,34 @@ var SlackAdapter = class _SlackAdapter {
           raw: result2
         };
       }
+      const tableResult = this.renderWithTableBlocks(message);
+      if (tableResult) {
+        this.logger.debug("Slack API: chat.postEphemeral (table blocks)", {
+          channel,
+          threadTs,
+          userId,
+          blockCount: tableResult.blocks.length
+        });
+        const result2 = await this.client.chat.postEphemeral(
+          this.withToken({
+            channel,
+            thread_ts: threadTs || void 0,
+            user: userId,
+            text: tableResult.text,
+            blocks: tableResult.blocks
+          })
+        );
+        this.logger.debug("Slack API: chat.postEphemeral response", {
+          messageTs: result2.message_ts,
+          ok: result2.ok
+        });
+        return {
+          id: result2.message_ts || "",
+          threadId,
+          usedFallback: false,
+          raw: result2
+        };
+      }
       const text = convertEmojiPlaceholders(
         this.formatConverter.renderPostable(message),
         "slack"
@@ -1814,6 +2022,95 @@ var SlackAdapter = class _SlackAdapter {
       this.handleSlackError(error);
     }
   }
+  async scheduleMessage(threadId, message, options) {
+    const { channel, threadTs } = this.decodeThreadId(threadId);
+    const postAtUnix = Math.floor(options.postAt.getTime() / 1e3);
+    if (postAtUnix <= Math.floor(Date.now() / 1e3)) {
+      throw new ValidationError("slack", "postAt must be in the future");
+    }
+    const files = extractFiles(message);
+    if (files.length > 0) {
+      throw new ValidationError(
+        "slack",
+        "File uploads are not supported in scheduled messages"
+      );
+    }
+    const token = this.getToken();
+    try {
+      const card = extractCard(message);
+      if (card) {
+        const blocks = cardToBlockKit(card);
+        const fallbackText = cardToFallbackText(card);
+        this.logger.debug("Slack API: chat.scheduleMessage (blocks)", {
+          channel,
+          threadTs,
+          postAt: postAtUnix,
+          blockCount: blocks.length
+        });
+        const result2 = await this.client.chat.scheduleMessage({
+          token,
+          channel,
+          thread_ts: threadTs || void 0,
+          post_at: postAtUnix,
+          text: fallbackText,
+          blocks,
+          unfurl_links: false,
+          unfurl_media: false
+        });
+        const scheduledMessageId2 = result2.scheduled_message_id;
+        const adapter2 = this;
+        return {
+          scheduledMessageId: scheduledMessageId2,
+          channelId: channel,
+          postAt: options.postAt,
+          raw: result2,
+          async cancel() {
+            await adapter2.client.chat.deleteScheduledMessage({
+              token,
+              channel,
+              scheduled_message_id: scheduledMessageId2
+            });
+          }
+        };
+      }
+      const text = convertEmojiPlaceholders(
+        this.formatConverter.renderPostable(message),
+        "slack"
+      );
+      this.logger.debug("Slack API: chat.scheduleMessage", {
+        channel,
+        threadTs,
+        postAt: postAtUnix,
+        textLength: text.length
+      });
+      const result = await this.client.chat.scheduleMessage({
+        token,
+        channel,
+        thread_ts: threadTs || void 0,
+        post_at: postAtUnix,
+        text,
+        unfurl_links: false,
+        unfurl_media: false
+      });
+      const scheduledMessageId = result.scheduled_message_id;
+      const adapter = this;
+      return {
+        scheduledMessageId,
+        channelId: channel,
+        postAt: options.postAt,
+        raw: result,
+        async cancel() {
+          await adapter.client.chat.deleteScheduledMessage({
+            token,
+            channel,
+            scheduled_message_id: scheduledMessageId
+          });
+        }
+      };
+    } catch (error) {
+      this.handleSlackError(error);
+    }
+  }
   async openModal(triggerId, modal, contextId) {
     const metadata = encodeModalMetadata({
       contextId,
@@ -1958,6 +2255,31 @@ var SlackAdapter = class _SlackAdapter {
           raw: result2
         };
       }
+      const tableResult = this.renderWithTableBlocks(message);
+      if (tableResult) {
+        this.logger.debug("Slack API: chat.update (table blocks)", {
+          channel,
+          messageId,
+          blockCount: tableResult.blocks.length
+        });
+        const result2 = await this.client.chat.update(
+          this.withToken({
+            channel,
+            ts: messageId,
+            text: tableResult.text,
+            blocks: tableResult.blocks
+          })
+        );
+        this.logger.debug("Slack API: chat.update response", {
+          messageId: result2.ts,
+          ok: result2.ok
+        });
+        return {
+          id: result2.ts,
+          threadId,
+          raw: result2
+        };
+      }
       const text = convertEmojiPlaceholders(
         this.formatConverter.renderPostable(message),
         "slack"
@@ -2106,9 +2428,9 @@ var SlackAdapter = class _SlackAdapter {
    */
   async stream(threadId, textStream, options) {
     if (!(options?.recipientUserId && options?.recipientTeamId)) {
-      throw new ChatError(
-        "Slack streaming requires recipientUserId and recipientTeamId in options",
-        "MISSING_STREAM_OPTIONS"
+      throw new ValidationError(
+        "slack",
+        "Slack streaming requires recipientUserId and recipientTeamId in options"
       );
     }
     const { channel, threadTs } = this.decodeThreadId(threadId);
@@ -2751,9 +3073,9 @@ var SlackAdapter = class _SlackAdapter {
     } else {
       const message = options?.message;
       if (!message) {
-        throw new ChatError(
-          "Message required for replace action",
-          "INVALID_ARGS"
+        throw new ValidationError(
+          "slack",
+          "Message required for replace action"
         );
       }
       const card = extractCard(message);
@@ -2764,13 +3086,22 @@ var SlackAdapter = class _SlackAdapter {
           blocks: cardToBlockKit(card)
         };
       } else {
-        payload = {
-          replace_original: true,
-          text: convertEmojiPlaceholders(
-            this.formatConverter.renderPostable(message),
-            "slack"
-          )
-        };
+        const tableResult = this.renderWithTableBlocks(message);
+        if (tableResult) {
+          payload = {
+            replace_original: true,
+            text: tableResult.text,
+            blocks: tableResult.blocks
+          };
+        } else {
+          payload = {
+            replace_original: true,
+            text: convertEmojiPlaceholders(
+              this.formatConverter.renderPostable(message),
+              "slack"
+            )
+          };
+        }
       }
       if (options?.threadTs) {
         payload.thread_ts = options.threadTs;
@@ -2792,9 +3123,9 @@ var SlackAdapter = class _SlackAdapter {
         status: response.status,
         body: errorText
       });
-      throw new ChatError(
-        `Failed to ${action} via response_url: ${errorText}`,
-        response.status.toString()
+      throw new NetworkError(
+        "slack",
+        `Failed to ${action} via response_url: ${errorText}`
       );
     }
     const responseText = await response.text();