In default dune builds, the brr library raises an exception in web workers

[andrewray]

I think without aggressive dead code elimination (which is the case with --profile=dev dune builds) there is something that accesses the global document object, and this causes an exception when the code is run in a web worker context.

With --profile=release things seem to be OK.

Maybe worth some comment in the documentation, though it would be really nice if things worked smoothly in dev mode also.

[dbuenzli]

Yes ! It was fixed in 655a3d6. I will make a release anytime soon.

[dbuenzli]

(if you can confirm that opam pin brr --dev works for you, it's even better)

[andrewray]

Yup, it works by pinning the dev version. Thanks!

[dbuenzli]

Excellent, thanks for the confirmation.

Maybe worth some comment in the documentation, though it would be really nice if things worked smoothly in dev mode also.

It's not only about dev mode. I like to be able to have a single js artefact and simply:

let () = if Brr_webworkers.Worker.ami () then worker () else main ()

(if you write the the artefact in the html file and perform terrible circonvolutions it's even possible to make it work over the file:// protocol).