feat(lint): enable useExhaustiveDependencies rule with suppressions

Enable the biome correctness/useExhaustiveDependencies lint rule (was previously "off"). All existing violations are suppressed with biome-ignore comments added via `biome lint --suppress`. Co-authored-by: Claude <noreply@anthropic.com> https://claude.ai/code/session_01EHt5dM7Rts48s8HzcCaZCz
artsy · MounirDhahri · Feb 18, 2026 · Feb 19, 2026 · Feb 19, 2026 · Feb 19, 2026
commit bd8e6524be42d9f24c7fe8883bfce05a676e29a0
diff --git a/biome.json b/biome.json
@@ -41,7 +41,7 @@
         "noUnusedFunctionParameters": "off",
         "noUnusedImports": "off",
         "noUnusedVariables": "off",
-        "useExhaustiveDependencies": "off",
+        "useExhaustiveDependencies": "warn",
         "useHookAtTopLevel": "off",
         "useJsxKeyInIterable": "off",
         "useParseIntRadix": "off"

diff --git a/src/Apps/Article/Components/ArticleHTML.tsx b/src/Apps/Article/Components/ArticleHTML.tsx
@@ -87,6 +87,7 @@ export const ArticleHTML: FC<React.PropsWithChildren<ArticleHTMLProps>> = ({
 
   const [transformed, setTransformed] = useState<string | null>(null)
 
+  // biome-ignore lint/correctness/useExhaustiveDependencies: ignored using `--suppress`
   useEffect(() => {
     // Relies on the DOMParser global being available in the browser.
     import("@artsy/react-html-parser").then(({ default: reactHtmlParser }) => {

diff --git a/src/Apps/Article/Components/ArticleZoomGallery/ArticleZoomGalleryFigure.tsx b/src/Apps/Article/Components/ArticleZoomGallery/ArticleZoomGalleryFigure.tsx
@@ -179,6 +179,7 @@ const Image: FC<
 
   const ref = useRef<HTMLImageElement>(null)
 
+  // biome-ignore lint/correctness/useExhaustiveDependencies: ignored using `--suppress`
   useEffect(() => {
     if (ref.current?.complete) {
       setMode("Ready")

diff --git a/src/Apps/Artist/Routes/AuctionResults/Components/AuctionFilterMobileActionSheet.tsx b/src/Apps/Artist/Routes/AuctionResults/Components/AuctionFilterMobileActionSheet.tsx
@@ -32,6 +32,7 @@ export const AuctionFilterMobileActionSheet: FC<
     filterContext.stagedFilters,
   )
 
+  // biome-ignore lint/correctness/useExhaustiveDependencies: ignored using `--suppress`
   useEffect(() => {
     // While mobile sheet is mounted, the effect of the user's filter selections
     // should be merely staged until the Apply button is pressed, rather than

diff --git a/src/Apps/Artist/Routes/AuctionResults/Components/AuctionFilters/KeywordFilter.tsx b/src/Apps/Artist/Routes/AuctionResults/Components/AuctionFilters/KeywordFilter.tsx
@@ -15,14 +15,16 @@ export const KeywordFilter: React.FC<React.PropsWithChildren<unknown>> = () => {
     filterContext.setFilter?.("keyword", text)
   }
 
+  // biome-ignore lint/correctness/useExhaustiveDependencies: ignored using `--suppress`
   const handleChangeText = useMemo(
     () => debounce(updateKeywordFilter, DEBOUNCE_DELAY),
     // FIXME:
     [filterContext],
   )
 
   // Stop the invocation of the debounced function after unmounting
-  useEffect(() => {
+  // biome-ignore lint/correctness/useExhaustiveDependencies: ignored using `--suppress`
+    useEffect(() => {
     return () => handleChangeText?.cancel?.()
     // FIXME:
   }, [])

diff --git a/src/Apps/Artwork/ArtworkApp.tsx b/src/Apps/Artwork/ArtworkApp.tsx
@@ -105,7 +105,8 @@ export const ArtworkApp: React.FC<React.PropsWithChildren<Props>> = props => {
 
   // If the user is expecting a partner offer, require login and remove
   // the query param from the URL after login.
-  useEffect(() => {
+  // biome-ignore lint/correctness/useExhaustiveDependencies: ignored using `--suppress`
+      useEffect(() => {
     const expectingPartnerOffer = !!match?.location?.query?.partner_offer_id
     const isLoggedIn = !!me
 

diff --git a/src/Apps/Artwork/Components/ArtworkErrorApp/ArtworkErrorApp.tsx b/src/Apps/Artwork/Components/ArtworkErrorApp/ArtworkErrorApp.tsx
@@ -46,6 +46,7 @@ export const ArtworkErrorApp: React.FC<
     }
   }, [])
 
+  // biome-ignore lint/correctness/useExhaustiveDependencies: ignored using `--suppress`
   useEffect(() => {
     trackPageview()
   }, [])

diff --git a/src/Apps/Artwork/Components/ArtworkSidebar/ArtworkSidebarAuctionInfoPolling.tsx b/src/Apps/Artwork/Components/ArtworkSidebar/ArtworkSidebarAuctionInfoPolling.tsx
@@ -31,6 +31,7 @@ export const ArtworkSidebarAuctionPolling: React.FC<
   const isMounted = useRef(false)
   const tracking = useTracking()
 
+  // biome-ignore lint/correctness/useExhaustiveDependencies: ignored using `--suppress`
   useEffect(() => {
     if (isMounted.current) {
       setCurrentBidChanged(true)

diff --git a/src/Apps/Artwork/Components/ArtworkVideoPlayer/ArtworkVideoPlayer.tsx b/src/Apps/Artwork/Components/ArtworkVideoPlayer/ArtworkVideoPlayer.tsx
@@ -59,6 +59,7 @@ const ArtworkVideoPlayer: FC<
     }, 0)
   }
 
+  // biome-ignore lint/correctness/useExhaustiveDependencies: ignored using `--suppress`
   useEffect(() => {
     window.addEventListener("blur", trackClickedPlayVideo)
 

diff --git a/src/Apps/Auction/AuctionApp.tsx b/src/Apps/Auction/AuctionApp.tsx
@@ -54,6 +54,7 @@ export const AuctionApp: React.FC<React.PropsWithChildren<AuctionAppProps>> = ({
 
   const isFullBleedHeaderFixed = !cascadingEndTimeIntervalMinutes
 
+  // biome-ignore lint/correctness/useExhaustiveDependencies: ignored using `--suppress`
   useEffect(() => {
     tracking.auctionPageView({ sale, me })
   }, [])

diff --git a/src/Apps/Auction/Routes/AuctionConfirmRegistrationRoute.tsx b/src/Apps/Auction/Routes/AuctionConfirmRegistrationRoute.tsx
@@ -68,7 +68,8 @@ const AuctionConfirmRegistrationRoute: React.FC<
   }
 
   // Track page view or redirect
-  useEffect(() => {
+  // biome-ignore lint/correctness/useExhaustiveDependencies: ignored using `--suppress`
+    useEffect(() => {
     if (redirectToSaleHome(sale)) {
       router.replace(`/auction/${sale.slug}`)
     } else if (!me.hasQualifiedCreditCards) {

diff --git a/src/Apps/Auction/Routes/AuctionRegistrationRoute.tsx b/src/Apps/Auction/Routes/AuctionRegistrationRoute.tsx
@@ -43,7 +43,8 @@ const AuctionRegistrationRoute: React.FC<
   }
 
   // Track page view or redirect
-  useEffect(() => {
+  // biome-ignore lint/correctness/useExhaustiveDependencies: ignored using `--suppress`
+    useEffect(() => {
     if (redirectToSaleHome(sale)) {
       router.replace(`/auction/${sale.slug}`)
     } else if (me.hasQualifiedCreditCards) {

diff --git a/src/Apps/Auction/Routes/Bid/Components/PricingTransparency.tsx b/src/Apps/Auction/Routes/Bid/Components/PricingTransparency.tsx
@@ -95,7 +95,8 @@ export const PricingTransparencyQueryRenderer = ({
   // when the url changes after user places a successful bid and we redirect
   // back to artwork/id. If / when we remove the transition to the auction page
   // from artwork/id we can remove this hack.
-  const variables = useMemo(() => {
+  // biome-ignore lint/correctness/useExhaustiveDependencies: ignored using `--suppress`
+          const variables = useMemo(() => {
     return {
       saleId,
       artworkId,

diff --git a/...ectorProfile/Routes/MyCollection/EditArtwork/Components/MyCollectionArtworkFormImages.tsx b/...ectorProfile/Routes/MyCollection/EditArtwork/Components/MyCollectionArtworkFormImages.tsx
@@ -62,6 +62,7 @@ export const MyCollectionArtworkFormImages: React.FC<
     return photos.filter(c => !(c.geminiToken || c.url) && !c.loading)
   }
 
+  // biome-ignore lint/correctness/useExhaustiveDependencies: ignored using `--suppress`
   useEffect(() => {
     const imagesToUpload = getImagesToUpload(values.newPhotos)
 

diff --git a/src/Apps/CollectorProfile/Routes/MyCollection/EditArtwork/MyCollectionCreateArtwork.tsx b/src/Apps/CollectorProfile/Routes/MyCollection/EditArtwork/MyCollectionCreateArtwork.tsx
@@ -87,6 +87,7 @@ export const MyCollectionCreateArtwork: React.FC<
     }
   }
 
+  // biome-ignore lint/correctness/useExhaustiveDependencies: ignored using `--suppress`
   useEffect(() => {
     window?.scrollTo?.({ top: 0 })
   }, [currentStep])

diff --git a/src/Apps/CollectorProfile/Routes/Saves/CollectorProfileSavesRoute.tsx b/src/Apps/CollectorProfile/Routes/Saves/CollectorProfileSavesRoute.tsx
@@ -80,7 +80,9 @@ const CollectorProfileSavesRoute: FC<
    * To prevent this from happening, `useMemo` is used here
    */
 
-  const selectedArtworkList = useMemo(() => {
+
+// biome-ignore lint/correctness/useExhaustiveDependencies: ignored using `--suppress`
+const  selectedArtworkList = useMemo(() => {
     return artworkLists.find(
       artworkList => artworkList.internalID === selectedArtworkListId,
     )

diff --git a/src/Apps/CollectorProfile/Routes/Saves/Components/SavesArtworks.tsx b/src/Apps/CollectorProfile/Routes/Saves/Components/SavesArtworks.tsx
@@ -64,6 +64,7 @@ export const SavesArtworks: FC<React.PropsWithChildren<SavesArtworksProps>> = ({
     )
   }, [state.priceMax, state.priceMin])
 
+  // biome-ignore lint/correctness/useExhaustiveDependencies: ignored using `--suppress`
   const SaveArtworksInvalidFallback = useMemo(() => {
     return ({ onReset }: ErrorFallbackProps) => {
       return (

diff --git a/src/Apps/Components/AppShell.tsx b/src/Apps/Components/AppShell.tsx
@@ -32,7 +32,8 @@ export const AppShell: React.FC<
   // Check to see if a route has a onServerSideRender key; if so call it. Used
   // typically to preload bundle-split components (import()) while the route is
   // fetching data in the background.
-  useEffect(() => {
+  // biome-ignore lint/correctness/useExhaustiveDependencies: ignored using `--suppress`
+        useEffect(() => {
     try {
       if (match) {
         routeConfig?.onClientSideRender?.({ match })

diff --git a/src/Apps/Conversations/components/ArtsyGuaranteeMessage.tsx b/src/Apps/Conversations/components/ArtsyGuaranteeMessage.tsx
@@ -9,6 +9,7 @@ export const ArtsyGuaranteeMessage: React.FC<
   const { match } = useRouter()
   const [showBanner, setShowBanner] = useState(false)
 
+  // biome-ignore lint/correctness/useExhaustiveDependencies: ignored using `--suppress`
   useEffect(() => {
     setShowBanner(true)
 

diff --git a/src/Apps/Conversations/components/ConversationReply.tsx b/src/Apps/Conversations/components/ConversationReply.tsx
@@ -137,6 +137,7 @@ export const ConversationReply: FC<
     },
   )
 
+  // biome-ignore lint/correctness/useExhaustiveDependencies: ignored using `--suppress`
   useEffect(() => {
     resetForm()
   }, [data?.internalID, resetForm])

diff --git a/src/Apps/Conversations/components/Message/ConversationMessages.tsx b/src/Apps/Conversations/components/Message/ConversationMessages.tsx
@@ -406,7 +406,8 @@ const useAutoScrollToBottom = ({
   }, [lastMessageId, triggerAutoScroll])
 
   // Additional effect to ensure initial scroll after component mount
-  useEffect(() => {
+  // biome-ignore lint/correctness/useExhaustiveDependencies: ignored using `--suppress`
+    useEffect(() => {
     if (lastMessageId) {
       const timeoutId = setTimeout(() => {
         triggerAutoScroll({

diff --git a/src/Apps/Conversations/components/Sidebar/ConversationsSidebar.tsx b/src/Apps/Conversations/components/Sidebar/ConversationsSidebar.tsx
@@ -45,6 +45,7 @@ export const ConversationsSidebar: React.FC<
     totalDisplayedCount = SIDEBAR_FETCH_PAGE_SIZE
   }
 
+  // biome-ignore lint/correctness/useExhaustiveDependencies: ignored using `--suppress`
   useEffect(() => {
     if (!match.params.conversationId) {
       return

diff --git a/src/Apps/Conversations/components/Sidebar/ConversationsSidebarItem.tsx b/src/Apps/Conversations/components/Sidebar/ConversationsSidebarItem.tsx
@@ -31,6 +31,7 @@ export const ConversationsSidebarItem: React.FC<
 
   const shouldHighlight = isSelected || isUnread
 
+  // biome-ignore lint/correctness/useExhaustiveDependencies: ignored using `--suppress`
   useEffect(() => {
     if (isSelected) {
       scrollRef.current?.scrollIntoView?.({ block: "center" })

diff --git a/src/Apps/Conversations/hooks/useLoadMore.ts b/src/Apps/Conversations/hooks/useLoadMore.ts
@@ -34,6 +34,7 @@ export const useLoadMore = ({
     return isFunction(isLoadingNext) ? isLoadingNext() : isLoadingNext
   }
 
+  // biome-ignore lint/correctness/useExhaustiveDependencies: ignored using `--suppress`
   useEffect(() => {
     if (shouldLoadMore && hasMore() && !isLoading() && when !== false) {
       loadNext(pageSize, () => {

diff --git a/src/Apps/FairOrginizer/Routes/FairOrganizerDedicatedArticles.tsx b/src/Apps/FairOrginizer/Routes/FairOrganizerDedicatedArticles.tsx
@@ -40,6 +40,7 @@ export const FairOrganizerDedicatedArticles: React.FC<
   const [isLoading, setIsLoading] = useState(false)
   const logger = createLogger("FairOrganizerDedicatedArticles.tsx")
 
+  // biome-ignore lint/correctness/useExhaustiveDependencies: ignored using `--suppress`
   useEffect(() => {
     setIsLoading(true)
 

diff --git a/src/Apps/Notifications/NotificationsApp.tsx b/src/Apps/Notifications/NotificationsApp.tsx
@@ -48,7 +48,8 @@ const NotificationsApp: React.FC<
   const isNotificationsPage = pathname.startsWith("/notifications")
 
   // Scroll to top of content pane when route changes
-  useEffect(() => {
+  // biome-ignore lint/correctness/useExhaustiveDependencies: ignored using `--suppress`
+    useEffect(() => {
     contentRef.current?.scrollTo(0, 0)
   }, [pathname])
 

diff --git a/src/Apps/Order/Components/AddressVerificationFlow.tsx b/src/Apps/Order/Components/AddressVerificationFlow.tsx
@@ -151,7 +151,8 @@ const AddressVerificationFlow: React.FC<
   >(addressOptions[0]?.key)
 
   // perform only once when the flow first loads
-  useEffect(() => {
+  // biome-ignore lint/correctness/useExhaustiveDependencies: ignored using `--suppress`
+    useEffect(() => {
     if (verificationPath === VerificationPath.ERROR_IMMEDIATE_CONFIRM) {
       const fallbackOption = fallbackFromFormValues(verificationInput)
       onChosenAddress(AddressVerifiedBy.USER, fallbackOption.address)
@@ -186,6 +187,7 @@ const AddressVerificationFlow: React.FC<
       setShowModal(false)
       onChosenAddress(verifiedBy, selectedAddress.address)
     }
+  // biome-ignore lint/correctness/useExhaustiveDependencies: ignored using `--suppress`
   }, [addressOptions, onChosenAddress, selectedAddressKey])
 
   const handleCloseModal = ({

diff --git a/src/Apps/Order/Components/PriceOptions.tsx b/src/Apps/Order/Components/PriceOptions.tsx
@@ -63,6 +63,7 @@ export const PriceOptions: React.FC<
     selectedPriceOption,
   )
 
+  // biome-ignore lint/correctness/useExhaustiveDependencies: ignored using `--suppress`
   useEffect(() => {
     if (lastOffer) {
       onChange(lastOffer)
@@ -71,6 +72,7 @@ export const PriceOptions: React.FC<
     }
   }, [])
 
+  // biome-ignore lint/correctness/useExhaustiveDependencies: ignored using `--suppress`
   useEffect(() => {
     if (toggle) trackClick("Different amount", 0)
   }, [toggle])

diff --git a/src/Apps/Order/Hooks/useStripePaymentBySetupIntentId.tsx b/src/Apps/Order/Hooks/useStripePaymentBySetupIntentId.tsx
@@ -21,6 +21,7 @@ export function useStripePaymentBySetupIntentId(orderId: string) {
     null,
   )
 
+  // biome-ignore lint/correctness/useExhaustiveDependencies: ignored using `--suppress`
   useEffect(() => {
     // pull necessary params from Stripe redirect URL
     const {

diff --git a/src/Apps/Order/OrderApp.tsx b/src/Apps/Order/OrderApp.tsx
@@ -45,6 +45,7 @@ const OrderApp: FC<React.PropsWithChildren<OrderAppProps>> = props => {
 
   const removeNavigationListenerRef = useRef<null | (() => void)>(null)
 
+  // biome-ignore lint/correctness/useExhaustiveDependencies: ignored using `--suppress`
   useEffect(() => {
     if (!removeNavigationListenerRef.current) {
       removeNavigationListenerRef.current =

diff --git a/src/Apps/Order/Routes/Details/Components/OrderDetailsPage.tsx b/src/Apps/Order/Routes/Details/Components/OrderDetailsPage.tsx
@@ -22,6 +22,7 @@ export const OrderDetailsPage = ({ order, me }: OrderDetailsPageProps) => {
   const meData = useFragment(ME_FRAGMENT, me)
   const tracking = useOrder2Tracking(orderData.source, orderData.mode)
 
+  // biome-ignore lint/correctness/useExhaustiveDependencies: ignored using `--suppress`
   useEffect(() => {
     if (!!orderData) {
       tracking.orderDetailsViewed(

diff --git a/src/Apps/Order/Routes/Payment/index.tsx b/src/Apps/Order/Routes/Payment/index.tsx
@@ -103,6 +103,7 @@ export const PaymentRoute: FC<
 
   const artworkVersion = extractNodes(order.lineItems)[0]?.artworkVersion
 
+  // biome-ignore lint/correctness/useExhaustiveDependencies: ignored using `--suppress`
   useEffect(() => {
     const bankAccountsArray =
       selectedPaymentMethod !== "SEPA_DEBIT"
@@ -121,6 +122,7 @@ export const PaymentRoute: FC<
     }
   }, [order])
 
+  // biome-ignore lint/correctness/useExhaustiveDependencies: ignored using `--suppress`
   useEffect(() => {
     setSelectedPaymentMethod(getInitialPaymentMethodValue(order))
   }, [order])
@@ -360,7 +362,8 @@ export const PaymentRoute: FC<
   }
 
   // complete payment when balance check is disabled and bank account is set
-  useEffect(() => {
+  // biome-ignore lint/correctness/useExhaustiveDependencies: ignored using `--suppress`
+    useEffect(() => {
     if (
       !balanceCheckEnabled &&
       (selectedBankAccountId || isPaymentSetupSuccessful) &&
@@ -377,7 +380,8 @@ export const PaymentRoute: FC<
   ])
 
   // show error modal when payment setup error is set
-  useEffect(() => {
+  // biome-ignore lint/correctness/useExhaustiveDependencies: ignored using `--suppress`
+    useEffect(() => {
     if (paymentSetupError) {
       let title = "An error occurred"
       let message =
@@ -453,6 +457,7 @@ export const PaymentRoute: FC<
     })
   }
 
+  // biome-ignore lint/correctness/useExhaustiveDependencies: ignored using `--suppress`
   useEffect(() => {
     if (order && me) {
       setIsLoading(false)

diff --git a/src/Apps/Order/Routes/Shipping/Components/AddressModal.tsx b/src/Apps/Order/Routes/Shipping/Components/AddressModal.tsx
@@ -188,7 +188,8 @@ const AddressModalForm: FC<
     addressModalAction.type === "edit" ? Object.keys(errors.address || {}) : []
 
   // Touch fields that have errors on edit
-  useEffect(() => {
+  // biome-ignore lint/correctness/useExhaustiveDependencies: ignored using `--suppress`
+    useEffect(() => {
     if (attributeErrorFieldsForEdit.length > 0) {
       attributeErrorFieldsForEdit.forEach(field => {
         setFieldTouched(`attributes.${field}`, true)

diff --git a/src/Apps/Order/Routes/Shipping/Components/FulfillmentDetails.tsx b/src/Apps/Order/Routes/Shipping/Components/FulfillmentDetails.tsx
@@ -67,7 +67,9 @@ export const FulfillmentDetails: FC<
    * the rest of its life and reset values
    */
 
-  useEffect(() => {
+
+// biome-ignore lint/correctness/useExhaustiveDependencies: ignored using `--suppress`
+useEffect(() => {
     const formLoaded =
       typeof shippingContext.state.fulfillmentDetailsFormikContext.setValues ===
       "function"
@@ -95,7 +97,9 @@ export const FulfillmentDetails: FC<
    * require artsy shipping
    */
 
-  useEffect(() => {
+
+// biome-ignore lint/correctness/useExhaustiveDependencies: ignored using `--suppress`
+useEffect(() => {
     const { savedFulfillmentDetails } = shippingContext.orderData
 
     const isArtsyShippingSaved =

diff --git a/src/Apps/Order/Routes/Shipping/Components/FulfillmentDetailsForm.tsx b/src/Apps/Order/Routes/Shipping/Components/FulfillmentDetailsForm.tsx
@@ -115,7 +115,9 @@ const FulfillmentDetailsFormLayout = (
    * via `shippingContext.state.fulfillmentDetailsCtx`
    */
 
-  useEffect(() => {
+
+// biome-ignore lint/correctness/useExhaustiveDependencies: ignored using `--suppress`
+useEffect(() => {
     shippingContext.actions.setFulfillmentDetailsFormikContext(formikContext)
   }, [formikContext.values, formikContext.isValid])
 
@@ -137,6 +139,7 @@ const FulfillmentDetailsFormLayout = (
 
   const serializedValues = JSON.stringify(formikContext.values)
 
+  // biome-ignore lint/correctness/useExhaustiveDependencies: ignored using `--suppress`
   const handleSelectSavedAddress = useCallback(
     (address: SavedAddressType) => {
       shippingContext.actions.setStage("fulfillment_details")
@@ -157,7 +160,8 @@ const FulfillmentDetailsFormLayout = (
   )
 
   // Use useEffect to submit the form after values are updated
-  useEffect(() => {
+  // biome-ignore lint/correctness/useExhaustiveDependencies: ignored using `--suppress`
+    useEffect(() => {
     if (shouldSubmit) {
       formikContext.submitForm()
       setShouldSubmit(false) // Reset the state after submission
-Original file line number
+Diff line change
@@ Expand Up / @@ -46,6 +46,7 @@ export const ArtworkErrorApp: React.FC< @@
         }
       }, [])
+      // biome-ignore lint/correctness/useExhaustiveDependencies: ignored using `--suppress`
       useEffect(() => {
         trackPageview()
       }, [])
@@ Expand Down @@