modelcontextprotocol / modelcontextprotocol/php-sdk
[Server] (Re-)Implement missing Notification & Request Handlers
Nessuno ha ancora preso questa issue.
- Lingua principale
- PHP
- Stelle
- 1.6k
- Fork
- 173
- Merge medio
- 2g 49m
- PR unite (30g)
- 23
Descrizione
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.
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Inizia esaminando gli handler esistenti in src/Server/NotificationHandler e src/Server/RequestHandler, quindi confronta la loro copertura con Symfony MCP SDK e php-mcp's Dispatcher.php. L'issue identifica SetLogLevelRequest e CancelledNotification come ancora aperti, ma potrebbero rimanere altre lacune del protocollo. Il lavoro è completato quando sono stati concordati i sub-issue e i relativi handler per le richieste e le notifiche di protocollo mancanti.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- php
- Ambito
- backend-api-design
- Tipo di issue
- Funzionalità
- Difficoltà
- 5/5
- Tempo stimato
- Più di una settimana
- Stato di attività
- Ferma
- Chiarezza
- Da chiarire
- Idoneità per principianti
- 25/100