Explore rollup-plugin-visualizer

[jaredpalmer]

Seems useful. Perhaps we add an --inspect flag?

[arvinsim]

Were you guys able to make rollup-plugin-visualizer work without resorting to the objectHashIgnoreUnknownHack hack to circumvent the asyncfunction issue?

[agilgur5]

@arvinsim I'm fairly certain your hunch is correct that this is blocked on puleos/object-hash#90 .
In theory disabling rpts2's cache only during --inspect isn't that bad, but I'm not a big fan of adding a hack on top of a hack to the codebase as that can lead to unexpected issues, esp. when they involve caching.

[agilgur5]

Ok so puleos/object-hash#90 was just merged and released so now we're blocked on ezolenko/rollup-plugin-typescript2#203 (PR from me).

Notably, rollup-plugin-visualizer was mentioned in #200 as an explicit example of one of the things that wasn't possible due to rpts2's lack of async/await support.

I mentioned in puleos/object-hash#90 that it'd be difficult to fork/patch a transitive dependency, but this actually goes a step further as rpts2 doesn't have externals for its dependencies and actually bundles them in its own release (ezolenko/rollup-plugin-typescript2#80, ezolenko/rollup-plugin-typescript2#70), so TSDX can't override the version of object-hash that rpts2 uses (on top of having duplicated dependencies included in it...) 😕 😕