Remove Debug-level logging, keep only Trace-level logging for sent messages

Addresses review feedback to only have Trace-level logging for JSON-RPC payloads.

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

Author: Copilot

src/ModelContextProtocol.Core/Client/SseClientSessionTransport.cs

@@ -80,14 +80,7 @@ public override async Task SendMessageAsync(
             messageId = messageWithId.Id.ToString();
         }
 
-        if (_logger.IsEnabled(LogLevel.Trace))
-        {
-            LogTransportSendingMessageSensitive(Name, JsonSerializer.Serialize(message, McpJsonUtilities.JsonContext.Default.JsonRpcMessage));
-        }
-        else
-        {
-            LogTransportSendingMessage(Name, messageId);
-        }
+        LogTransportSendingMessageSensitive(Name, JsonSerializer.Serialize(message, McpJsonUtilities.JsonContext.Default.JsonRpcMessage));
 
         using var httpRequestMessage = new HttpRequestMessage(HttpMethod.Post, _messageEndpoint);
         StreamableHttpClientSessionTransport.CopyAdditionalHeaders(httpRequestMessage.Headers, _options.AdditionalHeaders, sessionId: null, protocolVersion: null);

src/ModelContextProtocol.Core/Client/StreamClientSessionTransport.cs

@@ -105,14 +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);
-        }
-        else
-        {
-            LogTransportSendingMessage(Name, id);
-        }
+        LogTransportSendingMessageSensitive(Name, json);
 
         using var _ = await _sendLock.LockAsync(cancellationToken).ConfigureAwait(false);
         try

src/ModelContextProtocol.Core/Client/StreamableHttpClientSessionTransport.cs

@@ -75,20 +75,7 @@ internal async Task<HttpResponseMessage> SendHttpRequestAsync(JsonRpcMessage mes
                 $"Call {nameof(McpClient)}.{nameof(McpClient.ResumeSessionAsync)} to resume existing sessions.");
         }
 
-        string messageId = "(no id)";
-        if (message is JsonRpcMessageWithId messageWithId)
-        {
-            messageId = messageWithId.Id.ToString();
-        }
-
-        if (_logger.IsEnabled(LogLevel.Trace))
-        {
-            LogTransportSendingMessageSensitive(Name, JsonSerializer.Serialize(message, McpJsonUtilities.JsonContext.Default.JsonRpcMessage));
-        }
-        else
-        {
-            LogTransportSendingMessage(Name, messageId);
-        }
+        LogTransportSendingMessageSensitive(Name, JsonSerializer.Serialize(message, McpJsonUtilities.JsonContext.Default.JsonRpcMessage));
 
         using var sendCts = CancellationTokenSource.CreateLinkedTokenSource(cancellationToken, _connectionCts.Token);
         cancellationToken = sendCts.Token;

src/ModelContextProtocol.Core/Protocol/TransportBase.cs

@@ -166,9 +166,6 @@ protected void SetDisconnected(Exception? error = null)
     [LoggerMessage(Level = LogLevel.Error, Message = "{EndpointName} transport send failed for message ID '{MessageId}'.")]
     private protected partial void LogTransportSendFailed(string endpointName, string messageId, Exception exception);
 
-    [LoggerMessage(Level = LogLevel.Debug, Message = "{EndpointName} transport sending message with ID '{MessageId}'.")]
-    private protected partial void LogTransportSendingMessage(string endpointName, string messageId);
-
     [LoggerMessage(Level = LogLevel.Trace, Message = "{EndpointName} transport sending message. Message: '{Message}'.")]
     private protected partial void LogTransportSendingMessageSensitive(string endpointName, string message);
 

src/ModelContextProtocol.Core/Server/StreamServerTransport.cs

@@ -74,17 +74,9 @@ public override async Task SendMessageAsync(JsonRpcMessage message, Cancellation
 
         try
         {
-            if (Logger.IsEnabled(LogLevel.Trace))
-            {
-                var json = JsonSerializer.Serialize(message, McpJsonUtilities.JsonContext.Default.JsonRpcMessage);
-                LogTransportSendingMessageSensitive(Name, json);
-                await _outputStream.WriteAsync(Encoding.UTF8.GetBytes(json), cancellationToken).ConfigureAwait(false);
-            }
-            else
-            {
-                LogTransportSendingMessage(Name, id);
-                await JsonSerializer.SerializeAsync(_outputStream, message, McpJsonUtilities.DefaultOptions.GetTypeInfo(typeof(JsonRpcMessage)), cancellationToken).ConfigureAwait(false);
-            }
+            var json = JsonSerializer.Serialize(message, McpJsonUtilities.JsonContext.Default.JsonRpcMessage);
+            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);
         }