Add multiple recipients (To/CC/BCC) to Send Email workflow action #17385

neo773 · 2026-01-23T09:27:16Z

Adds support for multiple recipients in the Send Email workflow action

greptile-apps · 2026-01-23T09:30:54Z

Greptile Overview

Greptile Summary

This PR successfully extends the Send Email workflow action to support multiple recipients across To, CC, and BCC fields while maintaining backward compatibility with existing workflows using the single email field.

Key Changes:

Added EmailRecipientsInput component with dynamic row management for entering multiple email addresses
Updated WorkflowEditActionSendEmail to use three separate recipient input fields (To/CC/BCC)
Refactored SendEmailTool backend to normalize and validate multiple recipients from both new and legacy formats
Extended MessagingSendMessageService to handle email arrays for all providers (Google, Microsoft, SMTP)
Added toMicrosoftRecipients utility with comprehensive test coverage
Updated schemas and types across frontend, backend, and shared packages

Implementation Quality:

Clean separation of concerns with dedicated normalization and validation methods
Proper error handling with descriptive messages for invalid emails
Comprehensive Storybook stories including legacy format testing
Good test coverage for the Microsoft recipients utility

Confidence Score: 5/5

This PR is safe to merge with minimal risk
The implementation is well-structured with proper backward compatibility, comprehensive validation, and good test coverage. The changes are additive and don't break existing functionality. Code follows repository conventions and patterns.
No files require special attention

Important Files Changed

Filename	Overview
packages/twenty-front/src/modules/workflow/workflow-steps/workflow-actions/components/EmailRecipientsInput.tsx	New component for managing multiple email recipients with dynamic add/remove functionality
packages/twenty-front/src/modules/workflow/workflow-steps/workflow-actions/components/WorkflowEditActionSendEmail.tsx	Updated to support multiple recipients (To/CC/BCC) with backward compatibility for legacy email field
packages/twenty-server/src/engine/core-modules/tool/tools/send-email-tool/send-email-tool.schema.ts	Added EmailRecipientsZodSchema with validation for to/cc/bcc arrays, maintains backward compatibility with legacy email field
packages/twenty-server/src/engine/core-modules/tool/tools/send-email-tool/send-email-tool.ts	Refactored to support multiple recipients with normalizeRecipients and validateEmails methods, maintains backward compatibility
packages/twenty-server/src/modules/messaging/message-import-manager/services/messaging-send-message.service.ts	Updated SendMessageInput type to accept string arrays for cc/bcc, uses toMicrosoftRecipients helper for Microsoft provider

Sequence Diagram

sequenceDiagram
    participant User
    participant EmailRecipientsInput
    participant WorkflowEditActionSendEmail
    participant SendEmailTool
    participant MessagingSendMessageService
    participant EmailProvider

    User->>WorkflowEditActionSendEmail: Configure email action
    WorkflowEditActionSendEmail->>EmailRecipientsInput: Render To/CC/BCC inputs
    Note over EmailRecipientsInput: Initializes with defaultValue<br/>or legacy email field
    
    User->>EmailRecipientsInput: Enter recipient emails
    EmailRecipientsInput->>EmailRecipientsInput: handleEntryChange()
    EmailRecipientsInput->>EmailRecipientsInput: Auto-add empty row if last filled
    EmailRecipientsInput->>WorkflowEditActionSendEmail: onChange(emails[])
    
    WorkflowEditActionSendEmail->>WorkflowEditActionSendEmail: handleFieldChange('recipients', {...})
    WorkflowEditActionSendEmail->>WorkflowEditActionSendEmail: saveAction (debounced)
    
    Note over User,EmailProvider: Workflow Execution Phase
    
    User->>SendEmailTool: Execute workflow
    SendEmailTool->>SendEmailTool: normalizeRecipients()
    Note over SendEmailTool: Handles both new recipients<br/>format and legacy email field
    
    SendEmailTool->>SendEmailTool: validateEmails()
    Note over SendEmailTool: Validates all to/cc/bcc emails
    
    alt Invalid emails found
        SendEmailTool-->>User: Return error with invalid emails
    end
    
    SendEmailTool->>MessagingSendMessageService: sendMessage({to[], cc[], bcc[]})
    
    alt Google Provider
        MessagingSendMessageService->>MessagingSendMessageService: Build MailComposer with arrays
        MessagingSendMessageService->>EmailProvider: Gmail API send
    else Microsoft Provider
        MessagingSendMessageService->>MessagingSendMessageService: toMicrosoftRecipients()
        MessagingSendMessageService->>EmailProvider: Microsoft Graph API send
    else SMTP Provider
        MessagingSendMessageService->>MessagingSendMessageService: Build MailComposer with arrays
        MessagingSendMessageService->>EmailProvider: SMTP send
    end
    
    EmailProvider-->>MessagingSendMessageService: Success
    MessagingSendMessageService-->>SendEmailTool: Email sent
    SendEmailTool-->>User: Success with recipient details

greptile-apps

_{1 file reviewed, 1 comment}

_{Edit Code Review Agent Settings | Greptile}

greptile-apps · 2026-01-23T09:30:58Z

...ont/src/modules/workflow/workflow-steps/workflow-actions/components/EmailRecipientsInput.tsx

+
+    const nonEmptyValues = newEntries
+      .map((entry) => entry.value.trim())
+      .filter((value) => value.length > 0);


Use isNonEmptyString utility instead of manual check.

Suggested change

