eslint-plugin-putout 14.4.0 → 14.7.0

package/README.md

@@ -132,24 +132,24 @@ Disabled **ESLint** 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);
-- 🐲[`remove-unused-variables`](https://github.com/coderaiser/putout/tree/v24.0.0/packages/plugin-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/plugin-remove-unused-for-of-variables#readme);
-- 🐲[`remove-unused-expressions`](https://github.com/coderaiser/putout/tree/v24.0.0/packages/plugin-removeunused-expressions#readme);
-- 🐲[`remove-unreferenced-variables`](https://github.com/coderaiser/putout/tree/24.1.0/packages/plugin-remove-unreferenced-variables#readme);
-- 🐲[`remove-useless-arguments`](https://github.com/coderaiser/putout/tree/master/packages/plugin-remove-useless-arguments#readme);
-- 🐲[`remove-useless-for-of`](https://github.com/coderaiser/putout/tree/master/packages/plugin-remove-useless-for-of#readme);
-- 🐲[`remove-useless-return`](https://github.com/coderaiser/putout/tree/master/packages/plugin-remove-useless-return#readme);
-- 🐲[`remove-useless-spread`](https://github.com/coderaiser/putout/tree/master/packages/plugin-remove-useless-spread/#readme#readme);
-- 🐲[`remove-useless-variables/rename`](https://github.com/coderaiser/putout/tree/master/packages/plugin-remove-useless-arguments#rename#readme);
-- 🐲[`remove-skip`](https://github.com/coderaiser/putout/tree/v24.0.0/packages/plugin-tape#remove-skip);
-- 🐲[`remove-only`](https://github.com/coderaiser/putout/tree/v24.0.0/packages/plugin-tape#remove-only);
-- 🐲[`remove-console`](https://github.com/coderaiser/putout/tree/v24.0.0/packages/plugin-remove-console#readme);
-- 🐲[`remove-debugger`](https://github.com/coderaiser/putout/tree/v24.0.0/packages/plugin-remove-debugger#readme);
-- 🐲[`remove-unreachable-code`](https://github.com/coderaiser/putout/tree/v24.0.0/packages/plugin-remove-unreachable-code#readme);
-- 🐲[`convert-for-to-for-of`](https://github.com/coderaiser/putout/tree/v24.0.0/packages/plugin-convert-for-to-for-of#readme);
+- ✅ [`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/plugin-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/plugin-remove-unused-for-of-variables#readme);
+- ✅ [`remove-unused-expressions`](https://github.com/coderaiser/putout/tree/v24.0.0/packages/plugin-removeunused-expressions#readme);
+- ✅ [`remove-unreferenced-variables`](https://github.com/coderaiser/putout/tree/24.1.0/packages/plugin-remove-unreferenced-variables#readme);
+- ✅ [`remove-useless-arguments`](https://github.com/coderaiser/putout/tree/master/packages/plugin-remove-useless-arguments#readme);
+- ✅ [`remove-useless-for-of`](https://github.com/coderaiser/putout/tree/master/packages/plugin-remove-useless-for-of#readme);
+- ✅ [`remove-useless-return`](https://github.com/coderaiser/putout/tree/master/packages/plugin-remove-useless-return#readme);
+- ✅ [`remove-useless-spread`](https://github.com/coderaiser/putout/tree/master/packages/plugin-remove-useless-spread/#readme#readme);
+- ✅ [`remove-useless-variables/rename`](https://github.com/coderaiser/putout/tree/master/packages/plugin-remove-useless-arguments#rename#readme);
+- ✅ [`remove-skip`](https://github.com/coderaiser/putout/tree/v24.0.0/packages/plugin-tape#remove-skip);
+- ✅ [`remove-only`](https://github.com/coderaiser/putout/tree/v24.0.0/packages/plugin-tape#remove-only);
+- ✅ [`remove-console`](https://github.com/coderaiser/putout/tree/v24.0.0/packages/plugin-remove-console#readme);
+- ✅ [`remove-debugger`](https://github.com/coderaiser/putout/tree/v24.0.0/packages/plugin-remove-debugger#readme);
+- ✅ [`remove-unreachable-code`](https://github.com/coderaiser/putout/tree/v24.0.0/packages/plugin-remove-unreachable-code#readme);
+- ✅ [`convert-for-to-for-of`](https://github.com/coderaiser/putout/tree/v24.0.0/packages/plugin-convert-for-to-for-of#readme);
 
 ### safe+align
 

package/lib/keyword-spacing/index.js

@@ -87,13 +87,13 @@ function checkSwitch(text) {
 
 function fixCatch(text) {
     return text
-        .replace(/catch{/g, 'catch {')
-        .replace(/}catch/g, '} catch')
-        .replace(/catch \(/g, 'catch(');
+        .replaceAll('catch{', 'catch {')
+        .replaceAll('}catch', '} catch')
+        .replaceAll('catch (', 'catch(');
 }
 
 function fixSwitch(text) {
     return text
-        .replace(/switch \(/g, 'switch(');
+        .replaceAll('switch (', 'switch(');
 }
 

package/lib/newline-function-call-arguments/README.md

@@ -1,12 +1,11 @@
-# Keep curly braces in one line when property is single (`newline-function-call-arguments`)
+# newline-function-call-arguments
 
 Addition to eslint's [function-paren-newline](https://eslint.org/docs/rules/function-paren-newline)
+This rule aims to fix eslint transform adding new line after opening brace.
 
-## Rule Details
+Part of [**eslint-plugin-putout**](https://github.com/coderaiser/putout/tree/master/packages/eslint-plugin-putout#rules).
 
-This rule aims to fix eslint transform.
-
-Examples of **incorrect** code for this rule:
+## ❌ Example of incorrect code
 
 ```js
 const onConnectError = squad(superFn(connect_error),
@@ -15,7 +14,7 @@ const onConnectError = squad(superFn(connect_error),
     getDescription);
 ```
 
-Examples of **correct** code for this rule:
+## ✅ Example of correct code
 
 ```js
 const onConnectError = squad(

package/lib/object-init/README.md

@@ -1,10 +1,10 @@
-# Keep each property on separate line when initializing an object(`object-init`)
+# object-init
 
-In the same way as eslint [object-property-newline](https://github.com/coderaiser/putout/tree/master/packages/eslint-plugin-putout/lib/multiple-properties-destructuring) but for initializing variables with `object expression`.
+Keep each property on separate line when initializing an object. In the same way as **ESLint** [`object-property-newline`](https://eslint.org/docs/rules/object-property-newline) but for initializing variables with [**Object Expression**](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Guide/Working_with_Objects).
 
-## 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 user = {name};
@@ -12,7 +12,7 @@ const user = {name};
 module.exports = {lint: 'putout lint'};
 ```
 
-Examples of **correct** code for this rule:
+## ✅ Example of correct code
 
 ```js
 const user = {

package/lib/objects-braces-inside-array/README.md

@@ -1,8 +1,9 @@
-# Keep braces on the same line as brackets (`object-braces-inside-array`)
+# object-braces-inside-array
 
-## Rule Details
+Keep braces on the same line as brackets.
+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 expected = [
@@ -15,7 +16,7 @@ const expected = [
 ];
 ```
 
-Examples of **correct** code for this rule:
+## ✅ Example of correct code
 
 ```js
 const expected = [{

package/lib/putout/index.js

@@ -10,10 +10,6 @@ const {
     parse,
 } = require('putout');
 
-const traverse = require('@babel/traverse').default;
-
-const v8 = require('v8');
-
 const parseOptions = require('putout/parse-options');
 const {parseError} = require('./parse-error');
 
@@ -23,9 +19,6 @@ const getContextOptions = ({options}) => {
     return allContextOptions;
 };
 
-const copyAST = (a) => v8.deserialize(v8.serialize(a));
-const returns = (a) => () => a;
-
 const EMPTY_VISITORS = {};
 
 module.exports = {
@@ -54,17 +47,20 @@ module.exports = {
         const {text} = source;
         const node = source.ast;
         
-        const ast = parse(text, {
-            parser: createParser(node),
+        const [errorParser, ast] = tryCatch(parse, text, {
+            isTS: true,
         });
         
-        const [error, places = []] = tryCatch(findPlaces, ast, text, resultOptions);
-        
-        if (error)
+        if (errorParser) {
             context.report({
-                message: `${parseError(error)} (putout)`,
+                message: `${parseError(errorParser)} (putout)`,
                 node,
             });
+            
+            return EMPTY_VISITORS;
+        }
+        
+        const places = findPlaces(ast, text, resultOptions);
         
         for (const {rule, message, position} of places) {
             context.report({
@@ -97,33 +93,8 @@ const fix = ({ast, text, node, source, resultOptions}) => (fixer) => {
     
     const [, last] = lastToken.range;
     
-    removeParent(ast);
     const code = print(ast);
     
     return fixer.replaceTextRange([0, last], code);
 };
 
-const createParser = (node) => {
-    const ast = copyAST(node);
-    
-    const parser = {
-        parse: returns(ast),
-    };
-    
-    return parser;
-};
-
-// ESLint adds parent to each node
-// it makes recase go crazy
-// so we better drop them
-//
-// https://github.com/eslint/eslint/blob/v8.4.0/lib/linter/linter.js#L964
-function removeParent(ast) {
-    traverse(ast, {
-        noScope: true,
-        enter(path) {
-            delete path.node.parent;
-        },
-    });
-}
-

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

@@ -1,10 +1,8 @@
-# Remove empty newline after import (`remove-empty-newline-after-import`)
+# remove-empty-newline-after-import
 
-## Rule Details
+This rule aims to remove empty newline after [`import` statement](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/import). Part of [**eslint-plugin-putout**](https://github.com/coderaiser/putout/tree/master/packages/eslint-plugin-putout#rules).
 
-This rule aims to remove empty newline after `import`.
-
-Examples of **incorrect** code for this rule:
+## ❌ Example of incorrect code
 
 ```js
 import {readFile} from 'fs';
@@ -14,7 +12,7 @@ import {promisify} from 'util';
 import index from './index.js';
 ```
 
-Examples of **correct** code for this rule:
+## ✅ Example of correct code
 
 ```js
 import {readFile} from 'fs';

package/lib/remove-empty-newline-after-import/index.js

@@ -68,6 +68,7 @@ module.exports = {
 
 function getImports(node) {
     const imports = [];
+    
     for (const current of node.body) {
         if (current.type === 'ImportDeclaration')
             imports.push(current);

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

@@ -1,7 +1,7 @@
 # remove-empty-newline-after-last-element
 
 Remove empty newline after last element.
-Part of [`eslint-plugin-putout`](https://github.com/coderaiser/putout/tree/master/packages/eslint-plugin-putout#rules).
+Part of [**eslint-plugin-putout**](https://github.com/coderaiser/putout/tree/master/packages/eslint-plugin-putout#rules).
 
 ## ❌ Example of incorrect code
 

package/lib/remove-empty-newline-before-first-specifier/README.md

@@ -1,10 +1,9 @@
-# Remove empty new line before first specifier (`remove-empty-newline-before-first-specifier`)
-
-## Rule Details
+# remove-empty-newline-before-first-specifier
 
 This rule aims to remove empty newline before first specifier.
+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
 import {
@@ -20,7 +19,7 @@ push({
 });
 ```
 
-Examples of **correct** code for this rule:
+## ✅ Example of correct code
 
 ```js
 import {

package/lib/remove-empty-specifiers/README.md

@@ -1,16 +1,15 @@
-# Remove empty specifiers(`remove-empty-specifiers`)
-
-## Rule Details
+# remove-empty-specifiers
 
 This rule aims to remove empty specifiers.
+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
 import putout, {} from 'putout';
 ```
 
-Examples of **correct** code for this rule:
+## ✅ Example of correct code
 
 ```js
 import putout from 'putout';

package/lib/remove-newline-from-empty-object/README.md

@@ -1,17 +1,16 @@
-# Remove new line in empty object (`remove-newline-from-empty-object`)
-
-## Rule Details
+# remove-newline-from-empty-object
 
 This rule aims to remove newline from empty object.
+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 = {
 };
 ```
 
-Examples of **correct** code for this rule:
+## ✅ Example of correct code
 
 ```js
 const a = {};

package/lib/ts.js

@@ -25,9 +25,7 @@ const extensionRules = {
     // broken
     '@typescript-eslint/indent': 'off',
     
-    'lines-between-class-members': 'off',
-    '@typescript-eslint/lines-between-class-members': 'error',
-    
+    '@typescript-eslint/lines-between-class-members': 'off',
     '@typescript-eslint/no-array-constructor': 'off',
     
     'no-extra-parens': 'off',
@@ -106,6 +104,7 @@ module.exports = [
 
 function convertPaddingLines([state, ...lines]) {
     const newLines = [];
+    
     for (const line of lines) {
         let {prev, next} = line;
         

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-putout",
-  "version": "14.4.0",
+  "version": "14.7.0",
   "type": "commonjs",
   "description": "eslint plugin for putout",
   "release": false,
@@ -52,7 +52,7 @@
     "@putout/test": "^5.0.0",
     "c8": "^7.5.0",
     "eslint": "^8.0.1",
-    "eslint-plugin-eslint-plugin": "^3.2.0",
+    "eslint-plugin-eslint-plugin": "^4.1.0",
     "madrun": "^9.0.0",
     "mocha": "^9.0.1",
     "montag": "^1.0.0",