@chat-adapter/slack 4.16.1 → 4.18.0

package/dist/index.d.ts

@@ -59,6 +59,12 @@ declare class SlackFormatConverter extends BaseFormatConverter {
      * Parse Slack mrkdwn into an AST.
      */
     toAst(mrkdwn: string): Root;
+    /**
+     * 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: Root): SlackBlock[] | null;
     private nodeToMrkdwn;
 }
 
@@ -81,10 +87,10 @@ interface SlackAdapterConfig {
      * Defaults to `slack:installation`. The full key will be `{prefix}:{teamId}`.
      */
     installationKeyPrefix?: string;
-    /** Logger instance for error reporting */
-    logger: Logger;
-    /** Signing secret for webhook verification */
-    signingSecret: string;
+    /** Logger instance for error reporting. Defaults to ConsoleLogger. */
+    logger?: Logger;
+    /** Signing secret for webhook verification. Defaults to SLACK_SIGNING_SECRET env var. */
+    signingSecret?: string;
     /** Override bot username (optional) */
     userName?: string;
 }
@@ -163,7 +169,7 @@ declare class SlackAdapter implements Adapter<SlackThreadId, unknown> {
     private readonly requestContext;
     /** Bot user ID (e.g., U_BOT_123) used for mention detection */
     get botUserId(): string | undefined;
-    constructor(config: SlackAdapterConfig);
+    constructor(config?: SlackAdapterConfig);
     /**
      * Get the current bot token for API calls.
      * Checks request context (multi-workspace) → default token (single-workspace) → throws.
@@ -307,6 +313,11 @@ declare class SlackAdapter implements Adapter<SlackThreadId, unknown> {
      * Includes a fetchData method that uses the bot token for auth.
      */
     private createAttachment;
+    /**
+     * 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>;
     openModal(triggerId: string, modal: ModalElement, contextId?: string): Promise<{
@@ -445,6 +456,6 @@ declare class SlackAdapter implements Adapter<SlackThreadId, unknown> {
      */
     private sendToResponseUrl;
 }
-declare function createSlackAdapter(config?: Partial<SlackAdapterConfig>): SlackAdapter;
+declare function createSlackAdapter(config?: SlackAdapterConfig): SlackAdapter;
 
 export { type EncryptedTokenData, SlackAdapter, type SlackAdapterConfig, type SlackEvent, SlackFormatConverter, type SlackInstallation, SlackFormatConverter as SlackMarkdownConverter, type SlackReactionEvent, type SlackThreadId, cardToBlockKit, cardToFallbackText, createSlackAdapter, decodeKey };

package/dist/index.js

@@ -17,6 +17,7 @@ import {
   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;
@@ -664,18 +736,28 @@ var SlackAdapter = class _SlackAdapter {
     }
     return this._botUserId || void 0;
   }
