modelcontextprotocol / modelcontextprotocol/python-sdk
Audit MCPServer constructor parameters
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
From the V2 checklist in #1253:
We need to recheck the constructor parameters of the MCPServer class. I did remove and refactor a bit, but we need to audit it a bit better.
PR #1732 renamed FastMCP to MCPServer and moved transport-specific options (host, port, stateless_http, etc.) to run(). However, the constructor still has parameters that may need further review:
- Are all remaining constructor parameters appropriate for the constructor vs.
run()? - Are there parameters that should be removed, renamed, or have their defaults reconsidered?
- Is the constructor signature clean and minimal for the common case?
This is a follow-up audit to ensure the MCPServer API surface is well-designed before V2 stabilizes.
Ref: Kludex's V2 checklist in #1253, follow-up to #1732
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 revisando o construtor de MCPServer e seu ponto de entrada run(), juntamente com as alterações de PR #1732 e a checklist de V2 em #1253. Audite quais parâmetros pertencem a cada API e documente ou teste as decisões resultantes para que o construtor seja mínimo e adequado para V2.
Escrita pelo modelo de indexação a partir do texto da issue.
Avaliação
- Stack de tecnologia
- python
- Domínio
- backend-api-design
- Tipo de issue
- Refatoração
- Dificuldade
- 5/5
- Tempo estimado
- Mais de uma semana
- Status de atividade
- Pouca atividade
- Clareza
- Precisa de esclarecimento
- Facilidade para iniciantes
- 35/100