fix(client): release owned dispatcher exception hook - #3306
Conversation
|
Thanks for the contribution. This repository only keeps pull requests open when they're linked to an issue that a maintainer has assigned to the author — CONTRIBUTING.md explains why and how we work. This PR has been closed for now because its description doesn't yet link an open issue in this repository (with If there isn't an issue for this yet, please open one — a clear description of the problem is genuinely the most useful thing for us. Then add There's no need to open a new PR — this one will be reopened. While it's closed, please push any updates as new commits rather than force-pushing, since GitHub can't reopen a PR whose branch has been rewritten. Maintainers: reopening this PR, removing the |
Summary
ClientSessioninstalls on an injectedJSONRPCDispatcherduring cleanupTests
uv run pytest tests/client -q(714 passed, 1 xfailed)uv run ruff check src/mcp/client/session.py tests/client/test_session.pyuv run pyright src/mcp/client/session.py tests/client/test_session.pyCloses no issue: identified by the existing TODO at the dispatcher/session ownership seam.