@aarongoldenthal/eslint-config-standard 42.0.1 → 43.0.0

package/README.md

@@ -5,58 +5,71 @@
 Custom ESLint configuration for all projects. Includes flat-config formatted
 configurations compatible with ESLint v9+ for the following:
 
-| Plugin Name                                       | Config name                                                                | Rule Prefix  |
-| ------------------------------------------------- | -------------------------------------------------------------------------- | ------------ |
-| `eslint`                                          | [`base-configs`](./base-configs.js)                                        | none, `core` |
-| `@eslint-community/eslint-plugin-eslint-comments` | [`eslint-comments-config`](./eslint-comments-config.js)                    | `comments`   |
-| `eslint-plugin-jest`                              | [`jest-config`](./jest-config.js)                                          | `jest`       |
-| `eslint-plugin-jsdoc`                             | [`jsdoc-config`](./jsdoc-config.js)                                        | `jsdoc`      |
-| `eslint-plugin-n`                                 | [`node-config`](./node-config.js)                                          | `node`       |
-| `eslint-plugin-playwright`                        | [`playwright-config`](./playwright-config.js)                              | `playwright` |
-| `eslint-plugin-promise`                           | [`promise-config`](./promise-config.js)                                    | `promise`    |
-| `eslint-plugin-unicorn`                           | [`unicorn-configs`](./unicorn-configs.js), [`esm-config`](./esm-config.js) | `unicorn`    |
-| `eslint-plugin-vitest`                            | [`vitest-config`](./vitest-config.js)                                      | `vitest`     |
+| Plugin Name                                       | Config name                                                                | Rule Prefix          |
+| ------------------------------------------------- | -------------------------------------------------------------------------- | -------------------- |
+| `eslint`                                          | [`base-configs`](./base-configs.js)                                        | none, `core`         |
+| `@eslint-community/eslint-plugin-eslint-comments` | [`eslint-comments-config`](./eslint-comments-config.js)                    | `comments`           |
+| `eslint-plugin-jest`                              | [`jest-config`](./jest-config.js)                                          | `jest`               |
+| `eslint-plugin-jsdoc`                             | [`jsdoc-configs`](./jsdoc-configs.js)                                      | `jsdoc`              |
+| `eslint-plugin-n`                                 | [`node-config`](./node-config.js)                                          | `node`               |
+| `eslint-plugin-playwright`                        | [`playwright-config`](./playwright-config.js)                              | `playwright`         |
+| `eslint-plugin-promise`                           | [`promise-config`](./promise-config.js)                                    | `promise`            |
+| `eslint-plugin-unicorn`                           | [`unicorn-configs`](./unicorn-configs.js), [`esm-config`](./esm-config.js) | `unicorn`            |
+| `eslint-plugin-vitest`                            | [`vitest-config`](./vitest-config.js)                                      | `vitest`             |
+| `typescript-eslint`                               | [`typescript-eslint-configs`](./typescript-eslint-configs.js)              | `@typescript-eslint` |
 
 As flat configs, the package defines all required plugins/configurations as
 `dependencies`. Since flat config allows flexibility in the rule prefixes (that
 is, they don't have to match the plugin name), the rules prefixes are adapted
-in some cases to be more intuitive or readable. Since they may be combined, and
-flat config doesn't allow nested arrays of rules, file names that are singular
-export a single config object (for example `jsdoc-config`), and file names
-that are plural export an array of config objects (for example `base-configs`).
+in some cases to be more intuitive or readable. Since they may be single
+configs or arrays of configs, file names that are singular export a single
+config object (for example `promise-config`) and file names that are plural
+export an array of config objects (for example `base-configs`).
 
 The `core` prefix has only one rule, `core/complexity`, that has a higher threshold.
 This provides a secondary check for cases where the lower threshold in the `complexity`
-rule is disabled, which otherwise allows unbounded complexity. Since re-use of core
+rule is turned off, which otherwise allows unbounded complexity. Since re-use of core
 rules is an experimental capability, this must be enabled with environment variable
 `ENABLE_ESLINT_CORE_RULE_DUPLICATES=true`.
 
-Most rule configurations are applicable to files matching `'**/*.{js,mjs,cjs}'`. The
-following configurations are exceptions and are applicable to files as noted:
+Most rule configurations are applicable to all JavaScript and TypeScript files
+(matching `'**/*.{js,mjs,cjs,ts,mts,cts}'`). The following configurations are
+exceptions and are applicable to files as noted:
 
-- `base-configs` sets `sourceType` to `commonjs` for `*.js`/`*.cjs` files and
-  `module` for `*.mjs` files.
+- `base-configs` sets `sourceType` to `commonjs` for `*.{js,cjs,ts,cts}` files and
+  `module` for `*.{mjs,mts}` files.
 - `base-configs` includes a config that disables some rules for test files
   matching any of the following test patterns (for example `max-lines`,
   `max-lines-per-function`).
 - `jest-config` applies rules to files matching
-  `'**/__tests__/**/*.{js,mjs,cjs}'` or `'**/?(*.)+(spec|test).{js,mjs,cjs}'`.
-- `vitest-config` applies rules to files matching
-  `'**/__tests__/**/*.{js,mjs}'` or `'**/?(*.)+(spec|test).{js,mjs}'`.
-- `playwright-config`: applies rules to files matching `'**/*.pwtest.{js,mjs,cjs}'`,
-  which differentiates them from Jest/Vitest test files.
+  `'**/__tests__/**/*.{js,mjs,cjs,ts,mts,cts}'` or
+  `'**/?(*.)+(spec|test).{js,mjs,cjs,ts,mts,cts}'`.
+- `vitest-config` applies rules to ESM files matching
+  `'**/__tests__/**/*.{js,mjs,ts,mts}'` or `'**/?(*.)+(spec|test).{js,mjs,ts,mts}'`.
+- `playwright-config`: applies rules to files matching
+  `'**/*.pwtest.{js,mjs,cjs,ts,mts,cts}'`, which differentiates them from
+  Jest/Vitest test files.
 - `unicorn-configs` includes a config that disables some rules for test files
   (matching any of the following test patterns), and applies some ESM-specific
-  rules only applicable to `*.mjs` files.
+  rules only applicable to `*.{mjs,mts}` files.
+- `typescript-eslint-configs` applies `typescript-eslint` rules to JavaScript
+  and all TypeScript files. Some of these rules extend core ESLint rules,
+  so those core ESLint rules are turned off in this config to avoid conflicts.
+  It also applies TypeScript-specific rules to TypeScript files only, and
+  configures the `@typescript-eslint/parser` set to use the closest
+  `tsconfig.json` file. For rules that are ESM/CJS specific, it assumes
+  `*.{js,ts}` files are ESM, which differs from `base-configs`. To ensure
+  proper rule overrides, this config should be applied after `base-configs`
+  and `esm-config`.
 
 With ESLint v9 the majority of formatting rules are deprecated and removed from
 `base-configs`, but the `eslint-config-prettier` package is included and can be
 added to the config if `prettier` is also being used to ensure it takes priority
-for formatting.
+for formatting, although it should be included last.
 
 There is also an `esm-config` included for projects using ES modules instead of
-CommonJS modules. This config sets `sourceType` to `module` for `*.js` files and
-includes `unicorn-config` ESM-specific rules.
+CommonJS modules. This config sets `sourceType` to `module` for `*.{js,ts}`
+files and includes `unicorn-configs` ESM-specific rules.
 
 ### Usage
 
@@ -64,10 +77,18 @@ This module is now pure ESM, so must be called from an ESM file, either
 `eslint.config.js` (if the project is ESM) or `eslint.config.mjs` (if the
 project is CJS).
 
-There is a `recommended` configuration for CJS with all plugin configurations
-enabled except `vitest-config` (it does include `jest-config`, and applies
-ESM-specific rules to `*.mjs files`). To configure `eslint.config.mjs` with
-this configuration:
+There are three versions of `recommended` configurations
+covering CommonJS, ESM, and TypeScript, and the individual configurations can
+be manually configured as well.
+
+#### `recommended` configuration
+
+The `recommended` configuration is for CommonJS with all plugin configurations
+enabled except `vitest-config` (it does include `jest-config`) and
+`typescript-eslint-configs`. It does apply ESM-specific rules to
+`*.{mjs,mts}` files. The rules are applicable to all JavaScript and TypeScript
+files, but don't require TypeScript (no rules with types). To configure
+`eslint.config.mjs` with this configuration:
 
 ```js
 import { defineConfig, globalIgnores } from 'eslint/config';
@@ -75,17 +96,21 @@ import recommendedConfig from '@aarongoldenthal/eslint-config-standard/recommend
 
 export default defineConfig([
   recommendedConfig,
-  globalIgnores(['.vscode/**', 'archive/**', 'node_modules/**', 'coverage/**'])
+  globalIgnores(['node_modules/**', 'coverage/**'])
 ]);
 ```
 
-Note the optional `ignores` config that can be added last to ignore certain
+Note the optional `globalIgnores` config that can be added last to ignore certain
 directories.
 
-There is also a `recommended-esm` configuration that's the same as the
-`recommended` config, but includes the `vitest-config` instead of the
-`jest-config`, and also the `esm-config` (applicable to `*.js` files).
-It can be configured with:
+#### `recommended-esm` configuration
+
+The `recommended-esm` configuration is the same as the `recommended` config,
+but for ESM. It includes the `vitest-config` instead of the
+`jest-config`, and also the `esm-config` (applicable to `*.{js,ts}` files),
+but doesn't include `typescript-eslint-configs`. The rules are also applicable
+to all JavaScript and TypeScript files, but don't require TypeScript (no rules
+with types). It can be configured with:
 
 ```js
 import { defineConfig, globalIgnores } from 'eslint/config';
@@ -93,13 +118,31 @@ import recommendedConfig from '@aarongoldenthal/eslint-config-standard/recommend
 
 export default defineConfig([
   recommendedConfig,
-  globalIgnores(['.vscode/**', 'archive/**', 'node_modules/**', 'coverage/**'])
+  globalIgnores(['node_modules/**', 'coverage/**'])
+]);
+```
+
+#### `recommended-ts` configuration
+
+The `recommended-ts` configuration is the same as the `recommended-esm` config,
+but also includes `typescript-eslint-configs`, and therefore requires
+TypeScript (an optional peer dependency) and a `tsconfig.json` file. It can
+be configured with:
+
+```js
+import { defineConfig, globalIgnores } from 'eslint/config';
+import recommendedConfig from '@aarongoldenthal/eslint-config-standard/recommended-ts';
+
+export default defineConfig([
+  recommendedConfig,
+  globalIgnores(['node_modules/**', 'coverage/**'])
 ]);
 ```
 
