Renée

It doesn't detect that `require`: ```js detective('// require("xyz")') // → [] detective('require("xyz")') // → ["xyz"] ``` The `require` string occuring inside a comment just means detective will parse the file...

Not without a code sample that reproduces the problem :sweat_smile:

How does `src/main.js` look like? Sometimes modules don't have side effects. Like if they're just doing: ```js function initApp () { // stuff here } module.exports = initApp ``` tinyify...

If you do `window.MyApp = initApp` in your main file it should also work.

Ack. you can use `--no-flat` in the CLI or `flat: false` in the API to disable browser-pack-flat, does it still crash then? bundle-collapser is significanly less complex than browser-pack-flat.

ow, it's possible that bundle-collapser doesn't support some new syntax. in case you're using async functions or something. it should be updated.

There are some syntax errors in that snippet so I'm not sure if I'm reproducing it correctly. Name changes are done by terser, so you could try putting the actual...

tinyify was depending on an older version of terser. I just published [email protected], could you try again with that?

Hmm it might be a bit more complicated then :disappointed: I tried just bundling this with browserify + tinyify and it seemed OK: ```js window.x={ addOnclick: function(){ element.onclick = ()=>{this.foo("some...

Awesome, thanks. It looks like common-shakeify does not count the `this.bar()` call as a use of the exported `bar` function, and decides to remove it because it appears unused. You...