More verbose logging for threads and searches

Author: keoghpe

src/commands/mcp-tools/search.ts

@@ -8,6 +8,7 @@ import { searchSlackMessages } from '../../services/slack-services';
 const queryDescription = `
 Search query with Slack search modifiers.
 Available modifiers: in:<channel/user>, from:<user>, has:<emoji reaction>, is:thread, before:YYYY-MM-DD, after:YYYY-MM-DD, has:pin, with:<user>.
+Filtering by date is supported, but you cannot filter by a specific time.
 Identify users with "@me", "@display.name" or "<@U12345>".
 Identify channels with "#channel-name" or "<#C12345>".
 Exclude results with a dash (-) in front of the modifier.

src/commands/mcp-tools/thread-replies.ts

@@ -12,7 +12,7 @@ const threadRepliesParams = z.object({
   ts: z
     .string()
     .describe(
-      'Unique identifier of either a thread’s parent message or a message in the thread. ts must be the timestamp of an existing message with 0 or more replies. If there are no replies then just the single message referenced by ts will return - it is just an ordinary, unthreaded message. convert TS to the form 1234567890.123456',
+      'Unique identifier of either a thread\'s parent message or a message in the thread. ts must be the timestamp of an existing message with 0 or more replies. If there are no replies then just the single message referenced by ts will return - it is just an ordinary, unthreaded message. convert TS to the form 1234567890.123456',
     ),
   limit: z
     .number()
@@ -43,7 +43,22 @@ export const threadRepliesTool = tool({
     const formattedReplies = result.replies.map((reply) => {
       const user = result.entities[reply.user ?? '']?.displayName || reply.user;
       const time = reply.ts ? new Date(parseInt(reply.ts) * 1000).toLocaleString() : 'Unknown time';
-      return `${user} - ${time}: ${reply.text}`;
+      let base = `${user} - ${time}: ${reply.text}`;
+
+      // Add context from attachments fields if present
+      if (Array.isArray(reply.attachments)) {
+        reply.attachments.forEach((attachment) => {
+          if (Array.isArray(attachment.fields)) {
+            attachment.fields.forEach((field) => {
+              if (field.title || field.value) {
+                base += `\n  *${field.title || ''}*: ${field.value || ''}`;
+              }
+            });
+          }
+        });
+      }
+
+      return base;
     });
 
     return objectToMarkdown({

src/services/formatting-service.ts

@@ -1,4 +1,4 @@
-import { Match } from '@slack/web-api/dist/types/response/SearchMessagesResponse';
+import { Match, MatchBlock } from '@slack/web-api/dist/types/response/SearchMessagesResponse';
 import { GlobalContext } from '../context';
 import { SlackCache, ThreadMessage } from '../commands/my_messages/types';
 import { objectToMarkdown } from '../utils/markdown-utils';
@@ -47,6 +47,33 @@ export function generateSearchResultsMarkdown(
   });
 }
 
+
+function formatSlackBlocksToMarkdown(blocks: MatchBlock[]) {
+  return blocks.map(msg => {
+    // Format timestamp
+    let header = '';
+    let section = '';
+
+    if (Array.isArray(blocks)) {
+      for (const block of blocks) {
+        if (block.type === 'header' && block.text && block.text.text) {
+          header = block.text.text;
+        }
+        if (block.type === 'section' && block.text && block.text.text) {
+          section = block.text.text;
+        }
+      }
+    }
+
+    // Build markdown
+    return [
+      `### [${header || 'Message'}]`,
+      section ? `\n${section}` : '',
+      '---'
+    ].filter(Boolean).join('\n');
+  }).join('\n\n');
+}
+
 export function formatMessage(
   message: Match,
   cache: SlackCache,
@@ -64,7 +91,16 @@ export function formatMessage(
   const messageTs = message.ts || '';
   const permalink = message.permalink || '';
 
-  const formattedText = formatSlackText(message.text || '', cache);
+  // need to handle blocks as well
+  let messageText = '';
+
+  if (message.blocks && message.blocks.length > 0) {
+    messageText = formatSlackBlocksToMarkdown(message.blocks);
+  } else {
+    messageText = message.text || '';
+  }
+
+  const formattedText = formatSlackText(messageText, cache);
   const messageLines = formattedText.split('\n');
 
   let threadIndicator = '';

test/unit/services/formatting-service.test.ts

@@ -111,6 +111,37 @@ describe('Formatting Service', () => {
       expect(earlierIndex).toBeLessThan(laterIndex);
     });
 
+    it('should format search results with blocks', () => {
+      const messages: Match[] = [
+        {
+          channel: { id: 'C123' },
+          ts: '1609459200.000000',
+          text: '',
+          permalink: 'https://slack.com/message/1',
+          blocks: [
+            {
+              type: 'header',
+              block_id: 'm2cmJ',
+              text: { type: 'plain_text', text: ':canary: Deploying to my server (canary)', emoji: true },
+            },
+            {
+              type: 'section',
+              block_id: '9Dggb',
+              text: { type: 'mrkdwn', text: 'Revision `8750b93` contains:\n- <@U03STEKTYLV>: Some cool code <https://github.com/Shopify/cool-code/pull/105498|#105498>', verbatim: false },
+            },
+          ],
+        },
+      ];
+
+      const result = generateSearchResultsMarkdown(messages, mockCache);
+
+      expect(result).toContain(':canary: Deploying to my server (canary)');
+      expect(result).toContain('Shopify/cool-code');
+      expect(result).toContain('Revision `8750b93` contains:');
+      expect(result).toContain('Some cool code');
+    });
+
+
     it('should handle messages without user data', () => {
       const messages: Match[] = [
         {