Skip to content

Add ModelContextProtocol.AspNetCore#160

Merged
halter73 merged 6 commits intomodelcontextprotocol:mainfrom
halter73:aspnet
Mar 31, 2025
Merged

Add ModelContextProtocol.AspNetCore#160
halter73 merged 6 commits intomodelcontextprotocol:mainfrom
halter73:aspnet

Conversation

@halter73
Copy link
Copy Markdown
Contributor

  • Remove IServerTransport and HttpListenerSseServerTransport. Replace its usage in tests with ASP.NET Core.

- Remove IServerTransport

- Canceling read from stdin in StdioServerTransport.DisposeAsync is also flaky on Windows
- https://github.com/modelcontextprotocol/csharp-sdk/actions/runs/14161333333/job/39667212874
Comment thread src/ModelContextProtocol.AspNetCore/McpEndpointRouteBuilderExtensions.cs Outdated
Comment thread src/ModelContextProtocol.AspNetCore/ModelContextProtocol.AspNetCore.csproj Outdated
Comment thread src/ModelContextProtocol.AspNetCore/README.md
…tCore.csproj

Co-authored-by: Eirik Tsarpalis <eirik.tsarpalis@gmail.com>
Comment thread src/ModelContextProtocol.AspNetCore/README.md Outdated
Comment thread src/ModelContextProtocol.AspNetCore/McpEndpointRouteBuilderExtensions.cs Outdated
Comment thread src/ModelContextProtocol.AspNetCore/McpEndpointRouteBuilderExtensions.cs Outdated
Comment thread src/ModelContextProtocol.AspNetCore/README.md
eiriktsarpalis and others added 2 commits March 31, 2025 17:46
Co-authored-by: Jeff Handley <jeffhandley@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

@jeffhandley jeffhandley left a comment

Choose a reason for hiding this comment

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

Approved from the build/infrastructure perspective

@halter73 halter73 merged commit 973c7c0 into modelcontextprotocol:main Mar 31, 2025
8 checks passed
@stvansolano stvansolano mentioned this pull request Mar 31, 2025
10 tasks
@IshamMohamed
Copy link
Copy Markdown

Invalidates #168

@jeffhandley
Copy link
Copy Markdown
Contributor

Adding the breaking-change label retroactively during release notes revision.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaking-change This issue or PR introduces a breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants