@oxlint/migrate 1.25.0 → 1.26.0

package/README.md

@@ -21,6 +21,7 @@ When no config file provided, the script searches for the default eslint config
 | `--merge`                   | \* merge eslint configuration with an existing .oxlintrc.json configuration                                                                 |
 | `--type-aware`              | Include type aware rules, which are supported with `oxlint --type-aware`                                                                    |
 | `--with-nursery`            | Include oxlint rules which are currently under development                                                                                  |
+| `--js-plugins`              | \*\* Include ESLint plugins via `jsPlugins` key.                                                                                            |
 | `--output-file <file>`      | The oxlint configuration file where to eslint v9 rules will be written to, default: `.oxlintrc.json`                                        |
 | `--replace-eslint-comments` | Search in the project files for eslint comments and replaces them with oxlint. Some eslint comments are not supported and will be reported. |
 
@@ -28,6 +29,10 @@ When no config file provided, the script searches for the default eslint config
 Else we need to disable each rule `plugin/categories` combination, which is not covered by your eslint configuration.
 This behavior can change in the future.
 
+\*\* WARNING: Tries to guess the plugin name. Should work with most of the plugin names.
+Not every ESLint API is integrated with `oxlint`.
+Tested ESLint Plugins with `oxlint` can be found in this [Oxc Discussion](https://github.com/oxc-project/oxc/discussions/14862).
+
 ### User Flow
 
 - Upgrade `oxlint` and `@oxlint/migrate` to the same version.

package/dist/bin/oxlint-migrate.mjs

@@ -36,6 +36,9 @@ program.name("oxlint-migrate").version(packageJson.version).argument("[eslint-co
 ).option(
   "--type-aware",
   "Includes supported type-aware rules. Needs the same flag in `oxlint` to enable it."
+).option(
+  "--js-plugins",
+  "Tries to convert unsupported oxlint plugins with `jsPlugins`."
 ).action(async (filePath) => {
   const cliOptions = program.opts();
   const oxlintFilePath = path.join(cwd, cliOptions.outputFile);
@@ -44,7 +47,8 @@ program.name("oxlint-migrate").version(packageJson.version).argument("[eslint-co
     reporter,
     merge: !!cliOptions.merge,
     withNursery: !!cliOptions.withNursery,
-    typeAware: !!cliOptions.typeAware
+    typeAware: !!cliOptions.typeAware,
+    jsPlugins: !!cliOptions.jsPlugins
   };
   if (cliOptions.replaceEslintComments) {
     await walkAndReplaceProjectFiles(

package/dist/package.json.mjs

@@ -1,4 +1,4 @@
-const version = "1.25.0";
+const version = "1.26.0";
 const packageJson = {
   version
 };

package/dist/src/generated/rules.mjs

@@ -35,6 +35,7 @@ const pedanticRules = [
   "jsdoc/require-returns-description",
   "jsdoc/require-returns-type",
   "react/checked-requires-onchange-or-readonly",
+  "react/jsx-no-target-blank",
   "react/jsx-no-useless-fragment",
   "react/no-unescaped-entities",
   "react-hooks/rules-of-hooks",
@@ -42,6 +43,7 @@ const pedanticRules = [
   "@typescript-eslint/ban-types",
   "@typescript-eslint/no-misused-promises",
   "@typescript-eslint/no-confusing-void-expression",
+  "@typescript-eslint/no-deprecated",
   "@typescript-eslint/no-mixed-enums",
   "@typescript-eslint/no-unsafe-argument",
   "@typescript-eslint/no-unsafe-assignment",
@@ -216,6 +218,7 @@ const styleRules = [
   "react/no-set-state",
   "react/prefer-es6-class",
   "react/self-closing-comp",
+  "react/state-in-constructor",
   "@typescript-eslint/adjacent-overload-signatures",
   "@typescript-eslint/array-type",
   "@typescript-eslint/ban-tslint-comment",
@@ -595,7 +598,6 @@ const correctnessRules = [
   "react/forward-ref-uses-ref",
   "react/jsx-key",
   "react/jsx-no-duplicate-props",
-  "react/jsx-no-target-blank",
   "react/jsx-no-undef",
   "react/jsx-props-no-spread-multi",
   "react/no-children-prop",

package/dist/src/index.mjs

@@ -59,13 +59,13 @@ const buildConfig = (configs, oxlintConfig, options) => {
     transformRuleEntry(config, targetConfig, options);
     transformEnvAndGlobals(config, targetConfig, options);
     if ("files" in targetConfig) {
-      detectNeededRulesPlugins(targetConfig, options);
+      detectNeededRulesPlugins(targetConfig);
       detectEnvironmentByGlobals(targetConfig);
       cleanUpOxlintConfig(targetConfig);
     }
   }
   oxlintConfig.overrides = overrides;
-  detectNeededRulesPlugins(oxlintConfig, options);
+  detectNeededRulesPlugins(oxlintConfig);
   detectEnvironmentByGlobals(oxlintConfig);
   cleanUpOxlintConfig(oxlintConfig);
   return oxlintConfig;

package/dist/src/jsPlugins.d.ts

@@ -0,0 +1,3 @@
+import { Linter } from 'eslint';
+import { OxlintConfigOrOverride } from './types.js';
+export declare const enableJsPluginRule: (targetConfig: OxlintConfigOrOverride, rule: string, ruleEntry: Linter.RuleEntry | undefined) => boolean;

package/dist/src/jsPlugins.mjs

@@ -0,0 +1,52 @@
+import { rulesPrefixesForPlugins } from "./constants.mjs";
+const ignorePlugins = /* @__PURE__ */ new Set([
+  ...Object.keys(rulesPrefixesForPlugins),
+  ...Object.values(rulesPrefixesForPlugins),
+  "local"
+  // ToDo: handle local plugin rules
+]);
+const guessEslintPluginName = (pluginName) => {
+  if (pluginName.startsWith("@")) {
+    const [scope, maybeSub] = pluginName.split("/");
+    if (maybeSub) {
+      return `${scope}/eslint-plugin-${maybeSub}`;
+    }
+    return `${scope}/eslint-plugin`;
+  }
+  return `eslint-plugin-${pluginName}`;
+};
+const extractPluginId = (ruleId) => {
+  const firstSlash = ruleId.indexOf("/");
+  if (firstSlash === -1) {
+    return;
+  }
+  if (ruleId.startsWith("@")) {
+    const secondSlash = ruleId.indexOf("/", firstSlash + 1);
+    if (secondSlash !== -1) {
+      return ruleId.substring(0, secondSlash);
+    }
+  }
+  return ruleId.substring(0, firstSlash);
+};
+const enableJsPluginRule = (targetConfig, rule, ruleEntry) => {
+  const pluginName = extractPluginId(rule);
+  if (pluginName === void 0) {
+    return false;
+  }
+  if (ignorePlugins.has(pluginName)) {
+    return false;
+  }
+  if (targetConfig.jsPlugins === void 0) {
+    targetConfig.jsPlugins = [];
+  }
+  const eslintPluginName = guessEslintPluginName(pluginName);
+  if (!targetConfig.jsPlugins.includes(eslintPluginName)) {
+    targetConfig.jsPlugins.push(eslintPluginName);
+  }
+  targetConfig.rules = targetConfig.rules || {};
+  targetConfig.rules[rule] = ruleEntry;
+  return true;
+};
+export {
+  enableJsPluginRule
+};

package/dist/src/js_plugin_fixes.mjs

@@ -3,7 +3,8 @@ const fixForAntfuEslintConfig = (config) => {
     return config.renamePlugins({
       ts: "@typescript-eslint",
       test: "vitest",
-      next: "@next/next"
+      next: "@next/next",
+      style: "@stylistic"
     });
   }
   return config;

package/dist/src/plugins_rules.d.ts

@@ -1,7 +1,7 @@
 import { Linter } from 'eslint';
 import { Options, OxlintConfig, OxlintConfigOrOverride } from './types.js';
 export declare const transformRuleEntry: (eslintConfig: Linter.Config, targetConfig: OxlintConfigOrOverride, options?: Options) => void;
-export declare const detectNeededRulesPlugins: (targetConfig: OxlintConfigOrOverride, options?: Options) => void;
+export declare const detectNeededRulesPlugins: (targetConfig: OxlintConfigOrOverride) => void;
 export declare const cleanUpUselessOverridesPlugins: (config: OxlintConfig) => void;
 export declare const cleanUpUselessOverridesRules: (config: OxlintConfig) => void;
 export declare const cleanUpRulesWhichAreCoveredByCategory: (config: OxlintConfigOrOverride) => void;

package/dist/src/plugins_rules.mjs

@@ -1,6 +1,7 @@
 import * as rules from "./generated/rules.mjs";
 import { nurseryRules } from "./generated/rules.mjs";
 import { typescriptTypeAwareRules, rulesPrefixesForPlugins, typescriptRulesExtendEslintRules } from "./constants.mjs";
+import { enableJsPluginRule } from "./jsPlugins.mjs";
 const allRules = Object.values(rules).flat();
 const isValueInSet = (value, validSet) => validSet.includes(value) || Array.isArray(value) && validSet.includes(value[0]);
 const isActiveValue = (value) => isValueInSet(value, ["error", "warn", 1, 2]);
@@ -62,13 +63,21 @@ const transformRuleEntry = (eslintConfig, targetConfig, options) => {
         targetConfig.rules[rule] = normalizeSeverityValue(config);
       }
     } else {
-      if (isActiveValue(config)) {
+      if (options?.jsPlugins) {
+        if (isActiveValue(config) && !enableJsPluginRule(
+          targetConfig,
+          rule,
+          normalizeSeverityValue(config)
+        )) {
+          options?.reporter?.report(`unsupported rule: ${rule}`);
+        }
+      } else if (isActiveValue(config)) {
         options?.reporter?.report(`unsupported rule: ${rule}`);
       }
     }
   }
 };
-const detectNeededRulesPlugins = (targetConfig, options) => {
+const detectNeededRulesPlugins = (targetConfig) => {
   if (targetConfig.rules === void 0) {
     return;
   }
@@ -79,18 +88,11 @@ const detectNeededRulesPlugins = (targetConfig, options) => {
     if (!rule.includes("/")) {
       continue;
     }
-    let found = false;
     for (const [prefix, plugin] of Object.entries(rulesPrefixesForPlugins)) {
-      if (rule.startsWith(`${prefix}/`)) {
-        if (!targetConfig.plugins.includes(plugin)) {
-          targetConfig.plugins.push(plugin);
-        }
-        found = true;
+      if (rule.startsWith(`${prefix}/`) && !targetConfig.plugins.includes(plugin)) {
+        targetConfig.plugins.push(plugin);
       }
     }
-    if (!found) {
-      options?.reporter?.report(`unsupported plugin for rule: ${rule}`);
-    }
   }
   if ("files" in targetConfig && targetConfig.plugins.length === 0) {
     delete targetConfig.plugins;

package/dist/src/types.d.ts

@@ -1,5 +1,6 @@
 import { Linter } from 'eslint';
 type OxlintConfigPlugins = string[];
+type OxlintConfigJsPlugins = string[];
 type OxlintConfigCategories = Record<string, unknown>;
 type OxlintConfigEnv = Record<string, boolean>;
 type OxlintConfigIgnorePatterns = string[];
@@ -8,6 +9,7 @@ export type OxlintConfigOverride = {
     env?: OxlintConfigEnv;
     globals?: Linter.Globals;
     plugins?: OxlintConfigPlugins;
+    jsPlugins?: OxlintConfigJsPlugins;
     categories?: OxlintConfigCategories;
     rules?: Partial<Linter.RulesRecord>;
 };
@@ -16,6 +18,7 @@ export type OxlintConfig = {
     env?: OxlintConfigEnv;
     globals?: Linter.Globals;
     plugins?: OxlintConfigPlugins;
+    jsPlugins?: OxlintConfigJsPlugins;
     categories?: OxlintConfigCategories;
     rules?: Partial<Linter.RulesRecord>;
     overrides?: OxlintConfigOverride[];
@@ -31,5 +34,6 @@ export type Options = {
     merge?: boolean;
     withNursery?: boolean;
     typeAware?: boolean;
+    jsPlugins?: boolean;
 };
 export {};

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@oxlint/migrate",
-  "version": "1.25.0",
+  "version": "1.26.0",
   "description": "Generates a `.oxlintrc.json` from a existing eslint flat config",
   "type": "module",
   "bin": {
@@ -69,7 +69,7 @@
     "jiti": "^2.4.2",
     "lint-staged": "^16.1.2",
     "next": "^16.0.0",
-    "oxlint": "^1.25.0",
+    "oxlint": "^1.26.0",
     "oxlint-tsgolint": "^0.2.0",
     "prettier": "^3.6.1",
     "typescript": "^5.8.3",
@@ -84,11 +84,11 @@
   "dependencies": {
     "commander": "^14.0.0",
     "globals": "^16.3.0",
-    "oxc-parser": "^0.95.0",
+    "oxc-parser": "^0.96.0",
     "tinyglobby": "^0.2.14"
   },
   "peerDependencies": {
     "jiti": "*"
   },
-  "packageManager": "pnpm@10.19.0"
+  "packageManager": "pnpm@10.20.0"
 }