fix(smartlead): correct request params and outputs found in re-validation

Three tools sent a parameter Smartlead's validator rejects outright with 400,
so the affected operations failed whenever the field was filled in:

- get_campaign_lead_statistics paginated with `skip`; the endpoint accepts
  `offset` and only echoes it back as `skip`.
- list_lead_activities and list_inbox_replies both sent a campaign filter.
  `campaign_id`, `campaignId`, `campaign_ids` and `email_campaign_id` are all
  rejected, so the filter is gone rather than advertised and broken.

mark_lead_complete reported `next_sequence: null` on every call, including when
a step remained: `status.nextSequence` is an object, not a number. It now maps
to `next_sequence_id` and `next_sequence_delay_in_days` — verified live
returning step 10093171 rather than null.

get_lead_by_id reused the by-email mapper, so it always claimed the lead belongs
to zero campaigns; `GET /leads/{id}` omits `lead_campaign_data` entirely. It now
declares the narrower shape it actually returns.

A stale advanced `clientId` leaked into list_email_accounts: advanced subblocks
serialize without evaluating their condition, and that tool consumes `clientId`
while sitting outside its condition list. The field is now offered for that
operation too, so the value is visible wherever it is sent.

Two dropdowns had defaults that act on their own. `status` defaulted to PAUSED,
so choosing Update Campaign Status and never opening the dropdown paused the
campaign; it now requires an explicit choice. `pauseLead` sent `false` on every
categorization, which risks resuming a paused lead; it now defaults to leaving
the state alone.

Also counts CSV export rows with a quote-aware scan so a newline inside a name,
location, or custom field no longer inflates the count, and fills in the block
output declarations for the fields the 47 tools actually return.

Author: waleedlatif1

apps/docs/content/docs/en/integrations/smartlead.mdx

@@ -183,11 +183,13 @@ export const SmartleadBlock: BlockConfig<SmartleadResponse> = {
       title: 'Status',
       type: 'dropdown',
       options: [
+        { label: 'Select a status', id: '' },
         { label: 'Start', id: 'START' },
         { label: 'Pause', id: 'PAUSED' },
         { label: 'Stop', id: 'STOPPED' },
       ],
-      value: () => 'PAUSED',
+      // No default: a materialized default would let an untouched dropdown pause a campaign.
+      value: () => '',
       required: { field: 'operation', value: 'update_campaign_status' },
       condition: { field: 'operation', value: 'update_campaign_status' },
     },
@@ -427,20 +429,15 @@ export const SmartleadBlock: BlockConfig<SmartleadResponse> = {
       },
       mode: 'advanced',
     },
