UNPKG

eslint-config-webpack

Version:

Provides Webpack's eslint rules as an extensible shared config

823 lines (611 loc) 21.1 kB
import { javascriptExtensions, typescriptExtensions, } from "./utils/extensions.js"; import getJsonFile from "./utils/get-json-file.js"; import isTypescriptInstalled from "./utils/is-typescript-installed.js"; const isTypescriptInstalledResult = isTypescriptInstalled(); /** * @returns {Promise<import("eslint").Linter.Config>} config */ async function getTypescriptJSDocRecommendedConfig() { if (!isTypescriptInstalledResult) { return { name: "typescript/please-install-typescript-to-enable-it" }; } const jsdocPlugin = (await import("eslint-plugin-jsdoc")).default; const jsdocConfig = jsdocPlugin.configs["flat/recommended-typescript-flavor-error"]; return { ...jsdocConfig, name: "typescript/jsdoc", files: [ `**/*.{${javascriptExtensions.map((item) => item.slice(1)).join(",")}}`, ], settings: { jsdoc: { mode: "typescript", // supported tags https://github.com/microsoft/TypeScript-wiki/blob/master/JSDoc-support-in-JavaScript.md tagNamePreference: { ...["memberof", "yields", "member"].reduce((acc, tag) => { acc[tag] = { message: `@${tag} currently not supported in TypeScript`, }; return acc; }, /** @type {Record<string, { message: string }>} */ ({})), extends: "extends", return: "returns", constructor: "constructor", prop: "property", property: "property", arg: "param", argument: "param", param: "param", augments: "extends", description: false, desc: false, inheritdoc: false, class: "constructor", returns: "returns", }, overrideReplacesDocs: false, }, }, rules: { ...jsdocConfig.rules, // From recommended // "jsdoc/check-access": "error", // From recommended // "jsdoc/check-alignment": "error", // No need // "jsdoc/check-examples": "error", "jsdoc/check-indentation": "error", "jsdoc/check-line-alignment": ["error", "never"], // From recommended // "jsdoc/check-param-names": "error", // From recommended // "jsdoc/check-property-names": "error", // No need // typescript does it // "jsdoc/check-syntax": "error", // Avoid conflict with jest special comment "jsdoc/check-tag-names": [ "error", { definedTags: ["jest-environment", "jest-environment-options"], }, ], // No need // "jsdoc/check-template-names": "error", // From recommended // "jsdoc/check-types": "error", // From recommended // "jsdoc/check-values": "error", // No need // "jsdoc/convert-to-jsdoc-comments": "error", // From recommended // "jsdoc/empty-tags": "error", // From recommended // "jsdoc/escape-inline-tags": "error", // "jsdoc/implements-on-classes": "error", // No need // "jsdoc/informative-docs": "error", // No need // "jsdoc/lines-before-block": "error", // No need // "jsdoc/match-description": "error", // No need // "jsdoc/match-name": "error", // From recommended // "jsdoc/multiline-blocks": "error", "jsdoc/no-bad-blocks": "error", "jsdoc/no-blank-block-descriptions": "error", "jsdoc/no-blank-blocks": "error", // From recommended // "jsdoc/no-defaults": "error", // No need // "jsdoc/no-missing-syntax": "error", // From recommended // "jsdoc/no-multi-asterisks": "error", "jsdoc/no-restricted-syntax": [ "error", { contexts: [ // Prefer TypeScript syntax for functions { comment: "JsdocBlock:has(JsdocTypeFunction[arrow=false])", message: "Please use TypeScript syntax - `(a: string, b: boolean) => number`", }, // Prefer `{string=}` over `{string} [arg]` { comment: "JsdocBlock:has(JsdocTag[tag=/^(property|param)$/][name=/[\\[\\]]/])", message: "Please use `@property {string=} property`/`@param {string=} arg` instead `[arg]` for optional properties and parameters", }, // No `?` type { comment: "JsdocBlock:has(JsdocTypeUnknown)", message: "Please use `unknown` or `any` (or `EXPECTED_ANY`) type", }, // No `Object` { comment: "JsdocBlock:has(JsdocTag[tag!=/^(typedef|template|param)$/]:has(JsdocTypeName[value=/^(Object|object)$/]))", message: "Please use provide types for object - `{ property: number:, result: () => number}` instead `Object`/`object` or use `EXPECTED_OBJECT` type", }, { comment: "JsdocBlock:has(JsdocTag[tag=typedef][parsedType.type!=JsdocTypeName]:has(JsdocTypeName[value=/^(Object|object)$/]))", message: "Please use provide types for object - `{ property: number:, result: () => number}` instead `Object`/`object` or use `EXPECTED_OBJECT` type", }, ], }, ], // No need // "jsdoc/no-types": "error", // No need // "jsdoc/no-undefined-types": "error", // TODO enable me in future // "jsdoc/prefer-import-tag": "error", "jsdoc/reject-any-type": "error", "jsdoc/reject-function-type": "error", "jsdoc/require-asterisk-prefix": "error", // No need // "jsdoc/require-description": "error", // No need // "jsdoc/require-description-complete-sentence": "error", // No need // "jsdoc/require-example": "error", // No need // "jsdoc/require-file-overview": "error", // No need "jsdoc/require-hyphen-before-param-description": ["error", "never"], // From recommended // "jsdoc/require-jsdoc": "error", // No need // "jsdoc/require-next-description":"error", // No need // "jsdoc/require-next-type": "error", // From recommended // "jsdoc/require-param": "error", // From recommended // "jsdoc/require-param-description": "error", // From recommended // "jsdoc/require-param-name": "error", // From recommended // "jsdoc/require-param-type": "error", // From recommended // "jsdoc/require-property": "error", // From recommended // "jsdoc/require-property-description": "error", // From recommended // "jsdoc/require-property-name": "error", // From recommended // "jsdoc/require-property-type": "error", // A lot of false positive with loops/`switch`/`if`/etc "jsdoc/require-returns-check": "off", // From recommended // "jsdoc/require-returns-description": "error", // From recommended // "jsdoc/require-returns-type": "error", // No need // "jsdoc/require-tags": "error", "jsdoc/require-template": "error", // No need // "jsdoc/require-template-description": "error", // No need // "jsdoc/require-throws": "error", // No need // "jsdoc/require-throws-description": "error", // No need // "jsdoc/require-throws-type": "error", // From recommended // "jsdoc/require-yields": "error", // No need // "jsdoc/require-yields-description": "error", // No need // "jsdoc/require-yields-type": "error", // From recommended // "jsdoc/require-yields-check": "error", // No need // "jsdoc/sort-tags": "error", // From recommended // "jsdoc/tag-lines": "error", // No need // "jsdoc/text-escaping": "error", "jsdoc/type-formatting": [ "error", { objectTypeBracketSpacing: " ", objectFieldSeparator: "comma-and-linebreak", objectFieldSeparatorTrailingPunctuation: true, trailingPunctuationMultilineOnly: true, }, ], // Doesn't support function overloading/tuples/`readonly`/module keyword/etc // Also `typescript` reports this itself "jsdoc/valid-types": "off", }, }; } /** * @returns {Promise<import("eslint").Linter.Config | import("eslint").Linter.Config[]>} config */ async function getTypescriptRecommendedConfig() { if (!isTypescriptInstalledResult) { return { name: "typescript/please-install-typescript-to-enable-it" }; } const tsconfigJson = getJsonFile("tsconfig.json"); const isNoEmitEnabled = (tsconfigJson && tsconfigJson.compilerOptions && tsconfigJson.compilerOptions.noEmit) || false; if (isNoEmitEnabled) { return { name: "typescript/no-emit-enabled" }; } const isStrict = (tsconfigJson && tsconfigJson.compilerOptions && tsconfigJson.compilerOptions.strict) || true; const typescriptPlugin = (await import("typescript-eslint")).default; const { configs } = typescriptPlugin; /** @type {import("eslint").Linter.Config} */ const baseConfig = configs.base; /** @type {import("eslint").Linter.Config} */ const eslintRecommendedConfig = configs.eslintRecommended; const recommendedConfig = /** @type {import("eslint").Linter.Config} */ ( configs.recommended.find( (item) => item.name === "typescript-eslint/recommended", ) ); const stylisticConfig = /** @type {import("eslint").Linter.Config} */ ( configs.stylistic.find( (item) => item.name === "typescript-eslint/stylistic", ) ); const allExtensions = [...typescriptExtensions, ...javascriptExtensions]; return [ { ...baseConfig, name: "typescript/recommended", files: [ `**/*.{${typescriptExtensions.map((item) => item.slice(1)).join(",")}}`, ], ignores: ["**/*.d.ts"], languageOptions: { parser: (baseConfig.languageOptions || {}).parser, }, plugins: { ...baseConfig.plugins, }, settings: { "import/extensions": allExtensions, "import/external-module-folders": [ "node_modules", "node_modules/@types", ], "import/parsers": { "@typescript-eslint/parser": typescriptExtensions, }, "import/resolver": { node: { extensions: allExtensions, }, }, }, rules: { ...eslintRecommendedConfig.rules, ...recommendedConfig.rules, ...stylisticConfig.rules, // From recommended // "@typescript-eslint/adjacent-overload-signatures": "error", // From recommended // "@typescript-eslint/array-type": "error", // No need // "@typescript-eslint/await-thenable": "error", // From recommended // "@typescript-eslint/ban-ts-comment": "error", // From recommended // "@typescript-eslint/ban-tslint-comment": "error", // From recommended // "@typescript-eslint/class-literal-property-style": "error", // No need // "@typescript-eslint/class-methods-use-this": "error", // From recommended // "@typescript-eslint/consistent-generic-constructors": "error", // From recommended // "@typescript-eslint/consistent-indexed-object-style": "error", // No need // "@typescript-eslint/consistent-return": "error", // From recommended // "@typescript-eslint/consistent-type-assertions": "error", // From recommended // "@typescript-eslint/consistent-type-definitions": "error", // No need // "@typescript-eslint/consistent-type-exports": "error", // No need // "@typescript-eslint/consistent-type-imports": "error", // The same as `default-param-last` "default-param-last": "off", "@typescript-eslint/default-param-last": "error", // No need // we have `dot-notation` // "@typescript-eslint/dot-notation": "error", // No need // "@typescript-eslint/explicit-function-return-type": "error", "@typescript-eslint/explicit-member-accessibility": [ "error", { accessibility: "no-public" }, ], // No need // "@typescript-eslint/explicit-module-boundary-types": "error", // No need // "@typescript-eslint/init-declarations": "error", // No need // "@typescript-eslint/max-params": "error", // No need // "@typescript-eslint/member-ordering": "error", // No need // "@typescript-eslint/method-signature-style": "error", // No need // "@typescript-eslint/naming-convention": "error", // From recommended // "@typescript-eslint/no-array-constructor": "error", // No need // "@typescript-eslint/no-array-delete": "error", // No need // "@typescript-eslint/no-base-to-string": "error", // From recommended // "@typescript-eslint/no-confusing-non-null-assertion": "error", // No need // "@typescript-eslint/no-confusing-void-expression": "error", // No need // Good rule, but some packages can change their API often, and it will create noise in CI // "@typescript-eslint/no-deprecated": "error", // No need // "@typescript-eslint/no-dupe-class-members": "error", // From recommended // "@typescript-eslint/no-duplicate-enum-values": "error", // No need // "@typescript-eslint/no-dynamic-delete": "error", // From recommended // "@typescript-eslint/no-empty-function": "error", // From recommended // "@typescript-eslint/no-empty-object-type": "error", // From recommended // "@typescript-eslint/no-explicit-any": "error", // From recommended // "@typescript-eslint/no-extra-non-null-assertion": "error", // No need // "@typescript-eslint/no-extraneous-class": "error", // No need // "@typescript-eslint/no-floating-promises": "error", // No need // "@typescript-eslint/no-for-in-array": "error", // No need // "@typescript-eslint/no-implied-eval": "error", // No need // "@typescript-eslint/no-import-type-side-effects": "error", // From recommended // "@typescript-eslint/no-inferrable-types": "error", // No need // "@typescript-eslint/no-invalid-this": "error", // No need // "@typescript-eslint/no-invalid-void-type": "error", // The same as `no-loop-func` "no-loop-func": "off", "@typescript-eslint/no-loop-func": "error", // No need // "@typescript-eslint/no-magic-numbers": "error", // No need // "@typescript-eslint/no-meaningless-void-operator": "error", // From recommended // "@typescript-eslint/no-misused-new": "error", // No need // "@typescript-eslint/no-misused-promises": "error", // No need // "@typescript-eslint/no-misused-spread": "error", // No need // "@typescript-eslint/no-mixed-enums": "error", // No need // "@typescript-eslint/no-namespace": "error", // No need // "@typescript-eslint/no-non-null-asserted-nullish-coalescing": "error", // From recommended // "@typescript-eslint/no-non-null-asserted-optional-chain": "error", // No need // "@typescript-eslint/no-non-null-assertion": "error", // No need // "@typescript-eslint/no-redeclare": "error", // No need // "@typescript-eslint/no-redundant-type-constituents": "error", // Module system provided in `node/module`/`node/commonjs`/etc configurations "@typescript-eslint/no-require-imports": "off", // No need // "@typescript-eslint/no-restricted-imports": "error", // No need // "@typescript-eslint/no-restricted-types": "error", // No need // "@typescript-eslint/no-shadow": "error", // From recommended "@typescript-eslint/no-this-alias": [ "error", { allowedNames: ["self"] }, ], // No need // "@typescript-eslint/no-unnecessary-boolean-literal-compare": "error", // "@typescript-eslint/no-unnecessary-condition": "error", "@typescript-eslint/no-unnecessary-parameter-property-assignment": "error", // No need // "@typescript-eslint/no-unnecessary-qualifier": "error", // No need // "@typescript-eslint/no-unnecessary-template-expression": "error", // No need // "@typescript-eslint/no-unnecessary-type-arguments": "error", // No need // "@typescript-eslint/no-unnecessary-type-assertion": "error", // From recommended // "@typescript-eslint/no-unnecessary-type-constraint": "error", // No need // "@typescript-eslint/no-unnecessary-type-conversion": "error", // No need // "@typescript-eslint/no-unnecessary-type-parameters": "error", // No need // "@typescript-eslint/no-unsafe-argument": "error", // No need // "@typescript-eslint/no-unsafe-assignment": "error", // No need // "@typescript-eslint/no-unsafe-call": "error", // From recommended // "@typescript-eslint/no-unsafe-declaration-merging": "error", // No need // "@typescript-eslint/no-unsafe-enum-comparison": "error", // From recommended // "@typescript-eslint/no-unsafe-function-type": "error", // No need // "@typescript-eslint/no-unsafe-member-access": "error", // No need // "@typescript-eslint/no-unsafe-return": "error", // No need // "@typescript-eslint/no-unsafe-type-assertion": "error", // No need // "@typescript-eslint/no-unsafe-unary-minus": "error", // From recommended // "@typescript-eslint/no-unused-expressions": "error", "no-unused-private-class-members": "off", "@typescript-eslint/no-unused-private-class-members": "error", // Provide better options "no-unused-vars": "off", "@typescript-eslint/no-unused-vars": [ "error", { args: "after-used", argsIgnorePattern: "^_", caughtErrors: "all", caughtErrorsIgnorePattern: "^_", destructuredArrayIgnorePattern: "^_", ignoreRestSiblings: true, ignoreClassWithStaticInitBlock: false, reportUsedIgnorePattern: false, }, ], // From recommended // "@typescript-eslint/no-unused-vars": "error", // The same as `no-use-before-define` "no-use-before-define": "off", "@typescript-eslint/no-use-before-define": [ "error", { functions: true, classes: true, variables: true, enums: true, typedefs: true, }, ], // No need // "@typescript-eslint/no-useless-constructor": "error", // No need // "@typescript-eslint/no-useless-default-assignment": "error", "@typescript-eslint/no-useless-empty-export": "error", // From recommended "@typescript-eslint/no-wrapper-object-types": "error", // No need // "@typescript-eslint/non-nullable-type-assertion-style": "error", // No need // "@typescript-eslint/only-throw-error": "error", // No need // "@typescript-eslint/parameter-properties": "error", // From recommended // "@typescript-eslint/prefer-as-const": "error", // No need // "@typescript-eslint/prefer-destructuring": "error", // No need // "@typescript-eslint/prefer-enum-initializers": "error", // No need // "@typescript-eslint/prefer-find": "error", // From recommended // "@typescript-eslint/prefer-for-of": "error", // From recommended // "@typescript-eslint/prefer-function-type": "error", // No need // "@typescript-eslint/prefer-includes": "error", // No need // "@typescript-eslint/prefer-literal-enum-member": "error", // From recommended // "@typescript-eslint/prefer-namespace-keyword": "error", // No need // "@typescript-eslint/prefer-nullish-coalescing": "error", // No need // "@typescript-eslint/prefer-optional-chain": "error", // No need // "@typescript-eslint/prefer-promise-reject-errors": "error", // No need // "@typescript-eslint/prefer-readonly": "error", // No need // "@typescript-eslint/prefer-readonly-parameter-types": "error", // No need // "@typescript-eslint/prefer-reduce-type-parameter": "error", // No need // "@typescript-eslint/prefer-regexp-exec": "error", // No need // "@typescript-eslint/prefer-return-this-type": "error", // No need // "@typescript-eslint/prefer-string-starts-ends-with": "error", // No need // "@typescript-eslint/promise-function-async": "error", // No need // "@typescript-eslint/related-getter-setter-pairs": "error", // No need // "@typescript-eslint/require-array-sort-compare": "error", // No need // "@typescript-eslint/require-await": "error", // No need // "@typescript-eslint/restrict-plus-operands": "error", // No need // "@typescript-eslint/restrict-template-expressions": "error", // No need // "@typescript-eslint/return-await": "error", // No need // "@typescript-eslint/strict-boolean-expressions": "error", // No need // "@typescript-eslint/switch-exhaustiveness-check": "error", // From recommended // "@typescript-eslint/triple-slash-reference": "error", // No need // "@typescript-eslint/unbound-method": "error", // No need // "@typescript-eslint/unified-signatures": "error", // No need // "use-unknown-in-catch-callback-variable": "error", // TypeScript compilation already ensures that named imports exist in the referenced module "import/named": "off", // TypeScript handles this for us "import/no-unresolved": "off", }, }, isStrict ? { files: [ `**/*.{${typescriptExtensions.map((item) => item.slice(1)).join(",")}}`, ], ignores: ["**/*.d.ts"], rules: { strict: "off" }, } : {}, ]; } export default { "typescript/jsdoc": await getTypescriptJSDocRecommendedConfig(), "typescript/recommended": await getTypescriptRecommendedConfig(), };