Clarify supported MCP transport protocol for hosted API Platform servers#180
Clarify supported MCP transport protocol for hosted API Platform servers#180Rami2212 wants to merge 2 commits into
Conversation
Clarify MCP transport support: restrict API Platform to HTTP/SSE and document stdio as local only.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
✅ Files skipped from review due to trivial changes (1)
📝 WalkthroughSummaryThis documentation update clarifies the supported transport mechanism for MCP servers in the WSO2 API Platform hosted environment. It adds callouts to the "Get Started with MCP Servers" guide to state that platform-hosted MCP servers are network-accessible and therefore support only streamable HTTP (SSE); the Changes
ImpactImproves developer guidance and prevents transport misconfiguration by distinguishing MCP protocol transports from the transport mechanisms supported by the hosted API Platform. No functional changes to the platform. WalkthroughThis update adds documentation clarifications for MCP transport constraints when using WSO2 API Platform. Two callout notes were inserted specifying that MCP servers hosted on API Platform are accessed over the network and therefore only support streamable HTTP (SSE) transport, while 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
|
There was a problem hiding this comment.
Pull request overview
This PR clarifies which MCP transport is supported when MCP servers are deployed/hosted on the WSO2 API Platform, to prevent users from attempting to use stdio with remote endpoints.
Changes:
- Added an overview note distinguishing MCP protocol transports (
stdiovs streamable HTTP/SSE) vs platform support. - Added a “Transport Requirement” note under the remote-hosted MCP servers section stating
stdiois not supported and HTTP/SSE is required.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| > **Note:** | ||
| > The Model Context Protocol defines both `stdio` and streamable HTTP transports. | ||
| > - `stdio` is designed for **local, process-level communication** (for example, between a CLI/IDE and a locally running MCP server). | ||
| > - Streamable HTTP (SSE) is designed for **remote, network-based communication**. |
There was a problem hiding this comment.
This page uses a plain Markdown blockquote for the note, but most docs in this repo use MkDocs-style admonitions (e.g., !!! note in en/docs/api-platform-gateway/getting-started.md:5). Consider switching this to an admonition for consistent rendering/styling across the site.
| > **Transport Requirement:** | ||
| > Since API Platform exposes MCP servers as **remote, network-accessible endpoints**, it **does not support `stdio` transport**. | ||
| > All MCP interactions with API Platform must use **streamable HTTP (SSE)**. |
There was a problem hiding this comment.
This "Transport Requirement" callout is also formatted as a blockquote; for consistency with the rest of the docs (which use !!! note/!!! important admonitions), consider converting this to an admonition block so it renders consistently.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Description
This PR updates the Get Started with MCP Servers documentation to clearly distinguish between the transport mechanisms defined by the Model Context Protocol and the transport mechanism supported by the WSO2 API Platform hosted environment.
The existing documentation mentions both
stdioand streamable HTTP as primary MCP transport mechanisms, but does not explicitly clarify which transport is supported by the platform implementation. This may lead to confusion for developers attempting to configure remote MCP clients.Since the API Platform exposes MCP servers as remote, network-accessible endpoints, the
stdiotransport is not applicable in this context because it is designed only for local process-level communication.This update adds explicit notes to clarify that:
stdiois intended for local communication between a client and a server running on the same machineChanges Made
stdiois not supported for hosted MCP server connectionsProblem Solved
This resolves ambiguity between:
and improves onboarding clarity for developers new to MCP integrations.
Impact
This is a documentation-only change with no functional impact on the platform.
It improves: