chore(devdeps): update dependency eslint-plugin-oxlint to v0.18.1 (#5252)

* chore(devdeps): update dependency eslint-plugin-oxlint to v0.18.1

* fix: lint

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Matthias <[email protected]>

Author: renovate[bot]

package.json

@@ -157,7 +157,7 @@
     "emoji-toolkit": "9.0.1",
     "esbuild-plugin-browserslist": "1.0.1",
     "eslint": "9.29.0",
-    "eslint-plugin-oxlint": "0.16.12",
+    "eslint-plugin-oxlint": "0.18.1",
     "eslint-plugin-testing-library": "7.4.0",
     "expect": "29.7.0",
     "file-loader": "6.2.0",

packages/plus/src/components/Navigation/NavigationContent.tsx

@@ -189,7 +189,6 @@ export const NavigationContent = ({
     sliderRef.current?.addEventListener('mousedown', mousedown)
 
     return () => {
-      // eslint-disable-next-line react-hooks/exhaustive-deps
       sliderRef.current?.removeEventListener('mousedown', mousedown)
     }
   }, [

packages/plus/src/components/Navigation/components/Item.tsx

@@ -392,15 +392,11 @@ export const Item = memo(
       animationType,
     } = context
 
-    useEffect(
-      () => {
-        if (type !== 'pinnedGroup' && pinnedFeature) {
-          registerItem({ [id]: { label, active, onToggle, onClickPinUnpin } })
-        }
-      },
-      // eslint-disable-next-line react-hooks/exhaustive-deps
-      [active, id, label, registerItem],
-    )
+    useEffect(() => {
+      if (type !== 'pinnedGroup' && pinnedFeature) {
+        registerItem({ [id]: { label, active, onToggle, onClickPinUnpin } })
+      }
+    }, [active, id, label, registerItem])
 
     const [internalExpanded, onToggleExpand] = useReducer(
       prevState => !prevState,

packages/ui/src/components/DateInput/index.tsx

@@ -220,7 +220,6 @@ export const DateInput = <IsRange extends undefined | boolean>({
         end: endDate ?? computedRange.end,
       })
     }
-    // eslint-disable-next-line react-hooks/exhaustive-deps
   }, [endDate, startDate, value])
 
   const manageOnChange = (event: ChangeEvent<HTMLInputElement>) => {

packages/ui/src/components/List/ListContext.tsx

@@ -314,7 +314,6 @@ export const ListProvider = ({
     return () => {
       handlers.forEach(cleanup => cleanup())
     }
-    // eslint-disable-next-line react-hooks/exhaustive-deps
   }, [lastCheckedCheckbox, selectRows])
 
   useEffect(() => {

packages/ui/src/components/List/index.tsx

@@ -60,7 +60,6 @@ const TableContainer = ({ children }: { children: ReactNode }) => {
       setRefList([])
       setChildrenMemory(Children.toArray(children))
     }
-    // eslint-disable-next-line react-hooks/exhaustive-deps
   }, [children, setRefList])
 
   return <TableContainerStyle>{children}</TableContainerStyle>

packages/ui/src/components/NumberInputV2/index.tsx

@@ -297,37 +297,31 @@ export const NumberInputV2 = forwardRef(
       [localRef, min, onChange],
     )
 
-    const isMinusDisabled = useMemo(
-      () => {
-        if (!localRef?.current?.value || localRef?.current?.value === '') {
-          return false
-        }
+    const isMinusDisabled = useMemo(() => {
+      if (!localRef?.current?.value || localRef?.current?.value === '') {
+        return false
+      }
 
-        const numericValue = Number(localRef?.current?.value)
-        if (Number.isNaN(numericValue)) return false
+      const numericValue = Number(localRef?.current?.value)
+      if (Number.isNaN(numericValue)) return false
 
-        const minValue = typeof min === 'number' ? min : Number(min)
+      const minValue = typeof min === 'number' ? min : Number(min)
 
-        return Number.isNaN(numericValue) || numericValue <= minValue
-      }, // eslint-disable-next-line react-hooks/exhaustive-deps
-      [localRef?.current?.value, min],
-    )
+      return Number.isNaN(numericValue) || numericValue <= minValue
+    }, [localRef?.current?.value, min])
 
-    const isPlusDisabled = useMemo(
-      () => {
-        if (!localRef?.current?.value || localRef?.current?.value === '') {
-          return false
-        }
+    const isPlusDisabled = useMemo(() => {
+      if (!localRef?.current?.value || localRef?.current?.value === '') {
+        return false
+      }
 
-        const numericValue = Number(localRef?.current?.value)
-        if (Number.isNaN(numericValue)) return false
+      const numericValue = Number(localRef?.current?.value)
+      if (Number.isNaN(numericValue)) return false
 
-        const maxValue = typeof max === 'number' ? max : Number(max)
+      const maxValue = typeof max === 'number' ? max : Number(max)
 
-        return numericValue >= maxValue
-      }, // eslint-disable-next-line react-hooks/exhaustive-deps
-      [localRef?.current?.value, max],
-    )
+      return numericValue >= maxValue
+    }, [localRef?.current?.value, max])
 
     const helperSentiment = useMemo(() => {
       if (error) {

packages/ui/src/components/Popup/index.tsx

@@ -259,7 +259,6 @@ export const Popup = forwardRef(
       }
 
       return null
-      // eslint-disable-next-line react-hooks/exhaustive-deps
     }, [portalTarget, role, childrenRef.current])
 
     // There are some issue when mixing animation and maxHeight on some browsers, so we disable animation if maxHeight is set.
@@ -430,15 +429,11 @@ export const Popup = forwardRef(
     ])
 
     // This will be triggered when positions are computed and popup is visible in the dom.
-    useEffect(
-      () => {
-        if (visibleInDom && innerPopupRef.current) {
-          innerPopupRef.current.style.opacity = '1'
-        }
-      },
-      // eslint-disable-next-line react-hooks/exhaustive-deps
-      [positions],
-    )
+    useEffect(() => {
+      if (visibleInDom && innerPopupRef.current) {
+        innerPopupRef.current.style.opacity = '1'
+      }
+    }, [positions])
 
     /**
      * If popup has `visible` prop it means the popup is manually controlled through this prop.

packages/ui/src/components/SelectInputV2/Dropdown.tsx

@@ -721,7 +721,6 @@ export const Dropdown = ({
         } else window.scrollBy({ top: overflow, behavior: 'smooth' })
       }
     }
-    // eslint-disable-next-line react-hooks/exhaustive-deps
   }, [isDropdownVisible, refSelect, size, ref.current])
 
   const resizeDropdown = useCallback(() => {

packages/ui/src/components/Table/index.tsx

@@ -109,7 +109,6 @@ const TableContainer = ({ children }: { children: ReactNode }) => {
       setRefList([])
       setChildrenMemory(Children.toArray(children))
     }
-    // eslint-disable-next-line react-hooks/exhaustive-deps
   }, [children, setRefList])
 
   return <TableContainerStyle>{children}</TableContainerStyle>