@kirklin/eslint-config 2.8.0 → 3.0.0

package/dist/index.js

@@ -386,7 +386,7 @@ async function stylistic(options = {}) {
         "style/brace-style": ["error", "1tbs", { allowSingleLine: false }],
         "style/member-delimiter-style": ["error", { multiline: { delimiter: "semi" } }],
         ...lessOpinionated ? {
-          curly: ["error", "multi-or-nest", "consistent"]
+          curly: ["error", "all"]
         } : {
           "curly": ["error", "all"],
           "kirklin/if-newline": "error",
@@ -705,6 +705,7 @@ async function imports(options = {}) {
         kirklin: default3
       },
       rules: {
+        "import/consistent-type-specifier-style": ["error", "prefer-top-level"],
         "import/first": "error",
         "import/no-duplicates": "error",
         "import/no-mutable-exports": "error",
@@ -893,7 +894,7 @@ async function javascript(options = {}) {
           }
         ],
         "prefer-const": [
-          "error",
+          isInEditor ? "warn" : "error",
           {
             destructuring: "all",
             ignoreReadBeforeAssign: true
@@ -907,7 +908,7 @@ async function javascript(options = {}) {
         "prefer-template": "error",
         "symbol-description": "error",
         "unicode-bom": ["error", "never"],
-        "unused-imports/no-unused-imports": isInEditor ? "off" : "error",
+        "unused-imports/no-unused-imports": isInEditor ? "warn" : "error",
         "unused-imports/no-unused-vars": [
           "error",
           {
@@ -1202,6 +1203,12 @@ var RemixPackages = [
   "@remix-run/serve",
   "@remix-run/dev"
 ];
+var ReactRouterPackages = [
+  "@react-router/node",
+  "@react-router/react",
+  "@react-router/serve",
+  "@react-router/dev"
+];
 var NextJsPackages = [
   "next"
 ];
@@ -1236,6 +1243,7 @@ async function react(options = {}) {
   ]);
   const isAllowConstantExport = ReactRefreshAllowConstantExportPackages.some((i) => isPackageExists3(i));
   const isUsingRemix = RemixPackages.some((i) => isPackageExists3(i));
+  const isUsingReactRouter = ReactRouterPackages.some((i) => isPackageExists3(i));
   const isUsingNext = NextJsPackages.some((i) => isPackageExists3(i));
   const plugins = pluginReact.configs.all.plugins;
   return [
@@ -1247,7 +1255,8 @@ async function react(options = {}) {
         "react-hooks": pluginReactHooks,
         "react-hooks-extra": plugins["@eslint-react/hooks-extra"],
         "react-naming-convention": plugins["@eslint-react/naming-convention"],
-        "react-refresh": pluginReactRefresh
+        "react-refresh": pluginReactRefresh,
+        "react-web-api": plugins["@eslint-react/web-api"]
       }
     },
     {
@@ -1298,7 +1307,7 @@ async function react(options = {}) {
                 "viewport",
                 "generateViewport"
               ] : [],
-              ...isUsingRemix ? [
+              ...isUsingRemix || isUsingReactRouter ? [
                 "meta",
                 "links",
                 "headers",
@@ -1308,27 +1317,37 @@ async function react(options = {}) {
             ]
           }
         ],
+        // recommended rules from @eslint-react/web-api
+        "react-web-api/no-leaked-event-listener": "warn",
+        "react-web-api/no-leaked-interval": "warn",
+        "react-web-api/no-leaked-resize-observer": "warn",
+        "react-web-api/no-leaked-timeout": "warn",
         // recommended rules from @eslint-react
         "react/ensure-forward-ref-using-ref": "warn",
+        "react/jsx-no-duplicate-props": "warn",
+        "react/jsx-uses-vars": "warn",
         "react/no-access-state-in-setstate": "error",
         "react/no-array-index-key": "warn",
         "react/no-children-count": "warn",
         "react/no-children-for-each": "warn",
         "react/no-children-map": "warn",
         "react/no-children-only": "warn",
-        "react/no-children-prop": "warn",
         "react/no-children-to-array": "warn",
         "react/no-clone-element": "warn",
         "react/no-comment-textnodes": "warn",
         "react/no-component-will-mount": "error",
         "react/no-component-will-receive-props": "error",
         "react/no-component-will-update": "error",
+        "react/no-context-provider": "warn",
         "react/no-create-ref": "error",
+        "react/no-default-props": "error",
         "react/no-direct-mutation-state": "error",
         "react/no-duplicate-key": "error",
-        "react/no-implicit-key": "error",
+        "react/no-forward-ref": "warn",
+        "react/no-implicit-key": "warn",
         "react/no-missing-key": "error",
-        "react/no-nested-components": "warn",
+        "react/no-nested-components": "error",
+        "react/no-prop-types": "error",
         "react/no-redundant-should-component-update": "error",
         "react/no-set-state-in-component-did-mount": "warn",
         "react/no-set-state-in-component-did-update": "warn",
@@ -1337,11 +1356,10 @@ async function react(options = {}) {
         "react/no-unsafe-component-will-mount": "warn",
         "react/no-unsafe-component-will-receive-props": "warn",
         "react/no-unsafe-component-will-update": "warn",
-        "react/no-unstable-context-value": "error",
-        "react/no-unstable-default-props": "error",
+        "react/no-unstable-context-value": "warn",
+        "react/no-unstable-default-props": "warn",
         "react/no-unused-class-component-members": "warn",
         "react/no-unused-state": "warn",
-        "react/no-useless-fragment": "warn",
         "react/prefer-destructuring-assignment": "warn",
         "react/prefer-shorthand-boolean": "warn",
         "react/prefer-shorthand-fragment": "warn",
@@ -1829,7 +1847,7 @@ async function test(options = {}) {
         "test/consistent-test-it": ["error", { fn: "it", withinDescribe: "it" }],
         "test/no-identical-title": "error",
         "test/no-import-node-test": "error",
-        "test/no-only-tests": isInEditor ? "off" : "error",
+        "test/no-only-tests": isInEditor ? "warn" : "error",
         "test/prefer-hooks-in-order": "error",
         "test/prefer-lowercase-title": "error",
         // Disables
@@ -2014,6 +2032,7 @@ async function typescript(options = {}) {
         "ts/consistent-type-definitions": ["error", "interface"],
         "ts/consistent-type-imports": ["error", {
           disallowTypeAnnotations: false,
+          fixStyle: "separate-type-imports",
           prefer: "type-imports"
         }],
         "ts/method-signature-style": ["error", "property"],
@@ -2405,7 +2424,7 @@ function kirklin(options = {}, ...userConfigs) {
   if (isInEditor == null) {
     isInEditor = isInEditorEnv();
     if (isInEditor) {
-      console.info("[@kirklin/eslint-config] Detected running in editor, some rules are disabled.");
+      console.log("[@kirklin/eslint-config] Detected running in editor, some rules are disabled.");
     }
   }
   const stylisticOptions = options.stylistic === false ? false : typeof options.stylistic === "object" ? options.stylistic : {};
@@ -2489,6 +2508,7 @@ function kirklin(options = {}, ...userConfigs) {
   }
   if (enableReact) {
     configs2.push(react({
+      ...typescriptOptions,
       overrides: getOverrides(options, "react"),
       tsconfigPath
     }));
@@ -2580,6 +2600,15 @@ function kirklin(options = {}, ...userConfigs) {
   if (autoRenamePlugins) {
     composer = composer.renamePlugins(defaultPluginRenaming);
   }
+  if (isInEditor) {
+    composer = composer.disableRulesFix([
+      "unused-imports/no-unused-imports",
+      "test/no-only-tests",
+      "prefer-const"
+    ], {
+      builtinRules: () => import(["eslint", "use-at-your-own-risk"].join("/")).then((r) => r.builtinRules)
+    });
+  }
   return composer;
 }
 function resolveSubOptions(options, key) {
@@ -2594,7 +2623,7 @@ function getOverrides(options, key) {
 }
 
 // src/index.ts
-var src_default = kirklin;
+var index_default = kirklin;
 export {
   GLOB_ALL_SRC,
   GLOB_ASTRO,
@@ -2631,7 +2660,7 @@ export {
   combine,
   command,
   comments,
-  src_default as default,
+  index_default as default,
   defaultPluginRenaming,
   disables,
   ensurePackages,

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@kirklin/eslint-config",
   "type": "module",
-  "version": "2.8.0",
+  "version": "3.0.0",
   "description": "Kirk Lin's ESLint config",
   "author": "Kirk Lin (https://github.com/kirklin/)",
   "license": "MIT",
@@ -11,10 +11,7 @@
     "eslint-config"
   ],
   "exports": {
-    ".": {
-      "import": "./dist/index.js",
-      "require": "./dist/index.cjs"
-    }
+    ".": "./dist/index.js"
   },
   "main": "./dist/index.js",
   "types": "./dist/index.d.ts",
@@ -24,7 +21,7 @@
     "dist"
   ],
   "peerDependencies": {
-    "@eslint-react/eslint-plugin": "^1.5.8",
+    "@eslint-react/eslint-plugin": "^1.19.0",
     "@prettier/plugin-xml": "^3.4.1",
     "@unocss/eslint-plugin": ">=0.50.0",
     "astro-eslint-parser": "^1.0.2",
@@ -81,35 +78,35 @@
     }
   },
   "dependencies": {
-    "@antfu/install-pkg": "^0.5.0",
-    "@clack/prompts": "^0.8.2",
+    "@antfu/install-pkg": "^1.0.0",
+    "@clack/prompts": "^0.10.0",
     "@eslint-community/eslint-plugin-eslint-comments": "^4.4.1",
-    "@eslint/markdown": "^6.2.1",
-    "@stylistic/eslint-plugin": "^2.11.0",
-    "@typescript-eslint/eslint-plugin": "^8.16.0",
-    "@typescript-eslint/parser": "^8.16.0",
-    "@vitest/eslint-plugin": "^1.1.12",
-    "eslint-config-flat-gitignore": "^0.3.0",
-    "eslint-flat-config-utils": "^0.4.0",
-    "eslint-merge-processors": "^0.1.0",
-    "eslint-plugin-command": "^0.2.6",
-    "eslint-plugin-import-x": "^4.4.3",
-    "eslint-plugin-jsdoc": "^50.6.0",
-    "eslint-plugin-jsonc": "^2.18.2",
-    "eslint-plugin-kirklin": "^1.5.0",
-    "eslint-plugin-n": "^17.14.0",
+    "@eslint/markdown": "^6.2.2",
+    "@stylistic/eslint-plugin": "^3.1.0",
+    "@typescript-eslint/eslint-plugin": "^8.23.0",
+    "@typescript-eslint/parser": "^8.23.0",
+    "@vitest/eslint-plugin": "^1.1.26",
+    "eslint-config-flat-gitignore": "^2.0.0",
+    "eslint-flat-config-utils": "^2.0.1",
+    "eslint-merge-processors": "^2.0.0",
+    "eslint-plugin-command": "^3.0.0",
+    "eslint-plugin-import-x": "^4.6.1",
+    "eslint-plugin-jsdoc": "^50.6.3",
+    "eslint-plugin-jsonc": "^2.19.1",
+    "eslint-plugin-kirklin": "^2.0.0",
+    "eslint-plugin-n": "^17.15.1",
     "eslint-plugin-no-only-tests": "^3.3.0",
-    "eslint-plugin-perfectionist": "^4.1.2",
+    "eslint-plugin-perfectionist": "^4.8.0",
     "eslint-plugin-regexp": "^2.7.0",
-    "eslint-plugin-toml": "^0.11.1",
+    "eslint-plugin-toml": "^0.12.0",
     "eslint-plugin-unicorn": "^56.0.1",
     "eslint-plugin-unused-imports": "^4.1.4",
-    "eslint-plugin-vue": "^9.31.0",
-    "eslint-plugin-yml": "^1.15.0",
-    "eslint-processor-vue-blocks": "^0.1.2",
-    "globals": "^15.12.0",
+    "eslint-plugin-vue": "^9.32.0",
+    "eslint-plugin-yml": "^1.16.0",
+    "eslint-processor-vue-blocks": "^2.0.0",
+    "globals": "^15.14.0",
     "jsonc-eslint-parser": "^2.4.0",
-    "local-pkg": "^0.5.1",
+    "local-pkg": "^1.0.0",
     "parse-gitignore": "^2.0.0",
     "picocolors": "^1.1.1",
     "toml-eslint-parser": "^0.10.0",
@@ -118,66 +115,65 @@
     "yargs": "^17.7.2"
   },
   "devDependencies": {
-    "@antfu/ni": "^0.23.1",
-    "@eslint-react/eslint-plugin": "^1.17.1",
-    "@eslint/config-inspector": "^0.5.6",
+    "@antfu/ni": "^23.3.1",
+    "@eslint-react/eslint-plugin": "^1.26.2",
+    "@eslint/config-inspector": "^1.0.0",
     "@prettier/plugin-xml": "^3.4.1",
-    "@stylistic/eslint-plugin-migrate": "^2.11.0",
+    "@stylistic/eslint-plugin-migrate": "^3.1.0",
     "@types/fs-extra": "^11.0.4",
-    "@types/node": "^22.10.1",
+    "@types/node": "^22.13.1",
     "@types/prompts": "^2.4.9",
     "@types/yargs": "^17.0.33",
-    "@unocss/eslint-plugin": "^0.64.1",
-    "astro-eslint-parser": "^1.1.0",
-    "bumpp": "^9.8.1",
-    "eslint": "^9.15.0",
+    "@unocss/eslint-plugin": "^65.4.3",
+    "astro-eslint-parser": "^1.2.1",
+    "bumpp": "^10.0.2",
+    "eslint": "^9.20.0",
     "eslint-plugin-astro": "^1.3.1",
-    "eslint-plugin-format": "^0.1.2",
-    "eslint-plugin-react-hooks": "^5.0.0",
-    "eslint-plugin-react-refresh": "^0.4.14",
-    "eslint-plugin-solid": "^0.14.4",
-    "eslint-plugin-svelte": "^2.46.0",
-    "eslint-typegen": "^0.3.2",
-    "esno": "^4.8.0",
-    "execa": "^9.5.1",
-    "fast-glob": "^3.3.2",
-    "fs-extra": "^11.2.0",
-    "jiti": "^2.4.0",
-    "lint-staged": "^15.2.10",
+    "eslint-plugin-format": "^1.0.1",
+    "eslint-plugin-react-hooks": "^5.1.0",
+    "eslint-plugin-react-refresh": "^0.4.18",
+    "eslint-plugin-solid": "^0.14.5",
+    "eslint-plugin-svelte": "^2.46.1",
+    "eslint-typegen": "^1.0.0",
+    "execa": "^9.5.2",
+    "fast-glob": "^3.3.3",
+    "fs-extra": "^11.3.0",
+    "jiti": "^2.4.2",
+    "lint-staged": "^15.4.3",
     "prettier-plugin-astro": "^0.14.1",
     "prettier-plugin-slidev": "^1.0.5",
     "rimraf": "^6.0.1",
     "simple-git-hooks": "^2.11.1",
-    "svelte": "^5.2.10",
+    "svelte": "^5.19.9",
     "svelte-eslint-parser": "^0.43.0",
-    "tsup": "^8.3.5",
+    "tsup": "^8.3.6",
     "tsx": "^4.19.2",
-    "typescript": "^5.7.2",
-    "vitest": "^2.1.6",
+    "typescript": "^5.7.3",
+    "vitest": "^3.0.5",
     "vue": "^3.5.13",
-    "@kirklin/eslint-config": "2.8.0"
+    "@kirklin/eslint-config": "3.0.0"
   },
   "resolutions": {
     "@eslint-community/eslint-utils": "^4.4.1",
-    "@typescript-eslint/utils": "^8.16.0",
-    "eslint": "^9.15.0",
+    "@typescript-eslint/utils": "^8.23.0",
+    "eslint": "^9.20.0",
     "tsx": "^4.19.2"
   },
   "simple-git-hooks": {
     "pre-commit": "npx lint-staged"
   },
   "lint-staged": {
-    "*": "eslint --flag unstable_ts_config --fix"
+    "*": "eslint --fix"
   },
   "scripts": {
-    "build": "nr typegen && tsup --format esm,cjs --clean --dts",
-    "stub": "tsup --format esm",
+    "build": "nr typegen && tsup --clean --dts",
+    "stub": "tsup",
     "dev": "npx @eslint/config-inspector --config eslint.config.ts",
     "build:inspector": "pnpm build && npx @eslint/config-inspector build",
-    "watch": "tsup --format esm,cjs --watch",
-    "lint": "eslint --flag unstable_ts_config .",
-    "lint:fix": "eslint --flag unstable_ts_config . --fix",
-    "typegen": "esno scripts/typegen.ts",
+    "watch": "tsup --watch",
+    "lint": "eslint .",
+    "lint:fix": "eslint . --fix",
+    "typegen": "tsx scripts/typegen.ts",
     "release": "bumpp && pnpm publish",
     "test": "vitest",
     "typecheck": "tsc --noEmit"