verify-skills: no negative test for the shipped skills/ validation path
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
- 72/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
- testing
Hướng nghiên cứu
Đọc scripts/verify-skills.mjs quanh các dòng 320–333 và các fixture trong verify-skills.main.test.mjs; trước tiên hãy chạy bài kiểm thử hiện có để hiểu cách nó gọi trình xác minh. Bổ sung kiểm thử bằng cách sử dụng một kho lưu trữ fixture có cả .claude/skills và skills, bao gồm một skill được cung cấp bị lỗi định dạng. Hoàn tất khi bài kiểm thử xác nhận rằng các skill được cung cấp bị lỗi định dạng sẽ bị từ chối.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Problem
Raised by Copilot on the v2.10.0 milestone merge (#2637, round 3, scripts/verify-skills.mjs:320). The validation of the packaged skills/*/SKILL.md (the !override branch, lines 320–333) is never reached by a test. Every fixture run in verify-skills.main.test.mjs passes an override directory, so a regression that stopped rejecting a missing or malformed shipped skill would leave the guard suite green.
Expected
An orchestration test against a fixture repository holding both .claude/skills and skills, including a malformed shipped skill that must fail. Alternatively, extract the traversal so it can be tested directly.
Priority
Low (rubric total 5): Severity 2 (test gap in repo tooling; nothing ships broken), Urgency 2, +1 milestoned.
- Ngôn ngữ chính
- TypeScript
- Star
- 11.1k
- Fork
- 1.5k
- Merge trung bình
- 4 giờ 28 phút
- Pull request đã merge (30 ngày)
- 151
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
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
modelcontextprotocol/inspector#2723 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 50/100
modelcontextprotocol/inspector#2724 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Modern subscriptions: a subscriptions/listen refused with HTTP 403 is retried eight times in silenceĐang mởbug v2
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 30/100
modelcontextprotocol/inspector#2722 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug v2
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 50/100
modelcontextprotocol/inspector#2721 · 2 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug v2
Độ khó 3/5 Nửa ngày Mức phù hợp với người mới 62/100
modelcontextprotocol/inspector#2719 ·
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ự
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 85/100
juice-shop/juice-shop#3686 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
dsh-market/dsh-market#877 ·
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 82/100
-
enhancement
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 74/100
ScottKirvan/GOMAC#61 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
stellar/stellar-dev-skill#151 ·
Maintainer thường phản hồi trong vòng 1 ngày