Open
Conversation
- Replace mcp[cli] dependency with fastmcp>=2.0.0 - Update imports to use fastmcp package directly - Add CLI arguments for transport selection (stdio/http) - Support configurable host, port, and path for HTTP transport
Contributor
There was a problem hiding this comment.
Pull request overview
This PR migrates multiple IntelliKit MCP servers from the mcp[cli] FastMCP implementation to the standalone fastmcp package and adds CLI-selectable transports to enable running servers over streamable HTTP.
Changes:
- Switch MCP dependencies from
mcp[cli]tofastmcp>=2.0.0across several tool packages. - Update MCP server entrypoints to import from
fastmcpand add--transport/--host/--port/--pathCLI options, usingstreamable-httpwhen--transport httpis selected. - Refresh the
uprof_mcplockfile and expand repo-level documentation about tools, tests, and MCP transport usage.
Reviewed changes
Copilot reviewed 11 out of 14 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
uprof_mcp/uv.lock |
Updates locked dependency versions for uprof_mcp. |
uprof_mcp/src/uprof_mcp/uprof_profiler_mcp.py |
Migrates imports to fastmcp and adds CLI transport selection for stdio vs streamable HTTP. |
uprof_mcp/pyproject.toml |
Replaces mcp[cli] with fastmcp>=2.0.0. |
rocm_mcp/src/rocm_mcp/compile/hip_compiler_mcp.py |
Migrates imports to fastmcp and adds CLI transport selection for stdio vs streamable HTTP. |
rocm_mcp/pyproject.toml |
Replaces mcp[cli] with fastmcp>=2.0.0. |
nexus/pyproject.toml |
Replaces mcp[cli] with fastmcp>=2.0.0. |
nexus/nexus/mcp/server.py |
Migrates imports to fastmcp and adds CLI transport selection for stdio vs streamable HTTP. |
metrix/src/metrix/mcp/server.py |
Migrates imports to fastmcp and adds CLI transport selection for stdio vs streamable HTTP. |
metrix/pyproject.toml |
Replaces mcp[cli] with fastmcp>=2.0.0. |
linex/src/linex/mcp/server.py |
Migrates imports to fastmcp and adds CLI transport selection for stdio vs streamable HTTP. |
linex/pyproject.toml |
Replaces mcp[cli] with fastmcp>=2.0.0. |
AGENTS.md |
Updates/expands repository documentation, including MCP transport options and local testing guidance. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
mawad-amd
approved these changes
Mar 18, 2026
Member
mawad-amd
left a comment
mawad-amd
left a comment
There was a problem hiding this comment.
Looks great. Thanks, Yiannis!
Contributor
|
@ypapadop-amd I've opened a new pull request, #77, to work on those changes. Once the pull request is ready, I'll request review from you. |
…t support (#77) * Initial plan * feat: migrate hip_docs_mcp and rocminfo_mcp to fastmcp with HTTP transport support Co-authored-by: ypapadop-amd <102817138+ypapadop-amd@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: ypapadop-amd <102817138+ypapadop-amd@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR adds streamable-http to all MCPs. I had to upgrade to FastMCP to support those options (also logging is a bit nicer) and to add ASGI support in the future if needed (
mcpdoes not seem to expose those).