@oriflame/config-eslint 4.3.0 → 5.0.3

package/lib/rules/unicorn.js

@@ -1,105 +0,0 @@
-"use strict";
-const config = {
-    // override ESLint rules
-    'no-nested-ternary': 'off',
-    'no-warning-comments': 'off',
-    // eslint-plugin-unicorn rules
-    'unicorn/better-regex': 'warn',
-    'unicorn/catch-error-name': 'warn',
-    'unicorn/consistent-destructuring': 'error',
-    'unicorn/consistent-function-scoping': 'error',
-    'unicorn/custom-error-definition': 'error',
-    'unicorn/empty-brace-spaces': 'off',
-    'unicorn/error-message': 'error',
-    'unicorn/escape-case': 'warn',
-    'unicorn/expiring-todo-comments': [
-        'warn',
-        {
-            terms: ['todo:', 'fixme:', 'debug:'],
-        },
-    ],
-    'unicorn/explicit-length-check': 'off',
-    'unicorn/filename-case': 'off',
-    'unicorn/import-index': 'error',
-    'unicorn/import-style': 'warn',
-    'unicorn/new-for-builtins': 'warn',
-    'unicorn/no-abusive-eslint-disable': 'off',
-    'unicorn/no-array-callback-reference': 'off',
-    'unicorn/no-array-for-each': 'off',
-    'unicorn/no-array-method-this-argument': 'error',
-    'unicorn/no-array-push-push': 'error',
-    'unicorn/no-array-reduce': 'off',
-    'unicorn/no-console-spaces': 'off',
-    'unicorn/no-document-cookie': 'error',
-    'unicorn/no-for-loop': 'warn',
-    'unicorn/no-hex-escape': 'warn',
-    'unicorn/no-instanceof-array': 'error',
-    'unicorn/no-keyword-prefix': 'off',
-    'unicorn/no-lonely-if': 'warn',
-    'unicorn/no-nested-ternary': 'error',
-    'unicorn/no-new-array': 'error',
-    'unicorn/no-new-buffer': 'error',
-    'unicorn/no-null': 'off',
-    'unicorn/no-object-as-default-parameter': 'error',
-    'unicorn/no-process-exit': 'off',
-    'unicorn/no-static-only-class': 'error',
-    'unicorn/no-this-assignment': 'off',
-    'unicorn/no-unreadable-array-destructuring': 'warn',
-    'unicorn/no-unsafe-regex': 'off',
-    'unicorn/no-unused-properties': 'off',
-    'unicorn/no-useless-length-check': 'off',
-    'unicorn/no-useless-spread': 'error',
-    'unicorn/no-useless-undefined': 'off',
-    'unicorn/no-zero-fractions': 'error',
-    'unicorn/number-literal-case': 'error',
-    'unicorn/numeric-separators-style': 'off',
-    'unicorn/prefer-add-event-listener': 'error',
-    'unicorn/prefer-array-find': 'warn',
-    'unicorn/prefer-array-flat': 'warn',
-    'unicorn/prefer-array-flat-map': 'warn',
-    'unicorn/prefer-array-index-of': 'warn',
-    'unicorn/prefer-array-some': 'warn',
-    'unicorn/prefer-at': 'warn',
-    'unicorn/prefer-date-now': 'warn',
-    'unicorn/prefer-default-parameters': 'error',
-    'unicorn/prefer-dom-node-append': 'warn',
-    'unicorn/prefer-dom-node-dataset': 'warn',
-    'unicorn/prefer-dom-node-remove': 'warn',
-    'unicorn/prefer-dom-node-text-content': 'warn',
-    'unicorn/prefer-includes': 'off',
-    'unicorn/prefer-keyboard-event-key': 'warn',
-    'unicorn/prefer-math-trunc': 'warn',
-    'unicorn/prefer-modern-dom-apis': 'warn',
-    // FIXME [@rajzik]: This is good rule but adoption isn't great so far
-    'unicorn/prefer-module': 'off',
-    // FIXME [eslint-plugin-import]: Once https://github.com/benmosher/eslint-plugin-import/issues/2035 is closed can be enabled
-    'unicorn/prefer-node-protocol': 'off',
-    'unicorn/prefer-negative-index': 'warn',
-    'unicorn/prefer-number-properties': 'warn',
-    'unicorn/prefer-object-from-entries': 'warn',
-    'unicorn/prefer-object-has-own': 'warn',
-    'unicorn/prefer-optional-catch-binding': 'off',
-    'unicorn/prefer-prototype-methods': 'warn',
-    'unicorn/prefer-query-selector': 'off',
-    'unicorn/prefer-reflect-apply': 'off',
-    'unicorn/prefer-regexp-test': 'warn',
-    // Autofix can be breaking
-    'unicorn/prefer-set-has': 'warn',
-    'unicorn/prefer-spread': 'error',
-    'unicorn/prefer-string-replace-all': 'warn',
-    'unicorn/prefer-string-slice': 'warn',
-    'unicorn/prefer-string-starts-ends-with': 'warn',
-    'unicorn/prefer-string-trim-start-end': 'warn',
-    'unicorn/prefer-switch': 'off',
-    'unicorn/prefer-ternary': 'off',
-    // FIXME [@jakubmazanec]: enable when we support top-level await
-    'unicorn/prefer-top-level-await': 'off',
-    'unicorn/prefer-type-error': 'warn',
-    'unicorn/prevent-abbreviations': 'off',
-    'unicorn/require-array-join-separator': 'error',
-    'unicorn/require-number-to-fixed-digits-argument': 'error',
-    'unicorn/require-post-message-target-origin': 'error',
-    'unicorn/string-content': 'off',
-    'unicorn/throw-new-error': 'error', // require new when throwing an error
-};
-module.exports = config;

