Refactor UUID handling and improve metrics data sanitization

Developing-Gamer · Developing-Gamer · commit d3bd0f2442db · 2026-03-12T11:18:17.000-07:00
- Updated the `bulkId` generation in `seedDummyEmails` for better idempotency.
- Introduced `normalizeUuidFromEvent` function to sanitize user and team IDs in metrics loading functions, ensuring only valid UUIDs are processed.
- Enhanced the logic in `loadDailyActiveUsersSplit` and `loadDailyActiveTeamsSplit` to utilize sanitized data, improving data integrity and accuracy in metrics retrieval.
- Removed unnecessary mouse leave event handlers in `HeroAnalyticsWidget` for cleaner interaction.
diff --git a/apps/backend/prisma/seed.ts b/apps/backend/prisma/seed.ts
@@ -2112,7 +2112,7 @@ async function seedDummyEmails(options: EmailSeedOptions) {
 
     for (let j = 0; j < count; j++) {
       // Deterministic ID so seeding is idempotent across runs
-      const bulkId = `66000000-bulk-seed-${String(emailBulkIndex).padStart(4, '0')}-000000000000`;
+      const bulkId = `66000000-0000-4000-8000-${emailBulkIndex.toString(16).padStart(12, '0')}`;
       const hour = 7 + Math.floor(emailBulkRand() * 14);
       const createdAt = emailDaysAgo(dayBack, hour);
       const subject = bulkEmailSubjects[Math.floor(emailBulkRand() * bulkEmailSubjects.length)];
diff --git a/apps/backend/src/app/api/latest/internal/metrics/route.tsx b/apps/backend/src/app/api/latest/internal/metrics/route.tsx
@@ -6,6 +6,7 @@ import { KnownErrors } from "@stackframe/stack-shared";
 import { UsersCrud } from "@stackframe/stack-shared/dist/interface/crud/users";
 import { getNodeEnvironment } from "@stackframe/stack-shared/dist/utils/env";
 import { captureError, StackAssertionError } from "@stackframe/stack-shared/dist/utils/errors";
+import { isUuid } from "@stackframe/stack-shared/dist/utils/uuids";
 import { adaptSchema, adminAuthTypeSchema, yupArray, yupMixed, yupNumber, yupObject, yupString } from "@stackframe/stack-shared/dist/schema-fields";
 import yup from 'yup';
 import { userFullInclude, userPrismaToCrud, usersCrudHandlers } from "../../users/crud";
@@ -24,6 +25,11 @@ function formatClickhouseDateTimeParam(date: Date): string {
   return date.toISOString().slice(0, 19);
 }
 
+function normalizeUuidFromEvent(value: string): string | null {
+  const normalized = value.trim().toLowerCase();
+  return isUuid(normalized) ? normalized : null;
+}
+
 async function loadUsersByCountry(tenancy: Tenancy, includeAnonymous: boolean = false): Promise<Record<string, number>> {
   const clickhouseClient = getClickhouseAdminClient();
   const res = await clickhouseClient.query({
@@ -252,7 +258,15 @@ async function loadDailyActiveUsersSplit(tenancy: Tenancy, now: Date, includeAno
     format: "JSONEachRow",
   }).then((result) => result.json() as Promise<{ day: string, user_id: string }[]>);
 
-  const activeUserIds = [...new Set(userRows.map((row) => row.user_id))];
+  const sanitizedUserRows = userRows.flatMap((row) => {
+    const userId = normalizeUuidFromEvent(row.user_id);
+    if (userId == null) {
+      return [];
+    }
+    return [{ ...row, user_id: userId }];
+  });
+
+  const activeUserIds = [...new Set(sanitizedUserRows.map((row) => row.user_id))];
   const users = activeUserIds.length === 0
     ? []
     : await prisma.projectUser.findMany({
@@ -274,7 +288,7 @@ async function loadDailyActiveUsersSplit(tenancy: Tenancy, now: Date, includeAno
     orderedDays.push(date);
     idsByDay.set(date, new Set<string>());
   }
-  for (const row of userRows) {
+  for (const row of sanitizedUserRows) {
     const day = row.day.split('T')[0];
     const daySet = idsByDay.get(day);
     if (daySet) {
@@ -324,7 +338,15 @@ async function loadDailyActiveTeamsSplit(tenancy: Tenancy, now: Date): Promise<A
     format: "JSONEachRow",
   }).then((result) => result.json() as Promise<{ day: string, team_id: string }[]>);
 
-  const activeTeamIds = [...new Set(teamRows.map((row) => row.team_id))];
+  const sanitizedTeamRows = teamRows.flatMap((row) => {
+    const teamId = normalizeUuidFromEvent(row.team_id);
+    if (teamId == null) {
+      return [];
+    }
+    return [{ ...row, team_id: teamId }];
+  });
+
+  const activeTeamIds = [...new Set(sanitizedTeamRows.map((row) => row.team_id))];
   const teams = activeTeamIds.length === 0
     ? []
     : await prisma.team.findMany({
@@ -345,7 +367,7 @@ async function loadDailyActiveTeamsSplit(tenancy: Tenancy, now: Date): Promise<A
     orderedDays.push(date);
     idsByDay.set(date, new Set<string>());
   }
-  for (const row of teamRows) {
+  for (const row of sanitizedTeamRows) {
     const day = row.day.split('T')[0];
     const daySet = idsByDay.get(day);
     if (daySet) {
diff --git a/apps/dashboard/src/app/(main)/(protected)/projects/[projectId]/(overview)/metrics-page.tsx b/apps/dashboard/src/app/(main)/(protected)/projects/[projectId]/(overview)/metrics-page.tsx
@@ -378,7 +378,6 @@ function HeroAnalyticsWidget({
               color={dauColor}
               isHovered={chartMode === 'dau'}
               onMouseEnter={() => handlePillMouseEnter('dau')}
-              onMouseLeave={handlePillMouseLeave}
             />
             <div className="w-px bg-foreground/[0.07] shrink-0 my-1.5 mx-1" />
             <HeroInChartPill
@@ -388,7 +387,6 @@ function HeroAnalyticsWidget({
               color={visitorsColor}
               isHovered={chartMode === 'visitors'}
               onMouseEnter={() => handlePillMouseEnter('visitors')}
-              onMouseLeave={handlePillMouseLeave}
             />
             <div className="w-px bg-foreground/[0.07] shrink-0 my-1.5 mx-1" />
             <HeroInChartPill
@@ -398,7 +396,6 @@ function HeroAnalyticsWidget({
               color={revenueColor}
               isHovered={chartMode === 'revenue'}
               onMouseEnter={() => handlePillMouseEnter('revenue')}
-              onMouseLeave={handlePillMouseLeave}
             />
           </div>
 
