chore(otlp): remove redundant comments

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Author: jchrostek-dd

bottlecap/src/otlp/agent.rs

@@ -204,13 +204,9 @@ impl Agent {
             }
         }
 
-        // This needs to be after process_traces() because process_traces()
-        // performs obfuscation, and we need to compute stats on the obfuscated traces.
         if compute_trace_stats_on_extension
             && let Err(err) = stats_generator.send(&processed_traces)
         {
-            // Just log the error. We don't think trace stats are critical, so we don't want to
-            // return an error if only stats fail to send.
             error!("OTLP | Error sending traces to the stats concentrator: {err}");
         }
 
@@ -307,7 +303,6 @@ mod tests {
 
     #[test]
     fn test_parse_port_with_valid_endpoint() {
-        // Test with a valid endpoint containing a port
         let endpoint = Some("localhost:8080".to_string());
         assert_eq!(
             Agent::parse_port(endpoint.as_ref(), OTLP_AGENT_HTTP_PORT),
@@ -317,7 +312,6 @@ mod tests {
 
     #[test]
     fn test_parse_port_with_invalid_port_format() {
-        // Test with an endpoint containing an invalid port format
         let endpoint = Some("localhost:invalid".to_string());
         assert_eq!(
             Agent::parse_port(endpoint.as_ref(), OTLP_AGENT_HTTP_PORT),
@@ -327,7 +321,6 @@ mod tests {
 
     #[test]
     fn test_parse_port_with_missing_port() {
-        // Test with an endpoint missing a port
         let endpoint = Some("localhost".to_string());
         assert_eq!(
             Agent::parse_port(endpoint.as_ref(), OTLP_AGENT_HTTP_PORT),
@@ -337,7 +330,6 @@ mod tests {
 
     #[test]
     fn test_parse_port_with_none_endpoint() {
-        // Test with None endpoint
         let endpoint: Option<String> = None;
         assert_eq!(
             Agent::parse_port(endpoint.as_ref(), OTLP_AGENT_HTTP_PORT),
@@ -347,7 +339,6 @@ mod tests {
 
     #[test]
     fn test_parse_port_with_empty_endpoint() {
-        // Test with an empty endpoint
         let endpoint = Some(String::new());
         assert_eq!(
             Agent::parse_port(endpoint.as_ref(), OTLP_AGENT_HTTP_PORT),

bottlecap/src/otlp/grpc_agent.rs

@@ -41,8 +41,6 @@ impl TraceService for OtlpGrpcService {
         request: Request<ExportTraceServiceRequest>,
     ) -> Result<Response<ExportTraceServiceResponse>, Status> {
         let inner_request = request.into_inner();
-
-        // Capture encoded size before processing for metrics
         let body_size = inner_request.encoded_len();
 
         let traces = match self.processor.process_request(inner_request) {
@@ -53,7 +51,6 @@ impl TraceService for OtlpGrpcService {
             }
         };
 
-        // Check if processor returned any actual traces
         if traces.iter().all(Vec::is_empty) {
             error!("OTLP gRPC | Not sending traces, processor returned empty data");
             return Err(Status::internal(
@@ -83,11 +80,9 @@ impl TraceService for OtlpGrpcService {
             debug!("OTLP gRPC | Successfully buffered traces to be aggregated.");
         }
 
-        // Compute trace stats after process_traces() which performs obfuscation
         if compute_trace_stats_on_extension
             && let Err(err) = self.stats_generator.send(&processed_traces)
         {
-            // Just log the error. Stats are not critical.
             error!("OTLP gRPC | Error sending traces to the stats concentrator: {err}");
         }
 
@@ -136,15 +131,13 @@ impl GrpcAgent {
 
     fn parse_port(endpoint: Option<&String>, default_port: u16) -> u16 {
         if let Some(endpoint) = endpoint {
-            // Strip scheme if present (e.g., "http://localhost:4317" -> "localhost:4317")
             let without_scheme = endpoint
                 .strip_prefix("http://")
                 .or_else(|| endpoint.strip_prefix("https://"))
                 .unwrap_or(endpoint);
 
-            // Use rsplit to get port from the last colon (handles IPv6 like [::1]:4317)
+            // rsplit handles IPv6 like [::1]:4317
             if let Some(port_str) = without_scheme.rsplit(':').next() {
-                // Ensure we got a port, not part of IPv6 address
                 if let Ok(port) = port_str.parse::<u16>() {
                     return port;
                 }