package/src/presets/SECRET_CONFIG_DO_NOT_USE_THIS.ts

@@ -1,34 +0,0 @@
-import { ESLintConfig } from '@beemo/driver-eslint';
-import { TSX_EXTS_GROUP } from '@oriflame/lumos-common';
-
-const config: ESLintConfig = {
-  rules: {
-    'node/no-sync': 'off',
-    'node/global-require': 'off',
-    'import/no-dynamic-require': 'off',
-    'no-underscore-dangle': 'off',
-    'eslint-comments/disable-enable-pair': 'off',
-    'eslint-comments/no-aggregating-enable': 'off',
-    'eslint-comments/no-duplicate-disable': 'off',
-    'eslint-comments/no-restricted-disable': 'off',
-    'eslint-comments/no-unlimited-disable': 'off',
-    'eslint-comments/no-unused-disable': 'off',
-    'eslint-comments/no-unused-enable': 'off',
-    'eslint-comments/no-use': 'off',
-    'eslint-comments/require-description': 'off',
-  },
-  overrides: [
-    {
-      files: [`*.${TSX_EXTS_GROUP}`],
-      rules: {
-        '@typescript-eslint/no-non-null-assertion': 'off',
-        '@typescript-eslint/naming-convention': 'off',
-        '@typescript-eslint/no-require-imports': 'off',
-        '@typescript-eslint/prefer-nullish-coalescing': 'off',
-        '@typescript-eslint/strict-boolean-expressions': 'off',
-      },
-    },
-  ],
-};
-
-export = config;

package/src/presets/base.ts

