Address review comments

Author: onurtemizkan

packages/node/package.json

@@ -65,7 +65,7 @@
     "access": "public"
   },
   "dependencies": {
-    "@fastify/otel": "0.5.0",
+    "@fastify/otel": "0.5.1",
     "@opentelemetry/api": "^1.9.0",
     "@opentelemetry/context-async-hooks": "^1.30.1",
     "@opentelemetry/core": "^1.30.1",

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

@@ -16,41 +16,6 @@ import * as diagnosticsChannel from 'node:diagnostics_channel';
 import { DEBUG_BUILD } from '../../../debug-build';
 import type { FastifyInstance, FastifyReply, FastifyRequest } from './types';
 
-// /**
-//  * Minimal request type containing properties around route information.
-//  * Works for Fastify 3, 4 and presumably 5.
-//  *
-//  * Based on https://github.com/fastify/fastify/blob/ce3811f5f718be278bbcd4392c615d64230065a6/types/request.d.ts
-//  */
-// // eslint-disable-next-line @typescript-eslint/no-explicit-any
-// interface MinimalFastifyRequest extends Record<string, any> {
-//   method?: string;
-//   // since [email protected]
-//   routeOptions?: {
-//     url?: string;
-//   };
-//   routerPath?: string;
-// }
-
-// /**
-//  * Minimal reply type containing properties needed for error handling.
-//  *
-//  * Based on https://github.com/fastify/fastify/blob/ce3811f5f718be278bbcd4392c615d64230065a6/types/reply.d.ts
-//  */
-// // eslint-disable-next-line @typescript-eslint/no-explicit-any
-// interface MinimalFastifyReply extends Record<string, any> {
-//   statusCode: number;
-// }
-
-// // We inline the types we care about here
-// interface Fastify {
-//   // eslint-disable-next-line @typescript-eslint/no-explicit-any
-//   version: string;
-//   register: (plugin: any) => Fastify;
-//   after: (listener?: (err: Error) => void) => Fastify;
-//   addHook: (name: string, handler: (...params: unknown[]) => void) => Fastify;
-// }
-
 interface FastifyHandlerOptions {
   /**
    * Callback method deciding whether error should be captured and sent to Sentry

yarn.lock

@@ -3921,10 +3921,10 @@
   resolved "https://registry.yarnpkg.com/@fastify/busboy/-/busboy-2.0.0.tgz#f22824caff3ae506b18207bad4126dbc6ccdb6b8"
   integrity sha512-JUFJad5lv7jxj926GPgymrWQxxjPYuJNiNjNMzqT+HiuP6Vl3dk5xzG+8sTX96np0ZAluvaMzPsjhHZ5rNuNQQ==
 
-"@fastify/[email protected].0":
-  version "0.5.0"
-  resolved "https://registry.yarnpkg.com/@fastify/otel/-/otel-0.5.0.tgz#1b18e9623768ad6d8a20e95771dacca2783aa0ba"
-  integrity sha512-wZ2yl/2iuW6jerdktR/mpkkI83/qHhbRShQtNA3x+6+bQ/RKqmhifc5M29fmHbyIw+DF3SXJAKMSdURSQt4W0w==
+"@fastify/[email protected].1":
+  version "0.5.1"
+  resolved "https://registry.yarnpkg.com/@fastify/otel/-/otel-0.5.1.tgz#fc8e56fcbdeb693c1a44f2da071817bc600ae1f5"
+  integrity sha512-xDoDFQHRlgZ1Koe9fWzz4MqgrdW6u66bMPy+rOgoWg4cHyGFnmpcnUusKaFOyBsFhGtaH5dCLmLo3gCGoMnxYQ==
   dependencies:
     "@opentelemetry/core" "^1.29.0"
     "@opentelemetry/instrumentation" "^0.57.0"