move log statement to calling functions

Author: lobsterkatie

packages/core/src/baseclient.ts

@@ -30,6 +30,8 @@ import {
 import { Backend, BackendClass } from './basebackend';
 import { IntegrationIndex, setupIntegrations } from './integration';
 
+const ALREADY_SEEN_ERROR = "Not capturing exception because it's already been captured.";
+
 /**
  * Base implementation for all JavaScript SDK clients.
  *
@@ -104,6 +106,7 @@ export abstract class BaseClient<B extends Backend, O extends Options> implement
   public captureException(exception: any, hint?: EventHint, scope?: Scope): string | undefined {
     // ensure we haven't captured this very object before
     if (checkOrSetAlreadyCaught(exception)) {
+      logger.log(ALREADY_SEEN_ERROR);
       return;
     }
 
@@ -148,6 +151,7 @@ export abstract class BaseClient<B extends Backend, O extends Options> implement
   public captureEvent(event: Event, hint?: EventHint, scope?: Scope): string | undefined {
     // ensure we haven't captured this very object before
     if (hint?.originalException && checkOrSetAlreadyCaught(hint.originalException)) {
+      logger.log(ALREADY_SEEN_ERROR);
       return;
     }
 

packages/utils/src/misc.ts

@@ -2,7 +2,6 @@
 import { Event, Mechanism, StackFrame } from '@sentry/types';
 
 import { getGlobalObject } from './global';
-import { logger } from './logger';
 import { snipLine } from './string';
 
 /**
@@ -260,7 +259,6 @@ export function stripUrlQueryAndFragment(urlPath: string): string {
 export function checkOrSetAlreadyCaught(exception: unknown): boolean {
   // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
   if ((exception as any)?.__sentry_captured__) {
-    logger.log("Not capturing exception because it's already been captured.");
     return true;
   }