Merge pull request #31 from codebridger/CU-86exvrzy8_Close-analytics-gaps-against-docsmetrics-convention-names-missing-events-Stripe-webhook-lifecycle_Navid-Shad

Title: Align Analytics Events with Documentation and Fix Unit Test Imports #86exvrzy8

Author: navidshad

.gitignore

@@ -8,4 +8,5 @@ static/key-file.json
 .npmrc
 /.claude
 /playwright-report
-/test-results
+/test-results
+.agent

src/common/services/translate.service.ts

@@ -6,7 +6,7 @@ import {
 import { Dictionary } from "../types/general.type";
 
 import proxy from "./proxy.service";
-import { functionProvider } from "@modular-rest/client";
+import { functionProvider, authentication } from "@modular-rest/client";
 import {
   Chunk,
   LanguageLearningData,
@@ -132,6 +132,8 @@ export class TranslateService {
           targetLanguage: this.languageTitle,
           phrase: text,
           context: context || "",
+          // For the server-side translation_requested analytics event.
+          userId: authentication.user?.id,
         },
       });
 
@@ -169,6 +171,8 @@ export class TranslateService {
           targetLanguage: this.languageTitle,
           phrase: text,
           context: context || "",
+          // For the server-side translation_requested analytics event.
+          userId: authentication.user?.id,
         },
       });
 

src/console-crane/components/Definition.vue

@@ -139,7 +139,7 @@ export default defineComponent({
           });
         });
 
-      analytic.track("Definition translated");
+      analytic.track("definition_translated");
     },
   },
 });

src/main.ts

@@ -89,7 +89,7 @@ function start() {
         .then((_) => {
           registerGlobalEvents();
         })
-        .catch((_) => analytic.track("Error on initiating"));
+        .catch((_) => analytic.track("extension_init-failed"));
     }
 
     //

src/popup/views/LoginView.vue

@@ -136,6 +136,7 @@ import {
   StoreUserTokenMessage,
 } from "../../common/types/messaging";
 import { sendMessage, sendMessageToTabs } from "../../common/helper/massage";
+import { analytic } from "../../plugins/mixpanel";
 import { get } from "../helper/http";
 import { joinToBaseUrl } from "../../common/helper/url";
 import {
@@ -240,6 +241,9 @@ async function handleTokenLogin(token: string) {
   await sendMessage(message);
   sendMessageToTabs(message);
   await loginWithLastSession();
+  // Fired only on an explicit login from this view — session restores on
+  // startup do not pass through here.
+  analytic.track("user_logged-in");
 }
 
 function launchWebAuthFlow(authURL) {

src/subtitle/_support-template/components/Subtitle.vue

@@ -186,7 +186,7 @@ export default defineComponent({
     hoveredWord(value) {
       if (value.length) {
         this.translateWord(value);
-        analytic.track("Word hovered", { word: value });
+        analytic.track("phrase_hovered", { word: value });
       }
     },
   },

src/subtitle/components/specific/Word.vue

@@ -96,6 +96,13 @@ function onMouseLeave(e: MouseEvent) {
 }
 
 function OpenWordDetail() {
+  analytic.track("phrase_clicked", {
+    word:
+      markerStore.words.length > 1
+        ? markerStore.selectedPhrase
+        : props.modelValue,
+    platform: window.location.hostname,
+  });
   if (markerStore.words.length > 1) {
     emitOpen({
       page: "word-detail",

tests/nibble-surface.test.ts

@@ -30,6 +30,7 @@ vi.mock("../src/nibble/composables/useTextSelection", async () => {
 // the child stub doesn't trigger a real fetch chain just by importing.
 vi.mock("@modular-rest/client", () => ({
   functionProvider: { run: vi.fn() },
+  authentication: { user: { id: "test-user-id" } },
 }));
 
 import NibbleSurface from "../src/nibble/components/NibbleSurface.vue";

tests/selection-popup.test.ts

@@ -6,6 +6,7 @@ import { createTestingPinia } from "@pinia/testing";
 // network layer so a stray click in another test can't fire a real request.
 vi.mock("@modular-rest/client", () => ({
   functionProvider: { run: vi.fn() },
+  authentication: { user: { id: "test-user-id" } },
 }));
 
 import SelectionPopup from "../src/nibble/components/SelectionPopup.vue";

tests/translate.service.test.ts

@@ -5,6 +5,9 @@ vi.mock("@modular-rest/client", () => ({
   functionProvider: {
     run: vi.fn(),
   },
+  // translate.service reads authentication.user?.id to attach a userId for the
+  // server-side translation_requested event.
+  authentication: { user: { id: "test-user-id" } },
 }));
 
 import { TranslateService } from "../src/common/services/translate.service";