fix: Rename Dispatcher global name to avoid conflicts with Nitro (#271)

* fix: Rename `Dispatcher` global name to avoid conflicts with Nitro

Since WebGPU uses an early copy of Nitro (kinda like a fork), there are some conflicts that users will run into when they install WebGPU in an app that already has Nitro Modules installed (or other way around).

An official full fix would be to migrate WebGPU to Nitro, not only for better maintenance, but also for better performance and stability. I really don't think the migration is a huge effort.

For now, this PR fixes the name of the `Dispatcher` that is injected into global, as Nitro uses the same name and when one of the two Nitro versions downcasts the global into the specific C++ symbol, we have conflics since it's two different C++ symbols - it's either `nitro::margelo::Dispatcher` (the official one) or the WebGPU `margelo::Dispatcher`.

It's essentially a race condition with no clear error log since its just a C++ pointer cast that fails (super hard to debug lol)

Fixes #270

* Bump version from 0.2.9 to 0.2.10

---------

Co-authored-by: William Candillon <wcandillon@gmail.com>

Author: mrousavy

packages/webgpu/cpp/threading/Dispatcher.cpp

@@ -12,7 +12,7 @@ namespace margelo {
 
 namespace jsi = facebook::jsi;
 
-static constexpr auto GLOBAL_DISPATCHER_HOLDER_NAME = "__nitroDispatcher";
+static constexpr auto GLOBAL_DISPATCHER_HOLDER_NAME = "__nitroDispatcher_WEBGPU_FORK";
 
 std::unordered_map<jsi::Runtime *, std::weak_ptr<Dispatcher>>
     Dispatcher::_globalCache;

packages/webgpu/package.json

@@ -1,6 +1,6 @@
 {
   "name": "react-native-wgpu",
-  "version": "0.2.9",
+  "version": "0.2.10",
   "description": "React Native WebGPU",
   "main": "lib/commonjs/index",
   "module": "lib/module/index",