From 49141a300c82834542f9ccb22fec81f0a711f590 Mon Sep 17 00:00:00 2001 From: Copilot <198982749+Copilot@users.noreply.github.com> Date: Thu, 19 Feb 2026 19:42:06 -0800 Subject: [PATCH] Remove unnecessary `_connectCts` field from `McpClientImpl` (#1323) Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: stephentoub <2642209+stephentoub@users.noreply.github.com> --- src/ModelContextProtocol.Core/Client/McpClientImpl.cs | 5 ----- 1 file changed, 5 deletions(-) diff --git a/src/ModelContextProtocol.Core/Client/McpClientImpl.cs b/src/ModelContextProtocol.Core/Client/McpClientImpl.cs index 6f4d938..9375e27 100644 --- a/src/ModelContextProtocol.Core/Client/McpClientImpl.cs +++ b/src/ModelContextProtocol.Core/Client/McpClientImpl.cs @@ -22,8 +22,6 @@ internal sealed partial class McpClientImpl : McpClient private readonly SemaphoreSlim _disposeLock = new(1, 1); private readonly McpTaskCancellationTokenProvider? _taskCancellationTokenProvider; - private CancellationTokenSource? _connectCts; - private ServerCapabilities? _serverCapabilities; private Implementation? _serverInfo; private string? _serverInstructions; @@ -526,9 +524,6 @@ internal sealed partial class McpClientImpl : McpClient /// public async Task ConnectAsync(CancellationToken cancellationToken = default) { - _connectCts = CancellationTokenSource.CreateLinkedTokenSource(cancellationToken); - cancellationToken = _connectCts.Token; - try { // We don't want the ConnectAsync token to cancel the message processing loop after we've successfully connected.