eslint-config-tamia 9.4.4 → 9.6.0

package/Readme.md

@@ -12,6 +12,7 @@ All presets include the following ESLint plugins:
 - [eslint-plugin-de-morgan](https://github.com/azat-io/eslint-plugin-de-morgan)
 - [eslint-plugin-unicorn](https://github.com/sindresorhus/eslint-plugin-unicorn)
 - [eslint-plugin-vitest](https://github.com/vitest-dev/eslint-plugin-vitest)
+- [eslint-plugin-washing-code](https://github.com/sapegin/eslint-plugin-washing-code/)
 
 ## Usage
 

package/package.json

@@ -1,7 +1,7 @@
 {
 	"name": "eslint-config-tamia",
 	"description": "Tâmia ESLint config",
-	"version": "9.4.4",
+	"version": "9.6.0",
 	"engines": {
 		"node": ">=22"
 	},
@@ -45,9 +45,9 @@
 	"homepage": "https://github.com/sapegin/eslint-config-tamia",
 	"devDependencies": {
 		"@types/eslint__js": "^8.42.3",
-		"@types/react": "^19.1.13",
-		"eslint": ">=9.16",
-		"react": "^19.1.1",
+		"@types/react": "^19.2.2",
+		"eslint": ">=9.38",
+		"react": "^19.2.0",
 		"vitest": "^3.2.4"
 	},
 	"peerDependencies": {
@@ -55,12 +55,13 @@
 	},
 	"dependencies": {
 		"@eslint-community/eslint-plugin-eslint-comments": "4.5.0",
-		"@eslint/js": "9.36.0",
-		"@vitest/eslint-plugin": "1.3.16",
+		"@eslint/js": "9.38.0",
+		"@vitest/eslint-plugin": "1.3.23",
 		"eslint-plugin-de-morgan": "2.0.0",
 		"eslint-plugin-react": "7.37.5",
 		"eslint-plugin-unicorn": "61.0.2",
+		"eslint-plugin-washing-code": "1.0.0",
 		"globals": "16.4.0",
-		"typescript-eslint": "8.44.1"
+		"typescript-eslint": "8.46.1"
 	}
 }

package/rules/misc.mjs

@@ -1,5 +1,6 @@
 import comments from '@eslint-community/eslint-plugin-eslint-comments/configs';
 import deMorgan from 'eslint-plugin-de-morgan';
+import washingCode from 'eslint-plugin-washing-code';
 
 export default [
 	// Disallows useless ESLint disable comments, etc.
@@ -7,6 +8,8 @@ export default [
 	// Enforces logical consistency by transforming negated boolean expressions
 	// according to De Morgan’s laws
 	deMorgan.configs.recommended,
+	// Rules inspired by Washing your code book
+	washingCode.configs.recommended,
 	{
 		rules: {},
 	},

package/rules/typescript.mjs

@@ -4,14 +4,16 @@ import tseslint from 'typescript-eslint';
 // produces false positives, for examples, in CommonJS files
 
 function findConfig(name) {
-	return tseslint.configs.recommended.find((x) => x.name === name);
+	return tseslint.configs.recommendedTypeChecked.find((x) => x.name === name);
 }
 
 const baseConfig = findConfig('typescript-eslint/base');
 const eslintRecommendedConfig = findConfig(
 	'typescript-eslint/eslint-recommended'
 );
-const recommendedConfig = findConfig('typescript-eslint/recommended');
+const recommendedTypeCheckedConfig = findConfig(
+	'typescript-eslint/recommended-type-checked'
+);
 
 /** @type { import("eslint").Linter.FlatConfig[] } */
 export default [
@@ -19,22 +21,25 @@ export default [
 		// Only apply to files with TypeScript
 		files: ['**/*.{ts,tsx,mts,mtsx,astro}'],
 
-		// Copy the basic options from the recommended config
-		languageOptions: baseConfig.languageOptions,
+		// Apply default TypeScript options and enable type checking
+		languageOptions: {
+			...baseConfig.languageOptions,
+			parserOptions: {
+				projectService: true,
+			},
+		},
 		plugins: baseConfig.plugins,
 
 		rules: {
 			// Recommended rules
 			...eslintRecommendedConfig.rules,
-			...recommendedConfig.rules,
+			...recommendedTypeCheckedConfig.rules,
 
 			// Disable generic rules that conflict with TypeScript
 			// Most of them are disabled in typescript-eslint/eslint-recommended
 			// config, but here are a few more
 			camelcase: 'off',
-			'no-array-constructor': 'off',
-			'no-unused-vars': 'off',
-			'no-unused-expressions': 'off',
+			'dot-notation': 'off',
 			'no-use-before-define': 'off',
 			'no-shadow': 'off',
 			// Conflicts with TypeScript check for unreachable code
@@ -46,8 +51,22 @@ export default [
 
 			// Require that function overload signatures be consecutive
 			'@typescript-eslint/adjacent-overload-signatures': 'error',
-			// Require consistently using either T[] or Array<T> for arrays
+			// Consistently use T[] instead of Array<T> for arrays
 			'@typescript-eslint/array-type': 'error',
+			// Enforce specifying generic type arguments on constructor name of a
+			// constructor call (new Map<string, number>())
+			'@typescript-eslint/consistent-generic-constructors': 'error',
+			// Require the Record type
+			'@typescript-eslint/consistent-indexed-object-style': 'error',
+			// Enforce consistent usage of type assertions (`value as Type` instead of
+			// `<Type>value`)
+			'@typescript-eslint/consistent-type-assertions': 'error',
+			// Enforce type definitions to consistently use `interface`
+			'@typescript-eslint/consistent-type-definitions': 'error',
+			// Enforce consistent usage of type exports
+			'@typescript-eslint/consistent-type-exports': 'error',
+			// Enforce dot notation whenever possible
+			'@typescript-eslint/dot-notation': 'error',
 			// Enforce naming conventions for everything across a codebase
 			'@typescript-eslint/naming-convention': [
 				'error',
@@ -81,18 +100,46 @@ export default [
 			],
 			// Require explicit accessibility modifiers on class properties and methods
 			'@typescript-eslint/explicit-member-accessibility': 'error',
-			// Enforce consistent usage of type assertions
-			'@typescript-eslint/consistent-type-assertions': 'error',
 			// Disallow explicit type declarations for variables or parameters
 			// initialized to a number, string, or boolean
 			'@typescript-eslint/no-inferrable-types': 'error',
+			// Disallow void type outside of generic or return types
+			'@typescript-eslint/no-invalid-void-type': 'error',
+			// Disallow the void operator except when used to discard a value
+			'@typescript-eslint/no-meaningless-void-operator': 'error',
+			// Disallow using the spread operator when it might cause unexpected behavior
+			'@typescript-eslint/no-misused-spread': 'error',
+			// Disallow enums from having both number and string members
+			'@typescript-eslint/no-mixed-enums': 'error',
 			// Disallow non-null assertions using the ! postfix operator
 			'@typescript-eslint/no-non-null-assertion': 'error',
-			// Disallow unused expressions
-			'@typescript-eslint/no-unused-expressions': 'error',
 			// Disallow variable declarations from shadowing variables declared
 			// in the outer scope
 			'@typescript-eslint/no-shadow': 'error',
+			// Disallow conditionals where the type is always truthy or always falsy
+			'@typescript-eslint/no-unnecessary-condition': 'error',
+			// Disallow unnecessary assignment of constructor property parameter
+			'@typescript-eslint/no-unnecessary-parameter-property-assignment':
+				'error',
+			// Disallow unnecessary template expressions
+			'@typescript-eslint/no-unnecessary-template-expression': 'error',
+			// Disallow type arguments that are equal to the default
+			'@typescript-eslint/no-unnecessary-type-arguments': 'error',
+			// Disallow conversion idioms when they do not change the type or value of
+			// the expression
+			'@typescript-eslint/no-unnecessary-type-conversion': 'error',
+			// Prefer optional chaining (`x.?y`) instead of chained logical ANDs
+			// (`x && x.y`)
+			'@typescript-eslint/prefer-optional-chain': 'error',
+			// Enforce using the nullish coalescing operator instead of logical
+			// assignments or chaining
+			'@typescript-eslint/prefer-nullish-coalescing': 'error',
+			// Require function parameters to be typed as readonly to prevent accidental
+			// mutation of inputs
+			'@typescript-eslint/prefer-readonly-parameter-types': 'error',
+			// Enforce using type parameter when calling `Array.reduce()` instead of
+			// using a type assertion
+			'@typescript-eslint/prefer-reduce-type-parameter': 'error',
 		},
 	},
 ];

package/rules/unicorn.mjs

@@ -18,7 +18,10 @@ export default [
 			// Avoid conflicts with Prettier
 			'unicorn/escape-case': ['warn', 'lowercase'],
 			// Avoid conflicts with Prettier
-			'unicorn/number-literal-case': ['warn', 'lowercase'],
+			'unicorn/number-literal-case': [
+				'warn',
+				{ hexadecimalValue: 'lowercase' },
+			],
 		},
 	},
 ];