eslint-plugin-putout 13.9.0 → 13.11.0

package/README.md

@@ -10,7 +10,7 @@
 ## Installation
 
 ```
-$ npm i putout eslint eslint-plugin-putout -D
+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.
@@ -52,6 +52,7 @@ Then configure the rules you want to use under the rules section.
         "putout/remove-newline-from-empty-object": "error",
         "putout/remove-empty-newline-before-first-specifier": "error",
         "putout/remove-empty-newline-after-last-specifier": "error",
+        "putout/remove-empty-newline-after-last-element": "error",
         "putout/remove-empty-newline-after-import": "error",
         "putout/remove-empty-specifiers": "error",
         "putout/objects-braces-inside-array": "error",
@@ -60,35 +61,36 @@ Then configure the rules you want to use under the rules section.
 }
 ```
 
-## Supported Rules
-
-- [Add newlines between types in union](/packages/eslint-plugin-putout/lib/add-newlines-between-types-in-union)
-- [Add newline before function call](/packages/eslint-plugin-putout/lib/add-newline-before-function-call)
-- [Add newline after function call](/packages/eslint-plugin-putout/lib/add-newline-after-function-call)
-- [Align spaces](/packages/eslint-plugin-putout/lib/align-spaces)
-- [Array element newline](/packages/eslint-plugin-putout/lib/array-element-newline)
-- [Evaluate](/packages/eslint-plugin-putout/lib/evaluate)
-- [Putout](lib/putout)
-- [Single property destructuring](/packages/eslint-plugin-putout/lib/single-property-destructuring)
-- [Multiple properties destructuring](/packages/eslint-plugin-putout/lib/multiple-properties-destructuring)
-- [For-of multiple properties destructuring](/packages/eslint-plugin-putout/lib/for-of-multiple-properties-destructuring)
-- [Long properties destructuring](/packages/eslint-plugin-putout/lib/long-properties-destructuring)
-- [Destructuring as function argument](/packages/eslint-plugin-putout/lib/destructuring-as-function-argument)
-- [Keyword spacing](/packages/eslint-plugin-putout/lib/keyword-spacing)
-- [Newline function call arguments](/packages/eslint-plugin-putout/lib/newline-function-call-arguments)
-- [Function declaration paren newline](/packages/eslint-plugin-putout/lib/function-declaration-paren-newline)
-- [Remove newline after default import](/packages/eslint-plugin-putout/lib/remove-newline-after-default-import)
-- [Remove newline from empty object](/packages/eslint-plugin-putout/lib/remove-newline-from-empty-object)
-- [Remove empty newline before first specifier](/packages/eslint-plugin-putout/lib/remove-empty-newline-before-first-specifier)
-- [Remove empty newline after last specifier](/packages/eslint-plugin-putout/lib/remove-empty-newline-after-last-specifier)
-- [Remove empty newline after import](/packages/eslint-plugin-putout/lib/remove-empty-newline-after-import)
-- [Remove empty specifiers](/packages/eslint-plugin-putout/lib/remove-empty-specifiers)
-- [Objects braces inside array](/packages/eslint-plugin-putout/lib/objects-braces-inside-array)
-- [Object init](/packages/eslint-plugin-putout/lib/object-init)
-- [No unresolved](/packages/eslint-plugin-putout/lib/no-unresolved)
-- [Tape: add newline before assertion]('/packages/eslint-plugin-putout/lib/tape-add-new-line-before-assertion)
-- [Tape: add newline between tests]('/packages/eslint-plugin-putout/lib/tape-add-new-line-between-tests)
-- [Tape: remove newline before t.end()]('/packages/eslint-plugin-putout/lib/tape-remove-newline-before-t-end)
+## Rules
+
+- [Add newlines between types in union](/packages/eslint-plugin-putout/lib/add-newlines-between-types-in-union#readme)
+- [Add newline before function call](/packages/eslint-plugin-putout/lib/add-newline-before-function-call#readme)
+- [Add newline after function call](/packages/eslint-plugin-putout/lib/add-newline-after-function-call#readme)
+- [Align spaces](/packages/eslint-plugin-putout/lib/align-spaces#readme)
+- [Array element newline](/packages/eslint-plugin-putout/lib/array-element-newline#readme)
+- [Evaluate](/packages/eslint-plugin-putout/lib/evaluate#readme)
+- [Putout](lib/putout#readme)
+- [Single property destructuring](/packages/eslint-plugin-putout/lib/single-property-destructuring#readme)
+- [Multiple properties destructuring](/packages/eslint-plugin-putout/lib/multiple-properties-destructuring#readme)
+- [For-of multiple properties destructuring](/packages/eslint-plugin-putout/lib/for-of-multiple-properties-destructuring#readme)
+- [Long properties destructuring](/packages/eslint-plugin-putout/lib/long-properties-destructuring#readme)
+- [Destructuring as function argument](/packages/eslint-plugin-putout/lib/destructuring-as-function-argument#readme)
+- [Keyword spacing](/packages/eslint-plugin-putout/lib/keyword-spacing#readme)
+- [Newline function call arguments](/packages/eslint-plugin-putout/lib/newline-function-call-arguments#readme)
+- [Function declaration paren newline](/packages/eslint-plugin-putout/lib/function-declaration-paren-newline#readme)
+- [Remove newline after default import](/packages/eslint-plugin-putout/lib/remove-newline-after-default-import#readme)
+- [Remove newline from empty object](/packages/eslint-plugin-putout/lib/remove-newline-from-empty-object#readme)
+- [Remove empty newline before first specifier](/packages/eslint-plugin-putout/lib/remove-empty-newline-before-first-specifier#readme)
+- [Remove empty newline after last specifier](/packages/eslint-plugin-putout/lib/remove-empty-newline-after-last-specifier#readme)
+- [Remove empty newline after last element](/packages/eslint-plugin-putout/lib/remove-empty-newline-after-last-element#readme)
+- [Remove empty newline after import](/packages/eslint-plugin-putout/lib/remove-empty-newline-after-import#readme)
+- [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)
+- [Tape: add newline before assertion]('/packages/eslint-plugin-putout/lib/tape-add-new-line-before-assertion#readme)
+- [Tape: add newline between tests]('/packages/eslint-plugin-putout/lib/tape-add-new-line-between-tests#readme)
+- [Tape: remove newline before t.end()]('/packages/eslint-plugin-putout/lib/tape-remove-newline-before-t-end#readme)
 
 ### Safe mode
 
@@ -107,28 +109,28 @@ When using 🐊`Putout` in IDE with `--fix` on save, or when you want to disable
 
 Disabled `ESLint` rules:
 
-- [`no-useless-return`](https://eslint.org/docs/rules/no-useless-return)
-- [`putout/remove-newline-from-empty-object`](https://eslint.org/docs/rules/no-useless-return)
+- [`no-useless-return`](https://eslint.org/docs/rules/no-useless-return#readme)
+- [`putout/remove-newline-from-empty-object`](https://eslint.org/docs/rules/no-useless-return#readme)
 
 Disabled 🐊`Putout` rules:
 
-- [`remove-empty`](https://github.com/coderaiser/putout/tree/v24.0.0/packages/plugin-remove-empty);
-- [`nodejs/remove-process-exit`](https://github.com/coderaiser/putout/tree/v24.0.0/packages/plugin-nodejs#remove-process-exit);
-- [`remove-unused-variables`](https://github.com/coderaiser/putout/tree/v24.0.0/packages/remove-unused-variables);
-- [`typescript/remove-unused-types`](https://github.com/coderaiser/putout/tree/v24.0.2/packages/plugin-typescript#remove-unused-types);
-- [`remove-unused-for-of-variables`](https://github.com/coderaiser/putout/tree/v24.0.0/packages/remove-unused-for-of-variables);
-- [`remove-unused-expressions`](https://github.com/coderaiser/putout/tree/v24.0.0/packages);
-- [`remove-unreferenced-variables`](https://github.com/coderaiser/putout/tree/24.1.0/packages);
-- [`remove-useless-return`](https://github.com/coderaiser/putout/tree/master/remove-useless-return);
-- [`remove-useless-arguments`](https://github.com/coderaiser/putout/tree/master/remove-useless-arguments);
-- [`remove-useless-spread`](https://github.com/coderaiser/putout/tree/master/remove-useless-spread/#readme);
-- [`remove-useless-variables/rename`](https://github.com/coderaiser/putout/tree/master/remove-useless-arguments#rename);
-- [`remove-skip`](https://github.com/coderaiser/putout/tree/v24.0.0/packages/remove-skip);
-- [`remove-only`](https://github.com/coderaiser/putout/tree/v24.0.0/packages/remove-only);
-- [`remove-console`](https://github.com/coderaiser/putout/tree/v24.0.0/packages/remove-console);
-- [`remove-debugger`](https://github.com/coderaiser/putout/tree/v24.0.0/packages/remove-debugger);
-- [`remove-unreachable-code`](https://github.com/coderaiser/putout/tree/v24.0.0/packages/remove-unreachable-code);
-- [`convert-for-to-for-of`](https://github.com/coderaiser/putout/tree/v24.0.0/packages/convert-for-to-for-of);
+- [`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);
+- [`remove-unused-variables`](https://github.com/coderaiser/putout/tree/v24.0.0/packages/remove-unused-variables#readme);
+- [`typescript/remove-unused-types`](https://github.com/coderaiser/putout/tree/v24.0.2/packages/plugin-typescript#remove-unused-types#readme);
+- [`remove-unused-for-of-variables`](https://github.com/coderaiser/putout/tree/v24.0.0/packages/remove-unused-for-of-variables#readme);
+- [`remove-unused-expressions`](https://github.com/coderaiser/putout/tree/v24.0.0/packages#readme);
+- [`remove-unreferenced-variables`](https://github.com/coderaiser/putout/tree/24.1.0/packages#readme);
+- [`remove-useless-return`](https://github.com/coderaiser/putout/tree/master/remove-useless-return#readme);
+- [`remove-useless-arguments`](https://github.com/coderaiser/putout/tree/master/remove-useless-arguments#readme);
+- [`remove-useless-spread`](https://github.com/coderaiser/putout/tree/master/remove-useless-spread/#readme#readme);
+- [`remove-useless-variables/rename`](https://github.com/coderaiser/putout/tree/master/remove-useless-arguments#rename#readme);
+- [`remove-skip`](https://github.com/coderaiser/putout/tree/v24.0.0/packages/remove-skip#readme);
+- [`remove-only`](https://github.com/coderaiser/putout/tree/v24.0.0/packages/remove-only#readme);
+- [`remove-console`](https://github.com/coderaiser/putout/tree/v24.0.0/packages/remove-console#readme);
+- [`remove-debugger`](https://github.com/coderaiser/putout/tree/v24.0.0/packages/remove-debugger#readme);
+- [`remove-unreachable-code`](https://github.com/coderaiser/putout/tree/v24.0.0/packages/remove-unreachable-code#readme);
+- [`convert-for-to-for-of`](https://github.com/coderaiser/putout/tree/v24.0.0/packages/convert-for-to-for-of#readme);
 
 ### safe+align
 

package/lib/function-declaration-paren-newline/README.md

@@ -1,12 +1,8 @@
-# Remove newlines between parens in function declaration (`function-declaration-paren-newline`)
+# function-declaration-paren-newline
 
-Similar to `eslint` [function-paren-newline](https://eslint.org/docs/rules/function-declaration-paren-newline), but forbids new lines in function declarations and expressions arguments.
+Remove newlines between parens in function declaration. Similar to `ESLint` rule [function-paren-newline](https://eslint.org/docs/rules/function-declaration-paren-newline), but forbids new lines in function declarations and expressions arguments.
 
-## Rule Details
-
-This rule aims to add spaces around `catch`
-
-Examples of **incorrect** code for this rule:
+## ❌ Incorrect code example
 
 ```js
 function f(
@@ -16,9 +12,17 @@ function f(
 const fn = (
     {a, b, c},
 ) => {};
+
+regexpTree.traverse(ast, {
+    RegExp(
+        {node},
+    ) {
+        const {body} = node;
+    },
+});
 ```
 
-Examples of **correct** code for this rule:
+## ✅ Correct code example
 
 ```js
 function f({a, b, c}) {}
@@ -31,4 +35,10 @@ const fnWithCall = ({a, b, c}) => {
         b,
     );
 };
+
+regexpTree.traverse(ast, {
+    RegExp({node}) {
+        const {body} = node;
+    },
+});
 ```

package/lib/function-declaration-paren-newline/index.js

@@ -1,16 +1,26 @@
 'use strict';
 
-const fixNewLines = (text) => {
-    return text
-        .replace(/\(\n\s+/, '(')
-        .replace(/,?\n\s+\)/, ')');
-};
-
 module.exports.report = () => {
     return `Unexpected new lines around arguments`;
 };
 
-module.exports.fix = ({text}) => fixNewLines(text);
+module.exports.fix = ({text, node, getText}) => {
+    const {body} = node.body;
+    node.body.body = [];
+    
+    const paramsText = getText(node);
+    
+    node.body.body = body;
+    
+    const newText = paramsText
+        .replace(/\(\n(\s+)?/, '(')
+        .replace(/,?\n\s+\)/, ')')
+        .replace(/,\n(\s+)?{/, ', {')
+        .replace(/},\n(\s+)?\)/, '})')
+        .replace(/,\n(\s+)?/, ', ');
+    
+    return text.replace(paramsText, newText);
+};
 
 module.exports.include = () => [
     'FunctionDeclaration',

package/lib/index.js

@@ -33,6 +33,7 @@ module.exports.rules = {
     ...getWrapRule('remove-newline-from-empty-object'),
     ...getWrapRule('remove-empty-newline-before-first-specifier'),
     ...getWrapRule('remove-empty-newline-after-last-specifier'),
+    ...getWrapRule('remove-empty-newline-after-last-element'),
     ...getWrapRule('remove-empty-specifiers'),
     ...getWrapRule('objects-braces-inside-array'),
     ...getWrapRule('object-init'),
@@ -72,6 +73,7 @@ const recommended = {
         'putout/remove-newline-from-empty-object': 'error',
         'putout/remove-empty-newline-before-first-specifier': 'error',
         'putout/remove-empty-newline-after-last-specifier': 'error',
+        'putout/remove-empty-newline-after-last-element': 'error',
         'putout/remove-empty-newline-after-import': 'error',
         'putout/remove-empty-specifiers': 'error',
         'putout/objects-braces-inside-array': 'error',

package/lib/putout/index.js

@@ -96,6 +96,8 @@ const fix = ({ast, text, node, source, resultOptions}) => (fixer) => {
     transform(ast, text, resultOptions);
     
     const [, last] = lastToken.range;
+    
+    removeParent(ast);
     const code = print(ast);
     
     return fixer.replaceTextRange([0, last], code);
@@ -103,7 +105,6 @@ const fix = ({ast, text, node, source, resultOptions}) => (fixer) => {
 
 const createParser = (node) => {
     const ast = copyAST(node);
-    removeParent(ast);
     
     const parser = {
         parse: returns(ast),

package/lib/remove-empty-newline-after-last-element/README.md

@@ -0,0 +1,25 @@
+# remove-empty-newline-after-last-element
+
+## Rule Details
+
+This rule aims to remove empty newline after last element.
+
+## ❌ Incorrect code example
+
+```js
+push([
+    a,
+    b,
+
+]);
+```
+
+## ✅ Correct code example
+
+```js
+push([
+    a,
+    b,
+
+]);
+```

package/lib/remove-empty-newline-after-last-element/index.js

@@ -0,0 +1,20 @@
+'use strict';
+
+module.exports.category = 'array';
+module.exports.report = () => 'Remove newline after last element';
+
+const regExp = /\n\n(\s+)?]/;
+
+module.exports.filter = ({text}) => {
+    return regExp.test(text);
+};
+
+module.exports.fix = ({text}) => {
+    return text
+        .replace(regExp, '\n]');
+};
+
+module.exports.include = () => [
+    'ArrayExpression',
+];
+

package/lib/remove-empty-newline-after-last-specifier/README.md

@@ -1,10 +1,10 @@
-# Remove empty new line after last specifier(`remove-empty-newline-after-last-specifier`)
+# remove-empty-newline-after-last-specifier
 
 ## Rule Details
 
 This rule aims to remove empty newline after last specifier.
 
-Examples of **incorrect** code for this rule:
+## ❌ Incorrect code example
 
 ```js
 import {
@@ -20,7 +20,7 @@ push({
 });
 ```
 
-Examples of **correct** code for this rule:
+## ✅ Correct code example
 
 ```js
 import {

package/lib/remove-newline-after-default-import/README.md

@@ -1,12 +1,10 @@
-# Keep opening curly brace on one line with default import (`remove-newline-after-default-import`)
+# remove-newline-after-default-import
 
 Fixes [`object-curly-newline`](https://eslint.org/docs/rules/object-curly-newline) + [`eslint-plugin-modules-newline`](https://github.com/gmsorrow/eslint-plugin-modules-newline).
 
-## Rule Details
+This rule aims to remove newline after default import, before opening curly brace (`{`).
 
-This rule aims to shorten destructuring of properties.
-
-Examples of **incorrect** code for this rule:
+## ❌ Incorrect code example
 
 ```js
 import x,
@@ -16,7 +14,7 @@ import x,
 } from 'y';
 ```
 
-Examples of **correct** code for this rule:
+## ✅ Correct code example
 
 ```js
 import x, {

package/package.json

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