SDK 1.0.26+ breaks Entra MCP sign-in unless the host registers its own Entra app (undocumented breaking change)

Đang mở
  • Có gì không ổn?
#2,896 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Chưa có ai nhận issue nà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
18/100
Loại issue
Lỗi
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
git, macos, nodejs, typescript
Lĩnh vực
authentication, devtools

Hướng nghiên cứu

The report centres on the CopilotClientOptions.entra option and EntraAuthConfig in the SDK's types.d.ts, plus whatever code handles Entra sign-in for MCP servers. Start by reading how the runtime's built-in public client is replaced by a required host client ID between SDK 1.0.24 and 1.0.26. Done means a clear decision on whether built-in sign-in is restored or the change is documented as breaking, with a note in the release notes.

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

Mô tả

What happened

Since @github/copilot-sdk 1.0.26 (runtime 1.0.103+), signing in to an MCP server protected by Microsoft Entra ID fails in an SDK host with:

Microsoft Entra authentication requires the host to configure an Entra client ID

The same servers signed in fine with SDK 1.0.24 (runtime 1.0.101). The runtime handled the Entra sign-in itself with a built-in public client, the same way the Copilot CLI still does today.

The release notes for v1.0.25 to v1.0.28 don't mention this. The only trace is the new CopilotClientOptions.entra?: EntraAuthConfig in types.d.ts, described as "Required for built-in Entra sign-in and token renewal" and marked @experimental. Updating the SDK silently broke Entra MCP servers for every user of my app.

Why a host client ID is not a fix

An SDK app is used across many organisations' tenants. Requiring the app's own Entra registration means every organisation must consent to that app before its users can reach their own MCP servers. That doesn't work in practice, and Copilot CLI users don't need it.

What I'm asking

Please undo this breaking change: restore the built-in Entra sign-in for SDK hosts as it worked in 1.0.24, the way the Copilot CLI still signs in, so the SDK is usable with Entra-protected MCP servers again without the host registering its own app.

If the change has to stay, please at least list it as a breaking change in the release notes.

Versions
  • Works: @github/copilot-sdk 1.0.24 (runtime 1.0.101).
  • Fails: @github/copilot-sdk 1.0.26 and later (runtime 1.0.103 and later), seen on macOS.
Ngôn ngữ chính
Java
Star
10.7k
Fork
1.5k
Không có pull request nào được merge trong 30 ngày

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 github/copilot-sdk

Tất cả issue của github/copilot-sdk

Issue tương tự

Thêm issue về Java

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.