Remove unnecessary IsEnabled guards from logging call sites

Since the arguments to logging calls are just passing existing variables
(no expensive work), the external guards are redundant - the source-generated
logger methods handle the IsEnabled check internally.

Co-authored-by: stephentoub <2642209+stephentoub@users.noreply.github.com>

Author: Copilot

src/ModelContextProtocol.Core/Client/SseClientSessionTransport.cs

@@ -195,10 +195,7 @@ private async Task ProcessSseMessage(string data, CancellationToken cancellation
             return;
         }
 
-        if (_logger.IsEnabled(LogLevel.Trace))
-        {
-            LogTransportReceivedMessageSensitive(Name, data);
-        }
+        LogTransportReceivedMessageSensitive(Name, data);
 
         try
         {

src/ModelContextProtocol.Core/Client/StreamClientSessionTransport.cs

@@ -105,10 +105,7 @@ public override async Task SendMessageAsync(JsonRpcMessage message, Cancellation
 
         var json = JsonSerializer.Serialize(message, McpJsonUtilities.JsonContext.Default.JsonRpcMessage);
 
-        if (Logger.IsEnabled(LogLevel.Trace))
-        {
-            LogTransportSendingMessageSensitive(Name, json);
-        }
+        LogTransportSendingMessageSensitive(Name, json);
 
         using var _ = await _sendLock.LockAsync(cancellationToken).ConfigureAwait(false);
         try
@@ -148,10 +145,7 @@ private async Task ReadMessagesAsync(CancellationToken cancellationToken)
                     continue;
                 }
 
-                if (Logger.IsEnabled(LogLevel.Trace))
-                {
-                    LogTransportReceivedMessageSensitive(Name, line);
-                }
+                LogTransportReceivedMessageSensitive(Name, line);
 
                 await ProcessMessageAsync(line, cancellationToken).ConfigureAwait(false);
             }

src/ModelContextProtocol.Core/Client/StreamableHttpClientSessionTransport.cs

@@ -344,10 +344,7 @@ private async Task<SseResponse> ProcessSseResponseAsync(
 
     private async Task<JsonRpcMessageWithId?> ProcessMessageAsync(string data, JsonRpcRequest? relatedRpcRequest, CancellationToken cancellationToken)
     {
-        if (_logger.IsEnabled(LogLevel.Trace))
-        {
-            LogTransportReceivedMessageSensitive(Name, data);
-        }
+        LogTransportReceivedMessageSensitive(Name, data);
 
         try
         {

src/ModelContextProtocol.Core/Server/StreamServerTransport.cs

@@ -75,10 +75,7 @@ public override async Task SendMessageAsync(JsonRpcMessage message, Cancellation
         try
         {
             var json = JsonSerializer.Serialize(message, McpJsonUtilities.JsonContext.Default.JsonRpcMessage);
-            if (Logger.IsEnabled(LogLevel.Trace))
-            {
-                LogTransportSendingMessageSensitive(Name, json);
-            }
+            LogTransportSendingMessageSensitive(Name, json);
             await _outputStream.WriteAsync(Encoding.UTF8.GetBytes(json), cancellationToken).ConfigureAwait(false);
             await _outputStream.WriteAsync(s_newlineBytes, cancellationToken).ConfigureAwait(false);
             await _outputStream.FlushAsync(cancellationToken).ConfigureAwait(false);
@@ -112,10 +109,7 @@ private async Task ReadMessagesAsync()
                     continue;
                 }
 
-                if (Logger.IsEnabled(LogLevel.Trace))
-                {
-                    LogTransportReceivedMessageSensitive(Name, line);
-                }
+                LogTransportReceivedMessageSensitive(Name, line);
 
                 try
                 {