modelcontextprotocol / modelcontextprotocol/python-sdk
Low-level Server support for mcp dev and mcp run command
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Python
- Sterne
- 24.3k
- Forks
- 4k
- Ø Merge
- 1 T. 1 Std.
- Gemergte PRs (30 T.)
- 31
Beschreibung
Currently the mcp dev and mcp run command does not support the low-level Server class. I think it's a better developer experience if we support both classes for these 2 commands, assuming there's no inherent reason for not doing so.
I'm interested in adding support for the low-level Server to both mcp dev and mcp run and I would like seek the maintainer's guidance on this. It looks like the only thing missing is the dependencies field, which is pulled from the Settings class and does not exist in the low-level Server. I'm thinking about adding a similar Settings class to server/lowlevel/server.py and thus introducing the dependencies field. The downside is more complexity in server/lowlevel/server.py which I'm not sure if it's acceptable to the maintainers and falls in line with the long term roadmap for the code base.
Please advise. Thanks.
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 Einstiegspunkte der Befehle mcp dev und mcp run nachzuverfolgen, und vergleiche, wie sie mit den High-Level- und Low-Level-Server-Klassen umgehen. Lies server/lowlevel/server.py und die vorhandene Settings-Klasse, um das fehlende Feld dependencies zu verstehen. Als abgeschlossen gilt die Aufgabe, wenn die Maintainer dem Design zugestimmt haben und die Unterstützung für Low-Level-Server in beiden Befehlen verifiziert wurde.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- python
- Bereich
- backend, cli
- Issue-Typ
- Feature
- Schwierigkeit
- 5/5
- Geschätzter Aufwand
- Über eine Woche
- Aktivitätsstatus
- Veraltet
- Klarheit
- Muss geklärt werden
- Anfängerfreundlichkeit
- 30/100