putout 24.2.0 → 24.4.0

package/ChangeLog

@@ -1,3 +1,69 @@
+2022.01.25, v24.4.0
+
+feature:
+- (putout) cli: move out runner
+- (@putout/eslint-config) padding-line-between-statements: add import
+- (@putout/eslint-config) padding-line-between-statements: add export
+- (@putout/eslint-config) enable operator-linebreak
+- (@putout/test) add ESLint support
+
+
+2022.01.24, v24.3.0
+
+feature:
+- (putout) eslint: add ability to enable putout/putout rule with a flag
+- (putout/eslint) add ability to enable putout
+
+
+2022.01.24, v24.2.2
+
+fix:
+- (@putout/plugin-declare-undefined-variables) node-js: exclude "promises" as it is to general
+- (@putout/engine-runner) avoid duplication of plugins
+
+feature:
+- (package) @putout/plugin-convert-esm-to-commonjs v4.0.0
+- (@putout/plugin-convert-esm-to-commonjs) drop support of putout < 24
+- (@putout/plugin-convert-esm-to-commonjs) Commonjs -> CommonJS
+- (@putout/engine-runner) add ability to debug with "putout:runner:replace"
+
+
+2022.01.21, v24.2.1
+
+fix:
+- (@putout/test) avoid updating noTransform files
+
+feature:
+- (package) @putout/plugin-remove-useless-variables v6.0.0
+- (@putout/plugin-remove-useless-variables) drop support of putout < 24
+- (package) @putout/plugin-remove-unused-for-of-variables v2.0.0
+- (package) @putout/plugin-convert-for-to-for-of v3.0.0
+- (package) @putout/plugin-tape v8.0.0
+- (@putout/plugin-tape) drop support of putout < 24
+- (@putout/plugin-convert-for-to-for-of) drop support of putout < 24
+- (package) @putout/operator-add-args v2.0.0
+- (@putout/operator-add-args) drop support of putout < 24
+- (@putout/operator-declare) improve variables declaration
+- (package) @putout/plugin-remove-nested-blocks v5.0.0
+- (@putout/plugin-remove-nested-blocks) drop support of putout < 24
+- (package) @putout/plugin-putout v10.0.0
+- (@putout/plugin-putout) drop support of putout < 24.2
+- (package) @putout/plugin-merge-duplicate-imports v5.0.0
+- (@putout/plugin-merge-duplicate-imports) drop support of putout < 24
+- (package) @putout/plugin-merge-destructuring-properties v6.0.0
+- (@putout/plugin-merge-destructuring-properties) drop support of putout < 24
+- (package) @putout/plugin-extract-object-properties v7.0.0
+- (@putout/plugin-extract-object-properties) drop support of putout < 24
+- (package) @putout/plugin-convert-mock-require-to-mock-import v2.0.0
+- (@putout/plugin-convert-mock-require-to-mock-import) drop support of putout < 24
+- (package) @putout/plugin-convert-for-each-to-for-of v7.0.0
+- (@putout/plugin-convert-for-each-to-for-of) drop support of putout < 24
+- (package) @putout/plugin-convert-commonjs-to-esm v8.0.0
+- (@putout/plugin-convert-commonjs-to-esm) drop support of putout < 24
+- (package) @putout/operator-declare v3.0.0
+- (@putout/operator-declare) drop support of putout < 24
+
+
 2022.01.20, v24.2.0
 
 fix:

package/lib/cli/eslint/index.js

@@ -49,7 +49,7 @@ const getESLint = ({fix, config}) => {
     };
 };
 
