feature: @putout/engine-runner: add error reason when throw during traverse

Author: coderaiser

packages/engine-runner/lib/index.js

@@ -13,6 +13,7 @@ const declare = require('./declare');
 const scanner = require('./scanner');
 const template = require('./template');
 const {createProgress} = require('./progress');
+const {tryThrowWithReason} = require('./try-throw-with-reason');
 
 const {getPath, getPosition} = require('./get-position');
 
@@ -80,7 +81,7 @@ function runWithMerge({ast, fix, shebang, template, pluginsTraverse, merge, trav
         template,
     });
 
-    traverse(ast, visitor);
+    tryThrowWithReason(traverse, ast, visitor);
 
     const places = [];
 
@@ -107,7 +108,7 @@ function runWithoutMerge({ast, fix, shebang, template, pluginsFind}) {
 
         const {report, find} = plugin;
 
-        const items = superFind({
+        const items = tryThrowWithReason(superFind, {
             rule,
             find,
             ast,

packages/engine-runner/lib/try-throw-with-reason.js

@@ -0,0 +1,14 @@
+'use strict';
+
+const tryCatch = require('try-catch');
+
+module.exports.tryThrowWithReason = (fn, ...args) => {
+    const [error, result] = tryCatch(fn, ...args);
+    
+    if (error) {
+        error.reason = 'traverse';
+        throw error;
+    }
+    
+    return result;
+};

packages/engine-runner/test/runner.js

@@ -10,7 +10,7 @@ const {loadPlugins} = require('@putout/engine-loader');
 
 const {readFixtures} = require('./fixture');
 const {runPlugins} = require('..');
-
+const noop = () => {};
 const {reRequire, stopAll} = mockRequire;
 const {parse} = putout;
 
@@ -1036,3 +1036,64 @@ test('putout: runner: runPlugins: override traverse', (t) => {
     t.calledWith(traverse, expected);
     t.end();
 });
+
+test('putout: runner: runPlugins: find: reason', (t) => {
+    const duplicator = {
+        report: () => '',
+        find: (ast, {traverse}) => {
+            traverse(ast, {
+                DebuggerStatement: () => {
+                    throw Error('x');
+                },
+            });
+        },
+    };
+    
+    const code = 'debugger';
+    const ast = parse(code);
+    
+    const plugins = loadPlugins({
+        pluginNames: [
+            ['duplicator', duplicator],
+        ],
+    });
+    
+    const [error] = tryCatch(runPlugins, {
+        ast,
+        fix: false,
+        plugins,
+    });
+    
+    t.equal(error.reason, 'traverse');
+    t.end();
+});
+
+test('putout: runner: runPlugins: traverse: reason', (t) => {
+    const duplicator = {
+        report: () => '',
+        fix: noop,
+        traverse: () => ({
+            DebuggerStatement: () => {
+                throw Error('x');
+            },
+        }),
+    };
+    
+    const code = 'debugger';
+    const ast = parse(code);
+    
+    const plugins = loadPlugins({
+        pluginNames: [
+            ['duplicator', duplicator],
+        ],
+    });
+    
+    const [error] = tryCatch(runPlugins, {
+        ast,
+        fix: false,
+        plugins,
+    });
+    
+    t.equal(error.reason, 'traverse');
+    t.end();
+});