From e2a5372c50ea6769916a86a47b42c95d1a599dc9 Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Fri, 21 Nov 2025 18:01:02 +0000
Subject: [PATCH 1/2] Initial plan


From 58d5bfbcf9365727021a31f95dae450e8d6807b9 Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Fri, 21 Nov 2025 18:04:39 +0000
Subject: [PATCH 2/2] Add comment and errorCount metric to telemetry

Co-authored-by: tnaum-ms <171359267+tnaum-ms@users.noreply.github.com>
---
 src/commands/removeConnection/removeConnection.ts | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/src/commands/removeConnection/removeConnection.ts b/src/commands/removeConnection/removeConnection.ts
index 1b486ca4d..483e2bad6 100644
--- a/src/commands/removeConnection/removeConnection.ts
+++ b/src/commands/removeConnection/removeConnection.ts
@@ -87,6 +87,8 @@ export async function removeConnection(
                 }),
             );
 
+            // Note: When multiple deletions fail, we intentionally capture only the last error's
+            // details in telemetry. The errorCount metric below provides context on total failures.
             context.telemetry.properties.error = 'RemoveConnectionError';
             context.telemetry.properties.errorMessage = error instanceof Error ? error.message : String(error);
 
@@ -99,6 +101,7 @@ export async function removeConnection(
 
     // Set telemetry for successfully deleted connections
     context.telemetry.measurements.connectionsDeleted = successCount;
+    context.telemetry.measurements.errorCount = failureCount;
 
     // Show summary message
     if (connectionsToDelete.length === 1) {