Simplify and harden session lifecycle with an explicit state machine
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 5/5
- Tempo estimado
- Mais de uma semana
- Facilidade para iniciantes
- 25/100
Direção de pesquisa
Nenhum arquivo ou ponto de entrada é nomeado; comece localizando a inicialização de sessão existente, o caminho HTTP stateless e a limpeza de AsyncExitStack. Revise a issue #756 e os testes do transporte em memória descritos na issue. Considera-se concluído quando houver estados explícitos do ciclo de vida, um caminho stateless dedicado, limpeza confiável em todos os caminhos de saída e testes de transição para os casos de erro e stateful/stateless.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
Description
Summary
Server and client sessions use an internal initialization state machine combined with various flags and manual AsyncExitStack management. Stateless mode complicates this further by setting an "Initialized" state early as a workaround.
This makes the lifecycle hard to reason about and contributes to subtle bugs (e.g., past issues like #756 in stateless mode).
Problems
- Scattered state management: Initialization and teardown logic are spread across methods and files.
- Special cases for stateless mode: Stateless HTTP sets the session as initialized even though no real protocol negotiation has occurred.
- Resource management risk: If initialization fails midway, transport tasks and resources may not be cleaned up reliably.
- Testing difficulty: Tests often bypass parts of the lifecycle using in-memory transports.
Proposal
-
Introduce an explicit session state machine
Represent distinct states as separate types, for example:
UninitializedSessionNegotiatingSessionActiveSessionClosedSessionStatelessSession(for per-request / ephemeral cases)
Each state exposes only the operations that are valid in that state, and transitions return the next state type.
-
Model stateless mode explicitly
- Instead of marking a stateful session as "Initialized" prematurely, model stateless HTTP as a dedicated
StatelessSessiontype with a simpler lifecycle. - Make behavior differences clear in code and docs.
- Instead of marking a stateful session as "Initialized" prematurely, model stateless HTTP as a dedicated
-
Centralize resource cleanup
- Ensure that all lifecycle paths (happy path, error, cancellation) lead through code that:
- cancels outstanding tasks,
- closes transports,
- releases resources stored in
AsyncExitStack.
- Ensure that all lifecycle paths (happy path, error, cancellation) lead through code that:
Why this matters
- Reliability: Fewer edge cases and surprise states where messages can be processed incorrectly.
- Debuggability: Easier to reason about what can happen in each state.
- Extensibility: Adding new lifecycle behavior (e.g., resuming sessions) becomes more manageable.
Acceptance criteria
- Session lifecycle is represented via explicit types or a clearly defined state machine.
- Stateless HTTP mode uses a dedicated path rather than setting "Initialized" as a hack.
- All entry/exit paths of sessions ensure proper cleanup of transports and tasks.
- Tests cover state transitions, including error cases and stateless/stat eful differences.
References
No response
- Linguagem predominante
- Python
- Estrelas
- 24.3k
- Forks
- 4k
- Merge médio
- 1d 11h
- PRs com merge (30d)
- 30
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
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 75/100
modelcontextprotocol/python-sdk#3566 ·
-
v1 v2
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 85/100
modelcontextprotocol/python-sdk#3546 · 5 comentários ·
-
v1 v2
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 76/100
modelcontextprotocol/python-sdk#3545 · 1 comentário ·
-
v1 v2
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 91/100
modelcontextprotocol/python-sdk#3508 · 2 comentários ·
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 64/100
modelcontextprotocol/python-sdk#3504 ·
Todas as issues de modelcontextprotocol/python-sdk
Issues semelhantes
-
bug
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 75/100
xinnan-tech/xiaozhi-fde-talk#263 ·
-
rules
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 90/100
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 70/100
huggingface/Repo2RLEnv#163 · 1 comentário ·
-
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 95/100
huggingface/sentence-transformers#4074 ·
-
comp/dashboard invalid P3
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 70/100
NousResearch/hermes-agent#121143 ·