Fix App Send Targeted Messages by updating ConversationReference User

[ShanmathiMayuramKrithivasan]

Problem

App.Send() cannot send targeted messages because it builds ConversationReference, dropping the required User field that specifies the recipient id for targeted messages.

Fix

• Set ConversationReference.User = activity.Recipient when isTargeted=true in App.Send()
• This ensures the sender plugin can properly set the outgoing activity's recipient
• Maintains backward compatibility for non-targeted messages (User remains null)

[Copilot]

Pull Request Overview

This pull request fixes a bug in the App.Send() method that prevented targeted messages from being delivered correctly. The fix sets the ConversationReference.User field to activity.Recipient when sending targeted messages, enabling the sender plugin to properly configure the outgoing activity's recipient.

Key Changes

• Sets ConversationReference.User = activity.Recipient when isTargeted=true in the App.Send() method
• Maintains backward compatibility by keeping User as null for non-targeted messages

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[singhk97]

lgtm

[rido-min]

/azp run

[azure-pipelines]

No pipelines are associated with this pull request.

[rido-min]

/azp run

[azure-pipelines]

No pipelines are associated with this pull request.

[Copilot]

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review for a chance to win a $100 gift card. Take the survey.

[Copilot]

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 3 comments.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

The ArgumentException should specify the parameter name as the second argument for better debugging and adherence to .NET conventions. Consider:

throw new ArgumentException("activity.Recipient is required for targeted messages", nameof(activity));

This helps developers identify which parameter caused the exception and follows the standard pattern for ArgumentException where the second parameter specifies the problematic parameter name.

Suggested change

	throw new ArgumentException("activity.Recipient is required for targeted messages");
	throw new ArgumentException("activity.Recipient is required for targeted messages", nameof(activity));

[Copilot]

The punctuation in this parameter description is inconsistent with other similar isTargeted parameter descriptions in the same file (lines 251 and 274), which use a semicolon instead of a period. For consistency, consider using a semicolon:

/// <param name="isTargeted">when true, sends the message privately to the specified recipient; when false, sends to all conversation participants</param>

This maintains consistency with the existing documentation style in the codebase.

Suggested change

	/// <param name="isTargeted">when true, sends the message privately to the specified recipient. when false, sends to all conversation participants</param>
	/// <param name="isTargeted">when true, sends the message privately to the specified recipient; when false, sends to all conversation participants</param>