No dedicated test fixture simulates a mid-session server crash/disconnect
Maintainer thường phản hồi trong vòng 1 ngày
Đánh giá
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức phù hợp với người mới
- 68/100
- Loại issue
- Tính năng
- Độ rõ ràng
- Khá rõ ràng
- Mức độ hoạt động
- Sôi nổi
- Công nghệ
- typescript
- Lĩnh vực
- testing
Hướng nghiên cứu
Bắt đầu với logic Crash-Reconciliation trong core/mcp/inspectorClient.ts, đặc biệt là các khu vực xung quanh dòng 1207-1260 và 1965-2160, sau đó kiểm tra các fixture test-servers hiện có. Thêm một fixture hoặc mode bị crash hoặc bị treo tại một điểm session được chọn, và thêm các test core thực thi những đường dẫn reconciliation đó và luôn vượt qua một cách ổn định.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Which client?
All / shared core
The problem
core/mcp/inspectorClient.ts has substantial crash-reconciliation logic (roughly lines 1207-1260 and 1965-2160) for handling a server process dying or disconnecting mid-session, but there's no dedicated test fixture/mock server that deliberately crashes at a chosen point in a session. Coverage of this logic currently relies on incidental paths rather than a targeted simulation, so a regression here could slip through.
Solution you have in mind (optional)
Add a test-servers fixture (or a mode flag on an existing one) that can be told to crash/hang at a specific point in a session, and add core tests that drive the client through the reconciliation paths using it.
Alternatives or workarounds you have tried
No response
Already built it locally?
No response
Before you submit
- I searched existing issues and this is not a duplicate.
- This is a request for the Inspector itself, not for the MCP specification or an SDK.
- Ngôn ngữ chính
- TypeScript
- Star
- 11.1k
- Fork
- 1.5k
- Merge trung bình
- 4 giờ 4 phút
- Pull request đã merge (30 ngày)
- 170
Chuẩn bị môi trường
- Có Dockerfile hoặc tệp Docker Compose
- Có mẫu pull request
- Đọc 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.
Issue khác của modelcontextprotocol/inspector
-
chore v2
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 48/100
modelcontextprotocol/inspector#2715 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
enhancement v2
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 35/100
modelcontextprotocol/inspector#2705 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug v2
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 35/100
modelcontextprotocol/inspector#2672 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
docs: record retroactive DCO signoffs for the 10 unsigned commits on v2/mainCó thể đã có người làm @olaservo đã nhận 6 ngày trước. Đang mởdocumentation v2
modelcontextprotocol/inspector#2617 · 2 bình luận · 2 người được giao ·
Maintainer thường phản hồi trong vòng 1 ngày
-
blocked chore v2
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 48/100
modelcontextprotocol/inspector#2548 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
Tất cả issue của modelcontextprotocol/inspector
Issue tương tự
-
Độ khó 1/5 1-3 giờ Mức phù hợp với người mới 70/100
siyuan-note/siyuan#20517 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
NaturalIntelligence/fast-xml-parser#890 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 3 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 62/100
zsviczian/obsidian-excalidraw-plugin#3000 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
pnpm/get.pnpm.io#68 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
nuxt-modules/better-auth#578 · 2 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày