shared eslint config

Author: wildmolasses

Diff for: .eslintignore

@@ -1,2 +1,3 @@
 /dist
-/node_modules
+/node_modules
+.eslintrc.js

Diff for: .eslintrc.js

@@ -1,55 +1,32 @@
 module.exports = {
-  root: true,
-
-  parserOptions: {
-    parser: 'babel-eslint',
-    sourceType: 'module'
-  },
-
   env: {
-    browser: true,
-    mocha: true,
+    es2021: true,
   },
 
   extends: [
-    'airbnb-base',
-    'plugin:chai-friendly/recommended',
-  ],
-
-  plugins: [
-    'chai-friendly',
+    'eslint:recommended',
+    'plugin:@typescript-eslint/recommended',
+    'plugin:@typescript-eslint/recommended-requiring-type-checking',
+    'plugin:vue/essential',
+    'prettier',
+    'prettier/@typescript-eslint',
+    'prettier/vue',
   ],
 
-  globals: {
-    'ga': true, // Google Analytics
-    'cordova': true,
-    '__statics': true,
-    'process': true,
-    'Capacitor': true,
-    'chrome': true
-  },
+  plugins: ['@typescript-eslint', 'vue'],
 
-  // add your custom rules here
   rules: {
-    'no-param-reassign': 'off',
-
-    'import/first': 'off',
-    'import/named': 'error',
-    'import/namespace': 'error',
-    'import/default': 'error',
-    'import/export': 'error',
-    'import/extensions': 'off',
-    'import/no-unresolved': 'off',
-    'import/no-extraneous-dependencies': 'off',
-    'import/prefer-default-export': 'off',
+    'prefer-const': 'error',
     'prefer-promise-reject-errors': 'off',
-
-    // allow debugger during development only
+    quotes: ['warn', 'single', { avoidEscape: true }],
+    '@typescript-eslint/explicit-function-return-type': 'off',
+    '@typescript-eslint/explicit-module-boundary-types': 'off',
+    '@typescript-eslint/no-unsafe-member-access': 'off',
+    '@typescript-eslint/no-unsafe-call': 'off',
+    '@typescript-eslint/no-unsafe-assignment': 'off',
+    '@typescript-eslint/no-var-requires': 'off',
+    '@typescript-eslint/unbound-method': ['error', { ignoreStatic: true }],
+    '@typescript-eslint/no-unused-vars': ['error', { ignoreRestSiblings: true }],
     'no-debugger': process.env.NODE_ENV === 'production' ? 'error' : 'off',
-
-    // Max line length of 100 characters, except for lines with comments or strings
-    'max-len': ['error', {
-      code: 100, ignoreComments: true, ignoreTrailingComments: true, ignoreStrings: true,
-    }],
-  }
-}
+  },
+};

Diff for: frontend/.eslintignore

@@ -4,4 +4,5 @@
 /src-cordova
 /.quasar
 /node_modules
-/src-ssr
+/src-ssr
+.eslintrc.js

Diff for: frontend/.eslintrc.js

