@fabdeh/eslint-config 0.9.0-beta.4 → 0.9.1

package/dist/index.mjs

@@ -5,7 +5,6 @@ import process from "node:process";
 import { isPackageExists } from "local-pkg";
 import eslintComments from "@eslint-community/eslint-plugin-eslint-comments";
 import * as importX from "eslint-plugin-import-x";
-import eslint from "@eslint/js";
 import preferArrowFunctions from "eslint-plugin-prefer-arrow-functions";
 import unusedImports from "eslint-plugin-unused-imports";
 import globals from "globals";
@@ -15,16 +14,10 @@ import nodePlugin from "eslint-plugin-n";
 import perfectionistPlugin from "eslint-plugin-perfectionist";
 import { configs } from "eslint-plugin-regexp";
 import unicornPlugin from "eslint-plugin-unicorn";
-
-//#region src/shared/globs.ts
-/**
-* A glob pattern that matches the extension of JavaScript and TypeScript source files.
-*/
-const GLOB_SRC_EXT = "?([cm])[jt]s?(x)";
 /**
 * A glob pattern that matches JavaScript and TypeScript source files (including JSX/TSX).
 */
-const GLOB_SRC = `**/*.${GLOB_SRC_EXT}`;
+const GLOB_SRC = `**/*.?([cm])[jt]s?(x)`;
 /**
 * A glob pattern that matches JavaScript source files (including JSX).
 */
@@ -147,7 +140,6 @@ const GLOB_EXCLUDE = [
 	"**/components.d.ts",
 	"**/prettier-types.ts"
 ];
-
 //#endregion
 //#region src/shared/utils.ts
 const SCOPE_URL = import.meta.dirname;
@@ -253,7 +245,6 @@ function resolveSubOptions(options, key) {
 	if (typeof option === "boolean") return {};
 	return option ?? {};
 }
-
 //#endregion
 //#region src/configs/angular.ts
 /**
@@ -359,7 +350,6 @@ async function angular(options = {}) {
 		}
 	});
 }
-
 //#endregion
 //#region src/configs/comments.ts
 /**
@@ -386,7 +376,6 @@ function comments() {
 		}
 	});
 }
-
 //#endregion
 //#region src/configs/stylistic.ts
 const STYLISTIC_CONFIG_DEFAULT = {
@@ -432,17 +421,16 @@ async function stylistic(options = {}) {
 		}
 	});
 }
-
 //#endregion
 //#region src/configs/formatters.ts
 /**
 * Merges the provided Prettier options with any overrides.
 *
 * @param options - The base Prettier options to merge.
-* @param overrides - Optional overrides for the Prettier options.
+* @param overrides - Required overrides for the Prettier options. Must include a `parser` property.
 * @returns The merged Prettier options.
 */
