@morgs32/eslint-config 2.0.1 → 2.0.2

package/.eslintrc.js

@@ -8,16 +8,20 @@ module.exports = {
     "**/dist", 
   ],
   plugins: [
+    "import",
     "unused-imports",
-    "react"
+    "react",
+    "react-hooks",
+    "jsx-a11y",
+    "@typescript-eslint"
   ],
+  
   rules: {
     "spaced-comment": ["error", "always", { "markers": ["/"] }],
     "unused-imports/no-unused-imports": "error",
     "comma-spacing": ["error", { "before": false, "after": true }],
     "react/jsx-indent": ["error", 2],
     "space-infix-ops": ["error"],
-    "react-hooks/rules-of-hooks": 0, // Checks rules of Hooks
     quotes: ["error", "double"],
     "space-before-blocks": "error",
     "keyword-spacing": "error",
@@ -117,6 +121,7 @@ module.exports = {
     "no-this-before-super": "warn",
     "no-throw-literal": "warn",
     "no-undef": "error",
+    // "no-restricted-globals": ["error"].concat(restrictedGlobals),
     "no-unreachable": "warn",
     "no-unused-expressions": [
       "error",
@@ -159,7 +164,35 @@ module.exports = {
     "react-hooks/exhaustive-deps": "warn",
     "require-yield": "warn",
     "rest-spread-spacing": ["warn", "never"],
-    
+    strict: ["warn", "never"],
+    "unicode-bom": ["warn", "never"],
+    "use-isnan": "warn",
+    "valid-typeof": "warn",
+    "no-restricted-properties": [
+      "error",
+      {
+        object: "require",
+        property: "ensure",
+        message:
+          "Please use import() instead. More info: https://facebook.github.io/create-react-app/docs/code-splitting",
+      },
+      {
+        object: "System",
+        property: "import",
+        message:
+          "Please use import() instead. More info: https://facebook.github.io/create-react-app/docs/code-splitting",
+      },
+    ],
+    "getter-return": "warn",
+
+    // https://github.com/benmosher/eslint-plugin-import/tree/master/docs/rules
+    "import/first": "error",
+    "import/no-amd": "error",
+    "import/no-anonymous-default-export": "warn",
+    "import/no-webpack-loader-syntax": "error",
+
+    // https://github.com/yannickcr/eslint-plugin-react/tree/master/docs/rules
+    "react/forbid-foreign-prop-types": ["warn", { allowInPropTypes: true }],
     "react/jsx-no-comment-textnodes": "warn",
     "react/jsx-no-duplicate-props": "warn",
     "react/jsx-no-target-blank": "warn",
@@ -180,8 +213,35 @@ module.exports = {
     "react/no-is-mounted": "warn",
     "react/no-typos": "error",
     "react/require-render-return": "error",
-    "react/style-prop-object": "warn"
+    "react/style-prop-object": "warn",
+
+    // https://github.com/evcohen/eslint-plugin-jsx-a11y/tree/master/docs/rules
+    "jsx-a11y/alt-text": "warn",
+    "jsx-a11y/anchor-has-content": "warn",
+    "jsx-a11y/anchor-is-valid": [
+      "warn",
+      {
+        aspects: ["noHref", "invalidHref"],
+      },
+    ],
+    "jsx-a11y/aria-activedescendant-has-tabindex": "warn",
+    "jsx-a11y/aria-props": "warn",
+    "jsx-a11y/aria-proptypes": "warn",
+    "jsx-a11y/aria-role": ["warn", { ignoreNonDOM: true }],
+    "jsx-a11y/aria-unsupported-elements": "warn",
+    "jsx-a11y/heading-has-content": "warn",
+    "jsx-a11y/iframe-has-title": "warn",
+    "jsx-a11y/img-redundant-alt": "warn",
+    "jsx-a11y/no-access-key": "warn",
+    "jsx-a11y/no-distracting-elements": "warn",
+    "jsx-a11y/no-redundant-roles": "warn",
+    "jsx-a11y/role-has-required-aria-props": "warn",
+    "jsx-a11y/role-supports-aria-props": "warn",
+    "jsx-a11y/scope": "warn",
 
+    // https://github.com/facebook/react/tree/main/packages/eslint-plugin-react-hooks
+    "react-hooks/rules-of-hooks": "error",
+    
   },
   overrides: [
     {
@@ -190,5 +250,66 @@ module.exports = {
         "import/no-anonymous-default-export": "off",
       },
     },
+    {
+      files: ["**/*.ts?(x)"],
+      // parser: "@typescript-eslint/parser",
+      // parserOptions: {
+      //   ecmaVersion: 2018,
+      //   sourceType: "module",
+      //   ecmaFeatures: {
+      //     jsx: true,
+      //   },
+
+      //   // typescript-eslint specific options
+      //   warnOnUnsupportedTypeScriptVersion: true,
+      // },
+      // plugins: ["@typescript-eslint"],
+      // If adding a typescript-eslint version of an existing ESLint rule,
+      // make sure to disable the ESLint rule here.
+      rules: {
+        // TypeScript's `noFallthroughCasesInSwitch` option is more robust (#6906)
+        "default-case": "off",
+        // 'tsc' already handles this (https://github.com/typescript-eslint/typescript-eslint/issues/291)
+        "no-dupe-class-members": "off",
+        // 'tsc' already handles this (https://github.com/typescript-eslint/typescript-eslint/issues/477)
+        "no-undef": "off",
+
+        // Add TypeScript specific rules (and turn off ESLint equivalents)
+        "@typescript-eslint/consistent-type-assertions": "warn",
+        "no-array-constructor": "off",
+        "@typescript-eslint/no-array-constructor": "warn",
+        "no-redeclare": "off",
+        "@typescript-eslint/no-redeclare": "warn",
+        "no-use-before-define": "off",
+        "@typescript-eslint/no-use-before-define": [
+          "warn",
+          {
+            functions: false,
+            classes: false,
+            variables: false,
+            typedefs: false,
+          },
+        ],
+        "no-unused-expressions": "off",
+        "@typescript-eslint/no-unused-expressions": [
+          "error",
+          {
+            allowShortCircuit: true,
+            allowTernary: true,
+            allowTaggedTemplates: true,
+          },
+        ],
+        "no-unused-vars": "off",
+        "@typescript-eslint/no-unused-vars": [
+          "warn",
+          {
+            args: "none",
+            ignoreRestSiblings: true,
+          },
+        ],
+        "no-useless-constructor": "off",
+        "@typescript-eslint/no-useless-constructor": "warn",
+      },
+    },
   ],
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@morgs32/eslint-config",
-  "version": "2.0.1",
+  "version": "2.0.2",
   "description": "",
   "main": ".eslintrc.js",
   "files": [
@@ -17,6 +17,7 @@
   },
   "homepage": "https://github.com/morgs32/eslint-config#readme",
   "dependencies": {
+    "@typescript-eslint/eslint-plugin": "^6.2.0",
     "eslint-config-next": "^13.4.12",
     "eslint-plugin-react": "^7.33.1",
     "eslint-plugin-unused-imports": "^3.0.0"