@lidofinance/eslint-config 0.0.2 → 0.12.0

package/.eslintrc.js

@@ -0,0 +1,14 @@
+const buildConfig = require("./lib/build-config");
+const rulesets = require("./rulesets/hard");
+
+const { plugins, rules, ...base } = buildConfig({}, rulesets);
+
+module.exports = {
+  root: true,
+  ...base,
+  plugins: [...plugins, "prettier"],
+  rules: {
+    ...rules,
+    "prettier/prettier": "error",
+  },
+};

package/README.md

@@ -1,6 +1,7 @@
 # Lido ESint config
-*Automated, non-opinionated ESLint config foundation*
-> ❗ _Please note that this ESLint config is still 0.x and is subject to significant changes; it is mainly used by internal Lido teams._
+Automated, non-opinionated ESLint config foundation
+> 
+>❗ _Please note that this ESLint config is still 0.x and is subject to significant changes; it is mainly used by internal Lido teams._
 
 
 ### Installation
@@ -19,7 +20,7 @@ If using TypeScript, alter `tsconfig.json` line `includes` to include everything
 Style rules are intentionally not provided; bring your own code style, whatever it is - prettier, airbnb, xo or standard.
 
 ```json5
-// .eslintrc
+// .eslintrc.js.js
 {
   "root": true,
   

package/all-hard.js

@@ -1,12 +1,12 @@
-const buildConfig = require('./lib/build-config')
-const rulesets = require('./rulesets/hard')
+const buildConfig = require("./lib/build-config");
+const rulesets = require("./rulesets/hard");
 
 module.exports = buildConfig(
-    {
-        typescript: true,
-        react: true,
-        next: true,
-        jest: true,
-    },
-    rulesets
-)
+  {
+    typescript: true,
+    react: true,
+    next: true,
+    jest: true,
+  },
+  rulesets
+);

package/all.js

@@ -1,12 +1,12 @@
-const buildConfig = require('./lib/build-config')
-const rulesets = require('./rulesets/easy')
+const buildConfig = require("./lib/build-config");
+const rulesets = require("./rulesets/easy");
 
 module.exports = buildConfig(
-    {
-        typescript: true,
-        react: true,
-        next: true,
-        jest: true,
-    },
-    rulesets
-)
+  {
+    typescript: true,
+    react: true,
+    next: true,
+    jest: true,
+  },
+  rulesets
+);

package/auto-hard.js

@@ -1,5 +1,5 @@
-const buildConfig = require('./lib/build-config')
-const rulesets = require('./rulesets/hard')
-const env = require('./lib/env')
+const buildConfig = require("./lib/build-config");
+const rulesets = require("./rulesets/hard");
+const env = require("./lib/env");
 
-module.exports = buildConfig(env, rulesets)
+module.exports = buildConfig(env, rulesets);

package/auto.js

@@ -1,5 +1,5 @@
-const buildConfig = require('./lib/build-config')
-const rulesets = require('./rulesets/easy')
-const env = require('./lib/env')
+const buildConfig = require("./lib/build-config");
+const rulesets = require("./rulesets/easy");
+const env = require("./lib/env");
 
-module.exports = buildConfig(env, rulesets)
+module.exports = buildConfig(env, rulesets);

package/lib/build-config.js

@@ -1,214 +1,229 @@
-const ts = require('typescript')
+const ts = require("typescript");
 
 const generalLoosenRules = {
-    'unicorn/consistent-function-scoping': 'off',
-    'unicorn/error-message': 'off',
-    'sonarjs/no-identical-conditions': 'off',
-    'sonarjs/no-identical-expressions': 'off',
-    'sonarjs/no-identical-functions': 'off',
-}
+  "unicorn/consistent-function-scoping": "off",
+  "unicorn/error-message": "off",
+  "sonarjs/no-identical-conditions": "off",
+  "sonarjs/no-identical-expressions": "off",
+  "sonarjs/no-identical-functions": "off",
+};
+
 const loosenTsRules = {
-    '@typescript-eslint/no-unsafe-argument': 'off',
-    '@typescript-eslint/no-unsafe-assignment': 'off',
-    '@typescript-eslint/no-unsafe-call': 'off',
-    '@typescript-eslint/no-unsafe-member-access': 'off',
-    '@typescript-eslint/no-unsafe-return': 'off',
-    '@typescript-eslint/require-await': 'off',
-    '@typescript-eslint/await-thenable': 'off',
-    '@typescript-eslint/no-explicit-any': 'off',
-    '@typescript-eslint/restrict-plus-operands': 'off',
-    '@typescript-eslint/restrict-template-expressions': 'off',
-}
+  "@typescript-eslint/no-unsafe-argument": "off",
+  "@typescript-eslint/no-unsafe-assignment": "off",
+  "@typescript-eslint/no-unsafe-call": "off",
+  "@typescript-eslint/no-unsafe-member-access": "off",
+  "@typescript-eslint/no-unsafe-return": "off",
+  "@typescript-eslint/require-await": "off",
+  "@typescript-eslint/await-thenable": "off",
+  "@typescript-eslint/no-explicit-any": "off",
+  "@typescript-eslint/restrict-plus-operands": "off",
+  "@typescript-eslint/restrict-template-expressions": "off",
+};
+
 const loosenReactRules = {
-    // loosen some rules
-    'react/prop-types': 'off',
-    'react/react-in-jsx-scope': 'off',
-}
+  // loosen some rules
+  "react/prop-types": "off",
+  "react/react-in-jsx-scope": "off",
+};
 
-module.exports = function buildConfig(env, { nextRules, tsRules, reactRules, defaultRules }) {
-    const config = {
-        extends: ['eslint:recommended'],
-        env: {
-            es2022: true,
-            browser: env.react,
-            node: true,
-        },
-        // future: add @brettz9, css-modules, eslint-comments plugins
-        plugins: ['sonarjs', 'unicorn', 'import', 'promise'],
-        settings: {
-            'import/resolver': {
-                [require.resolve('eslint-import-resolver-node')]: {
-                    extensions: ['.js', '.jsx'],
-                },
-            },
+module.exports = function buildConfig(
+  env,
+  { nextRules, tsRules, reactRules, defaultRules }
+) {
+  const config = {
+    extends: ["eslint:recommended"],
+    env: {
+      es2022: true,
+      browser: env.react,
+      node: true,
+    },
+    // future: add @brettz9, css-modules, eslint-comments plugins
+    plugins: ["sonarjs", "unicorn", "import", "promise"],
+    settings: {
+      "import/resolver": {
+        [require.resolve("eslint-import-resolver-node")]: {
+          extensions: [".js", ".jsx"],
         },
-        rules: defaultRules,
-        overrides: [],
-    }
+      },
+    },
+    rules: defaultRules,
+    overrides: [],
+  };
 
-    const loosenRules = {
-        ...generalLoosenRules,
-        ...(env.react ? loosenReactRules : {}),
-        ...(env.typescript ? loosenTsRules : {}),
-    }
+  const loosenRules = {
+    ...generalLoosenRules,
+    ...(env.react ? loosenReactRules : {}),
+    ...(env.typescript ? loosenTsRules : {}),
+  };
 
-    if (env.react) {
-        Object.assign(config, {
-            plugins: [...config.plugins, 'react', 'react-hooks'],
-            rules: {
-                ...config.rules,
-                ...reactRules,
-            },
-            settings: {
-                ...config.settings,
-                react: {
-                    version: 'detect',
-                },
-            },
-            overrides: [
-                ...config.overrides,
-                {
-                    files: ['*.stories.*', '*.test.*'],
-                    rules: loosenRules,
-                },
-            ],
-        })
-    }
-
-    if (env.next) {
-        Object.assign(config, {
-            plugins: [...config.plugins, '@next/next'],
-            rules: {
-                ...config.rules,
-                ...nextRules,
-            },
-        })
-    }
+  if (env.react) {
+    Object.assign(config, {
+      plugins: [...config.plugins, "react", "react-hooks"],
+      rules: {
+        ...config.rules,
+        ...reactRules,
+      },
+      settings: {
+        ...config.settings,
+        react: {
+          version: "detect",
+        },
+      },
+      overrides: [
+        ...config.overrides,
+        {
+          files: ["*.stories.*", "*.test.*"],
+          rules: loosenRules,
+        },
+      ],
+    });
+  }
 
-    // order is important here
-    if (env.jest) {
-        Object.assign(config, {
-            plugins: [...config.plugins, 'jest'],
-            overrides: [
-                ...config.overrides,
-                {
-                    extends: ['plugin:jest/recommended'],
-                    files: ['__test__/**', 'test/**', '*.spec.*'],
-                    env: {
-                        'jest/globals': true,
-                    },
-                    rules: {
-                        ...loosenRules,
-                        // sometimes there are problems with version detection causing this rule to crash ESLint
-                        'jest/no-deprecated-functions': 'off',
-                    },
-                },
-            ],
-        })
-    }
+  if (env.next) {
+    Object.assign(config, {
+      plugins: [...config.plugins, "@next/next"],
+      rules: {
+        ...config.rules,
+        ...nextRules,
+      },
+    });
+  }
 
-    if (env.typescript) {
-        let tsconfigPath
+  // order is important here
+  if (env.jest) {
+    Object.assign(config, {
+      plugins: [...config.plugins, "jest"],
+      overrides: [
+        ...config.overrides,
+        {
+          extends: ["plugin:jest/recommended"],
+          files: ["__test__/**", "test/**", "*.spec.*", "*.test.js"],
+          env: {
+            "jest/globals": true,
+          },
+          rules: {
+            ...loosenRules,
+            // sometimes there are problems with version detection causing this rule to crash ESLint
+            "jest/no-deprecated-functions": "off",
+          },
+        },
+      ],
+    });
+  }
 
-        try {
-            // eslint-disable-next-line @typescript-eslint/unbound-method
-            tsconfigPath = ts.findConfigFile(process.cwd(), ts.sys.fileExists, 'tsconfig.json')
-            // eslint-disable-next-line @typescript-eslint/unbound-method
-            const { config } = ts.readConfigFile(tsconfigPath, ts.sys.readFile)
-            if (config) {
-                // this are type-aware checks that require tsconfig presence
-                Object.assign(tsRules, {
-                    '@typescript-eslint/await-thenable': 'warn',
-                    '@typescript-eslint/no-floating-promises': 'error',
-                    'no-implied-eval': 'off',
-                    '@typescript-eslint/no-implied-eval': 'error',
-                    '@typescript-eslint/no-misused-promises': 'warn',
-                    '@typescript-eslint/no-unnecessary-type-assertion': 'warn',
-                    'require-await': 'off',
-                    '@typescript-eslint/require-await': 'error',
-                    '@typescript-eslint/unbound-method': ['error', { ignoreStatic: true }],
-                })
-            }
-            if (config?.compilerOptions?.strict) {
-                Object.assign(tsRules, {
-                    '@typescript-eslint/restrict-plus-operands': 'warn',
-                    '@typescript-eslint/no-explicit-any': 'error',
-                    '@typescript-eslint/no-unsafe-argument': 'error',
-                    '@typescript-eslint/no-unsafe-assignment': 'error',
-                    '@typescript-eslint/no-unsafe-call': 'error',
-                    '@typescript-eslint/no-unsafe-member-access': 'error',
-                    '@typescript-eslint/no-unsafe-return': 'error',
-                    '@typescript-eslint/restrict-template-expressions': 'error',
-                })
-            }
-        } catch {
-            void 0
-        }
+  if (env.typescript) {
+    let tsconfigPath;
 
-        Object.assign(config, {
-            plugins: [...config.plugins, '@typescript-eslint'],
-            parser: '@typescript-eslint/parser',
-            parserOptions: {
-                sourceType: 'module',
-                project: tsconfigPath,
-                ecmaFeatures: {
-                    jsx: env.react,
-                },
-                warnOnUnsupportedTypeScriptVersion: true,
-            },
-            settings: {
-                ...config.settings,
-                'import/resolver': {
-                    ...config.settings['import/resolver'],
-                    [require.resolve('eslint-import-resolver-typescript')]: {
-                        alwaysTryTypes: true,
-                    },
-                },
-            },
-            rules: {
-                ...config.rules,
-                ...tsRules,
-            },
-            overrides: [
-                ...config.overrides,
-                // ESLint can only apply one override per file, and takes first one matched,
-                // so multiple overrides should be combined.
-                // order is important.
-                {
-                    files: ['*.stories.ts', '*.stories.tsx', '*.stories.mts', '*.stories.cts'],
-                    rules: {
-                        ...loosenRules,
-                    },
-                },
-                {
-                    files: [
-                        '*.test.ts',
-                        '*.test.tsx',
-                        '*.test.mts',
-                        '*.test.cts',
-                        '__test__/**/*.ts',
-                        '__test__/**/*.tsx',
-                        '__test__/**/*.mts',
-                        '__test__/**/*.cts',
-                    ],
-                    extends: ['plugin:jest/recommended'],
-                    env: {
-                        'jest/globals': true,
-                    },
-                    rules: {
-                        ...loosenRules,
-                        'jest/no-deprecated-functions': 'off',
-                    },
-                },
-                {
-                    files: ['*.ts', '*.tsx', '*.mts', '*.cts'],
-                    rules: {
-                        ...tsRules,
-                    },
-                },
-            ],
-        })
+    try {
+      tsconfigPath = ts.findConfigFile(
+        process.cwd(),
+        ts.sys.fileExists,
+        "tsconfig.json"
+      );
+      const tsConfigFile = ts.readConfigFile(tsconfigPath, ts.sys.readFile);
+      if (tsConfigFile.config) {
+        // this are type-aware checks that require tsconfig presence
+        Object.assign(tsRules, {
+          "@typescript-eslint/await-thenable": "warn",
+          "@typescript-eslint/no-floating-promises": "error",
+          "no-implied-eval": "off",
+          "@typescript-eslint/no-implied-eval": "error",
+          "@typescript-eslint/no-misused-promises": "warn",
+          "@typescript-eslint/no-unnecessary-type-assertion": "warn",
+          "require-await": "off",
+          "@typescript-eslint/require-await": "error",
+          "@typescript-eslint/unbound-method": [
+            "error",
+            { ignoreStatic: true },
+          ],
+        });
+      }
+      if (config?.compilerOptions?.strict) {
+        Object.assign(tsRules, {
+          "@typescript-eslint/restrict-plus-operands": "warn",
+          "@typescript-eslint/no-explicit-any": "error",
+          "@typescript-eslint/no-unsafe-argument": "error",
+          "@typescript-eslint/no-unsafe-assignment": "error",
+          "@typescript-eslint/no-unsafe-call": "error",
+          "@typescript-eslint/no-unsafe-member-access": "error",
+          "@typescript-eslint/no-unsafe-return": "error",
+          "@typescript-eslint/restrict-template-expressions": "error",
+        });
+      }
+    } catch {
+      void 0;
     }
 
-    return config
-}
+    Object.assign(config, {
+      plugins: [...config.plugins, "@typescript-eslint"],
+      parser: "@typescript-eslint/parser",
+      parserOptions: {
+        sourceType: "module",
+        project: tsconfigPath,
+        ecmaFeatures: {
+          jsx: env.react,
+        },
+        warnOnUnsupportedTypeScriptVersion: true,
+      },
+      settings: {
+        ...config.settings,
+        "import/resolver": {
+          ...config.settings["import/resolver"],
+          [require.resolve("eslint-import-resolver-typescript")]: {
+            alwaysTryTypes: true,
+          },
+        },
+      },
+      rules: {
+        ...config.rules,
+        ...tsRules,
+      },
+      overrides: [
+        ...config.overrides,
+        // ESLint can only apply one override per file, and takes first one matched,
+        // so multiple overrides should be combined.
+        // order is important.
+        {
+          files: [
+            "*.stories.ts",
+            "*.stories.tsx",
+            "*.stories.mts",
+            "*.stories.cts",
+          ],
+          rules: {
+            ...loosenRules,
+          },
+        },
+        {
+          files: [
+            "*.test.ts",
+            "*.test.tsx",
+            "*.test.mts",
+            "*.test.cts",
+            "__test__/**/*.ts",
+            "__test__/**/*.tsx",
+            "__test__/**/*.mts",
+            "__test__/**/*.cts",
+          ],
+          extends: ["plugin:jest/recommended"],
+          env: {
+            "jest/globals": true,
+          },
+          rules: {
+            ...loosenRules,
+            "jest/no-deprecated-functions": "off",
+          },
+        },
+        {
+          files: ["*.ts", "*.tsx", "*.mts", "*.cts"],
+          rules: {
+            ...tsRules,
+          },
+        },
+      ],
+    });
+  }
+
+  return config;
+};

package/lib/env.js

@@ -1,29 +1,60 @@
-const fs = require('fs')
-const path = require('path')
+const fs = require("fs");
+const path = require("path");
 
-let packageJsonPath = process.cwd()
+let packageJsonPath = process.cwd();
 
-while (!fs.existsSync(path.join(packageJsonPath, 'package.json'))) {
-    packageJsonPath = path.dirname(packageJsonPath)
-    if (packageJsonPath === path.resolve('/')) {
-        packageJsonPath = null
-        break
-    }
+while (!fs.existsSync(path.join(packageJsonPath, "package.json"))) {
+  packageJsonPath = path.dirname(packageJsonPath);
+  if (packageJsonPath === path.resolve("/")) {
+    packageJsonPath = null;
+    break;
+  }
 }
 
-const packageJson = packageJsonPath === null ? null : require(path.join(packageJsonPath, 'package.json'))
+let workspacePackageJsonPath = packageJsonPath
+  ? path.dirname(packageJsonPath)
+  : "/";
 
-const has = (...libs) =>
-    libs.some((name) => {
-        if (packageJsonPath === null) {
-            return false
-        }
-        return !!packageJson.dependencies?.[name] || !!packageJson.devDependencies?.[name]
-    })
+while (!fs.existsSync(path.join(workspacePackageJsonPath, "package.json"))) {
+  workspacePackageJsonPath = path.dirname(workspacePackageJsonPath);
+  if (workspacePackageJsonPath === path.resolve("/")) {
+    workspacePackageJsonPath = null;
+    break;
+  }
+}
 
-module.exports = {
-    react: has('react'),
-    next: has('next'),
-    typescript: has('typescript'),
-    jest: has('jest'),
+const packageJson =
+  packageJsonPath === null
+    ? null
+    : require(path.join(packageJsonPath, "package.json"));
+
+let workspaceJson =
+  workspacePackageJsonPath === null
+    ? null
+    : require(path.join(workspacePackageJsonPath, "package.json"));
+
+if (!workspaceJson?.workspaces) {
+  workspaceJson = null;
 }
+
+const dependencies = new Set([
+  ...(packageJson?.dependencies ? Object.keys(packageJson.dependencies) : []),
+  ...(packageJson?.devDependencies
+    ? Object.keys(packageJson.devDependencies)
+    : []),
+  ...(workspaceJson?.dependencies
+    ? Object.keys(workspaceJson?.dependencies)
+    : []),
+  ...(workspaceJson?.devDependencies
+    ? Object.keys(workspaceJson?.devDependencies)
+    : []),
+]);
+
+const has = (...libs) => libs.some((name) => dependencies.has(name));
+
+module.exports = {
+  react: has("react"),
+  next: has("next"),
+  typescript: has("typescript"),
+  jest: has("jest"),
+};