modelcontextprotocol / modelcontextprotocol/python-sdk
Audit MCPServer constructor parameters
Chưa có ai nhận issue này.
- Ngôn ngữ chính
- Python
- Star
- 24.3k
- Fork
- 4k
- Merge trung bình
- 1 ngày 1 giờ
- Pull request đã merge (30 ngày)
- 31
Mô tả
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.
Ref: Kludex's V2 checklist in #1253, follow-up to #1732
Hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Hướng nghiên cứu
Bắt đầu bằng việc xem xét constructor của MCPServer và entry point run() của nó, cùng với các thay đổi từ PR #1732 và checklist V2 trong #1253. Kiểm tra xem tham số nào thuộc về từng API, đồng thời ghi lại hoặc kiểm thử các quyết định thu được để constructor tối giản và phù hợp với V2.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- python
- Lĩnh vực
- backend-api-design
- Loại issue
- Tái cấu trúc
- Độ khó
- 5/5
- Thời gian dự kiến
- Hơn một tuần
- Mức độ hoạt động
- Ít trao đổi
- Độ rõ ràng
- Cần làm rõ
- Mức phù hợp với người mới
- 35/100