fix(🔃): fix dependency cycles in header files (#257)

Author: wcandillon

packages/webgpu/cpp/jsi/RNFHybridObject.h

@@ -4,7 +4,6 @@
 
 #pragma once
 
-#include "RNFJSIConverter.h"
 #include "WGPULogger.h"
 #include "RNFWorkletRuntimeRegistry.h"
 #include <functional>
@@ -17,6 +16,15 @@
 #include <utility>
 #include <string>
 
+// Forward declare to avoid circular dependency
+namespace margelo {
+template <typename ArgType, typename SFINAE = void>
+struct JSIConverter;
+}
+
+// Include the converter - this must come after forward declaration
+#include "RNFJSIConverter.h"
+
 namespace margelo {
 
 namespace jsi = facebook::jsi;

packages/webgpu/cpp/jsi/RNFJSIConverter.h

@@ -20,14 +20,15 @@
 #include <jsi/jsi.h>
 
 #include "RNFEnumMapper.h"
-#include "RNFHybridObject.h"
 #include "RNFJSIHelper.h"
 #include "RNFPromise.h"
 #include "RNFWorkletRuntimeRegistry.h"
 
 #include "Dispatcher.h"
 #include "ThreadPool.h"
 
+#include "Unions.h"
+
 // This number is the maximum integer that can be represented exactly as a double
 #define MAX_SAFE_INTEGER static_cast<uint64_t>(9007199254740991)
 

packages/webgpu/cpp/rnwgpu/api/Convertors.h

@@ -60,6 +60,7 @@
 #include "GPUVertexAttribute.h"
 #include "GPUVertexBufferLayout.h"
 #include "GPUVertexState.h"
+#include "Unions.h"
 
 namespace rnwgpu {
 

packages/webgpu/cpp/rnwgpu/api/GPUError.h

@@ -1,8 +1,8 @@
 #pragma once
 
 #include <memory>
-#include <utility>
 #include <string>
+#include <utility>
 
 #include "webgpu/webgpu_cpp.h"
 

packages/webgpu/cpp/rnwgpu/api/GPURenderPassEncoder.cpp

@@ -2,8 +2,8 @@
 #include <string>
 #include <vector>
 
-#include "GPURenderPassEncoder.h"
 #include "Convertors.h"
+#include "GPURenderPassEncoder.h"
 
 namespace rnwgpu {