Continue on parse error, simply print details instead of exiting

danielstjules · Apr 4, 2017 · 8b11f70 · 8b11f70
1 parent 64115bf
commit 8b11f70
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 6 deletions.
diff --git a/lib/inspector.js b/lib/inspector.js
@@ -55,7 +55,11 @@ class Inspector extends EventEmitter {
     this._filePaths.forEach((filePath) => {
       var src = fs.readFileSync(filePath, {encoding: 'utf8'});
       this._fileContents[filePath] = src.split('\n');
-      var syntaxTree = parse(src, filePath);
+      try {
+        var syntaxTree = parse(src, filePath);
+      } catch (err) {
+        return console.error(err.message);
+      }
       this._traversals[filePath] = NodeUtils.getDFSTraversal(syntaxTree);
       this._walk(syntaxTree, (nodes) => this._insert(nodes));
     });

diff --git a/lib/parser.js b/lib/parser.js
@@ -12,16 +12,25 @@ var debug   = require('./debug');
 exports.parse = function(src, filePath) {
   debug(`parsing ${filePath}`);
   try {
-    return _parse(src, filePath, 'script');
+    return attempt(
+      () => _parse(src, filePath, 'script'),
+      () => _parse(src, filePath, 'module')
+    );
   } catch (err) {
+    let ctx = getErrorContext(err, src);
+    throw new Error(`Couldn't parse ${filePath}: ${err.message}${ctx}`);
+  }
+};
+
+function attempt(...fns) {
+  for (let i = 0; i < fns.length; i++) {
     try {
-      return _parse(src, filePath, 'module');
+      return fns[i]();
     } catch (err) {
-      let ctx = getErrorContext(err, src);
-      throw new Error(`Couldn't parse ${filePath}: ${err.message}${ctx}`);
+      if (i === fns.length - 1) throw err;
     }
   }
-};
+}
 
 function _parse(src, filePath, sourceType) {
   return babylon.parse(src, {