@ncontiero/eslint-config 8.2.0 → 8.2.2

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,46 +3739,46 @@ 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>;
   /**
-   * Enforces the context name to be a valid component name with the suffix 'Context'.
+   * Enforces identifier names assigned from `createContext` calls to be a valid component name with the suffix `Context`.
    * @see https://eslint-react.xyz/docs/rules/naming-convention-context-name
    */
   'react/naming-convention-context-name'?: Linter.RuleEntry<[]>;
@@ -3838,7 +3858,7 @@ interface RuleOptions {
    */
   'react/no-context-provider'?: Linter.RuleEntry<[]>;
   /**
-   * Disallows 'createRef' in function components.
+   * Disallows 'createRef' in function components and Hooks.
    * @see https://eslint-react.xyz/docs/rules/no-create-ref
    */
   'react/no-create-ref'?: Linter.RuleEntry<[]>;
@@ -3963,7 +3983,7 @@ interface RuleOptions {
    */
   'react/no-unused-props'?: Linter.RuleEntry<[]>;
   /**
-   * Warns about state variables that are defined but never used, or only used in effects.
+   * Warns about state variables that are defined but never used.
    * @see https://eslint-react.xyz/docs/rules/no-unused-state
    */
   'react/no-unused-state'?: Linter.RuleEntry<[]>;
@@ -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<[]>;
   /**
@@ -4133,7 +4153,7 @@ interface RuleOptions {
    */
   'react/x-no-context-provider'?: Linter.RuleEntry<[]>;
   /**
-   * Disallows 'createRef' in function components.
+   * Disallows 'createRef' in function components and Hooks.
    * @see https://eslint-react.xyz/docs/rules/no-create-ref
    */
   'react/x-no-create-ref'?: Linter.RuleEntry<[]>;
@@ -4258,7 +4278,7 @@ interface RuleOptions {
    */
   'react/x-no-unused-props'?: Linter.RuleEntry<[]>;
   /**
-   * Warns about state variables that are defined but never used, or only used in effects.
+   * Warns about state variables that are defined but never used.
    * @see https://eslint-react.xyz/docs/rules/no-unused-state
    */
   'react/x-no-unused-state'?: Linter.RuleEntry<[]>;
@@ -6874,6 +6894,7 @@ type HandleCallbackErr = [] | [string]; // ----- html/attrs-newline -----
 type HtmlAttrsNewline = [] | [{
   closeStyle?: ("newline" | "sameline");
   ifAttrsMoreThan?: number;
+  maxLen?: number;
   skip?: string[];
   inline?: string[];
 }]; // ----- html/element-newline -----
@@ -6969,6 +6990,7 @@ type HtmlRequireOpenGraphProtocol = [] | [string[]]; // ----- html/sort-attrs --
 type HtmlSortAttrs = [] | [{
   priority?: (string | {
     pattern: string;
+    order?: ("preserve" | "alphabetically");
   })[];
 }]; // ----- html/use-baseline -----
 type HtmlUseBaseline = [] | [{

package/dist/index.mjs

@@ -1403,7 +1403,7 @@ async function typescript(options = {}) {
 		"ts/require-await": "error",
 		"ts/restrict-plus-operands": "error",
 		"ts/restrict-template-expressions": "error",
-		"ts/return-await": ["error", "always"],
+		"ts/return-await": ["error", "in-try-catch"],
 		"ts/strict-boolean-expressions": ["error", {
 			allowNullableBoolean: true,
 			allowNullableString: true

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@ncontiero/eslint-config",
   "type": "module",
-  "version": "8.2.0",
+  "version": "8.2.2",
   "description": "Nicolas's ESLint config.",
   "author": {
     "name": "Nicolas Contiero",
@@ -37,15 +37,15 @@
     }
   },
   "dependencies": {
-    "@e18e/eslint-plugin": "^0.4.1",
-    "@eslint-community/eslint-plugin-eslint-comments": "^4.7.1",
-    "@eslint-react/eslint-plugin": "^5.7.6",
-    "@eslint/markdown": "^8.0.1",
-    "@html-eslint/eslint-plugin": "^0.60.0",
-    "@html-eslint/parser": "^0.60.0",
+    "@e18e/eslint-plugin": "^0.5.0",
+    "@eslint-community/eslint-plugin-eslint-comments": "^4.7.2",
+    "@eslint-react/eslint-plugin": "^5.8.8",
+    "@eslint/markdown": "^8.0.2",
+    "@html-eslint/eslint-plugin": "^0.61.0",
+    "@html-eslint/parser": "^0.61.0",
     "@next/eslint-plugin-next": "^16.2.6",
-    "@typescript-eslint/eslint-plugin": "^8.59.3",
-    "@typescript-eslint/parser": "^8.59.3",
+    "@typescript-eslint/eslint-plugin": "^8.60.0",
+    "@typescript-eslint/parser": "^8.60.0",
     "eslint-config-flat-gitignore": "^2.3.0",
     "eslint-merge-processors": "^2.0.0",
     "eslint-plugin-antfu": "^3.2.3",
@@ -53,45 +53,45 @@
     "eslint-plugin-command": "^3.5.2",
     "eslint-plugin-de-morgan": "^2.1.2",
     "eslint-plugin-import-x": "^4.16.2",
-    "eslint-plugin-jsdoc": "^62.9.0",
-    "eslint-plugin-jsonc": "^3.1.2",
+    "eslint-plugin-jsdoc": "^63.0.0",
+    "eslint-plugin-jsonc": "^3.2.0",
     "eslint-plugin-jsx-a11y": "^6.10.2",
     "eslint-plugin-n": "^18.0.1",
     "eslint-plugin-perfectionist": "^5.9.0",
-    "eslint-plugin-prettier": "^5.5.5",
+    "eslint-plugin-prettier": "^5.5.6",
     "eslint-plugin-promise": "^7.3.0",
     "eslint-plugin-react-refresh": "^0.5.2",
     "eslint-plugin-regexp": "^3.1.0",
-    "eslint-plugin-toml": "^1.3.1",
+    "eslint-plugin-toml": "^1.4.0",
     "eslint-plugin-unicorn": "^64.0.0",
     "eslint-plugin-unused-imports": "^4.4.1",
-    "eslint-plugin-yml": "^3.3.2",
+    "eslint-plugin-yml": "^3.4.0",
     "globals": "^17.6.0",
-    "local-pkg": "^1.1.2",
+    "local-pkg": "^1.2.1",
     "prettier": "^3.8.3",
     "toml-eslint-parser": "^1.0.3",
     "yaml-eslint-parser": "^2.0.0"
   },
   "devDependencies": {
     "@changesets/cli": "^2.31.0",
-    "@commitlint/cli": "^21.0.0",
-    "@commitlint/config-conventional": "^21.0.0",
-    "@eslint/config-inspector": "^2.0.1",
-    "@ncontiero/changelog-github": "^2.1.3",
+    "@commitlint/cli": "^21.0.2",
+    "@commitlint/config-conventional": "^21.0.2",
+    "@eslint/config-inspector": "^3.0.4",
+    "@ncontiero/changelog-github": "^2.1.4",
     "@ncontiero/prettier-config": "^1.0.0",
-    "@tanstack/eslint-plugin-query": "^5.100.10",
+    "@tanstack/eslint-plugin-query": "^5.100.14",
     "@types/eslint-plugin-jsx-a11y": "^6.10.1",
-    "@types/node": "^25.7.0",
-    "eslint": "^10.3.0",
+    "@types/node": "^25.9.1",
+    "eslint": "^10.4.1",
     "eslint-typegen": "^2.3.1",
     "husky": "^9.1.7",
     "nano-staged": "^1.0.2",
-    "tinyexec": "^1.1.2",
-    "tinyglobby": "^0.2.16",
-    "tsdown": "^0.22.0",
-    "tsx": "^4.21.0",
+    "tinyexec": "^1.2.4",
+    "tinyglobby": "^0.2.17",
+    "tsdown": "^0.22.1",
+    "tsx": "^4.22.4",
     "typescript": "^6.0.3",
-    "vitest": "^4.1.6"
+    "vitest": "^4.1.7"
   },
   "engines": {
     "node": ">=20.19.0"