@putout/engine-runner 12.0.0 → 12.3.0

package/README.md

@@ -15,7 +15,8 @@ npm i @putout/engine-runner
 
 ### Replacer
 
-`Replacer` converts code in declarative way. Simplest possible form, no need to use `fix`. Just `from` and `to` parts.
+`Replacer` converts code in declarative way. Simplest possible form, no need to use `fix`. Just `from` and `to` parts
+according to [`template variables syntax`](https://github.com/coderaiser/putout/tree/master/packages/compare#supported-template-variables).
 
 Simplest replace example:
 
@@ -170,34 +171,40 @@ module.exports.traverse = ({push}) => ({
 });
 ```
 
-#### listStore
+#### Storages
 
-To keep things during traverse in a safe way `listStore` can be used.
+Storages is preferred way of keeping data 🐊`Putout`, `traverse` init function called only once, and any other way
+of handling variables will most likely will lead to bugs.
+
+##### listStore
+
+To keep things and save them as a list during traverse in a safe way `listStore` can be used for code:
+
+```js
+debugger;
+const hello = '';
+debugger;
+const world = '';
+```
 
 ```js
-module.exports.traverse = ({push, listStore}) => ({
+module.exports.traverse = ({listStore}) => ({
     'debugger'(path) {
         listStore('x');
-        push(path);
     },
     
     Program: {
         exit() {
             console.log(listStore());
             // returns
-            ['x', 'x', 'x'];
+            ['x', 'x'];
             // for code
-            'debugger; debugger; debugger';
+            'debugger; debugger';
         },
     },
 });
 ```
 
-`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
-
 When you need `key-value` storage `store` can be used.
 
 ```js
@@ -262,24 +269,6 @@ module.exports.traverse = ({push, store}) => ({
 });
 ```
 
-#### 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/index.js

@@ -41,6 +41,8 @@ module.exports.runPlugins = ({ast, shebang, fix, fixCount, plugins, template = r
         
         if (!fix || !places.length)
             return places;
+        
+        replace.clearWatermark(ast);
     }
     
     return places;
@@ -120,6 +122,7 @@ function runWithoutMerge({ast, fix, shebang, template, pluginsFind}) {
                 path: item,
                 rule,
                 position,
+                options,
             });
         }
     }

package/lib/replace/index.js

@@ -61,6 +61,10 @@ module.exports = ({rule, plugin, msg, options}) => {
     };
 };
 
+module.exports.clearWatermark = (ast) => {
+    delete ast.program[watermark.REPLACE_WATERMARK];
+};
+
 const isFn = (a) => typeof a === 'function';
 
 const fix = (from, to, path) => {

package/lib/replace/watermark.js

@@ -30,6 +30,8 @@ module.exports = (from, to, path) => {
     };
 };
 
+module.exports.REPLACE_WATERMARK = name;
+
 module.exports.create = create;
 function create(from, to, path) {
     const watermark = `${from} -> ${to}`;
@@ -52,6 +54,7 @@ function init({path, program}) {
 module.exports.add = add;
 function add({path, program, watermark, highWatermark}) {
     init({path, program});
+    
     path.node[name].add(watermark);
     program.node[name].add(highWatermark);
 }

package/lib/run-fix.js

@@ -6,9 +6,14 @@ const {enabled} = debug;
 
 const tryToFix = (fix, {path, position, options}) => {
     const [e] = tryCatch(fix, path, {options});
+    const {scope} = path.path || path;
     
-    if (!e)
+    if (!e && scope)
+        scope.getProgramParent().crawl();
+    
+    if (!e) {
         return;
+    }
     
     e.loc = e.loc || position;
     

package/lib/store.js

@@ -57,9 +57,7 @@ function createStore({set}) {
             map = {};
         };
         
-        fn.entries = () => {
-            return entries(map);
-        };
+        fn.entries = () => entries(map);
         
         return fn;
     };

package/package.json

@@ -1,6 +1,7 @@
 {
   "name": "@putout/engine-runner",
-  "version": "12.0.0",
+  "version": "12.3.0",
+  "type": "commonjs",
   "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#readme",
@@ -27,7 +28,7 @@
     "@babel/types": "^7.12.7",
     "@putout/compare": "^8.0.0",
     "@putout/engine-parser": "^4.0.1",
-    "@putout/operate": "^6.0.0",
+    "@putout/operate": "^7.0.0",
     "debug": "^4.1.1",
     "jessy": "^3.0.0",
     "nessy": "^4.0.0",
@@ -45,7 +46,7 @@
     "c8": "^7.5.0",
     "eslint": "^8.0.1",
     "eslint-plugin-node": "^11.0.0",
-    "eslint-plugin-putout": "^12.0.0",
+    "eslint-plugin-putout": "^13.0.0",
     "just-camel-case": "^4.0.2",
     "lerna": "^4.0.0",
     "madrun": "^8.0.1",