Delete the dead pkg/vmcp/server/adapter package

Đang mở Phù hợp với người mới
  • Có gì không ổn?
#6,191 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

@suantea đang làm issue này rồi.

Từ ngày 10/10/2026.

  • #6791 của @suantea — đã đóng, không merge

Đá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
84/100
Loại issue
Tái cấu trúc
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Ít trao đổi
Công nghệ
go
Lĩnh vực
backend

Hướng nghiên cứu

Bắt đầu với pkg/vmcp/server/adapter/capability_adapter.go và handler_factory.go, bao gồm cả các bài kiểm thử của chúng. Thực hiện kiểm tra tham chiếu trên toàn bộ repository và xác nhận rằng cmd/ và operator không import package này. Hoàn tất có nghĩa là package và các bài kiểm thử của nó đã bị xóa, không còn tham chiếu nào và các bước kiểm tra của repository đều đạt.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

needs-triage

Surfaced in jhrozek's review of #6180.

Problem

pkg/vmcp/server/adapter/ (capability_adapter.go, handler_factory.go) has no non-test callers — NewCapabilityAdapter/NewDefaultHandlerFactory are dead code. Its CreateResourceHandler (handler_factory.go:164) still drops result.Meta while its sibling CreateToolHandler preserves it — it's now the only surviving copy of the resource-_meta drop that #6180 removed from the live paths, which invites re-divergence if anyone ever wires it back in.

Work

Delete the package and its tests. Verify with a repo-wide reference check first (the review found no non-test callers; confirm nothing in cmd/ or the operator imports it).

Ngôn ngữ chính
Go
Star
2.3k
Fork
326
Merge trung bình
1 ngày 53 phút
Pull request đã merge (30 ngày)
155

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. 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.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của stacklok/toolhive

Tất cả issue của stacklok/toolhive

Issue tương tự

Thêm issue về Go

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.