fix: Use addExceptionTypeValue in helpers

Author: HazAT

packages/browser/src/integrations/globalhandlers.ts

@@ -103,10 +103,6 @@ export class GlobalHandlers implements Integration {
       data.name = stacktrace.name;
     }
 
-    const newEvent: Event = {
-      ...event,
-    };
-
     const client = getCurrentHub().getClient();
     const maxValueLength = (client && client.getOptions().maxValueLength) || 250;
 
@@ -116,12 +112,12 @@ export class GlobalHandlers implements Integration {
     const fallbackType = stacktrace.mechanism === 'onunhandledrejection' ? 'UnhandledRejection' : 'Error';
 
     // This makes sure we have type/value in every exception
-    addExceptionTypeValue(newEvent, fallbackValue, fallbackType, {
+    addExceptionTypeValue(event, fallbackValue, fallbackType, {
       data,
       handled: false,
       type: stacktrace.mechanism,
     });
 
-    return newEvent;
+    return event;
   }
 }

packages/browser/src/integrations/helpers.ts

@@ -1,6 +1,6 @@
 import { captureException, getCurrentHub, withScope } from '@sentry/core';
 import { Event as SentryEvent, Mechanism, WrappedFunction } from '@sentry/types';
-import { htmlTreeAsString } from '@sentry/utils/misc';
+import { addExceptionTypeValue, htmlTreeAsString } from '@sentry/utils/misc';
 import { normalize } from '@sentry/utils/object';
 
 const debounceDuration: number = 1000;
@@ -91,11 +91,7 @@ export function wrap(
           const processedEvent = { ...event };
 
           if (options.mechanism) {
-            processedEvent.exception = processedEvent.exception || {};
-            processedEvent.exception.values = processedEvent.exception.values || [];
-            processedEvent.exception.values[0] = processedEvent.exception.values[0] || {};
-            processedEvent.exception.values[0].mechanism =
-              processedEvent.exception.values[0].mechanism || options.mechanism;
+            addExceptionTypeValue(processedEvent, undefined, undefined, options.mechanism);
           }
 
           processedEvent.extra = {

packages/node/src/backend.ts

@@ -1,5 +1,5 @@
 import { BaseBackend, Dsn, getCurrentHub } from '@sentry/core';
-import { Event, EventHint, Options, Mechanism, Severity, Transport } from '@sentry/types';
+import { Event, EventHint, Mechanism, Options, Severity, Transport } from '@sentry/types';
 import { isError, isPlainObject } from '@sentry/utils/is';
 import { addExceptionTypeValue } from '@sentry/utils/misc';
 import { normalizeToSize } from '@sentry/utils/object';
@@ -109,17 +109,11 @@ export class NodeBackend extends BaseBackend<NodeOptions> {
 
     return new SyncPromise<Event>(resolve =>
       parseError(ex as Error, this.options).then(event => {
-        const newEvent = event;
-        addExceptionTypeValue(
-          {
-            ...newEvent,
-            event_id: hint && hint.event_id,
-          },
-          undefined,
-          undefined,
-          mechanism,
-        );
-        resolve(newEvent);
+        addExceptionTypeValue(event, undefined, undefined, mechanism);
+        resolve({
+          ...event,
+          event_id: hint && hint.event_id,
+        });
       }),
     );
   }