Merge branch 'develop' into fix/elysia-sdk-metadata

Author: isaacs

packages/elysia/src/sdk.ts

@@ -3,6 +3,7 @@ import {
   bunServerIntegration,
   getDefaultIntegrations as getBunDefaultIntegrations,
   makeFetchTransport,
+  makeNodeTransport,
 } from '@sentry/bun';
 import type { Integration, Options } from '@sentry/core';
 import { applySdkMetadata } from '@sentry/core';
@@ -50,7 +51,7 @@ export function init(userOptions: ElysiaOptions = {}): NodeClient | undefined {
 
   applySdkMetadata(options, 'elysia', ['elysia', options.runtime.name]);
 
-  options.transport = options.transport || makeFetchTransport;
+  options.transport = options.transport || (options.runtime.name === 'bun' ? makeFetchTransport : makeNodeTransport);
 
   if (options.defaultIntegrations === undefined) {
     options.defaultIntegrations = getDefaultIntegrations(options);

packages/elysia/test/sdk.test.ts

@@ -4,11 +4,13 @@ import { afterEach, describe, expect, it, vi } from 'vitest';
 const mockInitNode = vi.fn();
 const mockGetBunDefaultIntegrations = vi.fn(() => [] as Integration[]);
 const mockMakeFetchTransport = vi.fn();
+const mockMakeNodeTransport = vi.fn();
 
 vi.mock('@sentry/bun', () => ({
   init: mockInitNode,
   getDefaultIntegrations: mockGetBunDefaultIntegrations,
   makeFetchTransport: mockMakeFetchTransport,
+  makeNodeTransport: mockMakeNodeTransport,
   bunServerIntegration: () => ({ name: 'BunServer', setupOnce: vi.fn() }),
 }));
 
@@ -51,16 +53,24 @@ describe('init', () => {
     );
   });
 
-  it('uses makeFetchTransport by default', () => {
+  it('uses makeNodeTransport by default on Node', () => {
     init({ dsn: 'https://examplePublicKey@o0.ingest.sentry.io/0' });
 
     expect(mockInitNode).toHaveBeenCalledWith(
       expect.objectContaining({
-        transport: mockMakeFetchTransport,
+        transport: mockMakeNodeTransport,
       }),
     );
   });
 
+  it('uses makeFetchTransport by default on Bun', () => {
+    vi.stubGlobal('Bun', { version: '1.2.3' });
+
+    init({ dsn: 'https://examplePublicKey@o0.ingest.sentry.io/0' });
+
+    expect(mockInitNode).toHaveBeenCalledWith(expect.objectContaining({ transport: mockMakeFetchTransport }));
+  });
+
   it('allows overriding transport', () => {
     const customTransport = vi.fn();
     init({ dsn: 'https://examplePublicKey@o0.ingest.sentry.io/0', transport: customTransport });