Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

server: registeredResource.update and registeredResourceTemplate.update desync title and metadata.title when updating metadata alone

Đang mở Phù hợp với người mới
  • Có gì không ổn?
#3,103 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

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
76/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
api

Hướng nghiên cứu

Bắt đầu tại server/src/server/mcp.ts, ở hai phương thức cập nhật quanh các dòng 783-787 và 824-828. Đọc cách mỗi phương thức áp dụng updates.metadata và cách các phương thức liệt kê tài nguyên kết hợp title với metadata. Kiểm tra các bài kiểm thử hiện có cho việc cập nhật tài nguyên và template đã đăng ký, sau đó bổ sung kiểm thử cho các cập nhật chỉ thay đổi metadata; hoàn tất khi title và title trong metadata nhất quán trong kết quả liệt kê.

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

Mô tả

While checking resource metadata updates in packages/server/src/server/mcp.ts (lines 783-787 and 824-828 on main commit b381559), I noticed _createRegisteredResource and _createRegisteredResourceTemplate copy updates.title into registeredResource.metadata when updates.title !== undefined, since resources/list and resources/templates/list spread ...resource.metadata after title: resource.title.

However, when update({ metadata }) is called without updates.title, registeredResource.metadata = updates.metadata replaces metadata without syncing registeredResource.title (so passing metadata: { title: 'New' } leaves registeredResource.title stale, while passing metadata with title: undefined overwrites title: resource.title with undefined in resources/list). Should update() keep registeredResource.title and registeredResource.metadata.title in sync when updates.metadata is provided? Can put up a small PR if that makes sense!

Ngôn ngữ chính
TypeScript
Star
13.7k
Fork
2.3k
Merge trung bình
1 ngày 2 giờ
Pull request đã merge (30 ngày)
60

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 modelcontextprotocol/typescript-sdk

Tất cả issue của modelcontextprotocol/typescript-sdk

Issue tương tự

Thêm issue về TypeScript

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.