Error behaviour difference between esprima.parse() and detective.find() about detective HOT 3 CLOSED

After a bit more poking around, I've tracked down the source of the inconsistency to https://github.com/substack/node-detective/blob/master/index.js#L49:

if (src.indexOf(word) == -1) return modules;

This short circuit (presumably for performance, to prevent an unnecessary esprima parse) means that sources with the string 'require' anywhere (including in a comment or within a string) will be parsed, but those that don't won't.

Calling detective on this source throws an error:

// I dont require any modules
return

This doesn't:

return

from detective.

Fixed in 2.1.1.

from detective.

Thanks!

from detective.