modelcontextprotocol / modelcontextprotocol/python-sdk
Low-level Server support for mcp dev and mcp run command
Ninguém assumiu esta issue ainda.
- Linguagem predominante
- Python
- Estrelas
- 24.3k
- Forks
- 4k
- Merge médio
- 1d 1h
- PRs com merge (30d)
- 31
Descrição
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.
Guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Direção de pesquisa
Comece rastreando os pontos de entrada dos comandos mcp dev e mcp run e compare como eles lidam com as classes Server de alto e baixo nível. Leia server/lowlevel/server.py e a classe Settings existente para entender o campo dependencies ausente. A conclusão exige a concordância dos maintainers sobre o design e a verificação do suporte a Server de baixo nível em ambos os comandos.
Escrita pelo modelo de indexação a partir do texto da issue.
Avaliação
- Stack de tecnologia
- python
- Domínio
- backend, cli
- Tipo de issue
- Funcionalidade
- Dificuldade
- 5/5
- Tempo estimado
- Mais de uma semana
- Status de atividade
- Estagnada
- Clareza
- Precisa de esclarecimento
- Facilidade para iniciantes
- 30/100