modelcontextprotocol / modelcontextprotocol/python-sdk
Low-level Server support for mcp dev and mcp run command
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ả
Currently the mcp dev and mcp run command does not support the low-level Server class. I think it's a better developer experience if we support both classes for these 2 commands, assuming there's no inherent reason for not doing so.
I'm interested in adding support for the low-level Server to both mcp dev and mcp run and I would like seek the maintainer's guidance on this. It looks like the only thing missing is the dependencies field, which is pulled from the Settings class and does not exist in the low-level Server. I'm thinking about adding a similar Settings class to server/lowlevel/server.py and thus introducing the dependencies field. The downside is more complexity in server/lowlevel/server.py which I'm not sure if it's acceptable to the maintainers and falls in line with the long term roadmap for the code base.
Please advise. Thanks.
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 cách lần theo các điểm vào của lệnh mcp dev và mcp run, rồi so sánh cách chúng xử lý các lớp Server cấp cao và cấp thấp. Đọc server/lowlevel/server.py và lớp Settings hiện có để hiểu trường dependencies còn thiếu. Công việc được hoàn tất khi maintainer đồng ý về thiết kế và đã xác minh hỗ trợ Server cấp thấp trong cả hai lệnh.
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, cli
- Loại issue
- Tính năng
- Độ khó
- 5/5
- Thời gian dự kiến
- Hơn một tuần
- Mức độ hoạt động
- Đình trệ
- Độ rõ ràng
- Cần làm rõ
- Mức phù hợp với người mới
- 30/100