Simple chatbot: unused Server.stdio_context attribute obscures transport ownership
Mantenedores costumam responder em até 1 dia
Avaliação
- Dificuldade
- 1/5
- Tempo estimado
- Menos de uma hora
- Facilidade para iniciantes
- 86/100
- Tipo de issue
- Refatoração
- Clareza
- Claramente especificada
- Status de atividade
- Ativa
- Stack de tecnologia
- python
- Domínio
- developer-experience
Direção de pesquisa
Leia examples/clients/simple-chatbot/mcp_simple_chatbot/main.py, com foco em Server.__init__, Server.initialize e Server.cleanup. Remova as duas atribuições não utilizadas a self.stdio_context sem alterar o ciclo de vida existente da pilha de saída; a tarefa estará concluída quando o atributo não for mais referenciado e os testes relevantes do exemplo passarem.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
While studying context management in the simple chatbot example, I noticed that Server.stdio_context suggests the transport context is stored on the instance, but it is never used.
In examples/clients/simple-chatbot/mcp_simple_chatbot/main.py, searching for stdio_context finds only:
# Server.__init__
self.stdio_context: Any | None = None
# Server.cleanup
self.stdio_context = None
Server.initialize() instead enters stdio_client(server_params) through self.exit_stack.enter_async_context(...), and cleanup() calls self.exit_stack.aclose(). The stack already owns the transport cleanup; the attribute is never read or assigned a context manager.
This is a minor example-code clarity issue, not an observed runtime failure. The unused attribute made it unclear whether initialization was missing an assignment. Suggested fix: remove the two self.stdio_context assignments, leaving the existing exit-stack lifecycle unchanged.
Reporting this first per the contribution guide. AI assistance: this report was prepared with OpenAI Codex after discussing the example and inspecting the source.
- Linguagem predominante
- Python
- Estrelas
- 24.5k
- Forks
- 4k
- Merge médio
- 18h 19min
- PRs com merge (30d)
- 36
Preparar o ambiente
- Sem Dockerfile nem arquivo Docker Compose
- Tem um modelo de pull request
- Ler o 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.
Mais de modelcontextprotocol/python-sdk
-
bug v2
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
modelcontextprotocol/python-sdk#3685 ·
Mantenedores costumam responder em até 1 dia
-
Type annotation gaps and unhandled FileNotFoundError in src/mcp/cli/cli.pyTalvez livre de novo Um pull request para esta issue foi fechado sem ser mesclado. Abertaenhancement
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 84/100
modelcontextprotocol/python-sdk#3673 · 3 comentários ·
Mantenedores costumam responder em até 1 dia
-
Streamable HTTP server rejects a mixed-case Content-Type with 415Talvez já em andamento @CRYPTONIKAV assumiu há 2 dias. Abertabug v1 v2
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 75/100
modelcontextprotocol/python-sdk#3670 · 4 comentários ·
Mantenedores costumam responder em até 1 dia
-
Bug in the MCP Apps documentation.Talvez já em andamento @Kludex assumiu há 1 dia. Abertadocumentation v2
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 82/100
modelcontextprotocol/python-sdk#3662 ·
Mantenedores costumam responder em até 1 dia
-
Audio(data=b"") raises "Either path or data can be provided", while Image(data=b"") worksTalvez já em andamento @ajay206 assumiu hoje. Abertabug v1 v2
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 85/100
modelcontextprotocol/python-sdk#3656 · 2 comentários ·
Mantenedores costumam responder em até 1 dia
Todas as issues de modelcontextprotocol/python-sdk
Issues semelhantes
-
SSM GetParameter throttling during concurrent workflow launchesTalvez já em andamento @alexiswl assumiu hoje. Aberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 80/100
-
[Bug] Dev image build fails: `npm install` in the root Dockerfile runs out of memory (exit 134)Talvez já em andamento @andest01 assumiu hoje. Aberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 75/100
Mantenedores costumam responder em até 1 dia
-
enhancement good first issue
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 79/100
pesu-dev/discord_bot#97 ·
-
Dificuldade 1/5 1-3 horas Facilidade para iniciantes 88/100
gradio-app/gradio#13964 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 93/100
BSData/horus-heresy-3rd-edition#3232 · 1 comentário ·
Mantenedores costumam responder em até 1 dia