ref: Mangle more internal names (#2036)

* ref: Mangle more internal names

* fix: Linter

Author: HazAT

.prettierignore

@@ -0,0 +1 @@
+*.md

CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## Unreleased
+
+- [browser] ref: Mangle more stuff, reduce bundle size
+
 ## 5.1.1
 
 - [browser] fix: Breadcrumb Integration: Fix #2034

packages/browser/src/backend.ts

@@ -12,7 +12,7 @@ import {
 } from '@sentry/utils';
 
 import { eventFromPlainObject, eventFromStacktrace, prepareFramesForEvent } from './parsers';
-import { computeStackTrace } from './tracekit';
+import { _computeStackTrace } from './tracekit';
 import { FetchTransport, XHRTransport } from './transports';
 
 /**
@@ -72,7 +72,7 @@ export class BrowserBackend extends BaseBackend<BrowserOptions> {
       // If it is an ErrorEvent with `error` property, extract it to get actual Error
       const errorEvent = exception as ErrorEvent;
       exception = errorEvent.error; // tslint:disable-line:no-parameter-reassignment
-      event = eventFromStacktrace(computeStackTrace(exception as Error));
+      event = eventFromStacktrace(_computeStackTrace(exception as Error));
       return SyncPromise.resolve(this._buildEvent(event, hint));
     }
     if (isDOMError(exception as DOMError) || isDOMException(exception as DOMException)) {
@@ -91,7 +91,7 @@ export class BrowserBackend extends BaseBackend<BrowserOptions> {
     }
     if (isError(exception as Error)) {
       // we have a real Error object, do nothing
-      event = eventFromStacktrace(computeStackTrace(exception as Error));
+      event = eventFromStacktrace(_computeStackTrace(exception as Error));
       return SyncPromise.resolve(this._buildEvent(event, hint));
     }
     if (isPlainObject(exception as {}) && hint && hint.syntheticException) {
@@ -148,7 +148,7 @@ export class BrowserBackend extends BaseBackend<BrowserOptions> {
     };
 
     if (this._options.attachStacktrace && hint && hint.syntheticException) {
-      const stacktrace = computeStackTrace(hint.syntheticException);
+      const stacktrace = _computeStackTrace(hint.syntheticException);
       const frames = prepareFramesForEvent(stacktrace.stack);
       event.stacktrace = {
         frames,

packages/browser/src/index.ts

@@ -40,6 +40,7 @@ import * as Transports from './transports';
 
 let windowIntegrations = {};
 
+// This block is needed to add compatibility with the integrations packages when used with a CDN
 // tslint:disable: no-unsafe-any
 const _window = getGlobalObject<Window>() as any;
 if (_window.Sentry && _window.Sentry.Integrations) {

packages/browser/src/integrations/globalhandlers.ts

@@ -4,10 +4,10 @@ import { addExceptionTypeValue, logger, normalize, truncate } from '@sentry/util
 
 import { eventFromStacktrace } from '../parsers';
 import {
-  installGlobalHandler,
-  installGlobalUnhandledRejectionHandler,
+  _installGlobalHandler,
+  _installGlobalUnhandledRejectionHandler,
+  _subscribe,
   StackTrace as TraceKitStackTrace,
-  subscribe,
 } from '../tracekit';
 
 import { shouldIgnoreOnError } from './helpers';
@@ -47,7 +47,7 @@ export class GlobalHandlers implements Integration {
   public setupOnce(): void {
     Error.stackTraceLimit = 50;
 
-    subscribe((stack: TraceKitStackTrace, _: boolean, error: Error) => {
+    _subscribe((stack: TraceKitStackTrace, _: boolean, error: Error) => {
       // TODO: use stack.context to get a valuable information from TraceKit, eg.
       // [
       //   0: "  })"
@@ -76,12 +76,12 @@ export class GlobalHandlers implements Integration {
 
     if (this._options.onerror) {
       logger.log('Global Handler attached: onerror');
-      installGlobalHandler();
+      _installGlobalHandler();
     }
 
     if (this._options.onunhandledrejection) {
       logger.log('Global Handler attached: onunhandledrejection');
-      installGlobalUnhandledRejectionHandler();
+      _installGlobalUnhandledRejectionHandler();
     }
   }
 

packages/browser/src/integrations/linkederrors.ts

@@ -2,7 +2,7 @@ import { addGlobalEventProcessor, getCurrentHub } from '@sentry/core';
 import { Event, EventHint, Exception, ExtendedError, Integration } from '@sentry/types';
 
 import { exceptionFromStacktrace } from '../parsers';
-import { computeStackTrace } from '../tracekit';
+import { _computeStackTrace } from '../tracekit';
 
 const DEFAULT_KEY = 'cause';
 const DEFAULT_LIMIT = 5;
@@ -44,7 +44,7 @@ export class LinkedErrors implements Integration {
     addGlobalEventProcessor((event: Event, hint?: EventHint) => {
       const self = getCurrentHub().getIntegration(LinkedErrors);
       if (self) {
-        return self.handler(event, hint);
+        return self._handler(event, hint);
       }
       return event;
     });
@@ -53,24 +53,24 @@ export class LinkedErrors implements Integration {
   /**
    * @inheritDoc
    */
-  public handler(event: Event, hint?: EventHint): Event | null {
+  private _handler(event: Event, hint?: EventHint): Event | null {
     if (!event.exception || !event.exception.values || !hint || !(hint.originalException instanceof Error)) {
       return event;
     }
-    const linkedErrors = this.walkErrorTree(hint.originalException, this._key);
+    const linkedErrors = this._walkErrorTree(hint.originalException, this._key);
     event.exception.values = [...linkedErrors, ...event.exception.values];
     return event;
   }
 
   /**
    * @inheritDoc
    */
-  public walkErrorTree(error: ExtendedError, key: string, stack: Exception[] = []): Exception[] {
+  private _walkErrorTree(error: ExtendedError, key: string, stack: Exception[] = []): Exception[] {
     if (!(error[key] instanceof Error) || stack.length + 1 >= this._limit) {
       return stack;
     }
-    const stacktrace = computeStackTrace(error[key]);
+    const stacktrace = _computeStackTrace(error[key]);
     const exception = exceptionFromStacktrace(stacktrace);
-    return this.walkErrorTree(error[key], key, [exception, ...stack]);
+    return this._walkErrorTree(error[key], key, [exception, ...stack]);
   }
 }

packages/browser/src/parsers.ts

@@ -1,7 +1,7 @@
 import { Event, Exception, StackFrame } from '@sentry/types';
 import { keysToEventMessage, normalizeToSize } from '@sentry/utils';
 
-import { computeStackTrace, StackFrame as TraceKitStackFrame, StackTrace as TraceKitStackTrace } from './tracekit';
+import { _computeStackTrace, StackFrame as TraceKitStackFrame, StackTrace as TraceKitStackTrace } from './tracekit';
 
 const STACKTRACE_LIMIT = 50;
 
@@ -43,7 +43,7 @@ export function eventFromPlainObject(exception: {}, syntheticException: Error |
   };
 
   if (syntheticException) {
-    const stacktrace = computeStackTrace(syntheticException);
+    const stacktrace = _computeStackTrace(syntheticException);
     const frames = prepareFramesForEvent(stacktrace.stack);
     event.stacktrace = {
       frames,