fix(sendgrid): dedupe active coercion between block and tool

waleedlatif1 · waleedlatif1 · commit 4d3663efc15f · 2026-07-02T11:02:18.000-07:00
Per Cursor Bugbot: the block's pre-coercion only recognized the
dropdown string 'true' or boolean true as active, so a dynamic
reference producing numeric 1 or string '1' fell through to 0 and
silently created an inactive template version. Exported the tool's
toActiveFlag and reused it in the block instead of duplicating the
inactive-value logic, so both layers can no longer drift out of sync.
diff --git a/apps/sim/blocks/blocks/sendgrid.ts b/apps/sim/blocks/blocks/sendgrid.ts
@@ -2,6 +2,7 @@ import { SendgridIcon } from '@/components/icons'
 import type { BlockConfig, BlockMeta } from '@/blocks/types'
 import { AuthMode, IntegrationType } from '@/blocks/types'
 import { normalizeFileInput } from '@/blocks/utils'
+import { toActiveFlag } from '@/tools/sendgrid/create_template_version'
 import type { SendMailResult } from '@/tools/sendgrid/types'
 
 export const SendGridBlock: BlockConfig<SendMailResult> = {
@@ -623,7 +624,7 @@ Return ONLY the HTML content.`,
           ...(operation === 'list_templates' &&
             templatePageToken && { pageToken: templatePageToken }),
           ...(normalizedAttachments && { attachments: normalizedAttachments }),
-          ...(active !== undefined && { active: active === 'true' || active === true ? 1 : 0 }),
+          ...(active !== undefined && { active: toActiveFlag(active) }),
         }
       },
     },
diff --git a/apps/sim/tools/sendgrid/create_template_version.ts b/apps/sim/tools/sendgrid/create_template_version.ts
@@ -7,7 +7,10 @@ import type { ToolConfig } from '@/tools/types'
 
 const INACTIVE_VALUES: unknown[] = [false, 'false', 0, '0']
 
-function toActiveFlag(active: CreateTemplateVersionParams['active']): 0 | 1 {
+/** Coerces any dynamic-reference form of SendGrid's active flag (boolean, string, or
+ *  number) to the 0/1 integer the API requires. Shared with the block's own
+ *  pre-coercion in blocks/blocks/sendgrid.ts so both layers stay in sync. */
+export function toActiveFlag(active: unknown): 0 | 1 {
   if (active === undefined) return 1
   return INACTIVE_VALUES.includes(active) ? 0 : 1
 }
