Skip to content

Add Communication send email tool#690

Merged
KarishmaGhiya merged 47 commits intomicrosoft:mainfrom
KarishmaGhiya:communication-email
Oct 10, 2025
Merged

Add Communication send email tool#690
KarishmaGhiya merged 47 commits intomicrosoft:mainfrom
KarishmaGhiya:communication-email

Conversation

@KarishmaGhiya
Copy link
Copy Markdown
Member

@KarishmaGhiya KarishmaGhiya commented Oct 3, 2025

What does this PR do?

Communication namespace - send email tool

GitHub issue number?

[Link to the GitHub issue this PR addresses]

Pre-merge Checklist

  • Required for All PRs
    • Read contribution guidelines
    • PR title clearly describes the change
    • Commit history is clean with descriptive messages (cleanup guide)
    • Added comprehensive tests for new/modified functionality
    • Updated servers/Azure.Mcp.Server/CHANGELOG.md and/or servers/Fabric.Mcp.Server/CHANGELOG.md for product changes (features, bug fixes, UI/UX, updated dependencies)
  • For MCP tool changes:
    • One tool per PR: This PR adds or modifies only one MCP tool for faster review cycles
    • Updated servers/Azure.Mcp.Server/README.md and/or servers/Fabric.Mcp.Server/README.md documentation
    • Updated command list in /docs/azmcp-commands.md and/or /docs/fabric-commands.md
    • For new or modified tool descriptions, ran ToolDescriptionEvaluator and obtained a score of 0.4 or more and a top 3 ranking for all related test prompts
    • For new tools associated with Azure services or publicly available tools/APIs/products, add URL to documentation in the PR description
  • Extra steps for Azure MCP Server tool changes:
    • Updated test prompts in /docs/e2eTestPrompts.md
    • 👉 For Community (non-Microsoft team member) PRs:
      • Security review: Reviewed code for security vulnerabilities, malicious code, or suspicious activities before running tests (crypto mining, spam, data exfiltration, etc.)
      • Manual tests run: added comment /azp run mcp - pullrequest - live to run Live Test Pipeline

@github-project-automation github-project-automation Bot moved this to Untriaged in Azure MCP Server Oct 3, 2025
@KarishmaGhiya KarishmaGhiya moved this from Untriaged to In Progress in Azure MCP Server Oct 3, 2025
@KarishmaGhiya KarishmaGhiya added this to the 2025-10 milestone Oct 3, 2025
@KarishmaGhiya KarishmaGhiya self-assigned this Oct 3, 2025
@KarishmaGhiya KarishmaGhiya force-pushed the communication-email branch 4 times, most recently from ba21c26 to 4ef8321 Compare October 3, 2025 23:12
@KarishmaGhiya KarishmaGhiya marked this pull request as ready for review October 7, 2025 23:07
@KarishmaGhiya KarishmaGhiya requested a review from a team as a code owner October 7, 2025 23:07
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds a new Communication email send tool that enables sending emails through Azure Communication Services. The implementation includes the service layer, command implementation, options definitions, model classes, and comprehensive test coverage.

  • Adds email sending functionality to Azure Communication Services MCP tools
  • Implements full email options including HTML content, CC/BCC, reply-to addresses
  • Provides comprehensive unit and live test coverage for the new email functionality

Reviewed Changes

Copilot reviewed 17 out of 17 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
CommunicationService.cs Implements SendEmailAsync method with Azure Communication Email client integration
EmailSendCommand.cs Command implementation for email sending with options binding and validation
ICommunicationService.cs Interface definition for the SendEmailAsync method
EmailSendOptions.cs Options class defining email-specific parameters
EmailSendResult.cs Model class for email send operation results
CommunicationOptionDefinitions.cs Option definitions for email-related command line parameters
BaseCommunicationCommand.cs Enhanced base class with subscription and resource group options
CommunicationSetup.cs Service registration and command group setup for email functionality
CommunicationJsonContext.cs JSON serialization context updates for email models
Azure.Mcp.Tools.Communication.csproj Package reference addition for Azure.Communication.Email
Directory.Packages.props Package version management for Azure.Communication.Email
EmailSendCommandTests.cs Comprehensive unit tests for email command functionality
CommunicationServiceTests.cs Unit tests for communication service email functionality
EmailSendCommandLiveTests.cs Live integration tests for email sending
azmcp-commands.md Documentation updates with email command examples
e2eTestPrompts.md Test prompts for email functionality validation

Comment thread tools/Azure.Mcp.Tools.Communication/src/Services/CommunicationService.cs Outdated
Comment thread tools/Azure.Mcp.Tools.Communication/src/Options/CommunicationOptionDefinitions.cs Outdated
@KarishmaGhiya KarishmaGhiya force-pushed the communication-email branch 2 times, most recently from 96e7514 to 2f6819d Compare October 8, 2025 19:17
@KarishmaGhiya KarishmaGhiya requested a review from jongio October 8, 2025 19:21
Comment thread global.json Outdated
Comment thread tools/Azure.Mcp.Tools.Communication/src/Commands/BaseCommunicationCommand.cs Outdated
Comment thread tools/Azure.Mcp.Tools.Communication/src/Commands/Email/EmailSendCommand.cs Outdated
Comment thread tools/Azure.Mcp.Tools.Communication/src/Commands/Email/EmailSendCommand.cs Outdated
Comment thread servers/Azure.Mcp.Server/CHANGELOG.md Outdated
Comment thread servers/Azure.Mcp.Server/docs/azmcp-commands.md Outdated
Comment thread servers/Azure.Mcp.Server/docs/azmcp-commands.md Outdated
Comment thread servers/Azure.Mcp.Server/docs/azmcp-commands.md Outdated
Comment thread servers/Azure.Mcp.Server/docs/azmcp-commands.md Outdated
Comment thread servers/Azure.Mcp.Server/docs/azmcp-commands.md
Comment thread tools/Azure.Mcp.Tools.Communication/src/Commands/Email/EmailSendCommand.cs Outdated
Comment thread servers/Azure.Mcp.Server/docs/azmcp-commands.md
Comment thread tools/Azure.Mcp.Tools.Communication/src/Commands/Email/EmailSendCommand.cs Outdated
@KarishmaGhiya KarishmaGhiya merged commit 4fcf13f into microsoft:main Oct 10, 2025
26 checks passed
@github-project-automation github-project-automation Bot moved this from In Progress to Done in Azure MCP Server Oct 10, 2025
@KarishmaGhiya KarishmaGhiya deleted the communication-email branch October 10, 2025 06:56
colbytimm pushed a commit to colbytimm/microsoft-mcp that referenced this pull request Dec 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

7 participants