feat(nestjs): Switch to OTel core instrumentation (#17068)

OTel's instrumentation supports v11 since
[0.46.0](https://github.com/open-telemetry/opentelemetry-js-contrib/blob/de2260023b12c395e6f85592d29113b718a30f63/packages/instrumentation-nestjs-core/CHANGELOG.md?plain=1#L31-L36).

Author: andreiborza

dev-packages/e2e-tests/test-applications/nestjs-fastify/tests/transactions.test.ts

@@ -97,7 +97,7 @@ test('Sends an API route transaction', async ({ baseURL }) => {
             component: '@nestjs/core',
             'nestjs.version': expect.any(String),
             'nestjs.type': 'request_context',
-            'http.request.method': 'GET',
+            'http.method': 'GET',
             'http.url': '/test-transaction',
             'http.route': '/test-transaction',
             'nestjs.controller': 'AppController',

packages/nestjs/src/decorators.ts

@@ -110,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/nest.ts

@@ -1,13 +1,13 @@
+import { NestInstrumentation as NestInstrumentationCore } from '@opentelemetry/instrumentation-nestjs-core';
 import { defineIntegration } from '@sentry/core';
 import { generateInstrumentOnce } from '@sentry/node';
-import { NestInstrumentation } from './sentry-nest-core-instrumentation';
 import { SentryNestEventInstrumentation } from './sentry-nest-event-instrumentation';
 import { SentryNestInstrumentation } from './sentry-nest-instrumentation';
 
 const INTEGRATION_NAME = 'Nest';
 
 const instrumentNestCore = generateInstrumentOnce('Nest-Core', () => {
-  return new NestInstrumentation();
+  return new NestInstrumentationCore();
 });
 
 const instrumentNestCommon = generateInstrumentOnce('Nest-Common', () => {

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

@@ -83,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