eslint-plugin-putout 14.6.0 → 14.7.0

package/lib/putout/index.js

@@ -10,10 +10,6 @@ const {
     parse,
 } = require('putout');
 
-const traverse = require('@babel/traverse').default;
-
-const v8 = require('v8');
-
 const parseOptions = require('putout/parse-options');
 const {parseError} = require('./parse-error');
 
@@ -23,9 +19,6 @@ const getContextOptions = ({options}) => {
     return allContextOptions;
 };
 
-const copyAST = (a) => v8.deserialize(v8.serialize(a));
-const returns = (a) => () => a;
-
 const EMPTY_VISITORS = {};
 
 module.exports = {
@@ -54,20 +47,20 @@ module.exports = {
         const {text} = source;
         const node = source.ast;
         
-        const ast = parse(text, {
-            parser: createParser(node),
+        const [errorParser, ast] = tryCatch(parse, text, {
+            isTS: true,
         });
         
-        // remove parent nodes that was ignored before converting to babel
-        removeParent(ast);
-        
-        const [error, places = []] = tryCatch(findPlaces, ast, text, resultOptions);
-        
-        if (error)
+        if (errorParser) {
             context.report({
-                message: `${parseError(error)} (putout)`,
+                message: `${parseError(errorParser)} (putout)`,
                 node,
             });
+            
+            return EMPTY_VISITORS;
+        }
+        
+        const places = findPlaces(ast, text, resultOptions);
         
         for (const {rule, message, position} of places) {
             context.report({
@@ -105,34 +98,3 @@ const fix = ({ast, text, node, source, resultOptions}) => (fixer) => {
     return fixer.replaceTextRange([0, last], code);
 };
 
-// 1. We cannot modify ESLint AST
-// 2. Parent nodes makes Recast go crazy, so they should be removed
-// 3. Recast creates original nodes with copies of each nodes
-// 4. Parser does nothing but returns original AST before estree to babel
-// 5. All this stuff made to gain performance benefit of avoiding a duble parsing: ESLint, and then Babel
-// 6. Always can be removed and switched to direct parsing by Putout, when benefits outweight supporting all of this magic
-const createParser = (node) => {
-    const ast = copyAST(node);
-    removeParent(ast);
-    
-    const parser = {
-        parse: returns(ast),
-    };
-    
-    return parser;
-};
-
-// ESLint adds parent to each node
-// it makes recase go crazy
-// so we better drop them out
-//
-// https://github.com/eslint/eslint/blob/v8.4.0/lib/linter/linter.js#L964
-function removeParent(ast) {
-    traverse(ast, {
-        noScope: true,
-        enter(path) {
-            delete path.node.parent;
-        },
-    });
-}
-

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-putout",
-  "version": "14.6.0",
+  "version": "14.7.0",
   "type": "commonjs",
   "description": "eslint plugin for putout",
   "release": false,