Remove version check.

Author: onurtemizkan

packages/node/src/integrations/tracing/fastify-v3/internal-types.ts

@@ -21,10 +21,21 @@ export type HookHandlerDoneFunction = <TError extends Error = FastifyError>(err?
 
 export type FastifyErrorCodes = any;
 
-export type FastifyPlugin = any;
+export type FastifyPlugin = (
+  instance: FastifyInstance,
+  opts: any,
+  done: HookHandlerDoneFunction,
+) => unknown | Promise<unknown>;
 
 export interface FastifyInstance {
-  register: (plugin: FastifyPlugin) => void;
+  version: string;
+  register: (
+    plugin: FastifyPlugin,
+    opts?: {
+      throw?: boolean;
+    },
+  ) => FastifyInstance;
+  after: (listener?: (err: Error) => void) => FastifyInstance;
   addHook(
     name:
       | 'onRequest'

packages/node/src/integrations/tracing/fastify.ts

@@ -6,14 +6,15 @@ import {
   defineIntegration,
   getClient,
   getIsolationScope,
-  parseSemver,
+  logger,
   spanToJSON,
 } from '@sentry/core';
 import type { IntegrationFn, Span } from '@sentry/core';
 import { generateInstrumentOnce } from '../../otel/instrument';
 import { FastifyInstrumentationV3 } from './fastify-v3/instrumentation';
 
 import type { FastifyInstance } from './fastify-v3/internal-types';
+import { DEBUG_BUILD } from '../../debug-build';
 /**
  * Minimal request type containing properties around route information.
  * Works for Fastify 3, 4 and 5.
@@ -42,22 +43,10 @@ export const instrumentFastifyV3 = generateInstrumentOnce(
 
 let fastifyOtelInstrumentationInstance: FastifyOtelInstrumentation | undefined;
 
-function checkFastifyVersion(): ReturnType<typeof parseSemver> | undefined {
-  try {
-    // eslint-disable-next-line @typescript-eslint/no-var-requires
-    const pkg = require('fastify/package.json') as { version?: string };
-    return pkg?.version ? parseSemver(pkg.version) : undefined;
-  } catch {
-    return undefined;
-  }
-}
-
 export const instrumentFastify = generateInstrumentOnce(INTEGRATION_NAME, () => {
   // FastifyOtelInstrumentation does not have a `requestHook`
   // so we can't use `addFastifySpanAttributes` here for now
-  fastifyOtelInstrumentationInstance = new FastifyOtelInstrumentation({
-    registerOnInitialization: true,
-  });
+  fastifyOtelInstrumentationInstance = new FastifyOtelInstrumentation();
   return fastifyOtelInstrumentationInstance;
 });
 
@@ -66,11 +55,7 @@ const _fastifyIntegration = (() => {
     name: INTEGRATION_NAME,
     setupOnce() {
       instrumentFastifyV3();
-
-      const fastifyVersion = checkFastifyVersion();
-      if (fastifyVersion?.major && fastifyVersion.major >= 4) {
-        instrumentFastify();
-      }
+      instrumentFastify();
     },
   };
 }) satisfies IntegrationFn;
@@ -151,6 +136,15 @@ export function setupFastifyErrorHandler(fastify: FastifyInstance): void {
     });
   }
 
+  if (fastifyOtelInstrumentationInstance) {
+    // Setting after callback to prevent this plugin from throwing version mismatch errors
+    fastify.register(fastifyOtelInstrumentationInstance.plugin()).after(err => {
+      if (err) {
+        // eslint-disable-next-line no-console
+        DEBUG_BUILD && logger.warn('Failed to register Fastify Otel instrumentation plugin', err);
+      }
+    });
+  }
   // eslint-disable-next-line @typescript-eslint/no-floating-promises
   fastify.register(plugin);
 }