chore(comments): correct hook comments and remove unnecessary ones

Author: giacomoquinalia

packages/instrumentation-nats/src/instrumentation.ts

@@ -62,9 +62,6 @@ interface SentMessagesAttributes extends Attributes {
   [ATTR_MESSAGING_OPERATION_NAME]: string;
 }
 
-/**
- * Attributes for received messages metric.
- */
 interface ReceivedMessagesAttributes extends Attributes {
   [ATTR_MESSAGING_SYSTEM]: string;
   [ATTR_MESSAGING_DESTINATION_NAME]: string;

packages/instrumentation-nats/src/propagator.ts

@@ -25,7 +25,6 @@ export const natsHeadersGetter: TextMapGetter<MsgHdrs | undefined> = {
       return undefined;
     }
 
-    // MsgHdrs.get() returns the header value as a string directly
     return carrier.get(key);
   },
 

packages/instrumentation-nats/src/types.ts

@@ -18,12 +18,12 @@ import { Span } from '@opentelemetry/api';
 import type { Msg } from 'nats';
 
 /**
- * Hook function called after a message is published.
+ * Hook function called before a message is published.
  */
 export type PublishHook = (span: Span, info: PublishInfo) => void;
 
 /**
- * Hook function called after a message is received/processed.
+ * Hook function called before a message is received/processed.
  */
 export type ConsumeHook = (span: Span, info: ConsumeInfo) => void;
 
@@ -37,8 +37,13 @@ export interface ConsumeInfo {
 }
 
 export interface NatsInstrumentationConfig extends InstrumentationConfig {
+  /**
+   * Hook function called before a message is published.
+   */
   publishHook?: PublishHook;
-
+  /**
+   * Hook function called before a message is received/processed.
+   */
   consumeHook?: ConsumeHook;
   /**
    * Whether to include message body size in span attributes.