Simple chatbot: unused Server.stdio_context attribute obscures transport ownership
Los mantenedores suelen responder en 1 día
Evaluación
- Dificultad
- 1/5
- Tiempo estimado
- Menos de una hora
- Aptitud para principiantes
- 86/100
- Tipo de issue
- Refactorización
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- python
- Área
- developer-experience
Línea de trabajo
Lee examples/clients/simple-chatbot/mcp_simple_chatbot/main.py, centrándote en Server.__init__, Server.initialize y Server.cleanup. Elimina las dos asignaciones sin uso a self.stdio_context sin modificar el ciclo de vida existente de la pila de salida; el trabajo estará hecho cuando ya no se haga referencia al atributo y pasen las pruebas pertinentes del ejemplo.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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.
- Lenguaje dominante
- Python
- Estrellas
- 24.5k
- Forks
- 4k
- Merge medio
- 15 h 19 min
- PR fusionados (30 d)
- 32
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de modelcontextprotocol/python-sdk
-
enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
modelcontextprotocol/python-sdk#3673 ·
Los mantenedores suelen responder en 1 día
-
Streamable HTTP server rejects a mixed-case Content-Type with 415Posiblemente ocupada @CRYPTONIKAV la tomó hace 1 día. Abiertobug v1 v2
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
modelcontextprotocol/python-sdk#3670 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
Bug in the MCP Apps documentation.Posiblemente ocupada @Kludex la tomó hoy. Abiertodocumentation v2
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
modelcontextprotocol/python-sdk#3662 ·
Los mantenedores suelen responder en 1 día
-
Audio(data=b"") raises "Either path or data can be provided", while Image(data=b"") worksPosiblemente ocupada @KaiyiQuan la tomó hace 3 días. Abiertobug v1 v2
Dificultad 1/5 Menos de una hora Aptitud para principiantes 85/100
modelcontextprotocol/python-sdk#3656 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 66/100
modelcontextprotocol/python-sdk#3655 ·
Los mantenedores suelen responder en 1 día
Todos los issues de modelcontextprotocol/python-sdk
Issues similares
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
lugassawan/swe-workbench#799 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
vllm-project/vllm#61131 ·
Los mantenedores suelen responder en 1 día
-
Non-string form field names raise AttributeError when files are attachedPosiblemente ocupada @nightcityblade la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
kubernetes-client/python#2735 ·
Los mantenedores suelen responder en 1 día
-
desktop help wanted test
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día