| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
|
Updated the branch after the CI coverage failure. The missing streamable_http.py exception path is now covered by a notification-error regression test. Local checks: targeted pytest passed; full tests/shared/test_streamable_http.py passed (63 passed); coverage report for src/mcp/client/streamable_http.py no longer lists line 478 as missing; ruff check, ruff format --check, pyright, py_compile, and git diff --check all passed. CI is running again on f218dc8. |
Sorry, something went wrong.
|
Updated once more after the Python 3.14 strict coverage run exposed a branch-coverage partial in the new regression test itself. The new test now follows the existing anyio.fail_after pragma style. Local checks: targeted pytest passed; full tests/shared/test_streamable_http.py passed (63 passed); coverage report for tests/shared/test_streamable_http.py is 100.00%; ruff check, ruff format --check, pyright, py_compile, and git diff --check all passed. CI is running again on f092047. |
Sorry, something went wrong.
|
CI is green now on f092047: all matrix test jobs, pre-commit, readme-snippets, conformance, and all-green passed. |
Sorry, something went wrong.
|
Rebased this onto current upstream/main and resolved the StreamableHTTP in-process transport test refactor conflict. Kept the request-error isolation behavior and migrated the surrounding tests to the current in-process test setup without bringing back the old socket test style. Validated locally on Windows:
|
Sorry, something went wrong.
|
Follow-up after the matrix failure: the request-error isolation was catching every exception from the request task, which also converted OAuth flow errors into JSON-RPC errors. I narrowed that handler to httpx.HTTPError, so transport/request failures are still isolated for JSON-RPC requests, while auth flow errors keep propagating as OAuthFlowError/OAuthRegistrationError. Validated locally on Windows:
|
Sorry, something went wrong.
|
You've opened a duplicated pull request, please search opened PRs before creating new ones. Duplicated from #1998. |
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
Summary
Closes #2604
To verify