Skip to content

Conversation

@abdulanu0
Copy link
Contributor

No description provided.

Copilot AI review requested due to automatic review settings January 7, 2026 19:10
@abdulanu0 abdulanu0 requested a review from a team as a code owner January 7, 2026 19:10
Copy link
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 pull request implements proper EmailResponse entity wrapping for email notification responses across multiple agent samples in both Python and Node.js implementations. The changes ensure that when agents respond to email notifications, the responses are properly formatted using the EmailResponse entity type as required by the Agent 365 framework.

Key Changes

  • Updated email notification handlers to use createEmailResponseActivity() helper (Node.js) or EmailResponse entity wrapping (Python) for proper response formatting
  • Replaced placeholder notification handlers with complete email notification processing implementations that retrieve email content and generate appropriate responses
  • Fixed spelling errors in notification messages ("Recieved" → "Received")

Reviewed changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 6 comments.

Show a summary per file
File Description
python/claude/sample-agent/host_agent_server.py Added EmailResponse entity wrapping for email notification responses with conditional check for notification type
nodejs/vercel-sdk/sample-agent/src/agent.ts Implemented full email notification handler using createEmailResponseActivity with error handling
nodejs/perplexity/sample-agent/src/notificationService.ts Replaced stream.sendFinal with createEmailResponseActivity for proper email response formatting
nodejs/openai/sample-agent/src/agent.ts Implemented full email notification handler using createEmailResponseActivity with error handling
nodejs/n8n/sample-agent/src/n8nAgent.ts Wrapped email response in createEmailResponseActivity entity
nodejs/langchain/sample-agent/src/agent.ts Implemented full email notification handler using createEmailResponseActivity with error handling
nodejs/devin/sample-agent/src/agent.ts Added complete agent notification handler with email-specific processing using createEmailResponseActivity
nodejs/claude/sample-agent/src/agent.ts Implemented full email notification handler using createEmailResponseActivity with error handling

pontemonti
pontemonti previously approved these changes Jan 7, 2026
@abdulanu0 abdulanu0 merged commit 84e81af into main Jan 8, 2026
19 checks passed
@abdulanu0 abdulanu0 deleted the users/anabdul/email_handling branch January 8, 2026 01:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants