fix: replace deprecated cherow with @babel/parser (in estree mode)

Author: 3cp

lib/ensure-parser-set.js

@@ -1,11 +1,39 @@
-const {parse} = require('cherow');
+const {parse} = require('@babel/parser');
 const astMatcher = require('ast-matcher');
 
 function _parser (contents) {
   // Turn on range to get position info in scope analysis.
   // Do not need to turn on JSX, dumber is designed to be
   // small, supposes to be used after transpiling.
-  return parse(contents, {ranges: true, loc: true, module: true, next: true});
+  const file = parse(contents, {sourceType: 'module', plugins: [
+    // 'jsx',
+    // 'typescript',
+    'asyncGenerators',
+    'bigInt',
+    'classProperties',
+    'classPrivateProperties',
+    'classPrivateMethods',
+    'decorators-legacy',
+    // ['decorators', {'decoratorsBeforeExport': true}],
+    'doExpressions',
+    'dynamicImport',
+    'exportDefaultFrom',
+    'exportNamespaceFrom',
+    'functionBind',
+    'functionSent',
+    'importMeta',
+    'logicalAssignment',
+    'nullishCoalescingOperator',
+    'numericSeparator',
+    'objectRestSpread',
+    'optionalCatchBinding',
+    'optionalChaining',
+    'partialApplication',
+    // ['pipelineOperator', {proposal: 'minimal'}],
+    'throwExpressions',
+    'estree' // VERY IMPORTANT to be compatible with ast-matcher
+  ]});
+  return file && file.program;
 }
 
 module.exports = function() {

lib/transformers/name-amd-define.js

@@ -98,7 +98,7 @@ function ensureNamedDefine(unit) {
 
   let namedDefine;
   let needsScanDeps = true;
-  let firstArgLoc = args[0].start; // .start for cherow, .range[0] for esprima
+  let firstArgLoc = args[0].start;
 
   if (args[0].type === 'Literal') {
     // already has named define
@@ -169,7 +169,7 @@ function ensureNamedDefine(unit) {
     }
 
     if (cjsDeps.length) {
-      m.insert(factory.start, depsString(cjsDeps) + ','); // .start for cherow, .range[0], for esprima
+      m.insert(factory.start, depsString(cjsDeps) + ',');
     }
   }
 

lib/transformers/replace.js

@@ -53,8 +53,8 @@ module.exports = function(unit) {
 
     if (node.value !== dep) {
       m.replace(
-        node.start, // .start for cherow, .range[0] for esprima
-        node.end, // .end cherow, .range[1] for esprima
+        node.start,
+        node.end,
         `'${dep}'`
       );
     }

package.json

@@ -48,27 +48,27 @@
     "@babel/plugin-transform-modules-commonjs": "^7.5.0",
     "ast-matcher": "^1.0.4",
     "base64-arraybuffer": "^0.2.0",
-    "cherow": "^1.6.9",
     "convert-source-map": "^1.6.0",
-    "del": "^5.0.0",
+    "del": "^5.1.0",
     "dumber-module-loader": "^0.18.0",
     "escope": "^3.6.0",
     "mkdirp": "^0.5.1",
     "modify-code": "^0.5.3",
     "node-fetch": "^2.6.0",
     "node-libs-browser": "^2.2.1",
     "readable-stream": "^2.3.6",
-    "resolve": "^1.11.1",
+    "resolve": "^1.12.0",
     "source-map": "^0.7.3"
   },
   "devDependencies": {
-    "browserify": "^16.3.0",
-    "eslint": "^6.1.0",
+    "@babel/parser": "^7.5.5",
+    "browser-do": "^0.3.2",
+    "browserify": "^16.5.0",
+    "eslint": "^6.2.2",
     "nyc": "^14.1.1",
     "sourcemap-codec": "^1.4.6",
     "standard-changelog": "^2.0.13",
     "tap-dot": "^2.0.0",
-    "tape": "^4.11.0",
-    "browser-do": "^0.3.2"
+    "tape": "^4.11.0"
   }
 }