-  constructor(config) {
-    this.client = new WebClient(config.botToken);
-    this.signingSecret = config.signingSecret;
-    this.defaultBotToken = config.botToken;
-    this.logger = config.logger;
+  constructor(config = {}) {
+    const signingSecret = config.signingSecret ?? process.env.SLACK_SIGNING_SECRET;
+    if (!signingSecret) {
+      throw new ValidationError(
+        "slack",
+        "signingSecret is required. Set SLACK_SIGNING_SECRET or provide it in config."
+      );
+    }
+    const zeroConfig = !(config.signingSecret || config.botToken || config.clientId || config.clientSecret);
+    const botToken = config.botToken ?? (zeroConfig ? process.env.SLACK_BOT_TOKEN : void 0);
+    this.client = new WebClient(botToken);
+    this.signingSecret = signingSecret;
+    this.defaultBotToken = botToken;
+    this.logger = config.logger ?? new ConsoleLogger("info").child("slack");
     this.userName = config.userName || "bot";
     this._botUserId = config.botUserId || null;
-    this.clientId = config.clientId;
-    this.clientSecret = config.clientSecret;
+    this.clientId = config.clientId ?? (zeroConfig ? process.env.SLACK_CLIENT_ID : void 0);
+    this.clientSecret = config.clientSecret ?? (zeroConfig ? process.env.SLACK_CLIENT_SECRET : void 0);
     this.installationKeyPrefix = config.installationKeyPrefix ?? "slack:installation";
-    if (config.encryptionKey) {
-      this.encryptionKey = decodeKey(config.encryptionKey);
+    const encryptionKey = config.encryptionKey ?? process.env.SLACK_ENCRYPTION_KEY;
+    if (encryptionKey) {
+      this.encryptionKey = decodeKey(encryptionKey);
     }
   }
   /**
@@ -1281,7 +1363,28 @@ var SlackAdapter = class _SlackAdapter {
       this.logger.warn("Chat instance not initialized, ignoring event");
       return;
     }
-    if (event.subtype && event.subtype !== "bot_message") {
+    const ignoredSubtypes = /* @__PURE__ */ new Set([
+      "message_changed",
+      "message_deleted",
+      "message_replied",
+      "channel_join",
+      "channel_leave",
+      "channel_topic",
+      "channel_purpose",
+      "channel_name",
+      "channel_archive",
+      "channel_unarchive",
+      "group_join",
+      "group_leave",
+      "group_topic",
+      "group_purpose",
+      "group_name",
+      "group_archive",
+      "group_unarchive",
+      "ekm_access_denied",
+      "tombstone"
+    ]);
+    if (event.subtype && ignoredSubtypes.has(event.subtype)) {
       this.logger.debug("Ignoring message subtype", {
         subtype: event.subtype
       });
@@ -1538,12 +1641,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;
@@ -1561,10 +1674,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);
@@ -1641,13 +1773,39 @@ 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 {
       const files = extractFiles(message);
       if (files.length > 0) {
         await this.uploadFiles(files, channel, threadTs || void 0);
-        const hasText = typeof message === "string" || typeof message === "object" && message !== null && ("raw" in message || "markdown" in message || "ast" in message);
+        const hasText = typeof message === "string" || typeof message === "object" && message !== null && ("raw" in message && message.raw || "markdown" in message && message.markdown || "ast" in message && message.ast);
         const card2 = extractCard(message);
         if (!(hasText || card2)) {
           return {
@@ -1687,6 +1845,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"
@@ -1751,6 +1936,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"
@@ -1927,6 +2140,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"
@@ -2733,13 +2971,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;
@@ -2778,26 +3025,7 @@ var SlackAdapter = class _SlackAdapter {
   }
 };
 function createSlackAdapter(config) {
-  const signingSecret = config?.signingSecret ?? process.env.SLACK_SIGNING_SECRET;
-  if (!signingSecret) {
-    throw new ValidationError(
-      "slack",
-      "signingSecret is required. Set SLACK_SIGNING_SECRET or provide it in config."
-    );
-  }
-  const zeroConfig = !config;
-  const resolved = {
-    signingSecret,
-    botToken: config?.botToken ?? (zeroConfig ? process.env.SLACK_BOT_TOKEN : void 0),
-    clientId: config?.clientId ?? (zeroConfig ? process.env.SLACK_CLIENT_ID : void 0),
-    clientSecret: config?.clientSecret ?? (zeroConfig ? process.env.SLACK_CLIENT_SECRET : void 0),
-    encryptionKey: config?.encryptionKey ?? process.env.SLACK_ENCRYPTION_KEY,
-    installationKeyPrefix: config?.installationKeyPrefix,
-    logger: config?.logger ?? new ConsoleLogger("info").child("slack"),
-    userName: config?.userName,
-    botUserId: config?.botUserId
-  };
-  return new SlackAdapter(resolved);
+  return new SlackAdapter(config ?? {});
 }
 export {
   SlackAdapter,