@wistia/oxlint-config 1.1.1 → 1.3.0

package/README.md

@@ -29,6 +29,8 @@ yarn add -D oxlint-tsgolint
 
 `oxlint-tsgolint` is a separate Go-based tool that builds your TypeScript program using `typescript-go` and runs type-aware rules (e.g. detecting unhandled promises, unsafe assignments). Without it installed, type-aware rules will not run.
 
+_Note: this will likely be unnecessary in the future when Typescript v7 is released._
+
 ## Quick start
 
 Create an `oxlint.config.ts` in your project root:
@@ -161,12 +163,6 @@ export default defineConfig({
 5. Add short description of rule and link to rule documentation in code comments
 6. Never delete a rule — set it to `off` with a comment explaining why
 
-## ESLint parity
-
-Differences between `@wistia/oxlint-config` and `@wistia/eslint-config`.
-
-oxlint and ESLint use different rule name prefixes. This document uses the eslint-config names. See [Rule Name Mapping](#rule-name-mapping) at the bottom for prefix translations.
-
 ### Why some rules are ESLint-only
 
 oxlint's jsPlugins feature can load ESLint plugins that export a standard `{ rules }` object. This works for packages like `eslint-plugin-import-x`, `eslint-plugin-storybook`, etc.
@@ -179,4 +175,5 @@ Several categories of rules cannot use this approach:
 
 3. **Rules requiring module resolution** (e.g. `import-x/no-unresolved`, `import-x/extensions`, `import-x/no-rename-default`). In ESLint, these rules use `eslint-import-resolver-typescript` to resolve TypeScript path aliases, barrel re-exports, and extensionless imports. The resolver is configured via ESLint's `settings['import-x/resolver']` — but oxlint's jsPlugin runner does not pass ESLint settings to plugins. Without a resolver, these rules can't find modules and produce thousands of false positives on every import.
 
-4. **Old-style ESLint plugins** (e.g. `eslint-plugin-filenames`). These export rules as plain functions instead of objects with a `create` method. oxlint's jsPlugin runner requires the modern format (`{ meta, create }`).
+4. **Old-style ESLint plugins** (e.g. `eslint-plugin-filenames`). These export rules as plain functions instead of objects with a `create` method, which oxlint's jsPlugin runner — requiring the modern `{ meta, create }` format — cannot load. The package is also abandoned. Where a rule from such a plugin is worth keeping, it is reimplemented in our own custom local jsPlugin (`custom`) under [`plugins/custom.mjs`](./plugins/custom.mjs).
+   - `custom/match-exported` is a faithful port of `eslint-plugin-filenames`' `match-exported`, enabled in `javascriptConfig` + `typescriptConfig`, matching eslint-config. (Its rule ID is namespaced `custom/` rather than `filenames/` to signal it is our own rule, not the upstream plugin.) The upstream `match-regex` and `no-index` rules are not ported — eslint-config disables both.

package/configs/javascript.mjs

@@ -3,16 +3,22 @@ import { baseRules } from '../rules/base.mjs';
 import { importRules } from '../rules/import.mjs';
 import { promiseRules } from '../rules/promise.mjs';
 import { barrelFilesRules } from '../rules/barrel-files.mjs';
+import { customRules } from '../rules/custom.mjs';
 import { unicornRules } from '../rules/unicorn.mjs';
 
 export default defineConfig({
   plugins: [...baseRules.plugins, ...importRules.plugins, ...promiseRules.plugins],
-  jsPlugins: [...(importRules.jsPlugins || []), ...(barrelFilesRules.jsPlugins || [])],
+  jsPlugins: [
+    ...(importRules.jsPlugins || []),
+    ...(barrelFilesRules.jsPlugins || []),
+    ...(customRules.jsPlugins || []),
+  ],
   rules: {
     ...baseRules.rules,
     ...importRules.rules,
     ...promiseRules.rules,
     ...barrelFilesRules.rules,
+    ...customRules.rules,
     ...unicornRules.rules,
   },
 });

package/configs/storybook.mjs

@@ -1,12 +1,18 @@
 import { defineConfig } from 'oxlint';
 import { storybookRules } from '../rules/storybook.mjs';
+import { customRules } from '../rules/custom.mjs';
 
 export default defineConfig({
   overrides: [
     {
       files: ['**/*.stories.ts', '**/*.stories.tsx', '**/*.stories.js', '**/*.stories.jsx'],
-      jsPlugins: storybookRules.jsPlugins,
-      rules: storybookRules.rules,
+      jsPlugins: [...(storybookRules.jsPlugins || []), ...(customRules.jsPlugins || [])],
+      rules: {
+        ...storybookRules.rules,
+        // Decision: stories always default-export an object named `meta`, so
+        // the filename can never match the exported name.
+        'custom/match-exported': 'off',
+      },
     },
   ],
 });

package/configs/typescript.mjs

@@ -4,6 +4,7 @@ import { importRules } from '../rules/import.mjs';
 import { promiseRules } from '../rules/promise.mjs';
 import { typescriptRules } from '../rules/typescript.mjs';
 import { barrelFilesRules } from '../rules/barrel-files.mjs';
+import { customRules } from '../rules/custom.mjs';
 import { unicornRules } from '../rules/unicorn.mjs';
 
 export default defineConfig({
@@ -17,13 +18,18 @@ export default defineConfig({
     ...promiseRules.plugins,
     ...typescriptRules.plugins,
   ],
-  jsPlugins: [...(importRules.jsPlugins || []), ...(barrelFilesRules.jsPlugins || [])],
+  jsPlugins: [
+    ...(importRules.jsPlugins || []),
+    ...(barrelFilesRules.jsPlugins || []),
+    ...(customRules.jsPlugins || []),
+  ],
   rules: {
     ...baseRules.rules,
     ...importRules.rules,
     ...promiseRules.rules,
     ...typescriptRules.rules,
     ...barrelFilesRules.rules,
+    ...customRules.rules,
     ...unicornRules.rules,
   },
 });

package/index.d.mts

@@ -21,6 +21,7 @@ export declare const storybookRules: RuleFile;
 export declare const styledComponentsRules: RuleFile;
 export declare const testingLibraryRules: RuleFile;
 export declare const barrelFilesRules: RuleFile;
+export declare const customRules: RuleFile;
 
 // Configs
 export declare const javascriptConfig: OxlintConfig;

package/index.mjs

@@ -13,6 +13,7 @@ export { storybookRules } from './rules/storybook.mjs';
 export { styledComponentsRules } from './rules/styled-components.mjs';
 export { testingLibraryRules } from './rules/testing-library.mjs';
 export { barrelFilesRules } from './rules/barrel-files.mjs';
+export { customRules } from './rules/custom.mjs';
 
 // Configs
 export { default as javascriptConfig } from './configs/javascript.mjs';

package/package.json

@@ -1,12 +1,13 @@
 {
   "name": "@wistia/oxlint-config",
-  "version": "1.1.1",
+  "version": "1.3.0",
   "description": "Wistia's Oxlint configurations",
   "packageManager": "yarn@4.17.0",
   "type": "module",
   "files": [
     "configs",
     "rules",
+    "plugins",
     "index.mjs",
     "index.d.mts"
   ],
@@ -27,6 +28,7 @@
     "./testing-library": "./configs/testing-library.mjs",
     "./rules/barrel-files": "./rules/barrel-files.mjs",
     "./rules/base": "./rules/base.mjs",
+    "./rules/custom": "./rules/custom.mjs",
     "./rules/import": "./rules/import.mjs",
     "./rules/node": "./rules/node.mjs",
     "./rules/playwright": "./rules/playwright.mjs",

package/plugins/custom.mjs

@@ -0,0 +1,188 @@
+// `custom` — Wistia's custom oxlint jsPlugin, holding rules with no native
+// oxlint or loadable-ESLint-plugin equivalent.
+//
+// `custom/match-exported` is a faithful port of `match-exported` from the
+// abandoned eslint-plugin-filenames (https://github.com/selaux/eslint-plugin-filenames),
+// rewritten in oxlint's modern `{ meta, create }` plugin format. The original
+// ships its rules as plain functions, which oxlint's jsPlugin runner cannot
+// load — see the ESLint-parity section of the README.
+//
+// The rule ensures a file's name matches the name of its default export
+// (`export default ...`) or `module.exports = ...`. Files with no default
+// export, or an anonymous/object default export, are ignored. `index.*` files
+// are matched against their parent directory name instead.
+import path from 'node:path';
+
+// --- case transforms (lodash-equivalent for identifier-shaped input) ----------
+const splitWords = (name) =>
+  name
+    .replace(/([a-z0-9])([A-Z])/g, '$1 $2') // camelCase boundary
+    .replace(/([A-Z]+)([A-Z][a-z])/g, '$1 $2') // ACRONYMCase boundary
+    .replace(/([a-zA-Z])([0-9])/g, '$1 $2') // letter→digit boundary (lodash treats digit runs as words)
+    .replace(/([0-9])([a-zA-Z])/g, '$1 $2') // digit→letter boundary
+    .split(/[^A-Za-z0-9]+/)
+    .filter(Boolean);
+
+const upperFirst = (word) => word.charAt(0).toUpperCase() + word.slice(1);
+
+const transforms = {
+  kebab: (name) =>
+    splitWords(name)
+      .map((w) => w.toLowerCase())
+      .join('-'),
+  snake: (name) =>
+    splitWords(name)
+      .map((w) => w.toLowerCase())
+      .join('_'),
+  camel: (name) =>
+    splitWords(name)
+      .map((w, i) => (i === 0 ? w.toLowerCase() : upperFirst(w.toLowerCase())))
+      .join(''),
+  pascal: (name) =>
+    splitWords(name)
+      .map((w) => upperFirst(w.toLowerCase()))
+      .join(''),
+};
+
+// --- helpers ported from the original plugin ----------------------------------
+const ignoredFilenames = ['<text>', '<input>'];
+const isIgnoredFilename = (filename) => ignoredFilenames.includes(filename);
+
+const parseFilename = (filename) => {
+  const ext = path.extname(filename);
+  return {
+    dir: path.dirname(filename),
+    base: path.basename(filename),
+    ext,
+    name: path.basename(filename, ext),
+  };
+};
+
+const isIndexFile = (parsed) => parsed.name === 'index';
+
+const getStringToCheckAgainstExport = (parsed, replacePattern) => {
+  const dirArray = parsed.dir.split(path.sep);
+  const lastDirectory = dirArray[dirArray.length - 1];
+
+  if (isIndexFile(parsed)) {
+    return lastDirectory;
+  }
+  return replacePattern ? parsed.name.replace(replacePattern, '') : parsed.name;
+};
+
+const getNodeName = (node, options) => {
+  const op = options || [];
+
+  if (node.type === 'Identifier') {
+    return node.name;
+  }
+  if (node.id && node.id.type === 'Identifier') {
+    return node.id.name;
+  }
+  if (op[2] && node.type === 'CallExpression' && node.callee.type === 'Identifier') {
+    return node.callee.name;
+  }
+  return undefined;
+};
+
+const getExportedName = (programNode, options) => {
+  for (const node of programNode.body) {
+    // export default ...
+    if (node.type === 'ExportDefaultDeclaration') {
+      return getNodeName(node.declaration, options);
+    }
+
+    // module.exports = ...
+    if (
+      node.type === 'ExpressionStatement' &&
+      node.expression.type === 'AssignmentExpression' &&
+      node.expression.left.type === 'MemberExpression' &&
+      node.expression.left.object.type === 'Identifier' &&
+      node.expression.left.object.name === 'module' &&
+      node.expression.left.property.type === 'Identifier' &&
+      node.expression.left.property.name === 'exports'
+    ) {
+      return getNodeName(node.expression.right, options);
+    }
+  }
+  return undefined;
+};
+
+const getTransformsFromOptions = (option) => {
+  const usedTransforms = Array.isArray(option) ? option : [option];
+  return usedTransforms.map((name) => transforms[name]);
+};
+
+const applyTransforms = (exportedName, usedTransforms) =>
+  usedTransforms.map((t) => (t ? t(exportedName) : exportedName));
+
+const anyMatch = (expectedExport, transformedNames) =>
+  transformedNames.some((name) => name === expectedExport);
+
+const getWhatToMatchMessage = (usedTransforms) => {
+  if (usedTransforms.length === 1 && !usedTransforms[0]) {
+    return 'the exported name';
+  }
+  if (usedTransforms.length > 1) {
+    return 'any of the exported and transformed names';
+  }
+  return 'the exported and transformed name';
+};
+
+const matchExported = {
+  meta: {
+    schema: [
+      {
+        oneOf: [
+          { enum: ['kebab', 'snake', 'camel', 'pascal', null] },
+          {
+            type: 'array',
+            items: { enum: ['kebab', 'snake', 'camel', 'pascal', null] },
+            minItems: 1,
+          },
+        ],
+      },
+      { type: ['string', 'null'] },
+      { type: ['boolean', 'null'] },
+    ],
+  },
+  create(context) {
+    const options = context.options || [];
+    const usedTransforms = getTransformsFromOptions(options[0]);
+    const replacePattern = options[1] ? new RegExp(options[1]) : null;
+    const filename =
+      context.physicalFilename ??
+      context.filename ??
+      (typeof context.getFilename === 'function' ? context.getFilename() : '');
+
+    return {
+      Program(node) {
+        if (isIgnoredFilename(filename)) return;
+
+        const parsed = parseFilename(path.resolve(filename));
+        const exportedName = getExportedName(node, options);
+        const isExporting = Boolean(exportedName);
+        if (!isExporting) return;
+
+        const expectedExport = getStringToCheckAgainstExport(parsed, replacePattern);
+        const transformedNames = applyTransforms(exportedName, usedTransforms);
+        const everythingIsIndex = exportedName === 'index' && parsed.name === 'index';
+        const matchesExported = anyMatch(expectedExport, transformedNames) || everythingIsIndex;
+
+        if (matchesExported) return;
+
+        const exportName = transformedNames.join("', '");
+        const message = isIndexFile(parsed)
+          ? `The directory '${expectedExport}' must be named '${exportName}', after the exported value of its index file.`
+          : `Filename '${expectedExport}' must match ${getWhatToMatchMessage(usedTransforms)} '${exportName}'.`;
+
+        context.report({ node, message });
+      },
+    };
+  },
+};
+
+export default {
+  meta: { name: 'custom' },
+  rules: { 'match-exported': matchExported },
+};

package/rules/custom.mjs

@@ -0,0 +1,21 @@
+import { fileURLToPath } from 'node:url';
+
+// `customRules` configures Wistia's custom local jsPlugin (plugins/custom.mjs),
+// which holds rules that have no native oxlint or loadable-ESLint-plugin
+// equivalent. The plugin path is resolved to an absolute path so it loads both
+// in this repo and from a consumer's node_modules.
+//
+// `custom/match-exported` ports `match-exported` from the abandoned
+// eslint-plugin-filenames (it ships its rules in a legacy format oxlint cannot
+// load). The upstream `match-regex` and `no-index` rules are intentionally not
+// implemented — @wistia/eslint-config disables both.
+const customPlugin = fileURLToPath(new URL('../plugins/custom.mjs', import.meta.url));
+
+export const customRules = {
+  jsPlugins: [customPlugin],
+  rules: {
+    // Match Exported Values
+    // https://github.com/selaux/eslint-plugin-filenames#matching-exported-values-match-exported
+    'custom/match-exported': 'error',
+  },
+};

package/rules/import.mjs

@@ -195,7 +195,7 @@ export const importRules = {
     'import-x-js/order': [
       'error',
       {
-        groups: [['builtin', 'external', 'internal'], 'index', ['parent', 'sibling']],
+        groups: ['type', ['builtin', 'external', 'internal'], 'index', ['parent', 'sibling']],
         'newlines-between': 'never',
       },
     ],

package/rules/react-a11y.mjs

@@ -2,15 +2,15 @@ export const reactA11yRules = {
   plugins: ['jsx-a11y'],
   rules: {
     // Enforce that all elements that require alternative text have meaningful information
-    // https://oxc.rs/docs/guide/usage/linter/rules/jsx-a11y/alt-text.html
+    // https://oxc.rs/docs/guide/usage/linter/rules/jsx_a11y/alt-text.html
     'jsx_a11y/alt-text': 'error',
 
     // Enforce that anchors have content
-    // https://oxc.rs/docs/guide/usage/linter/rules/jsx-a11y/anchor-has-content.html
+    // https://oxc.rs/docs/guide/usage/linter/rules/jsx_a11y/anchor-has-content.html
     'jsx_a11y/anchor-has-content': 'error',
 
     // Enforce all anchors are valid, navigable elements
-    // https://oxc.rs/docs/guide/usage/linter/rules/jsx-a11y/anchor-is-valid.html
+    // https://oxc.rs/docs/guide/usage/linter/rules/jsx_a11y/anchor-is-valid.html
     'jsx_a11y/anchor-is-valid': [
       'error',
       {
@@ -21,40 +21,40 @@ export const reactA11yRules = {
     ],
 
     // Enforce that elements with aria-activedescendant have tabindex
-    // https://oxc.rs/docs/guide/usage/linter/rules/jsx-a11y/aria-activedescendant-has-tabindex.html
+    // https://oxc.rs/docs/guide/usage/linter/rules/jsx_a11y/aria-activedescendant-has-tabindex.html
     'jsx_a11y/aria-activedescendant-has-tabindex': 'error',
 
     // Enforce that elements have valid aria-* props
-    // https://oxc.rs/docs/guide/usage/linter/rules/jsx-a11y/aria-props.html
+    // https://oxc.rs/docs/guide/usage/linter/rules/jsx_a11y/aria-props.html
     'jsx_a11y/aria-props': 'error',
 
     // Enforce that ARIA state and property values are valid
-    // https://oxc.rs/docs/guide/usage/linter/rules/jsx-a11y/aria-proptypes.html
+    // https://oxc.rs/docs/guide/usage/linter/rules/jsx_a11y/aria-proptypes.html
     'jsx_a11y/aria-proptypes': 'error',
 
     // Enforce that elements with ARIA roles have all required attributes for that role
-    // https://oxc.rs/docs/guide/usage/linter/rules/jsx-a11y/role-has-required-aria-props.html
+    // https://oxc.rs/docs/guide/usage/linter/rules/jsx_a11y/role-has-required-aria-props.html
     'jsx_a11y/role-has-required-aria-props': 'error',
 
     // Enforce that elements with explicit or implicit roles defined contain only aria-* properties supported by that role
-    // https://oxc.rs/docs/guide/usage/linter/rules/jsx-a11y/role-supports-aria-props.html
+    // https://oxc.rs/docs/guide/usage/linter/rules/jsx_a11y/role-supports-aria-props.html
     'jsx_a11y/role-supports-aria-props': 'error',
 
     // Enforce that elements with ARIA roles must use a valid, non-abstract ARIA role
-    // https://oxc.rs/docs/guide/usage/linter/rules/jsx-a11y/aria-role.html
+    // https://oxc.rs/docs/guide/usage/linter/rules/jsx_a11y/aria-role.html
     'jsx_a11y/aria-role': 'error',
 
     // Enforce that certain elements don't have ARIA roles, states, or properties
-    // https://oxc.rs/docs/guide/usage/linter/rules/jsx-a11y/aria-unsupported-elements.html
+    // https://oxc.rs/docs/guide/usage/linter/rules/jsx_a11y/aria-unsupported-elements.html
     'jsx_a11y/aria-unsupported-elements': 'error',
 
     // Enforce that autocomplete attribute is correct
-    // https://oxc.rs/docs/guide/usage/linter/rules/jsx-a11y/autocomplete-valid.html
+    // https://oxc.rs/docs/guide/usage/linter/rules/jsx_a11y/autocomplete-valid.html
     // Decision: too opinionated for general use
     'jsx_a11y/autocomplete-valid': 'off',
 
     // Enforce that a control (an interactive element) has a text label
-    // https://oxc.rs/docs/guide/usage/linter/rules/jsx-a11y/control-has-associated-label.html
+    // https://oxc.rs/docs/guide/usage/linter/rules/jsx_a11y/control-has-associated-label.html
     'jsx_a11y/control-has-associated-label': [
       'error',
       {
@@ -78,27 +78,27 @@ export const reactA11yRules = {
     ],
 
     // Enforce a clickable non-interactive element has at least one keyboard event listener
-    // https://oxc.rs/docs/guide/usage/linter/rules/jsx-a11y/click-events-have-key-events.html
+    // https://oxc.rs/docs/guide/usage/linter/rules/jsx_a11y/click-events-have-key-events.html
     'jsx_a11y/click-events-have-key-events': 'error',
 
     // Enforce heading elements have content
-    // https://oxc.rs/docs/guide/usage/linter/rules/jsx-a11y/heading-has-content.html
+    // https://oxc.rs/docs/guide/usage/linter/rules/jsx_a11y/heading-has-content.html
     'jsx_a11y/heading-has-content': ['error', { components: [''] }],
 
     // Enforce <html> element has lang prop
-    // https://oxc.rs/docs/guide/usage/linter/rules/jsx-a11y/html-has-lang.html
+    // https://oxc.rs/docs/guide/usage/linter/rules/jsx_a11y/html-has-lang.html
     'jsx_a11y/html-has-lang': 'error',
 
     // Enforce iframe elements have a title attribute
-    // https://oxc.rs/docs/guide/usage/linter/rules/jsx-a11y/iframe-has-title.html
+    // https://oxc.rs/docs/guide/usage/linter/rules/jsx_a11y/iframe-has-title.html
     'jsx_a11y/iframe-has-title': 'error',
 
     // Enforce <img> alt prop does not contain the word "image", "picture", or "photo"
-    // https://oxc.rs/docs/guide/usage/linter/rules/jsx-a11y/img-redundant-alt.html
+    // https://oxc.rs/docs/guide/usage/linter/rules/jsx_a11y/img-redundant-alt.html
     'jsx_a11y/img-redundant-alt': 'error',
 
     // Enforce that a label tag has a text label and an associated control
-    // https://oxc.rs/docs/guide/usage/linter/rules/jsx-a11y/label-has-associated-control.html
+    // https://oxc.rs/docs/guide/usage/linter/rules/jsx_a11y/label-has-associated-control.html
     'jsx_a11y/label-has-associated-control': [
       'error',
       {
@@ -111,31 +111,31 @@ export const reactA11yRules = {
     ],
 
     // Enforce lang attribute has a valid value
-    // https://oxc.rs/docs/guide/usage/linter/rules/jsx-a11y/lang.html
+    // https://oxc.rs/docs/guide/usage/linter/rules/jsx_a11y/lang.html
     'jsx_a11y/lang': 'error',
 
     // Enforce that media elements have captions
-    // https://oxc.rs/docs/guide/usage/linter/rules/jsx-a11y/media-has-caption.html
+    // https://oxc.rs/docs/guide/usage/linter/rules/jsx_a11y/media-has-caption.html
     'jsx_a11y/media-has-caption': 'error',
 
     // Enforce that onMouseOver/onMouseOut are accompanied by onFocus/onBlur
-    // https://oxc.rs/docs/guide/usage/linter/rules/jsx-a11y/mouse-events-have-key-events.html
+    // https://oxc.rs/docs/guide/usage/linter/rules/jsx_a11y/mouse-events-have-key-events.html
     'jsx_a11y/mouse-events-have-key-events': 'error',
 
     // Enforce that the accessKey prop is not used on any element
-    // https://oxc.rs/docs/guide/usage/linter/rules/jsx-a11y/no-access-key.html
+    // https://oxc.rs/docs/guide/usage/linter/rules/jsx_a11y/no-access-key.html
     'jsx_a11y/no-access-key': 'error',
 
     // Enforce autoFocus prop is not used
-    // https://oxc.rs/docs/guide/usage/linter/rules/jsx-a11y/no-autofocus.html
+    // https://oxc.rs/docs/guide/usage/linter/rules/jsx_a11y/no-autofocus.html
     'jsx_a11y/no-autofocus': ['error', { ignoreNonDOM: true }],
 
     // Enforce distracting elements are not used
-    // https://oxc.rs/docs/guide/usage/linter/rules/jsx-a11y/no-distracting-elements.html
+    // https://oxc.rs/docs/guide/usage/linter/rules/jsx_a11y/no-distracting-elements.html
     'jsx_a11y/no-distracting-elements': 'error',
 
     // Enforce tabIndex value is not greater than zero
-    // https://oxc.rs/docs/guide/usage/linter/rules/jsx-a11y/tabindex-no-positive.html
+    // https://oxc.rs/docs/guide/usage/linter/rules/jsx_a11y/tabindex-no-positive.html
     'jsx_a11y/tabindex-no-positive': 'error',
 
     // Elements with an interactive role and interaction handlers must be focusable
@@ -143,7 +143,7 @@ export const reactA11yRules = {
     'jsx_a11y/interactive-supports-focus': 'error',
 
     // Ensure interactive elements are not assigned non-interactive roles
-    // https://oxc.rs/docs/guide/usage/linter/rules/jsx-a11y/no-noninteractive-tabindex.html
+    // https://oxc.rs/docs/guide/usage/linter/rules/jsx_a11y/no-noninteractive-tabindex.html
     'jsx_a11y/no-noninteractive-tabindex': [
       'error',
       {
@@ -153,21 +153,21 @@ export const reactA11yRules = {
     ],
 
     // Ensure interactive elements are not assigned non-interactive roles
-    // https://oxc.rs/docs/guide/usage/linter/rules/jsx-a11y/no-interactive-element-to-noninteractive-role.html
+    // https://oxc.rs/docs/guide/usage/linter/rules/jsx_a11y/no-interactive-element-to-noninteractive-role.html
     'jsx_a11y/no-interactive-element-to-noninteractive-role': [
       'error',
       { tr: ['none', 'presentation'] },
     ],
 
     // Enforce that non-interactive elements do not have interaction handlers
-    // https://oxc.rs/docs/guide/usage/linter/rules/jsx-a11y/no-noninteractive-element-interactions.html
+    // https://oxc.rs/docs/guide/usage/linter/rules/jsx_a11y/no-noninteractive-element-interactions.html
     'jsx_a11y/no-noninteractive-element-interactions': [
       'error',
       { handlers: ['onClick', 'onMouseDown', 'onMouseUp', 'onKeyPress', 'onKeyDown', 'onKeyUp'] },
     ],
 
     // Enforce that non-interactive elements are not assigned interactive roles
-    // https://oxc.rs/docs/guide/usage/linter/rules/jsx-a11y/no-noninteractive-element-to-interactive-role.html
+    // https://oxc.rs/docs/guide/usage/linter/rules/jsx_a11y/no-noninteractive-element-to-interactive-role.html
     'jsx_a11y/no-noninteractive-element-to-interactive-role': [
       'error',
       {
@@ -180,27 +180,27 @@ export const reactA11yRules = {
     ],
 
     // Enforce explicit role is not redundant with implicit role of the element
-    // https://oxc.rs/docs/guide/usage/linter/rules/jsx-a11y/no-redundant-roles.html
+    // https://oxc.rs/docs/guide/usage/linter/rules/jsx_a11y/no-redundant-roles.html
     'jsx_a11y/no-redundant-roles': 'error',
 
     // Enforce that non-interactive, visible elements with click handlers use the role attribute
-    // https://oxc.rs/docs/guide/usage/linter/rules/jsx-a11y/no-static-element-interactions.html
+    // https://oxc.rs/docs/guide/usage/linter/rules/jsx_a11y/no-static-element-interactions.html
     'jsx_a11y/no-static-element-interactions': 'error',
 
     // Enforce scope prop is only used on <th> elements
-    // https://oxc.rs/docs/guide/usage/linter/rules/jsx-a11y/scope.html
+    // https://oxc.rs/docs/guide/usage/linter/rules/jsx_a11y/scope.html
     'jsx_a11y/scope': 'error',
 
     // Enforce that anchor elements have non-ambiguous text content
-    // https://oxc.rs/docs/guide/usage/linter/rules/jsx-a11y/anchor-ambiguous-text.html
+    // https://oxc.rs/docs/guide/usage/linter/rules/jsx_a11y/anchor-ambiguous-text.html
     'jsx_a11y/anchor-ambiguous-text': 'error',
 
     // Enforce that aria-hidden="true" is not set on focusable elements
-    // https://oxc.rs/docs/guide/usage/linter/rules/jsx-a11y/no-aria-hidden-on-focusable.html
+    // https://oxc.rs/docs/guide/usage/linter/rules/jsx_a11y/no-aria-hidden-on-focusable.html
     'jsx_a11y/no-aria-hidden-on-focusable': 'error',
 
     // Prefer semantic HTML elements over role attributes
-    // https://oxc.rs/docs/guide/usage/linter/rules/jsx-a11y/prefer-tag-over-role.html
+    // https://oxc.rs/docs/guide/usage/linter/rules/jsx_a11y/prefer-tag-over-role.html
     // Decision: left to implementer
     'jsx_a11y/prefer-tag-over-role': 'off',
   },