@oxlint/migrate 1.71.0 → 1.73.0

package/README.md

@@ -115,3 +115,11 @@ Depending on how this is implemented by the given config, these behaviors may no
 The JS Plugins API supports almost all ESLint v9+ plugins for linting JS/TS/JSX/TSX files, but there are still some minor holes in support. See the [JS Plugins documentation](https://oxc.rs/docs/guide/usage/linter/js-plugins.html) for specifics.
 
 For example, if you currently use `eslint-plugin-svelte`, only some of its rules will work via JS Plugin, as Oxlint does not yet fully support custom file formats like `.svelte`. This means that Oxlint will only be able to lint the script blocks in your `.svelte` files, and not the template blocks where many of the `eslint-plugin-svelte` rules apply.
+
+## ❤ Who's [Sponsoring Oxc](https://github.com/sponsors/Boshen)?
+
+<p align="center">
+  <a href="https://github.com/sponsors/Boshen">
+    <img src="https://raw.githubusercontent.com/Boshen/sponsors/main/sponsors.svg" alt="Our sponsors" />
+  </a>
+</p>

package/dist/bin/oxlint-migrate.mjs

@@ -1,5 +1,5 @@
 #!/usr/bin/env node
-import { a as nurseryRules, i as buildUnsupportedRuleExplanations, n as preFixForJsPlugins, o as rules_exports, r as isOffValue, s as normalizeRuleToCanonical, t as main } from "../src-CLLHiI8g.mjs";
+import { a as nurseryRules, i as buildUnsupportedRuleExplanations, n as preFixForJsPlugins, o as rules_exports, r as isOffValue, s as normalizeRuleToCanonical, t as main } from "../src-aj7n0D2s.mjs";
 import { program } from "commander";
 import { existsSync, readFileSync, renameSync, writeFileSync } from "node:fs";
 import path from "node:path";
@@ -30,7 +30,7 @@ const loadESLintConfig = async (filePath) => {
 };
 //#endregion
 //#region package.json
-var version = "1.71.0";
+var version = "1.73.0";
 //#endregion
 //#region src/walker/comments/replaceRuleDirectiveComment.ts
 const allRulesSet = new Set(Object.values(rules_exports).flat());
@@ -414,6 +414,25 @@ const getFileContent = (absoluteFilePath) => {
 	}
 };
 /**
+* Strips JSON comments (single-line `//` and multi-line) and trailing commas
+* so that `tsconfig.json` (which is actually JSONC) can be parsed with `JSON.parse`.
+*/
+const stripJsoncFeatures = (text) => text.replace(/\/\/.*$/gm, "").replace(/\/\*[\s\S]*?\*\//g, "").replace(/,\s*([}\]])/g, "$1");
+/**
+* When `--type-aware` is used, check whether the project's `tsconfig.json`
+* contains a `baseUrl` value in `compilerOptions`. If it does, emit a warning
+* because `tsgo` does not support `baseUrl`, which may cause issues with
+* type-aware linting.
+*/
+const warnIfTsconfigHasBaseUrl = (reporter) => {
+	const content = getFileContent(path.join(cwd, "tsconfig.json"));
+	if (content === void 0) return;
+	try {
+		const tsconfig = JSON.parse(stripJsoncFeatures(content));
+		if (tsconfig?.compilerOptions?.baseUrl !== void 0) reporter.addWarning(`tsconfig.json has \`baseUrl\` set to ${JSON.stringify(tsconfig.compilerOptions.baseUrl)}. \`baseUrl\` is not supported by tsgo, which may cause issues with type-aware linting. Consider removing \`baseUrl\` or using path aliases instead.`);
+	} catch {}
+};
+/**
 * Count enabled rules (excluding "off" rules) from both rules and overrides
 */
 const countEnabledRules = (config) => {
@@ -440,6 +459,7 @@ program.name("oxlint-migrate").version(version).argument("[eslint-config]", "The
 		typeAware: !!cliOptions.typeAware,
 		jsPlugins
 	};
+	if (options.typeAware) warnIfTsconfigHasBaseUrl(reporter);
 	if (cliOptions.replaceEslintComments) {
 		await walkAndReplaceProjectFiles(await getAllProjectFiles(), (filePath) => getFileContent(filePath), (filePath, content) => writeFile(filePath, content, "utf-8"), options);
 		return;

package/dist/src/index.d.mts

@@ -4,7 +4,7 @@
 /**
  * What to enforce for `/// <reference types="..." />` references.
  */
-//#region node_modules/.pnpm/oxlint@1.71.0_oxlint-tsgolint@0.23.0/node_modules/oxlint/dist/index.d.ts
+//#region node_modules/.pnpm/oxlint@1.73.0_oxlint-tsgolint@0.24.0/node_modules/oxlint/dist/index.d.ts
 //#region src-js/package/config.generated.d.ts
 type AllowWarnDeny = ("allow" | "off" | "warn" | "error" | "deny") | number;
 type GlobalValue = ("readonly" | "writable" | "off") | undefined;
@@ -116,11 +116,15 @@ type AllowKind = "functions" | "arrowFunctions" | "generatorFunctions" | "method
 type NoInnerDeclarationsConfig = "functions" | "both";
 type BlockScopedFunctions = "allow" | "disallow";
 type NoMagicNumbersNumber = number | string;
+type NoRestrictedImportsConfigEnum = string | RestrictedPath | NoRestrictedImportsConfig;
+type PossiblePaths = string | RestrictedPath;
+type PossiblePatterns = string | RestrictedPattern;
 type NoReturnAssignMode = "always" | "except-parens";
 /**
  * Controls how hoisting is handled when checking for shadowing.
  */
 type HoistOption = "all" | "functions" | "functions-and-types" | "never" | "types";
+type LoopType = "WhileStatement" | "DoWhileStatement" | "ForStatement" | "ForInStatement" | "ForOfStatement";
 type NoUnusedVarsConfig = VarsOption | NoUnusedVarsOptions;
 type VarsOption = "all" | "local";
 type ArgsOption = "after-used" | "all" | "none";
@@ -142,6 +146,7 @@ type CallbackReturn = string[];
  * - a regexp pattern (e.g. `"^(err|error)$"`)
  *
  * If the configured name of the error variable begins with a `^` it is considered to be a regexp pattern.
+ * Invalid regexp patterns are rejected during configuration parsing.
  *
  * Default: `"err"`.
  */
@@ -292,6 +297,7 @@ type PathOption = "always" | "never";
 type TypesOption = "always" | "never" | "prefer-import";
 type BomOptionType = "always" | "never";
 type NonZero = "greater-than" | "not-equal";
+type ExplicitTimerDelayMode = "always" | "never";
 type ModuleStylesOverride = (false | {
   /**
    * Whether default imports or whole-module `require()` assignments are allowed for this module.
@@ -1012,7 +1018,7 @@ interface DummyRuleMap {
   "no-cond-assign"?: RuleNoConfig | [AllowWarnDeny, NoCondAssignConfig];
   "no-console"?: RuleNoConfig | [AllowWarnDeny, NoConsoleConfig];
   "no-const-assign"?: RuleNoConfig;
-  "no-constant-binary-expression"?: RuleNoConfig;
+  "no-constant-binary-expression"?: RuleNoConfig | [AllowWarnDeny, NoConstantBinaryExpressionConfig];
   "no-constant-condition"?: RuleNoConfig | [AllowWarnDeny, NoConstantCondition];
   "no-constructor-return"?: RuleNoConfig;
   "no-continue"?: RuleNoConfig;
@@ -1078,7 +1084,7 @@ interface DummyRuleMap {
   "no-regex-spaces"?: RuleNoConfig;
   "no-restricted-exports"?: RuleNoConfig | [AllowWarnDeny, NoRestrictedExportsConfig];
   "no-restricted-globals"?: DummyRule;
-  "no-restricted-imports"?: DummyRule;
+  "no-restricted-imports"?: RuleNoConfig | [AllowWarnDeny, NoRestrictedImportsConfigEnum, ...NoRestrictedImportsConfigEnum[]];
   "no-restricted-properties"?: RuleNoConfig | [AllowWarnDeny, PropertyDetails, ...PropertyDetails[]];
   "no-return-assign"?: RuleNoConfig | [AllowWarnDeny, NoReturnAssignMode];
   "no-script-url"?: RuleNoConfig;
@@ -1101,6 +1107,7 @@ interface DummyRuleMap {
   "no-unmodified-loop-condition"?: RuleNoConfig;
   "no-unneeded-ternary"?: RuleNoConfig | [AllowWarnDeny, NoUnneededTernary];
   "no-unreachable"?: RuleNoConfig;
+  "no-unreachable-loop"?: RuleNoConfig | [AllowWarnDeny, NoUnreachableLoopConfig];
   "no-unsafe-finally"?: RuleNoConfig;
   "no-unsafe-negation"?: RuleNoConfig | [AllowWarnDeny, NoUnsafeNegation];
   "no-unsafe-optional-chaining"?: RuleNoConfig | [AllowWarnDeny, NoUnsafeOptionalChaining];
@@ -1388,6 +1395,7 @@ interface DummyRuleMap {
   "unicorn/error-message"?: RuleNoConfig;
   "unicorn/escape-case"?: RuleNoConfig;
   "unicorn/explicit-length-check"?: RuleNoConfig | [AllowWarnDeny, ExplicitLengthCheck];
+  "unicorn/explicit-timer-delay"?: RuleNoConfig | [AllowWarnDeny, ExplicitTimerDelayMode];
   "unicorn/filename-case"?: DummyRule;
   "unicorn/import-style"?: RuleNoConfig | [AllowWarnDeny, ImportStyleConfig];
   "unicorn/max-nested-calls"?: RuleNoConfig | [AllowWarnDeny, MaxNestedCalls];
@@ -1404,6 +1412,7 @@ interface DummyRuleMap {
   "unicorn/no-array-sort"?: RuleNoConfig | [AllowWarnDeny, NoArraySort];
   "unicorn/no-await-expression-member"?: RuleNoConfig;
   "unicorn/no-await-in-promise-methods"?: RuleNoConfig;
+  "unicorn/no-confusing-array-with"?: RuleNoConfig;
   "unicorn/no-console-spaces"?: RuleNoConfig;
   "unicorn/no-document-cookie"?: RuleNoConfig;
   "unicorn/no-empty-file"?: RuleNoConfig;
@@ -2265,6 +2274,7 @@ interface PreferEndingWithAnExpectConfig {
   additionalTestBlockFunctions?: string[];
   /**
    * A list of function names that should be treated as assertion functions.
+   * Default: `["expect"]`
    */
   assertFunctionNames?: string[];
 }
@@ -2789,6 +2799,9 @@ interface NoConsoleConfig {
    */
   allow?: string[];
 }
+interface NoConstantBinaryExpressionConfig {
+  checkRelationalComparisons?: boolean;
+}
 interface NoConstantCondition {
   /**
    * Configuration option to specify whether to check for constant conditions in loops.
@@ -3305,6 +3318,28 @@ interface RestrictDefaultExports {
    */
   namespaceFrom?: boolean;
 }
+interface RestrictedPath {
+  allowImportNames?: string[];
+  allowTypeImports?: boolean;
+  importNames?: string[];
+  message?: string;
+  name: string;
+}
+interface NoRestrictedImportsConfig {
+  paths?: PossiblePaths[];
+  patterns?: PossiblePatterns[];
+}
+interface RestrictedPattern {
+  allowImportNamePattern?: string;
+  allowImportNames?: string[];
+  allowTypeImports?: boolean;
+  caseSensitive?: boolean;
+  group?: string[];
+  importNamePattern?: string;
+  importNames?: string[];
+  message?: string;
+  regex?: string;
+}
 interface PropertyDetails {
   /**
    * Objects where property access should be allowed. This must be used with `property` and
@@ -3443,6 +3478,9 @@ interface NoUnneededTernary {
    */
   defaultAssignment?: boolean;
 }
+interface NoUnreachableLoopConfig {
+  ignore?: LoopType[];
+}
 interface NoUnsafeNegation {
   /**
    * The `enforceForOrderingRelations` option determines whether negation is allowed
@@ -4227,20 +4265,20 @@ interface ForbidItemObject {
    * Component names for which this prop is **allowed** (all others are
    * forbidden).
    */
-  allowedFor: string[];
+  allowedFor?: string[];
   /**
    * Glob patterns for component names where the prop is **allowed**.
    */
-  allowedForPatterns: string[];
+  allowedForPatterns?: string[];
   /**
    * Component names for which this prop is **disallowed** (all others are
    * allowed).
    */
-  disallowedFor: string[];
+  disallowedFor?: string[];
   /**
    * Glob patterns for component names where the prop is **disallowed**.
    */
-  disallowedForPatterns: string[];
+  disallowedForPatterns?: string[];
   /**
    * Custom message to display.
    */
@@ -4262,7 +4300,7 @@ interface ForbidDomPropsConfig {
    * An array of prop names or objects that are forbidden on DOM elements.
    *
    * Each array element can be a string with the property name, or an object
-   * with `propName`, an optional `disallowedFor` array of DOM node names,
+   * with `propName`, optional `disallowedFor` and `disallowedValues` arrays,
    * and an optional custom `message`.
    *
    * Examples:
@@ -4270,11 +4308,13 @@ interface ForbidDomPropsConfig {
    * - `["error", { "forbid": ["id", "style"] }]`
    * - `["error", { "forbid": [{ "propName": "className", "message": "Use class instead" }] }]`
    * - `["error", { "forbid": [{ "propName": "style", "disallowedFor": ["div", "span"] }] }]`
+   * - `["error", { "forbid": [{ "propName": "type", "disallowedValues": ["button"] }] }]`
    */
   forbid?: ForbidDomPropsItem[];
 }
 /**
- * A prop with optional `disallowedFor` DOM node list and custom `message`.
+ * A prop with optional `disallowedFor` DOM node list, optional `disallowedValues`
+ * value list, and custom `message`.
  */
 interface PropWithOptions {
   /**
@@ -4283,6 +4323,11 @@ interface PropWithOptions {
    * DOM elements.
    */
   disallowedFor?: string[];
+  /**
+   * A list of string literal values for which this prop is forbidden. If
+   * omitted, the prop is forbidden for all values.
+   */
+  disallowedValues?: string[];
   /**
    * A custom message to display when this prop is used.
    */
@@ -5934,6 +5979,16 @@ interface NoArrayReverse {
   allowExpressionStatement?: boolean;
 }
 interface NoArraySort {
+  /**
+   * When set to `true`, allows sorting a fresh array created by a spread, e.g. `[...iterable].sort()`.
+   * This avoids the double allocation of `toSorted()` when sorting an iterable such as a `Set`.
+   *
+   * Example of **correct** code for this rule with `allowAfterSpread` set to `true`:
+   * ```js
+   * const sorted = [...mySet].sort();
+   * ```
+   */
+  allowAfterSpread?: boolean;
   /**
    * When set to `true` (default), allows `array.sort()` as an expression statement.
    * Set to `false` to forbid `Array#sort()` even if it's an expression statement.

package/dist/src/index.mjs

@@ -1,2 +1,2 @@
-import { t as main } from "../src-CLLHiI8g.mjs";
+import { t as main } from "../src-aj7n0D2s.mjs";
 export { main as default };

package/dist/{src-CLLHiI8g.mjs → src-aj7n0D2s.mjs}

@@ -706,6 +706,7 @@ const styleRules = [
 	"unicorn/custom-error-definition",
 	"unicorn/empty-brace-spaces",
 	"unicorn/error-message",
+	"unicorn/explicit-timer-delay",
 	"unicorn/filename-case",
 	"unicorn/max-nested-calls",
 	"unicorn/no-array-method-this-argument",
@@ -862,6 +863,7 @@ const suspiciousRules = [
 	"unicorn/no-array-fill-with-reference-type",
 	"unicorn/no-array-reverse",
 	"unicorn/no-array-sort",
+	"unicorn/no-confusing-array-with",
 	"unicorn/no-instanceof-builtins",
 	"unicorn/prefer-add-event-listener",
 	"unicorn/require-module-specifiers",
@@ -1238,6 +1240,7 @@ const perfRules = [
 const nurseryRules = [
 	"no-restricted-exports",
 	"no-undef",
+	"no-unreachable-loop",
 	"no-useless-assignment",
 	"import/export",
 	"import/named",
@@ -1351,6 +1354,15 @@ var unsupportedRules = {
 	"import/no-deprecated": "No need to implement, already implemented by `typescript/no-deprecated` via tsgolint.",
 	"n/no-restricted-import": "No need to implement, already implemented by `no-restricted-imports` rule.",
 	"n/no-restricted-require": "No need to implement, already implemented by `no-restricted-imports` rule.",
+	"n/prefer-global/buffer": "No need to implement, the same policy can be enforced with generic restriction rules such as `no-restricted-imports` and `no-restricted-globals`.",
+	"n/prefer-global/console": "No need to implement, the same policy can be enforced with generic restriction rules such as `no-restricted-imports` and `no-restricted-globals`.",
+	"n/prefer-global/crypto": "No need to implement, the same policy can be enforced with generic restriction rules such as `no-restricted-imports` and `no-restricted-globals`.",
+	"n/prefer-global/process": "No need to implement, the same policy can be enforced with generic restriction rules such as `no-restricted-imports` and `no-restricted-globals`.",
+	"n/prefer-global/text-decoder": "No need to implement, the same policy can be enforced with generic restriction rules such as `no-restricted-imports` and `no-restricted-globals`.",
+	"n/prefer-global/text-encoder": "No need to implement, the same policy can be enforced with generic restriction rules such as `no-restricted-imports` and `no-restricted-globals`.",
+	"n/prefer-global/timers": "No need to implement, the same policy can be enforced with generic restriction rules such as `no-restricted-imports` and `no-restricted-globals`.",
+	"n/prefer-global/url": "No need to implement, the same policy can be enforced with generic restriction rules such as `no-restricted-imports` and `no-restricted-globals`.",
+	"n/prefer-global/url-search-params": "No need to implement, the same policy can be enforced with generic restriction rules such as `no-restricted-imports` and `no-restricted-globals`.",
 	"import/order": "Not implementing this in Oxlint as its behavior is covered very well by [Oxfmt's import sorting](https://oxc.rs/docs/guide/usage/formatter/sorting.html).",
 	"unicorn/template-indent": "Stylistic rule.",
 	"jsdoc/type-formatting": "Experimental rule in the original plugin, may reconsider once stable.",
@@ -1518,6 +1530,7 @@ var unsupportedRules = {
 	"react/jsx-uses-vars": "Handled by `eslint/no-unused-vars`, which already evaluates whether vars are used in JSX.",
 	"unicorn/no-named-default": "Implemented via `import/no-named-default`.",
 	"unicorn/no-double-comparison": "Use `oxc/double-comparisons` instead.",
+	"unicorn/no-chained-comparison": "Use `oxc/bad-comparison-sequence` instead. TypeScript also reports errors for these comparisons.",
 	"vue/no-lone-template": "Not currently possible, as it requires Vue template parsing.",
 	"vue/no-v-html": "Not currently possible, as it requires Vue template parsing.",
 	"vue/this-in-template": "Not currently possible, as it requires Vue template parsing.",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@oxlint/migrate",
-  "version": "1.71.0",
+  "version": "1.73.0",
   "description": "Generates a `.oxlintrc.json` from a existing eslint flat config",
   "keywords": [
     "eslint",
@@ -40,12 +40,12 @@
   "dependencies": {
     "commander": "^15.0.0",
     "globals": "^17.0.0",
-    "oxc-parser": "^0.137.0",
+    "oxc-parser": "^0.138.0",
     "tinyglobby": "^0.2.14"
   },
   "devDependencies": {
     "@antfu/eslint-config": "^9.0.0",
-    "@babel/eslint-plugin": "^7.27.1",
+    "@babel/eslint-plugin": "^8.0.0",
     "@e18e/eslint-plugin": "^0.5.0",
     "@eslint-react/eslint-plugin": "^5.0.0",
     "@eslint/js": "^10.0.0",
@@ -79,13 +79,13 @@
     "eslint-plugin-react-web-api": "^5.0.0",
     "eslint-plugin-regexp": "^3.0.0",
     "eslint-plugin-tsdoc": "^0.5.0",
-    "eslint-plugin-unicorn": "^65.0.0",
+    "eslint-plugin-unicorn": "^67.0.0",
     "husky": "^9.1.7",
     "lint-staged": "^17.0.0",
     "next": "^16.0.0",
-    "oxfmt": "^0.55.0",
-    "oxlint": "^1.71.0",
-    "oxlint-tsgolint": "^0.23.0",
+    "oxfmt": "^0.57.0",
+    "oxlint": "^1.73.0",
+    "oxlint-tsgolint": "^0.24.0",
     "tsdown": "^0.22.0",
     "typescript": "^6.0.0",
     "typescript-eslint": "^8.35.0",
@@ -94,5 +94,5 @@
   "lint-staged": {
     "*": "oxfmt --no-error-on-unmatched-pattern"
   },
-  "packageManager": "pnpm@11.8.0"
+  "packageManager": "pnpm@11.9.0"
 }