@vinicunca/eslint-config 4.10.0 → 5.1.0

package/dist/index.mjs

@@ -145,13 +145,17 @@ const GLOB_EXCLUDE = [
 	"**/.output",
 	"**/.vite-inspect",
 	"**/.yarn",
-	"**/vite.config.*.timestamp-*",
 	"**/CHANGELOG*.md",
-	"**/*.min.*",
 	"**/LICENSE*",
+	"**/*.min.*",
 	"**/__snapshots__",
+	"**/vite.config.*.timestamp-*",
 	"**/auto-import?(s).d.ts",
-	"**/components.d.ts"
+	"**/components.d.ts",
+	"**/.context",
+	"**/.claude",
+	"**/.agents",
+	"**/.*/skills"
 ];
 //#endregion
 //#region src/utils.ts
@@ -390,6 +394,9 @@ async function e18e(options = {}) {
 			...modernization ? { ...configs.modernization.rules } : {},
 			...moduleReplacements ? { ...configs.moduleReplacements.rules } : {},
 			...performanceImprovements ? { ...configs.performanceImprovements.rules } : {},
+			...type === "lib" ? {} : { "e18e/prefer-static-regex": "off" },
+			"e18e/prefer-array-at": "off",
+			"e18e/prefer-array-from-map": "off",
 			"e18e/prefer-array-to-reversed": "off",
 			"e18e/prefer-array-to-sorted": "off",
 			"e18e/prefer-array-to-spliced": "off",
@@ -401,18 +408,20 @@ async function e18e(options = {}) {
 //#endregion
 //#region src/configs/stylistic.ts
 const STYLISTIC_CONFIG_DEFAULTS = {
+	braceStyle: "stroustrup",
 	indent: 2,
 	jsx: true,
 	quotes: "single",
 	semi: true
 };
 async function stylistic(options = {}) {
-	const { experimental, indent, jsx, overrides = {}, quotes, semi } = {
+	const { braceStyle, experimental, indent, jsx, overrides = {}, quotes, semi } = {
 		...STYLISTIC_CONFIG_DEFAULTS,
 		...options
 	};
 	const pluginStylistic = await interopDefault(import("@stylistic/eslint-plugin"));
 	const config = pluginStylistic.configs.customize({
+		braceStyle,
 		experimental,
 		indent,
 		jsx,
@@ -637,8 +646,9 @@ async function formatters(options = {}, stylistic = {}) {
 }
 //#endregion
 //#region src/configs/ignores.ts
-async function ignores(userIgnores = []) {
+async function ignores(userIgnores = [], ignoreTypeScript = false) {
 	let ignores = [...GLOB_EXCLUDE];
+	if (ignoreTypeScript) ignores.push(GLOB_TS, GLOB_TSX);
 	if (e$1(userIgnores)) ignores = userIgnores(ignores);
 	else ignores = [...ignores, ...userIgnores];
 	return [{
@@ -1064,6 +1074,11 @@ async function markdown(options = {}) {
 			files,
 			ignores: [GLOB_MARKDOWN_IN_MARKDOWN],
 			name: "vinicunca/markdown/processor",
+			/**
+			* `eslint-plugin-markdown` only creates virtual files for code blocks,
+			* but not the markdown file itself. We use `eslint-merge-processors` to
+			* add a pass-through processor for the markdown file itself.
+			*/
 			processor: mergeProcessors([markdown.processors.markdown, processorPassThrough])
 		},
 		{
@@ -1182,7 +1197,8 @@ async function node() {
 *
 * @see https://github.com/azat-io/eslint-plugin-perfectionist
 */
-async function perfectionist() {
+async function perfectionist(options) {
+	const { overrides = {} } = options;
 	return [{
 		name: "vinicunca/perfectionist/rules",
 		plugins: { perfectionist: pluginPerfectionist },
@@ -1223,7 +1239,8 @@ async function perfectionist() {
 			"perfectionist/sort-named-imports": [ERROR, {
 				order: "asc",
 				type: "natural"
-			}]
+			}],
+			...overrides
 		}
 	}];
 }
@@ -1369,38 +1386,22 @@ const ReactRouterPackages = [
 	"@react-router/dev"
 ];
 const NextJsPackages = ["next"];
-const ReactCompilerPackages = ["babel-plugin-react-compiler"];
 async function react(options = {}) {
-	const { files = [GLOB_SRC], filesTypeAware = [GLOB_TS, GLOB_TSX], ignoresTypeAware = [`${GLOB_MARKDOWN}/**`, GLOB_ASTRO_TS], overrides = {}, reactCompiler = ReactCompilerPackages.some((i) => isPackageExists(i)), tsconfigPath } = options;
-	await ensurePackages([
-		"@eslint-react/eslint-plugin",
-		"eslint-plugin-react-hooks",
-		"eslint-plugin-react-refresh"
-	]);
+	const { files = [GLOB_SRC], filesTypeAware = [GLOB_TS, GLOB_TSX], ignoresTypeAware = [`${GLOB_MARKDOWN}/**`, GLOB_ASTRO_TS], overrides = {}, tsconfigPath } = options;
+	await ensurePackages(["@eslint-react/eslint-plugin", "eslint-plugin-react-refresh"]);
 	const isTypeAware = !!tsconfigPath;
-	const typeAwareRules = { "react/no-leaked-conditional-rendering": WARN };
-	const [pluginReact, pluginReactHooks, pluginReactRefresh] = await Promise.all([
-		interopDefault(import("@eslint-react/eslint-plugin")),
-		interopDefault(import("eslint-plugin-react-hooks")),
-		interopDefault(import("eslint-plugin-react-refresh"))
-	]);
+	const typeAwareRules = { "react/no-leaked-conditional-rendering": ERROR };
+	const [pluginReact, pluginReactRefresh] = await Promise.all([interopDefault(import("@eslint-react/eslint-plugin")), interopDefault(import("eslint-plugin-react-refresh"))]);
 	const isAllowConstantExport = ReactRefreshAllowConstantExportPackages.some((i) => isPackageExists(i));
 	const isUsingRemix = RemixPackages.some((i) => isPackageExists(i));
 	const isUsingReactRouter = ReactRouterPackages.some((i) => isPackageExists(i));
 	const isUsingNext = NextJsPackages.some((i) => isPackageExists(i));
-	const plugins = pluginReact.configs.all.plugins;
 	return [
 		{
 			name: "vinicunca/react/setup",
 			plugins: {
-				"react": plugins["@eslint-react"],
-				"react-dom": plugins["@eslint-react/dom"],
-				"react-hooks": pluginReactHooks,
-				"react-hooks-extra": plugins["@eslint-react/hooks-extra"],
-				"react-naming-convention": plugins["@eslint-react/naming-convention"],
-				"react-refresh": pluginReactRefresh,
-				"react-rsc": plugins["@eslint-react/rsc"],
-				"react-web-api": plugins["@eslint-react/web-api"]
+				"react": pluginReact.configs.all.plugins["@eslint-react"],
+				"react-refresh": pluginReactRefresh
 			}
 		},
 		{
@@ -1411,89 +1412,8 @@ async function react(options = {}) {
 			},
 			name: "vinicunca/react/rules",
 			rules: {
-				"react-dom/no-dangerously-set-innerhtml": WARN,
-				"react-dom/no-dangerously-set-innerhtml-with-children": ERROR,
-				"react-dom/no-find-dom-node": ERROR,
-				"react-dom/no-flush-sync": ERROR,
-				"react-dom/no-hydrate": ERROR,
-				"react-dom/no-missing-button-type": WARN,
-				"react-dom/no-missing-iframe-sandbox": WARN,
-				"react-dom/no-namespace": ERROR,
-				"react-dom/no-render": ERROR,
-				"react-dom/no-render-return-value": ERROR,
-				"react-dom/no-script-url": WARN,
-				"react-dom/no-unsafe-iframe-sandbox": WARN,
-				"react-dom/no-unsafe-target-blank": WARN,
-				"react-dom/no-use-form-state": ERROR,
-				"react-dom/no-void-elements-with-children": ERROR,
-				"react-hooks-extra/no-direct-set-state-in-use-effect": WARN,
-				"react-hooks/exhaustive-deps": WARN,
-				"react-hooks/rules-of-hooks": ERROR,
-				"react-naming-convention/context-name": WARN,
-				"react-naming-convention/ref-name": WARN,
-				"react-naming-convention/use-state": WARN,
-				"react-rsc/function-definition": ERROR,
-				"react-web-api/no-leaked-event-listener": WARN,
-				"react-web-api/no-leaked-interval": WARN,
-				"react-web-api/no-leaked-resize-observer": WARN,
-				"react-web-api/no-leaked-timeout": WARN,
-				"react/jsx-no-comment-textnodes": WARN,
-				"react/jsx-no-duplicate-props": WARN,
-				"react/jsx-uses-vars": WARN,
-				"react/no-access-state-in-setstate": ERROR,
-				"react/no-array-index-key": WARN,
-				"react/no-children-count": WARN,
-				"react/no-children-for-each": WARN,
-				"react/no-children-map": WARN,
-				"react/no-children-only": WARN,
-				"react/no-children-to-array": WARN,
-				"react/no-clone-element": WARN,
-				"react/no-component-will-mount": ERROR,
-				"react/no-component-will-receive-props": ERROR,
-				"react/no-component-will-update": ERROR,
-				"react/no-context-provider": WARN,
-				"react/no-create-ref": ERROR,
-				"react/no-default-props": ERROR,
-				"react/no-direct-mutation-state": ERROR,
-				"react/no-forward-ref": WARN,
-				"react/no-missing-key": ERROR,
-				"react/no-nested-component-definitions": ERROR,
-				"react/no-prop-types": ERROR,
-				"react/no-redundant-should-component-update": ERROR,
-				"react/no-set-state-in-component-did-mount": WARN,
-				"react/no-set-state-in-component-did-update": WARN,
-				"react/no-set-state-in-component-will-update": WARN,
-				"react/no-string-refs": ERROR,
-				"react/no-unnecessary-use-prefix": WARN,
-				"react/no-unsafe-component-will-mount": WARN,
-				"react/no-unsafe-component-will-receive-props": WARN,
-				"react/no-unsafe-component-will-update": WARN,
-				"react/no-unstable-context-value": WARN,
-				"react/no-unstable-default-props": WARN,
-				"react/no-unused-class-component-members": WARN,
-				"react/no-unused-state": WARN,
-				"react/no-use-context": WARN,
-				"react/no-useless-forward-ref": WARN,
-				"react/prefer-namespace-import": ERROR,
-				"react/prefer-use-state-lazy-initialization": WARN,
-				...reactCompiler ? {
-					"react-hooks/component-hook-factories": ERROR,
-					"react-hooks/config": ERROR,
-					"react-hooks/error-boundaries": ERROR,
-					"react-hooks/gating": ERROR,
-					"react-hooks/globals": ERROR,
-					"react-hooks/immutability": ERROR,
-					"react-hooks/incompatible-library": WARN,
-					"react-hooks/preserve-manual-memoization": ERROR,
-					"react-hooks/purity": ERROR,
-					"react-hooks/refs": ERROR,
-					"react-hooks/set-state-in-effect": ERROR,
-					"react-hooks/set-state-in-render": ERROR,
-					"react-hooks/static-components": ERROR,
-					"react-hooks/unsupported-syntax": WARN,
-					"react-hooks/use-memo": ERROR
-				} : {},
-				"react-refresh/only-export-components": [WARN, {
+				...pluginReact.configs.recommended.rules,
+				"react-refresh/only-export-components": [ERROR, {
 					allowConstantExport: isAllowConstantExport,
 					allowExportNames: [...isUsingNext ? [
 						"dynamic",
@@ -1503,12 +1423,13 @@ async function react(options = {}) {
 						"runtime",
 						"preferredRegion",
 						"maxDuration",
-						"config",
 						"generateStaticParams",
 						"metadata",
 						"generateMetadata",
 						"viewport",
-						"generateViewport"
+						"generateViewport",
+						"generateImageMetadata",
+						"generateSitemaps"
 					] : [], ...isUsingRemix || isUsingReactRouter ? [
 						"meta",
 						"links",
@@ -1528,12 +1449,8 @@ async function react(options = {}) {
 			files: filesTypeAware,
 			name: "vinicunca/react/typescript",
 			rules: {
-				"react-dom/no-string-style-prop": "off",
-				"react-dom/no-unknown-property": "off",
-				"react/jsx-no-duplicate-props": "off",
-				"react/jsx-no-undef": "off",
-				"react/jsx-uses-react": "off",
-				"react/jsx-uses-vars": "off"
+				"react/dom-no-string-style-prop": "off",
+				"react/dom-no-unknown-property": "off"
 			}
 		},
 		...isTypeAware ? [{
@@ -1615,11 +1532,13 @@ async function solid(options = {}) {
 //#region src/configs/sonar.ts
 async function sonar() {
 	return [{
+		files: [GLOB_SRC],
 		name: "vinicunca/sonar/rules",
 		plugins: { sonar: pluginSonar },
 		rules: {
-			...pluginSonar.configs.recommended.rules,
+			...(pluginSonar.configs?.recommended).rules,
 			"sonar/cognitive-complexity": "off",
+			"sonar/no-unused-import": "off",
 			"sonar/no-unused-vars": "off",
 			"sonar/pseudo-random": "off",
 			"sonar/slow-regex": "off",
@@ -2165,7 +2084,7 @@ async function typescript(options = {}) {
 		}] : [],
 		...erasableOnly ? [{
 			name: "antfu/typescript/erasable-syntax-only",
-			plugins: { "erasable-syntax-only": await interopDefault(import("./lib-D3Kr7UIJ.mjs")) },
+			plugins: { "erasable-syntax-only": await interopDefault(import("./lib-BXrNxJq9.mjs")) },
 			rules: {
 				"erasable-syntax-only/enums": ERROR,
 				"erasable-syntax-only/import-aliases": ERROR,
@@ -2221,7 +2140,7 @@ async function unocss(options = {}) {
 async function vue(options = {}) {
 	const { a11y = false, files = [GLOB_VUE], overrides = {}, stylistic = true } = options;
 	const sfcBlocks = options.sfcBlocks === true ? {} : options.sfcBlocks ?? {};
-	const { indent = 2 } = e$2(stylistic) ? {} : stylistic;
+	const { braceStyle = "stroustrup", indent = 2 } = e$2(stylistic) ? {} : stylistic;
 	if (a11y) await ensurePackages(["eslint-plugin-vuejs-accessibility"]);
 	const [pluginVue, parserVue, processorVueBlocks, pluginVueA11y] = await Promise.all([
 		interopDefault(import("eslint-plugin-vue")),
@@ -2363,7 +2282,7 @@ async function vue(options = {}) {
 				}],
 				"vue/brace-style": [
 					ERROR,
-					"stroustrup",
+					braceStyle,
 					{ allowSingleLine: true }
 				],
 				"vue/comma-dangle": [ERROR, "always-multiline"],
@@ -2486,9 +2405,6 @@ const VuePackages = [
 ];
 const defaultPluginRenaming = {
 	"@eslint-react": "react",
-	"@eslint-react/dom": "react-dom",
-	"@eslint-react/hooks-extra": "react-hooks-extra",
-	"@eslint-react/naming-convention": "react-naming-convention",
 	"@next/next": "next",
 	"@stylistic": "style",
 	"@typescript-eslint": "ts",
@@ -2509,7 +2425,7 @@ const defaultPluginRenaming = {
 *  The merged ESLint configurations.
 */
 function vinicuncaESLint(options = {}, ...userConfigs) {
-	const { astro: enableAstro = false, autoRenamePlugins = true, componentExts = [], e18e: enableE18e = true, gitignore: enableGitignore = true, ignores: userIgnores = [], imports: enableImports = true, jsdoc: enableJsdoc = true, jsx: enableJsx = true, nextjs: enableNextjs = false, node: enableNode = true, pnpm: enableCatalogs = !!findUpSync("pnpm-workspace.yaml"), react: enableReact = false, regexp: enableRegexp = true, solid: enableSolid = false, svelte: enableSvelte = false, type: appType = "app", typescript: enableTypeScript = isPackageExists("typescript"), unicorn: enableUnicorn = true, unocss: enableUnoCSS = false, vue: enableVue = VuePackages.some((i) => isPackageExists(i)) } = options;
+	const { astro: enableAstro = false, autoRenamePlugins = true, componentExts = [], e18e: enableE18e = true, gitignore: enableGitignore = true, ignores: userIgnores = [], imports: enableImports = true, jsdoc: enableJsdoc = true, jsx: enableJsx = true, nextjs: enableNextjs = false, node: enableNode = true, perfectionist: enablePerfectionist = true, pnpm: enableCatalogs = !!findUpSync("pnpm-workspace.yaml"), react: enableReact = false, regexp: enableRegexp = true, solid: enableSolid = false, svelte: enableSvelte = false, type: appType = "app", typescript: enableTypeScript = isPackageExists("typescript") || isPackageExists("@typescript/native-preview"), unicorn: enableUnicorn = true, unocss: enableUnoCSS = false, vue: enableVue = VuePackages.some((i) => isPackageExists(i)) } = options;
 	let isInEditor = options.isInEditor;
 	if (isInEditor == null) {
 		isInEditor = isInEditorEnv();
@@ -2528,10 +2444,11 @@ function vinicuncaESLint(options = {}, ...userConfigs) {
 	})]));
 	const typescriptOptions = resolveSubOptions(options, "typescript");
 	const tsconfigPath = "tsconfigPath" in typescriptOptions ? typescriptOptions.tsconfigPath : void 0;
-	configs.push(ignores(userIgnores), javascript({
+	configs.push(ignores(userIgnores, !enableTypeScript), javascript({
 		isInEditor,
 		overrides: getOverrides(options, "javascript")
-	}), comments(), imports({ stylistic: stylisticOptions }), command(), perfectionist(), sonar());
+	}), comments(), command(), sonar());
+	if (enablePerfectionist) configs.push(perfectionist({ overrides: getOverrides(options, "perfectionist") }));
 	if (enableNode) configs.push(node());
 	if (enableJsdoc) configs.push(jsdoc({ stylistic: stylisticOptions }));
 	if (enableImports) configs.push(imports({