Event processors

Author: timfish

packages/integrations/src/extraerrordata.ts

@@ -37,7 +37,7 @@ export class ExtraErrorData implements Integration {
    * @inheritDoc
    */
   public setupOnce(addGlobalEventProcessor: (callback: EventProcessor) => void, getCurrentHub: () => Hub): void {
-    addGlobalEventProcessor((event: Event, hint?: EventHint) => {
+    addGlobalEventProcessor((event: Event, hint: EventHint) => {
       const self = getCurrentHub().getIntegration(ExtraErrorData);
       if (!self) {
         return event;
@@ -49,8 +49,8 @@ export class ExtraErrorData implements Integration {
   /**
    * Attaches extracted information from the Error object to extra field in the Event
    */
-  public enhanceEventWithErrorData(event: Event, hint?: EventHint): Event {
-    if (!hint || !hint.originalException || !isError(hint.originalException)) {
+  public enhanceEventWithErrorData(event: Event, hint: EventHint): Event {
+    if (!hint.originalException || !isError(hint.originalException)) {
       return event;
     }
     const exceptionName = (hint.originalException as ExtendedError).name || hint.originalException.constructor.name;

packages/node/src/integrations/linkederrors.ts

@@ -43,7 +43,7 @@ export class LinkedErrors implements Integration {
    * @inheritDoc
    */
   public setupOnce(): void {
-    addGlobalEventProcessor(async (event: Event, hint?: EventHint) => {
+    addGlobalEventProcessor(async (event: Event, hint: EventHint) => {
       const hub = getCurrentHub();
       const self = hub.getIntegration(LinkedErrors);
       const client = hub.getClient<NodeClient>();
@@ -57,8 +57,8 @@ export class LinkedErrors implements Integration {
   /**
    * @inheritDoc
    */
-  private _handler(stackParser: StackParser, event: Event, hint?: EventHint): PromiseLike<Event> {
-    if (!event.exception || !event.exception.values || !hint || !isInstanceOf(hint.originalException, Error)) {
+  private _handler(stackParser: StackParser, event: Event, hint: EventHint): PromiseLike<Event> {
+    if (!event.exception || !event.exception.values || !isInstanceOf(hint.originalException, Error)) {
       return resolvedSyncPromise(event);
     }
 

packages/types/src/eventprocessor.ts

@@ -8,5 +8,5 @@ import { Event, EventHint } from './event';
  */
 export interface EventProcessor {
   id?: string; // This field can't be named "name" because functions already have this field natively
-  (event: Event, hint?: EventHint): PromiseLike<Event | null> | Event | null;
+  (event: Event, hint: EventHint): PromiseLike<Event | null> | Event | null;
 }