@fabdeh/eslint-config 0.12.1 → 0.13.0

package/dist/index.mjs

@@ -14,7 +14,7 @@ import { mergeProcessors, processorPassThrough } from "eslint-merge-processors";
 import nodePlugin from "eslint-plugin-n";
 import perfectionistPlugin from "eslint-plugin-perfectionist";
 import { readFile } from "node:fs/promises";
-import { findUp, findUpSync } from "find-up-simple";
+import { up } from "empathic/find";
 import { configs } from "eslint-plugin-regexp";
 import unicornPlugin from "eslint-plugin-unicorn";
 import { FlatConfigComposer } from "eslint-flat-config-utils";
@@ -349,7 +349,8 @@ function getPlaywrightDirectory() {
 		const match = /testDir:\s*["'](.+)["'],?/.exec(configContent);
 		if (match) {
 			const testDir = match[1];
-			const resolvedTestDir = resolve(dirname(config), testDir);
+			const configDir = dirname(config);
+			const resolvedTestDir = resolve(configDir, testDir);
 			if (isDirectory(resolvedTestDir)) return resolvedTestDir;
 		}
 	} catch {}
@@ -396,6 +397,7 @@ async function angular(options = {}) {
 				"Output",
 				"Pipe",
 				"Self",
+				"Service",
 				"SkipSelf",
 				"ViewChild",
 				"ViewChildren"
@@ -1519,7 +1521,7 @@ async function playwright(options = {}) {
 * @returns `true` when catalog configuration is present; otherwise `false`.
 */
 async function detectCatalogUsage() {
-	const workspaceFile = await findUp("pnpm-workspace.yaml");
+	const workspaceFile = up("pnpm-workspace.yaml");
 	if (!workspaceFile) return false;
 	const yaml = await readFile(workspaceFile, "utf-8");
 	return yaml.includes("catalog:") || yaml.includes("catalogs:");
@@ -2094,7 +2096,7 @@ const MEMBER_ORDERING_OPTIONS = { default: [
 * @returns A TypedConfig[] containing the TypeScript ESLint configuration.
 */
 async function typescript(options = {}, isWorkspaceProject = false) {
-	const { enableErasableSyntaxOnly = false, overrides = {}, parserOptions = {}, stylistic = true, type = "app", useRelaxedNamingConventionForCamelAndPascalCases = false } = options;
+	const { enableErasableSyntaxOnly = false, overrides = {}, parserOptions = {}, stylistic = true, type = "app", useRelaxedNamingConventionForCamelAndPascalCases = false, allowJsx = false } = options;
 	let erasableSyntaxOnlyPlugin;
 	let erasableSyntaxOnlyRules;
 	if (enableErasableSyntaxOnly) {
@@ -2160,7 +2162,7 @@ async function typescript(options = {}, isWorkspaceProject = false) {
 				fixStyle: "separate-type-imports",
 				prefer: "type-imports"
 			}],
-			"@typescript-eslint/dot-notation": "error",
+			"@typescript-eslint/dot-notation": ["error", { allowIndexSignaturePropertyAccess: true }],
 			...type === "lib" ? { "@typescript-eslint/explicit-function-return-type": ["error", {
 				allowExpressions: true,
 				allowIIFEs: true
@@ -2169,7 +2171,7 @@ async function typescript(options = {}, isWorkspaceProject = false) {
 			...type === "workspace" ? {} : { "@typescript-eslint/explicit-module-boundary-types": "error" },
 			"@typescript-eslint/member-ordering": ["error", MEMBER_ORDERING_OPTIONS],
 			"@typescript-eslint/method-signature-style": "error",
-			"@typescript-eslint/naming-convention": ["error", ...namingConvention(!useRelaxedNamingConventionForCamelAndPascalCases)],
+			"@typescript-eslint/naming-convention": ["error", ...namingConvention(!useRelaxedNamingConventionForCamelAndPascalCases, allowJsx)],
 			"@typescript-eslint/no-base-to-string": "error",
 			"@typescript-eslint/no-confusing-non-null-assertion": "error",
 			"@typescript-eslint/no-confusing-void-expression": ["error", { ignoreArrowShorthand: true }],
@@ -2462,6 +2464,47 @@ const NGRX_PACKAGES = [
 const PLAYWRIGHT_PACKAGES = ["@playwright/test", "playwright"];
 const OPTIONS_SYMBOL = Symbol("options");
 //#endregion
+//#region src/factories/playwright-config.ts
+/**
+* Creates an ESLint configuration array for Playwright projects based on the provided base configuration,
+* options and user configurations.
+* This function is designed for use in an end-to-end project of a workspace environment and extends a base configuration
+* that is already defined in the root of the workspace.
+*
+* @param baseConfig - The base configuration to extend from.
+* @param options - Playwright-specific configuration options.
+* @param userConfigs - Additional user configurations that can be awaited.
+* @returns The merged ESLint configurations.
+* @example
+* ```typescript
+* import { definePlaywrightConfig } from '@fabdeh/eslint-config';
+*
+* import baseConfig from '../../eslint.base.js';
+*
+* export default definePlaywrightConfig(baseConfig, { ignores: ['playwright-report/**'] });
+*/
+function definePlaywrightConfig(baseConfig, options = {}, ...userConfigs) {
+	if (PLAYWRIGHT_PACKAGES.every((p) => !isPackageExists(p))) throw new Error("Playwright rules require the \"playwright\" package to be installed.");
+	let isInEditor = options.isInEditor;
+	if (isInEditor === void 0) {
+		isInEditor = isInEditorEnv();
+		if (isInEditor) console.log("[@fabdeh/eslint-config] Detected running in editor, some rules are disabled.");
+	}
+	const workspaceOptions = baseConfig[OPTIONS_SYMBOL];
+	const configs = [];
+	if (options.ignores && options.ignores.length > 0) configs.push(ignores(options.ignores, "project"));
+	if (workspaceOptions.typescript) configs.push(typescript({
+		overrides: {},
+		parserOptions: {},
+		type: "app"
+	}, true));
+	configs.push(playwright(options));
+	let composer = new FlatConfigComposer();
+	composer = composer.append(baseConfig, ...configs, ...userConfigs);
+	if (isInEditor) composer = composer.disableRulesFix(["prefer-const", "unused-imports/no-unused-imports"], { builtinRules: () => import("eslint/use-at-your-own-risk").then((m) => m.builtinRules) });
+	return composer;
+}
+//#endregion
 //#region src/factories/project-config.ts
 /**
 * Creates an ESLint configuration array based on the provided base configuration, options and user configurations.
@@ -2546,7 +2589,7 @@ function defineProjectConfig(baseConfig, options = {}, ...userConfigs) {
 * ```
 */
 function defineConfig(options = {}, ...userConfigs) {
-	const { angular: enableAngular = isPackageExists("@angular/core"), gitignore: enableGitignore = true, jsdoc: enableJsdoc = options.type === "lib", ngrx: enableNgrx = NGRX_PACKAGES.some((p) => isPackageExists(p)), playwright: enablePlaywright = PLAYWRIGHT_PACKAGES.some((p) => isPackageExists(p)), pnpm: enableCatalogs = !!findUpSync("pnpm-workspace.yaml"), regexp: enableRegexp = true, tailwindcss: enableTailwind = false, typescript: enableTypescript = isPackageExists("typescript"), unicorn: enableUnicorn = true, vitest: enableVitest = isPackageExists("vitest") } = options;
+	const { angular: enableAngular = isPackageExists("@angular/core"), gitignore: enableGitignore = true, jsdoc: enableJsdoc = options.type === "lib", ngrx: enableNgrx = NGRX_PACKAGES.some((p) => isPackageExists(p)), playwright: enablePlaywright = PLAYWRIGHT_PACKAGES.some((p) => isPackageExists(p)), pnpm: enableCatalogs = !!up("pnpm-workspace.yaml"), regexp: enableRegexp = true, tailwindcss: enableTailwind = false, typescript: enableTypescript = isPackageExists("typescript"), unicorn: enableUnicorn = true, vitest: enableVitest = isPackageExists("vitest") } = options;
 	let isInEditor = options.isInEditor;
 	if (isInEditor === void 0) {
 		isInEditor = isInEditorEnv();
@@ -2555,14 +2598,16 @@ function defineConfig(options = {}, ...userConfigs) {
 	if (enableNgrx && !enableAngular) throw new Error("NgRx rules can only be enabled if Angular rules are also enabled.");
 	const stylisticOptions = options.stylistic === false ? false : typeof options.stylistic === "object" ? options.stylistic : {};
 	const configs = [];
-	if (enableGitignore) if (typeof enableGitignore === "object") configs.push(interopDefault(import("eslint-config-flat-gitignore")).then((r) => [r({
-		name: "fabdeh/gitignore",
-		...enableGitignore
-	})]));
-	else configs.push(interopDefault(import("eslint-config-flat-gitignore")).then((r) => [r({
-		name: "fabdeh/gitignore",
-		strict: true
-	})]));
+	if (enableGitignore) {
+		if (typeof enableGitignore === "object") configs.push(interopDefault(import("eslint-config-flat-gitignore")).then((r) => [r({
+			name: "fabdeh/gitignore",
+			...enableGitignore
+		})]));
+		else configs.push(interopDefault(import("eslint-config-flat-gitignore")).then((r) => [r({
+			name: "fabdeh/gitignore",
+			strict: true
+		})]));
+	}
 	configs.push(ignores(options.ignores), javascript({
 		isInEditor,
 		overrides: options.javascript?.overrides
@@ -2677,7 +2722,7 @@ function defineConfig(options = {}, ...userConfigs) {
 * ```
 */
 function defineWorkspaceConfig(options = {}, ...userConfigs) {
-	const { gitignore: enableGitignore = true, pnpm: enableCatalogs = !!findUpSync("pnpm-workspace.yaml"), regexp: enableRegexp = true, typescript: enableTypescript = isPackageExists("typescript"), unicorn: enableUnicorn = true } = options;
+	const { gitignore: enableGitignore = true, pnpm: enableCatalogs = !!up("pnpm-workspace.yaml"), regexp: enableRegexp = true, typescript: enableTypescript = isPackageExists("typescript"), unicorn: enableUnicorn = true } = options;
 	let isInEditor = options.isInEditor;
 	if (isInEditor === void 0) {
 		isInEditor = isInEditorEnv();
@@ -2685,14 +2730,16 @@ function defineWorkspaceConfig(options = {}, ...userConfigs) {
 	}
 	const stylisticOptions = options.stylistic === false ? false : typeof options.stylistic === "object" ? options.stylistic : {};
 	const configs = [];
-	if (enableGitignore) if (typeof enableGitignore === "object") configs.push(interopDefault(import("eslint-config-flat-gitignore")).then((r) => [r({
-		name: "fabdeh/gitignore",
-		...enableGitignore
-	})]));
-	else configs.push(interopDefault(import("eslint-config-flat-gitignore")).then((r) => [r({
-		name: "fabdeh/gitignore",
-		strict: true
-	})]));
+	if (enableGitignore) {
+		if (typeof enableGitignore === "object") configs.push(interopDefault(import("eslint-config-flat-gitignore")).then((r) => [r({
+			name: "fabdeh/gitignore",
+			...enableGitignore
+		})]));
+		else configs.push(interopDefault(import("eslint-config-flat-gitignore")).then((r) => [r({
+			name: "fabdeh/gitignore",
+			strict: true
+		})]));
+	}
 	configs.push(ignores(options.ignores, "workspace"), javascript({
 		isInEditor,
 		overrides: options.javascript?.overrides
@@ -2756,4 +2803,4 @@ function defineWorkspaceConfig(options = {}, ...userConfigs) {
 	return composerWithOptions;
 }
 //#endregion
-export { CONFIG_PRESET_FULL_OFF, CONFIG_PRESET_FULL_ON, GLOB_HTML, GLOB_JS, GLOB_SRC, GLOB_TESTS, GLOB_TS, STYLISTIC_CONFIG_DEFAULT, angular, comments, convertPathToPosix, defineConfig, defineProjectConfig, defineWorkspaceConfig, ensurePackages, getPlaywrightDirectory, getTsConfigFileName, getWorkspaceRoot, ignores, imports, interopDefault, isDirectory, isInEditorEnv, isInGitHooksOrLintStaged, isPackageInScope, javascript, jsdoc, jsonc, markdown, ngrx, node, pathToGlobPattern, perfectionist, playwright, pnpm, regexp, resolveSubOptions, sortPackageJson, sortTsConfig, stylistic, tailwindcss, toml, typescript, unicorn, vitest, yaml };
+export { CONFIG_PRESET_FULL_OFF, CONFIG_PRESET_FULL_ON, GLOB_HTML, GLOB_JS, GLOB_SRC, GLOB_TESTS, GLOB_TS, STYLISTIC_CONFIG_DEFAULT, angular, comments, convertPathToPosix, defineConfig, definePlaywrightConfig, defineProjectConfig, defineWorkspaceConfig, ensurePackages, getPlaywrightDirectory, getTsConfigFileName, getWorkspaceRoot, ignores, imports, interopDefault, isDirectory, isInEditorEnv, isInGitHooksOrLintStaged, isPackageInScope, javascript, jsdoc, jsonc, markdown, ngrx, node, pathToGlobPattern, perfectionist, playwright, pnpm, regexp, resolveSubOptions, sortPackageJson, sortTsConfig, stylistic, tailwindcss, toml, typescript, unicorn, vitest, yaml };

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@fabdeh/eslint-config",
   "type": "module",
-  "version": "0.12.1",
+  "version": "0.13.0",
   "description": "My personal eslint config preset",
   "author": {
     "name": "Fabien Dehopré",
@@ -37,8 +37,8 @@
   ],
   "peerDependencies": {
     "eslint": "^9.38.0 || ^10.0.0",
-    "eslint-plugin-better-tailwindcss": "^4.5.0",
-    "eslint-plugin-erasable-syntax-only": "^0.4.1"
+    "eslint-plugin-better-tailwindcss": "^4.7.0",
+    "eslint-plugin-erasable-syntax-only": "^0.4.2"
   },
   "peerDependenciesMeta": {
     "eslint-plugin-better-tailwindcss": {
@@ -49,82 +49,84 @@
     }
   },
   "dependencies": {
-    "@antfu/install-pkg": "^1.1.0",
-    "@clack/prompts": "^1.5.1",
+    "@antfu/install-pkg": "^2.0.1",
+    "@clack/prompts": "^1.7.0",
     "@eslint-community/eslint-plugin-eslint-comments": "^4.7.2",
-    "@eslint/markdown": "^8.0.2",
+    "@eslint/markdown": "^8.0.3",
     "@ngrx/eslint-plugin": "^21.1.1",
     "@stylistic/eslint-plugin": "^5.10.0",
-    "@typescript-eslint/eslint-plugin": "^8.61.0",
-    "@typescript-eslint/parser": "^8.61.0",
-    "@typescript-eslint/utils": "^8.61.0",
-    "@vitest/eslint-plugin": "^1.6.19",
-    "angular-eslint": "^22.0.0",
+    "@typescript-eslint/eslint-plugin": "^8.66.0",
+    "@typescript-eslint/parser": "^8.66.0",
+    "@typescript-eslint/utils": "^8.66.0",
+    "@vitest/eslint-plugin": "^1.6.27",
+    "angular-eslint": "^22.1.0",
+    "empathic": "^2.0.1",
     "eslint-config-flat-gitignore": "^2.3.0",
     "eslint-flat-config-utils": "^3.2.0",
     "eslint-merge-processors": "^2.0.0",
-    "eslint-plugin-import-x": "^4.16.2",
+    "eslint-plugin-import-x": "^4.17.1",
     "eslint-plugin-jest-dom-ya": "^1.0.1",
-    "eslint-plugin-jsdoc": "^63.0.2",
-    "eslint-plugin-jsonc": "^3.2.0",
-    "eslint-plugin-n": "^18.1.0",
-    "eslint-plugin-perfectionist": "^5.9.0",
-    "eslint-plugin-playwright": "^2.10.4",
-    "eslint-plugin-pnpm": "^1.6.1",
-    "eslint-plugin-prefer-arrow-functions": "^3.9.1",
-    "eslint-plugin-regexp": "^3.1.0",
+    "eslint-plugin-jsdoc": "^63.3.3",
+    "eslint-plugin-jsonc": "^3.4.1",
+    "eslint-plugin-n": "^18.3.0",
+    "eslint-plugin-perfectionist": "^5.10.1",
+    "eslint-plugin-playwright": "^2.11.0",
+    "eslint-plugin-pnpm": "^1.7.0",
+    "eslint-plugin-prefer-arrow-functions": "^3.10.2",
+    "eslint-plugin-regexp": "^3.1.1",
     "eslint-plugin-testing-library": "^7.16.2",
-    "eslint-plugin-toml": "^1.4.0",
+    "eslint-plugin-toml": "^1.5.0",
     "eslint-plugin-unicorn": "^65.0.1",
     "eslint-plugin-unused-imports": "^4.4.1",
-    "eslint-plugin-yml": "^3.4.0",
-    "find-up-simple": "^1.0.1",
+    "eslint-plugin-yml": "^3.8.1",
     "glob": "^13.0.6",
-    "globals": "^17.6.0",
-    "jsonc-eslint-parser": "^3.1.0",
+    "globals": "^17.9.0",
+    "jsonc-eslint-parser": "^3.3.0",
     "local-pkg": "^1.2.1",
-    "typescript-eslint": "^8.61.0"
+    "typescript-eslint": "^8.66.0"
   },
   "devDependencies": {
-    "@angular/core": "^22.0.0",
-    "@arethetypeswrong/cli": "^0.18.3",
-    "@commitlint/cli": "^21.0.2",
-    "@commitlint/config-conventional": "^21.0.2",
-    "@commitlint/cz-commitlint": "^21.0.2",
-    "@commitlint/lint": "^21.0.2",
-    "@commitlint/load": "^21.0.2",
+    "@angular/cli": "^22.1.3",
+    "@angular/core": "^22.1.1",
+    "@arethetypeswrong/cli": "^0.18.5",
+    "@commitlint/cli": "^21.2.1",
+    "@commitlint/config-conventional": "^21.2.0",
+    "@commitlint/cz-commitlint": "^21.2.0",
+    "@commitlint/lint": "^21.2.0",
+    "@commitlint/load": "^21.2.0",
     "@ngrx/effects": "^21.1.1",
     "@ngrx/operators": "^21.1.1",
     "@ngrx/signals": "^21.1.1",
     "@ngrx/store": "^21.1.1",
-    "@testing-library/angular": "^19.4.1",
-    "@testing-library/jest-dom": "^6.9.1",
+    "@testing-library/angular": "^19.4.2",
+    "@testing-library/dom": "^10.4.1",
+    "@testing-library/jest-dom": "^7.0.1",
     "@types/fs-extra": "^11.0.4",
-    "@types/node": "^24.13.1",
-    "@vitest/coverage-v8": "^4.1.8",
-    "@vitest/ui": "^4.1.8",
-    "bumpp": "^11.1.0",
+    "@types/node": "^24.13.3",
+    "@vitest/coverage-v8": "^4.1.10",
+    "@vitest/ui": "^4.1.10",
+    "bumpp": "^12.2.0",
     "change-case": "^5.4.4",
-    "changelogithub": "^14.0.0",
-    "commitizen": "^4.3.1",
-    "eslint": "^10.4.1",
-    "eslint-plugin-better-tailwindcss": "^4.5.0",
-    "eslint-plugin-erasable-syntax-only": "^0.4.1",
+    "changelogithub": "^15.0.4",
+    "commitizen": "^4.3.2",
+    "eslint": "^10.8.1",
+    "eslint-plugin-better-tailwindcss": "^4.7.0",
+    "eslint-plugin-erasable-syntax-only": "^0.4.2",
     "eslint-typegen": "^2.3.1",
-    "execa": "^9.6.1",
-    "fs-extra": "^11.3.5",
+    "execa": "^10.0.1",
+    "fs-extra": "^11.4.0",
     "is-ci": "^4.1.0",
     "jiti": "^2.7.0",
     "nano-staged": "^1.0.2",
-    "playwright": "^1.60.0",
-    "semver": "^7.8.3",
+    "playwright": "^1.62.1",
+    "semver": "^7.8.5",
     "simple-git-hooks": "^2.13.1",
-    "tailwindcss": "^4.3.0",
-    "tsdown": "^0.22.2",
-    "tsx": "^4.22.4",
+    "tailwindcss": "^4.3.3",
+    "tsdown": "^0.22.14",
+    "tsx": "^4.23.12",
     "typescript": "^6.0.3",
-    "vitest": "^4.1.8",
-    "@fabdeh/eslint-config": "0.12.1"
+    "vitest": "^4.1.10",
+    "@fabdeh/eslint-config": "0.13.0"
   },
   "simple-git-hooks": {
     "pre-commit": "pnpm nano-staged",