This repository was archived by the owner on Mar 21, 2025. It is now read-only.
mcpdotnet.TestSseServer can work #97
Closed
Conversation
This file contains 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
- 使 `IMcpServer` 继承自 `IAsyncDisposable`，并添加多个请求处理程序设置方法。 - 在 `McpServer` 类中实现新方法，并将 `SetPingHandler` 和 `SetInitializeHandler` 修改为公共方法。 - 将 `McpServerHandlers` 类的访问修饰符改为 `internal`。 - 更新 `Program.cs` 中的服务器实例创建方式，使用 `McpServerFactory.Create` 方法，并调整处理程序注册逻辑。
修改 `IMcpClient` 和 `IMcpServer` 接口中的方法名称，从 `SendRequestAsync` 更改为 `SendRequest`，以简化接口的使用。更新 `McpClient` 和 `McpServer` 类中对该方法的调用，并在 `McpClientExtensions` 中替换所有相关调用，确保一致性。同时，更新 `McpJsonRpcEndpoint` 类中的实现。添加 `Main` 方法以启动服务器，并清理 `mcpdotnet.Tests.csproj` 中不再需要的测试文件。
将 IMcpClient 和 IMcpServer 接口中的 SendRequest 方法重命名为 SendRequestAsync，以符合异步编程的命名约定。更新了 McpClient 类及相关扩展类中的所有调用，确保一致性和异步操作的正确性。同时，McpJsonRpcEndpoint 抽象类中的实现也进行了相应的重命名和调整。
在 `McpServer.cs` 文件中，启用了 `SetToolsHandler`、`SetPromptsHandler` 和 `SetResourcesHandler` 方法，以确保服务器初始化时正确设置这些处理程序。同时，将原本抛出的异常注释掉，改为直接返回，以提高代码的容错性。
|
Thanks, but this has already been addressed by #81 |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
Pull Request
Description of Changes
sse server "mcpdotnet.TestSseServer" now work for me.
Related Issue(s)