Wrangler: Installing on every change wasm-pack / wranglerjs

Created on 7 Oct 2019 · 5Comments · Source: cloudflare/wrangler

🐛 Bug Report

When i run wrangler preview --watch it's installing on every change those packages. Is that normal? It takes lot of time on every change.

Environment

operating system: NixOs
output of rustc -V: Is not installed
output of node -v: v12.5.0
output of wrangler -V: wrangler 1.4.0

Steps to reproduce

Empty new Project and then preview it with watch

What did you expect to see?

Faster loading without installing on every change that package

What did you see instead?

Taking time on every change to install that package

bug status - PR attached user report

Source

shyim

👍6

Most helpful comment

currently we check to see if the latest version of the binary we need is installed on the system. We need to instead check if the latest version is installed in the install directory. this is difficult because our install directory is full of things like wranglerjs-789087 that don't have the version names in the directories.

1) When we download, we should put the version number in the directory number
2) When we check if we need an install, we should see if the version number already exists in the download folder (optionally also check the system version)
3) If there is an updated version we need to install, we should install that and clean up the old one

EverlastingBugstopper on 17 Dec 2019

👍3

All 5 comments

Bump! Would like a way to not have to install wrangler and wasm-pack every time I publish my worker.

@ashleymichal @ashleygwilliams Will this be implemented into the next release? Installing wrangler-js and wasm-pack on every publish is super slow :( (Plus I don't even use wasm in my code)

Maybe just showing a message saying there's an update to one of the packages would be helpful? So we know when we need to update it :)