-To configure `eslint.config.js` with individual plugins, see the
-[`recommended`](./recommended.js) or
-[`recommended-esm`](./recommended-esm.js) configurations as examples.
+### Using individual configurations
+
+To configure `eslint.config.js` with individual plugins, see the `recommended*`
+configurations as examples.
 
 Notes:
 
@@ -107,7 +150,9 @@ Notes:
   except `esm-config` and `prettier`, so those settings take precedence.
 - The `jest-config` and `vitest-config` have the same file applicability, so
   only one should be used.
-- If used, the `esm-config` should be configured after all functional rules
-  to ensure the overridden settings take precedence.
+- If used, the `esm-config` should be configured after all functional
+  JavaScript rules to ensure the overridden settings take precedence.
+- If used, the `typescript-eslint-configs` should be configured after all
+  other rules to ensure the overridden settings take precedence.
 - If used, the `prettier` config should be included last to take priority in
   disabling the applicable rules from all other configurations.

package/base-configs.js

@@ -1,30 +1,51 @@
 /* eslint-disable max-lines -- required given number of rules */
+import { filePatterns, thresholds } from './settings.js';
 import { defineConfig } from 'eslint/config';
 
-const complexityThresholdLow = 5;
-const maxFunctionLinesThreshold = 30;
-const maxLinesThreshold = 300;
-const maxParametersThreshold = 4;
-const maxStatementsThreshold = 50;
-const nestedThreshold = 5;
+const eslintRulesCoveredByTsc = {
+    // All applicable rules listed for clarity.
+    // Those already configured are commented out.
+    'constructor-super': 'off', // Ts(2335) & ts(2377)
+    'getter-return': 'off', // Ts(2378)
+    'no-class-assign': 'off', // Ts(2629)
+    'no-const-assign': 'off', // Ts(2588)
+    'no-dupe-args': 'off', // Ts(2300)
+    'no-dupe-class-members': 'off', // Ts(2393) & ts(2300)
+    'no-dupe-keys': 'off', // Ts(1117)
+    'no-func-assign': 'off', // Ts(2630)
+    'no-import-assign': 'off', // Ts(2632) & ts(2540)
+    'no-new-native-nonconstructor': 'off', // Ts(7009)
+    'no-obj-calls': 'off', // Ts(2349)
+    'no-redeclare': 'off', // Ts(2451)
+    'no-setter-return': 'off', // Ts(2408)
+    'no-this-before-super': 'off', // Ts(2376) & ts(17009)
+    'no-undef': 'off', // Ts(2304) & ts(2552)
+    'no-unreachable': 'off', // Ts(7027)
+    'no-unsafe-negation': 'off', // Ts(2365) & ts(2322) & ts(2358)
+    // 'no-var': 'error', // Ts transpiles let/const to var, so no need for vars any more
+    'no-with': 'off' // Ts(1101) & ts(2410)
+    // 'prefer-const': 'error', // Ts provides better types with const
+    // 'prefer-rest-params': 'error', // Ts provides better types with rest args over arguments
+    // 'prefer-spread': 'error' // Ts transpiles spread to apply, so no need for manual apply
+};
 
 const configs = [
     {
-        files: ['**/*.{js,cjs}'],
+        files: [...filePatterns.baseJsTs, ...filePatterns.commonJsTs],
         languageOptions: {
             sourceType: 'commonjs'
         },
         name: 'CJS files'
     },
     {
-        files: ['**/*.mjs'],
+        files: [...filePatterns.esmJsTs],
         languageOptions: {
             sourceType: 'module'
         },
         name: 'ESM files'
     },
     {
-        files: ['**/*.{js,mjs,cjs}'],
+        files: [...filePatterns.allJs, ...filePatterns.allTs],
         linterOptions: {
             reportUnusedDisableDirectives: 'error',
             reportUnusedInlineConfigs: 'error'
@@ -104,7 +125,7 @@ const configs = [
             'class-methods-use-this': 'error',
             complexity: [
                 'error',
-                { max: complexityThresholdLow, variant: 'modified' }
+                { max: thresholds.complexityLow, variant: 'modified' }
             ],
             'consistent-return': 'error',
             // Disabled due to 'unicorn/no-this-assignment
@@ -130,19 +151,19 @@ const configs = [
             'init-declarations': 'off',
             'logical-assignment-operators': ['error', 'always'],
             'max-classes-per-file': ['error', 1],
-            'max-depth': ['error', { max: nestedThreshold }],
-            'max-lines': ['error', maxLinesThreshold],
+            'max-depth': ['error', { max: thresholds.nested }],
+            'max-lines': ['error', thresholds.maxLines],
             'max-lines-per-function': [
                 'error',
                 {
-                    max: maxFunctionLinesThreshold,
+                    max: thresholds.maxFunctionLines,
                     skipBlankLines: true,
                     skipComments: true
                 }
             ],
-            'max-nested-callbacks': ['error', { max: nestedThreshold }],
-            'max-params': ['error', { max: maxParametersThreshold }],
-            'max-statements': ['error', { max: maxStatementsThreshold }],
+            'max-nested-callbacks': ['error', { max: thresholds.nested }],
+            'max-params': ['error', { max: thresholds.maxParameters }],
+            'max-statements': ['error', { max: thresholds.maxStatements }],
             'new-cap': 'error',
             'no-alert': 'error',
             'no-array-constructor': 'error',
@@ -354,10 +375,11 @@ const configs = [
     {
         files: [
             // Patterns for jest-config, vitest-config
-            '**/__tests__/**/*.{js,mjs,cjs}',
-            '**/?(*.)+(spec|test).{js,mjs,cjs}',
+            ...filePatterns.testAllJs,
+            ...filePatterns.testAllTs,
             // Pattern for playwright-config
-            '**/*.pwtest.{js,mjs,cjs}'
+            ...filePatterns.playwrightJs,
+            ...filePatterns.playwrightTs
         ],
         name: 'eslint (test and pwtest files)',
         rules: {
@@ -370,7 +392,7 @@ const configs = [
             // Undefined is a common value to test no value
             'no-undefined': 'off',
             // Tests frequently check array elements, destructuring is
-            // les intuitive
+            // less intuitive
             'prefer-destructuring': 'off',
             // Regexes in tests frequently check for match patterns, but
             // groups are not captured
@@ -379,6 +401,13 @@ const configs = [
             // runner awaits implicitly
             'require-await': 'off'
         }
+    },
+    {
+        files: [...filePatterns.allTs],
+        name: 'eslint (rules covered by typescript)',
+        rules: {
+            ...eslintRulesCoveredByTsc
+        }
     }
 ];
 
@@ -386,9 +415,8 @@ if (process.env.ENABLE_ESLINT_CORE_RULE_DUPLICATES === 'true') {
     const { builtinRules } = await import('eslint/use-at-your-own-risk');
     const complexity = builtinRules.get('complexity');
 
-    const complexityThresholdHigh = 15;
     const coreDuplicateConfig = {
-        files: ['**/*.{js,mjs,cjs}'],
+        files: [...filePatterns.allJs, ...filePatterns.allTs],
         name: 'eslint core duplicates (all files)',
         plugins: {
             // Add plugin that duplicates the required rules
@@ -397,7 +425,7 @@ if (process.env.ENABLE_ESLINT_CORE_RULE_DUPLICATES === 'true') {
         rules: {
             'core/complexity': [
                 'error',
-                { max: complexityThresholdHigh, variant: 'modified' }
+                { max: thresholds.complexityHigh, variant: 'modified' }
             ]
         }
     };

package/eslint-comments-config.js

@@ -1,9 +1,10 @@
 import { defineConfig } from 'eslint/config';
 import eslintCommentsPlugin from '@eslint-community/eslint-plugin-eslint-comments';
+import { filePatterns } from './settings.js';
 
 // Create local variable due to vitest v4 coverage changes
 const config = defineConfig({
-    files: ['**/*.{js,mjs,cjs}'],
+    files: [...filePatterns.allJs, ...filePatterns.allTs],
     name: 'eslint-comments (all files)',
     plugins: { comments: eslintCommentsPlugin },
     rules: {

package/esm-config.js

@@ -1,13 +1,14 @@
 import { defineConfig } from 'eslint/config';
+import { filePatterns } from './settings.js';
 import unicornPlugin from 'eslint-plugin-unicorn';
 
 // Create local variable due to vitest v4 coverage changes
 const config = defineConfig({
-    files: ['**/*.js'],
+    files: [...filePatterns.baseJsTs],
     languageOptions: {
         sourceType: 'module'
     },
-    name: 'esm (js files)',
+    name: 'esm (js/ts files)',
     plugins: { unicorn: unicornPlugin },
     rules: {
         'unicorn/prefer-module': 'error',

package/jest-config.js

@@ -1,13 +1,11 @@
 import { defineConfig } from 'eslint/config';
+import { filePatterns } from './settings.js';
 import globals from 'globals';
 import jestPlugin from 'eslint-plugin-jest';
 
 // Create local variable due to vitest v4 coverage changes
 const config = defineConfig({
-    files: [
-        '**/__tests__/**/*.{js,mjs,cjs}',
-        '**/?(*.)+(spec|test).{js,mjs,cjs}'
-    ],
+    files: [...filePatterns.testAllJs, ...filePatterns.testAllTs],
     languageOptions: {
         globals: {
             ...globals.jest
@@ -31,6 +29,7 @@ const config = defineConfig({
         'jest/no-disabled-tests': 'error',
         'jest/no-done-callback': 'error',
         'jest/no-duplicate-hooks': 'error',
+        'jest/no-error-equal': 'off',
         'jest/no-export': 'error',
         'jest/no-focused-tests': 'error',
         'jest/no-hooks': 'off',
@@ -59,6 +58,8 @@ const config = defineConfig({
         'jest/no-standalone-expect': 'error',
         'jest/no-test-prefixes': 'error',
         'jest/no-test-return-statement': 'error',
+        'jest/no-unnecessary-assertion': 'off',
+        'jest/no-unneeded-async-expect-function': 'error',
         'jest/no-untyped-mock-factory': 'off',
         'jest/prefer-called-with': 'error',
         'jest/prefer-comparison-matcher': 'error',
@@ -73,6 +74,7 @@ const config = defineConfig({
         'jest/prefer-jest-mocked': 'error',
         'jest/prefer-lowercase-title': 'error',
         'jest/prefer-mock-promise-shorthand': 'error',
+        'jest/prefer-mock-return-shorthand': 'error',
         'jest/prefer-snapshot-hint': 'error',
         'jest/prefer-spy-on': 'error',
         'jest/prefer-strict-equal': 'error',
@@ -83,9 +85,11 @@ const config = defineConfig({
         'jest/require-hook': 'error',
         'jest/require-to-throw-message': 'error',
         'jest/require-top-level-describe': 'error',
+        'jest/unbound-method': 'off',
         'jest/valid-describe-callback': 'error',
         'jest/valid-expect': 'error',
         'jest/valid-expect-in-promise': 'error',
+        'jest/valid-expect-with-promise': 'off',
         'jest/valid-mock-module-path': 'error',
         'jest/valid-title': 'error'
     }

package/jsdoc-configs.js

@@ -0,0 +1,134 @@
+import { defineConfig } from 'eslint/config';
+import { filePatterns } from './settings.js';
+import jsdocPlugin from 'eslint-plugin-jsdoc';
+
+// Create local variable due to vitest v4 coverage changes
+const config = defineConfig(
+    {
+        files: [...filePatterns.allJs, ...filePatterns.allTs],
+        name: 'jsdoc (all files)',
+        plugins: { jsdoc: jsdocPlugin },
+        rules: {
+            'jsdoc/check-access': 'error', // Recommended
+            'jsdoc/check-alignment': 'error', // Recommended
+            'jsdoc/check-indentation': [
+                'error',
+                // Exclude default, as well as tags covered by check-line-alignment
+                {
+                    excludeTags: [
+                        'example',
+                        'param',
+                        'property',
+                        'returns',
+                        'throws',
+                        'typedef'
+                    ]
+                }
+            ],
+            'jsdoc/check-line-alignment': [
+                'error',
+                'always',
+                {
+                    tags: ['param', 'property', 'returns', 'throws', 'typedef']
+                }
+            ],
+            'jsdoc/check-param-names': 'error', // Recommended
+            'jsdoc/check-property-names': 'error', // Recommended
+            'jsdoc/check-syntax': 'error',
+            'jsdoc/check-tag-names': 'error', // Recommended
+            'jsdoc/check-template-names': 'error',
+            'jsdoc/check-types': 'error', // Recommended
+            'jsdoc/check-values': 'error', // Recommended
+            'jsdoc/convert-to-jsdoc-comments': 'off',
+            'jsdoc/empty-tags': 'error', // Recommended
+            'jsdoc/escape-inline-tags': 'error',
+            'jsdoc/implements-on-classes': 'error', // Recommended
+            'jsdoc/imports-as-dependencies': 'error',
+            'jsdoc/informative-docs': 'error',
+            'jsdoc/lines-before-block': 'off',
+            'jsdoc/match-description': 'error',
+            'jsdoc/multiline-blocks': 'error', // Recommended
+            'jsdoc/no-bad-blocks': 'error',
+            'jsdoc/no-blank-blocks': 'error',
+            'jsdoc/no-defaults': ['error', { contexts: ['any'] }],
+            'jsdoc/no-missing-syntax': 'off',
+            'jsdoc/no-multi-asterisks': 'error', // Recommended
+            'jsdoc/no-restricted-syntax': 'off',
+            'jsdoc/no-types': 'off',
+            'jsdoc/no-undefined-types': 'error', // Recommended
+            'jsdoc/prefer-import-tag': 'error',
+            'jsdoc/reject-any-type': 'error', // Recommended
+            'jsdoc/reject-function-type': 'error', // Recommended
+            'jsdoc/require-asterisk-prefix': 'error',
+            'jsdoc/require-description': 'error',
+            'jsdoc/require-description-complete-sentence': 'error',
+            'jsdoc/require-example': 'off',
+            'jsdoc/require-file-overview': 'off',
+            'jsdoc/require-hyphen-before-param-description': ['error', 'never'],
+            'jsdoc/require-jsdoc': [
+                'error',
+                {
+                    publicOnly: true,
+                    require: {
+                        ArrowFunctionExpression: true,
+                        ClassDeclaration: true,
+                        ClassExpression: true,
+                        FunctionDeclaration: true,
+                        FunctionExpression: true
+                    }
+                }
+            ], // Recommended
+            'jsdoc/require-next-description': 'error',
+            'jsdoc/require-next-type': 'error', // Recommended
+            'jsdoc/require-param': 'error', // Recommended
+            'jsdoc/require-param-description': 'error', // Recommended
+            'jsdoc/require-param-name': 'error', // Recommended
+            'jsdoc/require-param-type': 'error', // Recommended
+            'jsdoc/require-property': 'error', // Recommended
+            'jsdoc/require-property-description': 'error', // Recommended
+            'jsdoc/require-property-name': 'error', // Recommended
+            'jsdoc/require-property-type': 'error', // Recommended
+            'jsdoc/require-rejects': 'off',
+            'jsdoc/require-returns': 'error', // Recommended
+            'jsdoc/require-returns-check': 'error', // Recommended
+            'jsdoc/require-returns-description': 'error', // Recommended
+            'jsdoc/require-returns-type': 'error', // Recommended
+            'jsdoc/require-template': 'error',
+            'jsdoc/require-template-description': 'error',
+            'jsdoc/require-throws': 'error',
+            'jsdoc/require-throws-description': 'error',
+            'jsdoc/require-throws-type': 'error', // Recommended
+            'jsdoc/require-yields': 'error', // Recommended
+            'jsdoc/require-yields-check': 'error', // Recommended
+            'jsdoc/require-yields-description': 'error',
+            'jsdoc/require-yields-type': 'error', // Recommended
+            'jsdoc/sort-tags': 'error',
+            'jsdoc/tag-lines': [
+                // Recommended
+                'error',
+                'never',
+                { applyToEndTag: false, startLines: 1 }
+            ],
+            'jsdoc/ts-method-signature-style': 'off',
+            'jsdoc/ts-no-empty-object-type': 'off',
+            'jsdoc/ts-no-unnecessary-template-expression': 'off',
+            'jsdoc/ts-prefer-function-type': 'off',
+            'jsdoc/type-formatting': 'off',
+            'jsdoc/valid-types': 'error' // Recommended
+        }
+    },
+    {
+        files: [...filePatterns.allTs],
+        name: 'jsdoc (rules covered by typescript)',
+        rules: {
+            // Disable types in JSDoc since redundant with TS.
+            'jsdoc/no-types': 'error',
+            'jsdoc/require-param-type': 'off',
+            'jsdoc/require-property-type': 'off',
+            'jsdoc/require-returns-type': 'off',
+            'jsdoc/require-throws-type': 'off'
+        }
+    }
+);
+
+export default config;

package/node-config.js

@@ -1,10 +1,11 @@
 import { defineConfig } from 'eslint/config';
+import { filePatterns } from './settings.js';
 import globals from 'globals';
 import nodePlugin from 'eslint-plugin-n';
 
 // Create local variable due to vitest v4 coverage changes
 const config = defineConfig({
-    files: ['**/*.{js,mjs,cjs}'],
+    files: [...filePatterns.allJs, ...filePatterns.allTs],
     languageOptions: {
         globals: {
             ...globals.node
@@ -15,7 +16,14 @@ const config = defineConfig({
     rules: {
         'node/callback-return': 'error',
         'node/exports-style': 'error',
-        'node/file-extension-in-import': 'error',
+        // Disable for "*.ts" since it adds a ".js" extension to all imports,
+        // regardless of the file type. See
+        // https://github.com/eslint-community/eslint-plugin-n/issues/422.
+        'node/file-extension-in-import': [
+            'error',
+            'always',
+            { '.cts': 'never', '.mts': 'never', '.ts': 'never' }
+        ],
         'node/global-require': 'error',
         'node/handle-callback-err': 'error',
         'node/hashbang': 'error',