@wistia/oxlint-config 1.8.0 → 1.9.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@wistia/oxlint-config",
-  "version": "1.8.0",
+  "version": "1.9.0",
   "description": "Wistia's Oxlint configurations",
   "packageManager": "yarn@4.17.1",
   "type": "module",
@@ -71,15 +71,15 @@
     }
   },
   "dependencies": {
-    "@eslint-react/eslint-plugin": "^5.10.0",
-    "@vitest/eslint-plugin": "^1.6.20",
+    "@eslint-react/eslint-plugin": "^5.11.3",
+    "@vitest/eslint-plugin": "^1.6.21",
     "confusing-browser-globals": "^1.0.11",
     "eslint-plugin-barrel-files": "^3.0.1",
     "eslint-plugin-import-x": "^4.17.1",
     "eslint-plugin-jest-dom": "^5.5.0",
     "eslint-plugin-n": "^18.2.1",
     "eslint-plugin-no-only-tests": "^3.4.0",
-    "eslint-plugin-playwright": "^2.10.4",
+    "eslint-plugin-playwright": "^2.10.5",
     "eslint-plugin-storybook": "^10.4.6",
     "eslint-plugin-styled-components": "^0.0.0",
     "eslint-plugin-styled-components-a11y": "^2.2.1",
@@ -94,7 +94,7 @@
     "oxlint": "^1.73.0",
     "oxlint-tsgolint": "^0.24.0",
     "storybook": "^10.4.6",
-    "vitest": "^4.1.9"
+    "vitest": "^4.1.10"
   },
   "engines": {
     "node": "^20.19.0 || ^22.13.0 || >=24 || >=26"

package/rules/playwright.mjs

@@ -191,5 +191,59 @@ export const playwrightRules = {
     // Require toThrow message
     // https://github.com/playwright-community/eslint-plugin-playwright/blob/main/docs/rules/require-to-throw-message.md
     'eslint-plugin-playwright/require-to-throw-message': 'error',
+
+    // Disallow duplicate slow() annotations
+    // https://github.com/playwright-community/eslint-plugin-playwright/blob/main/docs/rules/no-duplicate-slow.md
+    'eslint-plugin-playwright/no-duplicate-slow': 'error',
+
+    // Enforce blank lines between test blocks
+    // https://github.com/playwright-community/eslint-plugin-playwright/blob/main/docs/rules/consistent-spacing-between-blocks.md
+    // Decision: style preference, left to a formatter
+    'eslint-plugin-playwright/consistent-spacing-between-blocks': 'off',
+
+    // Disallow setup and teardown hooks
+    // https://github.com/playwright-community/eslint-plugin-playwright/blob/main/docs/rules/no-hooks.md
+    // Decision: too strict for general use
+    'eslint-plugin-playwright/no-hooks': 'off',
+
+    // Disallow configured locators
+    // https://github.com/playwright-community/eslint-plugin-playwright/blob/main/docs/rules/no-restricted-locators.md
+    // Decision: no-op without project-specific options
+    'eslint-plugin-playwright/no-restricted-locators': 'off',
+
+    // Disallow configured matchers
+    // https://github.com/playwright-community/eslint-plugin-playwright/blob/main/docs/rules/no-restricted-matchers.md
+    // Decision: no-op without project-specific options
+    'eslint-plugin-playwright/no-restricted-matchers': 'off',
+
+    // Disallow configured ARIA roles in locators
+    // https://github.com/playwright-community/eslint-plugin-playwright/blob/main/docs/rules/no-restricted-roles.md
+    // Decision: no-op without project-specific options
+    'eslint-plugin-playwright/no-restricted-roles': 'off',
+
+    // Disallow skipped tests
+    // https://github.com/playwright-community/eslint-plugin-playwright/blob/main/docs/rules/no-skipped-test.md
+    // Decision: skipping tests is legitimate, matching vitest/no-disabled-tests
+    'eslint-plugin-playwright/no-skipped-test': 'off',
+
+    // Enforce lowercase titles
+    // https://github.com/playwright-community/eslint-plugin-playwright/blob/main/docs/rules/prefer-lowercase-title.md
+    // Decision: too opinionated for general use, matching vitest/prefer-lowercase-title
+    'eslint-plugin-playwright/prefer-lowercase-title': 'off',
+
+    // Require soft assertions
+    // https://github.com/playwright-community/eslint-plugin-playwright/blob/main/docs/rules/require-soft-assertions.md
+    // Decision: too strict for general use
+    'eslint-plugin-playwright/require-soft-assertions': 'off',
+
+    // Require tags on all tests
+    // https://github.com/playwright-community/eslint-plugin-playwright/blob/main/docs/rules/require-tags.md
+    // Decision: too strict for general use
+    'eslint-plugin-playwright/require-tags': 'off',
+
+    // Require explicit timeout on toPass()
+    // https://github.com/playwright-community/eslint-plugin-playwright/blob/main/docs/rules/require-to-pass-timeout.md
+    // Decision: too strict for general use, matching vitest/require-test-timeout
+    'eslint-plugin-playwright/require-to-pass-timeout': 'off',
   },
 };

package/rules/react.mjs

