Initial load JS much too large about dao-dao-ui HOT 7 CLOSED

For the libsodium.js dependency, progress is here: cosmos/cosmjs#904

For the bip39 dependency, I think we can handle that on our side with a bit better optimization. A good static analyzer should be capable of removing it. It's just a regular (i.e., js, non-compiled) dependency of cosmjs/crypto: https://github.com/cosmos/cosmjs/blob/main/packages/crypto/package.json#L47

from dao-dao-ui.

I wonder why libsodium.js isn't optimized out entirely. In thise codebase you use none of its functionality. However, I'm not aware of state of the art in JS optimization.

from dao-dao-ui.

I have the same question @webmaster128. I think whatever the next build optimizer does is just not up to snuff---reasonable tree-shaking should get rid of the dependency. (libsodium.js size is still an issue, but shouldn't be our issue).

from dao-dao-ui.

Jupp. I found a pretty good explanation in gcanti/fp-ts#813 and gcanti/fp-ts#814 for what needs to be done in CosmJS to make this work.

from dao-dao-ui.

I think this is a "Won't Fix" because it's upstream. Closing for now.

from dao-dao-ui.

Re-opening this issue, as there is some hope of trimming the bundle sizes by removing dependencies on libsodium and bip39 using the tricks described here: cosmos/cosmjs#904 (comment)

from dao-dao-ui.

fixed in cosmjs v0.28 - just need to upgrade the package

from dao-dao-ui.