chore: use chat response as default source

Author: gagik

packages/compass-assistant/src/components/assistant-chat.spec.tsx

@@ -519,6 +519,34 @@ describe('AssistantChat', function () {
       });
     });
 
+    it('tracks it as "chat response" when source is not present', async function () {
+      const { result } = renderWithChat([
+        {
+          ...mockMessages[1],
+          metadata: {
+            ...mockMessages[1].metadata,
+            source: undefined,
+          },
+        },
+      ]);
+      const { track } = result;
+
+      const thumbsDownButton = within(
+        screen.getByTestId('assistant-message-assistant')
+      ).getByLabelText('Dislike this message');
+
+      userEvent.click(thumbsDownButton);
+
+      await waitFor(() => {
+        expect(track).to.have.been.calledWith('Assistant Feedback Submitted', {
+          feedback: 'negative',
+          text: undefined,
+          request_id: null,
+          source: 'chat response',
+        });
+      });
+    });
+
     it('does not show feedback buttons when there are no assistant messages', function () {
       const userOnlyMessages: AssistantMessage[] = [
         {
@@ -807,6 +835,29 @@ describe('AssistantChat', function () {
         );
       });
     });
+
+    it('tracks it as "chat response" when source is not present', async function () {
+      const { result } = renderWithChat([
+        {
+          ...mockConfirmationMessage,
+          metadata: {
+            ...mockConfirmationMessage.metadata,
+            source: undefined,
+          },
+        },
+      ]);
+      const { track } = result;
+
+      const confirmButton = screen.getByText('Confirm');
+      userEvent.click(confirmButton);
+
+      await waitFor(() => {
+        expect(track).to.have.been.calledWith(
+          'Assistant Confirmation Submitted',
+          { status: 'confirmed', source: 'chat response' }
+        );
+      });
+    });
   });
 
   describe('related sources', function () {

packages/compass-assistant/src/components/assistant-chat.tsx

@@ -222,7 +222,7 @@ export const AssistantChat: React.FunctionComponent<AssistantChatProps> = ({
         feedback,
         text: textFeedback,
         request_id: null,
-        source: message.metadata?.source,
+        source: message.metadata?.source ?? 'chat response',
       });
     },
     [track]
@@ -268,7 +268,7 @@ export const AssistantChat: React.FunctionComponent<AssistantChatProps> = ({
       });
       track('Assistant Confirmation Submitted', {
         status: newState,
-        source: confirmedMessage.metadata?.source,
+        source: confirmedMessage.metadata?.source ?? 'chat response',
       });
       if (newState === 'confirmed') {
         // Force the new message request to be sent

packages/compass-telemetry/src/telemetry-events.ts

@@ -1494,7 +1494,7 @@ type AssistantFeedbackSubmittedEvent = CommonEvent<{
     feedback: 'positive' | 'negative';
     text: string | undefined;
     request_id: string | null;
-    source: AssistantEntryPointUsedEvent['payload']['source'] | undefined;
+    source: AssistantEntryPointUsedEvent['payload']['source'] | 'chat response';
   };
 }>;
 
@@ -1507,7 +1507,7 @@ type AssistantConfirmationSubmittedEvent = CommonEvent<{
   name: 'Assistant Confirmation Submitted';
   payload: {
     status: 'confirmed' | 'rejected';
-    source: AssistantEntryPointUsedEvent['payload']['source'] | undefined;
+    source: AssistantEntryPointUsedEvent['payload']['source'] | 'chat response';
   };
 }>;