Skip to content

Commit dca7d2f

Browse files
lobsterkatielobsterkatie
committed
ref(build): Turn on isolatedModules TS option (#4497)
1 parent cf416fa commit dca7d2f

File tree

29 files changed

+130
-115
lines changed

29 files changed

+130
-115
lines changed

‎packages/angular/src/index.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,9 @@
1+
export type { ErrorHandlerOptions } from './errorhandler';
2+
13
export * from '@sentry/browser';
24

35
export { init } from './sdk';
4-
export { createErrorHandler, ErrorHandlerOptions, SentryErrorHandler } from './errorhandler';
6+
export { createErrorHandler, SentryErrorHandler } from './errorhandler';
57
export {
68
getActiveTransaction,
79
// TODO `instrumentAngularRouting` is just an alias for `routingInstrumentation`; deprecate the latter at some point

‎packages/browser/src/backend.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ export class BrowserBackend extends BaseBackend<BrowserOptions> {
4545
/**
4646
* @inheritDoc
4747
*/
48-
public eventFromMessage(message: string, level: Severity = Severity.Info, hint?: EventHint): PromiseLike<Event> {
48+
public eventFromMessage(message: string, level: Severity = 'info' as Severity, hint?: EventHint): PromiseLike<Event> {
4949
return eventFromMessage(message, level, hint, this._options.attachStacktrace);
5050
}
5151

‎packages/browser/src/eventbuilder.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -161,7 +161,7 @@ export function eventFromException(
161161
const syntheticException = (hint && hint.syntheticException) || undefined;
162162
const event = eventFromUnknownInput(exception, syntheticException, attachStacktrace);
163163
addExceptionMechanism(event); // defaults to { type: 'generic', handled: true }
164-
event.level = Severity.Error;
164+
event.level = 'error' as Severity;
165165
if (hint && hint.event_id) {
166166
event.event_id = hint.event_id;
167167
}
@@ -174,7 +174,7 @@ export function eventFromException(
174174
*/
175175
export function eventFromMessage(
176176
message: string,
177-
level: Severity = Severity.Info,
177+
level: Severity = 'info' as Severity,
178178
hint?: EventHint,
179179
attachStacktrace?: boolean,
180180
): PromiseLike<Event> {

‎packages/browser/src/exports.ts

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
export {
1+
export type {
22
Breadcrumb,
33
BreadcrumbHint,
44
Request,
@@ -15,7 +15,10 @@ export {
1515
User,
1616
} from '@sentry/types';
1717

18-
export { SeverityLevel } from '@sentry/utils';
18+
export type { SeverityLevel } from '@sentry/utils';
19+
20+
export type { BrowserOptions } from './backend';
21+
export type { ReportDialogOptions } from './helpers';
1922

2023
export {
2124
addGlobalEventProcessor,
@@ -41,9 +44,8 @@ export {
4144
withScope,
4245
} from '@sentry/core';
4346

44-
export { BrowserOptions } from './backend';
4547
export { BrowserClient } from './client';
46-
export { injectReportDialog, ReportDialogOptions } from './helpers';
48+
export { injectReportDialog } from './helpers';
4749
export { eventFromException, eventFromMessage } from './eventbuilder';
4850
export { defaultIntegrations, forceLoad, init, lastEventId, onLoad, showReportDialog, flush, close, wrap } from './sdk';
4951
export { SDK_NAME } from './version';

‎packages/browser/src/integrations/breadcrumbs.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -230,7 +230,7 @@ function _fetchBreadcrumb(handlerData: { [key: string]: any }): void {
230230
{
231231
category: 'fetch',
232232
data: handlerData.fetchData,
233-
level: Severity.Error,
233+
level: 'error' as Severity,
234234
type: 'http',
235235
},
236236
{

‎packages/browser/src/integrations/globalhandlers.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -98,7 +98,7 @@ function _installGlobalOnErrorHandler(): void {
9898
column,
9999
);
100100

101-
event.level = Severity.Error;
101+
event.level = 'error' as Severity;
102102

103103
addMechanismAndCapture(hub, error, event, 'onerror');
104104
},
@@ -144,7 +144,7 @@ function _installGlobalOnUnhandledRejectionHandler(): void {
144144
? _eventFromRejectionWithPrimitive(error)
145145
: eventFromUnknownInput(error, undefined, attachStacktrace, true);
146146

147-
event.level = Severity.Error;
147+
event.level = 'error' as Severity;
148148

149149
addMechanismAndCapture(hub, error, event, 'onunhandledrejection');
150150
return;

‎packages/core/src/index.ts

Lines changed: 15 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,14 @@
1+
export type { APIDetails } from './api';
2+
export type { BackendClass } from './basebackend';
3+
export type { ClientClass } from './sdk';
4+
export type {
5+
BaseTransportOptions,
6+
NewTransport,
7+
TransportMakeRequestResponse,
8+
TransportRequest,
9+
TransportRequestExecutor,
10+
} from './transports/base';
11+
112
export {
213
addBreadcrumb,
314
captureException,
@@ -17,26 +28,19 @@ export { addGlobalEventProcessor, getCurrentHub, getHubFromCarrier, Hub, makeMai
1728
export {
1829
// eslint-disable-next-line deprecation/deprecation
1930
API,
20-
APIDetails,
2131
getEnvelopeEndpointWithUrlEncodedAuth,
2232
getStoreEndpointWithUrlEncodedAuth,
2333
getRequestHeaders,
2434
initAPIDetails,
2535
getReportDialogEndpoint,
2636
} from './api';
2737
export { BaseClient } from './baseclient';
28-
export { BackendClass, BaseBackend } from './basebackend';
38+
export { BaseBackend } from './basebackend';
2939
export { eventToSentryRequest, sessionToSentryRequest } from './request';
30-
export { initAndBind, ClientClass } from './sdk';
40+
export { initAndBind } from './sdk';
3141
export { NoopTransport } from './transports/noop';
32-
export {
33-
BaseTransportOptions,
34-
createTransport,
35-
NewTransport,
36-
TransportMakeRequestResponse,
37-
TransportRequest,
38-
TransportRequestExecutor,
39-
} from './transports/base';
42+
export { createTransport } from './transports/base';
43+
4044
export { SDK_VERSION } from './version';
4145

4246
import * as Integrations from './integrations';

‎packages/core/test/mocks/backend.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ export class TestBackend extends BaseBackend<TestOptions> {
3838
});
3939
}
4040

41-
public eventFromMessage(message: string, level: Severity = Severity.Info): PromiseLike<Event> {
41+
public eventFromMessage(message: string, level: Severity = 'info' as Severity): PromiseLike<Event> {
4242
return resolvedSyncPromise({ message, level });
4343
}
4444

‎packages/hub/src/index.ts

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,10 @@
1+
export type {
2+
Carrier,
3+
// eslint-disable-next-line deprecation/deprecation
4+
DomainAsCarrier,
5+
Layer,
6+
} from './hub';
7+
18
export { addGlobalEventProcessor, Scope } from './scope';
29
export { Session } from './session';
310
export { SessionFlusher } from './sessionflusher';
@@ -10,8 +17,4 @@ export {
1017
Hub,
1118
makeMain,
1219
setHubOnCarrier,
13-
Carrier,
14-
// eslint-disable-next-line deprecation/deprecation
15-
DomainAsCarrier,
16-
Layer,
1720
} from './hub';

‎packages/hub/test/scope.test.ts

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -85,8 +85,8 @@ describe('Scope', () => {
8585

8686
test('setLevel', () => {
8787
const scope = new Scope();
88-
scope.setLevel(Severity.Critical);
89-
expect((scope as any)._level).toEqual(Severity.Critical);
88+
scope.setLevel('critical' as Severity);
89+
expect((scope as any)._level).toEqual('critical' as Severity);
9090
});
9191

9292
test('setTransactionName', () => {
@@ -137,8 +137,8 @@ describe('Scope', () => {
137137

138138
test('chaining', () => {
139139
const scope = new Scope();
140-
scope.setLevel(Severity.Critical).setUser({ id: '1' });
141-
expect((scope as any)._level).toEqual(Severity.Critical);
140+
scope.setLevel('critical' as Severity).setUser({ id: '1' });
141+
expect((scope as any)._level).toEqual('critical' as Severity);
142142
expect((scope as any)._user).toEqual({ id: '1' });
143143
});
144144
});
@@ -202,7 +202,7 @@ describe('Scope', () => {
202202
scope.setTag('a', 'b');
203203
scope.setUser({ id: '1' });
204204
scope.setFingerprint(['abcd']);
205-
scope.setLevel(Severity.Warning);
205+
scope.setLevel('warning' as Severity);
206206
scope.setTransactionName('/abc');
207207
scope.addBreadcrumb({ message: 'test' });
208208
scope.setContext('os', { id: '1' });
@@ -294,11 +294,11 @@ describe('Scope', () => {
294294
test('scope level should have priority over event level', () => {
295295
expect.assertions(1);
296296
const scope = new Scope();
297-
scope.setLevel(Severity.Warning);
297+
scope.setLevel('warning' as Severity);
298298
const event: Event = {};
299-
event.level = Severity.Critical;
299+
event.level = 'critical' as Severity;
300300
return scope.applyToEvent(event).then(processedEvent => {
301-
expect(processedEvent!.level).toEqual(Severity.Warning);
301+
expect(processedEvent!.level).toEqual('warning' as Severity);
302302
});
303303
});
304304

@@ -410,7 +410,7 @@ describe('Scope', () => {
410410
scope.setContext('foo', { id: '1' });
411411
scope.setContext('bar', { id: '2' });
412412
scope.setUser({ id: '1337' });
413-
scope.setLevel(Severity.Info);
413+
scope.setLevel('info' as Severity);
414414
scope.setFingerprint(['foo']);
415415
scope.setRequestSession({ status: 'ok' });
416416
});
@@ -458,7 +458,7 @@ describe('Scope', () => {
458458
localScope.setContext('bar', { id: '3' });
459459
localScope.setContext('baz', { id: '4' });
460460
localScope.setUser({ id: '42' });
461-
localScope.setLevel(Severity.Warning);
461+
localScope.setLevel('warning' as Severity);
462462
localScope.setFingerprint(['bar']);
463463
(localScope as any)._requestSession = { status: 'ok' };
464464

‎packages/minimal/test/lib/minimal.test.ts

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -165,8 +165,8 @@ describe('Minimal', () => {
165165
const client: any = new TestClient({});
166166
const scope = getCurrentHub().pushScope();
167167
getCurrentHub().bindClient(client);
168-
scope.setLevel(Severity.Warning);
169-
expect(global.__SENTRY__.hub._stack[1].scope._level).toEqual(Severity.Warning);
168+
scope.setLevel('warning' as Severity);
169+
expect(global.__SENTRY__.hub._stack[1].scope._level).toEqual('warning' as Severity);
170170
});
171171
});
172172

@@ -245,16 +245,16 @@ describe('Minimal', () => {
245245

246246
test('withScope', () => {
247247
withScope(scope => {
248-
scope.setLevel(Severity.Warning);
248+
scope.setLevel('warning' as Severity);
249249
scope.setFingerprint(['1']);
250250
withScope(scope2 => {
251-
scope2.setLevel(Severity.Info);
251+
scope2.setLevel('info' as Severity);
252252
scope2.setFingerprint(['2']);
253253
withScope(scope3 => {
254254
scope3.clear();
255-
expect(global.__SENTRY__.hub._stack[1].scope._level).toEqual(Severity.Warning);
255+
expect(global.__SENTRY__.hub._stack[1].scope._level).toEqual('warning' as Severity);
256256
expect(global.__SENTRY__.hub._stack[1].scope._fingerprint).toEqual(['1']);
257-
expect(global.__SENTRY__.hub._stack[2].scope._level).toEqual(Severity.Info);
257+
expect(global.__SENTRY__.hub._stack[2].scope._level).toEqual('info' as Severity);
258258
expect(global.__SENTRY__.hub._stack[2].scope._fingerprint).toEqual(['2']);
259259
expect(global.__SENTRY__.hub._stack[3].scope._level).toBeUndefined();
260260
});

‎packages/nextjs/src/index.server.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -134,8 +134,8 @@ function filterTransactions(event: Event): Event | null {
134134
return event.type === 'transaction' && event.transaction === '/404' ? null : event;
135135
}
136136

137+
export type { SentryWebpackPluginOptions } from './config/types';
137138
export { withSentryConfig } from './config';
138-
export { SentryWebpackPluginOptions } from './config/types';
139139
export { withSentry } from './utils/withSentry';
140140

141141
// Wrap various server methods to enable error monitoring and tracing. (Note: This only happens for non-Vercel

‎packages/node-integration-tests/suites/public-api/addBreadcrumb/multiple_breadcrumbs/scenario.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ Sentry.init({
88
Sentry.addBreadcrumb({
99
category: 'foo',
1010
message: 'bar',
11-
level: Sentry.Severity.Critical,
11+
level: 'critical' as Sentry.Severity,
1212
});
1313

1414
Sentry.addBreadcrumb({

‎packages/node-integration-tests/suites/public-api/addBreadcrumb/simple_breadcrumb/scenario.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ Sentry.init({
88
Sentry.addBreadcrumb({
99
category: 'foo',
1010
message: 'bar',
11-
level: Sentry.Severity.Critical,
11+
level: 'critical' as Sentry.Severity,
1212
});
1313

1414
Sentry.captureMessage('test_simple');

‎packages/node-integration-tests/suites/public-api/captureMessage/with_level/scenario.ts

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -5,10 +5,10 @@ Sentry.init({
55
release: '1.0',
66
});
77

8-
Sentry.captureMessage('debug_message', Sentry.Severity.Debug);
9-
Sentry.captureMessage('info_message', Sentry.Severity.Info);
10-
Sentry.captureMessage('warning_message', Sentry.Severity.Warning);
11-
Sentry.captureMessage('error_message', Sentry.Severity.Error);
12-
Sentry.captureMessage('fatal_message', Sentry.Severity.Fatal);
13-
Sentry.captureMessage('critical_message', Sentry.Severity.Critical);
14-
Sentry.captureMessage('log_message', Sentry.Severity.Log);
8+
Sentry.captureMessage('debug_message', 'debug' as Sentry.Severity);
9+
Sentry.captureMessage('info_message', 'info' as Sentry.Severity);
10+
Sentry.captureMessage('warning_message', 'warning' as Sentry.Severity);
11+
Sentry.captureMessage('error_message', 'error' as Sentry.Severity);
12+
Sentry.captureMessage('fatal_message', 'fatal' as Sentry.Severity);
13+
Sentry.captureMessage('critical_message', 'critical' as Sentry.Severity);
14+
Sentry.captureMessage('log_message', 'log' as Sentry.Severity);

‎packages/node/src/backend.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ export class NodeBackend extends BaseBackend<NodeOptions> {
2222
/**
2323
* @inheritDoc
2424
*/
25-
public eventFromMessage(message: string, level: Severity = Severity.Info, hint?: EventHint): PromiseLike<Event> {
25+
public eventFromMessage(message: string, level: Severity = 'info' as Severity, hint?: EventHint): PromiseLike<Event> {
2626
return resolvedSyncPromise(eventFromMessage(message, level, hint, this._options.attachStacktrace));
2727
}
2828

‎packages/node/src/eventbuilder.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -92,7 +92,7 @@ export function eventFromUnknownInput(exception: unknown, hint?: EventHint): Eve
9292
*/
9393
export function eventFromMessage(
9494
message: string,
95-
level: Severity = Severity.Info,
95+
level: Severity = 'info' as Severity,
9696
hint?: EventHint,
9797
attachStacktrace?: boolean,
9898
): Event {

‎packages/node/src/index.ts

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
export {
1+
export type {
22
Breadcrumb,
33
BreadcrumbHint,
44
Request,
@@ -15,7 +15,9 @@ export {
1515
User,
1616
} from '@sentry/types';
1717

18-
export { SeverityLevel } from '@sentry/utils';
18+
export type { SeverityLevel } from '@sentry/utils';
19+
20+
export type { NodeOptions } from './types';
1921

2022
export {
2123
addGlobalEventProcessor,
@@ -41,7 +43,6 @@ export {
4143
withScope,
4244
} from '@sentry/core';
4345

44-
export { NodeOptions } from './types';
4546
export { NodeBackend } from './backend';
4647
export { NodeClient } from './client';
4748
export { defaultIntegrations, init, lastEventId, flush, close, getSentryRelease } from './sdk';

‎packages/node/src/integrations/onuncaughtexception.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -77,7 +77,7 @@ export class OnUncaughtException implements Integration {
7777

7878
if (hub.getIntegration(OnUncaughtException)) {
7979
hub.withScope((scope: Scope) => {
80-
scope.setLevel(Severity.Fatal);
80+
scope.setLevel('fatal' as Severity);
8181
hub.captureException(error, {
8282
originalException: error,
8383
data: { mechanism: { handled: false, type: 'onuncaughtexception' } },

‎packages/node/src/transports/index.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
1+
export type { NodeTransportOptions } from './new';
12
export { BaseTransport } from './base';
23
export { HTTPTransport } from './http';
34
export { HTTPSTransport } from './https';
4-
export { makeNodeTransport, NodeTransportOptions } from './new';
5+
export { makeNodeTransport } from './new';

‎packages/serverless/src/awslambda.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -276,7 +276,7 @@ export function wrapHandler<TEvent, TResult>(
276276
timeoutWarningTimer = setTimeout(() => {
277277
withScope(scope => {
278278
scope.setTag('timeout', humanReadableTimeout);
279-
captureMessage(`Possible function timeout: ${context.functionName}`, Sentry.Severity.Warning);
279+
captureMessage(`Possible function timeout: ${context.functionName}`, 'warning' as Sentry.Severity);
280280
});
281281
}, timeoutWarningDelay);
282282
}

‎packages/serverless/src/gcpfunction/general.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -62,4 +62,4 @@ export function configureScopeWithContext(scope: Scope, context: Context): void
6262
scope.setContext('gcp.function.context', { ...context } as SentryContext);
6363
}
6464

65-
export { Request, Response };
65+
export type { Request, Response };

‎packages/tracing/src/browser/index.ts

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,4 @@
1+
export type { RequestInstrumentationOptions } from './request';
2+
13
export { BrowserTracing } from './browsertracing';
2-
export {
3-
instrumentOutgoingRequests,
4-
RequestInstrumentationOptions,
5-
defaultRequestInstrumentationOptions,
6-
} from './request';
4+
export { instrumentOutgoingRequests, defaultRequestInstrumentationOptions } from './request';

0 commit comments

Comments
 (0)