eslint-config-tamia 8.1.2 → 9.0.0

package/Readme.md

@@ -16,54 +16,50 @@ Our default export contains all of our ESLint rules, including EcmaScript 6+. It
 
 `npm install --save-dev eslint-config-tamia eslint`
 
-`.eslintrc`:
+`eslint.config.mjs`:
 
-```json
-{
-  "extends": "tamia"
-}
+```js
+import tamia from 'eslint-config-tamia';
+export default [...tamia];
 ```
 
 ### eslint-config-tamia/react
 
-Lints ES6+ and React. Requires `eslint` and `eslint-plugin-react`.
+Lints ES6+ and React. Uses `eslint-plugin-react`.
 
-`npm install --save-dev eslint-config-tamia eslint-plugin-react eslint`
+`npm install --save-dev eslint-config-tamia eslint`
 
-`.eslintrc`:
+`eslint.config.mjs`:
 
-```json
-{
-  "extends": "tamia/react"
-}
+```js
+import tamiaReact from 'eslint-config-tamia/react';
+export default [...tamiaReact];
 ```
 
 ### eslint-config-tamia/typescript
 
-Lints TypeScript. Requires `eslint` and `@typescript-eslint/eslint-plugin`.
+Lints TypeScript. Uses `typescript-eslint`.
 
-`npm install --save-dev eslint-config-tamia @typescript-eslint/eslint-plugin @typescript-eslint/parser eslint`
+`npm install --save-dev eslint-config-tamia eslint`
 
-`.eslintrc`:
+`eslint.config.mjs`:
 
-```json
-{
-  "extends": "tamia/typescript"
-}
+```js
+import tamiaTypeScript from 'eslint-config-tamia/typescript';
+export default [...tamiaTypeScript];
 ```
 
 ### eslint-config-tamia/typescript-react
 
-Lints TypeScript and React. Requires `eslint`, `@typescript-eslint/eslint-plugin` and `eslint-plugin-react`.
+Lints TypeScript and React. Uses `typescript-eslint` and `eslint-plugin-react`.
 
-`npm install --save-dev eslint-config-tamia @typescript-eslint/eslint-plugin @typescript-eslint/parser eslint-plugin-react eslint`
+`npm install --save-dev eslint-config-tamia eslint`
 
-`.eslintrc`:
+`eslint.config.mjs`:
 
-```json
-{
-  "extends": "tamia/typescript-react"
-}
+```js
+import tamiaTypeScriptReact from 'eslint-config-tamia/typescript-react';
+export default [...tamiaTypeScriptReact];
 ```
 
 ### eslint-config-tamia/legacy
@@ -72,21 +68,20 @@ Lints ES5 and below. Only requires `eslint`.
 
 `npm install --save-dev eslint-config-tamia eslint`
 
-`.eslintrc`:
+`eslint.config.mjs`:
 
-```json
-{
-  "extends": "tamia/legacy"
-}
+```js
+import tamiaTypeScript from 'eslint-config-tamia/typescript';
+export default [...tamiaTypeScript];
 ```
 
 ## Code style at a glance
 
-- Tab indentation.
+- ~~Tab indentation.~~
 - Single-quotes.
 - Semicolons.
 - Declare variables just before their first usage.
-- Multiple variable statements.
+- Multiple variable statements over multiple variable in a single statement.
 - Make `const`, not `var`.
 - Use `===` and `!==` over `==` and `!=`.
 - Return early.
@@ -98,14 +93,14 @@ Example:
 
 ```javascript
 function eatFood(food) {
-	if (!food.length) {
-		return ['No food'];
-	}
+  if (food.length === 0) {
+    return ['No food'];
+  }
 
-	return food.map(dish => `No ${dish.toLowerCase()}`);
+  return food.map(dish => `No ${dish.toLowerCase()}`);
 }
 
