eslint-plugin-putout 11.16.1 → 12.0.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/eslint-plugin-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).
 
 ## Installation
 
@@ -17,7 +17,7 @@ $ npm i putout eslint eslint-plugin-putout -D
 
 ## Usage
 
-Add `putout` to the plugins section of your `.eslintrc` configuration file. You can omit the `eslint-plugin-` prefix:
+Add `putout` to the plugins section of your `.eslintrc.json` configuration file. You can omit the `eslint-plugin-` prefix:
 
 ```json
 {
@@ -88,7 +88,7 @@ Then configure the rules you want to use under the rules section.
 
 ### 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
 {
@@ -101,16 +101,17 @@ When using `putout` in IDE with `--fix` on save, or when you want to disable the
 }
 ```
 
-List of disabled `putout` rules:
+List of disabled 🐊`Putout` rules:
 
-- [remove-empty](https://github.com/coderaiser/putout/tree/v20.0.0/packages/plugin-remove-empty);
-- [remove-unused-variables](https://github.com/coderaiser/putout/tree/v20.0.0/packages/remove-unused-variables);
-- [remove-unused-types](https://github.com/coderaiser/putout/tree/v20.0.0/packages/remove-unused-types);
-- [remove-unused-for-of-variables](https://github.com/coderaiser/putout/tree/v20.0.0/packages/remove-unused-for-of-variables);
-- [remove-unused-expressions](https://github.com/coderaiser/putout/tree/v20.0.0/packages);
+- [remove-empty](https://github.com/coderaiser/putout/tree/v22.0.0/packages/plugin-remove-empty);
+- [remove-unused-variables](https://github.com/coderaiser/putout/tree/v22.0.0/packages/remove-unused-variables);
+- [remove-unused-types](https://github.com/coderaiser/putout/tree/v22.0.0/packages/remove-unused-types);
+- [remove-unused-for-of-variables](https://github.com/coderaiser/putout/tree/v22.0.0/packages/remove-unused-for-of-variables);
+- [remove-unused-expressions](https://github.com/coderaiser/putout/tree/v22.0.0/packages);
 - [remove-useless-return](https://github.com/coderaiser/putout/tree/master/remove-useless-return);
-- [remove-skip](https://github.com/coderaiser/putout/tree/v20.0.0/packages/remove-skip);
-- [remove-only](https://github.com/coderaiser/putout/tree/v20.0.0/packages/remove-only);
-- [remove-console](https://github.com/coderaiser/putout/tree/v20.0.0/packages/remove-console);
-- [remove-debugger](https://github.com/coderaiser/putout/tree/v20.0.0/packages/remove-debugger);
-- [convert-for-to-for-of](https://github.com/coderaiser/putout/tree/v20.0.0/packages/convert-for-to-for-of);
+- [remove-skip](https://github.com/coderaiser/putout/tree/v22.0.0/packages/remove-skip);
+- [remove-only](https://github.com/coderaiser/putout/tree/v22.0.0/packages/remove-only);
+- [remove-console](https://github.com/coderaiser/putout/tree/v22.0.0/packages/remove-console);
+- [remove-debugger](https://github.com/coderaiser/putout/tree/v22.0.0/packages/remove-debugger);
+- [remove-unreachable-code](https://github.com/coderaiser/putout/tree/v22.0.0/packages/remove-unreachable-code);
+- [convert-for-to-for-of](https://github.com/coderaiser/putout/tree/v22.0.0/packages/convert-for-to-for-of);

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

@@ -1,4 +1,4 @@
-# Add new line after function call (add-newline-after-function-call)
+# Add new line after function call (`add-newline-after-function-call`)
 
 ## Rule Details
 

package/lib/add-newline-after-function-call/index.js

@@ -12,7 +12,7 @@ const {
 
 const regExp = /^;?\n( +)?\n +$/;
 
-module.exports.category = 'typescript';
+module.exports.category = 'layout';
 module.exports.report = () => 'Add newline after function call';
 
 module.exports.filter = ({text, node, getText, getCommentsAfter}) => {
@@ -47,6 +47,7 @@ module.exports.filter = ({text, node, getText, getCommentsAfter}) => {
         if (i === n - 1)
             break;
         
+        const prev = body[i - 1];
         const next = body[i + 1];
         
         if (!isVariableDeclaration(next))
@@ -62,8 +63,14 @@ module.exports.filter = ({text, node, getText, getCommentsAfter}) => {
         
         const spacesAfterNext = getSpacesAfterNode(next, {getText});
         
-        if (!regExp.test(spacesAfterNext))
+        if (regExp.test(spacesAfterNext))
+            break;
+        
+        if (!prev)
             return true;
+        
+        const spacesAfterPrev = getSpacesAfterNode(prev, {getText});
+        return !regExp.test(spacesAfterPrev);
     }
     
     return false;

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

@@ -1,4 +1,4 @@
-# Add new line before function call (add-newline-before-function-call)
+# Add new line before function call (`add-newline-before-function-call`)
 
 ## Rule Details
 

package/lib/add-newline-before-function-call/index.js

@@ -43,6 +43,7 @@ module.exports.filter = ({text, node, getText, getCommentsBefore}) => {
             continue;
         
         const prevA = body[i - 1];
+        const nextA = body[i + 1];
         
         if (!isVariableDeclaration(prevA))
             return false;
@@ -52,7 +53,11 @@ module.exports.filter = ({text, node, getText, getCommentsBefore}) => {
         if (regExp.test(spaces))
             return false;
         
-        return true;
+        if (!nextA)
+            return true;
+        
+        const nextSpaces = getSpacesBeforeNode(nextA, {getText});
+        return !regExp.test(nextSpaces);
     }
     
     return false;

package/lib/add-newlines-between-types-in-union/README.md

@@ -1,4 +1,4 @@
-# Add new lines between types in union (add-newlines-between-types-in-union)
+# Add new lines between types in union (`add-newlines-between-types-in-union`)
 
 ## Rule Details
 

package/lib/align-spaces/README.md

@@ -1,4 +1,4 @@
-# Spaces should be aligned (aling-spaces)
+# Spaces should be aligned (`aling-spaces`)
 
 After `putout` process files it can remove spaces from empty line, this `rule` fixes it.
 

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

@@ -1,4 +1,4 @@
-# Add new lines between types in union (add-newlines-between-types-in-union)
+# 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).

package/lib/destructuring-as-function-argument/README.md

@@ -1,4 +1,4 @@
-# keep curly bracesin in one line when you use destructuring as function argument (destructuring-as-function-argument)
+# Keep curly braces in one line when you use destructuring as function argument (`destructuring-as-function-argument`)
 
 When `putout` removes unused variable located in function argument object pattern, it formats in in a multiple lines.
 

package/lib/evaluate/README.md

@@ -1,4 +1,4 @@
-# evaluate expression (evaluate)
+# evaluate expression (`evaluate`)
 
 Evaluate expression started with `__putout_evaluate: `.
 Provided code is processed with [`@putout/plugin-declare-undefined-variables`](https://github.com/coderaiser/putout/tree/master/packages/plugin-declare-undefined-variables). So next code:

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

@@ -1,4 +1,4 @@
-# Keep all properties in one line when using destructuring in for-of (for-of-multiple-properties-destructuring)
+# Keep all properties in one line when using destructuring in `for-of` (`for-of-multiple-properties-destructuring`)
 
 ## Rule Details
 

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

@@ -1,4 +1,4 @@
-# Use spaces around "catch" (function-declaration-paren-newline)
+# Remove newlines between parens in function declaration (`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.
 

package/lib/index.js

@@ -105,6 +105,7 @@ const safe = {
                 'remove-unused-variables': 'off',
                 'remove-unused-expressions': 'off',
                 'remove-unused-for-of-variables': 'off',
+                'remove-unreachable-code': 'off',
                 'remove-useless-return': 'off',
                 'tape/remove-skip': 'off',
                 'tape/remove-only': 'off',

package/lib/keyword-spacing/README.md

@@ -1,4 +1,4 @@
-# Use spaces around "catch" (keyword-spacing)
+# Use spaces around `catch` (`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.
 

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

@@ -1,4 +1,4 @@
-# keep each property on separate line when destructuring long properties (long-properties-destructuring)
+# Keep each property on separate line when destructuring long properties (`long-properties-destructuring`)
 
 Always add new lines, when property name is longer then 15 characters.
 

package/lib/markdown.js

@@ -33,13 +33,19 @@ module.exports = [{
             parserOpts,
             plugins: [
                 '@babel/plugin-syntax-class-properties',
-                '@babel/plugin-syntax-top-level-await',
             ],
         },
     },
 }, {
     ...ts,
     files: '*.md{ts}',
-    rules: commonRules,
+    rules: {
+        ...commonRules,
+        ...ts.rules,
+        '@typescript-eslint/no-unused-vars': 'off',
+        '@typescript-eslint/no-explicit-any': 'off',
+        '@typescript-eslint/no-inferrable-types': 'off',
+        '@typescript-eslint/array-type': 'off',
+    },
 }];
 

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

@@ -1,4 +1,4 @@
-# keep each property on separate line when using multiple destructuring properties (multiple-properties-destructuring)
+# Keep each property on separate line when using multiple destructuring properties (`multiple-properties-destructuring`)
 
 In the same way as eslint [object-property-newline](https://eslint.org/docs/rules/object-property-newline), but for destructuring.
 

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

@@ -1,4 +1,4 @@
-# keep curly bracesin in one line when property is single (newline-function-call-arguments)
+# Keep curly braces in one line when property is single (`newline-function-call-arguments`)
 
 Addition to eslint's [function-paren-newline](https://eslint.org/docs/rules/function-paren-newline)
 

package/lib/no-unresolved/README.md

@@ -1,4 +1,4 @@
-# check if path can be resolved and fix if cannot (no-unresolved)
+# Check if path can be resolved and fix if cannot (`no-unresolved`)
 
 Similar to [`no-unresolved`](https://github.com/import-js/eslint-plugin-import/blob/HEAD/docs/rules/no-unresolved.md) from
 [`eslint-plugin-putout`](https://github.com/import-js/eslint-plugin-import). But supports only `ESM` and have `autofix`.

package/lib/object-init/README.md

@@ -1,4 +1,4 @@
-# keep each property on separate line when initializing an object(object-init)
+# Keep each property on separate line when initializing an object(`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`.
 

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

@@ -1,4 +1,4 @@
-# keep braces on the same line as brackets (object-braces-inside-array)
+# Keep braces on the same line as brackets (`object-braces-inside-array`)
 
 ## Rule Details
 

package/lib/putout/README.md

@@ -1,4 +1,4 @@
-# Putout (putout)
+# Putout (`putout`)
 
 🐊[`Putout`](https://github.com/coderaiser/putout) used as eslint plugin.
 

package/lib/putout/index.js

@@ -5,6 +5,7 @@ const {
     findPlaces,
     transform,
     print,
+    parse,
 } = require('putout');
 
 const v8 = require('v8');
@@ -47,7 +48,12 @@ module.exports = {
                 const source = context.getSourceCode();
                 const {text} = source;
                 
-                const ast = toBabel(copyAST(node));
+                const ast = parse(text, {
+                    parser: {
+                        parse: () => toBabel(copyAST(node)),
+                    },
+                });
+                
                 const places = findPlaces(ast, text, resultOptions);
                 
                 for (const {rule, message, position} of places) {

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

@@ -1,4 +1,4 @@
-# Remove empty new line after last specifier(remove-empty-newline-after-last-specifier)
+# Remove empty new line after last specifier(`remove-empty-newline-after-last-specifier`)
 
 ## Rule Details
 

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

@@ -1,4 +1,4 @@
-# Remove empty new line before first specifier(remove-empty-newline-before-first-specifier)
+# Remove empty new line before first specifier (`remove-empty-newline-before-first-specifier`)
 
 ## Rule Details
 

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

@@ -1,6 +1,6 @@
-# Keep opening curly brace on one line with default import (remove-newline-after-default-import)
+# Keep opening curly brace on one line with 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).
+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
 

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

@@ -1,4 +1,4 @@
-# Remove new line in empty object (remove-newline-from-empty-object)
+# Remove new line in empty object (`remove-newline-from-empty-object`)
 
 ## Rule Details
 

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

@@ -1,6 +1,6 @@
-# keep curly bracesin in one line when property is single (single-property-destructuring)
+# Keep curly braces in one line when property is single (`single-property-destructuring`)
 
-When `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.
 
 ## Rule Details
 

package/lib/tape-add-newline-before-assertion/README.md

@@ -1,6 +1,6 @@
-# Add newline before assertion (add-newline-before-assertion)
+# Add newline before assertion (`add-newline-before-assertion`)
 
-Add newline before `t.equal()` etc, for [supertape](https://github.com/coderaiser/supertape).
+Add newline before `t.equal()` etc, for [`supertape`](https://github.com/coderaiser/supertape).
 
 ## Rule Details
 

package/lib/tape-add-newline-between-tests/README.md

@@ -1,6 +1,6 @@
-# Add newline between tests (add-newline-between-tests)
+# Add newline between tests (`add-newline-between-tests`)
 
-Add newline between tests, for [supertape](https://github.com/coderaiser/supertape).
+Add newline between tests, for [`supertape`](https://github.com/coderaiser/supertape).
 
 ## Rule Details
 

package/lib/tape-remove-newline-before-t-end/README.md

@@ -1,6 +1,6 @@
-# Remove newline before `t.end()` (remove-newline-before-t-end)
+# Remove newline before `t.end()` (`remove-newline-before-t-end`)
 
-Remove newline before `t.end()`, for [supertape](https://github.com/coderaiser/supertape).
+Remove newline before `t.end()`, for [`supertape`](https://github.com/coderaiser/supertape).
 
 ## Rule Details
 

package/lib/ts.js

@@ -1,21 +1,44 @@
 'use strict';
 
-const parserOpts = require('@putout/engine-parser/babel/options');
-
-module.exports = [{
+const ts = {
     files: '*.ts',
-    parser: '@babel/eslint-parser/experimental-worker',
+    parser: '@typescript-eslint/parser',
     parserOptions: {
-        requireConfigFile: false,
-        babelOptions: {
-            parserOpts,
-            plugins: ['@babel/plugin-syntax-typescript'],
+        ecmaFeatures: {
+            jsx: false,
         },
     },
+    plugins: [
+        '@typescript-eslint',
+    ],
+    extends: [
+        'plugin:@typescript-eslint/recommended',
+    ],
     rules: {
         'no-undef': 'off',
         'no-var': 'off',
         'putout/no-unresolved': 'off',
+        'semi': 'off',
+        '@typescript-eslint/semi': 'error',
+        '@typescript-eslint/array-type': 'error',
+        'space-before-function-paren': 'off',
+        '@typescript-eslint/space-before-function-paren': ['error', {
+            anonymous: 'never',
+            named: 'never',
+            asyncArrow: 'always',
+        }],
+    },
+};
+
+module.exports = [
+    ts, {
+        ...ts,
+        files: '*.tsx',
+        parserOptions: {
+            ecmaFeatures: {
+                jsx: true,
+            },
+        },
     },
-}];
+];
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-putout",
-  "version": "11.16.1",
+  "version": "12.0.0",
   "description": "eslint plugin for putout",
   "release": false,
   "tag": false,
@@ -36,14 +36,15 @@
     "@babel/core": "^7.12.3",
     "@babel/eslint-parser": "^7.15.0",
     "@babel/plugin-syntax-class-properties": "^7.12.1",
-    "@babel/plugin-syntax-top-level-await": "^7.12.1",
-    "@babel/plugin-syntax-typescript": "^7.12.1",
     "@putout/engine-parser": "^4.6.0",
     "@putout/eslint-config": "^6.0.0",
+    "@typescript-eslint/eslint-plugin": "^5.5.0",
+    "@typescript-eslint/parser": "^5.4.0",
     "align-spaces": "^1.0.0",
     "eslint-plugin-node": "^11.0.0",
     "estree-to-babel": "^4.0.1",
-    "try-catch": "^3.0.0"
+    "try-catch": "^3.0.0",
+    "typescript": "^4.5.2"
   },
   "devDependencies": {
     "c8": "^7.5.0",
@@ -54,14 +55,15 @@
     "montag": "^1.0.0",
     "simport": "^1.2.0",
     "supertape": "^6.6.0",
-    "try-to-catch": "^3.0.0"
+    "try-to-catch": "^3.0.0",
+    "@babel/plugin-syntax-typescript": "^7.12.1"
   },
   "engines": {
     "node": ">=14"
   },
   "peerDependencies": {
     "eslint": ">=8.0.0",
-    "putout": ">=21"
+    "putout": ">=22"
   },
   "license": "MIT",
   "publishConfig": {