bun default integrations diverge from Node's

Đang mở Phù hợp với người mới
#23,887 1 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ó
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ính năng
Độ 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
observability-sre

Hướng nghiên cứu

So sánh packages/bun/src/sdk.ts:48 với node/src/sdk/index.ts:getBaseDefaultIntegrations, sau đó xem xét các định nghĩa integration cho conversationIdIntegration và systemErrorIntegration. Thêm conversation integration, quyết định behavior đối với system-error, và ghi lại các omission có chủ ý của child-process và worker-thread kèm tham chiếu đến issue #20237; hoàn thành khi các defaults và comment của Bun phản ánh những quyết định này.

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

Mô tả

Bun Feature javascript

packages/bun/src/sdk.ts:48 omits, relative to node/src/sdk/index.ts:getBaseDefaultIntegrations:

Integration Note
conversationIdIntegration pure core JS, no Node API. Cloudflare and vercel-edge both include it. Without it, setConversationId has no effect on gen_ai.* spans under Bun.
systemErrorIntegration needs a check
childProcessIntegration relies on child_process diagnostics channels Bun does not emit (see B1), so probably correct to omit, but say so in a comment
workerThreadsIntegration same

Work item. Add conversationIdIntegration() to Bun's defaults. For each remaining omission, either add it or leave a one-line comment saying which Bun gap makes it a no-op.

**Prior art **(tracked). #20237 (closed, not planned) already settled the childProcessIntegration and workerThreadsIntegration question, and documents the same root cause as B1: Bun implements the node:diagnostics_channel API but never publishes to the child_process or worker_threads channels. Omitting those two is correct; cite that issue in the comment. Nothing covers conversationIdIntegration or systemErrorIntegration on Bun.

Ngôn ngữ chính
TypeScript
Star
8.7k
Fork
1.9k
Merge trung bình
1 ngày 18 giờ
Pull request đã merge (30 ngày)
543

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

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 getsentry/sentry-javascript

Tất cả issue của getsentry/sentry-javascript

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.