@@ -1,143 +0,0 @@
-import { Path } from '@beemo/core';
-import { ESLintConfig } from '@beemo/driver-eslint';
-import {
-  ASSET_EXT_PATTERN,
-  CSS_EXT_PATTERN,
-  EXTS,
-  GQL_EXT_PATTERN,
-  TJSX_EXTS_GROUP,
-} from '@oriflame/lumos-common';
-
-import a11yRules from '../rules/a11y';
-import eslintRules from '../rules/eslint';
-import commentsRules from '../rules/eslint-comments';
-import importRules from '../rules/import';
-import jestRules from '../rules/jest';
-import promiseRules from '../rules/promise';
-import reactRules from '../rules/react';
-import reactHooksRules from '../rules/react-hooks';
-import testingLibraryRules from '../rules/testing-library';
-
-const config: ESLintConfig = {
-  root: true,
-
-  parser: 'babel-eslint',
-
-  parserOptions: {
-    requireConfigFile: false,
-    ecmaVersion: 2020,
-    sourceType: 'module',
-    ecmaFeatures: {
-      jsx: true,
-    },
-  },
-  // Extends list for base plugins
-  extends: [
-    'eslint:recommended',
-    'plugin:react/recommended',
-    'plugin:react-hooks/recommended',
-    'plugin:jsx-a11y/recommended',
-    'plugin:eslint-plugin-promise/recommended',
-  ],
-  // Required plugins; doesn't matter if they are used or not.
-  plugins: ['import', 'react', 'react-hooks', 'jsx-a11y', 'node', 'eslint-comments', 'promise'],
-
-  env: {
-    browser: true,
-    es2020: true,
-    worker: true,
-    serviceworker: true,
-  },
-
-  globals: {
-    __DEV__: 'readonly',
-
-    // metrics and analytics providers
-    ga: 'readonly',
-    newrelic: 'readonly',
-
-    // mostly for easier compatibility between browsers, workers, etc
-    global: 'readonly',
-
-    // mostly references to `process.env.NODE_ENV`
-    process: 'readonly',
-
-    // references for globalThis
-    globalThis: 'readonly',
-
-    // Webpack variables
-    __webpack_public_path__: 'writeable',
-    __webpack_require__: 'readonly',
-    __webpack_chunk_load__: 'readonly',
-    __webpack_modules__: 'readonly',
-    __webpack_hash__: 'readonly',
-    __non_webpack_require__: 'readonly',
-    __webpack_exports_info__: 'readonly',
-    DEBUG: 'readonly',
-  },
-
-  reportUnusedDisableDirectives: true,
-
-  settings: {
-    'import/ignore': [
-      'node_modules',
-      '\\.json$',
-      ASSET_EXT_PATTERN.source,
-      CSS_EXT_PATTERN.source,
-      GQL_EXT_PATTERN.source,
-    ],
-    'import/extensions': EXTS,
-    'import/resolver': {
-      node: {
-        extensions: EXTS,
-      },
-      [Path.resolve('../resolvers/graphql.js', __dirname).path()]: {
-        extensions: ['.gql', '.graphql'],
-      },
-    },
-    react: {
-      version: 'detect',
-    },
-    propWrapperFunctions: ['forbidExtraProps', 'exact', 'Object.freeze'],
-  },
-
-  rules: {
-    ...eslintRules,
-    ...commentsRules,
-    ...promiseRules,
-    ...importRules,
-    ...reactRules,
-    ...reactHooksRules,
-    ...a11yRules,
-  },
-  overrides: [
-    {
-      files: [`*.test.${TJSX_EXTS_GROUP}`],
-
-      plugins: ['jest', 'testing-library'],
-
-      extends: ['plugin:jest/recommended', 'plugin:testing-library/react'],
-
-      settings: {
-        'testing-library/custom-queries': 'off',
-        'testing-library/custom-renders': 'off',
-        'testing-library/utils-module': 'off',
-      },
-
-      globals: {
-        jsdom: 'readonly',
-      },
-
-      env: {
-        jest: true,
-        node: true,
-      },
-      rules: {
-        ...jestRules,
-        ...testingLibraryRules,
-      },
-    },
-  ],
-};
-
-export = config;

package/src/presets/future.ts

@@ -1,13 +0,0 @@
-import { ESLintConfig } from '@beemo/driver-eslint';
-
-import unicornRules from '../rules/unicorn';
-
-const config: ESLintConfig = {
-  extends: ['plugin:unicorn/recommended'],
-  plugins: ['unicorn'],
-  rules: {
-    ...unicornRules,
-  },
-};
-
-export = config;

package/src/presets/nextjs.ts

@@ -1,13 +0,0 @@
-import { ESLintConfig } from '@beemo/driver-eslint';
-
-import nextjsRules from '../rules/nextjs';
-
-const config: ESLintConfig = {
-  extends: ['plugin:@next/next/recommended'],
-  plugins: ['@next/next'],
-  rules: {
-    ...nextjsRules,
-  },
-};
-
-export = config;

package/src/presets/node.ts

