bun default integrations diverge from Node's

オープン 初心者向け
#23,887 コメント 1 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

評価

難易度
2/5
見積もり時間
1〜3時間
初心者へのやさしさ
84/100
issue の種類
機能追加
明瞭さ
明確に書かれている
活発さ
活発
技術スタック
typescript

調査の方向性

packages/bun/src/sdk.ts:48 と node/src/sdk/index.ts:getBaseDefaultIntegrations を比較し、その後 conversationIdIntegration と systemErrorIntegration の統合定義を確認します。conversation integration を追加し、system-error の動作を決定して、issue #20237 への参照とともに child-process と worker-thread を意図的に除外することを文書化します。Bun のデフォルトとコメントにこれらの決定が反映されていれば完了です。

索引モデルが issue の本文から書いたものです。

説明

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.

主要言語
TypeScript
スター
8.7k
フォーク
1.9k
平均マージ
1日 17時間
マージ済み PR(30日)
521

コントリビューションガイド

コントリビューションガイドを開く

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

getsentry/sentry-javascript のほかの issue

getsentry/sentry-javascript の issue をすべて見る

似ている issue

TypeScript の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。