Skip to content

Commit a5ea42e

Browse files
authored
feat(bun/v8): Remove all deprecations from Bun SDK (#10971)
1 parent 7f2e804 commit a5ea42e

File tree

4 files changed

+17
-27
lines changed

4 files changed

+17
-27
lines changed

dev-packages/e2e-tests/test-applications/node-exports-test-app/scripts/consistentExports.ts

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -54,6 +54,13 @@ const DEPENDENTS: Dependent[] = [
5454
'NodeClient',
5555
'hapiErrorPlugin',
5656
'makeNodeTransport',
57+
// TODO: remove these when we switch exports from nodeExperimentalExports to nodeExports
58+
'Integrations',
59+
'addGlobalEventProcessor',
60+
'getActiveTransaction',
61+
'getCurrentHub',
62+
'makeMain',
63+
'startTransaction',
5764
],
5865
},
5966
{

packages/bun/src/index.ts

Lines changed: 0 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -22,8 +22,6 @@ export type { TransactionNamingScheme } from '@sentry/node-experimental';
2222
export type { BunOptions } from './types';
2323

2424
export {
25-
// eslint-disable-next-line deprecation/deprecation
26-
addGlobalEventProcessor,
2725
addEventProcessor,
2826
addBreadcrumb,
2927
addIntegration,
@@ -33,22 +31,14 @@ export {
3331
close,
3432
createTransport,
3533
flush,
36-
// eslint-disable-next-line deprecation/deprecation
37-
getActiveTransaction,
38-
// eslint-disable-next-line deprecation/deprecation
39-
getCurrentHub,
4034
getClient,
4135
isInitialized,
4236
getCurrentScope,
4337
getGlobalScope,
4438
getIsolationScope,
4539
Hub,
46-
// eslint-disable-next-line deprecation/deprecation
47-
makeMain,
4840
setCurrentClient,
4941
Scope,
50-
// eslint-disable-next-line deprecation/deprecation
51-
startTransaction,
5242
SDK_VERSION,
5343
setContext,
5444
setExtra,
@@ -119,13 +109,4 @@ export {
119109
init,
120110
} from './sdk';
121111

122-
import { Integrations as NodeIntegrations } from '@sentry/node-experimental';
123-
import { BunServer } from './integrations/bunserver';
124112
export { bunServerIntegration } from './integrations/bunserver';
125-
126-
const INTEGRATIONS = {
127-
...NodeIntegrations,
128-
BunServer,
129-
};
130-
131-
export { INTEGRATIONS as Integrations };

packages/bun/src/integrations/bunserver.ts

Lines changed: 10 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,6 @@ import {
44
Transaction,
55
captureException,
66
continueTrace,
7-
convertIntegrationFnToClass,
87
defineIntegration,
98
getCurrentScope,
109
setHttpStatus,
@@ -25,15 +24,20 @@ const _bunServerIntegration = (() => {
2524
};
2625
}) satisfies IntegrationFn;
2726

28-
export const bunServerIntegration = defineIntegration(_bunServerIntegration);
29-
3027
/**
3128
* Instruments `Bun.serve` to automatically create transactions and capture errors.
3229
*
33-
* @deprecated Use `bunServerIntegration()` instead.
30+
* Enabled by default in the Bun SDK.
31+
*
32+
* ```js
33+
* Sentry.init({
34+
* integrations: [
35+
* Sentry.bunServerIntegration(),
36+
* ],
37+
* })
38+
* ```
3439
*/
35-
// eslint-disable-next-line deprecation/deprecation
36-
export const BunServer = convertIntegrationFnToClass(INTEGRATION_NAME, bunServerIntegration);
40+
export const bunServerIntegration = defineIntegration(_bunServerIntegration);
3741

3842
/**
3943
* Instruments Bun.serve by patching it's options.

packages/bun/test/integrations/bunserver.test.ts

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -79,8 +79,6 @@ describe('Bun Serve Integration', () => {
7979
// span.endTimestamp is already set in `finishTransaction` hook
8080
expect(transaction.isRecording()).toBe(false);
8181

82-
// eslint-disable-next-line deprecation/deprecation
83-
expect(transaction.metadata?.dynamicSamplingContext).toStrictEqual({ version: '1.0', environment: 'production' });
8482
expect(getDynamicSamplingContextFromSpan(transaction)).toStrictEqual({
8583
version: '1.0',
8684
environment: 'production',

0 commit comments

Comments
 (0)