@shayanthenerd/eslint-config 0.28.0 → 0.28.2

package/README.md

@@ -80,7 +80,7 @@ Legend:
    {
      "scripts": {
        "lint:inspect": "npx @eslint/config-inspector",
-       "lint": "eslint --fix --cache --cache-location='node_modules/.cache/.eslintcache'"
+       "lint": "eslint --fix --max-warnings=0 --cache --cache-location='node_modules/.cache/.eslintcache'"
      }
    }
    ```
@@ -363,7 +363,7 @@ _package.json_:
 {
   "scripts": {
     "format": "prettier --write . --cache",
-    "lint": "eslint --fix --cache --cache-location='node_modules/.cache/.eslintcache'"
+    "lint": "eslint --fix --max-warnings=0 --cache --cache-location='node_modules/.cache/.eslintcache'"
   }
 }
 ```
@@ -427,7 +427,7 @@ _.vscode/settings.json_:
     packageDir?: string,
     tsConfig?: false | {
       filename?: string,
-      rootDir: string,
+      rootDir?: string,
     },
 
     project?: {

package/dist/configs/importX.mjs

@@ -18,7 +18,7 @@ function getImportXConfig(options) {
 			isEnabled(astro) ? globs.astro : ""
 		].filter(isTruthy),
 		plugins: { "import-x": eslintPluginImportX },
-		settings: isEnabled(typescript) ? eslintPluginImportX.flatConfigs.typescript.settings : void 0,
+		settings: isEnabled(typescript) ? eslintPluginImportX.flatConfigs.typescript.settings : {},
 		rules: getImportXRules(options)
 	}, overrides);
 }

package/dist/configs/node.mjs

