Open
Conversation
Closed
jkawamoto
reviewed
Oct 22, 2025
Co-authored-by: Junpei Kawamoto <[email protected]>
There was a problem hiding this comment.
Pull Request Overview
This PR migrates the Eagle MCP Server from fastapi_mcp to the official FastMCP 2.0 SDK, transitioning from HTTP-based remote transport to stdio transport for improved local usage and compatibility with the MCP protocol standard.
Key Changes:
- Replaced
fastapi_mcpwithfastmcp>=2.12.4dependency - Converted FastAPI route-based architecture to FastMCP tool-based architecture
- Changed from HTTP/remote transport to stdio transport for MCP communication
Reviewed Changes
Copilot reviewed 21 out of 23 changed files in this pull request and generated 9 comments.
Show a summary per file
| File | Description |
|---|---|
| pyproject.toml | Updated dependency from fastapi-mcp to fastmcp>=2.12.4 and added project scripts/build configuration |
| main.py | Simplified to entry point wrapper calling server.main() instead of FastAPI/uvicorn server |
| server.py | New file implementing FastMCP server with stdio transport and tool registration |
| tools/*.py | New tool modules replacing route-based endpoints with @mcp.tool decorators |
| tools/init.py | New module exporting tool registration functions |
| routes/*.py | Removed FastAPI router implementations (replaced by tools) |
| schemas/*.py | Removed Pydantic request/response models (replaced by inline FastMCP annotations) |
| docs/* | Removed OpenAPI documentation files (no longer applicable with stdio transport) |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Co-authored-by: Copilot <[email protected]>
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 migrates the Eagle MCP Server from fastapi_mcp to the official FastMCP 2.0 SDK, replacing the remote transport with stdio transport for better local usage.
MCP config
MCP config: devepment