-
Notifications
You must be signed in to change notification settings - Fork 3.1k
Open
Labels
enhancementRequest for a new feature that's not currently supportedRequest for a new feature that's not currently supportedv2Ideas, requests and plans for v2 of the SDK which will incorporate major changes and fixesIdeas, requests and plans for v2 of the SDK which will incorporate major changes and fixes
Description
From the V2 checklist in #1253:
We need to recheck the constructor parameters of the MCPServer class. I did remove and refactor a bit, but we need to audit it a bit better.
PR #1732 renamed FastMCP to MCPServer and moved transport-specific options (host, port, stateless_http, etc.) to run(). However, the constructor still has parameters that may need further review:
- Are all remaining constructor parameters appropriate for the constructor vs.
run()? - Are there parameters that should be removed, renamed, or have their defaults reconsidered?
- Is the constructor signature clean and minimal for the common case?
This is a follow-up audit to ensure the MCPServer API surface is well-designed before V2 stabilizes.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
enhancementRequest for a new feature that's not currently supportedRequest for a new feature that's not currently supportedv2Ideas, requests and plans for v2 of the SDK which will incorporate major changes and fixesIdeas, requests and plans for v2 of the SDK which will incorporate major changes and fixes