-module.exports = async ({name, code, fix, config}) => {
+module.exports = async ({name, code, fix, config, putout = false}) => {
     const noChanges = [
         code,
         [],
@@ -81,7 +81,7 @@ module.exports = async ({name, code, fix, config}) => {
         ];
     }
     
-    disablePutout(finalConfig);
+    !putout && disablePutout(finalConfig);
     
     // that's right, we disabled "putout" rules in "config"
     // and now it located in eslint's cache

package/lib/cli/index.js

@@ -1,50 +1,39 @@
 'use strict';
 
-const {resolve} = require('path');
-const {readFileSync} = require('fs');
-
 const {red} = require('chalk');
 const yargsParser = require('yargs-parser');
 const {isCI} = require('ci-info');
 const memo = require('nano-memoize');
-const fullstore = require('fullstore');
 const tryCatch = require('try-catch');
-const tryToCatch = require('try-to-catch');
 const wraptile = require('wraptile');
+const fullstore = require('fullstore');
+
+const keyPress = require('@putout/cli-keypress');
 const {version} = require('../../package.json');
 const {simpleImport} = require('./simple-import');
-
-const {env} = process;
-const isIDE = /JetBrains/.test(env.TERMINAL_EMULATOR) || env.TERM_PROGRAM === 'vscode';
-const chooseName = (name, resolvedName) => !isIDE ? name : resolvedName;
+const {run} = require('./runner/runner.js');
 
 const {
-    runProcessors,
     getFilePatterns,
     getProcessorRunners,
     defaultProcessors,
 } = require('@putout/engine-processor');
 
 const merge = require('../merge');
-const ignores = require('../ignores');
 
-const initProcessFile = require('./process-file');
 const getFiles = require('./get-files');
 const {createCache} = require('@putout/cli-cache');
 const supportedFiles = require('./supported-files');
 const getFormatter = memo(require('./formatter').getFormatter);
 const getOptions = require('./get-options');
-const report = require('./report')();
-const keyPress = require('@putout/cli-keypress');
+
 const validateArgs = require('@putout/cli-validate-args');
-const parseError = require('./parse-error');
 
 const {
     OK,
     PLACE,
     STAGE,
     NO_FILES,
-    NO_PROCESSORS,
     CANNOT_LOAD_PROCESSOR,
     WAS_STOP,
     INVALID_OPTION,
@@ -64,20 +53,6 @@ const getExitCode = (wasStop) => wasStop() ? WAS_STOP : OK;
 
 const isStr = (a) => typeof a === 'string';
 const {isArray} = Array;
-const isParser = (rule) => /^parser/.test(rule);
-const isParsingError = ({rule}) => isParser(rule);
-
-const createFormatterProxy = (options) => {
-    return new Proxy(options, {
-        get(target, name) {
-            if (target[name])
-                return target[name];
-            
-            if (name === 'source')
-                return readFileSync(target.name, 'utf8');
-        },
-    });
-};
 
 module.exports = async ({argv, halt, log, write, logError, readFile, writeFile}) => {
     const {isStop} = keyPress();
@@ -299,114 +274,29 @@ module.exports = async ({argv, halt, log, write, logError, readFile, writeFile})
         plugins,
     };
     
-    const rawPlaces = [];
+    const {rawPlaces, exited} = await run({
+        fix,
+        exit,
+        readFile,
+        writeFile,
+        raw,
+        rulesdir,
+        names,
+        options,
+        fileCache,
+        currentFormat,
+        formatterOptions,
+        write,
+        log,
+        isStop,
+        wasStop,
+        noConfig,
+        plugins,
+        transform,
+    });
     
-    const processFile = initProcessFile(options);
-    const {length} = names;
-    
-    for (let index = 0; index < length; index++) {
-        if (wasStop())
-            break;
-        
-        wasStop(isStop());
-        
-        const currentIndex = isStop() ? length - 1 : index;
-        const name = names[index];
-        const resolvedName = resolve(name)
-            .replace(/^\./, cwd);
-        
-        const [configError, options] = tryCatch(getOptions, {
-            name: resolvedName,
-            rulesdir,
-            noConfig,
-            transform,
-            plugins,
-        });
-        
-        if (configError)
-            return exit(INVALID_CONFIG, configError);
-        
-        const {dir} = options;
-        
-        if (fileCache.canUseCache(name, options)) {
-            const places = fileCache.getPlaces(name);
-            const formatterProxy = createFormatterProxy({
-                report,
-                formatterOptions,
-                name: chooseName(name, resolvedName),
-                places,
-                index: currentIndex,
-                count: length,
-            });
-            
-            const line = await report(currentFormat, formatterProxy);
-            
-            write(line || '');
-            rawPlaces.push(places);
-            continue;
-        }
-        
-        let isProcessed = true;
-        let places = [];
-        let rawSource = '';
-        let processedSource = '';
-        
-        if (!ignores(dir, resolvedName, options)) {
-            rawSource = await readFile(resolvedName, 'utf8');
-            
-            const [error, result] = await tryToCatch(runProcessors, {
-                name: resolvedName,
-                fix,
-                processFile,
-                options,
-                rawSource,
-                processorRunners,
-            });
-            
-            if (error) {
-                places = parseError(error);
-                
-                isProcessed = true;
-                processedSource = rawSource;
-                
-                if (raw)
-                    log(error);
-            } else {
-                ({
-                    isProcessed,
-                    places,
-                    processedSource,
-                } = result);
-            }
-        }
-        
-        const line = await report(currentFormat, {
-            report,
-            formatterOptions,
-            name: chooseName(name, resolvedName),
-            source: rawSource,
-            places,
-            index: currentIndex,
-            count: length,
-        });
-        
-        write(line || '');
-        
-        if (!isProcessed)
-            return exit(NO_PROCESSORS, Error(`No processors found for ${name}`));
-        
-        if (rawSource !== processedSource) {
-            fileCache.removeEntry(name);
-            await writeFile(name, processedSource);
-        }
-        
-        const fixable = !places.filter(isParsingError).length;
-        
-        if (fixable)
-            fileCache.setInfo(name, places, options);
-        
-        rawPlaces.push(places);
-    }
+    if (exited)
+        return;
     
     const mergedPlaces = merge(...rawPlaces);
     
@@ -445,6 +335,10 @@ const getExit = ({halt, raw, logError}) => (code, e) => {
     
     logError(message);
     halt(code);
+    
+    return {
+        exited: true,
+    };
 };
 
 module.exports._addOnce = addOnce;
@@ -452,4 +346,3 @@ function addOnce(emitter, name, fn) {
     if (!emitter.listenerCount(name))
         emitter.on(name, fn);
 }
-

package/lib/cli/process-file.js

@@ -3,13 +3,12 @@
 const tryCatch = require('try-catch');
 
 const putout = require('../..');
+const merge = require('../merge');
+const parseMatch = require('../parse-options/parse-match');
 
 const eslint = require('./eslint');
 const parseError = require('./parse-error');
 
-const merge = require('../merge');
-const parseMatch = require('../parse-options/parse-match');
-
 const getMatchedOptions = (name, options) => {
     if (!name.includes('{'))
         return options;

package/lib/cli/runner/runner.js

@@ -0,0 +1,49 @@
+'use strict';
+
+const runWorker = require('./worker.js');
+const initProcessFile = require('../process-file.js');
+const Report = require('../report.js');
+
+const report = Report();
+
+module.exports.run = async ({transform, plugins, noConfig, readFile, writeFile, exit, isStop, wasStop, names, write, log, rulesdir, fix, processorRunners, fileCache, currentFormat, formatterOptions, options, raw}) => {
+    const processFile = initProcessFile(options);
+    const {length} = names;
+    const rawPlaces = [];
+    
+    for (let index = 0; index < length; index++) {
+        if (wasStop())
+            break;
+        
+        const {exited} = await runWorker({
+            readFile,
+            writeFile,
+            exit,
+            isStop,
+            wasStop,
+            fix,
+            processorRunners,
+            rulesdir,
+            currentFormat,
+            formatterOptions,
+            index,
+            names,
+            length,
+            rawPlaces,
+            processFile,
+            fileCache,
+            raw,
+            write,
+            log,
+            report,
+            noConfig,
+            plugins,
+            transform,
+        });
+        
+        if (exited)
+            return {exited, rawPlaces};
+    }
+    
+    return {rawPlaces};
+};

package/lib/cli/runner/worker.js

@@ -0,0 +1,146 @@
+'use strict';
+
+const {resolve} = require('path');
+const {
+    cwd,
+    env,
+} = require('process');
+const tryCatch = require('try-catch');
+const {readFileSync} = require('fs');
+const tryToCatch = require('try-to-catch');
+const {runProcessors} = require('@putout/engine-processor');
+
+const parseError = require('../parse-error.js');
+const getOptions = require('../get-options.js');
+const {
+    INVALID_CONFIG,
+    NO_PROCESSORS,
+} = require('../exit-codes.js');
+const ignores = require('../../ignores.js');
+
+const isParser = (rule) => /^parser/.test(rule);
+const isParsingError = ({rule}) => isParser(rule);
+const chooseName = (name, resolvedName) => !isIDE ? name : resolvedName;
+const isIDE = /JetBrains/.test(env.TERMINAL_EMULATOR) || env.TERM_PROGRAM === 'vscode';
+
+const createFormatterProxy = (options) => {
+    return new Proxy(options, {
+        get(target, name) {
+            if (target[name])
+                return target[name];
+            
+            if (name === 'source')
+                return readFileSync(target.name, 'utf8');
+        },
+    });
+};
+
+module.exports = async ({readFile, report, writeFile, exit, raw, write, log, currentFormat, rulesdir, formatterOptions, noConfig, transform, plugins, index, fix, processFile, processorRunners, fileCache, rawPlaces, wasStop, isStop, names, length}) => {
+    wasStop(isStop());
+    
+    const currentIndex = isStop() ? length - 1 : index;
+    const name = names[index];
+    const resolvedName = resolve(name)
+        .replace(/^\./, cwd);
+    
+    const [configError, options] = tryCatch(getOptions, {
+        name: resolvedName,
+        rulesdir,
+        noConfig,
+        transform,
+        plugins,
+    });
+    
+    if (configError)
+        return exit(INVALID_CONFIG, configError);
+    
+    const {dir} = options;
+    
+    if (fileCache.canUseCache(name, options)) {
+        const places = fileCache.getPlaces(name);
+        const formatterProxy = createFormatterProxy({
+            report,
+            formatterOptions,
+            name: chooseName(name, resolvedName),
+            places,
+            index: currentIndex,
+            count: length,
+        });
+        
+        const line = await report(currentFormat, formatterProxy);
+        
+        write(line || '');
+        rawPlaces.push(places);
+        
+        return {
+            success: true,
+        };
+    }
+    
+    let isProcessed = true;
+    let places = [];
+    let rawSource = '';
+    let processedSource = '';
+    
+    if (!ignores(dir, resolvedName, options)) {
+        rawSource = await readFile(resolvedName, 'utf8');
+        
+        const [error, result] = await tryToCatch(runProcessors, {
+            name: resolvedName,
+            fix,
+            processFile,
+            options,
+            rawSource,
+            processorRunners,
+        });
+        
+        if (error) {
+            places = parseError(error);
+            
+            isProcessed = true;
+            processedSource = rawSource;
+            
+            if (raw)
+                log(error);
+        } else {
+            ({
+                isProcessed,
+                places,
+                processedSource,
+            } = result);
+        }
+    }
+    
+    const line = await report(currentFormat, {
+        report,
+        formatterOptions,
+        name: chooseName(name, resolvedName),
+        source: rawSource,
+        places,
+        index: currentIndex,
+        count: length,
+    });
+    
+    write(line || '');
+    
+    if (!isProcessed)
+        return exit(NO_PROCESSORS, Error(`No processors found for ${name}`));
+    
+    if (rawSource !== processedSource) {
+        fileCache.removeEntry(name);
+        await writeFile(name, processedSource);
+    }
+    
+    const fixable = !places.filter(isParsingError).length;
+    
+    if (fixable)
+        fileCache.setInfo(name, places, options);
+    
+    rawPlaces.push(places);
+    
+    return {
+        rawPlaces,
+        success: true,
+    };
+};
+

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "putout",
-  "version": "24.2.0",
+  "version": "24.4.0",
   "type": "commonjs",
   "author": "coderaiser <mnemonic.enemy@gmail.com> (https://github.com/coderaiser)",
   "description": "🐊 Pluggable and configurable code transformer with built-in eslint and babel plugins support of js, jsx typescript, flow files, markdown, yaml and json",
@@ -16,6 +16,7 @@
     "./process-file": "./lib/cli/process-file.js",
     "./exit-codes": "./lib/cli/exit-codes.mjs",
     "./cli": "./lib/cli/index.js",
+    "./cli/run": "./lib/cli/runner/work",
     "./loader": "./lib/loader.mjs",
     "./eslint": "./lib/cli/eslint/index.js",
     "./package.json": "./package.json"
@@ -66,7 +67,7 @@
     "@putout/formatter-progress-bar": "^2.0.0",
     "@putout/formatter-stream": "^3.0.0",
     "@putout/operate": "^7.0.0",
-    "@putout/operator-add-args": "^1.0.0",
+    "@putout/operator-add-args": "^2.0.0",
     "@putout/operator-declare": "^3.0.0",
     "@putout/operator-regexp": "^1.0.0",
     "@putout/plugin-apply-await-import": "^1.0.0",
@@ -86,10 +87,10 @@
     "@putout/plugin-convert-comparison-to-boolean": "^2.0.0",
     "@putout/plugin-convert-concat-to-flat": "^1.0.0",
     "@putout/plugin-convert-equal-to-strict-equal": "^1.0.0",
-    "@putout/plugin-convert-esm-to-commonjs": "^3.0.0",
+    "@putout/plugin-convert-esm-to-commonjs": "^4.0.0",
     "@putout/plugin-convert-for-each-to-for-of": "^7.0.0",
     "@putout/plugin-convert-for-in-to-for-of": "^2.0.0",
-    "@putout/plugin-convert-for-to-for-of": "^2.0.0",
+    "@putout/plugin-convert-for-to-for-of": "^3.0.0",
     "@putout/plugin-convert-index-of-to-includes": "^1.0.0",
     "@putout/plugin-convert-map-to-for-of": "^1.0.0",
     "@putout/plugin-convert-math-pow": "^4.0.0",
@@ -130,7 +131,7 @@
     "@putout/plugin-remove-unreachable-code": "^1.0.0",
     "@putout/plugin-remove-unreferenced-variables": "^1.0.0",
     "@putout/plugin-remove-unused-expressions": "^4.0.0",
-    "@putout/plugin-remove-unused-for-of-variables": "^1.0.0",
+    "@putout/plugin-remove-unused-for-of-variables": "^2.0.0",
     "@putout/plugin-remove-unused-private-fields": "^1.0.0",
     "@putout/plugin-remove-unused-variables": "*",
     "@putout/plugin-remove-useless-arguments": "^5.0.0",
@@ -151,7 +152,7 @@
     "@putout/plugin-remove-useless-template-expressions": "^1.0.0",
     "@putout/plugin-remove-useless-type-conversion": "^1.0.0",
     "@putout/plugin-remove-useless-typeof": "^1.0.0",
-    "@putout/plugin-remove-useless-variables": "^5.0.0",
+    "@putout/plugin-remove-useless-variables": "^6.0.0",
     "@putout/plugin-reuse-duplicate-init": "^3.0.0",
     "@putout/plugin-simplify-assignment": "^1.0.0",
     "@putout/plugin-simplify-logical-expressions": "^1.0.0",
@@ -159,7 +160,7 @@
     "@putout/plugin-split-nested-destructuring": "^1.0.0",
     "@putout/plugin-split-variable-declarations": "^2.0.0",
     "@putout/plugin-strict-mode": "^2.0.0",
-    "@putout/plugin-tape": "^7.0.0",
+    "@putout/plugin-tape": "^8.0.0",
     "@putout/plugin-typescript": "^1.0.0",
     "@putout/plugin-webpack": "^1.0.0",
     "@putout/processor-css": "^3.0.0",
@@ -184,6 +185,7 @@
     "nano-memoize": "^1.1.11",
     "once": "^1.4.0",
     "picomatch": "^2.2.2",
+    "thread-it": "^1.1.3",
     "try-catch": "^3.0.0",
     "try-to-catch": "^3.0.0",
     "wraptile": "^3.0.0",