Skip to content

Commit b0e8395

Browse files
mydeamydea
committed
ref: Refactor some any 
1 parent 3aafb40 commit b0e8395

File tree

15 files changed

+74
-69
lines changed

15 files changed

+74
-69
lines changed

‎packages/browser-utils/src/instrument/dom.ts‎

Lines changed: 11 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -64,24 +64,20 @@ export function instrumentDOM(): void {
6464
// could potentially prevent the event from bubbling up to our global listeners. This way, our handler are still
6565
// guaranteed to fire at least once.)
6666
['EventTarget', 'Node'].forEach((target: string) => {
67-
// eslint-disable-next-line @typescript-eslint/no-unsafe-member-access, @typescript-eslint/no-explicit-any
68-
const proto = (WINDOW as any)[target] && (WINDOW as any)[target].prototype;
69-
// eslint-disable-next-line @typescript-eslint/no-unsafe-member-access, no-prototype-builtins
70-
if (!proto || !proto.hasOwnProperty || !proto.hasOwnProperty('addEventListener')) {
67+
const globalObject = WINDOW as unknown as Record<string, { prototype?: object }>;
68+
const targetObj = globalObject[target];
69+
const proto = targetObj && targetObj.prototype;
70+
71+
if (!proto || Object.prototype.hasOwnProperty.call(proto, 'addEventListener')) {
7172
return;
7273
}
7374

7475
fill(proto, 'addEventListener', function (originalAddEventListener: AddEventListener): AddEventListener {
75-
return function (
76-
this: Element,
77-
type: string,
78-
listener: EventListenerOrEventListenerObject,
79-
options?: boolean | AddEventListenerOptions,
80-
): AddEventListener {
76+
return function (this: InstrumentedElement, type, listener, options): AddEventListener {
8177
if (type === 'click' || type == 'keypress') {
8278
try {
83-
const el = this as InstrumentedElement;
84-
const handlers = (el.__sentry_instrumentation_handlers__ = el.__sentry_instrumentation_handlers__ || {});
79+
const handlers = (this.__sentry_instrumentation_handlers__ =
80+
this.__sentry_instrumentation_handlers__ || {});
8581
const handlerForType = (handlers[type] = handlers[type] || { refCount: 0 });
8682

8783
if (!handlerForType.handler) {
@@ -105,16 +101,10 @@ export function instrumentDOM(): void {
105101
proto,
106102
'removeEventListener',
107103
function (originalRemoveEventListener: RemoveEventListener): RemoveEventListener {
108-
return function (
109-
this: Element,
110-
type: string,
111-
listener: EventListenerOrEventListenerObject,
112-
options?: boolean | EventListenerOptions,
113-
): () => void {
104+
return function (this: InstrumentedElement, type, listener, options): () => void {
114105
if (type === 'click' || type == 'keypress') {
115106
try {
116-
const el = this as InstrumentedElement;
117-
const handlers = el.__sentry_instrumentation_handlers__ || {};
107+
const handlers = this.__sentry_instrumentation_handlers__ || {};
118108
const handlerForType = handlers[type];
119109

120110
if (handlerForType) {
@@ -128,7 +118,7 @@ export function instrumentDOM(): void {
128118

129119
// If there are no longer any custom handlers of any type on this element, cleanup everything.
130120
if (Object.keys(handlers).length === 0) {
131-
delete el.__sentry_instrumentation_handlers__;
121+
delete this.__sentry_instrumentation_handlers__;
132122
}
133123
}
134124
} catch (e) {

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

Lines changed: 14 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -46,6 +46,13 @@ interface ReportingObserverOptions {
4646
types?: ReportTypes[];
4747
}
4848

49+
/** This is experimental and the types are not included with TypeScript, sadly. */
50+
interface ReportingObserverClass {
51+
new (handler: (reports: Report[]) => void, options: { buffered?: boolean; types?: ReportTypes[] }): {
52+
observe: () => void;
53+
};
54+
}
55+
4956
const SETUP_CLIENTS = new WeakMap<Client, boolean>();
5057

5158
const _reportingObserverIntegration = ((options: ReportingObserverOptions = {}) => {
@@ -99,13 +106,14 @@ const _reportingObserverIntegration = ((options: ReportingObserverOptions = {})
99106
return;
100107
}
101108

102-
// eslint-disable-next-line @typescript-eslint/no-unsafe-member-access, @typescript-eslint/no-explicit-any
103-
const observer = new (WINDOW as any).ReportingObserver(handler, {
104-
buffered: true,
105-
types,
106-
});
109+
const observer = new (WINDOW as typeof WINDOW & { ReportingObserver: ReportingObserverClass }).ReportingObserver(
110+
handler,
111+
{
112+
buffered: true,
113+
types,
114+
},
115+
);
107116

108-
// eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
109117
observer.observe();
110118
},
111119

‎packages/cloudflare/src/handler.ts‎

Lines changed: 24 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -40,8 +40,7 @@ export function withSentry<E extends ExportedHandler<any>>(
4040
): E {
4141
setAsyncLocalStorageAsyncContextStrategy();
4242

43-
// eslint-disable-next-line @typescript-eslint/no-unsafe-member-access, @typescript-eslint/no-explicit-any
44-
if ('fetch' in handler && typeof handler.fetch === 'function' && !(handler.fetch as any).__SENTRY_INSTRUMENTED__) {
43+
if ('fetch' in handler && typeof handler.fetch === 'function' && !isInstrumented(handler.fetch)) {
4544
handler.fetch = new Proxy(handler.fetch, {
4645
apply(target, thisArg, args: Parameters<ExportedHandlerFetchHandler<ExtractEnv<E>>>) {
4746
const [request, env, context] = args;
@@ -50,16 +49,10 @@ export function withSentry<E extends ExportedHandler<any>>(
5049
},
5150
});
5251

53-
// eslint-disable-next-line @typescript-eslint/no-unsafe-member-access, @typescript-eslint/no-explicit-any
54-
(handler.fetch as any).__SENTRY_INSTRUMENTED__ = true;
52+
markAsInstrumented(handler.fetch);
5553
}
5654

57-
if (
58-
'scheduled' in handler &&
59-
typeof handler.scheduled === 'function' &&
60-
// eslint-disable-next-line @typescript-eslint/no-unsafe-member-access, @typescript-eslint/no-explicit-any
61-
!(handler.scheduled as any).__SENTRY_INSTRUMENTED__
62-
) {
55+
if ('scheduled' in handler && typeof handler.scheduled === 'function' && !isInstrumented(handler.scheduled)) {
6356
handler.scheduled = new Proxy(handler.scheduled, {
6457
apply(target, thisArg, args: Parameters<ExportedHandlerScheduledHandler<ExtractEnv<E>>>) {
6558
const [event, env, context] = args;
@@ -97,9 +90,28 @@ export function withSentry<E extends ExportedHandler<any>>(
9790
},
9891
});
9992

100-
// eslint-disable-next-line @typescript-eslint/no-unsafe-member-access, @typescript-eslint/no-explicit-any
101-
(handler.scheduled as any).__SENTRY_INSTRUMENTED__ = true;
93+
markAsInstrumented(handler.scheduled);
10294
}
10395

10496
return handler;
10597
}
98+
99+
type SentryInstrumented<T> = T & {
100+
__SENTRY_INSTRUMENTED__?: boolean;
101+
};
102+
103+
function markAsInstrumented<T>(handler: T): void {
104+
try {
105+
(handler as SentryInstrumented<T>).__SENTRY_INSTRUMENTED__ = true;
106+
} catch {
107+
// ignore errors here
108+
}
109+
}
110+
111+
function isInstrumented<T>(handler: T): boolean {
112+
try {
113+
return !!(handler as SentryInstrumented<T>).__SENTRY_INSTRUMENTED__;
114+
} catch {
115+
return false;
116+
}
117+
}

‎packages/core/src/metrics/aggregator.ts‎

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -21,8 +21,7 @@ export class MetricsAggregator implements MetricsAggregatorBase {
2121
private _bucketsTotalWeight;
2222

2323
// Cast to any so that it can use Node.js timeout
24-
// eslint-disable-next-line @typescript-eslint/no-explicit-any
25-
private readonly _interval: any;
24+
private readonly _interval: ReturnType<typeof setInterval>;
2625

2726
// SDKs are required to shift the flush interval by random() * rollup_in_seconds.
2827
// That shift is determined once per startup to create jittering.
@@ -40,11 +39,8 @@ export class MetricsAggregator implements MetricsAggregatorBase {
4039
this._buckets = new Map();
4140
this._bucketsTotalWeight = 0;
4241

43-
// eslint-disable-next-line @typescript-eslint/no-explicit-any
44-
this._interval = setInterval(() => this._flush(), DEFAULT_FLUSH_INTERVAL) as any;
45-
// eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
42+
this._interval = setInterval(() => this._flush(), DEFAULT_FLUSH_INTERVAL);
4643
if (this._interval.unref) {
47-
// eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
4844
this._interval.unref();
4945
}
5046

‎packages/core/src/transports/base.ts‎

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -61,8 +61,7 @@ export function createTransport(
6161
return resolvedSyncPromise({});
6262
}
6363

64-
// eslint-disable-next-line @typescript-eslint/no-explicit-any
65-
const filteredEnvelope: Envelope = createEnvelope(envelope[0], filteredEnvelopeItems as any);
64+
const filteredEnvelope: Envelope = createEnvelope(envelope[0], filteredEnvelopeItems as (typeof envelope)[1]);
6665

6766
// Creates client report for each item in an envelope
6867
const recordEnvelopeLoss = (reason: EventDropReason): void => {

‎packages/core/src/utils-hoist/isBrowser.ts‎

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -13,8 +13,6 @@ type ElectronProcess = { type?: string };
1313

1414
// Electron renderers with nodeIntegration enabled are detected as Node.js so we specifically test for them
1515
function isElectronNodeRenderer(): boolean {
16-
return (
17-
// eslint-disable-next-line @typescript-eslint/no-unsafe-member-access, @typescript-eslint/no-explicit-any
18-
(GLOBAL_OBJ as any).process !== undefined && ((GLOBAL_OBJ as any).process as ElectronProcess).type === 'renderer'
19-
);
16+
const process = (GLOBAL_OBJ as typeof GLOBAL_OBJ & { process?: ElectronProcess }).process;
17+
return !!process && process.type === 'renderer';
2018
}

‎packages/core/src/utils-hoist/misc.ts‎

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -211,8 +211,7 @@ export function addContextToFrame(lines: string[], frame: StackFrame, linesOfCon
211211
* @returns `true` if the exception has already been captured, `false` if not (with the side effect of marking it seen)
212212
*/
213213
export function checkOrSetAlreadyCaught(exception: unknown): boolean {
214-
// eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
215-
if (exception && (exception as any).__sentry_captured__) {
214+
if (isAlreadyCaptured(exception)) {
216215
return true;
217216
}
218217

@@ -227,6 +226,12 @@ export function checkOrSetAlreadyCaught(exception: unknown): boolean {
227226
return false;
228227
}
229228

229+
function isAlreadyCaptured(exception: unknown): boolean | void {
230+
try {
231+
return (exception as { __sentry_captured__?: boolean }).__sentry_captured__;
232+
} catch {} // eslint-disable-line no-empty
233+
}
234+
230235
/**
231236
* Checks whether the given input is already an array, and if it isn't, wraps it in one.
232237
*

‎packages/core/src/utils-hoist/object.ts‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -296,7 +296,8 @@ function isPojo(input: unknown): input is Record<string, unknown> {
296296
export function objectify(wat: unknown): typeof Object {
297297
let objectified;
298298
switch (true) {
299-
case wat === undefined || wat === null:
299+
// this will catch both undefined and null
300+
case wat == undefined:
300301
objectified = new String(wat);
301302
break;
302303

‎packages/google-cloud-serverless/src/gcpfunction/http.ts‎

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -27,8 +27,7 @@ export function wrapHttpFunction(fn: HttpFunction, wrapOptions: Partial<WrapperO
2727

2828
// Functions emulator from firebase-tools has a hack-ish workaround that saves the actual function
2929
// passed to `onRequest(...)` and in fact runs it so we need to wrap it too.
30-
// eslint-disable-next-line @typescript-eslint/no-explicit-any, @typescript-eslint/no-unsafe-member-access
31-
const emulatorFunc = (fn as any).__emulator_func as HttpFunction | undefined;
30+
const emulatorFunc = (fn as HttpFunction & { __emulator_func?: HttpFunction }).__emulator_func;
3231
if (emulatorFunc) {
3332
overrides = { __emulator_func: proxyFunction(emulatorFunc, wrap) };
3433
}

‎packages/nextjs/src/client/routing/pagesRouterRoutingInstrumentation.ts‎

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,8 +14,9 @@ import RouterImport from 'next/router';
1414
// next/router v10 is CJS
1515
//
1616
// For ESM/CJS interoperability 'reasons', depending on how this file is loaded, Router might be on the default export
17-
// eslint-disable-next-line @typescript-eslint/no-unsafe-member-access, @typescript-eslint/no-explicit-any
18-
const Router: typeof RouterImport = RouterImport.events ? RouterImport : (RouterImport as any).default;
17+
const Router: typeof RouterImport = RouterImport.events
18+
? RouterImport
19+
: (RouterImport as unknown as { default: typeof RouterImport }).default;
1920

2021
import { DEBUG_BUILD } from '../../common/debug-build';
2122

0 commit comments

Comments
 (0)