chore(webapp): scope route effect synchronization

apps/webapp/app/routes/_app.orgs.$organizationSlug.projects.$projectParam.env.$envParam.apikeys/route.tsx

@@ -617,6 +617,7 @@ function NewApiKeyDialog({
     }
 
     if (actionData?.ok && actionData.action === "create") {
+      // oxlint-disable-next-line react/react-compiler -- This effect intentionally synchronizes route state after an external or lifecycle change.
       setCreatedApiKey(actionData.apiKey);
     } else if (actionData && !actionData.ok) {
       setShowError(true);

apps/webapp/app/routes/_app.orgs.$organizationSlug.projects.$projectParam.env.$envParam.branches/route.tsx

@@ -667,6 +667,7 @@ function PurchaseBranchesModal({
       "ok" in data &&
       data.ok
     ) {
+      // oxlint-disable-next-line react/react-compiler -- This effect intentionally synchronizes route state after an external or lifecycle change.
       setOpen(false);
     }
   }, [fetcher.state, fetcher.data]);

apps/webapp/app/routes/_app.orgs.$organizationSlug.projects.$projectParam.env.$envParam.concurrency/route.tsx

@@ -648,6 +648,7 @@ function PurchaseConcurrencyModal({
   const [open, setOpen] = useState(false);
   useEffect(() => {
     if (purchaseSucceeded) {
+      // oxlint-disable-next-line react/react-compiler -- This effect intentionally synchronizes route state after an external or lifecycle change.
       setOpen(false);
       setSearchParams((s) => {
         s.delete("success");

apps/webapp/app/routes/_app.orgs.$organizationSlug.projects.$projectParam.env.$envParam.dashboards.$dashboardKey/route.tsx

@@ -444,6 +444,7 @@ function useContainerWidth(initialWidth = 1280) {
 
   useEffect(() => {
     measureWidth();
+    // oxlint-disable-next-line react/react-compiler -- This effect intentionally synchronizes route state after an external or lifecycle change.
     setMounted(true);
 
     const element = containerRef.current;

apps/webapp/app/routes/_app.orgs.$organizationSlug.projects.$projectParam.env.$envParam.dashboards.custom.$dashboardId/route.tsx

@@ -677,6 +677,7 @@ function RenameDashboardDialog({ title }: { title: string }) {
   // Close dialog when navigation completes
   useEffect(() => {
     if (navigation.state === "idle") {
+      // oxlint-disable-next-line react/react-compiler -- This effect intentionally synchronizes route state after an external or lifecycle change.
       setIsOpen(false);
     }
   }, [navigation.state]);
@@ -750,6 +751,7 @@ function DeleteDashboardDialog({ title }: { title: string }) {
   // Close dialog when navigation completes
   useEffect(() => {
     if (navigation.state === "idle") {
+      // oxlint-disable-next-line react/react-compiler -- This effect intentionally synchronizes route state after an external or lifecycle change.
       setIsOpen(false);
     }
   }, [navigation.state]);

apps/webapp/app/routes/_app.orgs.$organizationSlug.projects.$projectParam.env.$envParam.deployments.$deploymentParam/route.tsx

@@ -199,6 +199,7 @@ export default function Page() {
 
     const abortController = new AbortController();
 
+    // oxlint-disable-next-line react/react-compiler -- This effect intentionally synchronizes route state after an external or lifecycle change.
     setLogs([]);
     setStreamError(null);
     setIsStreaming(true);

apps/webapp/app/routes/_app.orgs.$organizationSlug.projects.$projectParam.env.$envParam.environment-variables/route.tsx

@@ -451,6 +451,7 @@ function EnvironmentVariablesListPage({
   const [isVirtualized, setIsVirtualized] = useState(false);
 
   useLayoutEffect(() => {
+    // oxlint-disable-next-line react/react-compiler -- This effect intentionally synchronizes route state after an external or lifecycle change.
     setIsVirtualized(shouldVirtualize);
   }, [shouldVirtualize]);
 
@@ -815,6 +816,7 @@ function EditEnvironmentVariablePanel({
   // Close dialog on successful submission
   useEffect(() => {
     if (lastSubmission?.success && fetcher.state === "idle") {
+      // oxlint-disable-next-line react/react-compiler -- This effect intentionally synchronizes route state after an external or lifecycle change.
       setIsOpen(false);
     }
   }, [lastSubmission?.success, fetcher.state]);

apps/webapp/app/routes/_app.orgs.$organizationSlug.projects.$projectParam.env.$envParam.logs/route.tsx

@@ -318,6 +318,7 @@ function LogsList({
 
   // Clear accumulated logs immediately when filters change (for instant visual feedback)
   useEffect(() => {
+    // oxlint-disable-next-line react/react-compiler -- This effect intentionally synchronizes route state after an external or lifecycle change.
     setAccumulatedLogs([]);
     setNextCursor(undefined);
     // Preserve log selection from URL param, clear if not present
@@ -327,6 +328,7 @@ function LogsList({
 
   // Populate accumulated logs when new data arrives
   useEffect(() => {
+    // oxlint-disable-next-line react/react-compiler -- This effect intentionally synchronizes route state after an external or lifecycle change.
     setAccumulatedLogs(list.logs);
     setNextCursor(list.pagination.next);
   }, [list.logs, list.pagination.next]);

apps/webapp/app/routes/_app.orgs.$organizationSlug.projects.$projectParam.env.$envParam.prompts.$promptSlug/route.tsx

@@ -909,6 +909,7 @@ function OverrideDialog({
   // Reset when dialog opens
   useEffect(() => {
     if (open) {
+      // oxlint-disable-next-line react/react-compiler -- This effect intentionally synchronizes route state after an external or lifecycle change.
       setEditedContent(content);
       setCommitMessage("");
       setModel(currentOverrideModel ?? prompt.defaultModel ?? "");
@@ -1332,6 +1333,7 @@ function GenerationsTab({
   // Append fetched rows when fetcher completes
   useEffect(() => {
     if (fetcher.data && fetcher.state === "idle") {
+      // oxlint-disable-next-line react/react-compiler -- This effect intentionally synchronizes route state after an external or lifecycle change.
       setGenerations((prev) => {
         const existingIds = new Set(prev.map((g) => g.span_id));
         const newRows = fetcher.data!.generations.filter((g) => !existingIds.has(g.span_id));
@@ -1422,6 +1424,7 @@ function GenerationsTab({
   const [showSpinner, setShowSpinner] = useState(false);
   useEffect(() => {
     if (!isLoadingMore) {
+      // oxlint-disable-next-line react/react-compiler -- This effect intentionally synchronizes route state after an external or lifecycle change.
       setShowSpinner(false);
       return;
     }

apps/webapp/app/routes/_app.orgs.$organizationSlug.projects.$projectParam.env.$envParam.queues/route.tsx

@@ -1035,6 +1035,7 @@ function EnvironmentPauseResumeButton({
 
   useEffect(() => {
     if (navigation.state === "loading" || navigation.state === "idle") {
+      // oxlint-disable-next-line react/react-compiler -- This effect intentionally synchronizes route state after an external or lifecycle change.
       setIsOpen(false);
     }
   }, [navigation.state]);

apps/webapp/app/routes/_app.orgs.$organizationSlug.projects.$projectParam.env.$envParam.queues_.$queueParam/route.tsx

@@ -898,6 +898,7 @@ function useConcurrencyKeys(opts: {
   }, [body]);
 
   useEffect(() => {
+    // oxlint-disable-next-line react/react-compiler -- This effect intentionally synchronizes route state after an external or lifecycle change.
     load();
     return () => abortRef.current?.abort();
   }, [load]);

apps/webapp/app/routes/_app.orgs.$organizationSlug.projects.$projectParam.env.$envParam.runs.$runParam/route.tsx

@@ -1264,6 +1264,7 @@ function TimelineView({
   const [duration, setDuration] = useState(queueAdjustedNs(totalDuration, queuedDuration));
   useEffect(() => {
     if (rootSpanStatus !== "executing" || !rootStartedAt) {
+      // oxlint-disable-next-line react/react-compiler -- This effect intentionally synchronizes route state after an external or lifecycle change.
       setDuration(queueAdjustedNs(totalDuration, queuedDuration));
       return;
     }

apps/webapp/app/routes/_app.orgs.$organizationSlug.projects.$projectParam.env.$envParam.runs._index/useRunsLiveReload.ts

@@ -192,6 +192,7 @@ export function useRunsLiveReload({
   // Single reset path: new loader data or changed filters re-baseline both the
   // visible rows and new-run tracking.
   useEffect(() => {
+    // oxlint-disable-next-line react/react-compiler -- This effect intentionally synchronizes route state after an external or lifecycle change.
     setVisibleRuns(runs);
     resetNewRunsTracking();
   }, [runs, searchKeyWithoutPagination, resetNewRunsTracking]);
@@ -202,6 +203,7 @@ export function useRunsLiveReload({
     const data = runsPollFetcher.data;
     if (!data?.runs.length) return;
 
+    // oxlint-disable-next-line react/react-compiler -- This effect intentionally synchronizes route state after an external or lifecycle change.
     setVisibleRuns((currentRuns) => patchVisibleRunsWithLiveUpdates(currentRuns, data.runs));
   }, [runsPollFetcher.data]);
 

apps/webapp/app/routes/_app.orgs.$organizationSlug.projects.$projectParam.env.$envParam.settings.integrations/route.tsx

@@ -261,6 +261,7 @@ export default function IntegrationsSettingsPage() {
       if (onboardingData && vercelFetcher.state === "idle") {
         // Data is loaded, ensure modal is open (query param takes precedence)
         if (!isModalOpen) {
+          // oxlint-disable-next-line react/react-compiler -- This effect intentionally synchronizes route state after an external or lifecycle change.
           openVercelOnboarding();
         }
       } else if (vercelFetcher.state === "idle" && !hasVercelFetcherData) {
@@ -289,6 +290,7 @@ export default function IntegrationsSettingsPage() {
     if (hasQueryParam && !isModalOpen) {
       // Query param is present but modal is closed, open it
       // This ensures the modal stays open during the onboarding flow
+      // oxlint-disable-next-line react/react-compiler -- This effect intentionally synchronizes route state after an external or lifecycle change.
       openVercelOnboarding();
     }
   }, [hasQueryParam, isModalOpen, openVercelOnboarding]);
@@ -298,6 +300,7 @@ export default function IntegrationsSettingsPage() {
     if (hasQueryParam && onboardingData && vercelFetcher.state === "idle") {
       // Data loaded and query param is present, ensure modal is open
       if (!isModalOpen) {
+        // oxlint-disable-next-line react/react-compiler -- This effect intentionally synchronizes route state after an external or lifecycle change.
         openVercelOnboarding();
       }
     }
@@ -442,6 +445,7 @@ function BuildSettingsForm({ buildSettings }: { buildSettings: BuildSettings })
       buildSettingsValues.installCommand !== (buildSettings?.installCommand || "") ||
       buildSettingsValues.triggerConfigFilePath !== (buildSettings?.triggerConfigFilePath || "") ||
       buildSettingsValues.useNativeBuildServer !== nativeBuildServerEnabled;
+    // oxlint-disable-next-line react/react-compiler -- This effect intentionally synchronizes route state after an external or lifecycle change.
     setHasBuildSettingsChanges(hasChanges);
   }, [buildSettingsValues, buildSettings, nativeBuildServerEnabled]);
 

apps/webapp/app/routes/_app.orgs.$organizationSlug.projects.$projectParam.env.$envParam.tasks.scheduled.$taskParam/route.tsx

@@ -651,6 +651,7 @@ function ScheduleSheet({
 
   // Always reopen in inspect mode.
   useEffect(() => {
+    // oxlint-disable-next-line react/react-compiler -- This effect intentionally synchronizes route state after an external or lifecycle change.
     setMode("inspect");
   }, [openScheduleId]);
 
@@ -693,6 +694,7 @@ function ScheduleSheet({
     handledUpdateRef.current = data;
     if (data.ok) {
       toast.success(data.message ?? "Schedule updated");
+      // oxlint-disable-next-line react/react-compiler -- This effect intentionally synchronizes route state after an external or lifecycle change.
       setMode("inspect");
       if (detailPath) loadScheduleDetail(detailPath);
       revalidator.revalidate();

apps/webapp/app/routes/_app.orgs.$organizationSlug.projects.$projectParam.env.$envParam.test.tasks.$taskParam/route.tsx

@@ -1585,6 +1585,7 @@ function RunTemplatesPopover({
 
   useEffect(() => {
     if (lastSubmission && "success" in lastSubmission && lastSubmission.success === true) {
+      // oxlint-disable-next-line react/react-compiler -- This effect intentionally synchronizes route state after an external or lifecycle change.
       setIsDeleteDialogOpen(false);
     }
   }, [lastSubmission]);
@@ -1772,6 +1773,7 @@ function CreateTemplateModal({
 
   useEffect(() => {
     if (lastSubmission && "success" in lastSubmission && lastSubmission.success === true) {
+      // oxlint-disable-next-line react/react-compiler -- This effect intentionally synchronizes route state after an external or lifecycle change.
       setIsModalOpen(false);
       setShowCreatedSuccessMessage(true);
       clearTimeout(successMessageTimeoutRef.current);

apps/webapp/app/routes/_app.orgs.$organizationSlug.projects.$projectParam.env.$envParam.webhooks.endpoints.$endpointParam/route.tsx

@@ -614,6 +614,7 @@ function SetSecretDialog({
   // Close on a successful save; the loader revalidates and the state flips to "Set".
   useEffect(() => {
     if (fetcher.state === "idle" && fetcher.data?.success) {
+      // oxlint-disable-next-line react/react-compiler -- This effect intentionally synchronizes route state after an external or lifecycle change.
       setOpen(false);
     }
   }, [fetcher.state, fetcher.data]);

apps/webapp/app/routes/_app.orgs.$organizationSlug.settings.sso/route.tsx

@@ -375,6 +375,7 @@ function useOverrideDraft<T>(serverValue: T): {
   const [override, setOverride] = useState<{ value: T } | null>(null);
   useEffect(() => {
     // Server matches the pending edit → clear the override.
+    // oxlint-disable-next-line react/react-compiler -- This effect intentionally synchronizes route state after an external or lifecycle change.
     setOverride((current) => (current && Object.is(current.value, serverValue) ? null : current));
   }, [serverValue]);
   const value = override ? override.value : serverValue;
@@ -419,6 +420,7 @@ export default function Page() {
 
   useEffect(() => {
     if (portalFetcher.data?.ok && portalFetcher.data.url) {
+      // oxlint-disable-next-line react/react-compiler -- This effect intentionally synchronizes route state after an external or lifecycle change.
       setPortalUrl(portalFetcher.data.url);
     }
   }, [portalFetcher.data]);
@@ -917,6 +919,7 @@ function DirectorySyncSection({
   // server value so polled-in groups appear and matched overrides drop.
   const [draftGroupRoles, setDraftGroupRoles] = useState<Record<string, string>>({});
   useEffect(() => {
+    // oxlint-disable-next-line react/react-compiler -- This effect intentionally synchronizes route state after an external or lifecycle change.
     setDraftGroupRoles((current) => {
       const next: Record<string, string> = {};
       for (const g of directorySync.groups) {

apps/webapp/app/routes/_app.orgs.$organizationSlug.settings.team/route.tsx

@@ -985,6 +985,7 @@ export function PurchaseSeatsModal({
       "ok" in data &&
       data.ok
     ) {
+      // oxlint-disable-next-line react/react-compiler -- This effect intentionally synchronizes route state after an external or lifecycle change.
       setOpen(false);
     }
   }, [fetcher.state, fetcher.data]);

apps/webapp/app/routes/_app.orgs.$organizationSlug_.projects.new/route.tsx

@@ -360,6 +360,7 @@ export default function Page() {
 
   useEffect(() => {
     const nonOther = workingOnOptions.filter((o) => o !== WORKING_ON_OTHER);
+    // oxlint-disable-next-line react/react-compiler -- This effect intentionally synchronizes route state after an external or lifecycle change.
     setShuffledWorkingOn([...shuffleArray(nonOther), WORKING_ON_OTHER]);
 
     const nonOtherGoals = goalOptions.filter((o) => o !== GOALS_OTHER);

apps/webapp/app/routes/account._index/route.tsx

@@ -221,6 +221,7 @@ export default function Page() {
   const [contrastPreview, setContrastPreview] = useState(contrast);
   useEffect(() => {
     if (contrastFetcher.state === "idle") {
+      // oxlint-disable-next-line react/react-compiler -- This effect intentionally synchronizes route state after an external or lifecycle change.
       setContrastPreview(contrast);
       document.documentElement.style.setProperty("--theme-contrast", String(contrast / 100));
     }

apps/webapp/app/routes/admin.feature-flags.tsx

@@ -182,12 +182,14 @@ export default function AdminFeatureFlagsRoute() {
         editable[key] = value;
       }
     }
+    // oxlint-disable-next-line react/react-compiler -- This effect intentionally synchronizes route state after an external or lifecycle change.
     setValues({ ...editable });
     setInitialValues({ ...editable });
   }, [globalFlags, unlocked]);
 
   useEffect(() => {
     if (saveFetcher.data?.success) {
+      // oxlint-disable-next-line react/react-compiler -- This effect intentionally synchronizes route state after an external or lifecycle change.
       setSaveError(null);
       setConfirmOpen(false);
     } else if (saveFetcher.data?.error) {

apps/webapp/app/routes/admin.queue-metrics.tsx

@@ -65,6 +65,7 @@ export default function AdminQueueMetricsRoute() {
   const handledSaveDataRef = useRef(saveFetcher.data);
 
   useEffect(() => {
+    // oxlint-disable-next-line react/react-compiler -- This effect intentionally synchronizes route state after an external or lifecycle change.
     setEnabled(controls.enabled);
     setSampleRate(String(controls.sampleRate));
   }, [controls.enabled, controls.sampleRate]);
@@ -76,6 +77,7 @@ export default function AdminQueueMetricsRoute() {
     handledSaveDataRef.current = saveFetcher.data;
 
     if (saveFetcher.data.success) {
+      // oxlint-disable-next-line react/react-compiler -- This effect intentionally synchronizes route state after an external or lifecycle change.
       setError(null);
       revalidate();
     } else if (saveFetcher.data.error) {

apps/webapp/app/routes/confirm-basic-details.tsx

@@ -228,6 +228,7 @@ export default function Page() {
 
   useEffect(() => {
     const nonOtherReferral = referralSourceOptions.filter((r) => r !== "Other");
+    // oxlint-disable-next-line react/react-compiler -- This effect intentionally synchronizes route state after an external or lifecycle change.
     setShuffledReferralSources([...shuffleArray(nonOtherReferral), "Other"]);
 
     const nonOtherRoles = roleOptions.filter((r) => r !== "Other");

apps/webapp/app/routes/login.mfa/route.tsx

@@ -197,6 +197,7 @@ export default function LoginMfaPage() {
   // Reset hideError when a new error appears
   React.useEffect(() => {
     if (rawMfaError) {
+      // oxlint-disable-next-line react/react-compiler -- This effect intentionally synchronizes route state after an external or lifecycle change.
       setHideError(false);
     }
   }, [rawMfaError]);

apps/webapp/app/routes/resources.branches.create.tsx

@@ -99,6 +99,7 @@ export function NewBranchPanel({
         s.delete("dialogClosed");
         return s;
       });
+      // oxlint-disable-next-line react/react-compiler -- This effect intentionally synchronizes route state after an external or lifecycle change.
       setIsOpen(false);
     }
   }, [searchParams, setSearchParams]);

apps/webapp/app/routes/resources.orgs.$organizationSlug.projects.$projectParam.env.$envParam.github.tsx

@@ -442,6 +442,7 @@ export function ConnectGitHubRepoModal({
     const params = new URLSearchParams(searchParams);
 
     if (params.get("openGithubRepoModal") === "1") {
+      // oxlint-disable-next-line react/react-compiler -- This effect intentionally synchronizes route state after an external or lifecycle change.
       setIsModalOpen(true);
       params.delete("openGithubRepoModal");
       setSearchParams(params);
@@ -450,6 +451,7 @@ export function ConnectGitHubRepoModal({
 
   useEffect(() => {
     if (lastSubmission && "success" in lastSubmission && lastSubmission.success === true) {
+      // oxlint-disable-next-line react/react-compiler -- This effect intentionally synchronizes route state after an external or lifecycle change.
       setIsModalOpen(false);
     }
   }, [lastSubmission]);
@@ -810,6 +812,7 @@ export function ConnectedGitHubRepoForm({
       gitSettingsValues.stagingBranch !==
         (connectedGitHubRepo.branchTracking?.staging?.branch || "") ||
       gitSettingsValues.previewDeploymentsEnabled !== connectedGitHubRepo.previewDeploymentsEnabled;
+    // oxlint-disable-next-line react/react-compiler -- This effect intentionally synchronizes route state after an external or lifecycle change.
     setHasGitSettingsChanges(hasChanges);
   }, [gitSettingsValues, connectedGitHubRepo]);
 

apps/webapp/app/routes/resources.orgs.$organizationSlug.projects.$projectParam.env.$envParam.runs.$runParam.streams.$streamKey/route.tsx

@@ -540,6 +540,7 @@ export function useRealtimeStream(resourcePath: string, startIndex?: number) {
   const [isConnected, setIsConnected] = useState(false);
 
   useEffect(() => {
+    // oxlint-disable-next-line react/react-compiler -- This effect intentionally synchronizes route state after an external or lifecycle change.
     setChunks([]);
     setError(null);
 

apps/webapp/app/routes/resources.orgs.$organizationSlug.projects.$projectParam.env.$envParam.vercel.tsx

@@ -1118,6 +1118,7 @@ function VercelSettingsPanel({
   useEffect(() => {
     if (!data?.authInvalid && !hasError && !data && !hasFetched) {
       load(vercelResourcePath(organizationSlug, projectSlug, environmentSlug));
+      // oxlint-disable-next-line react/react-compiler -- This effect intentionally synchronizes route state after an external or lifecycle change.
       setHasFetched(true);
     }
   }, [

apps/webapp/app/routes/resources.orgs.$organizationSlug.select-plan.tsx

@@ -327,6 +327,7 @@ export function TierFree({
   const [isLackingFeaturesChecked, setIsLackingFeaturesChecked] = useState(false);
 
   useEffect(() => {
+    // oxlint-disable-next-line react/react-compiler -- This effect intentionally synchronizes route state after an external or lifecycle change.
     setIsDialogOpen(false);
   }, [subscription]);
 
@@ -493,6 +494,7 @@ export function TierHobby({
   const [isDialogOpen, setIsDialogOpen] = useState(false);
 
   useEffect(() => {
+    // oxlint-disable-next-line react/react-compiler -- This effect intentionally synchronizes route state after an external or lifecycle change.
     setIsDialogOpen(false);
   }, [subscription]);
 
@@ -635,6 +637,7 @@ export function TierPro({
   const [isDialogOpen, setIsDialogOpen] = useState(false);
 
   useEffect(() => {
+    // oxlint-disable-next-line react/react-compiler -- This effect intentionally synchronizes route state after an external or lifecycle change.
     setIsDialogOpen(false);
   }, [subscription]);
 

apps/webapp/app/routes/vercel.onboarding.tsx

@@ -328,6 +328,7 @@ export default function VercelOnboardingPage() {
   // Reset isInstalling when navigation returns to idle (e.g. on error)
   useEffect(() => {
     if (navigation.state === "idle" && isInstalling) {
+      // oxlint-disable-next-line react/react-compiler -- This effect intentionally synchronizes route state after an external or lifecycle change.
       setIsInstalling(false);
     }
   }, [navigation.state, isInstalling]);