@@ -1,27 +0,0 @@
-import { ESLintConfig } from '@beemo/driver-eslint';
-
-import nodeRules from '../rules/node';
-
-const config: ESLintConfig = {
-  // TODO: this also sets up globals and `parserOptions.sourceType`, so we should decide in later versions of this config if we want to use this `extends`
-  // extends: ['plugin:node/recommended'],
-
-  env: {
-    browser: false,
-    node: true,
-  },
-  settings: {
-    node: {
-      tryExtensions: ['.js', '.jsx', '.json'],
-    },
-    convertPath: {
-      'src/**/*.ts': ['^src/(.+?)\\.ts$', 'lib/$1.js'],
-      'src/**/*.tsx': ['^src/(.+?)\\.tsx$', 'lib/$1.js'],
-    },
-  },
-  rules: {
-    ...nodeRules,
-  },
-};
-
-export = config;

package/src/presets/prettier.ts

@@ -1,13 +0,0 @@
-import { ESLintConfig } from '@beemo/driver-eslint';
-
-import prettierRules from '../rules/prettier';
-
-const config: ESLintConfig = {
-  extends: ['eslint-config-prettier'],
-  plugins: ['eslint-plugin-prettier'],
-  rules: {
-    ...prettierRules,
-  },
-};
-
-export = config;

package/src/presets/typescript.ts

@@ -1,45 +0,0 @@
-import { ESLintConfig } from '@beemo/driver-eslint';
-import { EXTS, TSX_EXTS_GROUP, fromRoot } from '@oriflame/lumos-common';
-
-import typescriptRules from '../rules/typescript';
-
-const project = fromRoot('tsconfig.eslint.json', true) || fromRoot('tsconfig.json');
-
-const config: ESLintConfig = {
-  overrides: [
-    {
-      files: [`*.${TSX_EXTS_GROUP}`],
-
-      plugins: ['@typescript-eslint'],
-
-      parser: '@typescript-eslint/parser',
-
-      parserOptions: {
-        project,
-      },
-
-      settings: {
-        node: {
-          tryExtensions: EXTS,
-        },
-        'import/resolver': {
-          typescript: {},
-        },
-        'import/parsers': {
-          '@typescript-eslint/parser': ['.ts', '.tsx'],
-        },
-      },
-      rules: {
-        ...typescriptRules,
-      },
-    },
-    {
-      files: [`*.test.${TSX_EXTS_GROUP}`],
-      rules: {
-        '@typescript-eslint/ban-ts-comment': 'off',
-      },
-    },
-  ],
-};
-
-export = config;

package/src/resolvers/graphql.ts

@@ -1,11 +0,0 @@
-import { GQL_EXT_PATTERN } from '@oriflame/lumos-common';
-
-export const interfaceVersion = 2;
-
-export function resolve(source: string): { found: boolean; path?: unknown } {
-  if (GQL_EXT_PATTERN.test(source)) {
-    return { found: true, path: null };
-  }
-
-  return { found: false };
-}

package/src/rules/a11y.ts

