@timobechtel/style 1.14.0 → 2.1.0

package/eslint/rules/import.cjs

@@ -1,39 +0,0 @@
-// @ts-check
-const { defineConfig } = require('eslint-define-config');
-
-module.exports = defineConfig({
-  rules: {
-    // Disallow non-import statements appearing before import statements.
-    'import/first': 'error',
-    // Require a newline after the last import/require.
-    'import/newline-after-import': 'warn',
-    //  Disallow a module from importing itself.
-    'import/no-self-import': 'error',
-    // Ensures that there are no useless path segments.
-    'import/no-useless-path-segments': ['error'],
-    // Enforce a module import order convention.
-    'import/order': [
-      'warn',
-      {
-        groups: [
-          'builtin', // Node.js built-in modules
-          'external', // Packages
-          'internal', // Aliased modules
-          'parent', // Relative parent
-          'sibling', // Relative sibling
-          'index', // Relative index
-        ],
-      },
-    ],
-    // allow default exports
-    'import/no-default-export': 'off',
-    /**
-     * These are enabled by `import/recommended`, but are better handled by
-     * TypeScript and @typescript-eslint.
-     */
-    'import/default': 'off',
-    'import/export': 'off',
-    'import/namespace': 'off',
-    'import/no-unresolved': 'off',
-  },
-});

package/eslint/rules/react.cjs

@@ -1,90 +0,0 @@
-// @ts-check
-const { defineConfig } = require('eslint-define-config');
-
-module.exports = defineConfig({
-  rules: {
-    // We should prefer TypeScript over prop-types.
-    'react/prop-types': 'off',
-    // Disable requiring React to be imported, as this is no longer required.
-    'react/react-in-jsx-scope': 'off',
-    /**
-     * Require an explicit type when using button elements. Prevents common mistakes where `type="button"` is omitted on `<button>` elements.
-     */
-    'react/button-has-type': 'error',
-    /**
-     * Require consistent function type for function components.
-     */
-    'react/function-component-definition': 'warn',
-    /**
-     * Require destructuring and symmetric naming of `useState` hook value and setter variables.
-     */
-    'react/hook-use-state': 'warn',
-    /**
-     * Require consistent boolean attributes notation in JSX.
-     */
-    'react/jsx-boolean-value': 'warn',
-    /**
-     * Disallow unnecessary curly braces in JSX props and children.
-     */
-    'react/jsx-curly-brace-presence': [
-      'warn',
-      {
-        props: 'never',
-        children: 'ignore',
-        propElementValues: 'always',
-      },
-    ],
-    /**
-     * Require using shorthand form for React fragments, unless required.
-     */
-    'react/jsx-fragments': 'warn',
-    /**
-     * Prevent problematic leaked values from being rendered.
-     */
-    'react/jsx-no-leaked-render': 'error',
-    /**
-     * Prevents usage of unsafe `target='_blank'`.
-     *
-     * This rule is a part of `react/recommended`, but modified to
-     * enable allowReferrer.
-     */
-    'react/jsx-no-target-blank': [
-      'error',
-      {
-        allowReferrer: true,
-      },
-    ],
-    /**
-     * Disallow empty React fragments.
-     */
-    'react/jsx-no-useless-fragment': ['warn', { allowExpressions: true }],
-    /**
-     * Require the use of PascalCase for user-defined JSX components.
-     */
-    'react/jsx-pascal-case': 'warn',
-    /**
-     * Require props to be sorted alphabetically.
-     */
-    'react/jsx-sort-props': [
-      'warn',
-      {
-        // list callbacks after all other props
-        callbacksLast: true,
-      },
-    ],
-    /**
-     * Disallow creating unstable components inside components.
-     */
-    'react/no-unstable-nested-components': 'error',
-    /**
-     * Disallow closing tags for components without children.
-     */
-    'react/self-closing-comp': 'warn',
-    /**
-     * Enforce exhaustive dependencies in `useEffect` and `useCallback` hooks.
-     */
-    'react-hooks/exhaustive-deps': 'error',
-    // prefer destructuring props
-    'react/destructuring-assignment': ['warn', 'always'],
-  },
-});

package/eslint/rules/typescript.cjs

@@ -1,108 +0,0 @@
-const { defineConfig } = require('eslint-define-config');
-const noUnusedVarsConfig = require('./base.cjs').rules['no-unused-vars'];
-
-// @ts-check
-module.exports = defineConfig({
-  rules: {
-    // Require consistent usage of type exports.
-    '@typescript-eslint/consistent-type-exports': [
-      'error',
-      { fixMixedExportsWithInlineTypeSpecifier: true },
-    ],
-    // Require consistent usage of type imports.
-    '@typescript-eslint/consistent-type-imports': [
-      'error',
-      {
-        prefer: 'type-imports',
-        fixStyle: 'inline-type-imports',
-      },
-    ],
-    /**
-     * Require using function property types in method signatures.
-     * These have enhanced typechecking, whereas method signatures do not.
-     */
-    '@typescript-eslint/method-signature-style': 'error',
-    /**
-     * Require consistent naming conventions.
-     * Improves IntelliSense suggestions and avoids name collisions.
-     */
-    '@typescript-eslint/naming-convention': [
-      'error',
-      // Anything type-like should be written in PascalCase.
-      {
-        format: ['PascalCase'],
-        selector: ['typeLike', 'enumMember'],
-      },
-      // Interfaces cannot be prefixed with `I`, or have restricted names.
-      {
-        custom: {
-          match: false,
-          regex: '^I[A-Z]|^(Interface|Props|State)$',
-        },
-        format: ['PascalCase'],
-        selector: 'interface',
-      },
-    ],
-    // Disallow members of unions and intersections that do nothing or override type information.
-    '@typescript-eslint/no-redundant-type-constituents': 'error',
-    // Require using `RegExp.exec()` over `String.match()` for consistency.
-    '@typescript-eslint/prefer-regexp-exec': 'warn',
-    //
-    '@typescript-eslint/require-array-sort-compare': [
-      'error',
-      { ignoreStringArrays: true },
-    ],
-    /**
-     * Require exhaustive checks when using union types in switch statements.
-     * This ensures cases are considered when items are later added to a union.
-     */
-    '@typescript-eslint/switch-exhaustiveness-check': 'error',
-    // Require default parameters to be last.
-    'default-param-last': 'off',
-    '@typescript-eslint/default-param-last': 'error',
-    // Disallow creation of functions within loops.
-    'no-loop-func': 'off',
-    '@typescript-eslint/no-loop-func': 'error',
-    // Disallow unused variables.
-    'no-unused-vars': 'off',
-    '@typescript-eslint/no-unused-vars': noUnusedVarsConfig,
-
-    /**
-     * These are enabled by `import/recommended`, but are better handled by
-     * TypeScript and @typescript-eslint.
-     */
-    'import/default': 'off',
-    'import/export': 'off',
-    'import/namespace': 'off',
-    'import/no-unresolved': 'off',
-
-    // This is disabled as I feel that checking empty strings is a valid use
-    // of `||` over `??`.
-    '@typescript-eslint/prefer-nullish-coalescing': 'off',
-
-    // Disallow Promises in places not designed to handle them.
-    '@typescript-eslint/no-misused-promises': [
-      'error',
-      {
-        // Disabled as I feel that passing a async callback to a function expecting
-        // a void callback is a valid use case.
-        // e.g. fn.on('event', async () => {})
-        // Strictly requiring to return 'undefined' does not have a functional benefit,
-        // but makes the code more verbose.
-        checksVoidReturn: false,
-      },
-    ],
-    // While I'd prefer using types over interfaces, there are a lot of cases
-    // where interfaces are needed, e.g. merging declarations and this rule is too strict.
-    '@typescript-eslint/consistent-type-definitions': 'off',
-    // while using property signatures provides better typechecking,
-    // method signatures provide better hinting in the editor.
-    // (different color for methods vs properties)
-    '@typescript-eslint/method-signature-style': 'off',
-    // There are cases where using the index signature is more descriptive
-    // as the index can be named
-    '@typescript-eslint/consistent-indexed-object-style': 'off',
-    // Enforce using a top-level type qualifier for imports when only types are imported using the inline type qualifier
-    '@typescript-eslint/no-import-type-side-effects': 'error',
-  },
-});

package/eslint/rules/unicorn.cjs

@@ -1,129 +0,0 @@
-// @ts-check
-const { defineConfig } = require('eslint-define-config');
-
-module.exports = defineConfig({
-  plugins: ['unicorn'],
-  rules: {
-    // Use destructured variables over properties.
-    'unicorn/consistent-destructuring': 'warn',
-    // Move function definitions to the highest possible scope.
-    'unicorn/consistent-function-scoping': [
-      'warn',
-      {
-        checkArrowFunctions: false,
-      },
-    ],
-    //  Enforce passing a message value when creating a built-in error.
-    'unicorn/error-message': 'error',
-    //  Require escape sequences to use uppercase values.
-    'unicorn/escape-case': 'warn',
-    // Disallow empty files.
-    'unicorn/no-empty-file': 'error',
-    // Do not use a for loop that can be replaced with a for-of loop.
-    'unicorn/no-for-loop': 'warn',
-    // Require Array.isArray() instead of instanceof Array.
-    'unicorn/no-instanceof-array': 'error',
-    // Prevent calling EventTarget#removeEventListener() with the result of an expression.
-    'unicorn/no-invalid-remove-event-listener': 'error',
-    // Disallow the use of objects as default parameters.
-    'unicorn/no-object-as-default-parameter': 'error',
-    // Disallow then property.
-    'unicorn/no-thenable': 'error',
-    //  Disallow awaiting non-promise values.
-    'unicorn/no-unnecessary-await': 'error',
-    // Disallow unreadable IIFEs.
-    'unicorn/no-unreadable-iife': 'warn',
-    // Disallow useless fallback when spreading in object literals.
-    'unicorn/no-useless-fallback-in-spread': 'warn',
-    // Disallow useless array length check.
-    'unicorn/no-useless-length-check': 'warn',
-    // Disallow returning/yielding Promise.resolve/reject() in async functions or promise callbacks
-    'unicorn/no-useless-promise-resolve-reject': 'error',
-    // Disallow unnecessary spread.
-    'unicorn/no-useless-spread': 'error',
-    // Disallow number literals with zero fractions or dangling dots.
-    'unicorn/no-zero-fractions': 'warn',
-    // Enforce the style of numeric separators by correctly grouping digits.
-    'unicorn/numeric-separators-style': '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 .filter(…).
-    'unicorn/prefer-array-find': 'error',
-    // Prefer Array#flat() over legacy techniques to flatten arrays.
-    'unicorn/prefer-array-flat': 'error',
-    // Prefer .flatMap(…) over .map(…).flat().
-    'unicorn/prefer-array-flat-map': 'error',
-    // Prefer Array#{indexOf,lastIndexOf}() over Array#{findIndex,findLastIndex}() when looking for the index of an item.
-    'unicorn/prefer-array-index-of': 'error',
-    // Prefer .some(…) over .filter(…).length check and .{find,findLast}(…).
-    'unicorn/prefer-array-some': 'error',
-    // Prefer .at() method for index access and String#charAt().
-    'unicorn/prefer-at': 'error',
-    // Prefer Blob#arrayBuffer() over FileReader#readAsArrayBuffer(…) and Blob#text() over FileReader#readAsText(…).
-    'unicorn/prefer-blob-reading-methods': 'error',
-    // Prefer Date.now() to get the number of milliseconds since the Unix Epoch.
-    'unicorn/prefer-date-now': 'error',
-    // Prefer default parameters over reassignment.
-    'unicorn/prefer-default-parameters': 'warn',
-    // Prefer Node#append() over Node#appendChild().
-    'unicorn/prefer-dom-node-append': 'error',
-    // Prefer using .dataset on DOM elements over calling attribute methods.
-    'unicorn/prefer-dom-node-dataset': 'error',
-    // Prefer childNode.remove() over parentNode.removeChild(childNode).
-    'unicorn/prefer-dom-node-remove': 'error',
-    // Prefer EventTarget over EventEmitter.
-    'unicorn/prefer-event-target': 'warn',
-    // Prefer export…from when re-exporting.
-    'unicorn/prefer-export-from': [
-      'warn',
-      {
-        ignoreUsedVariables: true,
-      },
-    ],
-    // Prefer .includes() over .indexOf() and Array#some() when checking for existence or non-existence.
-    'unicorn/prefer-includes': 'error',
-    // Prefer KeyboardEvent#key over KeyboardEvent#keyCode.
-    'unicorn/prefer-keyboard-event-key': 'error',
-    // Prefer using a logical operator over a ternary.
-    'unicorn/prefer-logical-operator-over-ternary': 'error',
-    // Enforce the use of Math.trunc instead of bitwise operators.
-    'unicorn/prefer-math-trunc': 'error',
-    // Prefer .before() over .insertBefore(), .replaceWith() over .replaceChild(), prefer one of .before(), .after(), .append() or .prepend() over insertAdjacentText() and insertAdjacentElement().
-    'unicorn/prefer-modern-dom-apis': 'error',
-    // Prefer modern Math APIs over legacy patterns.
-    'unicorn/prefer-modern-math-apis': 'error',
-    // Prefer negative index over .length - index when possible.
-    'unicorn/prefer-negative-index': 'error',
-    // Require using the `node:` protocol when importing Node.js built-in modules.
-    'unicorn/prefer-node-protocol': 'error',
-    // Prefer Number static properties over global ones.
-    'unicorn/prefer-number-properties': ['error', { checkInfinity: false }],
-    // Prefer using Object.fromEntries(…) to transform a list of key-value pairs into an object.
-    'unicorn/prefer-object-from-entries': 'error',
-    // Prefer Reflect.apply() over Function#apply().
-    'unicorn/prefer-reflect-apply': 'error',
-    // using RegExp.test() is faster than string.match()
-    // note: you should not use the global flag /g with RegExp.test() though!
-    'unicorn/prefer-regexp-test': 'error',
-    // Prefer Set#has() over Array#includes() when checking for existence or non-existence.
-    'unicorn/prefer-set-has': 'error',
-    // Prefer using Set#size instead of Array#length.
-    'unicorn/prefer-set-size': 'error',
-    // Prefer the spread operator over Array.from().
-    'unicorn/prefer-spread': 'error',
-    // Prefer String#replaceAll() over regex searches with the global flag.
-    'unicorn/prefer-string-replace-all': 'error',
-    // Prefer String#slice() over String#substr() and String#substring().
-    'unicorn/prefer-string-slice': 'error',
-    // Prefer String#startsWith() & String#endsWith() over RegExp#test().
-    'unicorn/prefer-string-starts-ends-with': 'error',
-    // Prefer ternary expressions over simple if-else statements.
-    'unicorn/prefer-ternary': ['warn', 'only-single-line'],
-    // Prefer top-level await over top-level promises and async function calls.
-    'unicorn/prefer-top-level-await': 'error',
-    // Enforce using the separator argument with Array#join().
-    'unicorn/require-array-join-separator': 'error',
-    // Enforce consistent brace style for case clauses.
-    'unicorn/switch-case-braces': 'error',
-  },
-});