@ethang/eslint-config 19.0.7 → 19.0.8

package/package.json

@@ -1,12 +1,12 @@
 {
   "name": "@ethang/eslint-config",
-  "version": "19.0.7",
+  "version": "19.0.8",
   "repository": {
     "url": "git+https://github.com/eglove/eslint-config-ethang.git"
   },
   "main": "eslint.config.js",
   "scripts": {
-    "lint": "eslint . --fix --ignore-pattern=\".eslint-config-inspector\"",
+    "lint": "prettier . -w",
     "build": "pnpx @eslint/config-inspector build"
   },
   "engines": {
@@ -17,22 +17,22 @@
   "license": "ISC",
   "peerDependencies": {
     "@eslint-react/eslint-plugin": "^1.12.1",
-    "@eslint/js": "^9.9.0",
+    "@eslint/js": "^9.9.1",
     "@stylistic/eslint-plugin": "^2.6.4",
     "@tanstack/eslint-plugin-query": "^5.52.0",
     "@typescript-eslint/parser": "^8.2.0",
-    "eslint": "^9.9.0",
+    "eslint": "^9.9.1",
     "eslint-plugin-astro": "^1.2.3",
     "eslint-plugin-barrel-files": "^2.1.0",
     "eslint-plugin-compat": "^6.0.0",
-    "eslint-plugin-depend": "^0.10.0",
+    "eslint-plugin-depend": "^0.11.0",
     "eslint-plugin-jsx-a11y": "^6.9.0",
     "eslint-plugin-lodash": "^8.0.0",
     "eslint-plugin-n": "^17.10.2",
     "eslint-plugin-perfectionist": "^3.2.0",
     "eslint-plugin-react-hooks": "^4.6.2",
     "eslint-plugin-solid": "^0.14.2",
-    "eslint-plugin-sonarjs": "2.0.0",
+    "eslint-plugin-sonarjs": "2.0.1",
     "eslint-plugin-tailwindcss": "^3.17.4",
     "eslint-plugin-unicorn": "^55.0.0",
     "typescript": "^5.5.4",
@@ -40,22 +40,22 @@
   },
   "dependencies": {
     "@eslint-react/eslint-plugin": "^1.12.1",
-    "@eslint/js": "^9.9.0",
+    "@eslint/js": "^9.9.1",
     "@stylistic/eslint-plugin": "^2.6.4",
     "@tanstack/eslint-plugin-query": "^5.52.0",
     "@typescript-eslint/parser": "^8.2.0",
-    "eslint": "^9.9.0",
+    "eslint": "^9.9.1",
     "eslint-plugin-astro": "^1.2.3",
     "eslint-plugin-barrel-files": "^2.1.0",
     "eslint-plugin-compat": "^6.0.0",
-    "eslint-plugin-depend": "^0.10.0",
+    "eslint-plugin-depend": "^0.11.0",
     "eslint-plugin-jsx-a11y": "^6.9.0",
     "eslint-plugin-lodash": "^8.0.0",
     "eslint-plugin-n": "^17.10.2",
     "eslint-plugin-perfectionist": "^3.2.0",
     "eslint-plugin-react-hooks": "^4.6.2",
     "eslint-plugin-solid": "^0.14.2",
-    "eslint-plugin-sonarjs": "2.0.0",
+    "eslint-plugin-sonarjs": "2.0.1",
     "eslint-plugin-tailwindcss": "^3.17.4",
     "eslint-plugin-unicorn": "^55.0.0",
     "typescript": "^5.5.4",
@@ -64,6 +64,7 @@
   "devDependencies": {
     "@ethang/project-builder": "^2.3.10",
     "@tsconfig/node-lts": "^20.1.3",
-    "@tsconfig/strictest": "^2.0.5"
+    "@tsconfig/strictest": "^2.0.5",
+    "prettier": "^3.3.3"
   }
 }

package/setup/a11y.js

@@ -0,0 +1,25 @@
+import a11y from "eslint-plugin-jsx-a11y";
+
+import { genRules } from "./gen-rules.js";
+
+const ruleNames = Object.keys(a11y.rules);
+const customRules = [
+  {
+    name: "accessible-emoji",
+    rule: "off",
+  },
+  {
+    name: "label-has-for",
+    rule: "off",
+  },
+  {
+    name: "media-has-caption",
+    rule: "off",
+  },
+  {
+    name: "no-onchange",
+    rule: "off",
+  },
+];
+
+export const a11yRules = genRules(ruleNames, customRules, "a11y");

package/setup/barrel.js

@@ -0,0 +1,9 @@
+/* eslint-disable unicorn/no-abusive-eslint-disable */
+/* eslint-disable */
+import barrel from "eslint-plugin-barrel-files";
+import { genRules } from "./gen-rules.js";
+
+const ruleNames = Object.keys(barrel.rules);
+const customRules = [];
+
+export const barrelRules = genRules(ruleNames, customRules, "barrel");

package/setup/compat.js

@@ -0,0 +1,10 @@
+/* eslint-disable unicorn/no-abusive-eslint-disable */
+/* eslint-disable */
+import compat from "eslint-plugin-compat";
+
+import { genRules } from "./gen-rules.js";
+
+const ruleNames = Object.keys(compat.rules);
+const changedRules = [];
+
+export const compatRules = genRules(ruleNames, changedRules, "compat");

package/setup/depend.js

@@ -0,0 +1,11 @@
+/* eslint-disable unicorn/no-abusive-eslint-disable */
+/* eslint-disable */
+import depend from "eslint-plugin-depend";
+import { genRules } from "./gen-rules.js";
+
+const ruleNames = Object.keys(depend.rules);
+const changedRules = [
+  { name: "ban-dependencies", rule: ["error", { allowed: ["lodash"] }] },
+];
+
+export const dependRules = genRules(ruleNames, changedRules, "depend");

package/setup/eslint.js

@@ -0,0 +1,72 @@
+/* eslint-disable */
+import eslint from "@eslint/js";
+import { genRules } from "./gen-rules.js";
+
+const ruleNames = Object.keys(eslint.configs.all.rules);
+const changedRules = [
+  { name: "arrow-body-style", rule: ["error", "always"] },
+  { name: "camelcase", rule: "off" },
+  { name: "capitalized-comments", rule: "off" },
+  { name: "class-methods-use-this", rule: "off" },
+  { name: "complexity", rule: "off" },
+  { name: "consistent-return", rule: "off" },
+  { name: "consistent-this", rule: "off" },
+  { name: "curly", rule: "off" },
+  { name: "default-case", rule: "off" },
+  { name: "default-case-last", rule: "off" },
+  { name: "default-param-last", rule: "off" },
+  { name: "dot-notation", rule: "off" },
+  { name: "func-names", rule: "off" },
+  {
+    name: "func-style",
+    rule: ["error", "declaration", { allowArrowFunctions: true }],
+  },
+  { name: "id-denylist", rule: "off" },
+  { name: "id-length", rule: "off" },
+  { name: "id-match", rule: "off" },
+  { name: "init-declarations", rule: "off" },
+  { name: "max-lines", rule: "off" },
+  { name: "max-lines-per-function", rule: "off" },
+  { name: "max-params", rule: "off" },
+  { name: "new-cap", rule: "off" },
+  { name: "no-array-constructor", rule: "off" },
+  { name: "no-duplicate-imports", rule: "off" },
+  { name: "no-empty-function", rule: "off" },
+  { name: "no-implicit-globals", rule: "off" },
+  { name: "no-implied-eval", rule: "off" },
+  { name: "no-inline-comments", rule: "off" },
+  { name: "no-magic-numbers", rule: "off" },
+  { name: "no-redeclare", rule: "off" },
+  { name: "no-restricted-exports", rule: "off" },
+  { name: "no-restricted-globals", rule: "off" },
+  { name: "no-restricted-imports", rule: "off" },
+  { name: "no-restricted-properties", rule: "off" },
+  { name: "no-restricted-syntax", rule: "off" },
+  { name: "no-shadow", rule: "off" },
+  { name: "no-template-curly-in-string", rule: "off" },
+  { name: "no-ternary", rule: "off" },
+  { name: "no-throw-literal", rule: "off" },
+  { name: "no-undef", rule: "off" },
+  { name: "no-undefined", rule: "off" },
+  { name: "no-underscore-dangle", rule: "off" },
+  { name: "no-unexpected-multiline", rule: "off" },
+  {
+    name: "no-unsafe-optional-chaining",
+    rule: ["error", { disallowArithmeticOperators: true }],
+  },
+  { name: "no-unused-expressions", rule: "off" },
+  { name: "no-unused-vars", rule: "off" },
+  { name: "no-use-before-define", rule: "off" },
+  { name: "no-useless-assignment", rule: "off" },
+  { name: "no-useless-constructor", rule: "off" },
+  { name: "no-warning-comments", rule: "off" },
+  { name: "one-var", rule: "off" },
+  { name: "prefer-destructuring", rule: "off" },
+  { name: "prefer-promise-reject-errors", rule: "off" },
+  { name: "require-await", rule: "off" },
+  { name: "sort-imports", rule: "off" },
+  { name: "sort-keys", rule: "off" },
+  { name: "yoda", rule: ["error", "always"] },
+];
+
+export const eslintRules = genRules(ruleNames, changedRules);

package/setup/gen-rules.js

@@ -0,0 +1,34 @@
+/**
+ * @param {Array<String>} ruleNames
+ * @param {Array<{name: string, rule: unknown}>} customRules
+ * @param {string?} prefix
+ * @return {Object}
+ */
+export const genRules = (ruleNames, customRules, prefix) => {
+  const rules = {};
+
+  for (const rule of ruleNames) {
+    if (prefix === undefined) {
+      rules[rule] = "error";
+    } else {
+      rules[`${prefix}/${rule}`] = "error";
+    }
+  }
+
+  if (customRules) {
+    for (const rule of customRules) {
+      if (ruleNames.includes(rule.name)) {
+        if (prefix === undefined) {
+          rules[rule.name] = rule.rule;
+        } else {
+          rules[`${prefix}/${rule.name}`] = rule.rule;
+        }
+      } else {
+        // eslint-disable-next-line no-console
+        console.error(`${rule.name} does not exist.`);
+      }
+    }
+  }
+
+  return rules;
+};

package/setup/lodash.js

@@ -0,0 +1,19 @@
+/* eslint-disable */
+import lodash from "eslint-plugin-lodash";
+import { genRules } from "./gen-rules.js";
+
+const ruleNames = Object.keys(lodash.rules);
+const customRules = [
+  { name: "chain-style", rule: ["error", "as-needed"] },
+  { name: "consistent-compose", rule: ["error", "flow"] },
+  { name: "identity-shorthand", rule: ["error", "always"] },
+  { name: "import-scope", rule: ["error", "method"] },
+  { name: "matches-prop-shorthand", rule: ["error", "always"] },
+  { name: "matches-shorthand", rule: ["error", "always", 3] },
+  { name: "path-style", rule: ["error", "array"] },
+  { name: "prefer-includes", rule: ["error", { includeNative: true }] },
+  { name: "prefer-some", rule: ["error", { includeNative: true }] },
+  { name: "prop-shorthand", rule: ["error", "always"] },
+];
+
+export const lodashRules = genRules(ruleNames, customRules, "lodash");

package/setup/n.js

@@ -0,0 +1,31 @@
+/* eslint-disable unicorn/no-abusive-eslint-disable */
+/* eslint-disable */
+import n from "eslint-plugin-n";
+import { genRules } from "./gen-rules.js";
+
+const ruleNames = Object.keys(n.rules);
+const changedRules = [
+  { name: "callback-return", rule: "off" },
+  { name: "exports-style", rule: ["error", "exports"] },
+  { name: "file-extension-in-import", rule: "off" },
+  { name: "global-require", rule: "off" },
+  { name: "no-callback-literal", rule: "off" },
+  { name: "no-missing-import", rule: "off" },
+  { name: "no-mixed-requires", rule: "off" },
+  { name: "no-new-require", rule: "off" },
+  { name: "no-process-env", rule: "off" },
+  { name: "no-process-exit", rule: "off" },
+  { name: "no-restricted-import", rule: "off" },
+  { name: "no-restricted-require", rule: "off" },
+  { name: "no-sync", rule: "off" },
+  { name: "no-unpublished-import", rule: "off" },
+  { name: "no-unpublished-require", rule: "off" },
+  { name: "prefer-promises/dns", rule: "off" },
+  { name: "prefer-promises/fs", rule: "off" },
+  { name: "process-exit-as-throw", rule: "off" },
+  { name: "hashbang", rule: "off" },
+  { name: "no-hide-core-modules", rule: "off" },
+  { name: "shebang", rule: "off" },
+];
+
+export const nRules = genRules(ruleNames, changedRules, "n");

package/setup/perfectionist.js

@@ -0,0 +1,78 @@
+import perfectionist from "eslint-plugin-perfectionist";
+
+import { genRules } from "./gen-rules.js";
+
+const ruleNames = Object.keys(perfectionist.rules);
+const customRules = [
+  {
+    name: "sort-astro-attributes",
+    rule: [
+      "error",
+      {
+        groups: ["astro-shorthand", "shorthand", "multiline"],
+      },
+    ],
+  },
+  {
+    name: "sort-imports",
+    rule: [
+      "error",
+      {
+        groups: [
+          "type",
+          ["builtin", "external"],
+          "internal-type",
+          "internal",
+          ["parent-type", "sibling-type", "index-type"],
+          ["parent", "sibling", "index"],
+          "object",
+          "unknown",
+        ],
+        newlinesBetween: "always",
+      },
+    ],
+  },
+  {
+    name: "sort-jsx-props",
+    rule: ["error", { groups: ["shorthand", "multiline"] }],
+  },
+  {
+    name: "sort-svelte-attributes",
+    rule: [
+      "error",
+      {
+        groups: ["svelte-shorthand", "shorthand", "multiline"],
+      },
+    ],
+  },
+  {
+    name: "sort-switch-case",
+    rule: [
+      "error",
+      {
+        order: "asc",
+        type: "alphabetical",
+      },
+    ],
+  },
+  {
+    name: "sort-variable-declarations",
+    rule: [
+      "error",
+      {
+        order: "asc",
+        type: "alphabetical",
+      },
+    ],
+  },
+  {
+    name: "sort-vue-attributes",
+    rule: ["error", { groups: ["shorthand", "multiline"] }],
+  },
+];
+
+export const perfectionistRules = genRules(
+  ruleNames,
+  customRules,
+  "perfectionist",
+);

package/setup/sonar.js

@@ -0,0 +1,61 @@
+/* eslint-disable */
+import sonar from "eslint-plugin-sonarjs";
+import { genRules } from "./gen-rules.js";
+
+const ruleNames = Object.keys(sonar.rules);
+const customRules = [
+  { name: "aws-s3-bucket-server-encryption", rule: "off" },
+  { name: "arrow-function-convention", rule: "off" },
+  { name: "brace-style", rule: "off" },
+  { name: "conditional-indentation", rule: "off" },
+  { name: "comment-regex", rule: "off" },
+  { name: "cookies", rule: "off" },
+  { name: "cyclomatic-complexity", rule: "off" },
+  { name: "dns-prefetching", rule: "off" },
+  { name: "encryption", rule: "off" },
+  { name: "enforce-trailing-comma", rule: "off" },
+  { name: "file-header", rule: "off" },
+  {
+    name: "function-name",
+    rule: [
+      "error",
+      {
+        format: "^(?:[a-z][a-zA-Z0-9]*|[A-Z][A-Z0-9]*)$",
+      },
+    ],
+  },
+  { name: "function-return-type", rule: "off" },
+  { name: "max-union-size", rule: "off" },
+  { name: "new-cap", rule: "off" },
+  { name: "no-extra-semi", rule: "off" },
+  { name: "no-implicit-dependencies", rule: "off" },
+  { name: "no-inconsistent-returns", rule: "off" },
+  { name: "no-redundant-parentheses", rule: "off" }, // Not working correctly, TS can handle this
+  { name: "no-reference-error", rule: "off" }, // Not working correctly, TS can handle this
+  { name: "no-tab", rule: "off" }, // Not working correctly, TS can handle this
+  { name: "no-undefined-assignment", rule: "off" },
+  { name: "process-argv", rule: "off" },
+  { name: "regular-expr", rule: "off" },
+  { name: "semi", rule: "off" },
+  { name: "shorthand-property-grouping", rule: "off" }, // Conflicts with perfectionist sorting
+  { name: "sockets", rule: "off" },
+  { name: "sonar-no-fallthrough", rule: "off" }, // TODO v9 compatibility?
+  { name: "sonar-no-magic-numbers", rule: "off" },
+  { name: "standard-input", rule: "off" },
+  { name: "switch-without-default", rule: "off" }, // @typescript-eslint/switch-exhaustiveness-check works better
+  { name: "todo-tag", rule: "off" },
+  { name: "useless-string-operation", rule: "off" },
+  { name: "web-sql-database", rule: "off" },
+  { name: "xpath", rule: "off" },
+];
+
+const rules = genRules(ruleNames, customRules, "sonar");
+
+// Turn off duplicate S# rules
+for (const key of Object.keys(rules)) {
+  if (/^sonar\/S\d+/.test(key)) {
+    rules[key] = "off";
+  }
+}
+
+export const sonarRules = rules;

package/setup/stylistic.js

@@ -0,0 +1,62 @@
+/* eslint-disable */
+import stylistic from "@stylistic/eslint-plugin";
+import { genRules } from "./gen-rules.js";
+
+const ruleNames = Object.keys(stylistic.rules);
+const customRules = [
+  { name: "array-bracket-newline", rule: ["error", "consistent"] },
+  { name: "array-element-newline", rule: ["error", "consistent"] },
+  { name: "comma-dangle", rule: ["error", "always-multiline"] },
+  { name: "dot-location", rule: ["error", "property"] },
+  { name: "function-call-argument-newline", rule: ["error", "consistent"] },
+  { name: "function-paren-newline", rule: ["error", "consistent"] },
+  { name: "indent", rule: ["error", 2] },
+  { name: "line-comment-position", rule: "off" },
+  {
+    name: "max-len",
+    rule: [
+      "error",
+      {
+        ignoreComments: true,
+        ignoreTrailingComments: true,
+        ignoreUrls: true,
+        ignoreStrings: true,
+        ignoreTemplateLiterals: true,
+        ignoreRegExpLiterals: true,
+      },
+    ],
+  },
+  { name: "multiline-comment-style", rule: "off" },
+  {
+    name: "no-extra-parens",
+    rule: ["error", "all", { ignoreJSX: "multi-line" }],
+  },
+  {
+    name: "object-curly-newline",
+    rule: ["error", { multiline: true, consistent: true }],
+  },
+  { name: "object-curly-spacing", rule: ["error", "always"] },
+  { name: "padded-blocks", rule: ["error", "never"] },
+  { name: "quote-props", rule: ["error", "as-needed"] },
+  {
+    name: "space-before-function-paren",
+    rule: [
+      "error",
+      {
+        anonymous: "always",
+        named: "never",
+        asyncArrow: "always",
+      },
+    ],
+  },
+  { name: "jsx-indent", rule: "off" },
+  { name: "jsx-indent-props", rule: ["error", 2] },
+  { name: "jsx-newline", rule: ["error", { prevent: true }] },
+  { name: "jsx-sort-props", rule: "off" },
+  {
+    name: "jsx-wrap-multilines",
+    rule: ["error", { declaration: "parens-new-line" }],
+  },
+];
+
+export const stylisticRules = genRules(ruleNames, customRules, "stylistic");

package/setup/tailwind.js

@@ -0,0 +1,8 @@
+/* eslint-disable */
+import tailwind from "eslint-plugin-tailwindcss";
+import { genRules } from "./gen-rules.js";
+
+const ruleNames = Object.keys(tailwind.rules);
+const customRules = [{ name: "no-custom-classname", rule: "off" }];
+
+export const tailwindRules = genRules(ruleNames, customRules, "tailwind");

package/setup/tanstack-query.js

@@ -0,0 +1,7 @@
+/* eslint-disable */
+import tanstack from "@tanstack/eslint-plugin-query";
+import { genRules } from "./gen-rules.js";
+
+const ruleNames = Object.keys(tanstack.rules);
+
+export const tanstackQueryRules = genRules(ruleNames, [], "@tanstack/query");

package/setup/typescript-eslint.js

@@ -0,0 +1,124 @@
+import tseslint from "typescript-eslint";
+
+import { genRules } from "./gen-rules.js";
+
+const ruleNames = Object.keys(tseslint.plugin.rules);
+const customRules = [
+  {
+    name: "adjacent-overload-signatures",
+    rule: "off",
+  },
+  {
+    name: "class-methods-use-this",
+    rule: "off",
+  },
+  {
+    name: "consistent-return",
+    rule: "off",
+  },
+  {
+    name: "consistent-type-definitions",
+    rule: ["error", "type"],
+  },
+  {
+    name: "explicit-function-return-type",
+    rule: "off",
+  },
+  {
+    name: "explicit-module-boundary-types",
+    rule: "off",
+  },
+  {
+    name: "init-declarations",
+    rule: "off",
+  },
+  {
+    name: "member-ordering",
+    rule: "off",
+  },
+  {
+    name: "naming-convention",
+    rule: "off",
+  },
+  {
+    name: "no-dupe-class-members",
+    rule: "off",
+  },
+  {
+    name: "no-empty-interface",
+    rule: "off",
+  },
+  {
+    name: "no-invalid-this",
+    rule: "off",
+  },
+  {
+    name: "no-loss-of-precision",
+    rule: "off",
+  },
+  {
+    name: "no-magic-numbers",
+    rule: "off",
+  },
+  {
+    name: "no-redeclare",
+    rule: "off",
+  },
+  {
+    name: "no-type-alias",
+    rule: "off",
+  },
+  {
+    name: "no-use-before-define",
+    rule: "off",
+  },
+  {
+    name: "no-var-requires",
+    rule: "off",
+  },
+  {
+    name: "parameter-properties",
+    rule: "off",
+  },
+  {
+    name: "prefer-readonly-parameter-types",
+    rule: "off",
+  },
+  {
+    name: "prefer-ts-expect-error",
+    rule: "off",
+  },
+  {
+    name: "sort-type-constituents",
+    rule: "off",
+  },
+  {
+    name: "typedef",
+    rule: "off",
+  },
+  {
+    name: "max-params",
+    rule: "off",
+  },
+  {
+    name: "no-unused-vars",
+    rule: [
+      "error",
+      {
+        args: "all",
+        argsIgnorePattern: "^_",
+        caughtErrors: "all",
+        caughtErrorsIgnorePattern: "^_",
+        destructuredArrayIgnorePattern: "^_",
+        ignoreRestSiblings: true,
+        varsIgnorePattern: "^_",
+      },
+    ],
+  },
+];
+
+export const typescriptRules = genRules(
+  ruleNames,
+  customRules,
+  "@typescript-eslint",
+);

package/setup/unicorn.js

@@ -0,0 +1,32 @@
+import unicorn from "eslint-plugin-unicorn";
+import { genRules } from "./gen-rules.js";
+
+const ruleNames = Object.keys(unicorn.rules);
+const customRules = [
+  { name: "empty-brace-spaces", rule: "off" },
+  { name: "explicit-length-check", rule: "off" },
+  { name: "no-keyword-prefix", rule: "off" },
+  { name: "no-nested-ternary", rule: "off" },
+  { name: "no-null", rule: "off" },
+  { name: "number-literal-case", rule: "off" },
+  { name: "template-indent", rule: "off" },
+  { name: "import-index", rule: "off" },
+  { name: "no-array-instanceof", rule: "off" },
+  { name: "no-fn-reference-in-iterator", rule: "off" },
+  { name: "no-reduce", rule: "off" },
+  { name: "no-unsafe-regex", rule: "off" },
+  { name: "prefer-dataset", rule: "off" },
+  { name: "prefer-event-key", rule: "off" },
+  { name: "prefer-exponentiation-operator", rule: "off" },
+  { name: "prefer-flat-map", rule: "off" },
+  { name: "prefer-node-append", rule: "off" },
+  { name: "prefer-node-remove", rule: "off" },
+  { name: "prefer-object-has-own", rule: "off" },
+  { name: "prefer-replace-all", rule: "off" },
+  { name: "prefer-starts-ends-with", rule: "off" },
+  { name: "prefer-text-content", rule: "off" },
+  { name: "prefer-trim-start-end", rule: "off" },
+  { name: "regex-shorthand", rule: "off" },
+];
+
+export const unicornRules = genRules(ruleNames, customRules, "unicorn");