@@ -10,7 +10,7 @@ import path from "node:path";
 function getNodeConfig(options) {
 	const { tsConfig, configs: { vue, node, astro } } = options;
 	const { overrides } = isEnabled(node) ? node : defaultOptions.configs.node;
-	const tsconfig = tsConfig ? path.resolve(tsConfig.rootDir, tsConfig.filename) : void 0;
+	const tsconfigPath = tsConfig ? path.resolve(tsConfig.rootDir, tsConfig.filename) : void 0;
 	return mergeConfigs({
 		name: "shayanthenerd/node",
 		files: [
@@ -20,7 +20,7 @@ function getNodeConfig(options) {
 			isEnabled(astro) ? globs.astro : ""
 		].filter(isTruthy),
 		plugins: { n: eslintPluginNode },
-		settings: { n: { tsconfigPath: tsconfig } },
+		settings: { n: tsconfigPath ? { tsconfigPath } : void 0 },
 		rules: getNodeRules(options)
 	}, overrides);
 }

package/dist/configs/tailwind.mjs

@@ -45,7 +45,7 @@ const nuxtUiAppConfigUiFields = {
 function getTailwindConfig(options) {
 	const { tsConfig, configs: { vue, nuxt, html, astro, tailwind } } = options;
 	const { cwd, config, overrides, entryPoint } = isEnabled(tailwind) ? tailwind : defaultOptions.configs.tailwind;
-	const tsconfig = tsConfig ? path.resolve(tsConfig.rootDir, tsConfig.filename) : void 0;
+	const tsconfigPath = tsConfig ? path.resolve(tsConfig.rootDir, tsConfig.filename) : void 0;
 	const isNuxtUiEnabled = isEnabled(nuxt) && isEnabled(nuxt.ui);
 	const selectors = [
 		...getDefaultSelectors(),
@@ -66,10 +66,10 @@ function getTailwindConfig(options) {
 		plugins: { "better-tailwindcss": eslintPluginTailwind },
 		settings: { "better-tailwindcss": {
 			cwd,
-			tsconfig,
 			selectors,
 			entryPoint,
 			tailwindConfig: config,
+			tsconfig: tsconfigPath,
 			detectComponentClasses: true
 		} },
 		rules: getTailwindRules(options)

package/dist/helpers/options/defaultOptions.mjs

@@ -4,6 +4,7 @@ const defaultOptions = {
 	env: "browser",
 	gitignore: ".gitignore",
 	packageDir: ".",
+	tsConfig: false,
 	project: {
 		basePath: ".",
 		globals: {

package/dist/helpers/options/enableDetectedConfigs.mjs

@@ -1,4 +1,5 @@
 import { isEnabled } from "../../utils/isEnabled.mjs";
+import { defaultOptions } from "./defaultOptions.mjs";
 import { isPackageDetected, logDetectedPackages } from "../isPackageDetected.mjs";
 //#region src/helpers/options/enableDetectedConfigs.ts
 function enableDetectedConfigs(options) {
@@ -58,11 +59,20 @@ function enableDetectedConfigs(options) {
 	options.configs.test.cypress ??= autoDetectedPackages.cypress;
 	options.configs.test.storybook ??= autoDetectedPackages.storybook;
 	options.configs.test.playwright ??= autoDetectedPackages.playwright;
-	options.tsConfig ??= options.configs.typescript ? {
-		rootDir: ".",
-		filename: "tsconfig.json"
-	} : void 0;
-	if (options.configs.vue && options.configs.typescript) {
+	const isTypeScriptEnabled = isEnabled(options.configs.typescript);
+	if (isTypeScriptEnabled) {
+		const defaultTsConfigRootDir = options.packageDir ?? defaultOptions.packageDir;
+		const defaultTsConfigFilename = "tsconfig.json";
+		if (options.tsConfig === void 0) options.tsConfig = {
+			rootDir: defaultTsConfigRootDir,
+			filename: defaultTsConfigFilename
+		};
+		else if (options.tsConfig !== false) {
+			options.tsConfig.rootDir ??= defaultTsConfigRootDir;
+			options.tsConfig.filename ??= defaultTsConfigFilename;
+		}
+	}
+	if (isTypeScriptEnabled && options.configs.vue) {
 		options.configs.vue = options.configs.vue === true ? {} : options.configs.vue;
 		options.configs.vue.blockLang = { script: "ts" };
 	}

package/dist/index.mjs

@@ -82,9 +82,9 @@ function defineConfig(...args) {
 	return defineConfig$1([
 		{
 			name: "shayanthenerd/project",
-			rules,
 			linterOptions,
-			...Object.keys(settings).length > 0 && { settings }
+			settings,
+			rules
 		},
 		globalIgnores(ignorePatterns, "shayanthenerd/ignores"),
 		getBaseConfig(mergedOptions),

package/dist/rules/perfectionist.mjs

@@ -114,7 +114,7 @@ function getPerfectionistRules(options) {
 		"perfectionist/sort-named-exports": "warn",
 		"perfectionist/sort-imports": ["warn", {
 			environment: env === "bun" ? "bun" : "node",
-			tsconfig: tsConfig || void 0,
+			tsconfig: tsConfig || { rootDir: "" },
 			sortSideEffects: true,
 			fallbackSort: {
 				order: "asc",

package/dist/rules/stylistic.mjs

@@ -165,10 +165,10 @@ function getStylisticRules(options) {
 			},
 			{
 				prev: [
-					"var",
-					"let",
 					"const",
-					"using"
+					"let",
+					"using",
+					"var"
 				],
 				next: "block-like",
 				blankLine: "any"
@@ -178,10 +178,28 @@ function getStylisticRules(options) {
 				next: "*",
 				blankLine: "always"
 			},
+			{
+				prev: [
+					"if",
+					"switch",
+					"try",
+					"for",
+					"while",
+					"do",
+					"with"
+				],
+				next: [
+					"break",
+					"continue",
+					"return",
+					"throw"
+				],
+				blankLine: "any"
+			},
 			{
 				prev: ["case", "default"],
 				next: ["case", "default"],
-				blankLine: "any"
+				blankLine: "never"
 			}
 		],
 		"@stylistic/quote-props": ["warn", "consistent-as-needed"],

package/dist/rules/vue.mjs

@@ -56,11 +56,12 @@ function getVueRules(options) {
 		"vuejs-accessibility/tabindex-no-positive": "error"
 	};
 	const vueAndNuxtRules = {
+		"@typescript-eslint/no-useless-default-assignment": "off",
 		"vue/jsx-uses-vars": "error",
 		"vue/comment-directive": ["error", { reportUnusedDisableDirectives: true }],
 		"vue/multi-word-component-names": "error",
 		"vue/no-arrow-functions-in-watch": "warn",
-		"vue/no-async-in-computed-properties": ["error", { ignoredObjectNames: ["z"] }],
+		"vue/no-async-in-computed-properties": ["error", { ignoredObjectNames: ["z", "zod"] }],
 		"vue/no-child-content": "error",
 		"vue/no-computed-properties-in-data": "error",
 		"vue/no-deprecated-data-object-declaration": "error",

package/dist/rules/zod.mjs

@@ -12,12 +12,15 @@ function getZodRules(options) {
 		"zod-mini/consistent-schema-var-name": "warn",
 		"zod-mini/no-any-schema": "error",
 		"zod-mini/no-coerce-boolean": "warn",
+		"zod-mini/no-conflicting-checks": "warn",
 		"zod-mini/no-duplicate-schema-methods": "error",
 		"zod-mini/no-empty-custom-schema": "error",
 		"zod-mini/no-throw-in-refine": "error",
 		"zod-mini/no-transform-in-record-key": "error",
+		"zod-mini/no-unnecessary-readonly": "warn",
 		"zod-mini/prefer-enum-over-literal-union": "warn",
 		"zod-mini/prefer-meta": "warn",
+		"zod-mini/prefer-tuple-over-array-length": "warn",
 		"zod-mini/require-brand-type-parameter": "error",
 		"zod-mini/require-error-message": "warn"
 	} : {
@@ -28,6 +31,7 @@ function getZodRules(options) {
 		"zod/consistent-schema-var-name": "warn",
 		"zod/no-any-schema": "error",
 		"zod/no-coerce-boolean": "warn",
+		"zod/no-conflicting-checks": "error",
 		"zod/no-duplicate-schema-methods": "error",
 		"zod/no-empty-custom-schema": "error",
 		"zod/no-native-enum": "warn",
@@ -43,6 +47,7 @@ function getZodRules(options) {
 		"zod/no-schema-with-is-optional": "warn",
 		"zod/no-throw-in-refine": "error",
 		"zod/no-transform-in-record-key": "error",
+		"zod/no-unnecessary-readonly": "warn",
 		"zod/prefer-enum-over-literal-union": "warn",
 		"zod/prefer-loose-object": "warn",
 		"zod/prefer-meta": "warn",
@@ -50,6 +55,7 @@ function getZodRules(options) {
 		"zod/prefer-strict-object": "warn",
 		"zod/prefer-top-level-string-formats": "warn",
 		"zod/prefer-trim-before-string-length-checks": "error",
+		"zod/prefer-tuple-over-array-length": "warn",
 		"zod/require-brand-type-parameter": "error",
 		"zod/require-error-message": "warn"
 	};

package/dist/types/eslint-schema.d.mts

@@ -9731,6 +9731,11 @@ interface ESLintSchema {
    * @see https://github.com/marcalexiei/eslint-zod/blob/HEAD/plugins/eslint-plugin-zod-mini/docs/rules/no-coerce-boolean.md
    */
   'zod-mini/no-coerce-boolean'?: Linter.RuleEntry<[]>;
+  /**
+   * Disallow check combinations that can never match, are redundant, or do not apply to the schema type
+   * @see https://github.com/marcalexiei/eslint-zod/blob/HEAD/plugins/eslint-plugin-zod-mini/docs/rules/no-conflicting-checks.md
+   */
+  'zod-mini/no-conflicting-checks'?: Linter.RuleEntry<ZodMiniNoConflictingChecks>;
   /**
    * Disallow calling the same schema method more than once in a single chain
    * @see https://github.com/marcalexiei/eslint-zod/blob/HEAD/plugins/eslint-plugin-zod-mini/docs/rules/no-duplicate-schema-methods.md
@@ -9756,6 +9761,11 @@ interface ESLintSchema {
    * @see https://github.com/marcalexiei/eslint-zod/blob/HEAD/plugins/eslint-plugin-zod-mini/docs/rules/no-unknown-schema.md
    */
   'zod-mini/no-unknown-schema'?: Linter.RuleEntry<[]>;
+  /**
+   * Disallow `z.readonly()` on schemas whose output is already immutable
+   * @see https://github.com/marcalexiei/eslint-zod/blob/HEAD/plugins/eslint-plugin-zod-mini/docs/rules/no-unnecessary-readonly.md
+   */
+  'zod-mini/no-unnecessary-readonly'?: Linter.RuleEntry<[]>;
   /**
    * Prefer `z.enum()` over `z.union()` when all members are string literals.
    * @see https://github.com/marcalexiei/eslint-zod/blob/HEAD/plugins/eslint-plugin-zod-mini/docs/rules/prefer-enum-over-literal-union.md
@@ -9766,6 +9776,11 @@ interface ESLintSchema {
    * @see https://github.com/marcalexiei/eslint-zod/blob/HEAD/plugins/eslint-plugin-zod-mini/docs/rules/prefer-meta.md
    */
   'zod-mini/prefer-meta'?: Linter.RuleEntry<[]>;
+  /**
+   * Prefer `z.tuple()` over a length-constrained `z.array()` so the length is preserved in the inferred type.
+   * @see https://github.com/marcalexiei/eslint-zod/blob/HEAD/plugins/eslint-plugin-zod-mini/docs/rules/prefer-tuple-over-array-length.md
+   */
+  'zod-mini/prefer-tuple-over-array-length'?: Linter.RuleEntry<[]>;
   /**
    * Require type parameter on `.brand()` functions
    * @see https://github.com/marcalexiei/eslint-zod/blob/HEAD/plugins/eslint-plugin-zod-mini/docs/rules/require-brand-type-parameter.md
@@ -9821,6 +9836,11 @@ interface ESLintSchema {
    * @see https://github.com/marcalexiei/eslint-zod/blob/HEAD/plugins/eslint-plugin-zod/docs/rules/no-coerce-boolean.md
    */
   'zod/no-coerce-boolean'?: Linter.RuleEntry<[]>;
+  /**
+   * Disallow check combinations that can never match, are redundant, or do not apply to the schema type
+   * @see https://github.com/marcalexiei/eslint-zod/blob/HEAD/plugins/eslint-plugin-zod/docs/rules/no-conflicting-checks.md
+   */
+  'zod/no-conflicting-checks'?: Linter.RuleEntry<ZodNoConflictingChecks>;
   /**
    * Disallow calling the same schema method more than once in a single chain
    * @see https://github.com/marcalexiei/eslint-zod/blob/HEAD/plugins/eslint-plugin-zod/docs/rules/no-duplicate-schema-methods.md
@@ -9907,6 +9927,11 @@ interface ESLintSchema {
    * @see https://github.com/marcalexiei/eslint-zod/blob/HEAD/plugins/eslint-plugin-zod/docs/rules/no-unknown-schema.md
    */
   'zod/no-unknown-schema'?: Linter.RuleEntry<[]>;
+  /**
+   * Disallow `.readonly()` on schemas whose output is already immutable
+   * @see https://github.com/marcalexiei/eslint-zod/blob/HEAD/plugins/eslint-plugin-zod/docs/rules/no-unnecessary-readonly.md
+   */
+  'zod/no-unnecessary-readonly'?: Linter.RuleEntry<[]>;
   /**
    * Prefer `z.enum()` over `z.union()` when all members are string literals.
    * @see https://github.com/marcalexiei/eslint-zod/blob/HEAD/plugins/eslint-plugin-zod/docs/rules/prefer-enum-over-literal-union.md
@@ -9947,6 +9972,11 @@ interface ESLintSchema {
    * @see https://github.com/marcalexiei/eslint-zod/blob/HEAD/plugins/eslint-plugin-zod/docs/rules/prefer-trim-before-string-length-checks.md
    */
   'zod/prefer-trim-before-string-length-checks'?: Linter.RuleEntry<[]>;
+  /**
+   * Prefer `z.tuple()` over a length-constrained `z.array()` so the length is preserved in the inferred type.
+   * @see https://github.com/marcalexiei/eslint-zod/blob/HEAD/plugins/eslint-plugin-zod/docs/rules/prefer-tuple-over-array-length.md
+   */
+  'zod/prefer-tuple-over-array-length'?: Linter.RuleEntry<[]>;
   /**
    * Require type parameter on `.brand()` functions
    * @see https://github.com/marcalexiei/eslint-zod/blob/HEAD/plugins/eslint-plugin-zod/docs/rules/require-brand-type-parameter.md
@@ -23108,6 +23138,12 @@ type ZodMiniConsistentSchemaVarName = [] | [{
   before?: string;
   after?: string;
 }];
+// ----- zod-mini/no-conflicting-checks -----
+type ZodMiniNoConflictingChecks = [] | [{
+  checkImpossibleCases?: boolean;
+  checkConfusingCases?: boolean;
+  checkInapplicableChecks?: boolean;
+}];
 // ----- zod-mini/schema-error-property-style -----
 type ZodMiniSchemaErrorPropertyStyle = [] | [{
   selector?: string;
@@ -23138,6 +23174,12 @@ type ZodConsistentSchemaVarName = [] | [{
   before?: string;
   after?: string;
 }];
+// ----- zod/no-conflicting-checks -----
+type ZodNoConflictingChecks = [] | [{
+  checkImpossibleCases?: boolean;
+  checkConfusingCases?: boolean;
+  checkInapplicableChecks?: boolean;
+}];
 // ----- zod/no-optional-and-default-together -----
 type ZodNoOptionalAndDefaultTogether = [] | [{
   preferredMethod?: ("none" | "default" | "optional");

package/dist/types/index.d.mts

@@ -81,25 +81,25 @@ interface Options {
    * - [better-tailwindcss: `tsconfig` option](https://github.com/schoero/eslint-plugin-better-tailwindcss/blob/main/docs/settings/settings.md#tsconfig)
    * - [perfectionist/sort-imports: `tsconfig` option](https://perfectionist.dev/rules/sort-imports#tsconfig)
    *
-   * @default undefined // `{ rootDir: '.', filename: 'tsconfig.json' }` if TypeScript integration is enabled
+   * @default false // `{ rootDir: options.packageDir ?? '.', filename: 'tsconfig.json' }` if TypeScript integration is enabled
    */
   tsConfig?: false | {
     /**
-     * The name of the root TypeScript config file.
+     * The directory of the root TypeScript config file.
      *
      * It will fall back to the default value if set to an empty string (`''`).
      *
-     * @default 'tsconfig.json'
+     * @default options.packageDir ?? '.'
      */
-    filename?: string;
+    rootDir?: string;
     /**
-     * The directory of the root TypeScript config file.
+     * The name of the root TypeScript config file.
      *
      * It will fall back to the default value if set to an empty string (`''`).
      *
-     * @default '.'
+     * @default 'tsconfig.json'
      */
-    rootDir: string;
+    filename?: string;
   };
   /*** Project ***/
   /**

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@shayanthenerd/eslint-config",
-  "version": "0.28.0",
+  "version": "0.28.2",
   "description": "A modern, flexible ESLint configuration for enforcing best practices and maintaining a consistent coding style",
   "keywords": [
     "eslint",
@@ -49,7 +49,7 @@
     "{**/*.ts,package.json}": "pnpm lint"
   },
   "dependencies": {
-    "@eslint-react/eslint-plugin": "5.14.10",
+    "@eslint-react/eslint-plugin": "5.17.1",
     "@eslint/css": "1.4.0",
     "@eslint/markdown": "8.0.3",
     "@html-eslint/eslint-plugin": "0.64.0",
@@ -61,7 +61,7 @@
     "defu": "6.1.7",
     "eslint-flat-config-utils": "3.2.0",
     "eslint-import-resolver-typescript": "4.4.5",
-    "eslint-plugin-astro": "3.0.0",
+    "eslint-plugin-astro": "3.0.1",
     "eslint-plugin-baseline-js": "0.7.0",
     "eslint-plugin-better-tailwindcss": "4.6.1",
     "eslint-plugin-cypress": "6.4.3",
@@ -72,12 +72,12 @@
     "eslint-plugin-perfectionist": "5.10.0",
     "eslint-plugin-playwright": "2.10.5",
     "eslint-plugin-promise": "7.3.0",
-    "eslint-plugin-storybook": "10.5.0",
+    "eslint-plugin-storybook": "10.5.2",
     "eslint-plugin-unicorn": "72.0.0",
     "eslint-plugin-vue": "10.9.2",
     "eslint-plugin-vuejs-accessibility": "2.5.0",
-    "eslint-plugin-zod": "4.7.1",
-    "eslint-plugin-zod-mini": "1.4.0",
+    "eslint-plugin-zod": "4.8.0",
+    "eslint-plugin-zod-mini": "1.5.0",
     "globals": "17.7.0",
     "local-pkg": "1.2.1",
     "tailwind-csstree": "0.3.3",
@@ -97,7 +97,7 @@
     "serve": "14.2.6",
     "simple-git-hooks": "2.13.1",
     "ts-morph": "28.0.0",
-    "tsdown": "0.22.7",
+    "tsdown": "0.22.9",
     "typescript": "6.0.3",
     "unplugin-unused": "0.5.7"
   },
@@ -110,7 +110,7 @@
   "devEngines": {
     "packageManager": {
       "name": "pnpm",
-      "version": "11.13.0",
+      "version": "11.13.1",
       "onFail": "warn"
     },
     "runtime": {