Skip to content

Migrate all rocm_mcp servers to fastmcp with streamable-HTTP transport support#77

Merged
ypapadop-amd merged 2 commits intoypapadop-amd/streamable-httpfrom
copilot/sub-pr-74
Mar 18, 2026
Merged

Migrate all rocm_mcp servers to fastmcp with streamable-HTTP transport support#77
ypapadop-amd merged 2 commits intoypapadop-amd/streamable-httpfrom
copilot/sub-pr-74

Conversation

Copy link
Contributor

Copilot AI commented Mar 18, 2026

hip_docs_mcp.py and rocminfo_mcp.py still imported from mcp.server.fastmcp and only supported stdio, making the fastmcp>=2.0.0 dependency switch and HTTP transport support incomplete across the rocm_mcp package.

Changes

  • hip_docs_mcp.py / rocminfo_mcp.py: Migrated imports from mcp.server.fastmcpfastmcp; replaced hardcoded mcp.run(transport="stdio") with argparse-based CLI matching the pattern in hip_compiler_mcp.py
  • Default HTTP paths: /rocm_mcp/hip_docs and /rocm_mcp/rocminfo
  • AGENTS.md: Added the two new servers to the default HTTP paths table

All three rocm_mcp servers now support:

hip-docs-mcp --transport http --host 127.0.0.1 --port 8000 --path /rocm_mcp/hip_docs
rocminfo-mcp --transport http --host 127.0.0.1 --port 8000 --path /rocm_mcp/rocminfo

📱 Kick off Copilot coding agent tasks wherever you are with GitHub Mobile, available on iOS and Android.

…sport support

Co-authored-by: ypapadop-amd <102817138+ypapadop-amd@users.noreply.github.com>
Copilot AI changed the title [WIP] [WIP] Address feedback on Streamable HTTP for MCPs PR Migrate all rocm_mcp servers to fastmcp with streamable-HTTP transport support Mar 18, 2026
Copilot AI requested a review from ypapadop-amd March 18, 2026 17:42
@ypapadop-amd ypapadop-amd marked this pull request as ready for review March 18, 2026 20:11
@ypapadop-amd ypapadop-amd merged commit 7c33874 into ypapadop-amd/streamable-http Mar 18, 2026
@ypapadop-amd ypapadop-amd deleted the copilot/sub-pr-74 branch March 18, 2026 20:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants