modelcontextprotocol / modelcontextprotocol/php-sdk
[Server] (Re-)Implement missing Notification & Request Handlers
还没有人认领这个 Issue。
- 主要语言
- PHP
- 星标
- 1.6k
- 派生
- 173
- 平均合并
- 2 天 49 分钟
- 30 天内合并 PR
- 23
描述
For kicking off this SDK we used the Noticiation and Request handlers shipped with the Symfony MCP SDK and adopted schema component and registry.
But the MCP schema supports way more requests and notifications, that would need handlers as well. On php-mcp side this was the responsibility of the Dispatcher implementation.
Still open are:
- ListResourceTemplatesRequest
#89 - ResourceSubscribeRequest
#220 - ResourceUnsubscribeRequest
#220 - SetLogLevelRequest
- CompletionCompleteRequest
#97 - CancelledNotification
And potentially other requests and notification of the protocol.
Please announce yourself and create corresponding sub-issues before working on a specific handler - so we can avoid duplicate effort.
贡献指南
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
调研方向
首先检查 src/Server/NotificationHandler 和 src/Server/RequestHandler 下现有的处理器,然后将其覆盖范围与 Symfony MCP SDK 和 php-mcp's Dispatcher.php 进行比较。该 issue 指出 SetLogLevelRequest 和 CancelledNotification 仍未完成,但协议中可能还存在其他缺口。对于缺失的协议请求和通知,达成相应的子 issue 以及处理器方案后,即视为完成。
由索引模型根据 Issue 内容生成。
评估
- 技术栈
- php
- 领域
- backend-api-design
- Issue 类型
- 功能
- 难度
- 5/5
- 预计耗时
- 一周以上
- 活跃度
- 停滞
- 描述清晰度
- 需要澄清
- 新手友好度
- 25/100