fix(smartlead): remove the dead campaign field that could target the wrong campaign

Removing the campaign filters from list_lead_activities and list_inbox_replies
left their `activityCampaignId` subblock, its params mapping, and its inputs
entry behind. Two problems, the second serious:

- On those two operations the field promised campaign scoping the API cannot
  do. Smartlead rejects every candidate key (`campaign_id`, `campaignId`,
  `campaign_ids`, `email_campaign_id`), so the value was silently discarded and
  account-wide results were reported as scoped.
- Worse, the field is `mode: 'advanced'`, and advanced subblocks serialize
  without evaluating their condition. A value left over from listing activities
  therefore fed `campaignId` on all 32 campaign operations through the
  `params.campaignId || params.activityCampaignId` fallback. Configuring List
  Lead Activities with campaign 111, then switching the block to Delete
  Campaign and leaving Campaign ID blank, would have passed required-validation
  and deleted campaign 111.

Both list tools now also say plainly that Smartlead exposes no campaign filter,
rather than advertising one in their descriptions.

Also: route mark_lead_complete's next-sequence id through the shared numeric
coercion, since Smartlead string-encodes numbers inconsistently and its sibling
field already arrives as a string; re-bind the two enum constants that lost
their last consumer so the literal descriptions cannot drift undetected; and
declare the 17 tool output keys the block was missing — `accounts` most
importantly, which is the entire payload of both email-account tools.

Author: waleedlatif1

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

@@ -418,17 +418,6 @@ export const SmartleadBlock: BlockConfig<SmartleadResponse> = {
       value: () => 'false',
       condition: { field: 'operation', value: 'list_inbox_replies' },
     },
