chore: replace several instances of any (#2264)

* chore: replace several instances of any

* fix: make sure build works

Author: SebastiaanNijland

packages/browser/src/helpers.ts

@@ -1,5 +1,5 @@
 import { captureException, getCurrentHub, withScope } from '@sentry/core';
-import { Event as SentryEvent, Mechanism, WrappedFunction } from '@sentry/types';
+import { Event as SentryEvent, Mechanism, Scope, WrappedFunction } from '@sentry/types';
 import { addExceptionMechanism, addExceptionTypeValue, htmlTreeAsString, normalize } from '@sentry/utils';
 
 const debounceDuration: number = 1000;
@@ -92,7 +92,7 @@ export function wrap(
     } catch (ex) {
       ignoreNextOnError();
 
-      withScope(scope => {
+      withScope((scope: Scope) => {
         scope.addEventProcessor((event: SentryEvent) => {
           const processedEvent = { ...event };
 

packages/browser/src/integrations/breadcrumbs.ts

@@ -468,7 +468,7 @@ export class Breadcrumbs implements Integration {
           });
 
           if ('onreadystatechange' in xhr && typeof xhr.onreadystatechange === 'function') {
-            fill(xhr, 'onreadystatechange', function(original: () => void): void {
+            fill(xhr, 'onreadystatechange', function(original: () => void): Function {
               return wrap(
                 original,
                 {

packages/browser/src/sdk.ts

@@ -163,6 +163,5 @@ export function close(timeout?: number): PromiseLike<boolean> {
  * @returns The result of wrapped function call.
  */
 export function wrap(fn: Function): any {
-  // tslint:disable-next-line: no-unsafe-any
   return internalWrap(fn)();
 }

packages/core/src/baseclient.ts

@@ -359,7 +359,8 @@ export abstract class BaseClient<B extends Backend, O extends Options> implement
             }
 
             const beforeSendResult = beforeSend(prepared, hint);
-            if ((typeof beforeSendResult as any) === 'undefined') {
+            // tslint:disable-next-line:strict-type-predicates
+            if (typeof beforeSendResult === 'undefined') {
               logger.error('`beforeSend` method has to return `null` or a valid event.');
             } else if (isThenable(beforeSendResult)) {
               this._handleAsyncBeforeSend(beforeSendResult as PromiseLike<Event | null>, resolve, reject);

packages/core/src/integrations/inboundfilters.ts

@@ -87,8 +87,7 @@ export class InboundFilters implements Integration {
     }
 
     try {
-      // tslint:disable-next-line:no-unsafe-any
-      return (event as any).exception.values[0].type === 'SentryError';
+      return event && event.exception && event.exception.values && event.exception.values[0] && event.exception.values[0].type === 'SentryError' || false;
     } catch (_oO) {
       return false;
     }
@@ -147,8 +146,7 @@ export class InboundFilters implements Integration {
     }
     if (event.exception) {
       try {
-        // tslint:disable-next-line:no-unsafe-any
-        const { type, value } = (event as any).exception.values[0];
+        const { type = '', value = '' } = event.exception.values && event.exception.values[0] || {};
         return [`${value}`, `${type}: ${value}`];
       } catch (oO) {
         logger.error(`Cannot extract message for event ${getEventDescription(event)}`);
@@ -162,14 +160,12 @@ export class InboundFilters implements Integration {
   private _getEventFilterUrl(event: Event): string | null {
     try {
       if (event.stacktrace) {
-        // tslint:disable:no-unsafe-any
-        const frames = (event as any).stacktrace.frames;
-        return frames[frames.length - 1].filename;
+        const frames = event.stacktrace.frames;
+        return frames && frames[frames.length - 1].filename || null;
       }
       if (event.exception) {
-        // tslint:disable:no-unsafe-any
-        const frames = (event as any).exception.values[0].stacktrace.frames;
-        return frames[frames.length - 1].filename;
+        const frames = event.exception.values && event.exception.values[0].stacktrace && event.exception.values[0].stacktrace.frames;
+        return frames && frames[frames.length - 1].filename || null;
       }
       return null;
     } catch (oO) {

packages/integrations/src/rewriteframes.ts

@@ -74,8 +74,7 @@ export class RewriteFrames implements Integration {
 
     if (exception) {
       try {
-        // tslint:disable-next-line:no-unsafe-any
-        return (exception as any).values[0].stacktrace.frames;
+        return exception.values && exception.values[0].stacktrace && exception.values[0].stacktrace.frames;
       } catch (_oO) {
         return undefined;
       }