Skip to content

Add hot reload functionality to SSE#249

Closed
TheRobotCarlson wants to merge 1 commit intomodelcontextprotocol:mainfrom
TheRobotCarlson:main
Closed

Add hot reload functionality to SSE#249
TheRobotCarlson wants to merge 1 commit intomodelcontextprotocol:mainfrom
TheRobotCarlson:main

Conversation

@TheRobotCarlson
Copy link
Copy Markdown

@TheRobotCarlson TheRobotCarlson commented Mar 8, 2025

Motivation and Context

I use Cursor to build and test my MCP servers and often it updates the servers, but can't restart them once the code has been edited.

This adds the hot reload option to the FastMCP settings and uses it in the SSE uvicorn config.

How Has This Been Tested?

Locally

Breaking Changes

No

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update

Checklist

  • I have read the MCP Documentation
  • My code follows the repository's style guidelines
  • New and existing tests pass locally
  • I have added appropriate error handling
  • I have added or updated documentation as needed

@TheRobotCarlson TheRobotCarlson changed the title Add hot reload functionality Add hot reload functionality to SSE Mar 8, 2025
@Kludex
Copy link
Copy Markdown
Member

Kludex commented Mar 10, 2025

Did you test this PR? I don't think this will reload, given the app is passed as an ASGI application object. It only works when it's passed as string.

@TheRobotCarlson
Copy link
Copy Markdown
Author

@Kludex I had monkey-patched that in my tests. I'll close out this PR until I can come back to address that.

@Edison-A-N
Copy link
Copy Markdown
Contributor

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.

3 participants