bun default integrations diverge from Node's
まだ誰も着手していません。
評価
- 難易度
- 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 の本文から書いたものです。
説明
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
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
getsentry/sentry-javascript のほかの issue
-
Browser Waiting for: Product Owner
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100
getsentry/sentry-javascript#24577 · コメント 1 件 ·
-
Task
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
getsentry/sentry-javascript#24558 · コメント 1 件 ·
-
Task
難易度 1/5 1時間未満 初心者へのやさしさ 90/100
getsentry/sentry-javascript#24557 · コメント 1 件 ·
-
Task
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
getsentry/sentry-javascript#24556 · コメント 1 件 ·
-
Task
難易度 1/5 1〜3時間 初心者へのやさしさ 90/100
getsentry/sentry-javascript#24555 · コメント 1 件 ·
getsentry/sentry-javascript の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
copse-dev/agent-pane#2953 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
Eynzof/Hermes-CN-Desktop#610 ·
-
bug clawsweeper:linked-pr-open clawsweeper:needs-live-repro clawsweeper:no-new-fix-pr impact:message-loss issue-rating: 🐚 platinum hermit P2 regression
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
-
enhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
-
calcite-components needs triage refactor
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
Esri/calcite-design-system#15203 ·