.filter((value) => value.length > 0);

.filter((value) => isNonEmptyString(value));

Context Used: Context from dashboard - Use the 'isNonEmptyString' utility function to check for non-empty strings instead of checking for e... (source)

_{Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!}

Prompt To Fix With AI

This is a comment left during a code review. Path: packages/twenty-front/src/modules/workflow/workflow-steps/workflow-actions/components/EmailRecipientsInput.tsx Line: 66:66 Comment: Use `isNonEmptyString` utility instead of manual check. ```suggestion .filter((value) => isNonEmptyString(value)); ``` **Context Used:** Context from `dashboard` - Use the 'isNonEmptyString' utility function to check for non-empty strings instead of checking for e... ([source](https://app.greptile.com/review/custom-context?memory=1553f120-8908-4962-bc22-796896959d06)) <sub>Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!</sub> How can I resolve this? If you propose a fix, please make it concise.

cubic-dev-ai

1 issue found across 11 files

Prompt for AI agents (all issues)


Check if these issues are valid — if so, understand the root cause of each and fix them.


<file name="packages/twenty-shared/src/workflow/schemas/send-email-action-settings-schema.ts">

<violation number="1" location="packages/twenty-shared/src/workflow/schemas/send-email-action-settings-schema.ts:15">
P2: The schema now allows a Send Email action with no recipients at all (both `email` and `recipients` can be missing), which can lead to invalid workflows or runtime errors when sending. Add validation to require at least one recipient (legacy `email` or any of `recipients.to/cc/bcc`).</violation>
</file>

_{Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.}

cubic-dev-ai · 2026-01-23T09:31:58Z

packages/twenty-shared/src/workflow/schemas/send-email-action-settings-schema.ts

    input: z.object({
      connectedAccountId: z.string(),
-      email: z.string(),
+      email: z.string().optional(),


P2: The schema now allows a Send Email action with no recipients at all (both email and recipients can be missing), which can lead to invalid workflows or runtime errors when sending. Add validation to require at least one recipient (legacy email or any of recipients.to/cc/bcc).

Prompt for AI agents

Check if this issue is valid — if so, understand the root cause and fix it. At packages/twenty-shared/src/workflow/schemas/send-email-action-settings-schema.ts, line 15: <comment>The schema now allows a Send Email action with no recipients at all (both `email` and `recipients` can be missing), which can lead to invalid workflows or runtime errors when sending. Add validation to require at least one recipient (legacy `email` or any of `recipients.to/cc/bcc`).</comment> <file context> @@ -2,11 +2,18 @@ import { z } from 'zod'; input: z.object({ connectedAccountId: z.string(), - email: z.string(), + email: z.string().optional(), + recipients: workflowEmailRecipientsSchema.optional(), subject: z.string().optional(), </file context>

github-actions · 2026-01-23T09:39:48Z

🚀 Preview Environment Ready!

Your preview environment is available at: http://bore.pub:39807

This environment will automatically shut down when the PR is closed or after 5 hours.

greptile-apps

_{No files reviewed, no comments}

_{Edit Code Review Agent Settings | Greptile}

Add multiple recipients support for Send Email workflow action

5c47f62

neo773 closed this Jan 23, 2026

greptile-apps bot reviewed Jan 23, 2026

View reviewed changes

cubic-dev-ai bot reviewed Jan 23, 2026

View reviewed changes

neo773 added 2 commits January 23, 2026 15:38

fix gmail bcc

055762f

fix bcc for real

bf5d253

neo773 reopened this Jan 23, 2026

greptile-apps bot reviewed Jan 23, 2026

View reviewed changes

fix front lint

4c90a4d

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add multiple recipients (To/CC/BCC) to Send Email workflow action #17385

Add multiple recipients (To/CC/BCC) to Send Email workflow action #17385

Uh oh!

neo773 commented Jan 23, 2026 •

edited

Loading

Uh oh!

greptile-apps bot commented Jan 23, 2026 •

edited

Loading

Uh oh!

greptile-apps bot left a comment

Uh oh!

greptile-apps bot Jan 23, 2026

Uh oh!

cubic-dev-ai bot left a comment

Uh oh!

cubic-dev-ai bot Jan 23, 2026 •

edited

Loading

Uh oh!

github-actions bot commented Jan 23, 2026 •

edited

Loading

Uh oh!

greptile-apps bot left a comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

	.filter((value) => value.length > 0);
	.filter((value) => isNonEmptyString(value));

Add multiple recipients (To/CC/BCC) to Send Email workflow action #17385

Are you sure you want to change the base?

Add multiple recipients (To/CC/BCC) to Send Email workflow action #17385

Uh oh!

Conversation

neo773 commented Jan 23, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

greptile-apps bot commented Jan 23, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Greptile Overview

Greptile Summary

Confidence Score: 5/5

Important Files Changed

Sequence Diagram

Uh oh!

greptile-apps bot left a comment

Choose a reason for hiding this comment

Uh oh!

greptile-apps bot Jan 23, 2026

Choose a reason for hiding this comment

Uh oh!

cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

Uh oh!

cubic-dev-ai bot Jan 23, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

github-actions bot commented Jan 23, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

greptile-apps bot left a comment

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

neo773 commented Jan 23, 2026 •

edited

Loading

greptile-apps bot commented Jan 23, 2026 •

edited

Loading

cubic-dev-ai bot Jan 23, 2026 •

edited

Loading

github-actions bot commented Jan 23, 2026 •

edited

Loading