refactor: remove unused noop

Author: logaretm

packages/nitro/src/runtime/hooks/captureStorageEvents.ts

@@ -77,9 +77,6 @@ function setupStorageTracingChannel(operation: TracedOperation): void {
   });
 
   channel.subscribe({
-    start() {},
-    asyncStart() {},
-    end() {},
     asyncEnd(data: TracingChannelContextWithSpan<TraceContext & { result?: unknown }>) {
       if (data._sentrySpan && CACHE_HIT_OPERATIONS.has(operation)) {
         data._sentrySpan.setAttribute(SEMANTIC_ATTRIBUTE_CACHE_HIT, isCacheHit(data.keys?.[0], data.result));

packages/nitro/src/runtime/hooks/captureTracingEvents.ts

@@ -41,11 +41,6 @@ export function captureTracingEvents(): void {
   globalWithTraceChannels.__SENTRY_NITRO_HTTP_CHANNELS_INSTRUMENTED__ = true;
 }
 
-/**
- * No-op function to satisfy the tracing channel subscribe callbacks
- */
-const NOOP = (): void => {};
-
 /**
  * Extracts the HTTP status code from a tracing channel result.
  * The result is the return value of the traced handler, which is a Response for srvx
@@ -126,8 +121,6 @@ function setupH3TracingChannels(): void {
     start: (data: H3TracingRequestEvent) => {
       setServerTimingHeaders(data.event);
     },
-    asyncStart: NOOP,
-    end: NOOP,
     asyncEnd: (data: TracingChannelContextWithSpan<H3TracingRequestEvent>) => {
       onTraceEnd(data);
 
@@ -192,9 +185,6 @@ function setupSrvxTracingChannels(): void {
 
   // Subscribe to events (span already created in bindStore)
   fetchChannel.subscribe({
-    start: () => {},
-    asyncStart: () => {},
-    end: () => {},
     asyncEnd: data => {
       onTraceEnd(data);
 
@@ -239,9 +229,6 @@ function setupSrvxTracingChannels(): void {
 
   // Subscribe to events (span already created in bindStore)
   middlewareChannel.subscribe({
-    start: () => {},
-    asyncStart: () => {},
-    end: () => {},
     asyncEnd: onTraceEnd,
     error: onTraceError,
   });