everything: misleading names in get-resource-links (isOdd) and copy-pasted JSDoc tool names
Maintainer thường phản hồi trong vòng 1 ngày
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 2/5
- Thời gian dự kiến
- 1-3 giờ
- Mức phù hợp với người mới
- 82/100
- Loại issue
- Lỗi
- Độ rõ ràng
- Đặc tả rõ ràng
- Mức độ hoạt động
- Sôi nổi
- Công nghệ
- typescript
- Lĩnh vực
- devtools, documentation
Hướng nghiên cứu
Đọc src/everything/tools/get-resource-links.ts, src/everything/tools/trigger-long-running-operation.ts và docs/features.md; các PR trước đây #4503 và #4769 được mô tả là nguyên mẫu cho mã đã tái cấu trúc. Đổi tên flag gây hiểu lầm, ghi lại ánh xạ ID chẵn/lẻ và sửa từng tên công cụ trong JSDoc. Hoàn tất khi flag khớp với hành vi của nó, ánh xạ được ghi lại và cả hai phần đầu đều khớp với các công cụ mà tệp tương ứng đăng ký.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Problem
Two small naming slips in the everything server, which people read to learn the protocol:
- In
src/everything/tools/get-resource-links.ts, the flagisOddis computed asresourceId % 2 === 0, so it is true for even IDs (which get Text resources). The behavior is right; the name says the opposite, anddocs/features.mddoesn't say which IDs map to which resource type. - The JSDoc header in
src/everything/tools/get-resource-links.tsnames theget-resource-referencetool (copied from its neighbor), and the one insrc/everything/tools/trigger-long-running-operation.tsmisspells its tool's name.
Expected
Rename the flag (for example to isText), document the even/odd mapping, and have each JSDoc header name the tool its file registers.
Prior work
- #4503 by @Joosboy
- #4769 by @simpleqt
The PR(s) above targeted the pre-refactor code and are closed as part of the backlog triage in #4875; this issue is where the work continues, with the PR(s) as a prototype. Thank you to their authors.
- Ngôn ngữ chính
- TypeScript
- Star
- 91.4k
- Fork
- 11.8k
- Merge trung bình
- 7 giờ 36 phút
- Pull request đã merge (30 ngày)
- 126
Chuẩn bị môi trường
- Không có Dockerfile hay 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/servers
-
filesystem: search_files does not expand non-wildcard excludePatterns like directory_tree doesĐang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
modelcontextprotocol/servers#5216 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug v2
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 79/100
modelcontextprotocol/servers#5215 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
documentation server-git server-time v2
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
modelcontextprotocol/servers#5213 · 2 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
documentation v2
Độ khó 1/5 1-3 giờ Mức phù hợp với người mới 78/100
modelcontextprotocol/servers#5212 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
documentation v2
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
modelcontextprotocol/servers#5211 · 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/servers
Issue tương tự
-
frontend
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 86/100
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 75/100
ranxianglei/billion-context#4172 · 1 bình luận ·
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 75/100
awslabs/threat-composer#362 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Area/Cloud Needs Triage Type/Bug
Độ khó 1/5 1-3 giờ Mức phù hợp với người mới 86/100
wso2/product-integrator#2847 ·
Maintainer thường phản hồi trong vòng 1 ngày