-    {
-      id: 'activityCampaignId',
-      title: 'Campaign ID',
-      type: 'short-input',
-      placeholder: 'Leave empty for all campaigns',
-      condition: {
-        field: 'operation',
-        value: ['list_lead_activities', 'list_inbox_replies'],
-      },
-      mode: 'advanced',
-    },
     {
       id: 'clientId',
       title: 'Client ID',
@@ -803,9 +792,7 @@ export const SmartleadBlock: BlockConfig<SmartleadResponse> = {
     config: {
       tool: (params) => `smartlead_${params.operation}`,
       params: (params) => ({
-        // `list_lead_activities` and `list_inbox_replies` scope by campaign through their
-        // own optional field, since they are not campaign-scoped operations.
-        campaignId: toNumberParam(params.campaignId || params.activityCampaignId),
+        campaignId: toNumberParam(params.campaignId),
         leadId: toNumberParam(params.leadId),
         clientId: toNumberParam(params.clientId),
         categoryId: toNumberParam(params.categoryId),
@@ -937,7 +924,6 @@ export const SmartleadBlock: BlockConfig<SmartleadResponse> = {
     leadListId: { type: 'number', description: 'Lead list ID' },
     listName: { type: 'string', description: 'Lead list name' },
     unreadOnly: { type: 'boolean', description: 'Return only unread inbox replies' },
-    activityCampaignId: { type: 'number', description: 'Campaign to scope results to' },
   },
   outputs: {
     campaigns: { type: 'array', description: 'List of campaigns' },
@@ -1026,6 +1012,26 @@ export const SmartleadBlock: BlockConfig<SmartleadResponse> = {
     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' },
+    accounts: {
+      type: 'array',
+      description: 'Sending email accounts, excluding their stored mailbox credentials',
+    },
+    user_id: { type: 'number', description: 'Owning Smartlead user ID' },
+    min_time_btwn_emails: { type: 'number', description: 'Minimum minutes between emails' },
+    max_leads_per_day: { type: 'number', description: 'Maximum new leads per day' },
+    stop_lead_settings: { type: 'string', description: 'Activity that stops a lead sequence' },
+    schedule_start_time: { type: 'string', description: 'Scheduled start time' },
+    enable_ai_esp_matching: { type: 'boolean', description: 'Whether AI ESP matching is enabled' },
+    send_as_plain_text: { type: 'boolean', description: 'Whether emails send as plain text' },
+    follow_up_percentage: { type: 'number', description: 'Follow-up percentage' },
+    unsubscribe_text: { type: 'string', description: 'Unsubscribe text' },
+    parent_campaign_id: { type: 'number', description: 'Parent campaign ID' },
+    client_id: { type: 'number', description: 'Client ID for agency accounts' },
+    client_name: { type: 'string', description: 'Client name' },
+    client_email: { type: 'string', description: 'Client email' },
+    client_company_name: { type: 'string', description: 'Client company name' },
+    tags: { type: 'array', description: 'Tags on the record' },
+    email_campaign_id: { type: 'number', description: 'Campaign the webhook belongs to' },
   },
 }
 

apps/sim/blocks/blocks/smartlead.ts

@@ -1,5 +1,5 @@
 {
-  "updatedAt": "2026-08-06",
+  "updatedAt": "2026-08-07",
   "integrations": [
     {
       "type": "onepassword",
@@ -17867,7 +17867,7 @@
         },
         {
           "name": "List Lead Activities",
-          "description": "Retrieves recent lead activity across Smartlead campaigns — opens, clicks, replies, and status changes."
+          "description": "Retrieves recent lead activity across all Smartlead campaigns — opens, clicks, replies, and status changes. Smartlead exposes no campaign filter on this endpoint."
         },
         {
           "name": "Get Lead by ID",
@@ -17891,7 +17891,7 @@
         },
         {
           "name": "List Inbox Replies",
-          "description": "Retrieves replies from the Smartlead master inbox, optionally limited to unread replies or a single campaign."
+          "description": "Retrieves replies from the Smartlead master inbox across all campaigns, optionally limited to unread replies. Smartlead exposes no campaign filter on this endpoint."
         },
         {
           "name": "List Lead Lists",

apps/sim/lib/integrations/integrations.json

@@ -28,7 +28,7 @@ export const listInboxRepliesTool: ToolConfig<
   id: 'smartlead_list_inbox_replies',
   name: 'Smartlead List Inbox Replies',
   description:
-    'Retrieves replies from the Smartlead master inbox, optionally limited to unread replies or a single campaign.',
+    'Retrieves replies from the Smartlead master inbox across all campaigns, optionally limited to unread replies. Smartlead exposes no campaign filter on this endpoint.',
   version: '1.0.0',
   errorExtractor: ErrorExtractorId.SMARTLEAD_ERRORS,
   params: {

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

@@ -28,7 +28,7 @@ export const listLeadActivitiesTool: ToolConfig<
   id: 'smartlead_list_lead_activities',
   name: 'Smartlead List Lead Activities',
   description:
-    'Retrieves recent lead activity across Smartlead campaigns — opens, clicks, replies, and status changes.',
+    'Retrieves recent lead activity across all Smartlead campaigns — opens, clicks, replies, and status changes. Smartlead exposes no campaign filter on this endpoint.',
   version: '1.0.0',
   errorExtractor: ErrorExtractorId.SMARTLEAD_ERRORS,
   params: {

apps/sim/tools/smartlead/list_inbox_replies.ts

@@ -13,6 +13,7 @@ import {
   smartleadHeaders,
   smartleadRecord,
   smartleadUrl,
+  toNullableNumber,
 } from '@/tools/smartlead/utils'
 import type { ToolConfig } from '@/tools/types'
 
@@ -55,17 +56,17 @@ export const markLeadCompleteTool: ToolConfig<
     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.
-    // `delayInDays` arrives as a string and is legitimately "0" for an immediate step.
+    // Both members go through the shared numeric coercion: Smartlead string-encodes
+    // numbers inconsistently, and `delayInDays` is legitimately "0" for an immediate step.
     const next = isRecordLike(status.nextSequence) ? status.nextSequence : null
-    const delay = next === null ? Number.NaN : Number(next.delayInDays)
 
     return {
       success: true,
       output: {
         success: isOk(record),
         is_last_sequence: typeof status.isLastSequence === 'boolean' ? status.isLastSequence : null,
-        next_sequence_id: next && typeof next.id === 'number' ? next.id : null,
-        next_sequence_delay_in_days: Number.isFinite(delay) ? delay : null,
+        next_sequence_id: next === null ? null : toNullableNumber(next.id),
+        next_sequence_delay_in_days: next === null ? null : toNullableNumber(next.delayInDays),
       },
     }
   },

apps/sim/tools/smartlead/list_lead_activities.ts

@@ -6,6 +6,7 @@ import {
   jsonBody,
   pathSegment,
   type SMARTLEAD_STOP_LEAD_SETTINGS,
+  type SMARTLEAD_TRACK_SETTINGS,
   smartleadBaseParamFields,
   smartleadCampaignIdParamField,
   smartleadHeaders,
@@ -15,7 +16,7 @@ import {
 import type { ToolConfig } from '@/tools/types'
 
 interface UpdateCampaignSettingsParams extends SmartleadCampaignIdParams {
-  trackSettings?: string[]
+  trackSettings?: (typeof SMARTLEAD_TRACK_SETTINGS)[number][]
   stopLeadSettings?: (typeof SMARTLEAD_STOP_LEAD_SETTINGS)[number]
   sendAsPlainText?: boolean
   followUpPercentage?: number

apps/sim/tools/smartlead/mark_lead_complete.ts

@@ -6,6 +6,7 @@ import type {
 import {
   mapSavedWebhook,
   pathSegment,
+  type SMARTLEAD_WEBHOOK_EVENT_TYPES,
   smartleadBaseParamFields,
   smartleadCampaignIdParamField,
   smartleadHeaders,
@@ -18,7 +19,7 @@ import type { ToolConfig } from '@/tools/types'
 interface UpsertCampaignWebhookParams extends SmartleadCampaignIdParams {
   name: string
   webhookUrl: string
-  eventTypes: string[]
+  eventTypes: (typeof SMARTLEAD_WEBHOOK_EVENT_TYPES)[number][]
   categories: string[]
   webhookId?: number
 }

apps/sim/tools/smartlead/update_campaign_settings.ts

@@ -572,7 +572,7 @@ function toStringOrNull(value: unknown): string | null {
 }
 
 /** Normalizes Smartlead's string-encoded numbers (`"0"`, `"3499513771"`) to numbers. */
-function toNullableNumber(value: unknown): number | null {
+export function toNullableNumber(value: unknown): number | null {
   if (value === undefined || value === null) return null
   if (typeof value === 'number') return Number.isFinite(value) ? value : null
   if (typeof value === 'string' && value.trim() !== '') {