@vinicunca/eslint-config 3.8.0 → 3.10.0

package/dist/index.js

@@ -1314,6 +1314,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"
 ];
@@ -1330,7 +1336,7 @@ async function react(options = {}) {
   } = options;
   const isTypeAware = !!tsconfigPath;
   const typeAwareRules = {
-    "react/no-leaked-conditional-rendering": "warn"
+    "react/no-leaked-conditional-rendering": WARN
   };
   const [
     pluginReact,
@@ -1343,6 +1349,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 [
@@ -1354,7 +1361,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"]
       }
     },
     {
@@ -1369,28 +1377,86 @@ async function react(options = {}) {
       },
       name: "vinicunca/react/rules",
       rules: {
-        // recommended rules from @eslint-react/dom
-        "react-dom/no-children-in-void-dom-elements": WARN,
+        // recommended rules from eslint-plugin-react-x https://eslint-react.xyz/docs/rules/overview#core-rules
+        "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-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-jsx-props": WARN,
+        "react/no-duplicate-key": WARN,
+        "react/no-forward-ref": WARN,
+        "react/no-implicit-key": WARN,
+        "react/no-missing-key": ERROR,
+        "react/no-nested-component-definitions": 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,
+        "react/no-set-state-in-component-will-update": WARN,
+        "react/no-string-refs": ERROR,
+        "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": WARN,
+        "react/no-unstable-default-props": WARN,
+        "react/no-unused-class-component-members": WARN,
+        "react/no-unused-state": WARN,
+        "react/no-use-context": WARN,
+        "react/no-useless-forward-ref": WARN,
+        "react/use-jsx-vars": WARN,
+        // recommended rules from eslint-plugin-react-dom https://eslint-react.xyz/docs/rules/overview#dom-rule
         "react-dom/no-dangerously-set-innerhtml": WARN,
         "react-dom/no-dangerously-set-innerhtml-with-children": ERROR,
         "react-dom/no-find-dom-node": ERROR,
+        "react-dom/no-flush-sync": "error",
+        "react-dom/no-hydrate": "error",
         "react-dom/no-missing-button-type": WARN,
         "react-dom/no-missing-iframe-sandbox": WARN,
         "react-dom/no-namespace": ERROR,
+        "react-dom/no-render": "error",
         "react-dom/no-render-return-value": ERROR,
         "react-dom/no-script-url": WARN,
         "react-dom/no-unsafe-iframe-sandbox": WARN,
         "react-dom/no-unsafe-target-blank": WARN,
-        // recommended rules react-hooks
+        "react-dom/no-use-form-state": "error",
+        "react-dom/no-void-elements-with-children": "error",
+        // recommended rules eslint-plugin-react-hooks https://github.com/facebook/react/tree/main/packages/eslint-plugin-react-hooks/src/rules
         "react-hooks/exhaustive-deps": WARN,
         "react-hooks/rules-of-hooks": ERROR,
-        // react refresh
+        // recommended rules from eslint-plugin-react-hooks-extra https://eslint-react.xyz/docs/rules/overview#hooks-extra-rules
+        "react-hooks-extra/no-direct-set-state-in-use-effect": "warn",
+        "react-hooks-extra/no-unnecessary-use-prefix": "warn",
+        // recommended rules from eslint-plugin-react-web-api https://eslint-react.xyz/docs/rules/overview#web-api-rules
+        "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",
+        // preconfigured rules from eslint-plugin-react-refresh https://github.com/ArnaudBarre/eslint-plugin-react-refresh/tree/main/src
         "react-refresh/only-export-components": [
           WARN,
           {
             allowConstantExport: isAllowConstantExport,
             allowExportNames: [
               ...isUsingNext ? [
+                "dynamic",
+                "dynamicParams",
+                "revalidate",
+                "fetchCache",
+                "runtime",
+                "preferredRegion",
+                "maxDuration",
                 "config",
                 "generateStaticParams",
                 "metadata",
@@ -1398,7 +1464,7 @@ async function react(options = {}) {
                 "viewport",
                 "generateViewport"
               ] : [],
-              ...isUsingRemix ? [
+              ...isUsingRemix || isUsingReactRouter ? [
                 "meta",
                 "links",
                 "headers",
@@ -1408,43 +1474,6 @@ async function react(options = {}) {
             ]
           }
         ],
-        // recommended rules from @eslint-react
-        "react/ensure-forward-ref-using-ref": 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-create-ref": ERROR,
-        "react/no-direct-mutation-state": ERROR,
-        "react/no-duplicate-key": ERROR,
-        "react/no-implicit-key": ERROR,
-        "react/no-missing-key": ERROR,
-        "react/no-nested-components": WARN,
-        "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,
-        "react/no-set-state-in-component-will-update": WARN,
-        "react/no-string-refs": ERROR,
-        "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-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,
         // overrides
         ...overrides
       }
@@ -1727,6 +1756,7 @@ function sortTsconfig() {
               "useDefineForClassFields",
               "emitDecoratorMetadata",
               "experimentalDecorators",
+              "libReplacement",
               /* Modules */
               "baseUrl",
               "rootDir",
@@ -1800,6 +1830,7 @@ function sortTsconfig() {
               "isolatedModules",
               "preserveSymlinks",
               "verbatimModuleSyntax",
+              "erasableSyntaxOnly",
               /* Completeness */
               "skipDefaultLibCheck",
               "skipLibCheck"
@@ -2269,26 +2300,35 @@ async function vue(options = {}) {
   const {
     files = [GLOB_VUE],
     overrides = {},
-    stylistic: stylistic2 = true
+    stylistic: stylistic2 = true,
+    a11y = false
   } = options;
   const sfcBlocks = options.sfcBlocks === true ? {} : options.sfcBlocks ?? {};
   const {
     indent = 2
   } = e2(stylistic2) ? {} : stylistic2;
+  if (a11y) {
+    await ensurePackages([
+      "eslint-plugin-vuejs-accessibility"
+    ]);
+  }
   const [
     pluginVue,
     parserVue,
-    processorVueBlocks
+    processorVueBlocks,
+    pluginVueA11y
   ] = await Promise.all([
     interopDefault(import("eslint-plugin-vue")),
     interopDefault(import("vue-eslint-parser")),
-    interopDefault(import("eslint-processor-vue-blocks"))
+    interopDefault(import("eslint-processor-vue-blocks")),
+    ...a11y ? [interopDefault(import("eslint-plugin-vuejs-accessibility"))] : []
   ]);
   return [
     {
       name: "vinicunca/vue/setup",
       plugins: {
-        vue: pluginVue
+        vue: pluginVue,
+        ...a11y ? { "vue-a11y": pluginVueA11y } : {}
       },
       // This allows Vue plugin to work with auto imports
       // https://github.com/vuejs/eslint-plugin-vue/pull/2422
@@ -2428,6 +2468,30 @@ async function vue(options = {}) {
           "vue/space-in-parens": [ERROR, NEVER],
           "vue/template-curly-spacing": ERROR
         } : {},
+        ...a11y ? {
+          "vue-a11y/alt-text": ERROR,
+          "vue-a11y/anchor-has-content": ERROR,
+          "vue-a11y/aria-props": ERROR,
+          "vue-a11y/aria-role": ERROR,
+          "vue-a11y/aria-unsupported-elements": ERROR,
+          "vue-a11y/click-events-have-key-events": ERROR,
+          "vue-a11y/form-control-has-label": ERROR,
+          "vue-a11y/heading-has-content": ERROR,
+          "vue-a11y/iframe-has-title": ERROR,
+          "vue-a11y/interactive-supports-focus": ERROR,
+          "vue-a11y/label-has-for": ERROR,
+          "vue-a11y/media-has-caption": WARN,
+          "vue-a11y/mouse-events-have-key-events": ERROR,
+          "vue-a11y/no-access-key": ERROR,
+          "vue-a11y/no-aria-hidden-on-focusable": ERROR,
+          "vue-a11y/no-autofocus": WARN,
+          "vue-a11y/no-distracting-elements": ERROR,
+          "vue-a11y/no-redundant-roles": ERROR,
+          "vue-a11y/no-role-presentation-on-focusable": ERROR,
+          "vue-a11y/no-static-element-interactions": ERROR,
+          "vue-a11y/role-has-required-aria-props": ERROR,
+          "vue-a11y/tabindex-no-positive": WARN
+        } : {},
         ...overrides
       }
     }

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@vinicunca/eslint-config",
   "type": "module",
-  "version": "3.8.0",
+  "version": "3.10.0",
   "description": "Vinicunca ESLint config",
   "author": {
     "name": "praburangki",
@@ -58,10 +58,11 @@
     "eslint": "^9.10.0",
     "eslint-plugin-astro": "^1.2.0",
     "eslint-plugin-format": ">=0.1.0",
-    "eslint-plugin-react-hooks": "^5.0.0",
-    "eslint-plugin-react-refresh": "^0.4.4",
+    "eslint-plugin-react-hooks": "^5.2.0",
+    "eslint-plugin-react-refresh": "^0.4.19",
     "eslint-plugin-solid": "^0.14.3",
     "eslint-plugin-svelte": ">=2.35.1",
+    "eslint-plugin-vuejs-accessibility": "^2.4.1",
     "prettier-plugin-astro": "^0.14.0",
     "prettier-plugin-slidev": "^1.0.5",
     "svelte-eslint-parser": ">=0.37.0"
@@ -94,6 +95,9 @@
     "eslint-plugin-svelte": {
       "optional": true
     },
+    "eslint-plugin-vuejs-accessibility": {
+      "optional": true
+    },
     "prettier-plugin-astro": {
       "optional": true
     },
@@ -110,19 +114,19 @@
     "@eslint-community/eslint-plugin-eslint-comments": "^4.4.1",
     "@eslint/markdown": "^6.3.0",
     "@stylistic/eslint-plugin": "^4.2.0",
-    "@typescript-eslint/eslint-plugin": "^8.26.1",
-    "@typescript-eslint/parser": "^8.26.1",
-    "@unocss/eslint-plugin": "^66.0.0",
-    "@vitest/eslint-plugin": "^1.1.37",
+    "@typescript-eslint/eslint-plugin": "^8.28.0",
+    "@typescript-eslint/parser": "^8.28.0",
+    "@unocss/eslint-plugin": "^66.1.0-beta.7",
+    "@vitest/eslint-plugin": "^1.1.38",
     "eslint-config-flat-gitignore": "^2.1.0",
     "eslint-flat-config-utils": "^2.0.1",
     "eslint-merge-processors": "^2.0.0",
     "eslint-plugin-antfu": "^3.1.1",
-    "eslint-plugin-command": "^3.1.0",
+    "eslint-plugin-command": "^3.2.0",
     "eslint-plugin-format": "^1.0.1",
-    "eslint-plugin-import-x": "^4.6.1",
-    "eslint-plugin-jsdoc": "^50.6.6",
-    "eslint-plugin-jsonc": "^2.19.1",
+    "eslint-plugin-import-x": "^4.9.2",
+    "eslint-plugin-jsdoc": "^50.6.9",
+    "eslint-plugin-jsonc": "^2.20.0",
     "eslint-plugin-n": "^17.16.2",
     "eslint-plugin-no-only-tests": "^3.3.0",
     "eslint-plugin-perfectionist": "^4.10.1",
@@ -130,7 +134,7 @@
     "eslint-plugin-regexp": "^2.7.0",
     "eslint-plugin-sonarjs": "^3.0.2",
     "eslint-plugin-toml": "^0.12.0",
-    "eslint-plugin-unicorn": "^57.0.0",
+    "eslint-plugin-unicorn": "^58.0.0",
     "eslint-plugin-unused-imports": "^4.1.4",
     "eslint-plugin-vue": "^10.0.0",
     "eslint-plugin-yml": "^1.17.0",