Remove @shopify/web-worker Dependency in @wordpress/upload-media

[SmushyTaco]

What problem does this address?

If you go to the NPM page for @shopify/web-worker, you'll see that it's no longer being supported. Removing this package would be a good measure to future proof.

What is your proposed solution?

Replace @shopify/web-worker with comlink or with native web workers.

[Mamaduka]

cc @swissspidy

[swissspidy]

It‘s fine to remove for now since the actual web worker implementation is missing anyway, so this can be revisited later.

That said, neither comlink nor plain workers wre necessarily a good alternative. Trust me, I did a lot of research on this :-)

Another option might be to fork the package in the future.

Anyway, we can remove & revisit.

[SmushyTaco]

@swissspidy could you elaborate on why comlink isn't a good alternative?

[swissspidy]

Too much to rehash here, I did all the research in my media-experiments repo. For one, it is effectively unmaintained. Not to mention poorer DX and fewer features compared to the Shopify library.

FWIW, the upload-media package is still work in progress, I wouldn‘t spend time worrying about its dependencies right now.