eslint-plugin-putout 13.2.0 → 13.6.0

package/README.md

@@ -52,6 +52,8 @@ 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-import": "error",
+        "putout/remove-empty-specifiers": "error",
         "putout/objects-braces-inside-array": "error",
         "putout/object-init": "error"
     }
@@ -79,6 +81,8 @@ Then configure the rules you want to use under the rules section.
 - [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)
@@ -101,10 +105,6 @@ When using 🐊`Putout` in IDE with `--fix` on save, or when you want to disable
 }
 ```
 
-### safe+align
-
-When you want to enable ability to align spaces on empty lines, use `safe+align`.
-
 Disabled `ESLint` rules:
 
 - [`no-useless-return`](https://eslint.org/docs/rules/no-useless-return)
@@ -118,11 +118,18 @@ Disabled 🐊`Putout` rules:
 - [`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-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);
+
+### safe+align
+
+When you want to enable ability to align spaces on empty lines, while have all benefits of `safe` preset: use `safe+align`.
+

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-specifiers'),
     ...getWrapRule('objects-braces-inside-array'),
     ...getWrapRule('object-init'),
     ...getWrapRule('no-unresolved'),
@@ -41,6 +42,7 @@ module.exports.rules = {
     ...getWrapRule('tape-add-newline-between-tests'),
     ...getWrapRule('tape-remove-newline-before-t-end'),
     ...getRule('putout'),
+    ...getRule('remove-empty-newline-after-import'),
 };
 
 const config = require('@putout/eslint-config');
@@ -70,6 +72,8 @@ 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-import': 'error',
+        'putout/remove-empty-specifiers': 'error',
         'putout/objects-braces-inside-array': 'error',
         'putout/object-init': 'error',
         'putout/no-unresolved': 'error',
@@ -111,6 +115,8 @@ const safe = {
                 'remove-unreachable-code': 'off',
                 'remove-useless-return': 'off',
                 'remove-useless-arguments': 'off',
+                'remove-useless-variables/rename': 'off',
+                'remove-unreferenced-variables': 'off',
                 'tape/remove-skip': 'off',
                 'tape/remove-only': 'off',
                 'remove-console': 'off',

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

@@ -0,0 +1,24 @@
+# Remove empty newline after import (`remove-empty-newline-after-import`)
+
+## Rule Details
+
+This rule aims to remove empty newline after `import`.
+
+Examples of **incorrect** code for this rule:
+
+```js
+import {readFile} from 'fs';
+
+import {promisify} from 'util';
+
+import index from './index.js';
+```
+
+Examples of **correct** code for this rule:
+
+```js
+import {readFile} from 'fs';
+import {promisify} from 'util';
+
+import index from './index.js';
+```

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

@@ -0,0 +1,77 @@
+'use strict';
+
+const {isBuiltIn} = require('./is-built-in');
+const isLocal = (a) => /^\./.test(a.source.value);
+const isNode = (a) => isBuiltIn(a.source.value);
+
+const isSameGroup = (a, b) => {
+    if (isLocal(a) && isLocal(b))
+        return true;
+    
+    if (isNode(a) && isNode(b))
+        return true;
+    
+    return false;
+};
+
+module.exports = {
+    meta: {
+        type: 'suggestion',
+        docs: {
+            description: 'Putout',
+            category: 'putout',
+            recommended: true,
+        },
+        fixable: 'code',
+    },
+    
+    create(context) {
+        return {
+            ImportDeclaration(node) {
+                const source = context.getSourceCode();
+                const text = source.getText(node);
+                const newline = source.getText(node, 0, 2).replace(text, '');
+                
+                if (node.specifiers.length > 1)
+                    return;
+                
+                if (newline !== '\n\n')
+                    return;
+                
+                const nextNode = context.getNodeByRangeIndex(node.range[1] + 2);
+                
+                if (!nextNode || nextNode.type !== 'ImportDeclaration')
+                    return;
+                
+                if (nextNode.specifiers.length > 1)
+                    return;
+                
+                const allImports = getImports(node.parent);
+                
+                if (!isSameGroup(node, nextNode) && allImports.length > 2)
+                    return;
+                
+                context.report({
+                    node,
+                    message: 'Remove empty newline after import',
+                    
+                    fix(fixer) {
+                        return [
+                            fixer.removeRange([node.range[1], node.range[1] + 1]),
+                        ];
+                    },
+                });
+            },
+        };
+    },
+};
+
+function getImports(node) {
+    const imports = [];
+    for (const current of node.body) {
+        if (current.type === 'ImportDeclaration')
+            imports.push(current);
+    }
+    
+    return imports;
+}

package/lib/remove-empty-newline-after-import/is-built-in.js

@@ -0,0 +1,46 @@
+'use strict';
+
+const builtInsList = [
+    'assert',
+    'async_hooks',
+    'buffer',
+    'child_process',
+    'cluster',
+    'console',
+    'crypto',
+    'dgram',
+    'diagnostics_channel',
+    'dns',
+    'domain',
+    'fs',
+    'fs/promises',
+    'http',
+    'http2',
+    'https',
+    'inspector',
+    'module',
+    'os',
+    'path',
+    'perf_hooks',
+    'process',
+    'punycode',
+    'querystring',
+    'readline',
+    'repl',
+    'stream',
+    'string_decoder',
+    'timers',
+    'tls',
+    'trace_events',
+    'tty',
+    'url',
+    'util',
+    'vm',
+    'web_crypto_api',
+    'web_streams_api',
+    'wasi',
+    'worker_threads',
+    'zlib',
+];
+
+module.exports.isBuiltIn = (name) => builtInsList.includes(name);

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

@@ -0,0 +1,17 @@
+# Remove empty specifiers(`remove-empty-specifiers`)
+
+## Rule Details
+
+This rule aims to remove empty specifiers.
+
+Examples of **incorrect** code for this rule:
+
+```js
+import putout, {} from 'putout';
+```
+
+Examples of **correct** code for this rule:
+
+```js
+import putout from 'putout';
+```

package/lib/remove-empty-specifiers/index.js

@@ -0,0 +1,24 @@
+'use strict';
+
+module.exports.category = 'object';
+module.exports.report = () => 'Remove empty import specifiers';
+
+module.exports.filter = ({text, node, getCommentsInside}) => {
+    const comments = getCommentsInside(node);
+    
+    if (comments.length)
+        return false;
+    
+    return text.includes('{}');
+};
+
+module.exports.fix = ({text}) => {
+    return text
+        .replace('import {} from', 'import')
+        .replace(/,? {}/, '');
+};
+
+module.exports.include = () => [
+    'ImportDeclaration',
+];
+

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

@@ -9,22 +9,23 @@ This rule aims to shorten destructuring of one property.
 Examples of **incorrect** code for this rule:
 
 ```js
+import {
+    password,
+} from './user.js';
+
 const {
     username,
 } = user;
 
-import {
-    password,
-} from './user.js';
 ```
 
 Examples of **correct** code for this rule:
 
 ```js
-const {username} = user;
 import {password} from './user.js';
-
 import {
     helloWorld as simpleHello,
 } from './hello.js';
+
+const {username} = user;
 ```

package/package.json

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