Change SSE transport to streaming transport for MCP samples #216
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
👋 Thanks for contributing @copilot! We will review the pull request and get back to you soon. |
Co-authored-by: jongalloway <[email protected]>
Copilot
AI
changed the title
Copilot finished work on behalf of
jongalloway
July 28, 2025 18:43
There was a problem hiding this comment.
Pull Request Overview
This PR renames MCP (Model Context Protocol) sample projects from SSE (Server-Sent Events) to streaming transport terminology. The changes update project names, namespaces, and references to align with the new streaming transport approach while maintaining the same underlying functionality.
Key changes:
- Renamed the main server project from
McpSample.AspNetCoreSseServertoMcpSample.AspNetCoreServer - Updated all namespace references in tool classes to match the new project structure
- Modified project references and solution file entries to reflect the renamed projects
Reviewed Changes
Copilot reviewed 6 out of 11 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
Program.cs (Chat) |
Updated transport display name from "AspNetCoreSse Server" to "AspNetCore Server" |
WeatherTool.cs |
Changed namespace from McpSample.AspNetCoreSseServer to McpSample.AspNetCoreServer |
Jokes.cs |
Changed namespace from McpSample.AspNetCoreSseServer to McpSample.AspNetCoreServer |
Program.cs (AppHost) |
Updated project references and variable names from aspnetsseserver to aspnetserver |
McpSample.AppHost.csproj |
Updated project reference path to match renamed server project |
Aspire.MCP.Sample.sln |
Updated solution file entry for the renamed server project |
Co-authored-by: Copilot <[email protected]>
jongalloway
approved these changes
Jul 28, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR migrates the MCP (Model Context Protocol) samples from SSE transport naming to streaming transport by renaming projects and namespaces as requested in the issue.
Changes Made
Project Renaming
McpSample.AspNetCoreSseServerproject toMcpSample.AspNetCoreServeraspnetsseservertoaspnetserverNamespace Updates
Tools/Jokes.csandTools/WeatherTool.csfromMcpSample.AspNetCoreSseServertoMcpSample.AspNetCoreServerTransport Configuration
HttpTransportMode.StreamableHttpfor the new streaming transport modeTechnical Details
The migration maintains the
SseClientTransportclass usage (which appears to be kept for API compatibility) but ensures it's configured withHttpTransportMode.StreamableHttpto use the new streaming transport mechanism instead of traditional Server-Sent Events.All projects build successfully and the MCP server starts correctly, confirming the migration maintains full functionality while removing SSE naming references as requested.
Fixes #215.
💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.