@@ -661,6 +661,10 @@ export const reactRules = {
     // https://eslint-react.xyz/docs/rules/web-api-no-leaked-resize-observer
     '@eslint-react/web-api-no-leaked-resize-observer': 'error',
 
+    // Prevent leaked IntersectionObserver instances
+    // https://eslint-react.xyz/docs/rules/web-api-no-leaked-intersection-observer
+    '@eslint-react/web-api-no-leaked-intersection-observer': 'error',
+
     // Prevent leaked setTimeout calls
     // https://eslint-react.xyz/docs/rules/web-api-no-leaked-timeout
     '@eslint-react/web-api-no-leaked-timeout': 'error',

package/rules/storybook.mjs

@@ -44,6 +44,11 @@ export const storybookRules = {
     // https://github.com/storybookjs/eslint-plugin-storybook/blob/main/docs/rules/no-renderer-packages.md
     'eslint-plugin-storybook/no-renderer-packages': 'error',
 
+    // Disallow title property in meta (forces CSF3 auto-generated titles)
+    // https://github.com/storybookjs/eslint-plugin-storybook/blob/main/docs/rules/no-title-property-in-meta.md
+    // Decision: explicit titles are a legitimate pattern
+    'eslint-plugin-storybook/no-title-property-in-meta': 'off',
+
     // No uninstalled addons
     // https://github.com/storybookjs/eslint-plugin-storybook/blob/main/docs/rules/no-uninstalled-addons.md
     'eslint-plugin-storybook/no-uninstalled-addons': 'error',

package/rules/styled-components.mjs

@@ -4,6 +4,10 @@ import { resolvePlugin } from '../resolve-plugin.mjs';
 export const styledComponentsRules = {
   jsPlugins: [resolvePlugin('eslint-plugin-styled-components-a11y')],
   rules: {
+    // Enforce emojis are wrapped and given accessible text
+    // Decision: deprecated in jsx-a11y
+    'eslint-plugin-styled-components-a11y/accessible-emoji': 'off',
+
     // Enforce that all elements that require alternative text have meaningful information
     // https://github.com/brendanmorrell/eslint-plugin-styled-components-a11y/blob/master/docs/rules/alt-text.md
     'eslint-plugin-styled-components-a11y/alt-text': 'error',
@@ -98,6 +102,16 @@ export const styledComponentsRules = {
     // Enforce label-control association
     'eslint-plugin-styled-components-a11y/label-has-associated-control': 'error',
 
+    // Enforce label has htmlFor
+    // Decision: deprecated in jsx-a11y, superseded by label-has-associated-control
+    'eslint-plugin-styled-components-a11y/label-has-for': 'off',
+
+    // Enforce lang attribute has a valid value
+    // Decision: broken in the plugin — exported but missing from its internal
+    // ruleNameToTypeDict, so its create() crashes (path.join on undefined) and
+    // oxlint reports a JS-plugin runtime error on every linted file
+    'eslint-plugin-styled-components-a11y/lang': 'off',
+
     // Enforce that media elements have captions
     'eslint-plugin-styled-components-a11y/media-has-caption': 'error',
 
@@ -132,6 +146,10 @@ export const styledComponentsRules = {
     // Enforce tabIndex value is not greater than zero
     'eslint-plugin-styled-components-a11y/no-noninteractive-tabindex': 'error',
 
+    // Disallow onChange without onBlur
+    // Decision: deprecated in jsx-a11y
+    'eslint-plugin-styled-components-a11y/no-onchange': 'off',
+
     // Enforce explicit role is not redundant
     'eslint-plugin-styled-components-a11y/no-redundant-roles': 'error',
 

package/rules/testing-library.mjs

@@ -97,6 +97,11 @@ export const testingLibraryRules = {
     // https://github.com/testing-library/eslint-plugin-testing-library/blob/main/docs/rules/prefer-find-by.md
     'eslint-plugin-testing-library/prefer-find-by': 'error',
 
+    // Prefer implicit assertions from getBy* queries over explicit ones
+    // https://github.com/testing-library/eslint-plugin-testing-library/blob/main/docs/rules/prefer-implicit-assert.md
+    // Decision: conflicts with explicit jest-dom assertions
+    'eslint-plugin-testing-library/prefer-implicit-assert': 'off',
+
     // Ensure appropriate get*/query* queries are used with their respective matchers
     // https://github.com/testing-library/eslint-plugin-testing-library/blob/main/docs/rules/prefer-presence-queries.md
     'eslint-plugin-testing-library/prefer-presence-queries': 'error',

package/rules/vitest.mjs

@@ -704,5 +704,15 @@ export const vitestRules = {
     // https://github.com/vitest-dev/eslint-plugin-vitest/blob/main/docs/rules/valid-title.md
     // Decision: handled by native vitest/valid-title
     'vitest-js/valid-title': 'off',
+
+    // Prefer mockReturnValue() over mockImplementation() returning a value
+    // https://github.com/vitest-dev/eslint-plugin-vitest/blob/main/docs/rules/prefer-mock-return-shorthand.md
+    // Decision: handled by native vitest/prefer-mock-return-shorthand
+    'vitest-js/prefer-mock-return-shorthand': 'off',
+
+    // Report todo tests
+    // https://github.com/vitest-dev/eslint-plugin-vitest/blob/main/docs/rules/warn-todo.md
+    // Decision: handled by native vitest/warn-todo
+    'vitest-js/warn-todo': 'off',
   },
 };