eslint-plugin-putout 21.0.2 → 21.1.0

package/README.md

@@ -142,7 +142,8 @@ Disabled 🐊**Putout** rules:
 - ❌ [`remove-useless-arguments`](https://github.com/coderaiser/putout/tree/master/packages/plugin-remove-useless-arguments#readme);
 - ❌ [`remove-useless-return`](https://github.com/coderaiser/putout/tree/v29.0.0/packages/plugin-remove-useless-return#readme);
 - ❌ [`remove-useless-spread`](https://github.com/coderaiser/putout/tree/v29.0.0/packages/plugin-remove-useless-spread/#readme);
-- ❌ [`remove-useless-variables/rename`](https://github.com/coderaiser/putout/tree/v29.0.0/packages/plugin-remove-useless-variables#rename);
+- ❌ [`remove-useless-variables/rename`](https://github.com/coderaiser/putout/tree/v33.13.4/packages/plugin-remove-useless-variables#rename);
+- ❌ [`remove-useless-variables/declaration`](https://github.com/coderaiser/putout/tree/v33.13.4/packages/plugin-remove-useless-variables#declaration);
 - ❌ [`remove-skip`](https://github.com/coderaiser/putout/tree/v29.0.0/packages/plugin-tape#remove-skip);
 - ❌ [`remove-only`](https://github.com/coderaiser/putout/tree/v29.0.0/packages/plugin-tape#remove-only);
 - ❌ [`remove-console`](https://github.com/coderaiser/putout/tree/v29.0.0/packages/plugin-remove-console#readme);

package/lib/evaluate/index.js

@@ -31,10 +31,11 @@ function evaluate({value, filename}) {
         fix: true,
         rules: {
             'nodejs/convert-top-level-return': 'off',
+            'nodejs/convert-commonjs-to-esm': 'off',
+            'nodejs/convert-esm-to-commonjs': 'on',
         },
         plugins: [
             'nodejs',
-            'convert-esm-to-commonjs',
             'declare',
         ],
     });

package/lib/index.js

@@ -118,6 +118,8 @@ const safeRules = {
     'remove-unused-expressions': 'off',
     'remove-unreachable-code': 'off',
     'remove-useless-arguments': 'off',
+    'remove-useless-variables': 'off',
+    'remove-useless-variables/declaration': 'off',
     'remove-useless-variables/rename': 'off',
     'remove-useless-return': 'off',
     'remove-useless-spread': 'off',
@@ -132,6 +134,8 @@ const safeRules = {
     'maybe/noop': 'off',
 };
 
+module.exports.safeRules = safeRules;
+
 const safe = {
     ...recommended,
     rules: {

package/lib/putout/parse-error.js

@@ -4,9 +4,8 @@ module.exports.parseError = ({code, message, name}) => {
     if (message.includes('buildError'))
         return 'Parser error';
     
-    if (code === 'ERR_REQUIRE_ESM') {
+    if (code === 'ERR_REQUIRE_ESM')
         return `☝️ Looks like '${name}' is ESM, extend from 'plugin:putout/esm'`;
-    }
     
     return message;
 };

package/lib/putout/sync/index.js

@@ -81,7 +81,9 @@ const fix = ({ast, text, node, source, resultOptions}) => (fixer) => {
         includeComments,
     });
     
-    transform(ast, text, resultOptions);
+    transform(ast, text, {
+        ...resultOptions,
+    });
     
     const [, last] = lastToken.range;
     const code = print(ast);

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

@@ -1,8 +1,8 @@
 'use strict';
 
-const {isBuiltIn} = require('./is-built-in');
+const {isBuiltin} = require('node:module');
 const isLocal = (a) => /^\./.test(a.source.value);
-const isNode = (a) => isBuiltIn(a.source.value);
+const isNode = (a) => isBuiltin(a.source.value);
 
 const isSameGroup = (a, b) => {
     if (isLocal(a) && isLocal(b))

package/lib/single-property-destructuring/index.js

@@ -48,9 +48,8 @@ module.exports.filter = ({node, text, getText, getCommentsInside}) => {
 };
 
 module.exports.fix = ({text, node, getText}) => {
-    if (isImportDeclaration(node)) {
+    if (isImportDeclaration(node))
         return text.replace(NewLinesReg, '');
-    }
     
     const {id} = node;
     const idText = getText(id);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-putout",
-  "version": "21.0.2",
+  "version": "21.1.0",
   "type": "commonjs",
   "description": "ESLint plugin for 🐊Putout",
   "release": false,

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

@@ -1,46 +0,0 @@
-'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);