@wistia/oxlint-config 1.10.1 → 1.11.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@wistia/oxlint-config",
-  "version": "1.10.1",
+  "version": "1.11.1",
   "description": "Wistia's Oxlint configurations",
   "keywords": [
     "linter",
@@ -95,16 +95,16 @@
     "@changesets/changelog-github": "^0.7.0",
     "@changesets/cli": "^2.31.1",
     "eslint": "^10.8.0",
-    "oxfmt": "^0.60.0",
-    "oxlint": "^1.75.0",
+    "oxfmt": "^0.62.0",
+    "oxlint": "^1.77.0",
     "oxlint-tsgolint": "^7.0.2001",
     "storybook": "^10.5.5",
     "vitest": "^4.1.10"
   },
   "peerDependencies": {
     "eslint": ">= 10.8.0",
-    "oxfmt": ">= 0.60.0",
-    "oxlint": ">= 1.75.0",
+    "oxfmt": ">= 0.62.0",
+    "oxlint": ">= 1.77.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/unicorn.mjs

@@ -65,20 +65,42 @@ 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. Entries merge into the built-in style per key
-    // rather than replacing it, so every key we want must be set explicitly.
-    // - `node:path` ships default-only. Of all node builtins the rule constrains
-    //   only `node:path` (default-only) and `node:util` (named-only); `fs`,
-    //   `os`, `crypto`, `url`, `events` and the rest are unconstrained. Named is
-    //   our style everywhere else, so override it.
+    // 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 },
+          'node:util': { named: true, default: false },
           chalk: { named: true, default: true },
         },
       },