detective

detective copied to clipboard

ignore require() calls inside already browserified bundles

5

If a function expression declares a `require` parameter, its body is not analyzed for `require()` calls. This is mostly helpful for bundles that were already browserified, `detective` will ignore the...

goto-bus-stop

fix(package): [email protected]

6

Closes https://github.com/browserify/detective/issues/88

aaditkamat

After encountering dependency-check-team/dependency-check#112 I tried looking into how that project uses `detective` and the various "extensions", such as [detective-typescript](https://github.com/pahen/detective-typescript) that enabled foreign syntax. It was not immediately obvious how this...

fatso83

Skip commented-out requires

12

Currently detective parses these as valid requires even though they are not. ``` javascript require('b') // require('c') /* require('d') */ ``` maybe with https://github.com/jonschlinkert/strip-comments

paulmillr

speed up by using acorn tokenizer when possible

3

Add a `findFast` function that's exactly like `detective.find`, but doesn't parse the source. detective.find automatically uses findFast if possible. we cannot do this if a custom `isRequire` function is provided,...

goto-bus-stop

Replace `acorn` parser for `babylon`?

5

Hi, I came across a problem when `node-detective` cannot operate on ES2015+ source code. This is not a problem solely related to this package. The same problem applies to `node-syntax-error`...

zdychacek

if require is aliased, detective incorrectly returns uses of it as dependencies

3

Eg: ``` var require = function() { console.log('test') } require("this isn't a dependency") ``` It should be pretty simple to detect scopes in which require is redefined and exclude requires...

fresheneesz

Support ES6 imports

3

Tried this: ``` javascript import detective from 'detective'; import fs from 'fs'; let src = fs.readFileSync(__filename); let requires = detective(src, { parse: { ecmaVersion: 6, sourceType: 'module' } }); console.log(requires);...

lukehorvat

Revert 'require' word optimization

1

@zertosh What do you think about reverting the regex optimization so that syntax-error can be removed upstream? Giant libraries with no require calls should be noParse'd anyway, and the performance...

ashaffer

It would be nice to know that there was a require that could not be parsed rather than it being ignored. ``` var reqStr = './some-thing' ; require(reqStr); ```

coolaj86