ack pr comments

Author: waleedlatif1

apps/sim/blocks/blocks/linear.ts

@@ -107,6 +107,12 @@ export const LinearBlock: BlockConfig<LinearResponse> = {
         }
 
         if (params.operation === 'write') {
+          if (!params.title?.trim()) {
+            throw new Error('Title is required for creating issues.')
+          }
+          if (!params.description?.trim()) {
+            throw new Error('Description is required for creating issues.')
+          }
           return {
             credential: params.credential,
             teamId: effectiveTeamId,

apps/sim/tools/browser_use/types.ts

@@ -33,6 +33,6 @@ export interface BrowserUseResponse extends ToolResponse {
     id: string
     success: boolean
     output: any
-    steps: any[]
+    steps: BrowserUseTaskStep[]
   }
 }

apps/sim/tools/google/types.ts

@@ -24,21 +24,3 @@ export interface GoogleSearchResponse extends ToolResponse {
     }
   }
 }
-
-export interface GoogleSearchResponse extends ToolResponse {
-  output: {
-    items: Array<{
-      title: string
-      link: string
-      snippet: string
-      displayLink?: string
-      pagemap?: Record<string, any>
-    }>
-    searchInformation: {
-      totalResults: string
-      searchTime: number
-      formattedSearchTime: string
-      formattedTotalResults: string
-    }
-  }
-}

apps/sim/tools/google_calendar/types.ts

@@ -283,3 +283,4 @@ export type GoogleCalendarResponse =
   | GoogleCalendarGetResponse
   | GoogleCalendarQuickAddResponse
   | GoogleCalendarInviteResponse
+  | GoogleCalendarUpdateResponse

apps/sim/tools/perplexity/types.ts

@@ -25,15 +25,3 @@ export interface PerplexityChatResponse extends ToolResponse {
     }
   }
 }
-
-export interface PerplexityChatResponse extends ToolResponse {
-  output: {
-    content: string
-    model: string
-    usage: {
-      prompt_tokens: number
-      completion_tokens: number
-      total_tokens: number
-    }
-  }
-}

apps/sim/tools/stagehand/types.ts

@@ -101,19 +101,3 @@ export function jsonSchemaToZod(jsonSchema: Record<string, any>): z.ZodTypeAny {
       return z.any()
   }
 }
-
-export interface StagehandAgentResponse extends ToolResponse {
-  output: {
-    agentResult: {
-      success: boolean
-      completed: boolean
-      message: string
-      actions: Array<{
-        type: string
-        params: Record<string, any>
-        result: Record<string, any>
-      }>
-    }
-    structuredOutput?: Record<string, any>
-  }
-}

apps/sim/tools/supabase/types.ts

@@ -13,26 +13,16 @@ export interface SupabaseInsertParams {
   data: any
 }
 
-export interface SupabaseQueryResponse extends ToolResponse {
-  error?: string
+export interface SupabaseBaseResponse extends ToolResponse {
   output: {
     message: string
     results: any
   }
-}
-
-export interface SupabaseInsertResponse extends ToolResponse {
   error?: string
-  output: {
-    message: string
-    results: any
-  }
 }
 
-export interface SupabaseResponse extends ToolResponse {
-  output: {
-    message: string
-    results: any
-  }
-  error?: string
-}
+export interface SupabaseQueryResponse extends SupabaseBaseResponse {}
+
+export interface SupabaseInsertResponse extends SupabaseBaseResponse {}
+
+export interface SupabaseResponse extends SupabaseBaseResponse {}

apps/sim/tools/typeform/types.ts

@@ -106,72 +106,7 @@ export interface TypeformResponsesResponse extends ToolResponse {
 
 export interface TypeformResponse extends ToolResponse {
   output:
-    | {
-        total_items: number
-        page_count: number
-        items: Array<{
-          landing_id: string
-          token: string
-          landed_at: string
-          submitted_at: string
-          metadata: {
-            user_agent: string
-            platform: string
-            referer: string
-            network_id: string
-            browser: string
-          }
-          answers: Array<{
-            field: {
-              id: string
-              type: string
-              ref: string
-            }
-            type: string
-            [key: string]: any // For different answer types (text, boolean, number, etc.)
-          }>
-          hidden: Record<string, any>
-          calculated: {
-            score: number
-          }
-          variables: Array<{
-            key: string
-            type: string
-            [key: string]: any // For different variable types
-          }>
-        }>
-      }
-    | {
-        fileUrl: string
-        contentType: string
-        filename: string
-      }
-    | {
-        fields: Array<{
-          dropoffs: number
-          id: string
-          label: string
-          ref: string
-          title: string
-          type: string
-          views: number
-        }>
-        form: {
-          platforms: Array<{
-            average_time: number
-            completion_rate: number
-            platform: string
-            responses_count: number
-            total_visits: number
-            unique_visits: number
-          }>
-          summary: {
-            average_time: number
-            completion_rate: number
-            responses_count: number
-            total_visits: number
-            unique_visits: number
-          }
-        }
-      }
+    | TypeformResponsesResponse['output']
+    | TypeformFilesResponse['output']
+    | TypeformInsightsData
 }

apps/sim/tools/whatsapp/send_message.ts

@@ -1,10 +1,10 @@
 import { createLogger } from '@/lib/logs/console-logger'
 import type { ToolConfig } from '../types'
-import type { WhatsAppResponse } from './types'
+import type { WhatsAppResponse, WhatsAppSendMessageParams } from './types'
 
 const logger = createLogger('WhatsAppSendMessageTool')
 
-export const sendMessageTool: ToolConfig<any, WhatsAppResponse> = {
+export const sendMessageTool: ToolConfig<WhatsAppSendMessageParams, WhatsAppResponse> = {
   id: 'whatsapp_send_message',
   name: 'WhatsApp',
   description: 'Send WhatsApp messages',

apps/sim/tools/whatsapp/types.ts

@@ -1,5 +1,12 @@
 import type { ToolResponse } from '../types'
 
+export interface WhatsAppSendMessageParams {
+  phoneNumber: string
+  message: string
+  phoneNumberId: string
+  accessToken: string
+}
+
 export interface WhatsAppResponse extends ToolResponse {
   output: {
     success: boolean