greptile fixes

apps/trigger/package.json

@@ -22,6 +22,7 @@
     "@buster/server-shared": "workspace:*",
     "@buster/ai": "workspace:*",
     "@buster/database": "workspace:*",
+    "@buster/slack": "workspace:*",
     "@buster/test-utils": "workspace:*",
     "@buster/typescript-config": "workspace:*",
     "@buster/vitest-config": "workspace:*",

apps/trigger/src/tasks/message-post-processing/helpers/slack-notifier.ts

@@ -9,6 +9,7 @@ import {
   slackIntegrations,
   slackMessageTracking,
 } from '@buster/database';
+import { SlackMessageType } from '@buster/slack';
 import { logger } from '@trigger.dev/sdk/v3';
 
 export interface SlackNotificationParams {
@@ -369,7 +370,10 @@ function formatSlackReplyMessage(params: SlackReplyNotificationParams): SlackMes
     };
   }
 
-  throw new Error('Invalid reply notification parameters');
+  throw new Error(
+    'Invalid reply notification parameters: Missing required fields. ' +
+      'Requires either formattedMessage, summaryTitle with summaryMessage, or toolCalled="flagChat" with message'
+  );
 }
 
 /**
@@ -449,7 +453,9 @@ function formatSlackMessage(params: SlackNotificationParams): SlackMessage {
   }
 
   // This shouldn't happen if shouldSendSlackNotification is working correctly
-  throw new Error('Invalid notification parameters');
+  throw new Error(
+    `Invalid notification parameters: Missing required fields. Requires either formattedMessage, summaryTitle with summaryMessage, or toolCalled="flagChat" with message. Received: formattedMessage=${!!params.formattedMessage}, summaryTitle=${!!params.summaryTitle}, summaryMessage=${!!params.summaryMessage}, toolCalled="${params.toolCalled}", message=${!!params.message}`
+  );
 }
 
 /**
@@ -528,7 +534,7 @@ export async function trackSlackNotification(params: {
           slackChannelId: params.channelId,
           slackMessageTs: params.messageTs,
           slackThreadTs: params.threadTs || null,
-          messageType: params.threadTs ? 'reply' : 'message',
+          messageType: params.threadTs ? SlackMessageType.REPLY : SlackMessageType.MESSAGE,
           content: params.slackBlocks
             ? JSON.stringify({ blocks: params.slackBlocks })
             : params.summaryTitle && params.summaryMessage

packages/slack/src/types/index.ts

@@ -99,3 +99,10 @@ export const SendMessageResultSchema = z.object({
 });
 
 export type SendMessageResult = z.infer<typeof SendMessageResultSchema>;
+
+// Message Type enum for tracking
+export enum SlackMessageType {
+  MESSAGE = 'message',
+  REPLY = 'reply',
+  UPDATE = 'update',
+}

pnpm-lock.yaml

@@ -163,6 +163,9 @@ importers:
       '@buster/server-shared':
         specifier: workspace:*
         version: link:../../packages/server-shared
+      '@buster/slack':
+        specifier: workspace:*
+        version: link:../../packages/slack
       '@buster/test-utils':
         specifier: workspace:*
         version: link:../../packages/test-utils