fix(slack): address PR review comments on conversation tools

Author: waleedlatif1

apps/sim/tools/slack/create_conversation.ts

@@ -1,24 +1,10 @@
-import type { SlackCreateConversationParams } from '@/tools/slack/types'
+import type {
+  SlackCreateConversationParams,
+  SlackCreateConversationResponse,
+} from '@/tools/slack/types'
 import { CHANNEL_OUTPUT_PROPERTIES } from '@/tools/slack/types'
 import type { ToolConfig } from '@/tools/types'
 
-interface SlackCreateConversationResponse {
-  success: boolean
-  output: {
-    channel: {
-      id: string
-      name: string
-      is_private: boolean
-      is_archived: boolean
-      is_member: boolean
-      topic: string
-      purpose: string
-      created: number
-      creator: string
-    }
-  }
-}
-
 export const slackCreateConversationTool: ToolConfig<
   SlackCreateConversationParams,
   SlackCreateConversationResponse
@@ -137,8 +123,8 @@ export const slackCreateConversationTool: ToolConfig<
           is_member: ch.is_member || false,
           topic: ch.topic?.value || '',
           purpose: ch.purpose?.value || '',
-          created: ch.created ?? null,
-          creator: ch.creator ?? null,
+          created: ch.created,
+          creator: ch.creator,
         },
       },
     }

apps/sim/tools/slack/invite_to_conversation.ts

@@ -1,24 +1,10 @@
-import type { SlackInviteToConversationParams } from '@/tools/slack/types'
+import type {
+  SlackInviteToConversationParams,
+  SlackInviteToConversationResponse,
+} from '@/tools/slack/types'
 import { CHANNEL_OUTPUT_PROPERTIES } from '@/tools/slack/types'
 import type { ToolConfig } from '@/tools/types'
 
-interface SlackInviteToConversationResponse {
-  success: boolean
-  output: {
-    channel: {
-      id: string
-      name: string
-      is_private: boolean
-      is_archived: boolean
-      is_member: boolean
-      topic: string
-      purpose: string
-      created: number
-      creator: string
-    }
-  }
-}
-
 export const slackInviteToConversationTool: ToolConfig<
   SlackInviteToConversationParams,
   SlackInviteToConversationResponse
@@ -149,9 +135,10 @@ export const slackInviteToConversationTool: ToolConfig<
           is_member: ch.is_member || false,
           topic: ch.topic?.value || '',
           purpose: ch.purpose?.value || '',
-          created: ch.created ?? null,
-          creator: ch.creator ?? null,
+          created: ch.created,
+          creator: ch.creator,
         },
+        ...(data.errors?.length ? { errors: data.errors } : {}),
       },
     }
   },
@@ -162,5 +149,18 @@ export const slackInviteToConversationTool: ToolConfig<
       description: 'The channel object after inviting users',
       properties: CHANNEL_OUTPUT_PROPERTIES,
     },
+    errors: {
+      type: 'array',
+      description: 'Per-user errors when force is true and some invitations failed',
+      optional: true,
+      items: {
+        type: 'object',
+        properties: {
+          user: { type: 'string', description: 'User ID that failed' },
+          ok: { type: 'boolean', description: 'Always false for error entries' },
+          error: { type: 'string', description: 'Error code for this user' },
+        },
+      },
+    },
   },
 }

apps/sim/tools/slack/types.ts

@@ -1051,6 +1051,7 @@ export interface SlackCreateConversationResponse extends ToolResponse {
 export interface SlackInviteToConversationResponse extends ToolResponse {
   output: {
     channel: SlackChannel
+    errors?: Array<{ user: string; ok: boolean; error: string }>
   }
 }