@putout/engine-runner 11.6.0 → 12.0.0

package/README.md

@@ -193,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
@@ -225,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,6 +37,7 @@ module.exports = (pluginsToMerge, {fix, shebang, template}) => {
             push,
             pull,
             store,
+            upstore,
             listStore,
         } = getStore(plugin, {
             fix,
@@ -48,6 +53,7 @@ module.exports = (pluginsToMerge, {fix, shebang, template}) => {
             push,
             store,
             listStore,
+            upstore,
             generate,
             options,
         });
@@ -82,6 +88,7 @@ module.exports = (pluginsToMerge, {fix, shebang, template}) => {
 function getStore(plugin, {fix, rule, shebang, msg, options}) {
     const store = mapStore();
     const list = listStore();
+    const upstore = upStore();
     const placesStore = listStore();
     
     const push = (path) => {
@@ -104,6 +111,7 @@ function getStore(plugin, {fix, rule, shebang, msg, options}) {
     const pull = () => {
         store.clear();
         list.clear();
+        upstore.clear();
         return placesStore.clear();
     };
     
@@ -112,6 +120,7 @@ function getStore(plugin, {fix, rule, shebang, msg, options}) {
         pull,
         store,
         listStore: list,
+        upstore,
     };
 }
 

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.6.0",
+  "version": "12.0.0",
   "author": "coderaiser <mnemonic.enemy@gmail.com> (https://github.com/coderaiser)",
   "description": "run putout plugins",
-  "homepage": "https://github.com/coderaiser/putout/tree/master/packages/engine-runner",
+  "homepage": "https://github.com/coderaiser/putout/tree/master/packages/engine-runner#readme",
   "main": "lib/index.js",
   "release": false,
   "tag": false,
@@ -49,6 +49,7 @@
     "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": "*",