modelcontextprotocol / modelcontextprotocol/php-sdk
[Server] (Re-)Implement missing Notification & Request Handlers
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- PHP
- Sterne
- 1.6k
- Forks
- 173
- Ø Merge
- 2 T. 49 Min.
- Gemergte PRs (30 T.)
- 23
Beschreibung
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.
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne damit, die vorhandenen Handler unter src/Server/NotificationHandler und src/Server/RequestHandler zu überprüfen, und vergleiche anschließend deren Abdeckung mit dem Symfony MCP SDK und php-mcp's Dispatcher.php. Das Issue identifiziert SetLogLevelRequest und CancelledNotification als noch offen; weitere Lücken im Protokoll können jedoch bestehen. Als abgeschlossen gilt die Vereinbarung zu Unter-Issues und den entsprechenden Handlern für die fehlenden Protokollanforderungen und -benachrichtigungen.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- php
- Bereich
- backend-api-design
- Issue-Typ
- Feature
- Schwierigkeit
- 5/5
- Geschätzter Aufwand
- Über eine Woche
- Aktivitätsstatus
- Veraltet
- Klarheit
- Muss geklärt werden
- Anfängerfreundlichkeit
- 25/100