remove correlation ids in Chat and UserAction

Author: NoriZC

shell/agents/Microsoft.Azure.Agent/AzureAgent.cs

@@ -124,21 +124,31 @@ public void Initialize(AgentConfig config)
     public void OnUserAction(UserActionPayload actionPayload) {
         // Send telemetry about the user action.
         bool isUserFeedback = false;
+        bool allowIdsForCorrelation = false;
         string details = null;
         UserAction action = actionPayload.Action;
 
         if (action is UserAction.Dislike)
         {
-            var dislike = (DislikePayload) actionPayload;
             isUserFeedback = true;
+            DislikePayload dislikePayload = (DislikePayload)actionPayload;
+            allowIdsForCorrelation = dislikePayload.ShareConversation;
+            var dislike = (DislikePayload) actionPayload;
             details = string.Format("{0} | {1}", dislike.ShortFeedback, dislike.LongFeedback);
         }
         else if (action is UserAction.Like)
         {
             isUserFeedback = true;
         }
 
-        Telemetry.Trace(AzTrace.UserAction(action.ToString(), _copilotResponse, details, isUserFeedback));
+        if (isUserFeedback)
+        {
+            Telemetry.Trace(AzTrace.Feedback(action.ToString(), _copilotResponse, details, allowIdsForCorrelation));
+        }
+        else
+        {
+            Telemetry.Trace(AzTrace.UserAction(action.ToString(), _copilotResponse, details));
+        }
     }
 
     public async Task RefreshChatAsync(IShell shell, bool force)

shell/agents/Microsoft.Azure.Agent/Telemetry.cs

@@ -76,20 +76,40 @@ internal static void Initialize()
     internal object Details { get; set; }
 
     internal static AzTrace UserAction(
+        string shellCommand,
+        CopilotResponse response,
+        object details)
+    {
+        if (Telemetry.Enabled)
+        {
+            return new()
+            {
+                TopicName = response.TopicName,
+                ShellCommand = shellCommand,
+                EventType = "UserAction",
+                Details = details
+            };
+        }
+
+        // Don't create an object when telemetry is disabled.
+        return null;
+    }
+
+    internal static AzTrace Feedback(
         string shellCommand,
         CopilotResponse response,
         object details,
-        bool isFeedback = false)
+        bool allowIdsForCorrelation)
     {
         if (Telemetry.Enabled)
         {
             return new()
             {
-                QueryId = response.ReplyToId,
+                QueryId = allowIdsForCorrelation ? response.ReplyToId : null,
                 TopicName = response.TopicName,
-                ConversationId = response.ConversationId,
+                ConversationId = allowIdsForCorrelation ? response.ConversationId : null,
                 ShellCommand = shellCommand,
-                EventType = isFeedback ? "Feedback" : "UserAction",
+                EventType = "Feedback",
                 Details = details
             };
         }
@@ -105,9 +125,7 @@ internal static AzTrace Chat(CopilotResponse response)
             return new()
             {
                 EventType = "Chat",
-                QueryId = response.ReplyToId,
-                TopicName = response.TopicName,
-                ConversationId = response.ConversationId
+                TopicName = response.TopicName
             };
         }