Add type ignores for Reflect instead

Author: andreiborza

packages/nestjs/src/decorators.ts

@@ -1,4 +1,3 @@
-import 'reflect-metadata';
 import type { MonitorConfig } from '@sentry/core';
 import { captureException, isThenable } from '@sentry/core';
 import * as Sentry from '@sentry/node';
@@ -111,10 +110,18 @@ function copyFunctionNameAndMetadata({
   });
 
   // copy metadata
+  // eslint-disable-next-line @typescript-eslint/ban-ts-comment
+  // @ts-ignore - reflect-metadata of nestjs adds these methods to Reflect
   if (typeof Reflect !== 'undefined' && typeof Reflect.getMetadataKeys === 'function') {
+    // eslint-disable-next-line @typescript-eslint/ban-ts-comment
+    // @ts-ignore - reflect-metadata of nestjs adds these methods to Reflect
     const originalMetaData = Reflect.getMetadataKeys(originalMethod);
     for (const key of originalMetaData) {
+      // eslint-disable-next-line @typescript-eslint/ban-ts-comment
+      // @ts-ignore - reflect-metadata of nestjs adds these methods to Reflect
       const value = Reflect.getMetadata(key, originalMethod);
+      // eslint-disable-next-line @typescript-eslint/ban-ts-comment
+      // @ts-ignore - reflect-metadata of nestjs adds these methods to Reflect
       Reflect.defineMetadata(key, value, descriptor.value);
     }
   }

packages/nestjs/src/integrations/sentry-nest-event-instrumentation.ts

@@ -1,4 +1,3 @@
-import 'reflect-metadata';
 import type { InstrumentationConfig } from '@opentelemetry/instrumentation';
 import {
   InstrumentationBase,
@@ -84,7 +83,11 @@ export class SentryNestEventInstrumentation extends InstrumentationBase {
           descriptor.value = async function (...args: unknown[]) {
             // When multiple @OnEvent decorators are used on a single method, we need to get all event names
             // from the reflector metadata as there is no information during execution which event triggered it
+            // eslint-disable-next-line @typescript-eslint/ban-ts-comment
+            // @ts-ignore - reflect-metadata of nestjs adds these methods to Reflect
             if (Reflect.getMetadataKeys(descriptor.value).includes('EVENT_LISTENER_METADATA')) {
+              // eslint-disable-next-line @typescript-eslint/ban-ts-comment
+              // @ts-ignore - reflect-metadata of nestjs adds these methods to Reflect
               const eventData = Reflect.getMetadata('EVENT_LISTENER_METADATA', descriptor.value);
               if (Array.isArray(eventData)) {
                 eventName = eventData