@ocavue/eslint-config 2.16.0 → 2.17.0

package/src/typescript.js

@@ -1,128 +0,0 @@
-// @ts-check
-
-import eslint from '@eslint/js'
-import tseslint from 'typescript-eslint'
-
-import { GLOB_JS, GLOB_JSX, GLOB_TEST, GLOB_TS, GLOB_TSX } from './shared.js'
-
-export { tseslint }
-
-export function typescript() {
-  const rules = [...tseslint.configs.recommended, ...tseslint.configs.stylistic]
-    .map((config) => config.rules || {})
-    .reduce((acc, cur) => ({ ...acc, ...cur }), {})
-
-  /** @type {import('@typescript-eslint/utils').TSESLint.FlatConfig.ConfigArray} */
-  const config = [
-    eslint.configs.recommended,
-    {
-      name: 'typescript',
-      files: [GLOB_TS, GLOB_TSX, GLOB_JS, GLOB_JSX],
-      languageOptions: {
-        parser: tseslint.parser,
-        parserOptions: {
-          projectService: true,
-          sourceType: 'module',
-          ecmaVersion: 'latest',
-        },
-      },
-      plugins: {
-        '@typescript-eslint': tseslint.plugin,
-      },
-      rules: {
-        ...rules,
-
-        '@typescript-eslint/consistent-type-definitions': 'off',
-        '@typescript-eslint/prefer-optional-chain': 'off',
-        '@typescript-eslint/prefer-nullish-coalescing': 'off',
-        '@typescript-eslint/consistent-indexed-object-style': 'off',
-        '@typescript-eslint/array-type': 'off',
-        '@typescript-eslint/dot-notation': 'off',
-        '@typescript-eslint/triple-slash-reference': 'off',
-        '@typescript-eslint/consistent-type-imports': [
-          'warn',
-          {
-            // Allow type imports in type annotations (e.g. `type T = import('Foo').Foo`)
-            disallowTypeAnnotations: false,
-          },
-        ],
-        '@typescript-eslint/no-import-type-side-effects': 'warn',
-        '@typescript-eslint/no-unnecessary-type-assertion': 'warn',
-        '@typescript-eslint/no-unnecessary-parameter-property-assignment':
-          'warn',
-        '@typescript-eslint/restrict-plus-operands': 'warn',
-        '@typescript-eslint/no-unsafe-call': 'warn',
-        '@typescript-eslint/no-unsafe-return': 'warn',
-        '@typescript-eslint/no-unsafe-argument': 'warn',
-        '@typescript-eslint/no-unsafe-member-access': 'warn',
-        '@typescript-eslint/no-unsafe-assignment': 'warn',
-        '@typescript-eslint/no-floating-promises': 'warn',
-        '@typescript-eslint/no-explicit-any': 'warn',
-        '@typescript-eslint/explicit-module-boundary-types': 'off',
-        '@typescript-eslint/no-non-null-assertion': 'off',
-        '@typescript-eslint/restrict-template-expressions': 'off',
-        '@typescript-eslint/no-unused-vars': [
-          'error',
-          {
-            argsIgnorePattern: '^_',
-            varsIgnorePattern: '^_',
-            caughtErrorsIgnorePattern: '^_',
-          },
-        ],
-        '@typescript-eslint/no-extra-semi': 'off',
-        '@typescript-eslint/prefer-function-type': 'warn',
-
-        '@typescript-eslint/no-misused-promises': [
-          'error',
-          { checksVoidReturn: false },
-        ],
-        '@typescript-eslint/await-thenable': 'error',
-        '@typescript-eslint/unbound-method': 'error',
-
-        // `type T1 = T0` and `interface T2 extends T0 {}` have the same meaning
-        // but different behavior in TypeScript type checking. `T1` and `T0` are
-        // the same type, while `T2` is different than `T0`. We allow `interface
-        // T2 extends T0 {}` explicitly.
-        '@typescript-eslint/no-empty-object-type': [
-          'error',
-          {
-            allowInterfaces: 'with-single-extends',
-          },
-        ],
-
-        // Turn off this rule because it's incompatible with the `--isolatedDeclarations` compiler option.
-        '@typescript-eslint/no-inferrable-types': 'off',
-
-        // TODO: We should set the rule below to error in the future
-        '@typescript-eslint/require-await': 'warn',
-      },
-    },
-    {
-      name: 'typescript-js',
-      files: [GLOB_JS, GLOB_JSX],
-      rules: {
-        '@typescript-eslint/no-require-imports': 'off',
-        '@typescript-eslint/no-var-requires': 'off',
-      },
-    },
-    {
-      name: 'typescript-test',
-      files: [GLOB_TEST],
-      rules: {
-        '@typescript-eslint/no-unsafe-call': 'warn',
-        '@typescript-eslint/no-unsafe-return': 'warn',
-        '@typescript-eslint/no-unsafe-argument': 'warn',
-        '@typescript-eslint/no-unsafe-member-access': 'warn',
-        '@typescript-eslint/no-unsafe-assignment': 'warn',
-        '@typescript-eslint/no-explicit-any': 'off',
-        '@typescript-eslint/no-empty-function': 'off',
-      },
-    },
-  ]
-
-  /** @type {import('eslint').Linter.Config[]} */
-  // @ts-expect-error: unmatched type
-  const configTyped = config
-
-  return configTyped
-}

