@ncontiero/eslint-config 8.1.0 → 8.2.1

package/dist/index.d.mts

@@ -277,10 +277,18 @@ interface RuleOptions {
    * Disallow dependencies in favor of more performant or secure alternatives
    */
   'e18e/ban-dependencies'?: Linter.RuleEntry<E18EBanDependencies>;
+  /**
+   * Disallow `delete` on properties — V8 deoptimizes the object to dictionary mode
+   */
+  'e18e/no-delete-property'?: Linter.RuleEntry<[]>;
   /**
    * Prefer optimized alternatives to `indexOf()` equality checks
    */
   'e18e/no-indexof-equality'?: Linter.RuleEntry<[]>;
+  /**
+   * Disallow spreading the accumulator inside a `reduce` callback (O(N²) growth)
+   */
+  'e18e/no-spread-in-reduce'?: Linter.RuleEntry<[]>;
   /**
    * Prefer Array.prototype.at() over length-based indexing
    */
@@ -294,7 +302,7 @@ interface RuleOptions {
    */
   'e18e/prefer-array-from-map'?: Linter.RuleEntry<[]>;
   /**
-   * Prefer Array.some() over Array.find() when checking for element existence
+   * Prefer Array.some() over Array.find() and Array.filter().length checks when checking for element existence
    */
   'e18e/prefer-array-some'?: Linter.RuleEntry<[]>;
   /**
@@ -321,6 +329,10 @@ interface RuleOptions {
    * Prefer .includes() over indexOf() comparisons for arrays and strings
    */
   'e18e/prefer-includes'?: Linter.RuleEntry<[]>;
+  /**
+   * Prefer String.prototype.{includes,startsWith,endsWith} over equivalent regex.test() calls
+   */
+  'e18e/prefer-includes-over-regex-test'?: Linter.RuleEntry<[]>;
   /**
    * Prefer inline equality checks over temporary object creation for simple comparisons
    */
@@ -341,10 +353,18 @@ interface RuleOptions {
    * Prefer spread syntax over Array.concat(), Array.from(), Object.assign({}, ...), and Function.apply()
    */
   'e18e/prefer-spread-syntax'?: Linter.RuleEntry<[]>;
+  /**
+   * Prefer hoisting an `Intl.Collator` instance over calling localeCompare in a sort callback
+   */
+  'e18e/prefer-static-collator'?: Linter.RuleEntry<[]>;
   /**
    * Prefer defining regular expressions at module scope to avoid re-compilation on every function call
    */
   'e18e/prefer-static-regex'?: Linter.RuleEntry<[]>;
+  /**
+   * Prefer String.fromCharCode() over String.fromCodePoint() for code points below 0x10000
+   */
+  'e18e/prefer-string-fromcharcode'?: Linter.RuleEntry<[]>;
   /**
    * Prefer passing function and arguments directly to setTimeout/setInterval instead of wrapping in an arrow function or using bind
    */
@@ -3719,42 +3739,42 @@ interface RuleOptions {
   'react/immutability'?: Linter.RuleEntry<[]>;
   /**
    * Disallows passing 'children' as a prop.
-   * @see https://eslint-react.xyz/docs/rules/no-children-prop
+   * @see https://eslint-react.xyz/docs/rules/jsx-no-children-prop
    */
   'react/jsx-no-children-prop'?: Linter.RuleEntry<[]>;
   /**
    * Disallows passing 'children' as a prop when children are also passed as nested content.
-   * @see https://eslint-react.xyz/docs/rules/no-children-prop-with-children
+   * @see https://eslint-react.xyz/docs/rules/jsx-no-children-prop-with-children
    */
   'react/jsx-no-children-prop-with-children'?: Linter.RuleEntry<[]>;
   /**
    * Prevents comment strings from being accidentally inserted into a JSX element's text nodes.
-   * @see https://eslint-react.xyz/docs/rules/no-comment-textnodes
+   * @see https://eslint-react.xyz/docs/rules/jsx-no-comment-textnodes
    */
   'react/jsx-no-comment-textnodes'?: Linter.RuleEntry<[]>;
   /**
    * Prevent patterns that cause deoptimization when using the automatic JSX runtime.
-   * @see https://eslint-react.xyz/docs/rules/no-key-after-spread
+   * @see https://eslint-react.xyz/docs/rules/jsx-no-key-after-spread
    */
   'react/jsx-no-key-after-spread'?: Linter.RuleEntry<[]>;
   /**
    * Catches `$` before `{expr}` in JSX — typically from template literal `${expr}` being copy-pasted into JSX without removing the `$`. The `$` "leaks" into the rendered output.
-   * @see https://eslint-react.xyz/docs/rules/no-leaked-dollar
+   * @see https://eslint-react.xyz/docs/rules/jsx-no-leaked-dollar
    */
   'react/jsx-no-leaked-dollar'?: Linter.RuleEntry<[]>;
   /**
    * Catches `;` at the start of JSX text nodes — typically from accidentally placing a statement-ending `;` inside JSX. The `;` "leaks" into the rendered output.
-   * @see https://eslint-react.xyz/docs/rules/no-leaked-semicolon
+   * @see https://eslint-react.xyz/docs/rules/jsx-no-leaked-semicolon
    */
   'react/jsx-no-leaked-semicolon'?: Linter.RuleEntry<[]>;
   /**
    * Disallow JSX namespace syntax, as React does not support them.
-   * @see https://eslint-react.xyz/docs/rules/no-namespace
+   * @see https://eslint-react.xyz/docs/rules/jsx-no-namespace
    */
   'react/jsx-no-namespace'?: Linter.RuleEntry<[]>;
   /**
    * Disallows useless fragment elements.
-   * @see https://eslint-react.xyz/docs/rules/no-useless-fragment
+   * @see https://eslint-react.xyz/docs/rules/jsx-no-useless-fragment
    */
   'react/jsx-no-useless-fragment'?: Linter.RuleEntry<ReactJsxNoUselessFragment>;
   /**
@@ -3984,7 +4004,7 @@ interface RuleOptions {
   'react/refs'?: Linter.RuleEntry<[]>;
   /**
    * Validates and transforms React Client/Server Function definitions.
-   * @see https://eslint-react.xyz/docs/rules/function-definition
+   * @see https://eslint-react.xyz/docs/rules/rsc-function-definition
    */
   'react/rsc-function-definition'?: Linter.RuleEntry<[]>;
   /**
@@ -16707,6 +16727,12 @@ interface OptionsConfig {
    * @default true
    */
   unicorn?: boolean | OptionsUnicorn;
+  /**
+   * Options for eslint-plugin-perfectionist.
+   *
+   * @default true
+   */
+  perfectionist?: boolean | OptionsOverrides;
   /**
    * Options for [@e18e/eslint-plugin](https://github.com/e18e/eslint-plugin)
    *
@@ -16843,7 +16869,7 @@ declare function node(): FlatConfigItem[];
  *
  * @see https://github.com/azat-io/eslint-plugin-perfectionist
  */
-declare function perfectionist(): FlatConfigItem[];
+declare function perfectionist(options?: OptionsOverrides): FlatConfigItem[];
 //#endregion
 //#region src/configs/prettier.d.ts
 declare function prettier(options?: PrettierOptions): Promise<FlatConfigItem[]>;

package/dist/index.mjs

@@ -465,7 +465,7 @@ function javascript(options = {}) {
 				"no-useless-escape": "error",
 				"no-useless-rename": "error",
 				"no-var": "error",
-				"no-void": "error",
+				"no-void": ["error", { allowAsStatement: true }],
 				"no-with": "error",
 				"object-shorthand": [
 					"error",
@@ -785,7 +785,8 @@ function node() {
 *
 * @see https://github.com/azat-io/eslint-plugin-perfectionist
 */
-function perfectionist() {
+function perfectionist(options = {}) {
+	const { overrides = {} } = options;
 	return [{
 		name: "ncontiero/perfectionist/rules",
 		plugins: { perfectionist: pluginPerfectionist },
@@ -823,7 +824,8 @@ function perfectionist() {
 				type: "natural"
 			}],
 			"perfectionist/sort-named-exports": ["warn", { groups: ["type-export", "value-export"] }],
-			"perfectionist/sort-named-imports": ["warn", { groups: ["type-import", "value-import"] }]
+			"perfectionist/sort-named-imports": ["warn", { groups: ["type-import", "value-import"] }],
+			...overrides
 		}
 	}];
 }
@@ -1387,7 +1389,7 @@ async function typescript(options = {}) {
 		"ts/no-floating-promises": "error",
 		"ts/no-for-in-array": "error",
 		"ts/no-implied-eval": "error",
-		"ts/no-misused-promises": "error",
+		"ts/no-misused-promises": ["error", { checksVoidReturn: { attributes: false } }],
 		"ts/no-unnecessary-type-arguments": "error",
 		"ts/no-unnecessary-type-assertion": "error",
 		"ts/no-unsafe-argument": "error",
@@ -1401,8 +1403,11 @@ async function typescript(options = {}) {
 		"ts/require-await": "error",
 		"ts/restrict-plus-operands": "error",
 		"ts/restrict-template-expressions": "error",
-		"ts/return-await": ["error", "always"],
-		"ts/strict-boolean-expressions": ["error", { allowNullableBoolean: true }],
+		"ts/return-await": ["error", "in-try-catch"],
+		"ts/strict-boolean-expressions": ["error", {
+			allowNullableBoolean: true,
+			allowNullableString: true
+		}],
 		"ts/switch-exhaustiveness-check": ["error", { considerDefaultExhaustiveForUnions: true }],
 		"ts/unbound-method": "error"
 	};
@@ -1720,7 +1725,7 @@ function getStyleOptions(options) {
 * @returns Merged ESLint configurations based on provided options.
 */
 function ncontiero(options = {}, ...userConfigs) {
-	const { deMorgan: enableDeMorgan = true, e18e: enableE18e = true, gitignore: enableGitignore = true, html: enableHtml = true, ignores: userIgnores = [], imports: enableImports = true, jsdoc: enableJsdoc = true, jsonc: enableJsonc = true, jsx: enableJsx = true, markdown: enableMarkdown = true, nextjs: enableNextJs = hasNextJs, node: enableNode = true, prettier: enablePrettier = true, promise: enablePromise = true, react: enableReact = hasReact, regexp: enableRegexp = true, tailwindcss: enableTailwindCSS = hasTailwind, tanstackQuery: enableTanStackQuery = false, typescript: enableTypescript = hasTypeScript, unicorn: enableUnicorn = true } = options;
+	const { deMorgan: enableDeMorgan = true, e18e: enableE18e = true, gitignore: enableGitignore = true, html: enableHtml = true, ignores: userIgnores = [], imports: enableImports = true, jsdoc: enableJsdoc = true, jsonc: enableJsonc = true, jsx: enableJsx = true, markdown: enableMarkdown = true, nextjs: enableNextJs = hasNextJs, node: enableNode = true, perfectionist: enablePerfectionist = true, prettier: enablePrettier = true, promise: enablePromise = true, react: enableReact = hasReact, regexp: enableRegexp = true, tailwindcss: enableTailwindCSS = hasTailwind, tanstackQuery: enableTanStackQuery = false, typescript: enableTypescript = hasTypeScript, unicorn: enableUnicorn = true } = options;
 	const prettierOptions = typeof options.prettier === "object" ? options.prettier : {};
 	const styleOptions = getStyleOptions(prettierOptions);
 	const configs = [];
@@ -1734,7 +1739,8 @@ function ncontiero(options = {}, ...userConfigs) {
 	})]));
 	const typescriptOptions = resolveSubOptions(options, "typescript");
 	const tsconfigPath = "tsconfigPath" in typescriptOptions ? typescriptOptions.tsconfigPath : void 0;
-	configs.push(ignores(userIgnores, !enableTypescript), javascript({ overrides: getOverrides(options, "javascript") }), comments(), command(), perfectionist());
+	configs.push(ignores(userIgnores, !enableTypescript), javascript({ overrides: getOverrides(options, "javascript") }), comments(), command());
+	if (enablePerfectionist) configs.push(perfectionist({ overrides: getOverrides(options, "perfectionist") }));
 	if (enableNode) configs.push(node());
 	if (enablePromise) configs.push(promise({ overrides: getOverrides(options, "promise") }));
 	if (enableJsdoc) configs.push(jsdoc());

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@ncontiero/eslint-config",
   "type": "module",
-  "version": "8.1.0",
+  "version": "8.2.1",
   "description": "Nicolas's ESLint config.",
   "author": {
     "name": "Nicolas Contiero",
@@ -37,15 +37,15 @@
     }
   },
   "dependencies": {
-    "@e18e/eslint-plugin": "^0.4.1",
+    "@e18e/eslint-plugin": "^0.5.0",
     "@eslint-community/eslint-plugin-eslint-comments": "^4.7.1",
-    "@eslint-react/eslint-plugin": "^5.7.5",
+    "@eslint-react/eslint-plugin": "^5.7.7",
     "@eslint/markdown": "^8.0.1",
     "@html-eslint/eslint-plugin": "^0.60.0",
     "@html-eslint/parser": "^0.60.0",
     "@next/eslint-plugin-next": "^16.2.6",
-    "@typescript-eslint/eslint-plugin": "^8.59.2",
-    "@typescript-eslint/parser": "^8.59.2",
+    "@typescript-eslint/eslint-plugin": "^8.59.3",
+    "@typescript-eslint/parser": "^8.59.3",
     "eslint-config-flat-gitignore": "^2.3.0",
     "eslint-merge-processors": "^2.0.0",
     "eslint-plugin-antfu": "^3.2.3",
@@ -74,14 +74,14 @@
   },
   "devDependencies": {
     "@changesets/cli": "^2.31.0",
-    "@commitlint/cli": "^21.0.0",
-    "@commitlint/config-conventional": "^21.0.0",
-    "@eslint/config-inspector": "^2.0.1",
+    "@commitlint/cli": "^21.0.1",
+    "@commitlint/config-conventional": "^21.0.1",
+    "@eslint/config-inspector": "^3.0.2",
     "@ncontiero/changelog-github": "^2.1.3",
     "@ncontiero/prettier-config": "^1.0.0",
     "@tanstack/eslint-plugin-query": "^5.100.10",
     "@types/eslint-plugin-jsx-a11y": "^6.10.1",
-    "@types/node": "^25.6.2",
+    "@types/node": "^25.7.0",
     "eslint": "^10.3.0",
     "eslint-typegen": "^2.3.1",
     "husky": "^9.1.7",