@wistia/oxlint-config 1.10.0 → 1.11.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@wistia/oxlint-config",
-  "version": "1.10.0",
+  "version": "1.11.0",
   "description": "Wistia's Oxlint configurations",
   "keywords": [
     "linter",
@@ -76,16 +76,16 @@
     "validate": "vitest run test/validate-configs.test.mjs"
   },
   "dependencies": {
-    "@eslint-react/eslint-plugin": "^5.17.3",
-    "@vitest/eslint-plugin": "^1.6.23",
+    "@eslint-react/eslint-plugin": "^5.18.0",
+    "@vitest/eslint-plugin": "^1.6.24",
     "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-jest-dom": "^5.6.0",
     "eslint-plugin-n": "^18.2.2",
     "eslint-plugin-no-only-tests": "^3.4.0",
-    "eslint-plugin-playwright": "^2.10.5",
-    "eslint-plugin-storybook": "^10.5.3",
+    "eslint-plugin-playwright": "^2.11.0",
+    "eslint-plugin-storybook": "^10.5.5",
     "eslint-plugin-styled-components": "^0.0.0",
     "eslint-plugin-styled-components-a11y": "^2.2.1",
     "eslint-plugin-testing-library": "^7.16.2",
@@ -94,17 +94,17 @@
   "devDependencies": {
     "@changesets/changelog-github": "^0.7.0",
     "@changesets/cli": "^2.31.1",
-    "eslint": "^10.7.0",
-    "oxfmt": "^0.60.0",
-    "oxlint": "^1.75.0",
+    "eslint": "^10.8.0",
+    "oxfmt": "^0.61.0",
+    "oxlint": "^1.76.0",
     "oxlint-tsgolint": "^7.0.2001",
-    "storybook": "^10.5.3",
+    "storybook": "^10.5.5",
     "vitest": "^4.1.10"
   },
   "peerDependencies": {
-    "eslint": ">= 10.7.0",
-    "oxfmt": ">= 0.60.0",
-    "oxlint": ">= 1.75.0",
+    "eslint": ">= 10.8.0",
+    "oxfmt": ">= 0.61.0",
+    "oxlint": ">= 1.76.0",
     "oxlint-tsgolint": ">= 7.0.2001"
   },
   "peerDependenciesMeta": {
@@ -115,5 +115,5 @@
   "engines": {
     "node": "^20.19.0 || ^22.13.0 || >=24 || >=26"
   },
-  "packageManager": "yarn@4.17.1"
+  "packageManager": "yarn@4.18.0"
 }

package/rules/base.mjs

@@ -276,6 +276,11 @@ export const baseRules = {
     // https://oxc.rs/docs/guide/usage/linter/rules/eslint/grouped-accessor-pairs.html
     'eslint/grouped-accessor-pairs': 'error',
 
+    // Disallow specified identifiers
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/id-denylist.html
+    // Decision: requires a project-specific identifier list; too opinionated for a shared config
+    'eslint/id-denylist': 'off',
+
     // Require identifiers to match a specified regular expression
     // https://oxc.rs/docs/guide/usage/linter/rules/eslint/id-match.html
     // Decision: too opinionated for general use
@@ -857,6 +862,10 @@ export const baseRules = {
     // https://oxc.rs/docs/guide/usage/linter/rules/oxc/bad-object-literal-comparison.html
     'oxc/bad-object-literal-comparison': 'error',
 
+    // Disallow matchAll with regex missing the global flag (throws at runtime)
+    // https://oxc.rs/docs/guide/usage/linter/rules/oxc/bad-match-all-arg.html
+    'oxc/bad-match-all-arg': 'error',
+
     // Disallow replaceAll with regex missing the global flag (throws at runtime)
     // https://oxc.rs/docs/guide/usage/linter/rules/oxc/bad-replace-all-arg.html
     'oxc/bad-replace-all-arg': 'error',

package/rules/node.mjs

@@ -14,6 +14,10 @@ export const nodeRules = {
     // https://oxc.rs/docs/guide/usage/linter/rules/node/handle-callback-err.html
     'node/handle-callback-err': 'error',
 
+    // Enforce either module.exports or exports
+    // https://oxc.rs/docs/guide/usage/linter/rules/node/exports-style.html
+    'node/exports-style': ['error', 'module.exports'],
+
     // Disallow the assignment to exports
     // https://oxc.rs/docs/guide/usage/linter/rules/node/no-exports-assign.html
     'node/no-exports-assign': 'error',
@@ -94,7 +98,8 @@ export const nodeRules = {
 
     // Enforce either module.exports or exports
     // https://github.com/eslint-community/eslint-plugin-n/blob/HEAD/docs/rules/exports-style.md
-    'n/exports-style': 'error',
+    // Decision: handled by native node/exports-style
+    'n/exports-style': 'off',
 
     // Require require() to be called in the top-level module scope
     // https://github.com/eslint-community/eslint-plugin-n/blob/HEAD/docs/rules/global-require.md

package/rules/playwright.mjs

@@ -196,6 +196,10 @@ export const playwrightRules = {
     // https://github.com/playwright-community/eslint-plugin-playwright/blob/main/docs/rules/no-duplicate-slow.md
     'eslint-plugin-playwright/no-duplicate-slow': 'error',
 
+    // Disallow assertions on a Locator that can never fail
+    // https://github.com/playwright-community/eslint-plugin-playwright/blob/main/docs/rules/no-unnecessary-assertions.md
+    'eslint-plugin-playwright/no-unnecessary-assertions': '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

package/rules/unicorn.mjs

@@ -65,15 +65,43 @@ export const unicornRules = {
 
     // Enforce specific import styles per module
     // https://oxc.rs/docs/guide/usage/linter/rules/unicorn/import-style.html
-    // Override built-in defaults: this rule ships preferring default imports
-    // for `chalk`, `path`, and `node:path`. We require named imports instead.
+    // Every option is set explicitly, including the four that match oxlint's
+    // default, so an upgrade cannot change consumer behavior without a diff
+    // here. `styles` is the complete set — nothing is inherited.
+    // - `node:path` and `node:util` are the only node builtins the rule
+    //   constrains at all (default-only and named-only respectively); `fs`,
+    //   `os`, `crypto`, `url`, `events` and the rest are unconstrained. Named
+    //   is our style, so both are set to named. Bare `path` needs no entry —
+    //   `unicorn/prefer-node-protocol` already rejects the non-prefixed form.
+    // - `chalk` ships default-only, which is right for the pre-built instance
+    //   (chalk's whole main API) but wrong for `import { Chalk } from 'chalk'`,
+    //   the class you instantiate for a custom instance. Both are legitimate.
     'unicorn/import-style': [
       'error',
       {
+        // Matches oxlint's default. Static imports are the primary surface.
+        checkImport: true,
+
+        // Matches oxlint's default. Covers `await import('node:path')`.
+        checkDynamicImport: true,
+
+        // Matches oxlint's default. Covers `require('node:path')` in CJS.
+        checkRequire: true,
+
+        // Decision: matches oxlint's default. The only form it adds is
+        // `export * from`, already an error via
+        // `barrel-files/avoid-re-export-all` — enabling it double-reports.
+        checkExportFrom: false,
+
+        // Differs from oxlint's default of `true`. Declaring the full style set
+        // instead of merging into the built-ins means an upgrade that adds or
+        // changes a built-in style cannot silently move consumer behavior.
+        extendDefaultStyles: false,
+
         styles: {
           'node:path': { named: true, default: false },
-          path: { named: true, default: false },
-          chalk: { named: true, default: false },
+          'node:util': { named: true, default: false },
+          chalk: { named: true, default: true },
         },
       },
     ],