@zayne-labs/eslint-config 0.9.17 → 0.9.18

package/dist/index.js

@@ -973,7 +973,7 @@ const defaultPluginRenameMap = defineEnum({
 	"@eslint-react/hooks-extra": "react-hooks-extra",
 	"@eslint-react/naming-convention": "react-naming-convention",
 	"@eslint-react/web-api": "react-web-api",
-	"@eslint-react": "react",
+	"@eslint-react": "react-x",
 	"@next/next": "nextjs",
 	"@stylistic": "stylistic",
 	"@tanstack/query": "tanstack-query",
@@ -1024,72 +1024,71 @@ const isAllowConstantExport = ReactRefreshAllowConstantExportPackages.some((i) =
 const isUsingReactRouter = ReactRouterPackages.some((i) => isPackageExists(i));
 const isUsingNext = NextJsPackages.some((i) => isPackageExists(i));
 const react = async (options = {}) => {
-	const { compiler = false, files = [GLOB_SRC], nextjs = isUsingNext, overrides, react: enableReact = true, refresh = true, typescript: typescript$1 = true } = options;
+	const { compiler = false, files = [GLOB_SRC], filesTypeAware = [GLOB_TS, GLOB_TSX], ignoresTypeAware = [`${GLOB_MARKDOWN}/**`, GLOB_ASTRO_TS], nextjs = isUsingNext, overrides, react: enableReact = true, refresh = true, typescript: typescript$1 = true, youMightNotNeedAnEffect = true } = options;
 	await ensurePackages([
 		"@eslint-react/eslint-plugin",
 		"eslint-plugin-react-hooks",
 		refresh ? "eslint-plugin-react-refresh" : void 0,
+		youMightNotNeedAnEffect ? "eslint-plugin-react-you-might-not-need-an-effect" : void 0,
 		nextjs ? "@next/eslint-plugin-next" : void 0
 	]);
-	const [eslintPluginReact, eslintReactHooks, eslintPluginReactRefresh, eslintPluginNextjs] = await Promise.all([
+	const [eslintPluginReact, eslintReactHooks, eslintPluginReactRefresh, eslintPluginReactYouMightNotNeedAnEffect, eslintPluginNextjs] = await Promise.all([
 		enableReact ? interopDefault(import("@eslint-react/eslint-plugin")) : void 0,
 		enableReact ? interopDefault(import("eslint-plugin-react-hooks")) : void 0,
 		refresh ? interopDefault(import("eslint-plugin-react-refresh")) : void 0,
+		youMightNotNeedAnEffect ? interopDefault(import("./src-DwSufEpw.js")) : void 0,
 		nextjs ? interopDefault(import("@next/eslint-plugin-next")) : void 0
 	]);
 	const recommendedReactConfig = eslintPluginReact?.configs[typescript$1 ? "recommended-type-checked" : "recommended"];
 	const config = [];
 	if (enableReact && recommendedReactConfig && eslintReactHooks) config.push({
+		files,
+		languageOptions: { parserOptions: {
+			ecmaFeatures: { jsx: true },
+			sourceType: "module"
+		} },
 		name: "zayne/react/setup",
 		plugins: {
 			...renamePlugins(recommendedReactConfig.plugins, defaultPluginRenameMap),
-			"react-hooks": eslintReactHooks
+			"react-hooks": eslintReactHooks,
+			"react-you-might-not-need-an-effect": eslintPluginReactYouMightNotNeedAnEffect
 		},
 		settings: recommendedReactConfig.settings
 	}, {
-		files,
-		languageOptions: { parserOptions: {
-			ecmaFeatures: { jsx: true },
-			sourceType: "module",
-			...typescript$1 && { parser: (await interopDefault(import("typescript-eslint"))).parser }
-		} },
-		name: "zayne/react/setup-processor"
-	}, {
-		files,
-		name: "zayne/react/recommended",
+		files: typescript$1 ? filesTypeAware : files,
+		...typescript$1 && { ignores: ignoresTypeAware },
+		name: `zayne/react/${typescript$1 ? "recommended-type-checked" : "recommended"}`,
 		rules: renameRules(recommendedReactConfig.rules, defaultPluginRenameMap)
 	}, {
 		files,
 		name: "zayne/react/rules",
 		rules: {
-			"react-hooks-extra/no-unnecessary-use-callback": "warn",
-			"react-hooks-extra/no-unnecessary-use-memo": "warn",
-			"react-hooks-extra/no-unnecessary-use-prefix": "error",
-			"react-hooks-extra/prefer-use-state-lazy-initialization": "error",
 			"react-hooks/exhaustive-deps": "warn",
 			"react-hooks/rules-of-hooks": "error",
 			"react-naming-convention/component-name": "warn",
 			"react-naming-convention/use-state": "off",
-			"react/avoid-shorthand-boolean": "error",
-			"react/function-component-definition": "off",
-			"react/no-array-index-key": "error",
-			"react/no-children-count": "off",
-			"react/no-children-only": "off",
-			"react/no-children-prop": "error",
-			"react/no-clone-element": "off",
-			"react/no-complex-conditional-rendering": "warn",
-			"react/no-missing-component-display-name": "error",
-			"react/no-useless-fragment": "error",
-			"react/prefer-destructuring-assignment": "error",
-			"react/prefer-read-only-props": "off",
-			"react/prefer-shorthand-fragment": "error",
+			"react-x/jsx-shorthand-boolean": "error",
+			"react-x/jsx-shorthand-fragment": "error",
+			"react-x/no-array-index-key": "error",
+			"react-x/no-children-count": "off",
+			"react-x/no-children-only": "off",
+			"react-x/no-children-prop": "error",
+			"react-x/no-clone-element": "off",
+			"react-x/no-missing-component-display-name": "error",
+			"react-x/no-unnecessary-use-callback": "warn",
+			"react-x/no-unnecessary-use-memo": "warn",
+			"react-x/no-unnecessary-use-prefix": "error",
+			"react-x/no-useless-fragment": "error",
+			"react-x/prefer-destructuring-assignment": "error",
+			"react-x/prefer-read-only-props": "off",
+			"react-x/prefer-use-state-lazy-initialization": "error",
 			...overrides,
 			...isObject$1(enableReact) && enableReact.overrides
 		}
 	});
 	if (refresh && eslintPluginReactRefresh) config.push({
 		files,
-		name: "zayne/react/refresh",
+		name: "zayne/react/refresh/rules",
 		plugins: { "react-refresh": eslintPluginReactRefresh },
 		rules: {
 			"react-refresh/only-export-components": ["warn", {
@@ -1100,9 +1099,22 @@ const react = async (options = {}) => {
 			...isObject$1(refresh) && refresh.overrides
 		}
 	});
+	if (youMightNotNeedAnEffect && eslintPluginReactYouMightNotNeedAnEffect) config.push({
+		files,
+		name: "zayne/react/you-might-not-need-an-effect/recommended",
+		plugins: { "react-you-might-not-need-an-effect": eslintPluginReactYouMightNotNeedAnEffect },
+		rules: eslintPluginReactYouMightNotNeedAnEffect.configs.recommended.rules
+	}, {
+		files,
+		name: "zayne/react/you-might-not-need-an-effect/rules",
+		rules: {
+			...overrides,
+			...isObject$1(youMightNotNeedAnEffect) && youMightNotNeedAnEffect.overrides
+		}
+	});
 	if (compiler) config.push({
 		files,
-		name: "zayne/react/compiler",
+		name: "zayne/react/compiler/rules",
 		rules: {
 			"react-hooks/react-compiler": "error",
 			...overrides,
@@ -1111,14 +1123,19 @@ const react = async (options = {}) => {
 	});
 	if (nextjs && eslintPluginNextjs) config.push({
 		files,
-		name: "zayne/react/nextjs",
+		name: "zayne/react/nextjs/recommended",
 		plugins: { nextjs: eslintPluginNextjs },
 		rules: renameRules({
 			...eslintPluginNextjs.configs?.recommended?.rules,
-			...eslintPluginNextjs.configs?.["core-web-vitals"]?.rules,
+			...eslintPluginNextjs.configs?.["core-web-vitals"]?.rules
+		}, defaultPluginRenameMap)
+	}, {
+		files,
+		name: "zayne/react/nextjs/rules",
+		rules: {
 			...overrides,
 			...isObject$1(nextjs) && nextjs.overrides
-		}, defaultPluginRenameMap)
+		}
 	});
 	return config;
 };
@@ -1537,7 +1554,7 @@ const typescript = async (options = {}) => {
 		GLOB_TS,
 		GLOB_TSX,
 		...componentExts.map((ext) => `**/*.${ext}`)
-	], ignoresTypeAware = [`${GLOB_MARKDOWN}/**`, GLOB_ASTRO_TS], tsconfigPath = true, isTypeAware = Boolean(tsconfigPath), overrides, overridesTypeAware, parserOptions, stylistic: stylistic$1 = true } = options;
+	], ignoresTypeAware = [`${GLOB_MARKDOWN}/**`, GLOB_ASTRO_TS], tsconfigPath = true, isTypeAware = Boolean(tsconfigPath), overrides, parserOptions, stylistic: stylistic$1 = true } = options;
 	const tsEslint = await interopDefault(import("typescript-eslint"));
 	const projectServiceObject = isTypeAware && (allowDefaultProject ? {
 		projectService: {
@@ -1578,14 +1595,13 @@ const typescript = async (options = {}) => {
 	};
 	return [
 		{
-			name: `zayne/ts-eslint/${isTypeAware ? "type-aware-setup" : "setup"}`,
-			...makeParser(files),
-			...isTypeAware && makeParser(filesTypeAware, ignoresTypeAware)
+			name: `zayne/ts-eslint/${isTypeAware ? "setup-type-aware" : "setup"}`,
+			...isTypeAware ? makeParser(filesTypeAware, ignoresTypeAware) : makeParser(files)
 		},
 		...renamePluginInConfigs({
 			configArray: tsEslint.configs[selectedBaseRuleSet],
 			overrides: {
-				files,
+				files: isTypeAware ? filesTypeAware : files,
 				name: `zayne/ts-eslint/${selectedBaseRuleSet}`
 			},
 			renameMap: { "@typescript-eslint": "ts-eslint" }
@@ -1593,13 +1609,13 @@ const typescript = async (options = {}) => {
 		...stylistic$1 ? renamePluginInConfigs({
 			configArray: tsEslint.configs[selectedStylisticRuleSet],
 			overrides: {
-				files,
+				files: isTypeAware ? filesTypeAware : files,
 				name: `zayne/ts-eslint/${selectedStylisticRuleSet}`
 			},
 			renameMap: { "@typescript-eslint": "ts-eslint" }
 		}) : [],
 		{
-			files,
+			files: isTypeAware ? filesTypeAware : files,
 			name: "zayne/ts-eslint/rules",
 			rules: {
 				"ts-eslint/array-type": ["error", { default: "array-simple" }],
@@ -1632,18 +1648,10 @@ const typescript = async (options = {}) => {
 				"ts-eslint/no-use-before-define": "off",
 				"ts-eslint/no-useless-constructor": "error",
 				"ts-eslint/no-useless-empty-export": "error",
+				...isTypeAware && typeAwareRules,
 				...overrides
 			}
-		},
-		isTypeAware ? {
-			files: filesTypeAware,
-			ignores: ignoresTypeAware,
-			name: "zayne/ts-eslint/rules-type-aware",
-			rules: {
-				...typeAwareRules,
-				...overridesTypeAware
-			}
-		} : {}
+		}
 	];
 };
 
@@ -1685,10 +1693,11 @@ const vue = async (options = {}) => {
 		"vue-eslint-parser",
 		sfcBlocks ? "eslint-processor-vue-blocks" : void 0
 	]);
-	const [pluginVue, parserVue, processorVueBlocks] = await Promise.all([
+	const [pluginVue, parserVue, processorVueBlocks, tsEslint] = await Promise.all([
 		interopDefault(import("eslint-plugin-vue")),
 		interopDefault(import("vue-eslint-parser")),
-		sfcBlocks ? interopDefault(import("eslint-processor-vue-blocks")) : void 0
+		sfcBlocks ? interopDefault(import("eslint-processor-vue-blocks")) : void 0,
+		typescript$1 ? interopDefault(import("typescript-eslint")) : void 0
 	]);
 	return [
 		{
@@ -1719,7 +1728,7 @@ const vue = async (options = {}) => {
 					ecmaFeatures: { jsx: true },
 					extraFileExtensions: [".vue"],
 					sourceType: "module",
-					...typescript$1 && { parser: (await interopDefault(import("typescript-eslint"))).parser }
+					...typescript$1 && { parser: tsEslint?.parser }
 				}
 			},
 			name: "zayne/vue/setup/file-processor",
@@ -1943,8 +1952,8 @@ const zayne = (options = {}, ...userConfigs) => {
 		componentExts,
 		isTypeAware,
 		stylistic: isStylistic,
-		tsconfigPath,
-		...resolveOptions(enableTypeScript)
+		...resolveOptions(enableTypeScript),
+		tsconfigPath
 	}));
 	if (enableStylistic) configs.push(stylistic({
 		jsx: enableJsx,
@@ -2002,7 +2011,7 @@ const zayne = (options = {}, ...userConfigs) => {
 		typescript: isTypeAware,
 		...resolveOptions(restOfOptions.astro)
 	}));
-	if (restOfOptions.expo) configs.push(expo());
+	if (restOfOptions.expo) configs.push(expo(resolveOptions(restOfOptions.expo)));
 	if (restOfOptions.tailwindcssBetter) configs.push(tailwindcssBetter(resolveOptions(restOfOptions.tailwindcssBetter)));
 	if (restOfOptions.tanstack) configs.push(tanstack(resolveOptions(restOfOptions.tanstack)));
 	if (restOfOptions.depend) configs.push(depend(resolveOptions(restOfOptions.depend)));