-    {
-      id: 'skip',
-      title: 'Skip',
-      type: 'short-input',
-      placeholder: '0',
-      condition: { field: 'operation', value: 'get_campaign_lead_statistics' },
-      mode: 'advanced',
-    },
     {
       id: 'clientId',
       title: 'Client ID',
       type: 'short-input',
       placeholder: 'Agency client ID',
-      condition: { field: 'operation', value: ['list_campaigns', 'create_campaign'] },
+      condition: {
+        field: 'operation',
+        value: ['list_campaigns', 'create_campaign', 'list_email_accounts'],
+      },
       mode: 'advanced',
     },
     {
@@ -581,10 +578,11 @@ export const SmartleadBlock: BlockConfig<SmartleadResponse> = {
       title: 'Pause Lead',
       type: 'dropdown',
       options: [
+        { label: 'Leave unchanged', id: '' },
         { label: 'No', id: 'false' },
         { label: 'Yes', id: 'true' },
       ],
-      value: () => 'false',
+      value: () => '',
       condition: { field: 'operation', value: 'update_lead_category' },
       mode: 'advanced',
     },
@@ -821,7 +819,6 @@ export const SmartleadBlock: BlockConfig<SmartleadResponse> = {
         fromTime: emptyToUndefined(params.fromTime),
         toTime: emptyToUndefined(params.toTime),
         unreadOnly: toBooleanParam(params.unreadOnly),
-        skip: toNumberParam(params.skip),
         offset: toNumberParam(params.offset),
         limit: toNumberParam(params.limit),
         emailSequenceNumber: toNumberParam(params.emailSequenceNumber),
@@ -833,6 +830,9 @@ export const SmartleadBlock: BlockConfig<SmartleadResponse> = {
         pauseLead: toBooleanParam(params.pauseLead),
         name: params.operation === 'create_campaign' ? params.campaignName : params.webhookName,
         email: emptyToUndefined(params.leadEmail),
+        // Empty means "not chosen", so required validation reports it instead of
+        // the API rejecting an empty status.
+        status: emptyToUndefined(params.status),
         emailStatus: emptyToUndefined(params.emailStatus),
         stopLeadSettings: emptyToUndefined(params.stopLeadSettings),
         daysOfTheWeek: parseNumberList(params.daysOfTheWeek),
@@ -929,7 +929,6 @@ export const SmartleadBlock: BlockConfig<SmartleadResponse> = {
     emailSequenceNumber: { type: 'number', description: 'Sequence step filter' },
     offset: { type: 'number', description: 'Pagination offset' },
     limit: { type: 'number', description: 'Pagination limit' },
-    skip: { type: 'number', description: 'Rows to skip' },
     deleteWebhookId: { type: 'number', description: 'Webhook ID to delete' },
     fromTime: { type: 'string', description: 'Start of the reporting window' },
     toTime: { type: 'string', description: 'End of the reporting window' },
@@ -977,6 +976,56 @@ export const SmartleadBlock: BlockConfig<SmartleadResponse> = {
     has_more: { type: 'boolean', description: 'Whether more rows are available' },
     list_name: { type: 'string', description: 'Lead list name' },
     positive_reply_count: { type: 'number', description: 'Replies categorized as positive' },
+    offset: { type: 'number', description: 'Pagination offset used' },
+    limit: { type: 'number', description: 'Pagination limit used' },
+    total_count: { type: 'number', description: 'Total records matching the request' },
+    from: { type: 'string', description: 'Start of the reported window' },
+    to: { type: 'string', description: 'End of the reported window' },
+    start_date: { type: 'string', description: 'Start of the reported range' },
+    end_date: { type: 'string', description: 'End of the reported range' },
+    skipped_count: { type: 'number', description: 'Emails skipped' },
+    failed_count: { type: 'number', description: 'Failed sends' },
+    stopped_count: { type: 'number', description: 'Stopped leads' },
+    unsubscribed_count: { type: 'number', description: 'Unsubscribes' },
+    unique_sent_count: { type: 'number', description: 'Unique leads emailed' },
+    unique_open_count: { type: 'number', description: 'Unique opens' },
+    unique_click_count: { type: 'number', description: 'Unique clicks' },
+    block_count: {
+      type: 'number',
+      description: 'Blocked sends, or leads skipped by the block list',
+    },
+    drafted_count: { type: 'number', description: 'Drafted emails' },
+    sequence_count: { type: 'number', description: 'Sequence steps in the campaign' },
+    duplicate_count: { type: 'number', description: 'Duplicate leads skipped on import' },
+    invalid_email_count: { type: 'number', description: 'Leads skipped for an invalid email' },
+    invalid_emails: { type: 'array', description: 'Emails rejected as invalid' },
+    already_added_to_campaign: { type: 'number', description: 'Leads already in the campaign' },
+    unsubscribed_leads: { type: 'array', description: 'Leads skipped because they unsubscribed' },
+    lead_import_stopped_count: { type: 'number', description: 'Leads whose import was stopped' },
+    is_lead_limit_exhausted: {
+      type: 'boolean',
+      description: 'Whether the plan lead limit was hit',
+    },
+    is_last_sequence: { type: 'boolean', description: 'Whether the lead was on the final step' },
+    next_sequence_id: { type: 'number', description: 'ID of the next sequence step' },
+    next_sequence_delay_in_days: { type: 'number', description: 'Days before the next step' },
+    first_name: { type: 'string', description: 'Lead first name' },
+    last_name: { type: 'string', description: 'Lead last name' },
+    phone_number: { type: 'string', description: 'Lead phone number' },
+    company_name: { type: 'string', description: 'Lead company name' },
+    website: { type: 'string', description: 'Lead website' },
+    location: { type: 'string', description: 'Lead location' },
+    linkedin_profile: { type: 'string', description: 'Lead LinkedIn profile URL' },
+    company_url: { type: 'string', description: 'Lead company URL' },
+    custom_fields: { type: 'json', description: 'Lead custom fields' },
+    is_unsubscribed: { type: 'boolean', description: 'Whether the lead is unsubscribed' },
+    lead_campaign_data: { type: 'array', description: 'Campaigns the lead belongs to' },
+    created_at: { type: 'string', description: 'Creation timestamp' },
+    updated_at: { type: 'string', description: 'Last update timestamp' },
+    leads_count: { type: 'number', description: 'Leads in the list' },
+    active_leads_count: { type: 'number', description: 'Active leads in the list' },
+    track_settings: { type: 'array', description: 'Disabled tracking settings' },
+    scheduler_cron_value: { type: 'json', description: 'Campaign sending schedule' },
   },
 }
 

apps/sim/blocks/blocks/smartlead.ts

@@ -12,6 +12,39 @@ import {
 } from '@/tools/smartlead/utils'
 import type { ToolConfig } from '@/tools/types'
 
+/**
+ * Counts data rows excluding the header, treating newlines inside quoted fields
+ * as part of the value — lead names, locations, and custom fields can contain them.
+ */
+function countCsvDataRows(csv: string): number {
+  let rows = 0
+  let inQuotes = false
+  let lineHasContent = false
+
+  for (let i = 0; i < csv.length; i++) {
+    const char = csv[i]
+    if (char === '"') {
+      // A doubled quote inside a quoted field is an escaped literal quote.
+      if (inQuotes && csv[i + 1] === '"') {
+        i++
+        continue
+      }
+      inQuotes = !inQuotes
+      lineHasContent = true
+      continue
+    }
+    if (char === '\n' && !inQuotes) {
+      if (lineHasContent) rows++
+      lineHasContent = false
+      continue
+    }
+    if (char !== '\r') lineHasContent = true
+  }
+  if (lineHasContent) rows++
+
+  return Math.max(0, rows - 1)
+}
+
 export const exportCampaignLeadsTool: ToolConfig<
   SmartleadCampaignIdParams,
   SmartleadExportLeadsResponse
@@ -34,14 +67,12 @@ export const exportCampaignLeadsTool: ToolConfig<
   },
   transformResponse: async (response) => {
     const csv = await response.text()
-    // The first line is the header row, so data rows are the remaining non-empty lines.
-    const rowCount = csv.split('\n').filter((line) => line.trim() !== '').length
 
     return {
       success: true,
       output: {
         csv,
-        row_count: Math.max(0, rowCount - 1),
+        row_count: countCsvDataRows(csv),
       },
     }
   },

apps/sim/tools/generated/tool-metadata.ts

@@ -17,7 +17,7 @@ import type { ToolConfig } from '@/tools/types'
 
 interface GetCampaignLeadStatisticsParams extends SmartleadCampaignIdParams {
   limit?: number
-  skip?: number
+  offset?: number
 }
 
 export const getCampaignLeadStatisticsTool: ToolConfig<
@@ -38,18 +38,19 @@ export const getCampaignLeadStatisticsTool: ToolConfig<
       visibility: 'user-or-llm',
       description: 'Rows to return (default 100)',
     },
-    skip: {
+    offset: {
       type: 'number',
       required: false,
       visibility: 'user-or-llm',
       description: 'Rows to skip (default 0)',
     },
   },
   request: {
+    // Smartlead takes `offset` here but echoes it back as `skip`.
     url: (params) =>
       smartleadUrl(`/campaigns/${pathSegment(params.campaignId)}/leads-statistics`, params.apiKey, {
         limit: params.limit,
-        skip: params.skip,
+        offset: params.offset,
       }),
     method: 'GET',
     headers: smartleadHeaders,

apps/sim/tools/generated/tool-outputs.ts

@@ -1,8 +1,8 @@
 import { ErrorExtractorId } from '@/tools/error-extractors'
 import type { SmartleadBaseParams, SmartleadLeadByIdResponse } from '@/tools/smartlead/types'
 import {
-  leadDetailOutputs,
-  mapLeadDetail,
+  leadRecordOutputs,
+  mapLead,
   pathSegment,
   smartleadBaseParamFields,
   smartleadHeaders,
@@ -41,11 +41,15 @@ export const getLeadByIdTool: ToolConfig<GetLeadByIdParams, SmartleadLeadByIdRes
     // This endpoint wraps the lead in a single-element `data` array.
     const rows = Array.isArray(record.data) ? record.data : []
     if (rows.length === 0) throw new Error('Smartlead lead not found')
+    const lead = rows[0] as Record<string, unknown>
 
     return {
       success: true,
-      output: mapLeadDetail(rows[0]),
+      output: {
+        ...mapLead(lead),
+        created_at: typeof lead.created_at === 'string' ? lead.created_at : null,
+      },
     }
   },
-  outputs: leadDetailOutputs,
+  outputs: leadRecordOutputs,
 }

apps/sim/tools/smartlead/export_campaign_leads.ts

@@ -11,10 +11,13 @@ import {
 } from '@/tools/smartlead/utils'
 import type { ToolConfig } from '@/tools/types'
 
+/**
+ * Smartlead rejects every campaign-filter key this endpoint might plausibly take
+ * (`campaign_id`, `campaign_ids`, `campaignIds`), so only pagination is offered.
+ */
 interface ListInboxRepliesParams extends SmartleadBaseParams {
   offset?: number
   limit?: number
-  campaignId?: number
   unreadOnly?: boolean
 }
 
@@ -36,12 +39,6 @@ export const listInboxRepliesTool: ToolConfig<
       visibility: 'user-or-llm',
       description: 'Return only unread replies',
     },