@@ -1,86 +1,16 @@
-const { resolve } = require('path');
 module.exports = {
-  // https://eslint.org/docs/user-guide/configuring#configuration-cascading-and-hierarchy
-  // This option interrupts the configuration hierarchy at this file
-  // Remove this if you have an higher level ESLint config file (it usually happens into a monorepos)
-  root: true,
-
-  // https://eslint.vuejs.org/user-guide/#how-to-use-custom-parser
-  // Must use parserOptions instead of "parser" to allow vue-eslint-parser to keep working
-  // `parser: 'vue-eslint-parser'` is already included with any 'plugin:vue/**' config and should be omitted
+  extends: '../.eslintrc',
   parserOptions: {
-    // https://github.com/typescript-eslint/typescript-eslint/tree/master/packages/parser#configuration
-    // https://github.com/TypeStrong/fork-ts-checker-webpack-plugin#eslint
-    // Needed to make the parser take into account 'vue' files
-    extraFileExtensions: ['.vue'],
+    project: 'tsconfig.json',
     parser: '@typescript-eslint/parser',
-    project: resolve(__dirname, './tsconfig.json'),
-    tsconfigRootDir: __dirname,
-    ecmaVersion: 2018, // Allows for the parsing of modern ECMAScript features
-    sourceType: 'module', // Allows for the use of imports
-  },
-
-  env: {
-    browser: true,
+    extraFileExtensions: ['.vue'],
   },
-
-  // Rules order is important, please avoid shuffling them
-  extends: [
-    // Base ESLint recommended rules
-    // 'eslint:recommended',
-
-    // https://github.com/typescript-eslint/typescript-eslint/tree/master/packages/eslint-plugin#usage
-    // ESLint typescript rules
-    'plugin:@typescript-eslint/recommended',
-    // consider disabling this class of rules if linting takes too long
-    'plugin:@typescript-eslint/recommended-requiring-type-checking',
-
-    // Uncomment any of the lines below to choose desired strictness,
-    // but leave only one uncommented!
-    // See https://eslint.vuejs.org/rules/#available-rules
-    'plugin:vue/essential', // Priority A: Essential (Error Prevention)
-    // 'plugin:vue/strongly-recommended', // Priority B: Strongly Recommended (Improving Readability)
-    // 'plugin:vue/recommended', // Priority C: Recommended (Minimizing Arbitrary Choices and Cognitive Overhead)
-
-    // https://github.com/prettier/eslint-config-prettier#installation
-    // usage with Prettier, provided by 'eslint-config-prettier'.
-    'prettier',
-    'prettier/@typescript-eslint',
-    'prettier/vue',
-  ],
-
-  plugins: [
-    // required to apply rules which need type information
-    '@typescript-eslint',
-
-    // https://eslint.vuejs.org/user-guide/#why-doesn-t-it-work-on-vue-file
-    // required to lint *.vue files
-    'vue',
-
-    // https://github.com/typescript-eslint/typescript-eslint/issues/389#issuecomment-509292674
-    // Prettier has not been included as plugin to avoid performance impact
-    // add it as an extension for your IDE
-  ],
-
   globals: {
-    ga: true, // Google Analytics
+    ga: true,
     cordova: true,
     __statics: true,
     process: true,
     Capacitor: true,
     chrome: true,
   },
-
-  // add your custom rules here
-  rules: {
-    'prefer-promise-reject-errors': 'off',
-
-    // TypeScript
-    quotes: ['warn', 'single', { avoidEscape: true }],
-    '@typescript-eslint/explicit-function-return-type': 'off',
-    '@typescript-eslint/explicit-module-boundary-types': 'off',
-
-    // allow debugger during development only
-    'no-debugger': process.env.NODE_ENV === 'production' ? 'error' : 'off',
-  },
 };

Diff for: package.json

@@ -21,7 +21,8 @@
   },
   "scripts": {
     "build": "lerna run build",
-    "clean": "lerna run clean"
+    "clean": "lerna run clean",
+    "lint": "lerna run lint"
   },
   "husky": {
     "hooks": {

Diff for: umbra-js/.eslintignore

@@ -1,3 +1,6 @@
 /dist
 /node_modules
 /build
+/test
+.eslintrc.js
+test-environment.config.js

Diff for: umbra-js/.eslintrc.js

@@ -1,48 +1,7 @@
 module.exports = {
-  // https://eslint.org/docs/user-guide/configuring#configuration-cascading-and-hierarchy
-  // This option interrupts the configuration hierarchy at this file
-  // Remove this if you have an higher level ESLint config file (it usually happens into a monorepos)
-  root: true,
-
+  extends: '../.eslintrc',
   parserOptions: {
-    parser: 'babel-eslint',
-    ecmaVersion: 2018, // Allows for the parsing of modern ECMAScript features
-    sourceType: 'module', // Allows for the use of imports
-  },
-
-  env: {
-    browser: true,
-  },
-
-  // Rules order is important, please avoid shuffling them
-  extends: [
-    // https://github.com/prettier/eslint-config-prettier#installation
-    // usage with Prettier, provided by 'eslint-config-prettier'.
-    'prettier',
-  ],
-
-  plugins: [],
-
-  globals: {
-    ga: true, // Google Analytics
-    cordova: true,
-    __statics: true,
-    process: true,
-    Capacitor: true,
-    chrome: true,
-  },
-
-  // add your custom rules here
-  rules: {
-    // Max line length of 100 characters, except for lines with comments or strings
-    'max-len': [
-      'error',
-      {
-        code: 100,
-        ignoreComments: true,
-        ignoreTrailingComments: true,
-        ignoreStrings: true,
-      },
-    ],
-  },
+    project: 'tsconfig.json',
+    parser: '@typescript-eslint/parser'
+  }
 };

Diff for: umbra-js/package.json

@@ -7,7 +7,7 @@
   "files": ["build/**/*"],
   "scripts": {
     "test": "yarn build && mocha -r ts-node/register ./test/*.test.ts --exit --timeout 0",
-    "lint": "eslint --ext .js .",
+    "lint": "eslint --ext .js,.ts ./",
     "prettier": "prettier --write .",
     "watch": "tsc --watch",
     "build": "tsc --build",

Diff for: umbra-js/tsconfig.json

@@ -2,6 +2,7 @@
   "extends": "../tsconfig.settings.json",
   "compilerOptions": {
     "composite": true,
+    "rootDir": "src",
     "outDir": "build",
     "resolveJsonModule": true,
     "forceConsistentCasingInFileNames": true,