Skip to content

Conversation

Dangoron
Copy link

Body:

Hi team,

This PR introduces support for HTTP Streamable transport to the Firecrawl MCP server. This enhancement provides an alternative way for clients to connect and interact with the server, aligning with the latest specifications and offering more flexibility for integrations.

Key Changes:

  • Added a new runHTTPStreamableServer() function in src/index.ts to enable the HTTP Streamable server.
  • Updated README.md to include:
    • Instructions on how to run the server in HTTP Streamable mode.
    • An example MCP client configuration for connecting via HTTP Streamable transport.
    • Relevant environment variable HTTP_STREAMABLE_SERVER and configuration examples.

This addition is in the spirit of open source and aims to enhance the capabilities and interoperability of the Firecrawl MCP server. We believe this will be a valuable addition for users looking for different transport options.

For more information on the Model Context Protocol and its specifications, please refer to the official documentation: https://modelcontextprotocol.io/specification/2025-03-26

We hope this contribution is a welcome addition to the project and look forward to your feedback!

Best regards,

Dangoron

@nickscamara nickscamara requested a review from tomkosm May 12, 2025 16:46
src/index.ts Outdated

const transports: { [sessionId: string]: StreamableHTTPServerTransport } = {};

// 单一端点处理所有MCP请求

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am not a reviewer. Nor am I affiliated.

But I could use clarification on what this comment means.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Based on google translate: // Single endpoint handles all MCP requests

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for pointing this out and for your interest in the code! The comment means that all MCP (Multi-Channel Protocol) requests are handled through a single API endpoint. I appreciate your feedback, and I will update the comment to English to make it clearer for everyone.

@tradeqvest
Copy link

tradeqvest commented Jul 30, 2025

Hello! Will this PR eventually be merged? We are currently waiting on a reliable streamable http implementation. Thanks! @nickscamara @tomkosm

@nickscamara
Copy link
Member

Hey @tradeqvest, yes of course, sorry about the delay! We are working on pushing improvements to our mcp earlier next month but in the meantime we should totally merge this! @tomkosm can you review it today if possible? Thanks!

@tradeqvest
Copy link

Thank you for the quick reply! Looking forward to it.

@micahstairs
Copy link
Member

Added in 6817c87

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.

7 participants