@@ -1,47 +0,0 @@
-import { ESLintConfig } from '@beemo/driver-eslint';
-
-const config: ESLintConfig['rules'] = {
-  // eslint-plugin-jsx-a11y rules
-  'jsx-a11y/alt-text': 'warn', // enforce all elements that require alternative text have meaningful information to relay back to end user
-  'jsx-a11y/anchor-has-content': ['warn', { components: ['Link'] }], // enforce all anchors to contain accessible content
-  'jsx-a11y/anchor-is-valid': [
-    'error',
-    {
-      components: ['Link'],
-      specialLink: ['hrefLeft', 'hrefRight', 'to'],
-    },
-  ], // enforce all anchors are valid, navigable elements
-  'jsx-a11y/aria-activedescendant-has-tabindex': 'warn', // enforce elements with aria-active descendant are tab-able
-  'jsx-a11y/aria-props': 'error', // enforce all aria-* props are valid
-  'jsx-a11y/aria-proptypes': 'error', // enforce ARIA state and property values are valid
-  'jsx-a11y/aria-role': ['error', { ignoreNonDOM: false }], // enforce that elements with ARIA roles must use a valid, non-abstract ARIA role
-  'jsx-a11y/aria-unsupported-elements': 'error', // enforce that elements that do not support ARIA roles, states, and properties do not have those attributes
-  'jsx-a11y/autocomplete-valid': 'warn', // enforce that autocomplete attributes are used correctly.
-  'jsx-a11y/click-events-have-key-events': 'warn', // enforce a clickable non-interactive element has at least one keyboard event listener
-  'jsx-a11y/control-has-associated-label': 'warn', // enforce that a control (an interactive element) has a text label
-  'jsx-a11y/heading-has-content': ['warn', { components: ['Typography'] }], // enforce heading (h1, h2, etc) elements contain accessible content
-  'jsx-a11y/html-has-lang': 'warn', // enforce <html> element has lang prop
-  'jsx-a11y/iframe-has-title': 'warn', // enforce iframe elements have a title attribute
-  'jsx-a11y/img-redundant-alt': 'warn', // enforce <img> alt prop does not contain the word "image", "picture", or "photo"
-  'jsx-a11y/interactive-supports-focus': 'warn', // enforce that elements with interactive handlers like onClick must be focusable
-  'jsx-a11y/label-has-associated-control': 'warn', // enforce that a label tag has a text label and an associated control
-  'jsx-a11y/lang': 'error', // enforce lang attribute has a valid value
-  'jsx-a11y/media-has-caption': 'warn', // enforces that <audio> and <video> elements must have a <track> for captions
-  'jsx-a11y/mouse-events-have-key-events': 'warn', // enforce that onMouseOver/onMouseOut are accompanied by onFocus/onBlur for keyboard-only users
-  'jsx-a11y/no-access-key': 'warn', // enforce that the accessKey prop is not used on any element to avoid complications with keyboard commands used by a screen reader
-  'jsx-a11y/no-autofocus': 'off', // enforce autoFocus prop is not used
-  'jsx-a11y/no-distracting-elements': 'error', // enforce distracting elements are not used
-  'jsx-a11y/no-interactive-element-to-noninteractive-role': 'warn', // interactive elements should not be assigned non-interactive roles
-  'jsx-a11y/no-noninteractive-element-interactions': 'warn', // non-interactive elements should not be assigned mouse or keyboard event listeners
-  'jsx-a11y/no-noninteractive-element-to-interactive-role': 'warn', // Non-interactive elements should not be assigned interactive roles
-  'jsx-a11y/no-noninteractive-tabindex': 'error', // tabIndex should only be declared on interactive elements
-  'jsx-a11y/no-onchange': 'warn', // enforce usage of onBlur over onChange on select menus for accessibility
-  'jsx-a11y/no-redundant-roles': 'error', // enforce explicit role property is not the same as implicit/default role property on element
-  'jsx-a11y/no-static-element-interactions': 'warn', // enforce that non-interactive, visible elements (such as <div>) that have click handlers use the role attribute
-  'jsx-a11y/role-has-required-aria-props': 'error', // enforce that elements with ARIA roles must have all required attributes for that role
-  'jsx-a11y/role-supports-aria-props': 'error', // enforce that elements with explicit or implicit roles defined contain only aria-* properties supported by that role
-  'jsx-a11y/scope': 'error', // enforce scope prop is only used on <th> elements
-  'jsx-a11y/tabindex-no-positive': 'error', // enforce tabIndex value is not greater than zero
-};
-
-export = config;

package/src/rules/eslint-comments.ts

@@ -1,16 +0,0 @@
-import { ESLintConfig } from '@beemo/driver-eslint';
-
-const config: ESLintConfig['rules'] = {
-  // eslint-plugin-eslint-comments rules
-  'eslint-comments/disable-enable-pair': ['error', { allowWholeFile: true }], // require a eslint-enable comment for every eslint-disable comment
-  'eslint-comments/no-aggregating-enable': 'error', // disallow a eslint-enable comment for multiple eslint-disable comments
-  'eslint-comments/no-duplicate-disable': 'error', // disallow duplicate eslint-disable comments
-  'eslint-comments/no-restricted-disable': 'off', // disallow eslint-disable comments about specific rules
-  'eslint-comments/no-unlimited-disable': 'error', // disallow eslint-disable comments without rule names
-  'eslint-comments/no-unused-disable': 'off', // disallow unused eslint-disable comments
-  'eslint-comments/no-unused-enable': 'error', // disallow unused eslint-enable comments
-  'eslint-comments/no-use': 'off', // disallow ESLint directive-comments
-  'eslint-comments/require-description': ['error', { ignore: ['eslint-enable'] }], // require include descriptions in ESLint directive-comments
-};
-
-export = config;