@oxlint/migrate 1.23.0 → 1.25.0

package/dist/bin/oxlint-migrate.mjs

@@ -9,6 +9,7 @@ import { walkAndReplaceProjectFiles } from "../src/walker/index.mjs";
 import { getAllProjectFiles } from "./project-loader.mjs";
 import { writeFile } from "node:fs/promises";
 import { preFixForJsPlugins } from "../src/js_plugin_fixes.mjs";
+import { DefaultReporter } from "../src/reporter.mjs";
 const cwd = process.cwd();
 const getFileContent = (absoluteFilePath) => {
   try {
@@ -38,8 +39,9 @@ program.name("oxlint-migrate").version(packageJson.version).argument("[eslint-co
 ).action(async (filePath) => {
   const cliOptions = program.opts();
   const oxlintFilePath = path.join(cwd, cliOptions.outputFile);
+  const reporter = new DefaultReporter();
   const options = {
-    reporter: console.warn,
+    reporter,
     merge: !!cliOptions.merge,
     withNursery: !!cliOptions.withNursery,
     typeAware: !!cliOptions.typeAware
@@ -75,5 +77,8 @@ program.name("oxlint-migrate").version(packageJson.version).argument("[eslint-co
     renameSync(oxlintFilePath, `${oxlintFilePath}.bak`);
   }
   writeFileSync(oxlintFilePath, JSON.stringify(oxlintConfig, null, 2));
+  for (const report of reporter.getReports()) {
+    console.warn(report);
+  }
 });
 program.parse();

package/dist/package.json.mjs

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

package/dist/src/cleanup.mjs

@@ -1,5 +1,5 @@
 import { removeGlobalsWithAreCoveredByEnv, transformBoolGlobalToString, ES_VERSIONS, cleanUpUselessOverridesEnv } from "./env_globals.mjs";
-import { replaceTypescriptAliasRules, replaceNodePluginName, cleanUpRulesWhichAreCoveredByCategory, cleanUpUselessOverridesRules, cleanUpUselessOverridesPlugins } from "./plugins_rules.mjs";
+import { replaceTypescriptAliasRules, replaceNodePluginName, cleanUpRulesWhichAreCoveredByCategory, cleanUpDisabledRootRules, cleanUpUselessOverridesRules, cleanUpUselessOverridesPlugins } from "./plugins_rules.mjs";
 import { isEqualDeep } from "./utilities.mjs";
 const TS_ESLINT_DEFAULT_OVERRIDE = {
   files: ["**/*.ts", "**/*.tsx", "**/*.mts", "**/*.cts"],
@@ -81,6 +81,7 @@ const cleanUpOxlintConfig = (config) => {
   }
   if (!("files" in config)) {
     cleanUpUselessOverridesEntries(config);
+    cleanUpDisabledRootRules(config);
   }
 };
 export {

package/dist/src/env_globals.mjs

@@ -107,14 +107,16 @@ const detectEnvironmentByGlobals = (config) => {
   }
 };
 const transformEnvAndGlobals = (eslintConfig, targetConfig, options) => {
-  if (eslintConfig.languageOptions?.parser !== void 0 && !SUPPORTED_ESLINT_PARSERS.includes(
+  if (eslintConfig.languageOptions?.parser !== void 0 && eslintConfig.languageOptions?.parser !== null && typeof eslintConfig.languageOptions.parser === "object" && "meta" in eslintConfig.languageOptions.parser && !SUPPORTED_ESLINT_PARSERS.includes(
+    // @ts-ignore
     eslintConfig.languageOptions.parser.meta?.name
   )) {
-    options?.reporter !== void 0 && options.reporter(
-      "special parser detected: " + eslintConfig.languageOptions.parser.meta?.name
+    options?.reporter?.report(
+      "special parser detected: " + // @ts-ignore
+      eslintConfig.languageOptions.parser.meta?.name
     );
   }
-  if (eslintConfig.languageOptions?.globals !== void 0) {
+  if (eslintConfig.languageOptions?.globals !== void 0 && eslintConfig.languageOptions?.globals !== null) {
     if (targetConfig.globals === void 0) {
       targetConfig.globals = {};
     }
@@ -131,7 +133,7 @@ const transformEnvAndGlobals = (eslintConfig, targetConfig, options) => {
     }
   }
   if (eslintConfig.languageOptions?.ecmaVersion !== void 0) {
-    if (eslintConfig.languageOptions?.ecmaVersion === "latest") {
+    if (eslintConfig.languageOptions.ecmaVersion === "latest") {
       if (targetConfig.env === void 0) {
         targetConfig.env = {};
       }
@@ -139,11 +141,11 @@ const transformEnvAndGlobals = (eslintConfig, targetConfig, options) => {
       if (!(latestVersion in targetConfig.env)) {
         targetConfig.env[latestVersion] = true;
       }
-    } else if (ES_VERSIONS.includes(eslintConfig.languageOptions?.ecmaVersion)) {
+    } else if (typeof eslintConfig.languageOptions.ecmaVersion === "number" && ES_VERSIONS.includes(eslintConfig.languageOptions.ecmaVersion)) {
       if (targetConfig.env === void 0) {
         targetConfig.env = {};
       }
-      const targetVersion = `es${eslintConfig.languageOptions?.ecmaVersion}`;
+      const targetVersion = `es${eslintConfig.languageOptions.ecmaVersion}`;
       if (!(targetVersion in targetConfig.env)) {
         targetConfig.env[targetVersion] = true;
       }

package/dist/src/generated/rules.d.ts

@@ -2,6 +2,6 @@ export declare const pedanticRules: string[];
 export declare const styleRules: string[];
 export declare const suspiciousRules: string[];
 export declare const restrictionRules: string[];
-export declare const correctnessRules: string[];
 export declare const nurseryRules: string[];
+export declare const correctnessRules: string[];
 export declare const perfRules: string[];

package/dist/src/generated/rules.mjs

@@ -20,6 +20,7 @@ const pedanticRules = [
   "no-redeclare",
   "no-self-compare",
   "no-throw-literal",
+  "no-warning-comments",
   "radix",
   "require-await",
   "sort-vars",
@@ -56,6 +57,7 @@ const pedanticRules = [
   "@typescript-eslint/require-await",
   "@typescript-eslint/restrict-plus-operands",
   "@typescript-eslint/return-await",
+  "@typescript-eslint/strict-boolean-expressions",
   "@typescript-eslint/switch-exhaustiveness-check",
   "unicorn/consistent-assert",
   "unicorn/consistent-empty-array-spread",
@@ -267,9 +269,9 @@ const styleRules = [
   "vitest/prefer-to-be-falsy",
   "vitest/prefer-to-be-object",
   "vitest/prefer-to-be-truthy",
-  "vue/define-props-destructuring",
   "vue/define-emits-declaration",
   "vue/define-props-declaration",
+  "vue/define-props-destructuring",
   "vue/require-typed-ref",
   "@typescript-eslint/default-param-last",
   "@typescript-eslint/init-declarations",
@@ -402,6 +404,7 @@ const restrictionRules = [
   "promise/catch-or-return",
   "promise/spec-only",
   "react/button-has-type",
+  "react/forbid-dom-props",
   "react/forbid-elements",
   "react/jsx-filename-extension",
   "react/no-danger",
@@ -449,6 +452,19 @@ const restrictionRules = [
   "import-x/no-webpack-loader-syntax",
   "import-x/unambiguous"
 ];
+const nurseryRules = [
+  "constructor-super",
+  "getter-return",
+  "no-misleading-character-class",
+  "no-undef",
+  "no-unreachable",
+  "import/export",
+  "import/named",
+  "promise/no-return-in-finally",
+  "react/require-render-return",
+  "import-x/export",
+  "import-x/named"
+];
 const correctnessRules = [
   "for-direction",
   "no-unassigned-vars",
@@ -649,18 +665,6 @@ const correctnessRules = [
   "vitest/valid-describe-callback",
   "vitest/valid-expect"
 ];
-const nurseryRules = [
-  "getter-return",
-  "no-misleading-character-class",
-  "no-undef",
-  "no-unreachable",
-  "import/export",
-  "import/named",
-  "promise/no-return-in-finally",
-  "react/require-render-return",
-  "import-x/export",
-  "import-x/named"
-];
 const perfRules = [
   "no-await-in-loop",
   "no-useless-call",

package/dist/src/ignorePatterns.mjs

@@ -3,7 +3,7 @@ const transformIgnorePatterns = (eslintConfig, targetConfig, options) => {
     return;
   }
   if ("files" in targetConfig) {
-    options?.reporter !== void 0 && options.reporter("ignore list inside overrides is not supported");
+    options?.reporter?.report("ignore list inside overrides is not supported");
     return;
   }
   if (targetConfig.ignorePatterns === void 0) {

package/dist/src/plugins_rules.d.ts

@@ -5,6 +5,7 @@ export declare const detectNeededRulesPlugins: (targetConfig: OxlintConfigOrOver
 export declare const cleanUpUselessOverridesPlugins: (config: OxlintConfig) => void;
 export declare const cleanUpUselessOverridesRules: (config: OxlintConfig) => void;
 export declare const cleanUpRulesWhichAreCoveredByCategory: (config: OxlintConfigOrOverride) => void;
+export declare const cleanUpDisabledRootRules: (config: OxlintConfig) => void;
 export declare const replaceTypescriptAliasRules: (config: OxlintConfigOrOverride) => void;
 /**
  * Oxlint support them only under the node plugin name

package/dist/src/plugins_rules.mjs

@@ -43,11 +43,13 @@ const transformRuleEntry = (eslintConfig, targetConfig, options) => {
   for (const [rule, config] of Object.entries(eslintConfig.rules)) {
     if (allRules.includes(rule)) {
       if (!options?.withNursery && nurseryRules.includes(rule)) {
-        options?.reporter !== void 0 && options.reporter(`unsupported rule, but in development: ${rule}`);
+        options?.reporter?.report(
+          `unsupported rule, but in development: ${rule}`
+        );
         continue;
       }
       if (!options?.typeAware && typescriptTypeAwareRules.includes(rule)) {
-        options?.reporter !== void 0 && options.reporter(
+        options?.reporter?.report(
           `type-aware rule detected, but \`--type-aware\` is not enabled: ${rule}`
         );
         continue;
@@ -61,7 +63,7 @@ const transformRuleEntry = (eslintConfig, targetConfig, options) => {
       }
     } else {
       if (isActiveValue(config)) {
-        options?.reporter !== void 0 && options.reporter(`unsupported rule: ${rule}`);
+        options?.reporter?.report(`unsupported rule: ${rule}`);
       }
     }
   }
@@ -87,7 +89,7 @@ const detectNeededRulesPlugins = (targetConfig, options) => {
       }
     }
     if (!found) {
-      options?.reporter !== void 0 && options.reporter(`unsupported plugin for rule: ${rule}`);
+      options?.reporter?.report(`unsupported plugin for rule: ${rule}`);
     }
   }
   if ("files" in targetConfig && targetConfig.plugins.length === 0) {
@@ -146,6 +148,36 @@ const cleanUpRulesWhichAreCoveredByCategory = (config) => {
     }
   }
 };
+const getEnabledCategories = (config) => {
+  if (config.categories === void 0) {
+    return ["correctness"];
+  }
+  const categories = Object.entries(config.categories).filter(([, severity]) => severity === "warn" || severity === "error").map(([category]) => category);
+  if (Object.keys(config.categories).includes("correctness")) {
+    return categories;
+  }
+  return [...categories, "correctness"];
+};
+const isRuleInEnabledCategory = (rule, enabledCategories) => {
+  for (const category of enabledCategories) {
+    if (`${category}Rules` in rules && // @ts-expect-error -- ts can not resolve the type
+    rules[`${category}Rules`].includes(rule)) {
+      return true;
+    }
+  }
+  return false;
+};
+const cleanUpDisabledRootRules = (config) => {
+  if (config.rules === void 0) {
+    return;
+  }
+  const enabledCategories = getEnabledCategories(config);
+  for (const [rule, settings] of Object.entries(config.rules)) {
+    if (isOffValue(settings) && !isRuleInEnabledCategory(rule, enabledCategories)) {
+      delete config.rules[rule];
+    }
+  }
+};
 const replaceTypescriptAliasRules = (config) => {
   if (config.rules === void 0) {
     return;
@@ -181,6 +213,7 @@ const replaceNodePluginName = (config) => {
   }
 };
 export {
+  cleanUpDisabledRootRules,
   cleanUpRulesWhichAreCoveredByCategory,
   cleanUpUselessOverridesPlugins,
   cleanUpUselessOverridesRules,

package/dist/src/reporter.d.ts

@@ -0,0 +1,10 @@
+import { Reporter } from './types.js';
+export declare class DefaultReporter implements Reporter {
+    private reports;
+    report(message: string): void;
+    getReports(): string[];
+}
+export declare class SilentReporter implements Reporter {
+    report(_message: string): void;
+    getReports(): string[];
+}

package/dist/src/reporter.mjs

@@ -0,0 +1,12 @@
+class DefaultReporter {
+  reports = /* @__PURE__ */ new Set();
+  report(message) {
+    this.reports.add(message);
+  }
+  getReports() {
+    return Array.from(this.reports);
+  }
+}
+export {
+  DefaultReporter
+};

package/dist/src/types.d.ts

@@ -22,7 +22,10 @@ export type OxlintConfig = {
     ignorePatterns?: OxlintConfigIgnorePatterns;
 };
 export type OxlintConfigOrOverride = OxlintConfig | OxlintConfigOverride;
-type Reporter = (warning: string) => void;
+export type Reporter = {
+    report(message: string): void;
+    getReports(): string[];
+};
 export type Options = {
     reporter?: Reporter;
     merge?: boolean;

package/dist/src/walker/replaceCommentsInFile.mjs

@@ -10,8 +10,8 @@ const getComments = (absoluteFilePath, partialSourceText, options) => {
       sourceType: partialSourceText.sourceType
     }
   );
-  if (parserResult.errors.length > 0 && options.reporter) {
-    options.reporter(`${absoluteFilePath}: failed to parse`);
+  if (parserResult.errors.length > 0) {
+    options.reporter?.report(`${absoluteFilePath}: failed to parse`);
   }
   return parserResult.comments;
 };
@@ -26,8 +26,8 @@ function replaceCommentsInSourceText(absoluteFilePath, partialSourceText, option
         sourceText = sourceText.slice(0, comment.start) + newComment + sourceText.slice(comment.end);
       }
     } catch (error) {
-      if (error instanceof Error && options.reporter) {
-        options.reporter(
+      if (error instanceof Error) {
+        options.reporter?.report(
           `${absoluteFilePath}, char offset ${comment.start + partialSourceText.offset}: ${error.message}`
         );
         continue;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@oxlint/migrate",
-  "version": "1.23.0",
+  "version": "1.25.0",
   "description": "Generates a `.oxlintrc.json` from a existing eslint flat config",
   "type": "module",
   "bin": {
@@ -37,10 +37,9 @@
   "author": "Sysix <sysix@sysix-coding.de>",
   "license": "MIT",
   "devDependencies": {
-    "@antfu/eslint-config": "^5.0.0",
-    "@eslint/eslintrc": "^3.3.1",
+    "@antfu/eslint-config": "^6.0.0",
     "@eslint/js": "^9.29.0",
-    "@logux/eslint-config": "^56.0.0",
+    "@logux/eslint-config": "^57.0.0",
     "@oxc-node/core": "^0.0.32",
     "@stylistic/eslint-plugin": "^5.0.0",
     "@stylistic/eslint-plugin-ts": "^4.4.1",
@@ -48,33 +47,36 @@
     "@types/node": "^24.0.4",
     "@typescript-eslint/eslint-plugin": "^8.35.0",
     "@typescript-eslint/parser": "^8.35.0",
-    "@vitest/coverage-v8": "^3.2.4",
+    "@vitest/coverage-v8": "^4.0.0",
     "eslint": "^9.29.0",
-    "eslint-config-next": "^15.5.0",
+    "eslint-config-next": "^16.0.0",
     "eslint-config-prettier": "^10.1.5",
     "eslint-plugin-header": "^3.1.1",
     "eslint-plugin-import": "^2.32.0",
     "eslint-plugin-import-x": "^4.16.0",
-    "eslint-plugin-jsdoc": "^60.0.0",
+    "eslint-plugin-jsdoc": "^61.0.0",
     "eslint-plugin-local": "^6.0.0",
+    "eslint-plugin-mocha": "^11.2.0",
     "eslint-plugin-oxlint": "^1.3.0",
+    "eslint-plugin-prettier": "^5.5.4",
     "eslint-plugin-react": "^7.37.5",
-    "eslint-plugin-react-hooks": "^5.2.0",
+    "eslint-plugin-react-hooks": "^7.0.1",
     "eslint-plugin-react-perf": "^3.3.3",
     "eslint-plugin-regexp": "^2.9.0",
-    "eslint-plugin-unicorn": "^61.0.0",
+    "eslint-plugin-tsdoc": "^0.4.0",
+    "eslint-plugin-unicorn": "^62.0.0",
     "husky": "^9.1.7",
     "jiti": "^2.4.2",
     "lint-staged": "^16.1.2",
-    "next": "^15.5.0",
-    "oxlint": "^1.23.0",
+    "next": "^16.0.0",
+    "oxlint": "^1.25.0",
     "oxlint-tsgolint": "^0.2.0",
     "prettier": "^3.6.1",
     "typescript": "^5.8.3",
     "typescript-eslint": "^8.35.0",
     "vite": "^7.0.0",
     "vite-plugin-dts": "^4.5.4",
-    "vitest": "^3.2.4"
+    "vitest": "^4.0.0"
   },
   "lint-staged": {
     "*": "prettier --ignore-unknown --write"
@@ -82,11 +84,11 @@
   "dependencies": {
     "commander": "^14.0.0",
     "globals": "^16.3.0",
-    "oxc-parser": "^0.94.0",
+    "oxc-parser": "^0.95.0",
     "tinyglobby": "^0.2.14"
   },
   "peerDependencies": {
     "jiti": "*"
   },
-  "packageManager": "pnpm@10.18.2"
+  "packageManager": "pnpm@10.19.0"
 }