package/src/unicorn.js

@@ -1,75 +0,0 @@
-// @ts-check
-
-import plugin from 'eslint-plugin-unicorn'
-
-export function unicorn() {
-  /** @type {import('eslint').Linter.Config[]} */
-  const config = [
-    {
-      name: 'unicorn',
-      plugins: {
-        unicorn: plugin,
-      },
-      rules: {
-        // Pass error message when throwing errors
-        'unicorn/error-message': 'error',
-        // Uppercase regex escapes
-        'unicorn/escape-case': 'error',
-        // Array.isArray instead of instanceof etc
-        'unicorn/no-instanceof-builtins': 'error',
-        // Prevent deprecated `new Buffer()`
-        'unicorn/no-new-buffer': 'error',
-        // Keep regex literals safe!
-        'unicorn/no-unsafe-regex': 'off',
-        // Lowercase number formatting for octal, hex, binary (0x1'error' instead of 0X1'error')
-        'unicorn/number-literal-case': 'error',
-        // includes over indexOf when checking for existence
-        'unicorn/prefer-includes': 'error',
-        // String methods startsWith/endsWith instead of more complicated stuff
-        'unicorn/prefer-string-starts-ends-with': 'error',
-        // Enforce throwing type error when throwing error while checking typeof
-        'unicorn/prefer-type-error': 'error',
-        // Use new when throwing error
-        'unicorn/throw-new-error': 'error',
-        // Prefer using the node: protocol
-        'unicorn/prefer-node-protocol': 'error',
-        // Enforce explicitly comparing the length or size property of a value
-        'unicorn/explicit-length-check': 'error',
-        // Prefer `.flatMap(…)` over `.map(…).flat()`
-        'unicorn/prefer-array-index-of': 'error',
-        // Improve regexes
-        'unicorn/better-regex': 'error',
-        // Enforce combining multiple `Array#push()` into one call.
-        'unicorn/no-array-push-push': 'warn',
-        // Do not use a `for` loop that can be replaced with a `for-of` loop.
-        'unicorn/no-for-loop': 'error',
-        // Disallow eslint-disable comments without specific rule names
-        'unicorn/no-abusive-eslint-disable': 'error',
-        // Prefer `.addEventListener()` and `.removeEventListener()` over `on`-functions.
-        'unicorn/prefer-add-event-listener': 'error',
-        // Prefer `.find(…)` and `.findLast(…)` over the first or last element from
-        'unicorn/prefer-array-find': 'error',
-        // Prefer `.flatMap(…)` over `.map(…).flat()`.
-        'unicorn/prefer-array-flat-map': 'error',
-        // Prefer `.some(…)` over `.filter(…).length` check
-        'unicorn/prefer-array-some': 'error',
-        // Prefer `KeyboardEvent#key` over `KeyboardEvent#keyCode`
-        'unicorn/prefer-keyboard-event-key': 'error',
-        // Enforce the use of `Math.trunc` instead of bitwise operators.
-        'unicorn/prefer-math-trunc': 'error',
-        // Prefer negative index over `.length - index` when possible
-        'unicorn/prefer-negative-index': 'error',
-        // Prefer `Number` static properties over global ones.
-        'unicorn/prefer-number-properties': 'warn',
-        // Prefer `RegExp#test()` over `String#match()` and `RegExp#exec()`
-        'unicorn/prefer-regexp-test': 'warn',
-        // Prefer using `structuredClone` to create a deep clone
-        'unicorn/prefer-structured-clone': 'warn',
-        // Prefer using the `String.raw` tag to avoid escaping `\`
-        'unicorn/prefer-string-raw': 'warn',
-      },
-    },
-  ]
-
-  return config
-}

package/src/vue.js

@@ -1,46 +0,0 @@
-// @ts-check
-
-import prettierConfig from 'eslint-config-prettier'
-import vuePlugin from 'eslint-plugin-vue'
-import globals from 'globals'
-import tseslint from 'typescript-eslint'
-
-import { GLOB_VUE } from './shared.js'
-
-/** @type {import('eslint').Linter.Config[]} */
-const vueRecommended = vuePlugin.configs['flat/recommended']
-
-export function vue() {
-  /** @type {import('eslint').Linter.Config[]} */
-  const config = [
-    ...vueRecommended,
-    {
-      name: 'vue:language-options',
-      files: [GLOB_VUE],
-      languageOptions: {
-        ecmaVersion: 'latest',
-        sourceType: 'module',
-        globals: globals.browser,
-        parserOptions: {
-          parser: tseslint.parser,
-        },
-      },
-    },
-    {
-      name: 'vue:rules-override',
-      rules: {
-        ...Object.fromEntries(
-          Object.entries(prettierConfig.rules)
-            .filter(([key]) => key.startsWith('vue/'))
-            .map(([key, value]) => [key, value]),
-        ),
-
-        'vue/multi-word-component-names': 'off',
-        'vue/one-component-per-file': 'off',
-        'vue/require-prop-types': 'off',
-      },
-    },
-  ]
-
-  return config
-}