@ncontiero/eslint-config 7.3.0-beta.0 → 7.3.0-beta.2

package/dist/index.d.mts

@@ -4208,11 +4208,6 @@ interface RuleOptions {
    */
   'radix'?: Linter.RuleEntry<Radix>;
   'react-refresh/only-export-components'?: Linter.RuleEntry<ReactRefreshOnlyExportComponents>;
-  /**
-   * Disallows higher order functions that define components or hooks inside them.
-   * @see https://eslint-react.xyz/docs/rules/component-hook-factories
-   */
-  'react/component-hook-factories'?: Linter.RuleEntry<[]>;
   /**
    * Disallows DOM elements from using 'dangerouslySetInnerHTML'.
    * @see https://eslint-react.xyz/docs/rules/dom-no-dangerously-set-innerhtml
@@ -4293,11 +4288,6 @@ interface RuleOptions {
    * @see https://eslint-react.xyz/docs/rules/dom-no-void-elements-with-children
    */
   'react/dom-no-void-elements-with-children'?: Linter.RuleEntry<[]>;
-  /**
-   * Enforces importing React DOM via a namespace import.
-   * @see https://eslint-react.xyz/docs/rules/dom-prefer-namespace-import
-   */
-  'react/dom-prefer-namespace-import'?: Linter.RuleEntry<[]>;
   /**
    * Validates usage of Error Boundaries instead of try/catch for errors in child components.
    * @see https://eslint-react.xyz/docs/rules/error-boundaries
@@ -4308,6 +4298,11 @@ interface RuleOptions {
    * @see https://github.com/facebook/react/issues/14920
    */
   'react/exhaustive-deps'?: Linter.RuleEntry<ReactExhaustiveDeps>;
+  /**
+   * Validates against assignment/mutation of globals during render, part of ensuring that side effects must run outside of render.
+   * @see https://eslint-react.xyz/docs/rules/globals
+   */
+  'react/globals'?: Linter.RuleEntry<[]>;
   /**
    * Validates against mutating props, state, and other values that are immutable.
    * @see https://eslint-react.xyz/docs/rules/immutability
@@ -4503,11 +4498,6 @@ interface RuleOptions {
    * @see https://eslint-react.xyz/docs/rules/no-nested-lazy-component-declarations
    */
   'react/no-nested-lazy-component-declarations'?: Linter.RuleEntry<[]>;
-  /**
-   * Disallows 'shouldComponentUpdate' when extending 'React.PureComponent'.
-   * @see https://eslint-react.xyz/docs/rules/no-redundant-should-component-update
-   */
-  'react/no-redundant-should-component-update'?: Linter.RuleEntry<[]>;
   /**
    * Disallows calling 'this.setState' in 'componentDidMount' outside functions such as callbacks.
    * @see https://eslint-react.xyz/docs/rules/no-set-state-in-component-did-mount
@@ -4523,16 +4513,6 @@ interface RuleOptions {
    * @see https://eslint-react.xyz/docs/rules/no-set-state-in-component-will-update
    */
   'react/no-set-state-in-component-will-update'?: Linter.RuleEntry<[]>;
-  /**
-   * Disallows unnecessary usage of 'useCallback'.
-   * @see https://eslint-react.xyz/docs/rules/no-unnecessary-use-callback
-   */
-  'react/no-unnecessary-use-callback'?: Linter.RuleEntry<[]>;
-  /**
-   * Disallows unnecessary usage of 'useMemo'.
-   * @see https://eslint-react.xyz/docs/rules/no-unnecessary-use-memo
-   */
-  'react/no-unnecessary-use-memo'?: Linter.RuleEntry<[]>;
   /**
    * Enforces that a function with the 'use' prefix uses at least one Hook inside it.
    * @see https://eslint-react.xyz/docs/rules/no-unnecessary-use-prefix
@@ -4573,26 +4553,11 @@ interface RuleOptions {
    * @see https://eslint-react.xyz/docs/rules/no-unused-props
    */
   'react/no-unused-props'?: Linter.RuleEntry<[]>;
-  /**
-   * Warns about unused class component state.
-   * @see https://eslint-react.xyz/docs/rules/no-unused-state
-   */
-  'react/no-unused-state'?: Linter.RuleEntry<[]>;
   /**
    * Replaces usage of 'useContext' with 'use'.
    * @see https://eslint-react.xyz/docs/rules/no-use-context
    */
   'react/no-use-context'?: Linter.RuleEntry<[]>;
-  /**
-   * Enforces destructuring assignment for component props and context.
-   * @see https://eslint-react.xyz/docs/rules/prefer-destructuring-assignment
-   */
-  'react/prefer-destructuring-assignment'?: Linter.RuleEntry<[]>;
-  /**
-   * Enforces importing React via a namespace import.
-   * @see https://eslint-react.xyz/docs/rules/prefer-namespace-import
-   */
-  'react/prefer-namespace-import'?: Linter.RuleEntry<[]>;
   /**
    * Validates that components and hooks are pure by checking that they do not call known-impure functions during render.
    * @see https://eslint-react.xyz/docs/rules/purity
@@ -4623,6 +4588,11 @@ interface RuleOptions {
    * @see https://eslint-react.xyz/docs/rules/set-state-in-render
    */
   'react/set-state-in-render'?: Linter.RuleEntry<[]>;
+  /**
+   * Validates that components are static, not recreated every render.
+   * @see https://eslint-react.xyz/docs/rules/static-components
+   */
+  'react/static-components'?: Linter.RuleEntry<[]>;
   /**
    * Validates against syntax that React Compiler does not support.
    * @see https://eslint-react.xyz/docs/rules/unsupported-syntax
@@ -4643,6 +4613,11 @@ interface RuleOptions {
    * @see https://eslint-react.xyz/docs/rules/web-api-no-leaked-event-listener
    */
   'react/web-api-no-leaked-event-listener'?: Linter.RuleEntry<[]>;
+  /**
+   * Enforces that every 'fetch' in a component or custom hook has a corresponding 'AbortController' abort in the cleanup function.
+   * @see https://eslint-react.xyz/docs/rules/web-api-no-leaked-fetch
+   */
+  'react/web-api-no-leaked-fetch'?: Linter.RuleEntry<[]>;
   /**
    * Enforces that every 'setInterval' in a component or custom hook has a corresponding 'clearInterval'.
    * @see https://eslint-react.xyz/docs/rules/web-api-no-leaked-interval
@@ -4658,11 +4633,6 @@ interface RuleOptions {
    * @see https://eslint-react.xyz/docs/rules/web-api-no-leaked-timeout
    */
   'react/web-api-no-leaked-timeout'?: Linter.RuleEntry<[]>;
-  /**
-   * Disallows higher order functions that define components or hooks inside them.
-   * @see https://eslint-react.xyz/docs/rules/component-hook-factories
-   */
-  'react/x-component-hook-factories'?: Linter.RuleEntry<[]>;
   /**
    * Validates usage of Error Boundaries instead of try/catch for errors in child components.
    * @see https://eslint-react.xyz/docs/rules/error-boundaries
@@ -4673,6 +4643,11 @@ interface RuleOptions {
    * @see https://github.com/facebook/react/issues/14920
    */
   'react/x-exhaustive-deps'?: Linter.RuleEntry<ReactXExhaustiveDeps>;
+  /**
+   * Validates against assignment/mutation of globals during render, part of ensuring that side effects must run outside of render.
+   * @see https://eslint-react.xyz/docs/rules/globals
+   */
+  'react/x-globals'?: Linter.RuleEntry<[]>;
   /**
    * Validates against mutating props, state, and other values that are immutable.
    * @see https://eslint-react.xyz/docs/rules/immutability
@@ -4813,11 +4788,6 @@ interface RuleOptions {
    * @see https://eslint-react.xyz/docs/rules/no-nested-lazy-component-declarations
    */
   'react/x-no-nested-lazy-component-declarations'?: Linter.RuleEntry<[]>;
-  /**
-   * Disallows 'shouldComponentUpdate' when extending 'React.PureComponent'.
-   * @see https://eslint-react.xyz/docs/rules/no-redundant-should-component-update
-   */
-  'react/x-no-redundant-should-component-update'?: Linter.RuleEntry<[]>;
   /**
    * Disallows calling 'this.setState' in 'componentDidMount' outside functions such as callbacks.
    * @see https://eslint-react.xyz/docs/rules/no-set-state-in-component-did-mount
@@ -4833,16 +4803,6 @@ interface RuleOptions {
    * @see https://eslint-react.xyz/docs/rules/no-set-state-in-component-will-update
    */
   'react/x-no-set-state-in-component-will-update'?: Linter.RuleEntry<[]>;
-  /**
-   * Disallows unnecessary usage of 'useCallback'.
-   * @see https://eslint-react.xyz/docs/rules/no-unnecessary-use-callback
-   */
-  'react/x-no-unnecessary-use-callback'?: Linter.RuleEntry<[]>;
-  /**
-   * Disallows unnecessary usage of 'useMemo'.
-   * @see https://eslint-react.xyz/docs/rules/no-unnecessary-use-memo
-   */
-  'react/x-no-unnecessary-use-memo'?: Linter.RuleEntry<[]>;
   /**
    * Enforces that a function with the 'use' prefix uses at least one Hook inside it.
    * @see https://eslint-react.xyz/docs/rules/no-unnecessary-use-prefix
@@ -4883,26 +4843,11 @@ interface RuleOptions {
    * @see https://eslint-react.xyz/docs/rules/no-unused-props
    */
   'react/x-no-unused-props'?: Linter.RuleEntry<[]>;
-  /**
-   * Warns about unused class component state.
-   * @see https://eslint-react.xyz/docs/rules/no-unused-state
-   */
-  'react/x-no-unused-state'?: Linter.RuleEntry<[]>;
   /**
    * Replaces usage of 'useContext' with 'use'.
    * @see https://eslint-react.xyz/docs/rules/no-use-context
    */
   'react/x-no-use-context'?: Linter.RuleEntry<[]>;
-  /**
-   * Enforces destructuring assignment for component props and context.
-   * @see https://eslint-react.xyz/docs/rules/prefer-destructuring-assignment
-   */
-  'react/x-prefer-destructuring-assignment'?: Linter.RuleEntry<[]>;
-  /**
-   * Enforces importing React via a namespace import.
-   * @see https://eslint-react.xyz/docs/rules/prefer-namespace-import
-   */
-  'react/x-prefer-namespace-import'?: Linter.RuleEntry<[]>;
   /**
    * Validates that components and hooks are pure by checking that they do not call known-impure functions during render.
    * @see https://eslint-react.xyz/docs/rules/purity
@@ -4928,6 +4873,11 @@ interface RuleOptions {
    * @see https://eslint-react.xyz/docs/rules/set-state-in-render
    */
   'react/x-set-state-in-render'?: Linter.RuleEntry<[]>;
+  /**
+   * Validates that components are static, not recreated every render.
+   * @see https://eslint-react.xyz/docs/rules/static-components
+   */
+  'react/x-static-components'?: Linter.RuleEntry<[]>;
   /**
    * Validates against syntax that React Compiler does not support.
    * @see https://eslint-react.xyz/docs/rules/unsupported-syntax
@@ -16695,6 +16645,9 @@ declare const hasTailwind: boolean;
 declare const hasTanStackReactQuery: boolean;
 //#endregion
 //#region src/factory.d.ts
+declare const defaultPluginRenaming: {
+  "@eslint-react": string;
+};
 type ResolvedOptions<T> = T extends boolean ? never : NonNullable<T>;
 declare function resolveSubOptions<K extends keyof OptionsConfig>(options: OptionsConfig, key: K): ResolvedOptions<OptionsConfig[K]>;
 declare function getOverrides<K extends keyof OptionsConfig>(options: OptionsConfig, key: K): Partial<Linter.RulesRecord & Rules>;
@@ -16769,7 +16722,49 @@ declare function interopDefault<T>(m: Awaitable<T>): Promise<T extends {
   default: infer U;
 } ? U : T>;
 declare function ensurePackages(packages: (string | undefined)[]): void;
-declare function composer(...items: Awaitable<FlatConfigItem[]>[]): Promise<FlatConfigItem[]>;
+interface ComposerOptions {
+  items: Awaitable<FlatConfigItem[]>[];
+  pluginsNameMap?: Record<string, string>;
+}
+declare function composer({
+  items,
+  pluginsNameMap
+}: ComposerOptions): Promise<FlatConfigItem[]>;
+/**
+ * Rename plugin prefixes in a rule object.
+ * Accepts a map of prefixes to rename.
+ *
+ * @example
+ * ```ts
+ * import { renameRules } from "@ncontiero/eslint-config";
+ *
+ * export default [
+ *   {
+ *     rules: renameRules(
+ *       {
+ *         "@typescript-eslint/indent": "error",
+ *       },
+ *       { "@typescript-eslint": "ts" },
+ *     ),
+ *   },
+ * ];
+ * ```
+ */
+declare function renameRules(rules: Record<string, any>, map: Record<string, string>): Record<string, any>;
+/**
+ * Rename plugin names a flat configs array
+ *
+ * @example
+ * ```ts
+ * import { renamePluginInConfigs } from "@ncontiero/eslint-config";
+ * import someConfigs from "./some-configs";
+ *
+ * export default renamePluginInConfigs(someConfigs, {
+ *   "@typescript-eslint": "ts",
+ * });
+ * ```
+ */
+declare function renamePluginInConfigs(configs: FlatConfigItem[], map: Record<string, string>): FlatConfigItem[];
 declare function toArray<T>(value: T | T[]): T[];
 //#endregion
-export { Awaitable, FlatConfigItem, GLOB_ALL_SRC, GLOB_CSS, GLOB_DIST, GLOB_EXCLUDE, GLOB_HTML, GLOB_JS, GLOB_JSON, GLOB_JSON5, GLOB_JSONC, GLOB_JSX, GLOB_LESS, GLOB_LOCKFILE, GLOB_MARKDOWN, GLOB_MARKDOWN_CODE, GLOB_MARKDOWN_IN_MARKDOWN, GLOB_NODE_MODULES, GLOB_POSTCSS, GLOB_REACT, GLOB_SCSS, GLOB_SRC, GLOB_SRC_EXT, GLOB_STYLE, GLOB_TOML, GLOB_TS, GLOB_TSX, GLOB_YAML, HTMLOptions, OptionsConfig, OptionsFiles, OptionsHasNextJs, OptionsHasRegexp, OptionsHasTanStackReactQuery, OptionsHasTypeScript, OptionsOverrides, OptionsTailwindCSS, OptionsTypeScriptParserOptions, OptionsTypescript, OptionsUnicorn, PrettierOptions, ResolvedOptions, Rules, StyleConfig, StyleOptions, combine, command, comments, composer, deMorgan, ensurePackages, getOverrides, hasNextJs, hasReact, hasTailwind, hasTanStackReactQuery, hasTypeScript, html, ignores, imports, interopDefault, javascript, jsdoc, jsonc, markdown, ncontiero, nextJs, node, parserPlain, perfectionist, prettier, promise, react, regexp, resolveSubOptions, restrictedSyntaxJs, sortPackageJson, sortPnpmWorkspace, sortTsconfig, tailwindcss, toArray, toml, typescript, unicorn, yml };
+export { Awaitable, ComposerOptions, FlatConfigItem, GLOB_ALL_SRC, GLOB_CSS, GLOB_DIST, GLOB_EXCLUDE, GLOB_HTML, GLOB_JS, GLOB_JSON, GLOB_JSON5, GLOB_JSONC, GLOB_JSX, GLOB_LESS, GLOB_LOCKFILE, GLOB_MARKDOWN, GLOB_MARKDOWN_CODE, GLOB_MARKDOWN_IN_MARKDOWN, GLOB_NODE_MODULES, GLOB_POSTCSS, GLOB_REACT, GLOB_SCSS, GLOB_SRC, GLOB_SRC_EXT, GLOB_STYLE, GLOB_TOML, GLOB_TS, GLOB_TSX, GLOB_YAML, HTMLOptions, OptionsConfig, OptionsFiles, OptionsHasNextJs, OptionsHasRegexp, OptionsHasTanStackReactQuery, OptionsHasTypeScript, OptionsOverrides, OptionsTailwindCSS, OptionsTypeScriptParserOptions, OptionsTypescript, OptionsUnicorn, PrettierOptions, ResolvedOptions, Rules, StyleConfig, StyleOptions, combine, command, comments, composer, deMorgan, defaultPluginRenaming, ensurePackages, getOverrides, hasNextJs, hasReact, hasTailwind, hasTanStackReactQuery, hasTypeScript, html, ignores, imports, interopDefault, javascript, jsdoc, jsonc, markdown, ncontiero, nextJs, node, parserPlain, perfectionist, prettier, promise, react, regexp, renamePluginInConfigs, renameRules, resolveSubOptions, restrictedSyntaxJs, sortPackageJson, sortPnpmWorkspace, sortTsconfig, tailwindcss, toArray, toml, typescript, unicorn, yml };

package/dist/index.mjs

@@ -153,12 +153,63 @@ function ensurePackages(packages) {
 	if (nonExistingPackages.length === 0) return;
 	throw new Error(`This package(s) are required for this config: ${nonExistingPackages.join(", ")}. Please install them.`);
 }
-async function composer(...items) {
+async function composer({ items, pluginsNameMap }) {
 	let configs = [];
 	const flatResolved = (await Promise.all(items)).flat();
 	configs = [...configs, ...flatResolved];
+	if (pluginsNameMap) configs = renamePluginInConfigs(configs, pluginsNameMap);
 	return configs;
 }
+/**
+* Rename plugin prefixes in a rule object.
+* Accepts a map of prefixes to rename.
+*
+* @example
+* ```ts
+* import { renameRules } from "@ncontiero/eslint-config";
+*
+* export default [
+*   {
+*     rules: renameRules(
+*       {
+*         "@typescript-eslint/indent": "error",
+*       },
+*       { "@typescript-eslint": "ts" },
+*     ),
+*   },
+* ];
+* ```
+*/
+function renameRules(rules, map) {
+	return Object.fromEntries(Object.entries(rules).map(([key, value]) => {
+		for (const [from, to] of Object.entries(map)) if (key.startsWith(`${from}/`)) return [to + key.slice(from.length), value];
+		return [key, value];
+	}));
+}
+/**
+* Rename plugin names a flat configs array
+*
+* @example
+* ```ts
+* import { renamePluginInConfigs } from "@ncontiero/eslint-config";
+* import someConfigs from "./some-configs";
+*
+* export default renamePluginInConfigs(someConfigs, {
+*   "@typescript-eslint": "ts",
+* });
+* ```
+*/
+function renamePluginInConfigs(configs, map) {
+	return configs.map((i) => {
+		const clone = { ...i };
+		if (clone.rules) clone.rules = renameRules(clone.rules, map);
+		if (clone.plugins) clone.plugins = Object.fromEntries(Object.entries(clone.plugins).map(([key, value]) => {
+			if (key in map) return [map[key], value];
+			return [key, value];
+		}));
+		return clone;
+	});
+}
 function toArray(value) {
 	return Array.isArray(value) ? value : [value];
 }
@@ -959,7 +1010,7 @@ async function react(options = {}) {
 				"react/no-leaked-conditional-rendering": "error",
 				"react/no-missing-component-display-name": "warn",
 				"react/no-unstable-context-value": "warn",
-				"react/no-unstable-default-props": "error",
+				"react/no-unstable-default-props": "warn",
 				"react/no-unused-props": "warn",
 				"react/refs": "warn",
 				...overrides
@@ -1557,6 +1608,7 @@ const flatConfigProps = [
 	"rules",
 	"settings"
 ];
+const defaultPluginRenaming = { "@eslint-react": "react" };
 function resolveSubOptions(options, key) {
 	return typeof options[key] === "boolean" ? {} : options[key] || {};
 }
@@ -1636,7 +1688,10 @@ function ncontiero(options = {}, ...userConfigs) {
 		return acc;
 	}, {});
 	if (Object.keys(fusedConfig).length > 0) configs.push([fusedConfig]);
-	return composer(...configs, ...userConfigs);
+	return composer({
+		items: [...configs, ...userConfigs],
+		pluginsNameMap: defaultPluginRenaming
+	});
 }
 //#endregion
-export { GLOB_ALL_SRC, GLOB_CSS, GLOB_DIST, GLOB_EXCLUDE, GLOB_HTML, GLOB_JS, GLOB_JSON, GLOB_JSON5, GLOB_JSONC, GLOB_JSX, GLOB_LESS, GLOB_LOCKFILE, GLOB_MARKDOWN, GLOB_MARKDOWN_CODE, GLOB_MARKDOWN_IN_MARKDOWN, GLOB_NODE_MODULES, GLOB_POSTCSS, GLOB_REACT, GLOB_SCSS, GLOB_SRC, GLOB_SRC_EXT, GLOB_STYLE, GLOB_TOML, GLOB_TS, GLOB_TSX, GLOB_YAML, combine, command, comments, composer, deMorgan, ensurePackages, getOverrides, hasNextJs, hasReact, hasTailwind, hasTanStackReactQuery, hasTypeScript, html, ignores, imports, interopDefault, javascript, jsdoc, jsonc, markdown, ncontiero, nextJs, node, parserPlain, perfectionist, prettier, promise, react, regexp, resolveSubOptions, restrictedSyntaxJs, sortPackageJson, sortPnpmWorkspace, sortTsconfig, tailwindcss, toArray, toml, typescript, unicorn, yml };
+export { GLOB_ALL_SRC, GLOB_CSS, GLOB_DIST, GLOB_EXCLUDE, GLOB_HTML, GLOB_JS, GLOB_JSON, GLOB_JSON5, GLOB_JSONC, GLOB_JSX, GLOB_LESS, GLOB_LOCKFILE, GLOB_MARKDOWN, GLOB_MARKDOWN_CODE, GLOB_MARKDOWN_IN_MARKDOWN, GLOB_NODE_MODULES, GLOB_POSTCSS, GLOB_REACT, GLOB_SCSS, GLOB_SRC, GLOB_SRC_EXT, GLOB_STYLE, GLOB_TOML, GLOB_TS, GLOB_TSX, GLOB_YAML, combine, command, comments, composer, deMorgan, defaultPluginRenaming, ensurePackages, getOverrides, hasNextJs, hasReact, hasTailwind, hasTanStackReactQuery, hasTypeScript, html, ignores, imports, interopDefault, javascript, jsdoc, jsonc, markdown, ncontiero, nextJs, node, parserPlain, perfectionist, prettier, promise, react, regexp, renamePluginInConfigs, renameRules, resolveSubOptions, restrictedSyntaxJs, sortPackageJson, sortPnpmWorkspace, sortTsconfig, tailwindcss, toArray, toml, typescript, unicorn, yml };

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@ncontiero/eslint-config",
   "type": "module",
-  "version": "7.3.0-beta.0",
+  "version": "7.3.0-beta.2",
   "packageManager": "pnpm@10.33.2",
   "description": "Nicolas's ESLint config.",
   "author": {
@@ -52,7 +52,7 @@
   },
   "dependencies": {
     "@eslint-community/eslint-plugin-eslint-comments": "^4.7.1",
-    "@eslint-react/eslint-plugin": "^4.2.3",
+    "@eslint-react/eslint-plugin": "5.5.1-beta.1",
     "@eslint/markdown": "^8.0.1",
     "@html-eslint/eslint-plugin": "^0.59.0",
     "@html-eslint/parser": "^0.59.0",