Merge branch 'duo-feedback-snowplow-frontend' into 'master'

Switch Duo Chat feedback event sending to the backend

See merge request https://gitlab.com/gitlab-org/gitlab/-/merge_requests/149890



Merged-by: Jannik Lehmann <jlehmann@gitlab.com>
Approved-by: Jannik Lehmann <jlehmann@gitlab.com>
Co-authored-by: Alejandro Rodríguez <alejandro@gitlab.com>

ee/app/assets/javascripts/ai/tanuki_bot/components/app.vue

@@ -172,17 +172,6 @@ export default {
       this.helpCenterState.showTanukiBotChatDrawer = true;
     },
     onTrackFeedback({ feedbackChoices, didWhat, improveWhat, message } = {}) {
-      this.track(TANUKI_BOT_TRACKING_EVENT_NAME, {
-        action: 'click_button',
-        label: 'response_feedback',
-        property: feedbackChoices,
-        extra: {
-          improveWhat,
-          didWhat,
-          prompt_location: 'after_content',
-        },
-      });
-
       if (message) {
         const { id, requestId, extras, role, content } = message;
         this.$apollo
@@ -191,6 +180,17 @@ export default {
             variables: {
               input: {
                 aiMessageId: id,
+                trackingEvent: {
+                  category: TANUKI_BOT_TRACKING_EVENT_NAME,
+                  action: 'click_button',
+                  label: 'response_feedback',
+                  property: feedbackChoices.join(','),
+                  extra: {
+                    improveWhat,
+                    didWhat,
+                    prompt_location: 'after_content',
+                  },
+                },
               },
             },
           })

ee/spec/frontend/ai/tanuki_bot/components/app_spec.js

@@ -285,37 +285,6 @@ describe('GitLab Duo Chat', () => {
     });
 
     describe('@track-feedback', () => {
-      let trackingSpy;
-
-      beforeEach(() => {
-        trackingSpy = mockTracking(undefined, wrapper.element, jest.spyOn);
-      });
-
-      afterEach(() => {
-        unmockTracking();
-      });
-
-      it('tracks the snowplow event on successful mutation for chat', async () => {
-        createComponent();
-        findGlDuoChat().vm.$emit('track-feedback', {
-          feedbackChoices: ['foo', 'bar'],
-          improveWhat: 'improveWhat',
-          didWhat: 'didWhat',
-        });
-
-        await waitForPromises();
-        expect(trackingSpy).toHaveBeenCalledWith(undefined, TANUKI_BOT_TRACKING_EVENT_NAME, {
-          action: 'click_button',
-          label: 'response_feedback',
-          property: ['foo', 'bar'],
-          extra: {
-            improveWhat: 'improveWhat',
-            didWhat: 'didWhat',
-            prompt_location: 'after_content',
-          },
-        });
-      });
-
       it('calls the feedback GraphQL mutation when message is passed', async () => {
         createComponent();
         findGlDuoChat().vm.$emit('track-feedback', feedbackData);
@@ -324,6 +293,17 @@ describe('GitLab Duo Chat', () => {
         expect(duoUserFeedbackMutationHandlerMock).toHaveBeenCalledWith({
           input: {
             aiMessageId: feedbackData.message.id,
+            trackingEvent: {
+              category: TANUKI_BOT_TRACKING_EVENT_NAME,
+              action: 'click_button',
+              label: 'response_feedback',
+              property: 'useful,not_relevant',
+              extra: {
+                improveWhat: 'more examples',
+                didWhat: 'provided clarity',
+                prompt_location: 'after_content',
+              },
+            },
           },
         });
       });