-    campaignId: {
-      type: 'number',
-      required: false,
-      visibility: 'user-or-llm',
-      description: 'Only return replies for this campaign',
-    },
     offset: {
       type: 'number',
       required: false,
@@ -67,7 +64,6 @@ export const listInboxRepliesTool: ToolConfig<
       jsonBody({
         offset: params.offset ?? 0,
         limit: params.limit,
-        campaign_id: params.campaignId,
       }),
   },
   transformResponse: async (response) => {

apps/sim/tools/smartlead/get_campaign_lead_statistics.ts

@@ -11,10 +11,14 @@ import {
 } from '@/tools/smartlead/utils'
 import type { ToolConfig } from '@/tools/types'
 
+/**
+ * Smartlead exposes no working campaign filter on this endpoint — `campaign_id`,
+ * `campaignId`, `campaign_ids` and `email_campaign_id` are all rejected by its
+ * validator, so only pagination is offered.
+ */
 interface ListLeadActivitiesParams extends SmartleadBaseParams {
   offset?: number
   limit?: number
-  campaignId?: number
 }
 
 export const listLeadActivitiesTool: ToolConfig<
@@ -29,12 +33,6 @@ export const listLeadActivitiesTool: ToolConfig<
   errorExtractor: ErrorExtractorId.SMARTLEAD_ERRORS,
   params: {
     ...smartleadBaseParamFields,
-    campaignId: {
-      type: 'number',
-      required: false,
-      visibility: 'user-or-llm',
-      description: 'Only return activity for this campaign',
-    },
     offset: {
       type: 'number',
       required: false,
@@ -53,7 +51,6 @@ export const listLeadActivitiesTool: ToolConfig<
       smartleadUrl('/campaigns/all-leads-activities', params.apiKey, {
         offset: params.offset,
         limit: params.limit,
-        campaign_id: params.campaignId,
       }),
     method: 'GET',
     headers: smartleadHeaders,

apps/sim/tools/smartlead/get_lead_by_id.ts

@@ -54,13 +54,16 @@ export const markLeadCompleteTool: ToolConfig<
   transformResponse: async (response) => {
     const record = await smartleadRecord(response, 'lead completion')
     const status = isRecordLike(record.status) ? record.status : {}
+    // `nextSequence` is an object ({ id, delayInDays }) when a step remains, else null.
+    const next = isRecordLike(status.nextSequence) ? status.nextSequence : null
 
     return {
       success: true,
       output: {
         success: isOk(record),
         is_last_sequence: typeof status.isLastSequence === 'boolean' ? status.isLastSequence : null,
-        next_sequence: typeof status.nextSequence === 'number' ? status.nextSequence : null,
+        next_sequence_id: next && typeof next.id === 'number' ? next.id : null,
+        next_sequence_delay_in_days: next ? Number(next.delayInDays) || null : null,
       },
     }
   },