chore(devdeps): update dependency @scaleway/oxlint-config to v0.2.0 (#6561)

renovate[bot] · jsulpis · philibea · web-flow · commit f38c4e883ef7 · 2026-06-29T11:33:45.000+02:00
* chore(devdeps): update dependency @scaleway/oxlint-config to v0.2.0

Signed-off-by: renovate[bot] &lt;29139614+renovate[bot]@users.noreply.github.com&gt;

* fix: broken lint

---------

Signed-off-by: renovate[bot] &lt;29139614+renovate[bot]@users.noreply.github.com&gt;
Co-authored-by: renovate[bot] &lt;29139614+renovate[bot]@users.noreply.github.com&gt;
Co-authored-by: Julien Sulpis &lt;jsulpis@scaleway.com&gt;
Co-authored-by: philibeaux &lt;aphilibeaux@scaleway.com&gt;
diff --git a/.oxlintrc.jsonc b/.oxlintrc.jsonc
@@ -3,7 +3,6 @@
   "extends": [
     "./node_modules/@scaleway/oxlint-config/configs/index.json",
     "./node_modules/@scaleway/oxlint-config/configs/react.json",
-    "./node_modules/@scaleway/oxlint-config/configs/typescript.json",
   ],
   "jsPlugins": ["./.oxlint/rules/storybook-a11y-partial.cjs"],
   "options": {
diff --git a/package.json b/package.json
@@ -45,7 +45,7 @@
     "@manypkg/cli": "0.25.1",
     "@playwright/test": "1.61.1",
     "@scaleway/oxfmt-config": "1.1.3",
-    "@scaleway/oxlint-config": "0.1.17",
+    "@scaleway/oxlint-config": "0.2.0",
     "@scaleway/tsconfig": "1.1.6",
     "@size-limit/file": "12.1.0",
     "@size-limit/preset-big-lib": "12.1.0",
diff --git a/packages/animations/src/hooks/__tests__/usePrefersReducedMotion.test.tsx b/packages/animations/src/hooks/__tests__/usePrefersReducedMotion.test.tsx
@@ -4,7 +4,6 @@ import { usePrefersReducedMotion } from '../usePrefersReducedMotion'
 
 describe(usePrefersReducedMotion, () => {
   it('should return true when prefers-reduced-motion is enabled', () => {
-    // oxlint-disable-next-line vitest/prefer-spy-on
     window.matchMedia = vi.fn().mockReturnValue({
       matches: false,
       addEventListener: vi.fn(),
@@ -17,7 +16,6 @@ describe(usePrefersReducedMotion, () => {
   })
 
   it('should return false when prefers-reduced-motion is disabled', () => {
-    // oxlint-disable-next-line vitest/prefer-spy-on
     window.matchMedia = vi.fn().mockReturnValue({
       matches: true,
       addEventListener: vi.fn(),
@@ -32,7 +30,6 @@ describe(usePrefersReducedMotion, () => {
   it('should cleanup event listener on unmount', () => {
     const mockRemoveListener = vi.fn()
 
-    // oxlint-disable-next-line vitest/prefer-spy-on
     window.matchMedia = vi.fn().mockReturnValue({
       matches: true,
       addEventListener: vi.fn(),
diff --git a/packages/form/src/components/Form/index.tsx b/packages/form/src/components/Form/index.tsx
@@ -7,12 +7,7 @@ import { ErrorProvider } from '../../providers'
 import type { FormErrors } from '../../types'
 import { defaultErrors } from './defaultErrors'
 
-type OnSubmitReturn =
-  | string
-  | null
-  | undefined
-  // oxlint-disable-next-line typescript/no-invalid-void-type
-  | void
+type OnSubmitReturn = string | null | undefined | void
 
 type FormProps<TFieldValues extends FieldValues> = {
   children?: ReactNode
diff --git a/packages/illustrations/src/assets/products/index.ts b/packages/illustrations/src/assets/products/index.ts
@@ -3,8 +3,6 @@
  * PLEASE DO NOT EDIT HERE
  */
 
-// oxlint-disable import/no-namespace
-
 import * as accountExperience from './accountExperience'
 import * as annotations from './annotations'
 import * as apiGateway from './apiGateway'
diff --git a/packages/illustrations/src/assets/various/index.ts b/packages/illustrations/src/assets/various/index.ts
@@ -1,5 +1,3 @@
-// oxlint-disable import/no-namespace
-
 import * as documentation from './documentation'
 import * as empty from './empty'
 import * as feedback from './feedback'
diff --git a/packages/illustrations/src/index.ts b/packages/illustrations/src/index.ts
@@ -1,5 +1,3 @@
-// oxlint-disable import/no-namespace
-
 import * as products from './assets/products'
 import * as various from './assets/various'
 import * as components from './components'
diff --git a/packages/ui/src/components/BarChart/__tests__/index.test.tsx b/packages/ui/src/components/BarChart/__tests__/index.test.tsx
@@ -51,7 +51,6 @@ describe('barChart', () => {
     )
 
     const bar = container.querySelector('svg[role="img"] g line')
-    // oxlint-disable-next-line vitest/no-conditional-in-test
     if (!bar) {
       throw new Error('BarChart column not found')
     }
diff --git a/packages/ui/src/components/DateInput/__stories__/I18n.stories.tsx b/packages/ui/src/components/DateInput/__stories__/I18n.stories.tsx
@@ -43,7 +43,6 @@ export const I18n: StoryFn = args => {
       <SelectInput
         label="Switch Locale"
         name="locale-switcher"
-        // oxlint-disable-next-line typescript/no-misused-promises
         onChange={onChange}
         options={localesOptions}
         searchable={false}
diff --git a/packages/ui/src/components/DateInput/__tests__/index.test.tsx b/packages/ui/src/components/DateInput/__tests__/index.test.tsx
@@ -21,7 +21,6 @@ describe('dateInput', () => {
   it('renders correctly with default props', () => {
     const { asFragment } = renderWithTheme(
       <DateInput
-        // oxlint-disable-next-line vitest/no-conditional-in-test
         format={value => (value instanceof Date ? value.toISOString() : value)}
         label="Date"
         locale={fr}
@@ -358,7 +357,6 @@ describe('dateInput', () => {
     const { asFragment } = renderWithTheme(
       <DateInput
         endDate={new Date('1995-12-11T03:24:00.000+00:00')}
-        // oxlint-disable-next-line vitest/no-conditional-in-test
         format={value => (value instanceof Date ? value.toISOString() : value)}
         label="Date"
         name="test"
diff --git a/packages/ui/src/components/ExpandableCard/__tests__/index.test.tsx b/packages/ui/src/components/ExpandableCard/__tests__/index.test.tsx
@@ -81,7 +81,6 @@ describe('expandableCard', () => {
     fireEvent.dragOver(dropZone)
     fireEvent.drop(dropZone, {
       dataTransfer: {
-        // oxlint-disable-next-line vitest/no-conditional-in-test
         getData: (type: string) => (type === 'text' ? data : ''),
       },
     })
diff --git a/packages/ui/src/components/FileInput/__tests__/index.test.tsx b/packages/ui/src/components/FileInput/__tests__/index.test.tsx
@@ -463,7 +463,6 @@ describe('fileInput', () => {
       name: 'dir',
       createReader: () => ({
         readEntries: (resolve: (entries: FileSystemEntry[]) => void) => {
-          // oxlint-disable-next-line vitest/no-conditional-in-test
           if (callCount === 0) {
             callCount += 1
             resolve([dirFileEntry])
diff --git a/packages/ui/src/components/LineChart/__tests__/index.test.tsx b/packages/ui/src/components/LineChart/__tests__/index.test.tsx
@@ -42,7 +42,6 @@ describe('lineChart', () => {
       <LineChart
         data={lineChartData}
         pointFormatters={{
-          // oxlint-disable-next-line vitest/no-conditional-in-test
           x: value => value?.toString() ?? '',
           y: value => `${value as number} unit`,
         }}
@@ -63,7 +62,6 @@ describe('lineChart', () => {
   it.skip('renders correctly when chart is hovered', async () => {
     const { asFragment } = renderWithTheme(<LineChart data={lineChartData} withLegend xScale={{ type: 'linear' }} />)
     const line = document.querySelector('svg[role="img"] g path')
-    // oxlint-disable-next-line vitest/no-conditional-in-test
     if (!line) {
       throw new Error('LineChart line path not found')
     }
diff --git a/packages/ui/src/components/List/__tests__/index.test.tsx b/packages/ui/src/components/List/__tests__/index.test.tsx
@@ -1,5 +1,3 @@
-// oxlint-disable vitest/no-conditional-in-test
-// oxlint-disable vitest/no-conditional-expect
 import { fireEvent, render, screen } from '@testing-library/react'
 import { userEvent } from '@testing-library/user-event'
 import { consoleLightTheme, ThemeProvider } from '@ultraviolet/themes'
diff --git a/packages/ui/src/components/List/index.tsx b/packages/ui/src/components/List/index.tsx
@@ -75,7 +75,6 @@ const BaseList = forwardRef<HTMLTableElement, NewListProps | LegacyListProps>(
               <HeaderCell
                 info={column.info}
                 isOrdered={column.isOrdered}
-                // oxlint-disable-next-line react/no-array-index-key
                 key={`header-column-${index}`}
                 maxWidth={column.maxWidth}
                 minWidth={column.minWidth}
@@ -104,10 +103,7 @@ type ListType = {
    * @deprecated Use `colMode="strict"`
    */
 
-  (
-    // oxlint-disable-next-line typescript/unified-signatures
-    props: LegacyListProps & RefAttributes<HTMLTableElement>,
-  ): ReactNode
+  (props: LegacyListProps & RefAttributes<HTMLTableElement>): ReactNode
   Cell: typeof Cell
   Row: typeof Row
   SelectBar: typeof SelectBar
diff --git a/packages/ui/src/components/Modal/components/Dialog.tsx b/packages/ui/src/components/Modal/components/Dialog.tsx
@@ -107,10 +107,7 @@ export const Dialog = ({
   // We need to reverse the array as the last opened modal should be the first to be with normal size
   // while the first opened modal should shrink
   const realPosition = [...openedModals].findIndex(object => object.id === id)
-  const position = [...openedModals]
-    // oxlint-disable-next-line unicorn/no-array-reverse
-    .reverse()
-    .findIndex(object => object.id === id) // reverse method mutate array so we need to create a new array
+  const position = [...openedModals].reverse().findIndex(object => object.id === id) // reverse method mutate array so we need to create a new array
   const modalAbove = openedModals[realPosition + 1]
   const currentModalHeight = dialogRef.current?.offsetHeight
   let top = 0
diff --git a/packages/ui/src/components/PhoneInput/index.tsx b/packages/ui/src/components/PhoneInput/index.tsx
@@ -126,7 +126,6 @@ export const PhoneInput: PhoneInputType = forwardRef(
           onValueChange?.(result)
 
           return formattedNumber
-          // oxlint-disable-next-line eslint/no-shadow
         } catch (error: unknown) {
           if (error instanceof Error) {
             onParsingError?.({
diff --git a/packages/ui/src/components/PieChart/__tests__/index.test.tsx b/packages/ui/src/components/PieChart/__tests__/index.test.tsx
@@ -13,7 +13,6 @@ import {
 describe('pieChart', () => {
   beforeAll(() => {
     // Have to mock ResizeObserver as Nivo doesn't add automatically ResizeObserver polyfill anymore (v0.79.0)
-    // oxlint-disable-next-line vitest/prefer-spy-on
     window.ResizeObserver = vi.fn().mockReturnValue({
       disconnect: vi.fn(),
       observe: vi.fn(),
@@ -65,7 +64,6 @@ describe('pieChart', () => {
   it.skip('renders correctly when chart is hovered', async () => {
     const { container } = renderWithTheme(<PieChart data={dataWithLegendsAndDetails} withLegend />)
     const slice = container.querySelector('svg g path')
-    // oxlint-disable-next-line vitest/no-conditional-in-test
     if (!slice) {
       throw new Error('PieChart slice path not found')
     }
diff --git a/packages/ui/src/components/SelectableCard/__tests__/index.test.tsx b/packages/ui/src/components/SelectableCard/__tests__/index.test.tsx
@@ -70,9 +70,7 @@ describe('selectableCard', () => {
           {({ checked, disabled }) => (
             <div
               style={{
-                // oxlint-disable-next-line vitest/no-conditional-in-test
                 background: disabled ? 'gray' : 'green',
-                // oxlint-disable-next-line vitest/no-conditional-in-test
                 color: checked ? 'green' : 'gray',
               }}
             >
@@ -174,7 +172,6 @@ describe('selectableCard', () => {
         </SelectableCard>,
       )
 
-      // oxlint-disable-next-line vitest/no-conditional-in-test
       const role = type === 'toggle' ? 'checkbox' : type
       const label = screen.getByRole(role, {
         name: 'labelName',
diff --git a/packages/ui/src/components/Skeleton/List.tsx b/packages/ui/src/components/Skeleton/List.tsx
@@ -1,4 +1,3 @@
-// oxlint-disable no-shadow
 'use client'
 
 import { Line } from './Line'
diff --git a/packages/ui/src/components/Table/__tests__/index.test.tsx b/packages/ui/src/components/Table/__tests__/index.test.tsx
@@ -1,5 +1,3 @@
-// oxlint-disable vitest/no-conditional-in-test
-// oxlint-disable vitest/no-conditional-expect
 import { fireEvent, render, screen } from '@testing-library/react'
 import { userEvent } from '@testing-library/user-event'
 import { ThemeProvider } from '@ultraviolet/themes'
@@ -132,11 +130,9 @@ describe('table', () => {
     const allCheckbox = checkboxes.find(({ value }) => value === 'all')
     expect(firstRowCheckbox).toBeInTheDocument()
     expect(allCheckbox).toBeInTheDocument()
-    // oxlint-disable-next-line vitest/no-conditional-in-test
     if (!firstRowCheckbox) {
       throw new Error('First checkbox is not defined')
     }
-    // oxlint-disable-next-line vitest/no-conditional-in-test
     if (!allCheckbox) {
       throw new Error('Select all checkbox is not defined')
     }
diff --git a/packages/ui/src/components/Table/index.tsx b/packages/ui/src/components/Table/index.tsx
@@ -85,7 +85,6 @@ export const BaseTable = forwardRef<HTMLTableElement, TableProps>(
                   align={column.align}
                   info={column.info}
                   isOrdered={column.isOrdered}
-                  // oxlint-disable-next-line react/no-array-index-key
                   key={`header-column-${index}`}
                   maxWidth={column.maxWidth}
                   minWidth={column.minWidth}
diff --git a/packages/ui/src/components/VerificationCode/index.tsx b/packages/ui/src/components/VerificationCode/index.tsx
@@ -238,7 +238,6 @@ export const VerificationCode = ({
             data-testid={index}
             disabled={disabled}
             id={`${id}-${index}`}
-            // oxlint-disable-next-line react/no-array-index-key
             key={`field-${index}`}
             onChange={inputOnChange(index)}
             onFocus={inputOnFocus}
diff --git a/packages/ui/src/compositions/Navigation/__tests__/index.test.tsx b/packages/ui/src/compositions/Navigation/__tests__/index.test.tsx
@@ -187,7 +187,6 @@ describe('navigation', () => {
     // Find the button that contains "Servers" text
     const firstServersButton = allButtonsNav.find(button => button.textContent?.includes('Servers'))
 
-    // oxlint-disable-next-line vitest/no-conditional-in-test
     if (firstServersButton) {
       // Hover over the button to reveal the pin button
       await userEvent.hover(firstServersButton)
@@ -233,12 +232,10 @@ describe('navigation', () => {
     // Find the button that contains "Servers" text
     const serversButton = allButtons.find(button => button.textContent?.includes('Servers'))
 
-    // oxlint-disable-next-line vitest/no-conditional-in-test
     if (serversButton) {
       await userEvent.hover(serversButton)
     }
 
-    // oxlint-disable-next-line vitest/no-conditional-in-test
     if (unpinButton) {
       await userEvent.click(unpinButton)
     }
diff --git a/packages/ui/src/compositions/OfferList/__tests__/index.test.tsx b/packages/ui/src/compositions/OfferList/__tests__/index.test.tsx
@@ -44,7 +44,6 @@ describe('offerList', () => {
         {data.map(planet => (
           <OfferList.Row
             banner={{
-              // oxlint-disable-next-line vitest/no-conditional-in-test
               sentiment: planet.id === 'mercury' ? 'warning' : undefined,
               text: 'text',
             }}
diff --git a/packages/ui/src/compositions/OrderSummary/components/SubCategory.tsx b/packages/ui/src/compositions/OrderSummary/components/SubCategory.tsx
@@ -59,13 +59,7 @@ export const SubCategory = ({ subCategory }: { subCategory: SubCategoryType }) =
       <Stack className={orderSummaryStyle.details} direction="column" gap={0.5}>
         {subCategory.details.map((detail, index) =>
           detail ? (
-            <Text
-              as="span"
-              // oxlint-disable-next-line react/no-array-index-key
-              key={`${subCategory.title}-${index}`}
-              sentiment="neutral"
-              variant="bodySmall"
-            >
+            <Text as="span" key={`${subCategory.title}-${index}`} sentiment="neutral" variant="bodySmall">
               {detail}
             </Text>
           ) : null,
diff --git a/packages/ui/src/helpers/__tests__/index.test.ts b/packages/ui/src/helpers/__tests__/index.test.ts
@@ -39,10 +39,7 @@ describe(onKeyOnlyNumbers, () => {
         preventDefault,
       } as unknown as KeyboardEvent)
 
-      expect(preventDefault).toHaveBeenCalledTimes(
-        // oxlint-disable-next-line vitest/no-conditional-in-test
-        keyCode < 48 || keyCode > 57 ? 1 : 0,
-      )
+      expect(preventDefault).toHaveBeenCalledTimes(keyCode < 48 || keyCode > 57 ? 1 : 0)
     })
   })
 })
diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml
diff --git a/utils/stories/src/Guides/CompositionsList/index.tsx b/utils/stories/src/Guides/CompositionsList/index.tsx
@@ -18,7 +18,6 @@ export const CompositionsList = () => {
   const componentsList: ComponentInfo[] = Object.entries(Compositions)
     .map(([name, component]) => ({
       name,
-      // oxlint-disable-next-line no-underscore-dangle
       description: (component as ReactComponentWithDocGen).__docgenInfo?.description,
     }))
     .filter(component => !NOT_COMPONENTS.has(component.name))
diff --git a/utils/stories/src/Properties/Properties.tsx b/utils/stories/src/Properties/Properties.tsx
@@ -1,4 +1,3 @@
-// oxlint-disable no-underscore-dangle
 import { Stack, Table, Text } from '@ultraviolet/ui'
 import type { ComponentType } from 'react'
 import * as components from '../../../../packages/ui/src/components'
diff --git a/utils/test/src/vitest/setup.ts b/utils/test/src/vitest/setup.ts
@@ -40,9 +40,7 @@ export const setup = () => {
     setupIntersectionMocking(vi.fn)
     vi.spyOn(globalThis.Math, 'random').mockReturnValue(0.415_591_366_944_480_4)
 
-    // oxlint-disable-next-line vitest/prefer-spy-on
     window.ResizeObserver = vi.fn().mockImplementation(MockResize)
-    // oxlint-disable-next-line vitest/prefer-spy-on
     window.matchMedia = vi.fn().mockImplementation(MockMatchMedia)
 
     if (!globalThis.navigator.clipboard) {

Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,3 @@`
`1`		`-// oxlint-disable import/no-namespace`
`2`		`-`
`3`	`1`	`import * as documentation from './documentation'`
`4`	`2`	`import * as empty from './empty'`
`5`	`3`	`import * as feedback from './feedback'`
Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,3 @@`
`1`		`-// oxlint-disable import/no-namespace`
`2`		`-`
`3`	`1`	`import * as products from './assets/products'`
`4`	`2`	`import * as various from './assets/various'`
`5`	`3`	`import * as components from './components'`
Original file line number	Diff line number	Diff line change
`@@ -51,7 +51,6 @@ describe('barChart', () => {`
`51`	`51`	`)`
`52`	`52`
`53`	`53`	`const bar = container.querySelector('svg[role="img"] g line')`
`54`		`- // oxlint-disable-next-line vitest/no-conditional-in-test`
`55`	`54`	`if (!bar) {`
`56`	`55`	`throw new Error('BarChart column not found')`
`57`	`56`	`}`