port captureMessage

Author: AbhiPrasad

packages/hub/src/exports.ts

@@ -1,4 +1,4 @@
-import { CaptureContext, Hub } from '@sentry/types';
+import { CaptureContext, Hub, Severity, SeverityLevel } from '@sentry/types';
 
 import { getCurrentHub } from './hub';
 
@@ -27,3 +27,22 @@ export function callOnHub<T>(method: string, ...args: any[]): T {
 export function captureException(exception: any, captureContext?: CaptureContext): string {
   return getCurrentHub().captureException(exception, { captureContext });
 }
+
+/**
+ * Captures a message event and sends it to Sentry.
+ *
+ * @param message The message to send to Sentry.
+ * @param Severity Define the level of the message.
+ * @returns The generated eventId.
+ */
+export function captureMessage(
+  message: string,
+  // eslint-disable-next-line deprecation/deprecation
+  captureContext?: CaptureContext | Severity | SeverityLevel,
+): string {
+  // This is necessary to provide explicit scopes upgrade, without changing the original
+  // arity of the `captureMessage(message, level)` method.
+  const level = typeof captureContext === 'string' ? captureContext : undefined;
+  const context = typeof captureContext !== 'string' ? { captureContext } : undefined;
+  return getCurrentHub().captureMessage(message, level, context);
+}

packages/hub/src/hub.ts

@@ -211,27 +211,18 @@ export class Hub implements HubInterface {
     hint?: EventHint,
   ): string {
     const eventId = (this._lastEventId = hint && hint.event_id ? hint.event_id : uuid4());
-    let finalHint = hint;
-
-    // If there's no explicit hint provided, mimic the same thing that would happen
-    // in the minimal itself to create a consistent behavior.
-    // We don't do this in the client, as it's the lowest level API, and doing this,
-    // would prevent user from having full control over direct calls.
-    if (!hint) {
-      let syntheticException: Error;
-      try {
-        throw new Error(message);
-      } catch (exception) {
-        syntheticException = exception as Error;
-      }
-      finalHint = {
-        originalException: message,
-        syntheticException,
-      };
+
+    let syntheticException: Error;
+    try {
+      throw new Error('Sentry syntheticException');
+    } catch (exception) {
+      syntheticException = exception as Error;
     }
 
     this._invokeClient('captureMessage', message, level, {
-      ...finalHint,
+      originalException: message,
+      syntheticException,
+      ...hint,
       event_id: eventId,
     });
     return eventId;

packages/minimal/src/index.ts

@@ -29,32 +29,6 @@ function callOnHub<T>(method: string, ...args: any[]): T {
   throw new Error(`No hub defined or ${method} was not found on the hub, please open a bug report.`);
 }
 
-/**
- * Captures a message event and sends it to Sentry.
- *
- * @param message The message to send to Sentry.
- * @param Severity Define the level of the message.
- * @returns The generated eventId.
- */
-export function captureMessage(
-  message: string,
-  // eslint-disable-next-line deprecation/deprecation
-  captureContext?: CaptureContext | Severity | SeverityLevel,
-): string {
-  const syntheticException = new Error(message);
-
-  // This is necessary to provide explicit scopes upgrade, without changing the original
-  // arity of the `captureMessage(message, level)` method.
-  const level = typeof captureContext === 'string' ? captureContext : undefined;
-  const context = typeof captureContext !== 'string' ? { captureContext } : undefined;
-
-  return callOnHub('captureMessage', message, level, {
-    originalException: message,
-    syntheticException,
-    ...context,
-  });
-}
-
 /**
  * Captures a manually created event and sends it to Sentry.
  *