Creating a script "worker.js" will lead to racing condition with Basis.

[Maksims]

If you create a script, that is called "worker.js" with basic ScriptType implementation called "worker", and use Basis in your project. Then there is a racing condition, where it can lead to Basis failing to load. Only replicable when loading with cleared cache, or there is a significant project size (some delay for loading?).

Project that replicates an issue: https://playcanvas.com/project/1440466/overview/model-viewer
Launch with "disable cache" checkbox in dev tools.

Error:

[mvaligursky]

I think the assert might be important here:

[Maksims]

I think the assert might be important here:

But that only when running with debug engine, while it breaks with live engine too.
I believe breaking cases should be clearly logged and prevented, regardless of engine flavor.

[LeXXik]

I think the assert might be important here

It is not. The issue here is that the ScriptType creates a global variable named Worker:

var Worker = pc.createScript('worker');

which basically overwrites WebWorker at window.Worker, and Basis then calling new Worker() calls the constructor of the script instance, instead of web worker.

One solution here would be to add worker as a restricted name for a script. But then, do we restrict it for all the js objects?...

Edit: perhaps BasisClient could store a static reference to window.Worker and call that instead of resolving Worker at runtime?