Repository navigation
chore(deps-dev): update mcp requirement from <2,>=1.30 to >=1.30,<3 - #384
dependabot[bot] wants to merge 1 commit into
Conversation
Updates the requirements on [mcp](https://github.com/modelcontextprotocol/python-sdk) to permit the latest version. - [Release notes](https://github.com/modelcontextprotocol/python-sdk/releases) - [Changelog](https://github.com/modelcontextprotocol/python-sdk/blob/main/RELEASE.md) - [Commits](modelcontextprotocol/python-sdk@v1.30.0...v2.3.0) --- updated-dependencies: - dependency-name: mcp dependency-version: 2.3.0 dependency-type: direct:development ... Signed-off-by: dependabot[bot] <support@github.com>
Codex Review Summary: ✅ CompletedThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 9af98b146c
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| [project.optional-dependencies] | ||
| relay = ["websockets>=17.1,<18; python_version >= '3.11'", "cryptography>=50.0.1,<51; python_version >= '3.11'"] | ||
| mcp = ["mcp>=1.30,<2; python_version >= '3.10'"] | ||
| mcp = ["mcp>=1.30,<3; python_version >= '3.10'"] |
There was a problem hiding this comment.
Migrate the server before allowing MCP 2.x
When a fresh or upgraded session-peer[mcp] installation resolves MCP 2.x, create_server() still imports FastMCP from mcp.server.fastmcp at session_peer_mcp.py:754; that module was removed in v2, so main() catches the resulting ImportError and exits with status 1 instead of starting the server. The official migration guide explicitly identifies this import as a v2 breaking change and recommends retaining <2 until the server is migrated to mcp.server.mcpserver.MCPServer.
Useful? React with 👍 / 👎.
| [project.optional-dependencies] | ||
| relay = ["websockets>=17.1,<18; python_version >= '3.11'", "cryptography>=50.0.1,<51; python_version >= '3.11'"] | ||
| mcp = ["mcp>=1.30,<2; python_version >= '3.10'"] | ||
| mcp = ["mcp>=1.30,<3; python_version >= '3.10'"] |
There was a problem hiding this comment.
Update the MCP CI profiles when admitting 2.x
Allowing 2.x makes the latest MCP job install the current 2.x release, but .github/workflows/ci.yml:239-240 still rejects every latest version that does not start with 1., so all six latest-profile matrix legs stop before the integration tests and the dependent release gate fails. The repository test tests/docs/test_mcp_dependency_profiles.py:23 also still asserts this manifest contains <2 and now fails immediately; the CI verifier and contract test need to be migrated alongside the range.
Useful? React with 👍 / 👎.
|
Looks like mcp is no longer updatable, so this is no longer needed. |
Updates the requirements on mcp to permit the latest version.
Release notes
Sourced from mcp's releases.
... (truncated)
Commits
2118f14docs: refresh translations for recent English changes (#3636)ed9b2d6Stop counting an interactive OAuth login against request timeouts (#3635)d5cebd1Keep inline-snapshot disabled when pytest runs in a terminal (#3634)c15566cLink What's new to the Header parameters page (#3632)4d29994Let a newer Deploy Docs run cancel the one in progress (#3633)0acea60Bump urllib3 from 2.7.0 to 2.8.0 (#3607)c54075cLook the tool schema up by name for Mcp-Param-* validation instead of running...9afccaeRetry a tool call once after a HeaderMismatch rejection (#3627)cafa33bRecord JSON-RPC error responses on the client OpenTelemetry span (#3629)0b2fd3eOmit an empty _meta and empty params from outbound requests (#3628)Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)