feat: Capture wrapped function arguments as extra

Author: kamilogorek

CHANGELOG.md

@@ -4,6 +4,11 @@
 
 - [browser] ref: Expose `ReportDialogOptions`
 - [browser] ref: Use better default message for ReportingObserver
+- [browser] feat: Capture wrapped function arguments as extra
+- [browser] ref: Unify integrations options and set proper defaults
+- [browser] fix: Array.from is not available in old mobile browsers
+- [browser] fix: Check for anonymous function before getting its name for mechanism
+- [browser] test: Add loader + integration tests
 
 ## 4.0.4
 

packages/browser/src/integrations/helpers.ts

@@ -2,6 +2,7 @@ import { getCurrentHub, withScope } from '@sentry/core';
 import { Mechanism, SentryEvent, SentryWrappedFunction } from '@sentry/types';
 import { isFunction } from '@sentry/utils/is';
 import { htmlTreeAsString } from '@sentry/utils/misc';
+import { serializeObject } from '@sentry/utils/object';
 
 const debounceDuration: number = 1000;
 let keypressTimeout: number | undefined;
@@ -61,12 +62,14 @@ export function wrap(
       before.apply(this, arguments);
     }
 
+    const args = Array.prototype.slice.call(arguments);
+
     try {
       // Attempt to invoke user-land function
       // NOTE: If you are a Sentry user, and you are seeing this stack frame, it
       //       means Raven caught an error invoking your application code. This is
       //       expected behavior and NOT indicative of a bug with Raven.js.
-      const wrappedArguments = Array.prototype.slice.call(arguments).map((arg: any) => wrap(arg, options));
+      const wrappedArguments = args.map((arg: any) => wrap(arg, options));
 
       if (fn.handleEvent) {
         return fn.handleEvent.apply(this, wrappedArguments);
@@ -85,6 +88,11 @@ export function wrap(
             processedEvent.exception.mechanism = options.mechanism;
           }
 
+          processedEvent.extra = {
+            ...processedEvent.extra,
+            arguments: serializeObject(args, 2),
+          };
+
           return processedEvent;
         });
 

packages/utils/src/object.ts

@@ -232,7 +232,7 @@ function serializeValue<T>(value: T): T | string {
 }
 
 /** JSDoc */
-function serializeObject<T>(value: T, depth: number): T | string | {} {
+export function serializeObject<T>(value: T, depth: number): T | string | {} {
   if (depth === 0) {
     return serializeValue(value);
   }