@remcohaszing/eslint 12.0.1 → 14.0.0

package/README.md

@@ -1,11 +1,9 @@
 # @remcohaszing/eslint
 
-> A strict ESLint configuration
-
 [![build status](https://github.com/remcohaszing/eslint/workflows/ci/badge.svg)](https://github.com/remcohaszing/eslint/actions)
 [![codecov](https://codecov.io/gh/remcohaszing/eslint/branch/main/graph/badge.svg)](https://codecov.io/gh/remcohaszing/eslint)
-[![npm](https://img.shields.io/npm/v/@remcohaszing/eslint)](https://www.npmjs.com/package/@remcohaszing/eslint)
-[![prettier](https://img.shields.io/badge/code_style-prettier-ff69b4.svg)](https://prettier.io)
+[![npm version](https://img.shields.io/npm/v/@remcohaszing/eslint)](https://www.npmjs.com/package/@remcohaszing/eslint)
+[![npm downloads](https://img.shields.io/npm/dm/@remcohaszing/eslint)](https://www.npmjs.com/package/@remcohaszing/eslint)
 
 This is a strict shareable ESLint configuration. All ESLint core rules and a variety of ESLint
 plugins have been carefully considered. Overrides are used to apply rules based on context.
@@ -24,42 +22,13 @@ plugins have been carefully considered. Overrides are used to apply rules based
 
 ## Installation
 
-Install `@remcohaszing/eslint` using `npm`.
-
 ```sh
-npm install --save-dev \
-  eslint \
-  @remcohaszing/eslint \
-  prettier
+npm install --save-dev @remcohaszing/eslint
 ```
 
 ## Configuration
 
-First [configure Prettier](https://prettier.io/docs/en/configuration.html). For example, create a
-`.editorconfig` with the following content:
-
-```ini
-root = true
-
-[*]
-charset = utf-8
-end_of_line = lf
-indent_size = 2
-indent_style = space
-insert_final_newline = true
-max_line_length = 100
-trim_trailing_whitespace = true
-```
-
-And create a `.prettierrc.yaml` with the following content:
-
-```yaml
-proseWrap: always
-singleQuote: true
-trailingComma: all
-```
-
-Next, create `eslint.config.js` with the following content:
+Create `eslint.config.js` with the following content:
 
 ```js
 export { default } from '@remcohaszing/eslint'
@@ -85,8 +54,6 @@ Code blocks in Markdown are linted by default using
 [`@eslint/markdown`](https://github.com/eslint/markdown). The type checking rules from
 `@typescript-eslint/eslint-plugin` don’t work with markdown code blocks.
 
-`prettier/prettier` is disabled, because it doesn’t play nice with `eslint-plugin-markdown`.
-
 Some other rules have been turned off, because these rules may conflict with the purpose of the
 documentation.
 
@@ -126,7 +93,7 @@ eslint
 
 ## Compatibility
 
-This project requires Node.js 22 or greater.
+This project requires Node.js 24 or greater.
 
 ## License
 

package/lib/config.js

@@ -6,7 +6,6 @@ import { dirname, join } from 'node:path'
 
 import { includeIgnoreFile } from '@eslint/compat'
 import markdown from '@eslint/markdown'
-// @ts-expect-error
 import eslintCommunityEslintComments from '@eslint-community/eslint-plugin-eslint-comments'
 import stylistic from '@stylistic/eslint-plugin'
 import typescriptEslint from '@typescript-eslint/eslint-plugin'
@@ -14,13 +13,10 @@ import typescriptEslintParser from '@typescript-eslint/parser'
 import confusingBrowserGlobals from 'confusing-browser-globals'
 import { defineConfig } from 'eslint/config'
 import importX from 'eslint-plugin-import-x'
-// @ts-expect-error
-import jestFormatting from 'eslint-plugin-jest-formatting'
 import jsdoc from 'eslint-plugin-jsdoc'
 import n from 'eslint-plugin-n'
 import perfectionist from 'eslint-plugin-perfectionist'
 import { Alphabet } from 'eslint-plugin-perfectionist/alphabet'
-import prettier from 'eslint-plugin-prettier'
 import unicorn from 'eslint-plugin-unicorn'
 import findUp from 'find-up'
 import globals from 'globals'
@@ -125,6 +121,8 @@ const alphabet = unshiftChar(unshiftChar(characters, '.'), '/')
  */
 const NON_COMPONENT_FUNCTION = 'FunctionDeclaration[id.name=/^[a-z]/]'
 const METHOD_DEFINITION = 'MethodDefinition'
+const TEST_SELECTOR =
+  'ExpressionStatement[expression.callee.name=/^(after|afterAll|afterEach|before|beforeAll|beforeEach|describe|it|suite|test)$/]'
 
 const config = defineConfig([
   getIgnorePatterns(),
@@ -154,13 +152,10 @@ const config = defineConfig([
       // @ts-expect-error
       '@typescript-eslint': typescriptEslint,
       '@eslint-community/eslint-comments': eslintCommunityEslintComments,
-      // @ts-expect-error
       'import-x': importX,
-      'jest-formatting': jestFormatting,
       jsdoc,
       n,
       perfectionist,
-      prettier,
       unicorn
     },
     settings: {
@@ -363,10 +358,6 @@ const config = defineConfig([
           selector:
             'CallExpression[callee.property.name="toString"][callee.optional=false][arguments.length=0][optional=false]',
           message: 'Use String() instead.'
-        },
-        {
-          selector: '[returnType.type="TSTypeAnnotation"]>TSTypeAnnotation>TSVoidKeyword',
-          message: 'Use undefined for non-returning functions of unknown for callbacks'
         }
       ],
       'no-return-assign': ['error', 'always'],
@@ -428,6 +419,7 @@ const config = defineConfig([
       'prefer-rest-params': 'error',
       'prefer-spread': 'error',
       'prefer-template': 'error',
+      'preserve-caught-error': ['error', { requireCatchParameter: true }],
       radix: ['error', 'as-needed'],
       'require-await': 'error',
       'require-yield': 'error',
@@ -442,7 +434,6 @@ const config = defineConfig([
       '@eslint-community/eslint-comments/no-aggregating-enable': 'error',
       '@eslint-community/eslint-comments/no-duplicate-disable': 'error',
       '@eslint-community/eslint-comments/no-unlimited-disable': 'error',
-      '@eslint-community/eslint-comments/no-unused-disable': 'error',
       '@eslint-community/eslint-comments/no-unused-enable': 'error',
       '@eslint-community/eslint-comments/no-use': [
         'error',
@@ -496,13 +487,13 @@ const config = defineConfig([
 
         {
           blankLine: 'always',
-          prev: 'function',
+          prev: ['function', { selector: TEST_SELECTOR }],
           next: '*'
         },
         {
           blankLine: 'always',
           prev: '*',
-          next: 'function'
+          next: ['function', { selector: TEST_SELECTOR }]
         }
       ],
       '@stylistic/quotes': ['error', 'single', { avoidEscape: true, ignoreStringLiterals: true }],
@@ -558,14 +549,6 @@ const config = defineConfig([
       ],
       'import-x/no-useless-path-segments': 'error',
 
-      // https://github.com/dangreenisrael/eslint-plugin-jest-formatting
-      'jest-formatting/padding-around-after-all-blocks': 'error',
-      'jest-formatting/padding-around-after-each-blocks': 'error',
-      'jest-formatting/padding-around-before-all-blocks': 'error',
-      'jest-formatting/padding-around-before-each-blocks': 'error',
-      'jest-formatting/padding-around-describe-blocks': 'error',
-      'jest-formatting/padding-around-test-blocks': 'error',
-
       // https://github.com/gajus/eslint-plugin-jsdoc
       'jsdoc/check-access': 'error',
       'jsdoc/check-indentation': [
@@ -577,6 +560,7 @@ const config = defineConfig([
       'jsdoc/check-tag-names': ['error', { jsxTags: true }],
       'jsdoc/check-template-names': 'error',
       'jsdoc/empty-tags': 'error',
+      'jsdoc/escape-inline-tags': 'error',
       'jsdoc/match-name': [
         'error',
         {
@@ -638,6 +622,9 @@ const config = defineConfig([
         }
       ],
       'jsdoc/no-undefined-types': 'error',
+      'jsdoc/prefer-import-tag': ['error', { exemptTypedefs: false, outputType: 'named-import' }],
+      'jsdoc/reject-any-type': 'error',
+      'jsdoc/reject-function-type': 'error',
       'jsdoc/require-asterisk-prefix': 'error',
       'jsdoc/require-description': 'error',
       'jsdoc/require-hyphen-before-param-description': ['error', 'never'],
@@ -669,8 +656,14 @@ const config = defineConfig([
       'jsdoc/require-returns-check': 'error',
       'jsdoc/require-returns-description': 'error',
       'jsdoc/require-returns-type': 'error',
+      'jsdoc/require-template-description': 'error',
+      'jsdoc/require-throws-description': 'error',
+      'jsdoc/require-throws-type': 'error',
       'jsdoc/sort-tags': ['error', { alphabetizeExtras: true }],
       'jsdoc/tag-lines': ['error', 'never', { applyToEndTag: false, startLines: 1, endLines: 0 }],
+      'jsdoc/ts-method-signature-style': 'error',
+      'jsdoc/ts-no-unnecessary-template-expression': 'error',
+      'jsdoc/ts-prefer-function-type': 'error',
 
       // https://github.com/eslint-community/eslint-plugin-n
       'n/callback-return': ['error', ['callback', 'cb']],
@@ -684,9 +677,11 @@ const config = defineConfig([
       'n/no-unpublished-bin': 'error',
       'n/prefer-global/buffer': 'error',
       'n/prefer-global/console': 'error',
+      'n/prefer-global/crypto': 'error',
       'n/prefer-global/process': 'error',
       'n/prefer-global/text-decoder': 'error',
       'n/prefer-global/text-encoder': 'error',
+      'n/prefer-global/timers': 'error',
       'n/prefer-global/url': 'error',
       'n/prefer-global/url-search-params': 'error',
       'n/prefer-promises/dns': 'error',
@@ -696,8 +691,10 @@ const config = defineConfig([
       // https://perfectionist.dev
       'perfectionist/sort-array-includes': 'error',
       'perfectionist/sort-classes': ['error', { type: 'unsorted' }],
+      'perfectionist/sort-export-attributes': 'error',
       'perfectionist/sort-exports': ['error', { type: 'custom', alphabet }],
       'perfectionist/sort-heritage-clauses': 'error',
+      'perfectionist/sort-import-attributes': 'error',
       'perfectionist/sort-imports': [
         'error',
         {
@@ -711,17 +708,16 @@ const config = defineConfig([
             'type-subpath',
             'type-internal',
             'type-parent',
-            { newlinesBetween: 'never' },
+            { newlinesBetween: 0 },
             ['type-sibling', 'type-index'],
             'builtin',
             'external',
             'subpath',
             'internal',
             'parent',
-            { newlinesBetween: 'never' },
+            { newlinesBetween: 0 },
             ['sibling', 'index'],
-            'unknown',
-            'object'
+            'unknown'
           ]
         }
       ],
@@ -736,20 +732,21 @@ const config = defineConfig([
       'perfectionist/sort-maps': 'error',
       'perfectionist/sort-named-exports': 'error',
       'perfectionist/sort-named-imports': 'error',
-      'perfectionist/sort-objects': ['error', { objectDeclarations: false }],
+      'perfectionist/sort-objects': [
+        'error',
+        { type: 'unsorted', useConfigurationIf: { objectType: 'non-destructured' } }
+      ],
       'perfectionist/sort-union-types': 'error',
 
-      // https://github.com/prettier/eslint-plugin-prettier
-      'prettier/prettier': 'error',
-
       // https://github.com/sindresorhus/eslint-plugin-unicorn
       'unicorn/better-regex': 'error',
-      'unicorn/catch-error-name': ['error', { name: 'error', ignore: ['err', /Error^/] }],
+      'unicorn/catch-error-name': ['error', { name: 'error', ignore: ['cause', 'err', /Error$/] }],
       'unicorn/consistent-assert': 'error',
       'unicorn/consistent-date-clone': 'error',
       'unicorn/consistent-destructuring': 'error',
       'unicorn/consistent-empty-array-spread': 'error',
       'unicorn/consistent-function-scoping': 'error',
+      'unicorn/consistent-template-literal-escape': 'error',
       'unicorn/custom-error-definition': 'error',
       'unicorn/error-message': 'error',
       'unicorn/escape-case': 'error',
@@ -783,12 +780,14 @@ const config = defineConfig([
       'unicorn/no-array-for-each': 'error',
       'unicorn/no-array-method-this-argument': 'error',
       'unicorn/no-array-reduce': 'error',
+      'unicorn/no-array-sort': 'error',
       'unicorn/no-await-in-promise-methods': 'error',
       'unicorn/no-console-spaces': 'error',
       'unicorn/no-document-cookie': 'error',
       'unicorn/no-empty-file': 'error',
       'unicorn/no-for-loop': 'error',
       'unicorn/no-hex-escape': 'error',
+      'unicorn/no-immediate-mutation': 'error',
       'unicorn/no-instanceof-builtins': 'error',
       'unicorn/no-invalid-fetch-options': 'error',
       'unicorn/no-invalid-remove-event-listener': 'error',
@@ -808,8 +807,10 @@ const config = defineConfig([
       'unicorn/no-unnecessary-slice-end': 'error',
       'unicorn/no-unreadable-iife': 'error',
       'unicorn/no-unused-properties': 'error',
+      'unicorn/no-useless-collection-argument': 'error',
       'unicorn/no-useless-error-capture-stack-trace': 'error',
       'unicorn/no-useless-fallback-in-spread': 'error',
+      'unicorn/no-useless-iterator-to-array': 'error',
       'unicorn/no-useless-length-check': 'error',
       'unicorn/no-useless-promise-resolve-reject': 'error',
       'unicorn/no-useless-spread': 'error',
@@ -823,8 +824,10 @@ const config = defineConfig([
       'unicorn/prefer-array-index-of': 'error',
       'unicorn/prefer-array-some': 'error',
       'unicorn/prefer-at': 'error',
+      'unicorn/prefer-bigint-literals': 'error',
       'unicorn/prefer-blob-reading-methods': 'error',
       'unicorn/prefer-class-fields': 'error',
+      'unicorn/prefer-classlist-toggle': 'error',
       'unicorn/prefer-date-now': 'error',
       'unicorn/prefer-default-parameters': 'error',
       'unicorn/prefer-dom-node-append': 'error',
@@ -852,6 +855,7 @@ const config = defineConfig([
       'unicorn/prefer-prototype-methods': 'error',
       'unicorn/prefer-reflect-apply': 'error',
       'unicorn/prefer-regexp-test': 'error',
+      'unicorn/prefer-response-static-json': 'error',
       'unicorn/prefer-set-has': 'error',
       'unicorn/prefer-set-size': 'error',
       'unicorn/prefer-single-call': 'error',
@@ -864,9 +868,11 @@ const config = defineConfig([
       'unicorn/prefer-top-level-await': 'error',
       'unicorn/prefer-type-error': 'error',
       'unicorn/relative-url-style': 'error',
+      'unicorn/require-module-attributes': 'error',
       'unicorn/require-module-specifiers': 'error',
       'unicorn/require-post-message-target-origin': 'error',
       'unicorn/switch-case-braces': ['error', 'avoid'],
+      'unicorn/switch-case-break-position': 'error',
       'unicorn/template-indent': 'error',
       'unicorn/text-encoding-identifier-case': 'error',
       'unicorn/throw-new-error': 'error'
@@ -1124,8 +1130,6 @@ const config = defineConfig([
       'n/no-unpublished-import': 'off',
       'n/no-unpublished-require': 'off',
 
-      'prettier/prettier': 'off',
-
       'unicorn/consistent-function-scoping': 'off'
     }
   }
@@ -1174,6 +1178,7 @@ export const typechecking = defineConfig([
       '@typescript-eslint/no-unsafe-member-access': 'error',
       '@typescript-eslint/no-unsafe-return': 'error',
       '@typescript-eslint/no-unsafe-unary-minus': 'error',
+      '@typescript-eslint/no-useless-default-assignment': 'error',
       '@typescript-eslint/non-nullable-type-assertion-style': 'error',
       '@typescript-eslint/only-throw-error': 'error',
       '@typescript-eslint/prefer-destructuring': [

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@remcohaszing/eslint",
-  "version": "12.0.1",
+  "version": "14.0.0",
   "description": "A strict ESLint configuration.",
   "keywords": [
     "eslint",
@@ -30,31 +30,29 @@
   },
   "dependencies": {
     "@eslint-community/eslint-plugin-eslint-comments": "^4.0.0",
-    "@eslint/compat": "^1.0.0",
-    "@eslint/markdown": "^7.0.0",
+    "@eslint/compat": "^2.0.0",
+    "@eslint/markdown": "^8.0.0",
     "@stylistic/eslint-plugin": "^5.0.0",
     "@typescript-eslint/eslint-plugin": "^8.0.0",
     "@typescript-eslint/parser": "^8.0.0",
     "confusing-browser-globals": "^1.0.0",
     "eslint-plugin-import-x": "^4.0.0",
-    "eslint-plugin-jest-formatting": "^3.0.0",
-    "eslint-plugin-jsdoc": "^54.0.0",
+    "eslint-plugin-jsdoc": "^62.0.0",
     "eslint-plugin-n": "^17.0.0",
-    "eslint-plugin-perfectionist": "^4.0.0",
-    "eslint-plugin-prettier": "^5.0.0",
-    "eslint-plugin-unicorn": "^60.0.0",
+    "eslint-plugin-perfectionist": "^5.0.0",
+    "eslint-plugin-unicorn": "^64.0.0",
     "find-up": "^5.0.0"
   },
   "devDependencies": {
     "@types/confusing-browser-globals": "^1.0.0",
-    "prettier-plugin-packagejson": "^2.0.0",
+    "prettier": "^3.0.0",
+    "prettier-plugin-packagejson": "^3.0.0",
     "remark-cli": "^12.0.0",
     "remark-preset-remcohaszing": "^3.0.0",
-    "type-fest": "^4.0.0",
-    "typescript": "^5.0.0"
+    "type-fest": "^5.0.0",
+    "typescript": "^6.0.0"
   },
   "peerDependencies": {
-    "eslint": "9",
-    "prettier": "3"
+    "eslint": "10"
   }
 }

package/types/config.d.ts

@@ -8,7 +8,7 @@
  */
 export function define(overrides: Linter.Config | Linter.Config[]): Linter.Config[];
 export default config;
-export const typechecking: Linter.Config<Linter.RulesRecord>[];
+export const typechecking: import("@eslint/config-helpers").Config[];
 import type { Linter } from 'eslint';
-declare const config: Linter.Config<Linter.RulesRecord>[];
+declare const config: import("@eslint/config-helpers").Config[];
 //# sourceMappingURL=config.d.ts.map

package/types/config.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"config.d.ts","sourceRoot":"","sources":["../lib/config.js"],"names":[],"mappings":"AA4rCA;;;;;;;GAOG;AACH,kCALW,aAAa,GAAG,aAAa,EAAE,GAE7B,aAAa,EAAE,CAK3B;;AAvFD,+DA2EE;4BAzrCuB,QAAQ;AA+HjC,0DA2+BE"}
+{"version":3,"file":"config.d.ts","sourceRoot":"","sources":["../lib/config.js"],"names":[],"mappings":"AAisCA;;;;;;;GAOG;AACH,kCALW,aAAa,GAAG,aAAa,EAAE,GAE7B,aAAa,EAAE,CAK3B;;AAxFD,qEA4EE;4BA9rCuB,QAAQ;AA6HjC,gEAi/BE"}