bun default integrations diverge from Node's

Abierto Apto para principiantes
#23,887 1 comentario 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
84/100
Tipo de issue
Nueva funcionalidad
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
typescript
Área
observability-sre

Línea de trabajo

Compara packages/bun/src/sdk.ts:48 con node/src/sdk/index.ts:getBaseDefaultIntegrations y luego revisa las definiciones de integración de conversationIdIntegration y systemErrorIntegration. Añade la integración de conversación, decide el comportamiento ante errores del sistema y documenta las omisiones intencionadas de los procesos hijo y los hilos de trabajo con una referencia al issue #20237; se considera terminado cuando los valores predeterminados y los comentarios de Bun reflejen estas decisiones.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

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.

Lenguaje dominante
TypeScript
Estrellas
8.7k
Forks
1.9k
Merge medio
1 d 17 h
PR fusionados (30 d)
521

Guía de contribución

Abrir la guía de contribución

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de getsentry/sentry-javascript

Todos los issues de getsentry/sentry-javascript

Issues similares

Más issues de TypeScript

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.