Merge pull request #490 from buster-so/staging

Staging

apps/server/src/api/v2/slack/index.ts

@@ -16,9 +16,37 @@ const app = new Hono()
   .delete('/integration', requireAuth, (c) => slackHandler.removeIntegration(c))
   // Events endpoint (no auth required for Slack webhooks)
   .post('/events', async (c) => {
-    const body = await c.req.json();
+    try {
+      // Slack sends different content types for different events
+      // For URL verification, it's application/x-www-form-urlencoded
+      // For actual events, it's application/json
+      const contentType = c.req.header('content-type');
+      
+      if (contentType?.includes('application/x-www-form-urlencoded')) {
+        // Handle URL verification challenge
+        const formData = await c.req.parseBody();
+        if (formData.challenge) {
+          return c.text(formData.challenge as string);
+        }
+      }
+      
+      // For JSON payloads, try to parse but don't fail
+      let body = null;
+      if (contentType?.includes('application/json')) {
+        try {
+          body = await c.req.json();
+        } catch {
+          // If JSON parsing fails, just continue
+        }
+      }
+      
       const response = await eventsHandler(body);
       return c.json(response);
+    } catch (error) {
+      // Log the error but always return 200 OK for Slack
+      console.error('Error processing Slack event:', error);
+      return c.json({ success: true });
+    }
   })
   // Error handling
   .onError((e, c) => {

apps/web/src/mocks/metricData.ts

@@ -62,7 +62,8 @@ const dataMetadata: DataMetadata = {
 const MOCK_DATA: Required<BusterMetricData> = {
   data: mockData(),
   metricId: 'mock-metric-1',
-  data_metadata: dataMetadata
+  data_metadata: dataMetadata,
+  has_more_records: false
 };
 
 export const createMockData = (metricId: string): Required<BusterMetricData> => {