eslint-plugin-putout 14.0.1 → 14.3.0

package/README.md

@@ -5,7 +5,7 @@
 [CoverageURL]: https://coveralls.io/github/coderaiser/putout?branch=master
 [CoverageIMGURL]: https://coveralls.io/repos/coderaiser/putout/badge.svg?branch=master&service=github
 
-[`ESLint`](https://eslint.org) plugin for 🐊[`Putout`](https://github.com/coderaiser/putout) with built-in rules from [`@putout/eslint-config`](https://github.com/coderaiser/putout/tree/master/packages/eslint-config).
+[**ESLint**](https://eslint.org) plugin for 🐊[**Putout**](https://github.com/coderaiser/putout) with built-in rules from [**@putout/eslint-config**](https://github.com/coderaiser/putout/tree/master/packages/eslint-config#readme).
 
 ## Installation
 
@@ -13,7 +13,7 @@
 npm i putout eslint eslint-plugin-putout -D
 ```
 
-**Note:** If you installed ESLint globally (using the `-g` flag) then you must also install `putout` and `eslint-plugin-putout` globally.
+☝️*If you installed `eslint` globally (using the `-g` flag) then you must also install `putout` and `eslint-plugin-putout` globally.*
 
 ## Usage
 
@@ -81,6 +81,10 @@ Then configure the rules you want to use under the rules section.
 
 - ✅ [Add newlines between types in union](/packages/eslint-plugin-putout/lib/add-newlines-between-types-in-union#readme)
 
+### ESM
+
+- ✅ [No unresolved](/packages/eslint-plugin-putout/lib/no-unresolved#readme)
+
 ### Formatting
 
 - ✅ [Add newline before function call](/packages/eslint-plugin-putout/lib/add-newline-before-function-call#readme)
@@ -104,11 +108,11 @@ Then configure the rules you want to use under the rules section.
 - ✅ [Remove empty specifiers](/packages/eslint-plugin-putout/lib/remove-empty-specifiers#readme)
 - ✅ [Objects braces inside array](/packages/eslint-plugin-putout/lib/objects-braces-inside-array#readme)
 - ✅ [Object init](/packages/eslint-plugin-putout/lib/object-init#readme)
-- ✅ [No unresolved](/packages/eslint-plugin-putout/lib/no-unresolved#readme)
+- ✅ [Nonblock statement body newline](/packages/eslint-plugin-putout/lib/non-block-statement-body-newline#readme)
 
 ### Safe mode
 
-When using 🐊`Putout` in IDE with `--fix` on save, or when you want to disable the most dangerous rules, use:
+When using 🐊**Putout** in IDE with `--fix` on save, or when you want to disable the most dangerous rules, use:
 
 ```json
 {
@@ -121,12 +125,12 @@ When using 🐊`Putout` in IDE with `--fix` on save, or when you want to disable
 }
 ```
 
-Disabled `ESLint` rules:
+Disabled **ESLint** rules:
 
 - [`no-useless-return`](https://eslint.org/docs/rules/no-useless-return#readme)
 - [`putout/remove-newline-from-empty-object`](https://github.com/coderaiser/putout/tree/master/packages/eslint-plugin-putout/lib/remove-newline-from-empty-object#readme)
 
-Disabled 🐊`Putout` rules:
+Disabled 🐊**Putout** rules:
 
 - 🐲[`remove-empty`](https://github.com/coderaiser/putout/tree/v24.0.0/packages/plugin-remove-empty#readme);
 - 🐲[`nodejs/remove-process-exit`](https://github.com/coderaiser/putout/tree/v24.0.0/packages/plugin-nodejs#remove-process-exit#readme);

package/lib/add-newline-after-function-call/README.md

@@ -1,10 +1,8 @@
-# Add new line after function call (`add-newline-after-function-call`)
-
-## Rule Details
+# add-newline-after-function-call
 
 This rule aims to add newline after function call.
 
-Examples of **incorrect** code for this rule:
+## ❌ Example of incorrect code
 
 ```js
 export function parse() {
@@ -15,7 +13,7 @@ export function parse() {
 }
 ```
 
-Examples of **correct** code for this rule:
+## ✅ Example of correct code
 
 ```js
 export function parse() {

package/lib/align-spaces/README.md

@@ -1,12 +1,8 @@
-# Spaces should be aligned (`aling-spaces`)
+# aling-spaces
 
-After `putout` process files it can remove spaces from empty line, this `rule` fixes it.
+When 🐊**Putout** processes files it can remove spaces from empty line, this `rule` fixes it.
 
-## Rule Details
-
-This rule aims to shorten destricturing of one property.
-
-Examples of **incorrect** code for this rule:
+## ❌ Example of incorrect code
 
 ```js
 function hello() {
@@ -18,7 +14,7 @@ function hello() {
 }
 ```
 
-Examples of **correct** code for this rule:
+## ✅ Example of correct code
 
 ```js
 function hello() {

package/lib/array-element-newline/README.md

@@ -1,19 +1,17 @@
-# Add new lines between types in union (`add-newlines-between-types-in-union`)
-
-Because [`array-element-newline`](https://eslint.org/docs/rules/array-element-newline) requires [`array-bracket-newline`](https://eslint.org/docs/rules/array-bracket-newline).
-And `array-bracket-newline` and conflicts with [`object-braces-inside-array`](https://github.com/coderaiser/putout/tree/master/packages/eslint-plugin-putout/lib/objects-braces-inside-array).
-
-## Rule Details
+# array-elements-newline
 
 This rule aims to add newlines between array elements.
+It exists because [`array-element-newline`](https://eslint.org/docs/rules/array-element-newline) requires [`array-bracket-newline`](https://eslint.org/docs/rules/array-bracket-newline) which conflicts with [`object-braces-inside-array`](https://github.com/coderaiser/putout/tree/master/packages/eslint-plugin-putout/lib/objects-braces-inside-array#readme).
+
+Part of [`eslint-plugin-putout`](https://github.com/coderaiser/putout/tree/master/packages/eslint-plugin-putout#rules).
 
-Examples of **incorrect** code for this rule:
+## ❌ Example of incorrect code
 
 ```js
 const a = [1, 2, 3, 4, 5, 6, 7];
 ```
 
-Examples of **correct** code for this rule:
+## ✅ Example of correct code
 
 ```js
 const a = [

package/lib/evaluate/README.md

@@ -1,11 +1,10 @@
 # Evaluate
 
-Evaluate expression started with `__putout_evaluate: `. Part of [`eslint-plugin-putout`](https://github.com/coderaiser/putout/tree/master/packages/eslint-plugin-putout#rules)
-
+Evaluate expression started with `__putout_evaluate: `. Part of [`eslint-plugin-putout`](https://github.com/coderaiser/putout/tree/master/packages/eslint-plugin-putout#rules).
 Provided code is processed with [`@putout/plugin-declare-undefined-variables`](https://github.com/coderaiser/putout/tree/master/packages/plugin-declare-undefined-variables):
 
 ```js
-__putout_evaluate: join("hello", " ", "world");
+__putout_evaluate: join('hello', ' ', 'world');
 ```
 
 and converted to:
@@ -13,7 +12,7 @@ and converted to:
 ```js
 const fn = (__filename, __dirname, require) => {
     const {join} = require('path');
-    return join("hello", " ", "world");
+    return join('hello', ' ', 'world');
 };
 ```
 

package/lib/for-of-multiple-properties-destructuring/README.md

@@ -1,10 +1,9 @@
-# Keep all properties in one line when using destructuring in `for-of` (`for-of-multiple-properties-destructuring`)
+# for-of-multiple-properties-destructuring
 
-## Rule Details
+This rule aims to shorten destructuring in `for-of` statements, keeping all properties in one line.
+Part of [**eslint-plugin-putout**](https://github.com/coderaiser/putout/tree/master/packages/eslint-plugin-putout#rules).
 
-This rule aims to shorten destructuring in `for-of` statements
-
-Examples of **incorrect** code for this rule:
+## ❌ Example of incorrect code
 
 ```js
 for ({
@@ -14,13 +13,13 @@ for ({
 }
 ```
 
-Examples of **correct** code for this rule:
-
-## Options
-
-`maxProperties` - maximum properties count to work with.
+## ✅ Example of correct code
 
 ```js
 for ({username, password} of users) {
 }
 ```
+
+## Options
+
+`maxProperties` - maximum properties count to work with, defaults to **8**.

package/lib/index.js

@@ -42,6 +42,7 @@ module.exports.rules = {
     ...getWrapRule('tape-add-newline-before-assertion'),
     ...getWrapRule('tape-add-newline-between-tests'),
     ...getWrapRule('tape-remove-newline-before-t-end'),
+    ...getWrapRule('nonblock-statement-body-newline'),
     ...getRule('putout'),
     ...getRule('remove-empty-newline-after-import'),
 };
@@ -83,6 +84,7 @@ const recommended = {
         'putout/tape-add-newline-before-assertion': 'error',
         'putout/tape-add-newline-between-tests': 'error',
         'putout/tape-remove-newline-before-t-end': 'error',
+        'putout/nonblock-statement-body-newline': 'error',
         'putout/putout': 'error',
         
         'node/no-unsupported-features/es-syntax': 'off',

package/lib/keyword-spacing/README.md

@@ -1,24 +1,22 @@
-# Use spaces around `catch` (`keyword-spacing`)
+# keyword-spacing
 
-When `putout` removes unused variable in `catch` [eslint keyword spacing](https://eslint.org/docs/rules/keyword-spacing) can't handle [optional catch binding](https://github.com/tc39/proposal-optional-catch-binding) correctly.
+When 🐊[**Putout**](https://github.com/coderaiser/putout) removes unused variable in `catch` [eslint keyword spacing](https://eslint.org/docs/rules/keyword-spacing) can't handle [optional catch binding](https://github.com/tc39/proposal-optional-catch-binding) correctly. This rule adds spaces around `catch`.
 
-## Rule Details
+Part of [**eslint-plugin-putout**](https://github.com/coderaiser/putout/tree/master/packages/eslint-plugin-putout#rules).
 
-This rule aims to add spaces around `catch`
-
-Examples of **incorrect** code for this rule:
+## ❌ Example of incorrect code
 
 ```js
-try {
-} catch {
+try{
+} catch{
 }
 
 try {
-} catch {
+}catch{
 }
 
 try {
-} catch(error) {
+}catch(error) {
 }
 
 if(a) {
@@ -33,7 +31,7 @@ async () => {
 };
 ```
 
-Examples of **correct** code for this rule:
+## ✅ Example of correct code
 
 ```js
 try {

package/lib/long-properties-destructuring/README.md

@@ -1,18 +1,16 @@
-# Keep each property on separate line when destructuring long properties (`long-properties-destructuring`)
+# long-properties-destructuring
 
-Always add new lines, when property name is longer then 15 characters.
+Keep each property on separate line when destructuring properties with name length **15** characters and more.
 
-## Rule Details
+Part of [**eslint-plugin-putout**](https://github.com/coderaiser/putout/tree/master/packages/eslint-plugin-putout#rules).
 
-This rule aims to shorten destricturing of one property.
-
-Examples of **incorrect** code for this rule:
+## ❌ Example of incorrect code
 
 ```js
 const {isIdentifier} = user;
 ```
 
-Examples of **correct** code for this rule:
+## ✅ Example of correct code
 
 ```js
 const {

package/lib/markdown.js

@@ -8,8 +8,6 @@ const commonRules = {
     'no-multi-spaces': 'off',
     'no-empty': 'off',
     'eol-last': 'off',
-    'quotes': 'off',
-    'quote-props': 'off',
     'no-unreachable': 'off',
     'no-constant-condition': 'off',
     'node/no-extraneous-require': 'off',

package/lib/multiple-properties-destructuring/README.md

@@ -1,10 +1,11 @@
-# Keep each property on separate line when using multiple destructuring properties (`multiple-properties-destructuring`)
+# multiple-properties-destructuring
 
-In the same way as eslint [object-property-newline](https://eslint.org/docs/rules/object-property-newline), but for destructuring.
+Keep each property on separate line when using multiple destructuring properties
+In the same way as **ESLint** [object-property-newline](https://eslint.org/docs/rules/object-property-newline), but for destructuring.
 
-## Rule Details
+Part of [**eslint-plugin-putout**](https://github.com/coderaiser/putout/tree/master/packages/eslint-plugin-putout#rules).
 
-Examples of **incorrect** code for this rule:
+## ❌ Example of incorrect code
 
 ```js
 const {a, b, c} = user;
@@ -14,7 +15,7 @@ const {a, b, c} = user;
 import {a, b, c} from 'user';
 ```
 
-Examples of **correct** code for this rule:
+## ✅ Example of correct code
 
 ```js
 const {
@@ -34,7 +35,7 @@ import {
 
 ## Options
 
-`minProperties` - minimum properties count to work with.
+`minProperties` - minimum properties count to work with, defaults: `2`.
 
 ```js
 const {

package/lib/nonblock-statement-body-newline/README.md

@@ -0,0 +1,39 @@
+# nonblock-statement-body-newline
+
+Remove newline inside statement body. Part of [**eslint-plugin-putout**](https://github.com/coderaiser/putout/tree/master/packages/eslint-plugin-putout#rules).
+
+## ❌ Example of incorrect code
+
+```js
+if (a)
+    
+    b();
+
+for (a of b)
+    
+    a();
+
+while (a)
+    
+    b();
+
+for (;;)
+    
+    a();
+```
+
+## ✅ Example of correct code
+
+```js
+if (a)
+    b();
+
+for (a of b)
+    a();
+
+while (a)
+    b();
+
+for (;;)
+    a();
+```

package/lib/nonblock-statement-body-newline/index.js

@@ -0,0 +1,55 @@
+'use strict';
+
+const {types} = require('putout');
+const {
+    isBlockStatement,
+    isIfStatement,
+    isExpressionStatement,
+} = types;
+
+const reg = /\)\n(\s+)?\n/;
+
+module.exports.report = () => `Remove useless newline`;
+
+module.exports.fix = ({text}) => {
+    const result = text.replace(reg, ')\n');
+    return result;
+};
+
+module.exports.include = () => [
+    'IfStatement',
+    'ForOfStatement',
+    'ForStatement',
+    'WhileStatement',
+];
+
+module.exports.filter = ({text, node}) => {
+    if (isIf(text, node))
+        return true;
+    
+    if (isBody(text, node))
+        return true;
+    
+    return false;
+};
+
+function isIf(text, node) {
+    if (!isIfStatement(node))
+        return false;
+    
+    const {consequent, alternate} = node;
+    
+    if (!isExpressionStatement(alternate) && !isExpressionStatement(consequent))
+        return false;
+    
+    return reg.test(text);
+}
+
+function isBody(text, node) {
+    const {body} = node;
+    
+    if (isBlockStatement(body))
+        return false;
+    
+    return reg.test(text);
+}

package/lib/single-property-destructuring/README.md

@@ -1,12 +1,11 @@
-# Keep curly braces in one line when property is single (`single-property-destructuring`)
+# single-property-destructuring
 
-When 🐊[`Putout`](https://github.com/coderaiser/putout) removes unused variables declared as destructured properties, it keeps one property to take 3 lines, instead of 1.
+When 🐊[**Putout**](https://github.com/coderaiser/putout) removes unused variables declared as destructured properties, it keeps one property to take 3 lines, instead of 1.
+This rule aims to shorten destructuring of one property, keeping curly braces in one line.
 
-## Rule Details
+Part of [**eslint-plugin-putout**](https://github.com/coderaiser/putout/tree/master/packages/eslint-plugin-putout#rules).
 
-This rule aims to shorten destructuring of one property.
-
-Examples of **incorrect** code for this rule:
+## ❌ Example of incorrect code
 
 ```js
 import {
@@ -19,7 +18,7 @@ const {
 
 ```
 
-Examples of **correct** code for this rule:
+## ✅ Example of correct code
 
 ```js
 import {password} from './user.js';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-putout",
-  "version": "14.0.1",
+  "version": "14.3.0",
   "type": "commonjs",
   "description": "eslint plugin for putout",
   "release": false,