Skip to content

.Net: Mcp sampling sample #11533

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 8 commits into from
Apr 14, 2025
Merged

.Net: Mcp sampling sample #11533

merged 8 commits into from
Apr 14, 2025

Conversation

SergeyMenshykh
Copy link
Member

@SergeyMenshykh SergeyMenshykh commented Apr 14, 2025 •
edited
Loading

Motivation, Context and Description

This PR:

  1. Adds a new sample demonstrating how to use the MCP sampling and enabling human-in-the-loop client-side to intercept the sampling request to decide whether to proceed with it or not.
  2. Moves embedded resources from different folders to the ProjectResources folder to keep all resources in one place.
  3. Fixes the MCP sample that demonstrates how to use MCP tools with ChatCompletionAgent. The sample now uses execution settings with the enabled "retain types" option, allowing it to successfully call the tools.
  4. Updates the MCP NuGet package to the latest version.

Contributes: #11199

@SergeyMenshykh
1. add mcp sampling sample
46fcf11 
2. add fix mcp sample with agent
3. move mcp embedded resource to a separate folder
@SergeyMenshykh
fix xml comment
b6e0e9b
@SergeyMenshykh
improve xml comments
d4b794c
@SergeyMenshykh SergeyMenshykh added .NET Issue or Pull requests regarding .NET code modelcontextprotocol labels Apr 14, 2025
@SergeyMenshykh SergeyMenshykh self-assigned this Apr 14, 2025
@SergeyMenshykh SergeyMenshykh requested a review from a team as a code owner April 14, 2025 09:14
@SergeyMenshykh SergeyMenshykh moved this to Sprint: In Review in Semantic Kernel Apr 14, 2025
@SergeyMenshykh SergeyMenshykh added this pull request to the merge queue Apr 14, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Apr 14, 2025
@SergeyMenshykh SergeyMenshykh added this pull request to the merge queue Apr 14, 2025
Merged via the queue into microsoft:main with commit 12bcfc3 Apr 14, 2025
20 checks passed
@SergeyMenshykh SergeyMenshykh deleted the mcp-sampling-sample branch April 14, 2025 14:15
@github-project-automation github-project-automation bot moved this from Sprint: In Review to Sprint: Done in Semantic Kernel Apr 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@RogerBarreto RogerBarreto RogerBarreto approved these changes

@markwallace-microsoft markwallace-microsoft markwallace-microsoft approved these changes

Assignees

@SergeyMenshykh SergeyMenshykh

Labels
modelcontextprotocol .NET Issue or Pull requests regarding .NET code
Projects
Semantic Kernel
Status: Sprint: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@SergeyMenshykh @RogerBarreto @markwallace-microsoft