-const food = ['Pizza', 'Buger', 'Coffee'];
+const food = ['Pizza', 'Burger', 'Coffee'];
 console.log(eatFood(food));
 ```
 

package/package.json

@@ -1,48 +1,49 @@
 {
-  "name": "eslint-config-tamia",
-  "description": "Tâmia ESLint config",
-  "version": "8.1.2",
-  "main": "index.js",
-  "scripts": {
-    "pretest": "npm run lint",
-    "test": "npm run test:jest",
-    "lint": "eslint . --cache --fix",
-    "test:jest": "jest",
-    "test:watch": "jest --watch",
-    "test:coverage": "jest --coverage"
-  },
-  "repository": {
-    "type": "git",
-    "url": "https://github.com/tamiadev/eslint-config-tamia"
-  },
-  "files": [
-    "rules",
-    "*.js"
-  ],
-  "keywords": [
-    "eslintconfig",
-    "tamia"
-  ],
-  "author": {
-    "name": "Artem Sapegin",
-    "url": "https://sapegin.me/"
-  },
-  "license": "MIT",
-  "bugs": {
-    "url": "https://github.com/tamiadev/eslint-config-tamia/issues"
-  },
-  "homepage": "https://github.com/tamiadev/eslint-config-tamia",
-  "devDependencies": {
-    "@babel/eslint-parser": "^7.22.9",
-    "@babel/preset-react": "^7.22.5",
-    "@typescript-eslint/eslint-plugin": "^6.0.0",
-    "@typescript-eslint/parser": "^6.0.0",
-    "eslint": "^8.14.0",
-    "eslint-plugin-react": "^7.32.2",
-    "jest": "^29.6.1",
-    "react": "^18.2.0"
-  },
-  "peerDependencies": {
-    "eslint": ">=8"
-  }
+	"name": "eslint-config-tamia",
+	"description": "Tâmia ESLint config",
+	"version": "9.0.0",
+	"main": "index.js",
+	"scripts": {
+		"pretest": "npm run lint",
+		"test": "npm run test:vitest",
+		"lint": "eslint . --cache --fix",
+		"test:vitest": "vitest run",
+		"test:watch": "vitest"
+	},
+	"repository": {
+		"type": "git",
+		"url": "https://github.com/tamiadev/eslint-config-tamia"
+	},
+	"files": [
+		"rules",
+		"*.js"
+	],
+	"keywords": [
+		"eslintconfig",
+		"tamia"
+	],
+	"author": {
+		"name": "Artem Sapegin",
+		"url": "https://sapegin.me/"
+	},
+	"license": "MIT",
+	"bugs": {
+		"url": "https://github.com/tamiadev/eslint-config-tamia/issues"
+	},
+	"homepage": "https://github.com/tamiadev/eslint-config-tamia",
+	"devDependencies": {
+		"@types/eslint__js": "^8.42.3",
+		"eslint": "^9.7.0",
+		"react": "^18.3.1",
+		"vitest": "^2.0.4"
+	},
+	"peerDependencies": {
+		"eslint": ">=9.7"
+	},
+	"dependencies": {
+		"@eslint/js": "^9.7.0",
+		"eslint-plugin-react": "^7.35.0",
+		"globals": "^15.8.0",
+		"typescript-eslint": "^8.0.0-alpha.10"
+	}
 }

package/rules/best-practices.mjs

@@ -0,0 +1,118 @@
+/** @type { import("eslint").Linter.FlatConfig[] } */
+export default [
+	{
+		rules: {
+			// Enforces return statements in callbacks of array’s methods
+			'array-callback-return': 'error',
+			// Treat var statements as if they were block scoped
+			'block-scoped-var': 'error',
+			// Require return statements to either always or never specify values
+			'consistent-return': 'error',
+			// Require curly braces for all control statements
+			curly: ['error', 'all'],
+			// Encourages use of dot notation whenever possible
+			'dot-notation': [
+				'error',
+				{
+					allowKeywords: true,
+				},
+			],
+			// Require the use of === and !==
+			eqeqeq: ['error', 'allow-null'],
+			// Disallow the use of alert, confirm, and prompt
+			'no-alert': 'error',
+			// Disallow using an async function as a Promise executor
+			'no-async-promise-executor': 'error',
+			// Disallow use of arguments.caller or arguments.callee
+			'no-caller': 'error',
+			// Disallow else after a return in an if
+			'no-else-return': 'off',
+			// Disallow use of eval()
+			'no-eval': 'error',
+			// Disallow adding to native types
+			'no-extend-native': 'error',
+			// Disallow unnecessary function binding
+			'no-extra-bind': 'error',
+			// Disallow the use of leading or trailing decimal points in numeric literals
+			'no-floating-decimal': 'error',
+			// Disallow the type conversions with shorter notations
+			'no-implicit-coercion': [
+				'error',
+				{
+					boolean: false,
+					number: true,
+					string: true,
+				},
+			],
+			// Disallow use of eval()-like methods
+			'no-implied-eval': 'error',
+			// Disallow usage of __iterator__ property
+			'no-iterator': 'error',
+			// Disallow use of labels for anything other then loops and switches
+			'no-labels': [
+				'error',
+				{
+					allowLoop: true,
+					allowSwitch: true,
+				},
+			],
+			// Disallow unnecessary nested blocks
+			'no-lone-blocks': 'error',
+			// Disallow creation of functions within loops
+			'no-loop-func': 'error',
+			// Disallow characters which are made with multiple code points in character class syntax
+			'no-misleading-character-class': 'error',
+			// Disallow use of multiline strings
+			'no-multi-str': 'error',
+			// Disallow use of new operator for Function object
+			'no-new-func': 'error',
+			// Disallows creating new instances of String, Number, and Boolean
+			'no-new-wrappers': 'error',
+			// Disallow use of octal escape sequences in string literals, such as var foo = 'Copyright \251';
+			'no-octal-escape': 'error',
+			// Disallow usage of __proto__ property
+			'no-proto': 'error',
+			// Disallow use of assignment in return statement
+			'no-return-assign': 'error',
+			// Disallow use of `javascript:` urls.
+			'no-script-url': 'error',
+			// Disallow comparisons where both sides are exactly the same
+			'no-self-compare': 'error',
+			// Disallow use of comma operator
+			'no-sequences': 'error',
+			// Restrict what can be thrown as an exception
+			'no-throw-literal': 'error',
+			// Disallow unmodified conditions of loops
+			'no-unmodified-loop-condition': 'error',
+			// Disallow usage of expressions in statement position
+			'no-unused-expressions': 'error',
+			// Disallow unnecessary .call() and .apply()
+			'no-useless-call': 'error',
+			// Disallow unnecessary catch clauses
+			'no-useless-catch': 'error',
+			// Disallow unnecessary concatenation of literals or template literals
+			'no-useless-concat': 'error',
+			// Disallow redundant return statements
+			'no-useless-return': 'error',
+			// Disallow use of void operator
+			'no-void': [
+				'error',
+				{
+					allowAsStatement: true,
+				},
+			],
+			// Disallow use of the with statement
+			'no-with': 'error',
+			// Disallow async functions which have no await expression
+			'require-await': 'error',
+			// Require or disallow Yoda conditions
+			yoda: [
+				'error',
+				'never',
+				{
+					exceptRange: true,
+				},
+			],
+		},
+	},
+];

package/rules/errors.mjs

@@ -0,0 +1,13 @@
+/** @type { import("eslint").Linter.FlatConfig[] } */
+export default [
+	{
+		// Most of possible error rules are inherited from eslint:recommended preset
+		rules: {
+			// Enforce “for” loop update clause moving the counter
+			// in the right direction
+			'for-direction': 2,
+			// Disallow use of Object.prototypes builtins directly
+			'no-prototype-builtins': 2,
+		},
+	},
+];

package/rules/es6.mjs

@@ -0,0 +1,37 @@
+import globals from 'globals';
+
+/** @type { import("eslint").Linter.FlatConfig[] } */
+export default [
+	{
+		languageOptions: {
+			parserOptions: {
+				ecmaVersion: 2024,
+				sourceType: 'module',
+				requireConfigFile: false,
+			},
+			globals: {
+				...globals.es2024,
+			},
+		},
+		rules: {
+			// Disallow duplicate imports
+			'no-duplicate-imports': 2,
+			// Disallow unnecessary constructor
+			'no-useless-constructor': 2,
+			// Disallow unnecessary computed property keys on objects
+			'no-useless-computed-key': 2,
+			// Require let or const instead of var
+			'no-var': 2,
+			// Require method and property shorthand syntax for object literals
+			'object-shorthand': [2, 'always'],
+			// Suggest using of const declaration for variables that are never modified after declared
+			'prefer-const': 2,
+			// Suggest using the spread operator instead of .apply()
+			'prefer-spread': 2,
+			// Suggest using the rest parameters instead of arguments
+			'prefer-rest-params': 2,
+			// Disallow renaming import, export, and destructured assignments to the same name
+			'no-useless-rename': 2,
+		},
+	},
+];

package/rules/legacy.mjs

@@ -0,0 +1,17 @@
+/** @type { import("eslint").Linter.FlatConfig[] } */
+export default [
+	{
+		rules: {
+			// Specify the maximum depth that blocks can be nested
+			'max-depth': [0, 5],
+			// Limits the number of parameters that can be used in the function declaration
+			'max-params': [0, 5],
+			// Specify the maximum number of statement allowed in a function
+			'max-statements': [0, 500],
+			// Disallow use of bitwise operators
+			'no-bitwise': 0,
+			// Disallow use of unary operators, ++ and --
+			'no-plusplus': 0,
+		},
+	},
+];

package/rules/node.mjs

@@ -0,0 +1,16 @@
+import globals from 'globals';
+
+/** @type { import("eslint").Linter.FlatConfig[] } */
+export default [
+	{
+		languageOptions: {
+			globals: {
+				...globals.node,
+			},
+		},
+		rules: {
+			// Disallow string concatenation with __dirname and __filename
+			'no-path-concat': 2,
+		},
+	},
+];

package/rules/react.mjs

@@ -0,0 +1,193 @@
+import react from 'eslint-plugin-react';
+
+/** @type { import("eslint").Linter.FlatConfig[] } */
+export default [
+	{
+		plugins: { react },
+		languageOptions: {
+			parserOptions: {
+				ecmaFeatures: {
+					jsx: true,
+				},
+			},
+		},
+		settings: {
+			react: {
+				version: 'detect',
+			},
+		},
+		// View link below for react rules documentation
+		// https://github.com/yannickcr/eslint-plugin-react#list-of-supported-rules
+		rules: {
+			// Prevent missing displayName in a React component definition
+			'react/display-name': 0,
+			// Forbid certain propTypes (any, array, object)
+			'react/forbid-prop-types': 0,
+			// Enforce boolean attributes notation in JSX
+			'react/jsx-boolean-value': [2, 'never'],
+			// Validate closing bracket location in JSX
+			'react/jsx-closing-bracket-location': 0,
+			// Enforce or disallow spaces inside of curly braces in JSX attributes
+			'react/jsx-curly-spacing': 0,
+			// Disallow spaces around equal signs in JSX attributes
+			'react/jsx-equals-spacing': 0,
+			// Enforce event handler naming conventions in JSX
+			'react/jsx-handler-names': [
+				2,
+				{
+					eventHandlerPrefix: 'handle',
+					eventHandlerPropPrefix: 'on',
+				},
+			],
+			// Validate JSX indentation
+			'react/jsx-indent': 0,
+			// Validate props indentation in JSX
+			'react/jsx-indent-props': 0,
+			// Validate JSX has key prop when in array or iterator
+			'react/jsx-key': 2,
+			// Limit maximum of props on a single line in JSX
+			'react/jsx-max-props-per-line': 0,
+			// Prevent usage of .bind() and arrow functions in JSX props
+			'react/jsx-no-bind': [
+				2,
+				{
+					allowArrowFunctions: true,
+				},
+			],
+			// Prevent duplicate props in JSX
+			'react/jsx-no-duplicate-props': [
+				2,
+				{
+					ignoreCase: false,
+				},
+			],
+			// Prevent usage of unwrapped JSX strings
+			'react/jsx-no-literals': 0,
+			// Prevent usage of unsafe target='_blank'
+			'react/jsx-no-target-blank': 2,
+			// Disallow undeclared variables in JSX
+			'react/jsx-no-undef': 2,
+			// Enforce PascalCase for user-defined JSX components
+			'react/jsx-pascal-case': 2,
+			// Enforce propTypes declarations alphabetical sorting
+			'react/jsx-sort-prop-types': 0,
+			// Enforce props alphabetical sorting
+			'react/jsx-sort-props': 0,
+			// Prevent React to be incorrectly marked as unused
+			'react/jsx-uses-react': 2,
+			// Prevent variables used in JSX to be incorrectly marked as unused
+			'react/jsx-uses-vars': 2,
+			// Prevent usage of dangerous JSX properties
+			'react/no-danger': 0,
+			// Prevent usage of deprecated methods
+			'react/no-deprecated': 2,
+			// Prevent usage of setState in componentDidMount
+			'react/no-did-mount-set-state': 2,
+			// Prevent usage of setState in componentDidUpdate
+			'react/no-did-update-set-state': 2,
+			// Prevent direct mutation of this.state
+			'react/no-direct-mutation-state': 2,
+			// Prevent usage of isMounted
+			'react/no-is-mounted': 2,
+			// Prevent multiple component definition per file
+			'react/no-multi-comp': [
+				2,
+				{
+					ignoreStateless: true,
+				},
+			],
+			// Prevent usage of setState
+			'react/no-set-state': 0,
+			// Prevent using string references
+			'react/no-string-refs': 2,
+			// Prevent usage of unknown DOM property
+			'react/no-unknown-property': 2,
+			// Require ES6 class declarations over React.createClass
+			'react/prefer-es6-class': [2, 'always'],
+			// Enforce stateless React Components to be written as a pure function
+			'react/prefer-stateless-function': 2,
+			// Prevent missing props validation in a React component definition
+			'react/prop-types': [
+				2,
+				{
+					ignore: [],
+					customValidators: [],
+				},
+			],
+			// Prevent missing React when using JSX
+			'react/react-in-jsx-scope': 2,
+			// Prevent extra closing tags for components without children
+			'react/self-closing-comp': [
+				2,
+				{
+					component: true,
+					html: false,
+				},
+			],
+			// Enforce component methods order
+			'react/sort-comp': [
+				2,
+				{
+					order: [
+						'static-variables',
+						'instance-variables',
+						'lifecycle',
+						'everything-else',
+						'render',
+					],
+					groups: {
+						lifecycle: [
+							'constructor',
+							'getDefaultProps',
+							'getInitialState',
+							'getChildContext',
+							'getDerivedStateFromProps',
+							'componentWillMount',
+							'UNSAFE_componentWillMount',
+							'componentDidMount',
+							'componentWillReceiveProps',
+							'UNSAFE_componentWillReceiveProps',
+							'shouldComponentUpdate',
+							'componentWillUpdate',
+							'UNSAFE_componentWillUpdate',
+							'getSnapshotBeforeUpdate',
+							'componentDidUpdate',
+							'componentDidCatch',
+							'componentWillUnmount',
+						],
+					},
+				},
+			],
+			// Prevent missing parentheses around multilines JSX
+			'react/jsx-wrap-multilines': 0,
+			// Restrict file extensions that may contain JSX
+			'react/jsx-filename-extension': 0,
+			// Prevent usage of the return value of React.render
+			'react/no-render-return-value': 2,
+			// Prevent comments from being inserted as text nodes
+			'react/jsx-no-comment-textnodes': 2,
+			// Prevent usage of findDOMNode
+			'react/no-find-dom-node': 2,
+			// Prevent problem with children and props.dangerouslySetInnerHTML
+			'react/no-danger-with-children': 2,
+			// Prevent passing of children as props
+			'react/no-children-prop': 2,
+			// Prevent invalid characters from appearing in markup
+			'react/no-unescaped-entities': 2,
+			// Configure the position of the first property
+			'react/jsx-first-prop-new-line': 0,
+			// Validate whitespace in and around the JSX opening and closing brackets
+			'react/jsx-tag-spacing': 0,
+			// Prevent void DOM elements (e.g. <img />, <br />) from receiving children
+			'react/void-dom-elements-no-children': 2,
+			// Enforce style prop value being an object
+			'react/style-prop-object': 2,
+			// Prevent usage of setState in componentWillUpdate
+			'react/no-will-update-set-state': 2,
+			// Enforce all defaultProps have a corresponding non-required PropType
+			'react/default-props-match-prop-types': 2,
+			// Prevents common typos
+			'react/no-typos': 2,
+		},
+	},
+];

package/rules/strict.mjs

@@ -0,0 +1,9 @@
+/** @type { import("eslint").Linter.FlatConfig[] } */
+export default [
+	{
+		rules: {
+			// Strict is implied in ECMAScript modules
+			strict: [2, 'never'],
+		},
+	},
+];

package/rules/style.mjs

@@ -0,0 +1,41 @@
+/** @type { import("eslint").Linter.FlatConfig[] } */
+export default [
+	{
+		// Code style should be handled by Prettier
+		rules: {
+			// Require camel case names
+			camelcase: [
+				2,
+				{
+					properties: 'never',
+				},
+			],
+			// Disallow mixed 'LF' and 'CRLF' as linebreaks
+			'linebreak-style': [2, 'unix'],
+			// Require a capital letter for constructors
+			'new-cap': [
+				2,
+				{
+					newIsCap: true,
+					capIsNew: false,
+				},
+			],
+			// Disallow use of the Array constructor
+			'no-array-constructor': 2,
+			// Disallow if as the only statement in an else block
+			'no-lonely-if': 2,
+			// Disallow use of the Object constructor
+			'no-new-object': 2,
+			// Disallow Use of Chained Assignment Expressions
+			'no-multi-assign': 2,
+			// Disallow the use of Boolean literals in conditional expressions
+			'no-unneeded-ternary': 2,
+			// Allow just one var statement per variable
+			'one-var': [2, 'never'],
+			// Disallow the Unicode Byte Order Mark (BOM)
+			'unicode-bom': [2, 'never'],
+			// Disallow mixed spaces and tabs (disabled because of Prettier)
+			'no-mixed-spaces-and-tabs': 0,
+		},
+	},
+];