[Bug]: ACP sign-in replaces the agent's error message with a generic one

Đang mở Phù hợp với người mới
#30,218 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

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

Từ ngày 3/11/2026.

  • #30295 của @iScabrous — đang mở
  • #30321 của @Mohsintl — đang mở

Đánh giá

Độ khó
1/5
Thời gian dự kiến
Dưới một giờ
Mức phù hợp với người mới
85/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
authentication

Hướng nghiên cứu

Bắt đầu trong packages/provider-acp-registry/src/server/auth.ts, tại failure() và luồng xác minh phiên sau khi đăng nhập, để xem chuỗi chi tiết cố định được sử dụng ở đâu và AcpRequestError được truyền qua cause như thế nào. Tái hiện lỗi từ Settings → Providers bằng một nhà cung cấp ACP lệnh cục bộ có authenticate trả về thông báo lỗi JSON-RPC. Hoàn tất khi cả lỗi đăng nhập lẫn lỗi xác minh sau khi đăng nhập đều hiển thị thông báo của tác tử cùng với chi tiết chung.

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

Mô tả

bug needs-triage
Before submitting
  • I searched existing issues and did not find a duplicate.
  • I included enough detail to reproduce or investigate the problem.
Area

apps/server

Steps to reproduce
  1. Add a local-command ACP provider whose authenticate answers with a JSON-RPC error that has a message, e.g. {"code": -32000, "message": "Connect to the VPN, then sign in again."}
  2. Open Settings → Providers, select the provider, and press Sign in.
Expected behavior

The card shows the agent's message, so the user knows what to do, e.g. "The ACP agent could not complete sign-in. Connect to the VPN, then sign in again."

Actual behavior

The card shows only "The ACP agent could not complete sign-in." The agent's message is kept on the error's cause but never shown, because failure() in packages/provider-acp-registry/src/server/auth.ts uses the fixed detail string for every failure. The post-sign-in verify step ("The provider could not create a session after sign-in.") hides it the same way.

Appending the AcpRequestError's message to the detail in failure() would fix both.

Impact

Minor bug or occasional failure

Version or commit

main @ cefb544af3

Environment

macOS 26, desktop app, a local-command ACP agent

Logs or stack traces

Screenshots, recordings, or supporting files

No response

Workaround

None in the app; the reason only appears in the server's logs.

Ngôn ngữ chính
TypeScript
Star
24.8k
Fork
6.4k
Merge trung bình
7 giờ 39 phút
Pull request đã merge (30 ngày)
123

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

Mở trong Codespaces

Khởi chạy dev container của dự án ngay trên trình duyệt, bằng tài khoản GitHub của bạn.

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 pingdotgg/t3code

Tất cả issue của pingdotgg/t3code

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.