@wistia/oxlint-config 0.8.0 → 0.8.1

package/README.md

@@ -163,4 +163,20 @@ export default defineConfig({
 
 ## ESLint parity
 
-See [docs/eslint-parity.md](docs/eslint-parity.md) for a full list of differences between this package and `@wistia/eslint-config`.
+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.
+
+Several categories of rules cannot use this approach:
+
+1. **Core ESLint rules** (e.g. `no-restricted-globals`, `camelcase`, `object-shorthand`). These are built into the `eslint` package, not exported as a plugin with a `{ rules }` object. `@eslint/js` only exports configs, not rules. There is no standalone ESLint plugin that re-exports core rules in the format oxlint's jsPlugins expect.
+
+2. **Rules requiring ESLint parser services** (e.g. `@eslint-react/*`, `@typescript-eslint/*`). These call `getParserServices()` from `@typescript-eslint/utils` internally, which requires ESLint's parser infrastructure. oxlint's jsPlugin runner does not provide this.
+
+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 }`).

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@wistia/oxlint-config",
-  "version": "0.8.0",
+  "version": "0.8.1",
   "description": "Wistia's Oxlint configurations",
   "packageManager": "yarn@4.14.1",
   "type": "module",

package/rules/base.mjs

@@ -325,6 +325,8 @@ export const baseRules = {
 
     // Disallow the use of console
     // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-console.html
+    // Decision: allow console methods that are commonly used for logging and error reporting,
+    // but disallow others that are often left in accidentally
     'eslint/no-console': ['error', { allow: ['log', 'warn', 'error'] }],
 
     // Disallow continue statements

package/rules/import.mjs

@@ -175,27 +175,8 @@ export const importRules = {
 
     // Forbid the use of extraneous packages
     // https://github.com/un-ts/eslint-plugin-import-x/blob/master/docs/rules/no-extraneous-dependencies.md
-    'import-x-js/no-extraneous-dependencies': [
-      'error',
-      {
-        devDependencies: [
-          '**/*{.,_}{test,vitest,spec,stories}.{js,jsx,ts,tsx}',
-          '**/oxfmt.config.*',
-          '**/.oxfmtrc.json*',
-          '**/vite.config.{ts,js,cjs,mjs,mts}',
-          '**/vitest.config.{ts,js,cjs,mjs,mts}',
-          '**/eslint.config.{js,cjs,mjs,mts}',
-          '**/.eslintrc.{js,cjs}',
-          '**/.stylelintrc.{cjs,js,json,yaml,yml}',
-          '**/stylelint.config.{cjs,mjs,mts,js}',
-          '**/esbuild.config.{js,cjs,mjs,mts}',
-          '**/tsup.config.{js,cjs,mjs,mts}',
-          '**/rollup.config.{js,cjs,mjs,mts}',
-          '**/rollup.config.*.{js,cjs,mjs,mts}',
-        ],
-        optionalDependencies: false,
-      },
-    ],
+    // Decision: this is better handled by knip
+    'import-x-js/no-extraneous-dependencies': 'off',
 
     // Reports if a default export is renamed during import
     // https://github.com/un-ts/eslint-plugin-import-x/blob/master/docs/rules/no-rename-default.md