eslint-plugin-putout 14.6.0 → 14.8.0

package/README.md

@@ -154,3 +154,7 @@ Disabled 🐊**Putout** rules:
 ### safe+align
 
 When you want to enable ability to align spaces on empty lines, while have all benefits of `safe` preset: use `safe+align`.
+
+## License
+
+MIT

package/lib/markdown.js

@@ -1,6 +1,7 @@
 'use strict';
 
 const parserOpts = require('@putout/engine-parser/babel/options');
+const parserPlugins = require('@putout/engine-parser/babel/plugins');
 const [ts] = require('./ts');
 
 const commonRules = {
@@ -28,7 +29,10 @@ module.exports = [{
         requireConfigFile: false,
         babelOptions: {
             sourceType: 'module',
-            parserOpts,
+            parserOpts: {
+                ...parserOpts,
+                plugins: parserPlugins,
+            },
             plugins: [
                 '@babel/plugin-syntax-class-properties',
             ],

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,29 @@ 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);
+        if (errorParser) {
+            context.report({
+                message: `${parseError(errorParser)} (putout)`,
+                node,
+            });
+            
+            return EMPTY_VISITORS;
+        }
         
         const [error, places = []] = tryCatch(findPlaces, ast, text, resultOptions);
         
-        if (error)
+        if (error) {
             context.report({
                 message: `${parseError(error)} (putout)`,
                 node,
             });
+            
+            return EMPTY_VISITORS;
+        }
         
         for (const {rule, message, position} of places) {
             context.report({
@@ -105,34 +107,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.8.0",
   "type": "commonjs",
   "description": "eslint plugin for putout",
   "release": false,