eslint-plugin-putout 14.1.0 → 14.4.0

package/README.md

@@ -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,7 +108,7 @@ 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
 

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

@@ -1,12 +1,11 @@
-# Keep curly braces in one line when you use destructuring as function argument (`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.
+When 🐊[**Putout**](https://github.com/coderaiser/putout) [removes unused variables](https://github.com/coderaiser/putout/tree/master/packages/plugin-remove-unused-variables#readme) located in function argument object pattern, it formats it in a multiple lines.
+This rule aims keep curly braces in one line when you use destructuring as function argument.
 
-## 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 properties.
-
-Examples of **incorrect** code for this rule:
+## ❌ Example of incorrect code
 
 ```js
 const login = ({
@@ -16,7 +15,7 @@ const login = ({
 };
 ```
 
-Examples of **correct** code for this rule:
+## ✅ Example of correct code
 
 ```js
 const login = ({username, password}) => {

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/no-unresolved/README.md

@@ -1,13 +1,19 @@
-# Check if path can be resolved and fix if cannot (`no-unresolved`)
+# no-unresolved
 
+Check if path can be resolved and fix if cannot.
 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`.
+[`eslint-plugin-import`](https://github.com/import-js/eslint-plugin-import). But supports only `ESM` and have `autofix`.
 
-## Rule Details
+Part of [`eslint-plugin-putout`](https://github.com/coderaiser/putout/tree/master/packages/eslint-plugin-putout#rules).
 
-This rule aims to fix `unresolved import`:
+[File extension is mandatory](https://nodejs.org/api/esm.html#esm_mandatory_file_extensions) and will produce an error from `node.js`:
 
-Examples of **incorrect** code for this rule:
+```
+Error [ERR_MODULE_NOT_FOUND]: Cannot find module '/Users/coderaiser/putout/y' imported from /Users/coderaiser/putout/x.mjs
+Did you mean to import ../y.js?
+```
+
+## ❌ Example of incorrect code
 
 ```js
 import x from './y';
@@ -18,14 +24,7 @@ export * as dir from './dir';
 export {m} from './y';
 ```
 
-[File extension is mandatory](https://nodejs.org/api/esm.html#esm_mandatory_file_extensions) and will produce an error from `node.js`:
-
-```
-Error [ERR_MODULE_NOT_FOUND]: Cannot find module '/Users/coderaiser/putout/y' imported from /Users/coderaiser/putout/x.mjs
-Did you mean to import ../y.js?
-```
-
-Examples of **correct** code for this rule:
+## ✅ Example of correct code
 
 ```js
 import x from './y.js';

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/ts.js

@@ -1,6 +1,7 @@
 'use strict';
 
 const {rules} = require('@putout/eslint-config');
+const warnOnUnsupportedTypeScriptVersion = false;
 
 const extensionRules = {
     'no-undef': 'off',
@@ -68,6 +69,7 @@ const ts = {
     files: '*.ts',
     parser: '@typescript-eslint/parser',
     parserOptions: {
+        warnOnUnsupportedTypeScriptVersion,
         ecmaFeatures: {
             jsx: false,
         },
@@ -94,6 +96,7 @@ module.exports = [
         ...ts,
         files: '*.tsx',
         parserOptions: {
+            warnOnUnsupportedTypeScriptVersion,
             ecmaFeatures: {
                 jsx: true,
             },

package/package.json

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