fix(credentials): review round 1 — site_not_found modal message, revert pipeline-deals pagination change

- Client-credential connect modal maps site_not_found (bad Salesforce My
  Domain host) to host-specific guidance instead of the generic fallback
- Reverts the pipeline-deals v1→v2 migration: it swapped the saved-workflow
  pagination contract (start/next_start → cursor/next_cursor), silently
  breaking deployed workflows; the v2 move needs a dedicated PR with a
  subblock migration. authStyle typing and helper usage are kept

Author: waleedlatif1

apps/sim/app/workspace/[workspaceId]/integrations/components/connect-service-account-modal/client-credential-account-modal.tsx

@@ -39,6 +39,8 @@ function messageForClientCredentialError(
     switch (err.code) {
       case 'invalid_credentials':
         return `We couldn't authenticate with those credentials. Check that the ${fieldLabels} all belong to the same ${descriptor.serviceLabel} app and that the app is authorized.`
+      case 'site_not_found':
+        return `We couldn't find a ${descriptor.serviceLabel} account at that host. Check the spelling of the host field and try again.`
       case 'provider_unavailable':
         return `We couldn't reach ${descriptor.serviceLabel} to verify these credentials. Try again in a moment.`
       case 'duplicate_display_name':

apps/sim/blocks/blocks/pipedrive.ts

@@ -319,7 +319,7 @@ Return ONLY the date string in YYYY-MM-DD format - no explanations, no quotes, n
       placeholder: 'Pagination cursor from previous response',
       condition: {
         field: 'operation',
-        value: ['get_all_deals', 'get_pipeline_deals', 'get_projects'],
+        value: ['get_all_deals', 'get_projects'],
       },
     },
     {
@@ -329,7 +329,14 @@ Return ONLY the date string in YYYY-MM-DD format - no explanations, no quotes, n
       placeholder: 'Pagination offset (e.g., 0, 100, 200)',
       condition: {
         field: 'operation',
-        value: ['get_activities', 'get_leads', 'get_files', 'get_mail_messages', 'get_pipelines'],
+        value: [
+          'get_activities',
+          'get_leads',
+          'get_files',
+          'get_pipeline_deals',
+          'get_mail_messages',
+          'get_pipelines',
+        ],
       },
     },
     {

apps/sim/tools/pipedrive/get_pipeline_deals.ts

@@ -49,23 +49,25 @@ export const pipedriveGetPipelineDealsTool: ToolConfig<
       visibility: 'user-or-llm',
       description: 'Number of results to return (e.g., "50", default: 100, max: 500)',
     },
-    cursor: {
+    start: {
       type: 'string',
       required: false,
       visibility: 'user-or-llm',
-      description: 'For pagination, the marker representing the first item on the next page',
+      description: 'Pagination start offset (0-based index of the first item to return)',
     },
   },
 
   request: {
     url: (params) => {
-      const queryParams = new URLSearchParams({ pipeline_id: params.pipeline_id })
+      const baseUrl = `https://api.pipedrive.com/v1/pipelines/${params.pipeline_id}/deals`
+      const queryParams = new URLSearchParams()
 
       if (params.stage_id) queryParams.append('stage_id', params.stage_id)
       if (params.limit) queryParams.append('limit', params.limit)
-      if (params.cursor) queryParams.append('cursor', params.cursor)
+      if (params.start) queryParams.append('start', params.start)
 
-      return `https://api.pipedrive.com/api/v2/deals?${queryParams.toString()}`
+      const queryString = queryParams.toString()
+      return queryString ? `${baseUrl}?${queryString}` : baseUrl
     },
     method: 'GET',
     headers: (params) => getPipedriveAuthHeaders(params),
@@ -80,7 +82,8 @@ export const pipedriveGetPipelineDealsTool: ToolConfig<
     }
 
     const deals = data.data || []
-    const nextCursor = data.additional_data?.next_cursor ?? null
+    const hasMore = data.additional_data?.pagination?.more_items_in_collection || false
+    const nextStart = data.additional_data?.pagination?.next_start ?? null
 
     return {
       success: true,
@@ -89,8 +92,8 @@ export const pipedriveGetPipelineDealsTool: ToolConfig<
         metadata: {
           pipeline_id: params?.pipeline_id || '',
           total_items: deals.length,
-          has_more: nextCursor !== null,
-          next_cursor: nextCursor,
+          has_more: hasMore,
+          next_start: nextStart,
         },
         success: true,
       },

apps/sim/tools/pipedrive/types.ts

@@ -529,7 +529,7 @@ export interface PipedriveGetPipelineDealsParams extends PipedriveBaseParams {
   pipeline_id: string
   stage_id?: string
   limit?: string
-  cursor?: string
+  start?: string
 }
 
 interface PipedriveGetPipelineDealsOutput {
@@ -538,7 +538,7 @@ interface PipedriveGetPipelineDealsOutput {
     pipeline_id: string
     total_items: number
     has_more?: boolean
-    next_cursor?: string | null
+    next_start?: number
   }
   success: boolean
 }