PR feedback

Author: mydea

packages/browser/src/eventbuilder.ts

@@ -290,8 +290,7 @@ function getNonErrorObjectExceptionValue(
 ): string {
   const keys = extractExceptionKeysForMessage(exception);
   const captureType = isUnhandledRejection ? 'promise rejection' : 'exception';
-  const prototype: Prototype | null = Object.getPrototypeOf(exception);
-  const className = prototype ? prototype.constructor.name : undefined;
+  const className = getObjectClassName(exception);
 
   // Some ErrorEvent instances do not have an `error` property, which is why they are not handled before
   // We still want to try to get a decent message for these cases
@@ -309,3 +308,12 @@ function getNonErrorObjectExceptionValue(
 
   return `${label} with keys: ${keys}`;
 }
+
+function getObjectClassName(obj: unknown): string | undefined | void {
+  try {
+    const prototype: Prototype | null = Object.getPrototypeOf(obj);
+    return prototype ? prototype.constructor.name : undefined;
+  } catch (e) {
+    // ignore errors here
+  }
+}

packages/utils/src/is.ts

@@ -41,7 +41,7 @@ function isBuiltin(wat: unknown, className: string): boolean {
  * @param wat A value to be checked.
  * @returns A boolean representing the result.
  */
-export function isErrorEvent(wat: unknown): wat is ErrorEvent {
+export function isErrorEvent(wat: unknown): boolean {
   return isBuiltin(wat, 'ErrorEvent');
 }