⚠️ Potential issue | 🔴 Critical

#!/bin/bash
# Check how other components use the loading state from useProgramEnrollment
rg -nP -C3 'useProgramEnrollment.*loading' --type=tsx --type=ts

#!/bin/bash
# Check how other components use useProgramEnrollment hook
rg -nP -C3 'useProgramEnrollment' --type=ts

#!/bin/bash
# Find the useProgramEnrollment hook definition
fd -e ts -e tsx | xargs grep -l 'useProgramEnrollment' | head -5

#!/bin/bash
# Check the redirect logic around line 30 in the page-client file
rg -nP -B5 -A5 'redirect' apps/web/app/\(ee\)/partners.dub.co/\(dashboard\)/programs/\[programSlug\]/\(enrolled\)/customers/\[customerId\]/page-client.tsx

#!/bin/bash
# Find and display the useProgramEnrollment hook implementation
cat apps/web/lib/swr/use-program-enrollment.ts

Check the loading state from useProgramEnrollment before redirecting based on showDetailedAnalytics.

The useProgramEnrollment hook returns a loading state, but the redirect condition at line 30 only checks isLoading from usePartnerCustomer. During initial load, programEnrollment is undefined, so totalCommissions defaults to 0, which can make showDetailedAnalytics false prematurely for the "perplexity" program. This causes a redirect before the enrollment data finishes loading.

Destructure loading from useProgramEnrollment and update the redirect condition to avoid redirecting while enrollment data is still loading:

const { programEnrollment, showDetailedAnalytics, loading } = useProgramEnrollment();

Then update line 30 to:

if ((!customer && !isLoading) || (!showDetailedAnalytics && !loading)) {
  redirect(`/programs/${programSlug}`);
}

In
apps/web/app/(ee)/partners.dub.co/(dashboard)/programs/[programSlug]/(enrolled)/customers/[customerId]/page-client.tsx
around line 20, the redirect logic uses showDetailedAnalytics before program
enrollment finishes loading; destructure loading from useProgramEnrollment
(const { programEnrollment, showDetailedAnalytics, loading } =
useProgramEnrollment()) and update the redirect condition to guard against
redirecting while enrollment is loading (i.e., only redirect when (!customer &&
!isLoading) || (!showDetailedAnalytics && !loading)), ensuring you don't
redirect for the "perplexity" program prematurely.

⚠️ Potential issue | 🟠 Major

Potential race condition in redirect logic.

The redirect condition checks !showDetailedAnalytics, but this flag depends on programEnrollment.totalCommissions which may not be loaded yet. During the loading state, totalCommissions defaults to 0, which could trigger a premature redirect for "perplexity" program users before their commission data loads.

Consider checking the loading state from useProgramEnrollment before redirecting:

-  const { programEnrollment, showDetailedAnalytics } = useProgramEnrollment();
+  const { programEnrollment, showDetailedAnalytics, loading: isProgramLoading } = useProgramEnrollment();
   const { programSlug, customerId } = useParams<{
     programSlug: string;
     customerId: string;
   }>();

   const { data: customer, isLoading } = usePartnerCustomer({
     customerId,
   });

-  if ((!customer && !isLoading) || !showDetailedAnalytics) {
+  if ((!customer && !isLoading) || (!showDetailedAnalytics && !isProgramLoading)) {
     redirect(`/programs/${programSlug}`);
   }

Committable suggestion skipped: line range outside the PR's diff.

In
apps/web/app/(ee)/partners.dub.co/(dashboard)/programs/[programSlug]/(enrolled)/customers/[customerId]/page-client.tsx
around lines 30-32, the redirect uses showDetailedAnalytics which depends on
programEnrollment.totalCommissions that may be 0 while enrollment data is still
loading, causing a premature redirect; update the condition to also wait for the
enrollment loading flag (e.g., isLoading from useProgramEnrollment or an
explicit enrollmentLoading) so the redirect only runs when both customer and
enrollment data have finished loading and showDetailedAnalytics is definitively
false.

⚠️ Potential issue | 🟠 Major

Race condition: showDetailedAnalytics evaluates incorrectly during loading.

When programEnrollment is still loading, it's undefined, so totalCommissions defaults to 0. For the "perplexity" program, this makes showDetailedAnalytics false during the loading state, even if the user should have access once data loads.

Components that use this flag for redirects (e.g., page-client.tsx at line 30) will redirect prematurely before enrollment data finishes loading. This causes users to be redirected away from pages they should have access to.

Consider one of these solutions:

Solution 1: Return showDetailedAnalytics as true during loading to avoid premature redirects:

  return {
    programEnrollment,
    showDetailedAnalytics:
+     status === "loading" ||
      programSlug !== "perplexity" ||
      (programEnrollment?.totalCommissions ?? 0) >=
        LARGE_PROGRAM_MIN_TOTAL_COMMISSIONS_CENTS,
    error,
    loading: status === "loading" || isLoading,
  };

Solution 2: Make consumers check the loading state before using showDetailedAnalytics. However, this is error-prone as it requires every consumer to remember to check loading.

In apps/web/lib/swr/use-program-enrollment.ts around lines 35 to 38,
showDetailedAnalytics currently treats an undefined programEnrollment as
totalCommissions=0 which causes premature false during loading; change the logic
so that while enrollment data is loading you return true for
showDetailedAnalytics (e.g., detect loading/undefined enrollment and, for
programSlug "perplexity", short-circuit to true) so consumers don’t redirect
before data arrives; ensure once the real enrollment is loaded the original
commission threshold check is applied.