fix: slack section limiting

Signed-off-by: Uroš Marolt <uros@marolt.me>

Author: themarolt

services/libs/slack/src/notify.ts

@@ -10,6 +10,7 @@ const log = getServiceLogger()
 // Slack message size limit (keeping it conservative at 30KB)
 const MAX_MESSAGE_SIZE = 30 * 1024 // 30KB in bytes
 const MAX_BLOCKS = 50
+const MAX_SECTION_TEXT = 2900 // Slack hard limit is 3000 chars per section block
 
 interface SlackBlock {
   type: string
@@ -25,32 +26,37 @@ interface SlackBlock {
 }
 
 /**
- * Build content blocks from either a simple string or an array of sections
+ * Build content blocks from either a simple string or an array of sections.
+ * Splits text that exceeds Slack's 3000 char per-section limit at line boundaries.
  */
 function buildContentBlocks(content: string | SlackMessageSection[]): SlackBlock[] {
   if (typeof content === 'string') {
-    // Simple string content - single section
-    return [
-      {
-        type: 'section',
-        text: {
-          type: 'mrkdwn',
-          text: content,
-        },
-      },
-    ]
+    return [{ type: 'section', text: { type: 'mrkdwn', text: content } }]
   }
 
-  // Multiple sections - create a block for each section
   const blocks: SlackBlock[] = []
   for (const section of content) {
-    blocks.push({
-      type: 'section',
-      text: {
-        type: 'mrkdwn',
-        text: `*${section.title}*\n${section.text}`,
-      },
-    })
+    const fullText = `*${section.title}*\n${section.text}`
+    if (fullText.length <= MAX_SECTION_TEXT) {
+      blocks.push({ type: 'section', text: { type: 'mrkdwn', text: fullText } })
+      continue
+    }
+
+    // Split at line boundaries into multiple blocks
+    const lines = fullText.split('\n')
+    let current = ''
+    for (const line of lines) {
+      const candidate = current ? `${current}\n${line}` : line
+      if (candidate.length > MAX_SECTION_TEXT && current) {
+        blocks.push({ type: 'section', text: { type: 'mrkdwn', text: current } })
+        current = line
+      } else {
+        current = candidate
+      }
+    }
+    if (current) {
+      blocks.push({ type: 'section', text: { type: 'mrkdwn', text: current } })
+    }
   }
   return blocks
 }