release: v1.12.0 (#481)

Automated changes by
[create-pull-request](https://github.com/peter-evans/create-pull-request)
GitHub action

---------

Co-authored-by: Boshen <[email protected]>
Co-authored-by: Sysix <[email protected]>

Author: 3 people

package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-oxlint",
-  "version": "1.11.2",
+  "version": "1.12.0",
   "description": "Turn off all rules already supported by oxlint",
   "type": "module",
   "types": "./dist/index.d.ts",
@@ -69,7 +69,7 @@
     "jiti": "^2.4.2",
     "lint-staged": "^16.0.0",
     "memfs": "^4.14.0",
-    "oxlint": "^1.11.2",
+    "oxlint": "^1.12.0",
     "prettier": "^3.3.3",
     "scule": "^1.3.0",
     "shelljs": "^0.10.0",

pnpm-lock.yaml

@@ -3,3 +3,69 @@ export const ignoreScope = new Set(['oxc', 'deepscan', 'security']);
 
 // these are the rules that are not fully implemented in oxc
 export const ignoreCategories = new Set(['nursery']);
+
+// we are ignoring typescript type-aware rules for now, until it is stable.
+// When support it with a flag, do the same for `ignoreCategories`.
+// List copied from:
+// https://github.com/typescript-eslint/typescript-eslint/blob/7319bad3a5022be2adfbcb331451cfd85d1d786a/packages/eslint-plugin/src/configs/flat/disable-type-checked.ts
+export const typescriptTypeAwareRules = [
+  'await-thenable',
+  'consistent-return',
+  'consistent-type-exports',
+  'dot-notation',
+  'naming-convention',
+  'no-array-delete',
+  'no-base-to-string',
+  'no-confusing-void-expression',
+  'no-deprecated',
+  'no-duplicate-type-constituents',
+  'no-floating-promises',
+  'no-for-in-array',
+  'no-implied-eval',
+  'no-meaningless-void-operator',
+  'no-misused-promises',
+  'no-misused-spread',
+  'no-mixed-enums',
+  'no-redundant-type-constituents',
+  'no-unnecessary-boolean-literal-compare',
+  'no-unnecessary-condition',
+  'no-unnecessary-qualifier',
+  'no-unnecessary-template-expression',
+  'no-unnecessary-type-arguments',
+  'no-unnecessary-type-assertion',
+  'no-unnecessary-type-conversion',
+  'no-unnecessary-type-parameters',
+  'no-unsafe-argument',
+  'no-unsafe-assignment',
+  'no-unsafe-call',
+  'no-unsafe-enum-comparison',
+  'no-unsafe-member-access',
+  'no-unsafe-return',
+  'no-unsafe-type-assertion',
+  'no-unsafe-unary-minus',
+  'non-nullable-type-assertion-style',
+  'only-throw-error',
+  'prefer-destructuring',
+  'prefer-find',
+  'prefer-includes',
+  'prefer-nullish-coalescing',
+  'prefer-optional-chain',
+  'prefer-promise-reject-errors',
+  'prefer-readonly',
+  'prefer-readonly-parameter-types',
+  'prefer-reduce-type-parameter',
+  'prefer-regexp-exec',
+  'prefer-return-this-type',
+  'prefer-string-starts-ends-with',
+  'promise-function-async',
+  'related-getter-setter-pairs',
+  'require-array-sort-compare',
+  'require-await',
+  'restrict-plus-operands',
+  'restrict-template-expressions',
+  'return-await',
+  'strict-boolean-expressions',
+  'switch-exhaustiveness-check',
+  'unbound-method',
+  'use-unknown-in-catch-callback-variable',
+];

scripts/constants.ts

@@ -1,5 +1,9 @@
 import { execSync } from 'node:child_process';
-import { ignoreCategories, ignoreScope } from './constants.js';
+import {
+  ignoreCategories,
+  ignoreScope,
+  typescriptTypeAwareRules,
+} from './constants.js';
 import {
   aliasPluginNames,
   reactHookRulesInsideReactScope,
@@ -94,7 +98,13 @@ export function traverseRules(): Rule[] {
   // get all rules and filter the ignored one
   const rules = readRulesFromCommand().filter(
     (rule) =>
-      !ignoreCategories.has(rule.category) && !ignoreScope.has(rule.scope)
+      !ignoreCategories.has(rule.category) &&
+      !ignoreScope.has(rule.scope) &&
+      // ignore type-aware rules
+      !(
+        rule.scope === 'typescript' &&
+        typescriptTypeAwareRules.includes(rule.value)
+      )
   );
 
   const aliasRules: Rule[] = [];

scripts/traverse-rules.ts

@@ -140,9 +140,6 @@ exports[`contains all the oxlint rules 1`] = `
   "@typescript-eslint/no-extraneous-class": [
     0,
   ],
-  "@typescript-eslint/no-floating-promises": [
-    0,
-  ],
   "@typescript-eslint/no-import-type-side-effects": [
     0,
   ],
@@ -158,9 +155,6 @@ exports[`contains all the oxlint rules 1`] = `
   "@typescript-eslint/no-misused-new": [
     0,
   ],
-  "@typescript-eslint/no-misused-promises": [
-    0,
-  ],
   "@typescript-eslint/no-namespace": [
     0,
   ],
@@ -1157,6 +1151,9 @@ exports[`contains all the oxlint rules 1`] = `
   "prefer-spread": [
     0,
   ],
+  "prefer-template": [
+    0,
+  ],
   "promise/avoid-new": [
     0,
   ],
@@ -1242,6 +1239,9 @@ exports[`contains all the oxlint rules 1`] = `
   "react/jsx-filename-extension": [
     0,
   ],
+  "react/jsx-fragments": [
+    0,
+  ],
   "react/jsx-key": [
     0,
   ],

src/__snapshots__/configs.spec.ts.snap

@@ -11,6 +11,7 @@ import {
   unicornRulesExtendEslintRules,
   viteTestCompatibleRules,
 } from './constants.js';
+import { typescriptTypeAwareRules } from '../scripts/constants.js';
 
 describe('buildFromOxlintConfigFile', () => {
   it('successfully parse oxlint json config', () => {
@@ -157,6 +158,10 @@ describe('integration test with oxlint', () => {
           config.plugins === undefined ||
           config.plugins.includes('typescript')
         ) {
+          expectedCount += typescriptTypeAwareRules.filter(
+            (tsRule) => `@typescript-eslint/${tsRule}` in buildConfig.rules!
+          ).length;
+
           expectedCount += typescriptRulesExtendEslintRules.filter(
             (aliasRule) => aliasRule in buildConfig.rules!
           ).length;

src/build-from-oxlint-config.spec.ts

@@ -51,7 +51,7 @@ const getEsLintRuleName = (rule: string): string | undefined => {
   const expectedRule =
     esPluginName === '' ? ruleName : `${esPluginName}/${ruleName}`;
 
-  return allRules.find((rule) => rule == expectedRule);
+  return allRules.find((rule) => rule === expectedRule);
 };
 
 /**

src/build-from-oxlint-config/rules.ts

@@ -118,6 +118,7 @@ const styleRules: Record<string, 'off'> = {
   'no-template-curly-in-string': 'off',
   'no-ternary': 'off',
   'operator-assignment': 'off',
+  'prefer-template': 'off',
   'prefer-destructuring': 'off',
   'prefer-promise-reject-errors': 'off',
   'prefer-exponentiation-operator': 'off',
@@ -185,6 +186,7 @@ const styleRules: Record<string, 'off'> = {
   'promise/prefer-catch': 'off',
   'promise/prefer-await-to-callbacks': 'off',
   'promise/prefer-await-to-then': 'off',
+  'react/jsx-fragments': 'off',
   'react/jsx-boolean-value': 'off',
   'react/jsx-curly-brace-presence': 'off',
   'react/no-set-state': 'off',
@@ -292,8 +294,6 @@ const suspiciousRules: Record<string, 'off'> = {
   'react/no-namespace': 'off',
   'react/react-in-jsx-scope': 'off',
   'react/style-prop-object': 'off',
-  '@typescript-eslint/no-misused-promises': 'off',
-  '@typescript-eslint/no-floating-promises': 'off',
   '@typescript-eslint/no-confusing-non-null-assertion': 'off',
   '@typescript-eslint/no-extraneous-class': 'off',
   '@typescript-eslint/no-unnecessary-type-constraint': 'off',

src/generated/rules-by-category.ts

@@ -126,6 +126,7 @@ const eslintRules: Record<string, 'off'> = {
   'no-void': 'off',
   'no-with': 'off',
   'operator-assignment': 'off',
+  'prefer-template': 'off',
   'prefer-destructuring': 'off',
   'prefer-promise-reject-errors': 'off',
   'prefer-exponentiation-operator': 'off',
@@ -334,6 +335,7 @@ const reactRules: Record<string, 'off'> = {
   'react/forbid-elements': 'off',
   'react/forward-ref-uses-ref': 'off',
   'react/iframe-missing-sandbox': 'off',
+  'react/jsx-fragments': 'off',
   'react/jsx-filename-extension': 'off',
   'react/jsx-boolean-value': 'off',
   'react/jsx-curly-brace-presence': 'off',
@@ -389,8 +391,6 @@ const typescriptRules: Record<string, 'off'> = {
   '@typescript-eslint/consistent-type-imports': 'off',
   '@typescript-eslint/explicit-module-boundary-types': 'off',
   '@typescript-eslint/explicit-function-return-type': 'off',
-  '@typescript-eslint/no-misused-promises': 'off',
-  '@typescript-eslint/no-floating-promises': 'off',
   '@typescript-eslint/no-inferrable-types': 'off',
   '@typescript-eslint/no-confusing-non-null-assertion': 'off',
   '@typescript-eslint/no-duplicate-enum-values': 'off',