apps/web/app/(ee)/admin.dub.co/(dashboard)/components/user-info.tsx (5)

19-29: Tighten status typing to prevent runtime lookups from breaking.

Constrain status to keys of PartnerStatusBadges so TS catches unsupported values at compile time.

Apply this diff:

   programs: {
     id: string;
     name: string;
     slug: string;
-    status: string;
+    status: keyof typeof PartnerStatusBadges;
     totalClicks: number;
     totalLeads: number;
     totalConversions: number;
     totalSaleAmount: number;
     totalCommissions: number;
   }[];

---

36-41: Give workspaceItems a keyed union for type-safe access.

This avoids workspace[item.id] being typed as any and ensures only allowed keys are used.

Apply this diff:

-const workspaceItems = [
-  { id: "clicks", label: "Clicks" },
-  { id: "links", label: "Links" },
-  { id: "totalClicks", label: "Total Clicks" },
-  { id: "totalLinks", label: "Total Links" },
-];
+const workspaceItems = [
+  { id: "clicks", label: "Clicks" },
+  { id: "links", label: "Links" },
+  { id: "totalClicks", label: "Total Clicks" },
+  { id: "totalLinks", label: "Total Links" },
+] as const satisfies ReadonlyArray<{
+  id: keyof UserInfoProps["workspaces"][number];
+  label: string;
+}>;

---

43-49: Same for programItems: type-safe ids and currency flag.

Apply this diff:

-const programItems = [
-  { id: "totalClicks", label: "Total Clicks" },
-  { id: "totalLeads", label: "Total Leads" },
-  { id: "totalConversions", label: "Total Conversions" },
-  { id: "totalSaleAmount", label: "Total Sales", isCurrency: true },
-  { id: "totalCommissions", label: "Total Commissions", isCurrency: true },
-];
+const programItems = [
+  { id: "totalClicks", label: "Total Clicks" },
+  { id: "totalLeads", label: "Total Leads" },
+  { id: "totalConversions", label: "Total Conversions" },
+  { id: "totalSaleAmount", label: "Total Sales", isCurrency: true },
+  { id: "totalCommissions", label: "Total Commissions", isCurrency: true },
+] as const satisfies ReadonlyArray<{
+  id: keyof UserInfoProps["programs"][number];
+  label: string;
+  isCurrency?: boolean;
+}>;

---

64-103: Use a stable key and keep grid responsive.

• Prefer workspace.id over slug for React keys.
• Optional: improve small-screen layout with a 1-col fallback.

Apply this diff:

-          <div className="grid grid-cols-2 gap-4">
+          <div className="grid grid-cols-1 md:grid-cols-2 gap-4">
             {data.workspaces.map((workspace) => (
               <div
-                key={workspace.slug}
+                key={workspace.id}
                 className="flex flex-col space-y-2 rounded-lg border border-neutral-200 p-2"
               >

---

109-152: Harden StatusBadge lookup and optionally improve currency formatting.

Guard against unknown statuses to avoid runtime errors, and consider a currency formatter for dollar values.

Apply this diff:

                 <div className="flex justify-between text-sm">
                   <span className="font-medium text-neutral-700">Status</span>
-                  <StatusBadge
-                    variant={PartnerStatusBadges[program.status].variant}
-                  >
-                    {PartnerStatusBadges[program.status].label}
-                  </StatusBadge>
+                  <StatusBadge
+                    variant={
+                      PartnerStatusBadges[
+                        program.status as keyof typeof PartnerStatusBadges
+                      ]?.variant ?? "neutral"
+                    }
+                  >
+                    {
+                      PartnerStatusBadges[
+                        program.status as keyof typeof PartnerStatusBadges
+                      ]?.label ?? capitalize(program.status)
+                    }
+                  </StatusBadge>
                 </div>
                 {programItems.map((item) => (
                   <div key={item.id} className="flex justify-between text-sm">
                     <span className="font-medium text-neutral-700">
                       {item.label}
                     </span>
                     <span className="text-neutral-500">
-                      {item.isCurrency
-                        ? `$${nFormatter(program[item.id], { full: true })}`
-                        : nFormatter(program[item.id], { full: true })}
+                      {item.isCurrency
+                        ? `$${nFormatter(program[item.id], { full: true })}`
+                        : nFormatter(program[item.id], { full: true })}
                     </span>
                   </div>
                 ))}

Optional: if available in @dub/utils, prefer a money formatter:

// replace `$${nFormatter(...)}` with formatCurrency(program[item.id])

apps/web/lib/tinybird/record-link.ts (1)

45-49: Remove redundant workspace_id override in recordLinkTB.

The dubLinksMetadataSchema already defines workspace_id with the exact same transformation (.string().nullish().transform((v) => (v ? v : ""))). The extend override duplicates this logic unnecessarily. Removing it maintains identical behavior while keeping a single source of truth.

 export const recordLinkTB = tb.buildIngestEndpoint({
   datasource: "dub_links_metadata",
-  event: dubLinksMetadataSchema.extend({
-    workspace_id: z
-      .string()
-      .nullish()
-      .transform((v) => (v ? v : "")),
-  }),
+  event: dubLinksMetadataSchema,
   wait: true,
 });

apps/web/app/(ee)/api/admin/impersonate/route.ts (1)

41-46: Add index on Project.totalClicks for nested orderBy performance.

Prisma 5.18.0 supports nested orderBy on relations, so the syntax is compatible. However, the totalClicks column lacks an index in the Project model (packages/prisma/schema/workspace.prisma). Ordering by an unindexed column can degrade performance at scale. Add @@index(totalClicks(sort: Desc)) to the Project model.

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro