feat(web): seed permission sync banner with server-side initial state

Refactors getPermissionSyncStatus into a reusable server action and
passes the result as initialData to the banner query, eliminating the
loading flash on first render. Also fixes hasPendingFirstSync to treat
accounts with no sync jobs yet as pending.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: brendan-kellam

packages/web/src/app/[domain]/components/permissionSyncBanner.tsx

@@ -11,7 +11,11 @@ import { usePrevious } from "@uidotdev/usehooks";
 
 const POLL_INTERVAL_MS = 5000;
 
-export function PermissionSyncBanner() {
+interface PermissionSyncBannerProps {
+    initialHasPendingFirstSync: boolean;
+}
+
+export function PermissionSyncBanner({ initialHasPendingFirstSync }: PermissionSyncBannerProps) {
     const router = useRouter();
 
     const { data: hasPendingFirstSync, isError, isPending } = useQuery({
@@ -25,6 +29,9 @@ export function PermissionSyncBanner() {
             // Keep polling while sync is in progress, stop when done
             return hasPendingFirstSync ? POLL_INTERVAL_MS : false;
         },
+        initialData: {
+            hasPendingFirstSync: initialHasPendingFirstSync,
+        }
     });
 
     const previousHasPendingFirstSync = usePrevious(hasPendingFirstSync);

packages/web/src/app/[domain]/layout.tsx

@@ -26,6 +26,7 @@ import { UpgradeToast } from "./components/upgradeToast";
 import { getLinkedAccountProviderStates } from "@/ee/features/permissionSyncing/actions";
 import { LinkAccounts } from "@/ee/features/permissionSyncing/components/linkAccounts";
 import { PermissionSyncBanner } from "./components/permissionSyncBanner";
+import { getPermissionSyncStatus } from "../api/(server)/ee/permissionSyncStatus/route";
 
 interface LayoutProps {
     children: React.ReactNode,
@@ -190,12 +191,18 @@ export default async function Layout(props: LayoutProps) {
         )
     }
     const isPermissionSyncBannerVisible = session && hasEntitlement("permission-syncing");
+    const hasPendingFirstSync = isPermissionSyncBannerVisible ? (await getPermissionSyncStatus()) : null;
 
     return (
         <SyntaxGuideProvider>
             {
                 isPermissionSyncBannerVisible ? (
-                    <PermissionSyncBanner />
+                    <PermissionSyncBanner
+                        initialHasPendingFirstSync={(isServiceError(hasPendingFirstSync) || hasPendingFirstSync === null) ?
+                            false :
+                            hasPendingFirstSync.hasPendingFirstSync
+                        }
+                    />
                 ) : null
             }
             {children}

packages/web/src/app/api/(server)/ee/permissionSyncStatus/route.ts

@@ -1,13 +1,14 @@
 'use server';
 
 import { apiHandler } from "@/lib/apiHandler";
-import { serviceErrorResponse } from "@/lib/serviceError";
+import { ServiceError, serviceErrorResponse } from "@/lib/serviceError";
 import { isServiceError } from "@/lib/utils";
 import { withAuthV2 } from "@/withAuthV2";
-import { getEntitlements } from "@sourcebot/shared";
+import { env, getEntitlements } from "@sourcebot/shared";
 import { AccountPermissionSyncJobStatus } from "@sourcebot/db";
 import { StatusCodes } from "http-status-codes";
 import { ErrorCode } from "@/lib/errorCodes";
+import { sew } from "@/actions";
 
 export interface PermissionSyncStatusResponse {
     hasPendingFirstSync: boolean;
@@ -18,16 +19,28 @@ export interface PermissionSyncStatusResponse {
  * synced for the first time.
  */
 export const GET = apiHandler(async () => {
-    const entitlements = getEntitlements();
-    if (!entitlements.includes('permission-syncing')) {
-        return serviceErrorResponse({
-            statusCode: StatusCodes.FORBIDDEN,
-            errorCode: ErrorCode.INSUFFICIENT_PERMISSIONS,
-            message: "Permission syncing is not enabled for your license",
-        });
+    const result = await getPermissionSyncStatus();
+
+    if (isServiceError(result)) {
+        return serviceErrorResponse(result);
     }
 
-    const result = await withAuthV2(async ({ prisma, user }) => {
+    return Response.json(result, { status: StatusCodes.OK });
+});
+
+
+export const getPermissionSyncStatus = async (): Promise<PermissionSyncStatusResponse | ServiceError> => sew(async () =>
+    withAuthV2(async ({ prisma, user }) => {
+        const entitlements = getEntitlements();
+        if (!entitlements.includes('permission-syncing')) {
+            return {
+                statusCode: StatusCodes.FORBIDDEN,
+                errorCode: ErrorCode.INSUFFICIENT_PERMISSIONS,
+                message: "Permission syncing is not enabled for your license",
+            } satisfies ServiceError;
+        }
+
+
         const accounts = await prisma.account.findMany({
             where: {
                 userId: user.id,
@@ -46,18 +59,15 @@ export const GET = apiHandler(async () => {
             AccountPermissionSyncJobStatus.IN_PROGRESS
         ];
 
-        const hasPendingFirstSync = accounts.some(account =>
-            account.permissionSyncedAt === null &&
-            account.permissionSyncJobs.length > 0 &&
-            activeStatuses.includes(account.permissionSyncJobs[0].status)
-        );
+        const hasPendingFirstSync = env.EXPERIMENT_EE_PERMISSION_SYNC_ENABLED === 'true' &&
+            accounts.some(account =>
+                account.permissionSyncedAt === null &&
+                // @note: to handle the case where the permission sync job
+                // has not yet been scheduled for a new account, we consider
+                // accounts with no permission sync jobs as having a pending first sync.
+                (account.permissionSyncJobs.length === 0 || (account.permissionSyncJobs.length > 0 && activeStatuses.includes(account.permissionSyncJobs[0].status)))
+            )
 
         return { hasPendingFirstSync } satisfies PermissionSyncStatusResponse;
-    });
-
-    if (isServiceError(result)) {
-        return serviceErrorResponse(result);
-    }
-
-    return Response.json(result, { status: StatusCodes.OK });
-});
+    })
+)