@putout/engine-runner 11.4.0 → 12.0.0

package/README.md

@@ -1,11 +1,9 @@
-# @putout/engine-runner [![NPM version][NPMIMGURL]][NPMURL] [![Dependency Status][DependencyStatusIMGURL]][DependencyStatusURL]
+# @putout/engine-runner [![NPM version][NPMIMGURL]][NPMURL]
 
 [NPMIMGURL]: https://img.shields.io/npm/v/@putout/engine-runner.svg?style=flat&longCache=true
 [NPMURL]: https://npmjs.org/package/@putout/engine-runner"npm"
-[DependencyStatusURL]: https://david-dm.org/coderaiser/putout?path=packages/engine-runner
-[DependencyStatusIMGURL]: https://david-dm.org/coderaiser/putout.svg?path=packages/engine-runner
 
-Run putout plugins.
+Run 🐊[`Putout`](https://github.com/coderaiser/putout) plugins.
 
 ## Install
 
@@ -161,7 +159,7 @@ Where `__` can be any node. All this possible with help of [@putout/compare](htt
 ```js
 module.exports.report = () => 'debugger statement should not be used';
 
-module.exports.fix = (path) => {
+module.exports.fix = (path, {options}) => {
     path.remove();
 };
 
@@ -195,7 +193,7 @@ module.exports.traverse = ({push, listStore}) => ({
 });
 ```
 
-`store` is preferred way of keeping array elements, because of caching of `putout`, `traverse` init function called only once, and any other way
+`store` is preferred way of keeping data, because of caching 🐊`putout`, `traverse` init function called only once, and any other way
 of handling variables will most likely will lead to bugs.
 
 #### Store
@@ -227,6 +225,61 @@ module.exports.traverse = ({push, store}) => ({
 });
 ```
 
+#### Upstore
+
+When you need to update already saved values, use `upstore`
+
+```js
+module.exports.traverse = ({push, store}) => ({
+    TSTypeAliasDeclaration(path) {
+        if (path.parentPath.isExportNamedDeclaration())
+            return;
+        
+        store(path.node.id.name, {
+            path,
+        });
+    },
+    
+    ObjectProperty(path) {
+        const {value} = path.node;
+        const {name} = value;
+        
+        store(name, {
+            used: true,
+        });
+    },
+    
+    Program: {
+        exit() {
+            for (const {path, used} of store()) {
+                if (used)
+                    continue;
+                
+                push(path);
+            }
+        },
+    },
+});
+```
+
+#### ListStore
+
+When you need to track list of elements, use `listStore`:
+
+```
+module.exports.traverse = ({push, listStore}) => ({
+    ImportDeclaration(path) {
+        listStore(path);
+    },
+
+    Program: {
+        exit: () => {
+            processImports(push, listStore());
+        },
+    },
+});
+```
+
 ### Finder
 
 `Find plugins` gives you all the control over traversing, but it's the slowest format.

package/lib/merge-visitors.js

@@ -6,7 +6,11 @@ const {generate} = require('@putout/engine-parser');
 const runFix = require('./run-fix');
 const {getPosition} = require('./get-position');
 const maybeArray = require('./maybe-array');
-const {listStore, mapStore} = require('./store');
+const {
+    listStore,
+    mapStore,
+    upStore,
+} = require('./store');
 
 const shouldSkip = (a) => !a.parent;
 const {merge} = traverse.visitors;
@@ -16,7 +20,7 @@ const parse = (name, plugin, options) => {
     const list = [];
     
     if (plugin[name])
-        list.push(...plugin[name]());
+        list.push(...maybeArray(plugin[name]()));
     
     if (options[name])
         list.push(...maybeArray(options[name]));
@@ -33,12 +37,14 @@ module.exports = (pluginsToMerge, {fix, shebang, template}) => {
             push,
             pull,
             store,
+            upstore,
             listStore,
         } = getStore(plugin, {
             fix,
             rule,
             shebang,
             msg,
+            options,
         });
         
         pushed[rule] = pull;
@@ -47,6 +53,7 @@ module.exports = (pluginsToMerge, {fix, shebang, template}) => {
             push,
             store,
             listStore,
+            upstore,
             generate,
             options,
         });
@@ -78,16 +85,17 @@ module.exports = (pluginsToMerge, {fix, shebang, template}) => {
     };
 };
 
-function getStore(plugin, {fix, rule, shebang, msg}) {
+function getStore(plugin, {fix, rule, shebang, msg, options}) {
     const store = mapStore();
     const list = listStore();
+    const upstore = upStore();
     const placesStore = listStore();
     
     const push = (path) => {
         const position = getPosition(path, shebang);
         const message = msg || plugin.report(path);
         
-        placesStore ({
+        placesStore({
             message,
             position,
         });
@@ -96,12 +104,14 @@ function getStore(plugin, {fix, rule, shebang, msg}) {
             path,
             rule,
             position,
+            options,
         });
     };
     
     const pull = () => {
         store.clear();
         list.clear();
+        upstore.clear();
         return placesStore.clear();
     };
     
@@ -110,6 +120,7 @@ function getStore(plugin, {fix, rule, shebang, msg}) {
         pull,
         store,
         listStore: list,
+        upstore,
     };
 }
 

package/lib/replace/index.js

@@ -26,7 +26,7 @@ const isObj = (a) => typeof a === 'object';
 const validateTemplateValues = (a, b) => {
     for (const key of keys(a)) {
         if (!b[key])
-            throw Error(`☝️ Looks like template values not linked: ${stringify(keys(a))} ${stringify(keys(b))}`);
+            throw Error(`☝️ Looks like template values not linked: ${stringify(keys(b))} -> ${stringify(keys(a))}`);
     }
 };
 

package/lib/run-fix.js

@@ -4,8 +4,8 @@ const tryCatch = require('try-catch');
 const debug = require('debug')('putout:runner:fix');
 const {enabled} = debug;
 
-const tryToFix = (fix, {path, position}) => {
-    const [e] = tryCatch(fix, path);
+const tryToFix = (fix, {path, position, options}) => {
+    const [e] = tryCatch(fix, path, {options});
     
     if (!e)
         return;
@@ -15,7 +15,7 @@ const tryToFix = (fix, {path, position}) => {
     throw e;
 };
 
-module.exports = (is, fix, {path, rule, position}) => {
+module.exports = (is, fix, {path, rule, position, options}) => {
     if (!is)
         return;
     
@@ -24,6 +24,7 @@ module.exports = (is, fix, {path, rule, position}) => {
     tryToFix(fix, {
         path,
         position,
+        options,
     });
 };
 

package/lib/store.js

@@ -5,7 +5,6 @@ const {
     entries,
     assign,
 } = Object;
-const isObject = (a) => typeof a === 'object';
 
 module.exports.listStore = (list = []) => {
     const fn = (...args) => {
@@ -27,33 +26,42 @@ module.exports.listStore = (list = []) => {
     return fn;
 };
 
-module.exports.mapStore = (map = {}) => {
-    const fn = (...args) => {
-        if (!args.length)
-            return values(map);
-        
-        const [name, data] = args;
+module.exports.mapStore = createStore({
+    set(map, name, data) {
+        map[name] = data;
+    },
+});
+
+module.exports.upStore = createStore({
+    set(map, name, data) {
+        map[name] = map[name] || {};
+        assign(map[name], data);
+    },
+});
+
+function createStore({set}) {
+    return (map = {}) => {
+        const fn = (...args) => {
+            if (!args.length)
+                return values(map);
+            
+            const [name, data] = args;
+            
+            if (args.length === 1)
+                return map[name];
+            
+            set(map, name, data);
+        };
         
-        if (args.length === 1)
-            return map[name];
+        fn.clear = () => {
+            map = {};
+        };
         
-        if (isObject(data)) {
-            map[name] = map[name] || {};
-            assign(map[name], data);
-            return;
-        }
+        fn.entries = () => {
+            return entries(map);
+        };
         
-        map[name] = data;
+        return fn;
     };
-    
-    fn.clear = () => {
-        map = {};
-    };
-    
-    fn.entries = () => {
-        return entries(map);
-    };
-    
-    return fn;
-};
+}
 

package/lib/template/index.js

@@ -24,7 +24,7 @@ const {stringify} = JSON;
 module.exports._log = log;
 
 const exclude = ({rule, tmpl, fn, nodesExclude}) => {
-    if (!nodesExclude.length)
+    if (!isFn(fn) || !nodesExclude.length)
         return {
             [tmpl]: fn,
         };

package/package.json

@@ -1,9 +1,9 @@
 {
   "name": "@putout/engine-runner",
-  "version": "11.4.0",
+  "version": "12.0.0",
   "author": "coderaiser <mnemonic.enemy@gmail.com> (https://github.com/coderaiser)",
   "description": "run putout plugins",
-  "homepage": "http://github.com/coderaiser/putout",
+  "homepage": "https://github.com/coderaiser/putout/tree/master/packages/engine-runner#readme",
   "main": "lib/index.js",
   "release": false,
   "tag": false,
@@ -25,7 +25,7 @@
   "dependencies": {
     "@babel/traverse": "^7.12.7",
     "@babel/types": "^7.12.7",
-    "@putout/compare": "^7.6.0",
+    "@putout/compare": "^8.0.0",
     "@putout/engine-parser": "^4.0.1",
     "@putout/operate": "^6.0.0",
     "debug": "^4.1.1",
@@ -43,12 +43,13 @@
     "@babel/plugin-codemod-optional-catch-binding": "^7.7.4",
     "@cloudcmd/stub": "^3.0.0",
     "c8": "^7.5.0",
-    "eslint": "^8.0.0-beta.0",
+    "eslint": "^8.0.1",
     "eslint-plugin-node": "^11.0.0",
-    "eslint-plugin-putout": "^10.0.0",
+    "eslint-plugin-putout": "^12.0.0",
     "just-camel-case": "^4.0.2",
     "lerna": "^4.0.0",
     "madrun": "^8.0.1",
+    "mock-require": "^3.0.3",
     "montag": "^1.0.0",
     "nodemon": "^2.0.1",
     "putout": "*",