ChainSafe / filsnap

MetaMask snap for interacting with Filecoin dapps.
https://filsnap.chainsafe.io
Apache License 2.0
107 stars 41 forks source link

Building snap locally takes over 3 minutes #251

Open meandavejustice opened 1 year ago

meandavejustice commented 1 year ago

It appears that we don't have a watch command setup for the snap when running yarn demo, we should speed up the snap build command before we add watch so that it can be useful for local dev.

nard@wintermute:/mnt/c/Users/davej/Code/filsnap/packages/snap$ npm run build

> @chainsafe/filsnap@2.3.12 prebuild
> rm -rf build && rm -rf dist

> @chainsafe/filsnap@2.3.12 build
> yarn run build:snap
Build success: 'src/index.ts' bundled as 'dist/bundle.js'!
Manifest Success: Validated snap.manifest.json!
WARNING: Missing strong random number source
Eval Success: evaluated 'dist/bundle.js' in SES!

real    3m1.599s
user    0m20.466s
sys     1m4.116s
mpetrunic commented 1 year ago

Pretty sure ses eval takes the longest, it could be disabled but there is a risk, Metamask won't install snap.