@ethang/eslint-config 16.0.14 → 16.1.1

package/README.md

@@ -1,11 +1,15 @@
 # Opinionated, Strict, Brutal, Unforgiving
 
-* Nearly every EsLint rule error's as a default.
-* [sindresorhus/eslint-plugin-unicorn](https://github.com/sindresorhus/eslint-plugin-unicorn)
-  * 100's of opinionated rules for a consistent codebase.
-* [@typescript-eslint/strict-type-checked](https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/eslint-plugin/src/configs/strict-type-checked.ts)
-* [@typescript-eslint/stylistic-type-checked](https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/eslint-plugin/src/configs/stylistic-type-checked.ts)
-  * Strictest config options from typescript-eslint
+* More than 500 errored rules.
+* Over 100 vanilla ESLint rules.
+* Over 100 rules from [@typescript/eslint](https://github.com/typescript-eslint/typescript-eslint)
+* Over 100 rules from [sindresorhus/eslint-plugin-unicorn](https://github.com/sindresorhus/eslint-plugin-unicorn)
+* Nearly 100 rules from [eslint-plugin-react](https://github.com/jsx-eslint/eslint-plugin-react)
+* Includes [eslint-plugin-react-hooks](https://github.com/facebook/react/tree/main/packages/eslint-plugin-react-hooks)
+* Includes [jsx-a11y](https://github.com/jsx-eslint/eslint-plugin-jsx-a11y)
+* Includes [eslint-plugin-astro](https://www.npmjs.com/package/eslint-plugin-astro)
+* Includes [eslint-plugin-sonarjs](https://github.com/SonarSource/eslint-plugin-sonarjs)
+* Import sorting with [eslint-plugin-simple-import-sort](https://github.com/lydell/eslint-plugin-simple-import-sort)
 * Includes Prettier built in (do NOT use this with a separate Prettier config.)
 * And more...
 
@@ -20,19 +24,21 @@
 In **eslint.config.js**
 
 ```js
-import config from "@ethang/eslint-config/index.js";
-
-export default [
-  ...config,
-  {
-    languageOptions: {
-      parserOptions: {
-        project: true,
-        tsconfigRootDir: import.meta.dirname,
-      },
+import config from "@ethang/eslint-config/eslint.config.js";
+import tseslint from "typescript-eslint";
+
+export default tseslint.config(...config, {
+  ignores: ["dist/"], // Your ignores directories
+  languageOptions: {
+    parserOptions: {
+      project: true,
+      tsconfigRootDir: "./tsconfig.json",
     },
   },
-];
+  rules: {
+    // your custom rules here
+  },
+});
 ```
 
 **Scripts**:

package/eslint.config.js

@@ -9,6 +9,7 @@ import reactHooks from "eslint-plugin-react-hooks";
 import importSort from "eslint-plugin-simple-import-sort";
 import unicorn from "eslint-plugin-unicorn";
 import tseslint from "typescript-eslint";
+import sonar from "eslint-plugin-sonarjs";
 
 const languageOptions = {
   parser,
@@ -65,20 +66,12 @@ export default tseslint.config(
       "react/hook-use-state": "error",
       "react/iframe-missing-sandbox": "error",
       "react/jsx-boolean-value": "error",
-      "react/jsx-child-element-spacing": "error",
-      "react/jsx-closing-bracket-location": "error",
-      "react/jsx-closing-tag-location": "error",
       "react/jsx-curly-brace-presence": "error",
-      "react/jsx-curly-newline": "error",
-      "react/jsx-curly-spacing": "error",
-      "react/jsx-equals-spacing": "error",
       "react/jsx-filename-extension": "error",
       "react/jsx-fragments": "error",
       "react/jsx-handler-names": "error",
       "react/jsx-key": "error",
       "react/jsx-max-depth": "error",
-      "react/jsx-max-props-per-line": "error",
-      "react/jsx-newline": "error",
       "react/jsx-no-bind": "error",
       "react/jsx-no-comment-textnodes": "error",
       "react/jsx-no-constructed-context-values": "error",
@@ -150,6 +143,7 @@ export default tseslint.config(
       "@typescript-eslint": tseslint.plugin,
       a11y,
       n,
+      sonar,
       "simple-import-sort": importSort,
       unicorn,
     },
@@ -157,7 +151,6 @@ export default tseslint.config(
       "accessor-pairs": "error",
       "array-callback-return": "error",
       "block-scoped-var": "error",
-      complexity: "error",
       "consistent-this": "error",
       "constructor-super": "error",
       eqeqeq: "error",
@@ -306,6 +299,7 @@ export default tseslint.config(
       yoda: ["error", "always"],
       // eslint overloads
 
+      complexity: "off", // Handled by Sonar
       "default-param-last": "off", // Handled by tseslint
       "dot-notation": "off", // Handled by tseslint
       "max-params": "off", // Handled by tseslint
@@ -494,7 +488,6 @@ export default tseslint.config(
       "unicorn/no-lonely-if": "error",
       "unicorn/no-magic-array-flat-depth": "error",
       "unicorn/no-negated-condition": "error",
-      "unicorn/no-nested-ternary": "error",
       "unicorn/no-new-array": "error",
       "unicorn/no-new-buffer": "error",
       "unicorn/no-object-as-default-parameter": "error",
@@ -516,7 +509,6 @@ export default tseslint.config(
       "unicorn/no-useless-switch-case": "error",
       "unicorn/no-useless-undefined": "error",
       "unicorn/no-zero-fractions": "error",
-      "unicorn/number-literal-case": "error",
       "unicorn/numeric-separators-style": "error",
       "unicorn/prefer-add-event-listener": "error",
       "unicorn/prefer-array-find": "error",
@@ -573,10 +565,42 @@ export default tseslint.config(
       "unicorn/require-post-message-target-origin": "error",
       "unicorn/string-content": "error",
       "unicorn/switch-case-braces": "error",
-      "unicorn/template-indent": "error",
       "unicorn/text-encoding-identifier-case": "error",
       "unicorn/throw-new-error": "error",
 
+      "sonar/cognitive-complexity": "error",
+      "sonar/elseif-without-else": "error",
+      "sonar/max-switch-cases": "error",
+      "sonar/no-all-duplicated-branches": "error",
+      "sonar/no-collapsible-if": "error",
+      "sonar/no-collection-size-mischeck": "error",
+      "sonar/no-duplicate-string": "error",
+      "sonar/no-duplicated-branches": "error",
+      "sonar/no-element-overwrite": "error",
+      "sonar/no-empty-collection": "error",
+      "sonar/no-extra-arguments": "error",
+      "sonar/no-gratuitous-expressions": "error",
+      "sonar/no-identical-conditions": "error",
+      "sonar/no-identical-expressions": "error",
+      "sonar/no-identical-functions": "error",
+      "sonar/no-ignored-return": "error",
+      "sonar/no-inverted-boolean-check": "error",
+      "sonar/no-nested-switch": "error",
+      "sonar/no-nested-template-literals": "error",
+      "sonar/no-one-iteration-loop": "error",
+      "sonar/no-redundant-boolean": "error",
+      "sonar/no-redundant-jump": "error",
+      "sonar/no-same-line-conditional": "error",
+      "sonar/no-small-switch": "error",
+      "sonar/no-unused-collection": "error",
+      "sonar/no-use-of-empty-return-value": "error",
+      "sonar/no-useless-catch": "error",
+      "sonar/non-existent-operator": "error",
+      "sonar/prefer-immediate-return": "error",
+      "sonar/prefer-object-literal": "error",
+      "sonar/prefer-single-boolean-return": "error",
+      "sonar/prefer-while": "error",
+
       "simple-import-sort/exports": "error",
       "simple-import-sort/imports": "error",
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ethang/eslint-config",
-  "version": "16.0.14",
+  "version": "16.1.1",
   "repository": {
     "url": "git+https://github.com/eglove/eslint-config-ethang.git"
   },
@@ -24,6 +24,7 @@
     "eslint-plugin-react": "^7.34.1",
     "eslint-plugin-react-hooks": "^4.6.2",
     "eslint-plugin-simple-import-sort": "^12.1.0",
+    "eslint-plugin-sonarjs": "^1.0.3",
     "eslint-plugin-unicorn": "^53.0.0",
     "prettier": "^3.2.5",
     "typescript": "^5.4.5",
@@ -41,6 +42,7 @@
     "eslint-plugin-react": "^7.34.1",
     "eslint-plugin-react-hooks": "^4.6.2",
     "eslint-plugin-simple-import-sort": "^12.1.0",
+    "eslint-plugin-sonarjs": "^1.0.3",
     "eslint-plugin-unicorn": "^53.0.0",
     "prettier": "^3.2.5",
     "typescript": "^5.4.5",