-function mergePrettierOptions(options, overrides = {}) {
+function mergePrettierOptions(options, overrides) {
 	return {
 		...options,
 		...overrides,
@@ -577,7 +565,6 @@ async function formatters(options = {}, stylistic = {}, hasAngularTemplateParser
 	});
 	return tseslint.config(configs);
 }
-
 //#endregion
 //#region src/configs/ignore.ts
 /**
@@ -593,7 +580,6 @@ function ignores(userIgnores = [], fromFactory) {
 		ignores: [...!fromFactory || fromFactory === "workspace" ? GLOB_EXCLUDE : [], ...userIgnores]
 	});
 }
-
 //#endregion
 //#region src/configs/imports.ts
 /**
@@ -641,7 +627,6 @@ function imports(options = {}) {
 		}
 	});
 }
-
 //#endregion
 //#region src/configs/javascript.ts
 /**
@@ -664,7 +649,7 @@ function javascript(options = {}) {
 	return tseslint.config({
 		name: "fabdeh/javascript/setup",
 		languageOptions: {
-			ecmaVersion: 2022,
+			ecmaVersion: "latest",
 			globals: {
 				...globals.browser,
 				...globals.es2021,
@@ -675,7 +660,7 @@ function javascript(options = {}) {
 			},
 			parserOptions: {
 				ecmaFeatures: { jsx: true },
-				ecmaVersion: 2022,
+				ecmaVersion: "latest",
 				sourceType: "module"
 			},
 			sourceType: "module"
@@ -690,13 +675,15 @@ function javascript(options = {}) {
 			"unused-imports": unusedImports
 		},
 		rules: {
-			...eslint.configs.recommended.rules,
 			"accessor-pairs": "error",
 			"array-callback-return": "error",
 			"block-scoped-var": "error",
+			"constructor-super": "error",
 			"default-case-last": "error",
 			"dot-notation": "error",
 			eqeqeq: "error",
+			"for-direction": "error",
+			"getter-return": "error",
 			"id-denylist": [
 				"error",
 				"any",
@@ -711,48 +698,156 @@ function javascript(options = {}) {
 			],
 			"new-cap": "error",
 			"no-alert": "error",
+			"no-async-promise-executor": "error",
 			"no-caller": "error",
+			"no-case-declarations": "error",
+			"no-class-assign": "error",
 			"no-cond-assign": ["error", "always"],
+			"no-compare-neg-zero": "error",
 			"no-console": ["error", { allow: ["warn", "error"] }],
+			"no-const-assign": "error",
+			"no-constant-binary-expression": "error",
+			"no-constant-condition": "error",
+			"no-control-regex": "error",
+			"no-debugger": "error",
+			"no-delete-var": "error",
+			"no-dupe-args": "error",
+			"no-dupe-class-members": "error",
+			"no-dupe-else-if": "error",
+			"no-dupe-keys": "error",
+			"no-duplicate-case": "error",
 			"no-empty": ["error", { allowEmptyCatch: true }],
+			"no-empty-character-class": "error",
+			"no-empty-pattern": "error",
+			"no-empty-static-block": "error",
 			"no-eval": "error",
+			"no-ex-assign": "error",
 			"no-extend-native": "error",
 			"no-extra-bind": "error",
+			"no-extra-boolean-cast": "error",
+			"no-fallthrough": "error",
+			"no-func-assign": "error",
+			"no-global-assign": "error",
+			"no-import-assign": "error",
 			"no-implied-eval": "error",
+			"no-invalid-regexp": "error",
+			"no-irregular-whitespace": "error",
 			"no-iterator": "error",
 			"no-labels": "error",
 			"no-lone-blocks": "error",
 			"no-lonely-if": "error",
+			"no-loss-of-precision": "error",
+			"no-misleading-character-class": "error",
 			"no-multi-str": "error",
 			"no-new": "error",
 			"no-new-func": "error",
+			"no-new-native-nonconstructor": "error",
 			"no-new-wrappers": "error",
+			"no-nonoctal-decimal-escape": "error",
+			"no-obj-calls": "error",
+			"no-octal": "error",
 			"no-octal-escape": "error",
 			"no-plusplus": ["error", { allowForLoopAfterthoughts: true }],
+			"no-prototype-builtins": "error",
 			"no-proto": "error",
+			"no-redeclare": "error",
+			"no-regex-spaces": "error",
+			"no-restricted-globals": [
+				"error",
+				{
+					message: "Use `globalThis` instead.",
+					name: "global"
+				},
+				{
+					message: "Use `globalThis` instead.",
+					name: "self"
+				}
+			],
+			"no-restricted-properties": [
+				"error",
+				{
+					message: "Use `Object.getPrototypeOf` or `Object.setPrototypeOf` instead.",
+					property: "__proto__"
+				},
+				{
+					message: "Use `Object.defineProperty` instead.",
+					property: "__defineGetter__"
+				},
+				{
+					message: "Use `Object.defineProperty` instead.",
+					property: "__defineSetter__"
+				},
+				{
+					message: "Use `Object.getOwnPropertyDescriptor` instead.",
+					property: "__lookupGetter__"
+				},
+				{
+					message: "Use `Object.getOwnPropertyDescriptor` instead.",
+					property: "__lookupSetter__"
+				}
+			],
+			"no-restricted-syntax": [
+				"error",
+				{
+					selector: "TSEnumDeclaration",
+					message: "Use `as const` objects instead of enums."
+				},
+				{
+					selector: "TSExportAssignment",
+					message: "Avoid `export =`. Use ESM-style exports instead (for example, `export default` or named exports)."
+				},
+				{
+					selector: "ForInStatement",
+					message: "Avoid using `for...in` statements. Use `Object.keys()`, `Object.values()`, or `Object.entries()` instead."
+				},
+				{
+					selector: ":matches(PropertyDefinition, MethodDefinition)[accessibility=\"private\"]",
+					message: "Use `#private` members instead."
+				}
+			],
+			"no-self-assign": "error",
 			"no-self-compare": "error",
 			"no-sequences": "error",
+			"no-setter-return": "error",
+			"no-shadow-restricted-names": "error",
+			"no-sparse-arrays": "error",
 			"no-template-curly-in-string": "error",
+			"no-this-before-super": "error",
 			"no-throw-literal": "error",
+			"no-undef": "error",
 			"no-undef-init": "error",
+			"no-unassigned-vars": "error",
+			"no-unexpected-multiline": "error",
 			"no-unmodified-loop-condition": "error",
 			"no-unneeded-ternary": ["error", { defaultAssignment: false }],
+			"no-unreachable": "error",
 			"no-unreachable-loop": "error",
+			"no-unsafe-finally": "error",
+			"no-unsafe-negation": "error",
+			"no-unsafe-optional-chaining": "error",
 			"no-unused-expressions": ["error", {
 				allowShortCircuit: true,
 				allowTaggedTemplates: true,
 				allowTernary: true
 			}],
+			"no-unused-labels": "error",
+			"no-unused-private-class-members": "error",
+			"no-unused-vars": "off",
 			"no-use-before-define": ["error", {
 				classes: false,
 				functions: false
 			}],
+			"no-useless-assignment": "error",
+			"no-useless-backreference": "error",
 			"no-useless-call": "error",
+			"no-useless-catch": "error",
 			"no-useless-computed-key": "error",
 			"no-useless-constructor": "error",
+			"no-useless-escape": "error",
 			"no-useless-rename": "error",
 			"no-useless-return": "error",
 			"no-var": "error",
+			"no-with": "error",
 			"object-shorthand": [
 				"error",
 				"always",
@@ -774,10 +869,12 @@ function javascript(options = {}) {
 			"prefer-rest-params": "error",
 			"prefer-spread": "error",
 			"prefer-template": "error",
+			"preserve-caught-error": "error",
+			"require-yield": "error",
 			"symbol-description": "error",
 			"unicode-bom": ["error", "never"],
+			"use-isnan": ["error", { enforceForIndexOf: true }],
 			"unused-imports/no-unused-imports": "error",
-			"no-unused-vars": "off",
 			"unused-imports/no-unused-vars": ["error", {
 				args: "after-used",
 				argsIgnorePattern: "^_",
@@ -785,7 +882,6 @@ function javascript(options = {}) {
 				vars: "all",
 				varsIgnorePattern: "^_"
 			}],
-			"use-isnan": ["error", { enforceForIndexOf: true }],
 			"valid-typeof": ["error", { requireStringLiterals: true }],
 			"vars-on-top": "error",
 			yoda: [
@@ -803,7 +899,6 @@ function javascript(options = {}) {
 		}
 	});
 }
-
 //#endregion
 //#region src/configs/jsdoc.ts
 /**
@@ -889,7 +984,6 @@ function jsdoc(options = {}) {
 		rules: getJsDocRules("warn", !!stylistic, "tsOnly")
 	});
 }
-
 //#endregion
 //#region src/configs/jsonc.ts
 /**
@@ -974,7 +1068,6 @@ async function jsonc(options = {}) {
 		}
 	});
 }
-
 //#endregion
 //#region src/configs/markdown.ts
 /**
@@ -1070,7 +1163,6 @@ async function markdown(options = {}) {
 		}
 	});
 }
-
 //#endregion
 //#region src/configs/rules-configs/naming-convention.ts
 /**
@@ -1144,7 +1236,6 @@ function namingConvention(strict, allowJsx = false) {
 		}
 	];
 }
-
 //#endregion
 //#region src/configs/ngrx.ts
 const DEFAULT_STORE_GLOB = [
@@ -1292,7 +1383,6 @@ async function ngrx(options = {}) {
 	});
 	return tseslint.config(configs);
 }
-
 //#endregion
 //#region src/configs/node.ts
 /**
@@ -1320,7 +1410,6 @@ function node() {
 		}
 	});
 }
-
 //#endregion
 //#region src/configs/rules-configs/perfectionist-groups.ts
 const SORT_IMPORT_GROUPS = [
@@ -1366,7 +1455,6 @@ const SORT_UNION_OR_INTERSECTION_GROUPS = [
 	"nullish",
 	"unknown"
 ];
-
 //#endregion
 //#region src/configs/perfectionist.ts
 /**
@@ -1426,7 +1514,6 @@ function perfectionist() {
 		}
 	});
 }
-
 //#endregion
 //#region src/configs/pnpm.ts
 /**
@@ -1461,7 +1548,6 @@ async function pnpm() {
 		}
 	});
 }
-
 //#endregion
 //#region src/configs/regexp.ts
 /**
@@ -1484,7 +1570,6 @@ function regexp(options = {}) {
 		}
 	});
 }
-
 //#endregion
 //#region src/configs/sort.ts
 /**
@@ -1721,7 +1806,6 @@ function sortTsConfig() {
 		] }
 	});
 }
-
 //#endregion
 //#region src/configs/tailwindcss.ts
 /**
@@ -1740,7 +1824,7 @@ function sortTsConfig() {
 async function tailwindcss(options = {}) {
 	await ensurePackages(["eslint-plugin-better-tailwindcss"]);
 	const betterTailwindcssPlugin = await interopDefault(import("eslint-plugin-better-tailwindcss"));
-	const { files: filesGlob, overrides, enableAllRules, parsers, ...settings } = options;
+	const { files: filesGlob, overrides, enableAllRules, parsers, stylistic = true, ...settings } = options;
 	let files;
 	let parserConfigs;
 	if (parsers) {
@@ -1751,7 +1835,7 @@ async function tailwindcss(options = {}) {
 			languageOptions: { parser }
 		}));
 	} else {
-		files = { files: filesGlob ?? [GLOB_SRC, GLOB_HTML] };
+		files = { files: filesGlob ?? [GLOB_SRC, "**/*.htm?(l)"] };
 		parserConfigs = [];
 	}
 	return tseslint.config(...parserConfigs, {
@@ -1760,7 +1844,8 @@ async function tailwindcss(options = {}) {
 		plugins: { "better-tailwindcss": betterTailwindcssPlugin },
 		settings: { "better-tailwindcss": settings },
 		rules: {
-			...betterTailwindcssPlugin.configs.recommended.rules,
+			...betterTailwindcssPlugin.configs.correctness.rules,
+			...stylistic || enableAllRules ? betterTailwindcssPlugin.configs.stylistic.rules : {},
 			...enableAllRules ? {
 				"better-tailwindcss/enforce-consistent-variable-syntax": "warn",
 				"better-tailwindcss/no-conflicting-classes": "error",
@@ -1769,7 +1854,6 @@ async function tailwindcss(options = {}) {
 		}
 	});
 }
-
 //#endregion
 //#region src/configs/toml.ts
 /**
@@ -1820,7 +1904,6 @@ async function toml(options = {}) {
 		}
 	});
 }
-
 //#endregion
 //#region src/configs/rules-configs/member-ordering.ts
 const MEMBER_ORDERING_OPTIONS = { default: [
@@ -1857,7 +1940,6 @@ const MEMBER_ORDERING_OPTIONS = { default: [
 	"private-method",
 	"#private-method"
 ] };
-
 //#endregion
 //#region src/configs/typescript.ts
 /**
@@ -1986,7 +2068,6 @@ async function typescript(options = {}, isWorkspaceProject = false) {
 		}
 	});
 }
-
 //#endregion
 //#region src/configs/unicorn.ts
 /**
@@ -2099,7 +2180,6 @@ function unicorn(options = {}) {
 		} }
 	});
 }
-
 //#endregion
 //#region src/configs/vitest.ts
 /**
@@ -2175,7 +2255,6 @@ async function vitest(options = {}) {
 		}
 	});
 }
-
 //#endregion
 //#region src/configs/yaml.ts
 /**
@@ -2229,7 +2308,6 @@ async function yaml(options = {}) {
 		}
 	});
 }
-
 //#endregion
 //#region src/shared/constants.ts
 const NGRX_PACKAGES = [
@@ -2239,7 +2317,6 @@ const NGRX_PACKAGES = [
 	"@ngrx/operators"
 ];
 const OPTIONS_SYMBOL = Symbol("options");
-
 //#endregion
 //#region src/factories/project-config.ts
 /**
@@ -2264,11 +2341,9 @@ async function defineProjectConfig(baseConfig, options = {}, ...userConfigs) {
 	const { angular: enableAngular = isPackageExists("@angular/core"), jsdoc: enableJsdoc = options.type === "lib", ngrx: enableNgrx = NGRX_PACKAGES.some((p) => isPackageExists(p)), tailwindcss: enableTailwind = false, typescript: typescriptOptions, vitest: enableVitest = isPackageExists("vitest") } = options;
 	if (enableNgrx && !enableAngular) throw new Error("NgRx rules can only be enabled if Angular rules are also enabled.");
 	const configs = [];
+	const stylisticOptions = workspaceOptions.stylistic === false ? false : typeof workspaceOptions.stylistic === "object" ? workspaceOptions.stylistic : {};
 	if (options.ignores && options.ignores.length > 0) configs.push(ignores(options.ignores, "project"));
-	if (enableJsdoc) {
-		const stylisticOptions = workspaceOptions.stylistic === false ? false : typeof workspaceOptions.stylistic === "object" ? workspaceOptions.stylistic : {};
-		configs.push(jsdoc({ stylistic: stylisticOptions }));
-	}
+	if (enableJsdoc) configs.push(jsdoc({ stylistic: stylisticOptions }));
 	if (typescriptOptions) {
 		if (!workspaceOptions.typescript) throw new Error("Project-specific TypeScript configuration requires a base configuration with TypeScript support.");
 		configs.push(typescript({
@@ -2298,11 +2373,13 @@ async function defineProjectConfig(baseConfig, options = {}, ...userConfigs) {
 	}
 	if (enableTailwind) {
 		const tailwindcssOptions = resolveSubOptions(options, "tailwindcss");
-		configs.push(tailwindcss(tailwindcssOptions));
+		configs.push(tailwindcss({
+			...tailwindcssOptions,
+			stylistic: stylisticOptions
+		}));
 	}
 	return tseslint.config(...resolvedBaseConfig, ...await Promise.all(configs), ...await Promise.all(userConfigs));
 }
-
 //#endregion
 //#region src/factories/standard-config.ts
 /**
@@ -2367,7 +2444,10 @@ async function defineConfig(options = {}, ...userConfigs) {
 	}
 	if (enableTailwind) {
 		const tailwindcssOptions = resolveSubOptions(options, "tailwindcss");
-		configs.push(tailwindcss(tailwindcssOptions));
+		configs.push(tailwindcss({
+			...tailwindcssOptions,
+			stylistic: stylisticOptions
+		}));
 	}
 	if (options.jsonc ?? true) {
 		const jsoncOptions = resolveSubOptions(options, "jsonc");
@@ -2398,7 +2478,6 @@ async function defineConfig(options = {}, ...userConfigs) {
 	if (options.formatters) configs.push(formatters(options.formatters, typeof stylisticOptions === "boolean" ? {} : stylisticOptions, Boolean(enableAngular)));
 	return tseslint.config(...await Promise.all(configs), ...await Promise.all(userConfigs));
 }
-
 //#endregion
 //#region src/factories/workspace-config.ts
 /**
@@ -2475,6 +2554,5 @@ async function defineWorkspaceConfig(options = {}, ...userConfigs) {
 	eslintConfigWithOptions[OPTIONS_SYMBOL] = options;
 	return eslintConfigWithOptions;
 }
-
 //#endregion
-export { GLOB_HTML, GLOB_JS, GLOB_SRC, GLOB_TESTS, GLOB_TS, STYLISTIC_CONFIG_DEFAULT, angular, comments, defineConfig, defineProjectConfig, defineWorkspaceConfig, ensurePackages, formatters, getTsConfigFileName, getWorkspaceRoot, ignores, imports, interopDefault, isPackageInScope, javascript, jsdoc, jsonc, markdown, ngrx, node, perfectionist, pnpm, regexp, resolveSubOptions, sortPackageJson, sortTsConfig, stylistic, tailwindcss, toml, typescript, unicorn, vitest, yaml };
+export { GLOB_HTML, GLOB_JS, GLOB_SRC, GLOB_TESTS, GLOB_TS, STYLISTIC_CONFIG_DEFAULT, angular, comments, defineConfig, defineProjectConfig, defineWorkspaceConfig, ensurePackages, formatters, getTsConfigFileName, getWorkspaceRoot, ignores, imports, interopDefault, isPackageInScope, javascript, jsdoc, jsonc, markdown, ngrx, node, perfectionist, pnpm, regexp, resolveSubOptions, sortPackageJson, sortTsConfig, stylistic, tailwindcss, toml, typescript, unicorn, vitest, yaml };