The parser is blowing up on a shebang

[dijs]

Can this "#!/usr/bin/env node" not throw a parse error?

[michaelficarra]

No, that is not valid JavaScript. You can strip shebangs yourself before giving input to esprima.

[dijs]

Sounds good.

[ariya]

For an illustrative example on how to do this (taken from jsparser):

function trimHashbang(code) {
    if (code.substring(0, 2) === '#!') {
        var end = code.indexOf('\n');
        var filler = '';
        for (var i = 0; i < end; ++i) {
           filler += ' ';
        }
        code = filler + code.substring(end, code.length);
    }
    return code;
}

Of course, you also need to account for a possible return not inside a function. Check the above referred jsparser for details.

[michaelficarra]

eslint uses essentially

function trimShebang(text) {
  return text.replace(/^#!([^\r\n]+)/, function(match, captured) { return "//" + captured; });
}

[ariya]

"I know, I'll use regular expressions." 😉

[dmethvin]

problems++