react-native-windows 0.82.3 → 0.83.0-preview.1

package/Microsoft.ReactNative/ReactNativeWindow.idl

@@ -0,0 +1,44 @@
+// Copyright (c) Microsoft Corporation.
+// Licensed under the MIT License.
+
+import "ReactNativeIsland.idl";
+
+#include "NamespaceRedirect.h"
+#include "DocString.h"
+
+namespace Microsoft.ReactNative
+{
+
+  [experimental]
+  enum ContentSizePolicy
+  {
+    None,
+    ResizeContentToParentWindow,
+    ResizeParentWindowToContent,
+  };
+
+  [default_interface]
+  [webhosthidden]
+  [experimental]
+  DOC_STRING("Simplifies hosting a ReactNativeIsland inside a Window.")
+  runtimeclass ReactNativeWindow : Windows.Foundation.IClosable
+  {
+    static ReactNativeWindow CreateFromCompositor(Microsoft.UI.Composition.Compositor compositor);
+    static ReactNativeWindow CreateFromWindow(Microsoft.UI.Windowing.AppWindow appWindow, Microsoft.UI.Composition.Compositor compositor);
+    static ReactNativeWindow CreateFromContentSiteBridgeAndIsland(Microsoft.UI.Content.IContentSiteBridge bridge, Microsoft.ReactNative.ReactNativeIsland island);
+    static ReactNativeWindow CreateFromWindowAndIsland(Microsoft.UI.Windowing.AppWindow appWindow, Microsoft.ReactNative.ReactNativeIsland island);
+
+    ReactNativeIsland ReactNativeIsland { get; };
+
+    Microsoft.UI.Windowing.AppWindow AppWindow { get; };
+
+    Microsoft.UI.Content.IContentSiteBridge ContentSiteBridge { get; };
+    ContentSizePolicy ResizePolicy { get; set; };
+
+    DOC_STRING("When the ResizePolicy is set to ResizeParentWindowToContent, these LayoutConstraints will control the layout of the content."
+     " When ResizePolicy is set to ResizeContentToParentWindow, these constraints will be ignored. Instead the window can be constrained using"
+     " OverlappedPresenter.PreferredMaximumHeight etc.")
+    LayoutConstraints LayoutConstraints { get; set; };
+  }
+
+} // namespace Microsoft.ReactNative

package/Microsoft.ReactNative.Cxx/AutoDraw.h

@@ -16,7 +16,10 @@ class AutoDrawDrawingSurface {
       POINT *offset) noexcept {
     drawingSurface.as(m_drawingSurfaceInterop);
     auto dpi = scaleFactor * 96.0f;
-    m_drawingSurfaceInterop->BeginDraw(m_d2dDeviceContext.put(), dpi, dpi, offset);
+    auto hr = m_drawingSurfaceInterop->BeginDraw(m_d2dDeviceContext.put(), dpi, dpi, offset);
+    if (FAILED(hr)) {
+      m_d2dDeviceContext = nullptr;
+    }
   }
 
   ~AutoDrawDrawingSurface() noexcept {
@@ -34,6 +37,11 @@ class AutoDrawDrawingSurface {
     return m_d2dDeviceContext.get();
   }
 
+  // Returns true if the device context is valid and safe to use for drawing operations
+  bool IsValid() const noexcept {
+    return m_d2dDeviceContext != nullptr;
+  }
+
  private:
   winrt::com_ptr<Experimental::ICompositionDrawingSurfaceInterop> m_drawingSurfaceInterop;
   winrt::com_ptr<ID2D1DeviceContext> m_d2dDeviceContext;

package/Microsoft.ReactNative.Cxx/ReactCommon/CallInvoker.h

@@ -18,7 +18,7 @@ class Runtime;
 
 namespace facebook::react {
 
-using CallFunc = std::function<void(jsi::Runtime&)>;
+using CallFunc = std::function<void(jsi::Runtime &)>;
 
 /**
  * An interface for a generic native-to-JS call invoker. See BridgeJSCallInvoker
@@ -26,23 +26,24 @@ using CallFunc = std::function<void(jsi::Runtime&)>;
  */
 class CallInvoker {
  public:
-  virtual void invokeAsync(CallFunc&& func) noexcept = 0;
-  virtual void invokeAsync(
-      SchedulerPriority /*priority*/,
-      CallFunc&& func) noexcept {
+  virtual void invokeAsync(CallFunc &&func) noexcept = 0;
+  virtual void invokeAsync(SchedulerPriority /*priority*/, CallFunc &&func) noexcept
+  {
     // When call with priority is not implemented, fall back to a regular async
     // execution
     invokeAsync(std::move(func));
   }
-  virtual void invokeSync(CallFunc&& func) = 0;
+  virtual void invokeSync(CallFunc &&func) = 0;
 
   // Backward compatibility only, prefer the CallFunc methods instead
-  virtual void invokeAsync(std::function<void()>&& func) noexcept {
-    invokeAsync([func](jsi::Runtime&) { func(); });
+  virtual void invokeAsync(std::function<void()> &&func) noexcept
+  {
+    invokeAsync([func = std::move(func)](jsi::Runtime &) { func(); });
   }
 
-  virtual void invokeSync(std::function<void()>&& func) {
-    invokeSync([func](jsi::Runtime&) { func(); });
+  virtual void invokeSync(std::function<void()> &&func)
+  {
+    invokeSync([func = std::move(func)](jsi::Runtime &) { func(); });
   }
 
   virtual ~CallInvoker() = default;
@@ -52,12 +53,8 @@ using NativeMethodCallFunc = std::function<void()>;
 
 class NativeMethodCallInvoker {
  public:
-  virtual void invokeAsync(
-      const std::string& methodName,
-      NativeMethodCallFunc&& func) noexcept = 0;
-  virtual void invokeSync(
-      const std::string& methodName,
-      NativeMethodCallFunc&& func) = 0;
+  virtual void invokeAsync(const std::string &methodName, NativeMethodCallFunc &&func) noexcept = 0;
+  virtual void invokeSync(const std::string &methodName, NativeMethodCallFunc &&func) = 0;
   virtual ~NativeMethodCallInvoker() = default;
 };
 

package/Microsoft.ReactNative.Cxx/ReactCommon/TurboModule.h

@@ -36,9 +36,7 @@ enum TurboModuleMethodValueKind {
 /**
  * Determines TurboModuleMethodValueKind based on the jsi::Value type.
  */
-TurboModuleMethodValueKind getTurboModuleMethodValueKind(
-    jsi::Runtime& rt,
-    const jsi::Value* value);
+TurboModuleMethodValueKind getTurboModuleMethodValueKind(jsi::Runtime &rt, const jsi::Value *value);
 
 class TurboCxxModule;
 class TurboModuleBinding;
@@ -55,8 +53,8 @@ class JSI_EXPORT TurboModule : public jsi::HostObject {
   // JS representation. To customize lookup of properties, override `create`.
   // Note: keep this method declared inline to avoid conflicts
   // between RTTI and non-RTTI compilation units
-  jsi::Value get(jsi::Runtime& runtime, const jsi::PropNameID& propName)
-      override {
+  jsi::Value get(jsi::Runtime &runtime, const jsi::PropNameID &propName) override
+  {
     auto prop = create(runtime, propName);
     // If we have a JS wrapper, cache the result of this lookup
     // We don't cache misses, to allow for methodMap_ to dynamically be
@@ -64,15 +62,14 @@ class JSI_EXPORT TurboModule : public jsi::HostObject {
     // [Windows] Reenable once https://github.com/microsoft/react-native-windows/issues/14128 is fixed
 #ifndef WINAPI_FAMILY
     if (jsRepresentation_ && !prop.isUndefined()) {
-      jsRepresentation_->lock(runtime).asObject(runtime).setProperty(
-          runtime, propName, prop);
+      jsRepresentation_->lock(runtime).asObject(runtime).setProperty(runtime, propName, prop);
     }
 #endif
     return prop;
   }
 
-  std::vector<jsi::PropNameID> getPropertyNames(
-      jsi::Runtime& runtime) override {
+  std::vector<jsi::PropNameID> getPropertyNames(jsi::Runtime &runtime) override
+  {
     std::vector<jsi::PropNameID> result;
     result.reserve(methodMap_.size());
     for (auto it = methodMap_.cbegin(); it != methodMap_.cend(); ++it) {
@@ -87,18 +84,14 @@ class JSI_EXPORT TurboModule : public jsi::HostObject {
 
   struct MethodMetadata {
     size_t argCount;
-    jsi::Value (*invoker)(
-        jsi::Runtime& rt,
-        TurboModule& turboModule,
-        const jsi::Value* args,
-        size_t count);
+    jsi::Value (*invoker)(jsi::Runtime &rt, TurboModule &turboModule, const jsi::Value *args, size_t count);
   };
   std::unordered_map<std::string, MethodMetadata> methodMap_;
-  std::unordered_map<std::string, std::shared_ptr<IAsyncEventEmitter>>
-      eventEmitterMap_;
 
-  using ArgFactory =
-      std::function<void(jsi::Runtime& runtime, std::vector<jsi::Value>& args)>;
+  friend class TurboModuleTestFixtureInternal;
+  std::unordered_map<std::string, std::shared_ptr<IAsyncEventEmitter>> eventEmitterMap_;
+
+  using ArgFactory = std::function<void(jsi::Runtime &runtime, std::vector<jsi::Value> &args)>;
 
   /**
    * Calls RCTDeviceEventEmitter.emit to JavaScript, with given event name and
@@ -112,35 +105,31 @@ class JSI_EXPORT TurboModule : public jsi::HostObject {
    *      args.emplace_back(jsi::Value(42));
    *  });
    */
-  void emitDeviceEvent(
-      const std::string& eventName,
-      ArgFactory&& argFactory = nullptr);
+  void emitDeviceEvent(const std::string &eventName, ArgFactory &&argFactory = nullptr);
 
   // Backwards compatibility version
   void emitDeviceEvent(
-      jsi::Runtime& /*runtime*/,
+      jsi::Runtime & /*runtime*/,
 
-      const std::string& eventName,
-      ArgFactory&& argFactory = nullptr) {
+      const std::string &eventName,
+      ArgFactory &&argFactory = nullptr)
+  {
     emitDeviceEvent(eventName, std::move(argFactory));
   }
 
-  virtual jsi::Value create(
-      jsi::Runtime& runtime,
-      const jsi::PropNameID& propName) {
+  virtual jsi::Value create(jsi::Runtime &runtime, const jsi::PropNameID &propName)
+  {
     std::string propNameUtf8 = propName.utf8(runtime);
-    if (auto methodIter = methodMap_.find(propNameUtf8);
-        methodIter != methodMap_.end()) {
-      const MethodMetadata& meta = methodIter->second;
+    if (auto methodIter = methodMap_.find(propNameUtf8); methodIter != methodMap_.end()) {
+      const MethodMetadata &meta = methodIter->second;
       return jsi::Function::createFromHostFunction(
           runtime,
           propName,
           static_cast<unsigned int>(meta.argCount),
           [this, meta](
-              jsi::Runtime& rt,
-              [[maybe_unused]] const jsi::Value& thisVal,
-              const jsi::Value* args,
-              size_t count) { return meta.invoker(rt, *this, args, count); });
+              jsi::Runtime &rt, [[maybe_unused]] const jsi::Value &thisVal, const jsi::Value *args, size_t count) {
+            return meta.invoker(rt, *this, args, count);
+          });
     } else if (auto eventEmitterIter = eventEmitterMap_.find(propNameUtf8);
                eventEmitterIter != eventEmitterMap_.end()) {
       return eventEmitterIter->second->get(runtime, jsInvoker_);
@@ -159,7 +148,6 @@ class JSI_EXPORT TurboModule : public jsi::HostObject {
  * An app/platform-specific provider function to get an instance of a module
  * given a name.
  */
-using TurboModuleProviderFunctionType =
-    std::function<std::shared_ptr<TurboModule>(const std::string& name)>;
+using TurboModuleProviderFunctionType = std::function<std::shared_ptr<TurboModule>(const std::string &name)>;
 
 } // namespace facebook::react

package/Microsoft.ReactNative.Cxx/ReactCommon/TurboModuleUtils.h

@@ -16,19 +16,16 @@
 namespace facebook::react {
 
 struct Promise : public LongLivedObject {
-  Promise(jsi::Runtime& rt, jsi::Function resolve, jsi::Function reject);
+  Promise(jsi::Runtime &rt, jsi::Function resolve, jsi::Function reject);
 
-  void resolve(const jsi::Value& result);
-  void reject(const std::string& message);
+  void resolve(const jsi::Value &result);
+  void reject(const std::string &message);
 
   jsi::Function resolve_;
   jsi::Function reject_;
 };
 
-using PromiseSetupFunctionType =
-    std::function<void(jsi::Runtime& rt, std::shared_ptr<Promise>)>;
-jsi::Value createPromiseAsJSIValue(
-    jsi::Runtime& rt,
-    PromiseSetupFunctionType&& func);
+using PromiseSetupFunctionType = std::function<void(jsi::Runtime &rt, std::shared_ptr<Promise>)>;
+jsi::Value createPromiseAsJSIValue(jsi::Runtime &rt, PromiseSetupFunctionType &&func);
 
 } // namespace facebook::react

package/Microsoft.ReactNative.Cxx/ReactCommon/react/bridging/AString.h

@@ -16,25 +16,27 @@ namespace facebook::react {
 
 template <>
 struct Bridging<std::string> {
-  static std::string fromJs(jsi::Runtime& rt, const jsi::String& value) {
+  static std::string fromJs(jsi::Runtime &rt, const jsi::String &value)
+  {
     return value.utf8(rt);
   }
 
-  static jsi::String toJs(jsi::Runtime& rt, const std::string& value) {
+  static jsi::String toJs(jsi::Runtime &rt, const std::string &value)
+  {
     return jsi::String::createFromUtf8(rt, value);
   }
 };
 
 template <>
 struct Bridging<std::string_view> {
-  static jsi::String toJs(jsi::Runtime& rt, std::string_view value) {
-    return jsi::String::createFromUtf8(
-        rt, reinterpret_cast<const uint8_t*>(value.data()), value.length());
+  static jsi::String toJs(jsi::Runtime &rt, std::string_view value)
+  {
+    return jsi::String::createFromUtf8(rt, reinterpret_cast<const uint8_t *>(value.data()), value.length());
   }
 };
 
 template <>
-struct Bridging<const char*> : Bridging<std::string_view> {};
+struct Bridging<const char *> : Bridging<std::string_view> {};
 
 template <size_t N>
 struct Bridging<char[N]> : Bridging<std::string_view> {};

package/Microsoft.ReactNative.Cxx/ReactCommon/react/bridging/Array.h

@@ -24,35 +24,31 @@ namespace array_detail {
 
 template <typename T, size_t N>
 struct BridgingStatic {
-  static jsi::Array toJs(
-      jsi::Runtime& rt,
-      const T& array,
-      const std::shared_ptr<CallInvoker>& jsInvoker) {
+  static jsi::Array toJs(jsi::Runtime &rt, const T &array, const std::shared_ptr<CallInvoker> &jsInvoker)
+  {
     return toJs(rt, array, jsInvoker, std::make_index_sequence<N>{});
   }
 
  private:
   template <size_t... Index>
   static jsi::Array toJs(
-      facebook::jsi::Runtime& rt,
-      const T& array,
-      const std::shared_ptr<CallInvoker>& jsInvoker,
-      std::index_sequence<Index...>) {
-    return jsi::Array::createWithElements(
-        rt, bridging::toJs(rt, std::get<Index>(array), jsInvoker)...);
+      facebook::jsi::Runtime &rt,
+      const T &array,
+      const std::shared_ptr<CallInvoker> &jsInvoker,
+      std::index_sequence<Index...> /*unused*/)
+  {
+    return jsi::Array::createWithElements(rt, bridging::toJs(rt, std::get<Index>(array), jsInvoker)...);
   }
 };
 
 template <typename T>
 struct BridgingDynamic {
-  static jsi::Array toJs(
-      jsi::Runtime& rt,
-      const T& list,
-      const std::shared_ptr<CallInvoker>& jsInvoker) {
+  static jsi::Array toJs(jsi::Runtime &rt, const T &list, const std::shared_ptr<CallInvoker> &jsInvoker)
+  {
     jsi::Array result(rt, list.size());
     size_t index = 0;
 
-    for (const auto& item : list) {
+    for (const auto &item : list) {
       result.setValueAtIndex(rt, index++, bridging::toJs(rt, item, jsInvoker));
     }
 
@@ -63,18 +59,15 @@ struct BridgingDynamic {
 } // namespace array_detail
 
 template <typename T, size_t N>
-struct Bridging<std::array<T, N>>
-    : array_detail::BridgingStatic<std::array<T, N>, N> {
-  static std::array<T, N> fromJs(
-      facebook::jsi::Runtime& rt,
-      const jsi::Array& array,
-      const std::shared_ptr<CallInvoker>& jsInvoker) {
+struct Bridging<std::array<T, N>> : array_detail::BridgingStatic<std::array<T, N>, N> {
+  static std::array<T, N>
+  fromJs(facebook::jsi::Runtime &rt, const jsi::Array &array, const std::shared_ptr<CallInvoker> &jsInvoker)
+  {
     size_t length = array.length(rt);
 
     std::array<T, N> result;
     for (size_t i = 0; i < length; i++) {
-      result[i] =
-          bridging::fromJs<T>(rt, array.getValueAtIndex(rt, i), jsInvoker);
+      result[i] = bridging::fromJs<T>(rt, array.getValueAtIndex(rt, i), jsInvoker);
     }
 
     return result;
@@ -82,12 +75,10 @@ struct Bridging<std::array<T, N>>
 };
 
 template <typename T1, typename T2>
-struct Bridging<std::pair<T1, T2>>
-    : array_detail::BridgingStatic<std::pair<T1, T2>, 2> {
-  static std::pair<T1, T1> fromJs(
-      facebook::jsi::Runtime& rt,
-      const jsi::Array& array,
-      const std::shared_ptr<CallInvoker>& jsInvoker) {
+struct Bridging<std::pair<T1, T2>> : array_detail::BridgingStatic<std::pair<T1, T2>, 2> {
+  static std::pair<T1, T1>
+  fromJs(facebook::jsi::Runtime &rt, const jsi::Array &array, const std::shared_ptr<CallInvoker> &jsInvoker)
+  {
     return std::make_pair(
         bridging::fromJs<T1>(rt, array.getValueAtIndex(rt, 0), jsInvoker),
         bridging::fromJs<T2>(rt, array.getValueAtIndex(rt, 1), jsInvoker));
@@ -95,35 +86,29 @@ struct Bridging<std::pair<T1, T2>>
 };
 
 template <typename... Types>
-struct Bridging<std::tuple<Types...>>
-    : array_detail::BridgingStatic<std::tuple<Types...>, sizeof...(Types)> {};
+struct Bridging<std::tuple<Types...>> : array_detail::BridgingStatic<std::tuple<Types...>, sizeof...(Types)> {};
 
 template <typename T>
-struct Bridging<std::deque<T>> : array_detail::BridgingDynamic<std::deque<T>> {
-};
+struct Bridging<std::deque<T>> : array_detail::BridgingDynamic<std::deque<T>> {};
 
 template <typename T>
-struct Bridging<std::initializer_list<T>>
-    : array_detail::BridgingDynamic<std::initializer_list<T>> {};
+struct Bridging<std::initializer_list<T>> : array_detail::BridgingDynamic<std::initializer_list<T>> {};
 
 template <typename T>
 struct Bridging<std::list<T>> : array_detail::BridgingDynamic<std::list<T>> {};
 
 template <typename T>
-struct Bridging<std::vector<T>>
-    : array_detail::BridgingDynamic<std::vector<T>> {
-  static std::vector<T> fromJs(
-      facebook::jsi::Runtime& rt,
-      const jsi::Array& array,
-      const std::shared_ptr<CallInvoker>& jsInvoker) {
+struct Bridging<std::vector<T>> : array_detail::BridgingDynamic<std::vector<T>> {
+  static std::vector<T>
+  fromJs(facebook::jsi::Runtime &rt, const jsi::Array &array, const std::shared_ptr<CallInvoker> &jsInvoker)
+  {
     size_t length = array.length(rt);
 
     std::vector<T> vector;
     vector.reserve(length);
 
     for (size_t i = 0; i < length; i++) {
-      vector.push_back(
-          bridging::fromJs<T>(rt, array.getValueAtIndex(rt, i), jsInvoker));
+      vector.push_back(bridging::fromJs<T>(rt, array.getValueAtIndex(rt, i), jsInvoker));
     }
 
     return vector;
@@ -132,16 +117,14 @@ struct Bridging<std::vector<T>>
 
 template <typename T>
 struct Bridging<std::set<T>> : array_detail::BridgingDynamic<std::set<T>> {
-  static std::set<T> fromJs(
-      facebook::jsi::Runtime& rt,
-      const jsi::Array& array,
-      const std::shared_ptr<CallInvoker>& jsInvoker) {
+  static std::set<T>
+  fromJs(facebook::jsi::Runtime &rt, const jsi::Array &array, const std::shared_ptr<CallInvoker> &jsInvoker)
+  {
     size_t length = array.length(rt);
 
     std::set<T> set;
     for (size_t i = 0; i < length; i++) {
-      set.insert(
-          bridging::fromJs<T>(rt, array.getValueAtIndex(rt, i), jsInvoker));
+      set.insert(bridging::fromJs<T>(rt, array.getValueAtIndex(rt, i), jsInvoker));
     }
 
     return set;

package/Microsoft.ReactNative.Cxx/ReactCommon/react/bridging/Base.h

@@ -24,7 +24,7 @@ struct Bridging;
 template <>
 struct Bridging<void> {
   // Highly generic code may result in "casting" to void.
-  static void fromJs(jsi::Runtime&, const jsi::Value&) {}
+  static void fromJs(jsi::Runtime & /*unused*/, const jsi::Value & /*unused*/) {}
 };
 
 namespace bridging {
@@ -51,9 +51,7 @@ struct bridging_wrapper {
 // Convert lambda types to move-only function types since we can't specialize
 // Bridging templates for arbitrary lambdas.
 template <typename T>
-struct bridging_wrapper<
-    T,
-    std::void_t<decltype(&remove_cvref_t<T>::operator())>>
+struct bridging_wrapper<T, std::void_t<decltype(&remove_cvref_t<T>::operator())>>
     : function_wrapper<decltype(&remove_cvref_t<T>::operator())> {};
 
 } // namespace detail
@@ -63,70 +61,45 @@ using bridging_t = typename detail::bridging_wrapper<T>::type;
 
 template <typename ReturnT, typename JSArgT>
   requires is_jsi_v<JSArgT>
-auto fromJs(
-    jsi::Runtime& rt,
-    JSArgT&& value,
-    const std::shared_ptr<CallInvoker>& /*unused*/)
-    -> decltype(static_cast<ReturnT>(
-        std::move(convert(rt, std::forward<JSArgT>(value)))))
+auto fromJs(jsi::Runtime &rt, JSArgT &&value, const std::shared_ptr<CallInvoker> & /*unused*/)
+    -> decltype(static_cast<ReturnT>(std::move(convert(rt, std::forward<JSArgT>(value)))))
 
 {
-  return static_cast<ReturnT>(
-      std::move(convert(rt, std::forward<JSArgT>(value))));
+  return static_cast<ReturnT>(std::move(convert(rt, std::forward<JSArgT>(value))));
 }
 
 template <typename ReturnT, typename JSArgT>
-auto fromJs(
-    jsi::Runtime& rt,
-    JSArgT&& value,
-    const std::shared_ptr<CallInvoker>& /*unused*/)
-    -> decltype(Bridging<remove_cvref_t<ReturnT>>::fromJs(
-        rt,
-        convert(rt, std::forward<JSArgT>(value)))) {
-  return Bridging<remove_cvref_t<ReturnT>>::fromJs(
-      rt, convert(rt, std::forward<JSArgT>(value)));
+auto fromJs(jsi::Runtime &rt, JSArgT &&value, const std::shared_ptr<CallInvoker> & /*unused*/)
+    -> decltype(Bridging<remove_cvref_t<ReturnT>>::fromJs(rt, convert(rt, std::forward<JSArgT>(value))))
+{
+  return Bridging<remove_cvref_t<ReturnT>>::fromJs(rt, convert(rt, std::forward<JSArgT>(value)));
 }
 
 template <typename ReturnT, typename JSArgT>
-auto fromJs(
-    jsi::Runtime& rt,
-    JSArgT&& value,
-    const std::shared_ptr<CallInvoker>& jsInvoker)
-    -> decltype(Bridging<remove_cvref_t<ReturnT>>::fromJs(
-        rt,
-        convert(rt, std::forward<JSArgT>(value)),
-        jsInvoker)) {
-  return Bridging<remove_cvref_t<ReturnT>>::fromJs(
-      rt, convert(rt, std::forward<JSArgT>(value)), jsInvoker);
+auto fromJs(jsi::Runtime &rt, JSArgT &&value, const std::shared_ptr<CallInvoker> &jsInvoker)
+    -> decltype(Bridging<remove_cvref_t<ReturnT>>::fromJs(rt, convert(rt, std::forward<JSArgT>(value)), jsInvoker))
+{
+  return Bridging<remove_cvref_t<ReturnT>>::fromJs(rt, convert(rt, std::forward<JSArgT>(value)), jsInvoker);
 }
 
 template <typename T>
   requires is_jsi_v<T>
-auto toJs(
-    jsi::Runtime& rt,
-    T&& value,
-    const std::shared_ptr<CallInvoker>& = nullptr) -> remove_cvref_t<T> {
+auto toJs(jsi::Runtime &rt, T &&value, const std::shared_ptr<CallInvoker> & /*unused*/ = nullptr) -> remove_cvref_t<T>
+{
   return convert(rt, std::forward<T>(value));
 }
 
 template <typename T>
-auto toJs(
-    jsi::Runtime& rt,
-    T&& value,
-    const std::shared_ptr<CallInvoker>& = nullptr)
-    -> decltype(Bridging<bridging_t<T>>::toJs(rt, std::forward<T>(value))) {
+auto toJs(jsi::Runtime &rt, T &&value, const std::shared_ptr<CallInvoker> & /*unused*/ = nullptr)
+    -> decltype(Bridging<bridging_t<T>>::toJs(rt, std::forward<T>(value)))
+{
   return Bridging<bridging_t<T>>::toJs(rt, std::forward<T>(value));
 }
 
 template <typename T>
-auto toJs(
-    jsi::Runtime& rt,
-    T&& value,
-    const std::shared_ptr<CallInvoker>& jsInvoker)
-    -> decltype(Bridging<bridging_t<T>>::toJs(
-        rt,
-        std::forward<T>(value),
-        jsInvoker)) {
+auto toJs(jsi::Runtime &rt, T &&value, const std::shared_ptr<CallInvoker> &jsInvoker)
+    -> decltype(Bridging<bridging_t<T>>::toJs(rt, std::forward<T>(value), jsInvoker))
+{
   return Bridging<bridging_t<T>>::toJs(rt, std::forward<T>(value), jsInvoker);
 }
 
@@ -137,19 +110,13 @@ template <typename T, typename Arg>
 inline constexpr bool supportsFromJs<
     T,
     Arg,
-    std::void_t<decltype(fromJs<T>(
-        std::declval<jsi::Runtime&>(),
-        std::declval<Arg>(),
-        nullptr))>> = true;
+    std::void_t<decltype(fromJs<T>(std::declval<jsi::Runtime &>(), std::declval<Arg>(), nullptr))>> = true;
 
 template <typename T>
 inline constexpr bool supportsFromJs<
     T,
     jsi::Value,
-    std::void_t<decltype(fromJs<T>(
-        std::declval<jsi::Runtime&>(),
-        std::declval<jsi::Value>(),
-        nullptr))>> = true;
+    std::void_t<decltype(fromJs<T>(std::declval<jsi::Runtime &>(), std::declval<jsi::Value>(), nullptr))>> = true;
 
 template <typename, typename = jsi::Value, typename = void>
 inline constexpr bool supportsToJs = false;
@@ -158,31 +125,15 @@ template <typename JSReturnT, typename ReturnT>
 inline constexpr bool supportsToJs<
     JSReturnT,
     ReturnT,
-    std::void_t<decltype(toJs(
-        std::declval<jsi::Runtime&>(),
-        std::declval<JSReturnT>(),
-        nullptr))>> =
-    std::is_convertible_v<
-        decltype(toJs(
-            std::declval<jsi::Runtime&>(),
-            std::declval<JSReturnT>(),
-            nullptr)),
-        ReturnT>;
+    std::void_t<decltype(toJs(std::declval<jsi::Runtime &>(), std::declval<JSReturnT>(), nullptr))>> =
+    std::is_convertible_v<decltype(toJs(std::declval<jsi::Runtime &>(), std::declval<JSReturnT>(), nullptr)), ReturnT>;
 
 template <typename ReturnT>
 inline constexpr bool supportsToJs<
     ReturnT,
     jsi::Value,
-    std::void_t<decltype(toJs(
-        std::declval<jsi::Runtime&>(),
-        std::declval<ReturnT>(),
-        nullptr))>> =
-    std::is_convertible_v<
-        decltype(toJs(
-            std::declval<jsi::Runtime&>(),
-            std::declval<ReturnT>(),
-            nullptr)),
-        jsi::Value>;
+    std::void_t<decltype(toJs(std::declval<jsi::Runtime &>(), std::declval<ReturnT>(), nullptr))>> =
+    std::is_convertible_v<decltype(toJs(std::declval<jsi::Runtime &>(), std::declval<ReturnT>(), nullptr)), jsi::Value>;
 
 } // namespace bridging
 } // namespace facebook::react

package/Microsoft.ReactNative.Cxx/ReactCommon/react/bridging/Bool.h

@@ -13,11 +13,13 @@ namespace facebook::react {
 
 template <>
 struct Bridging<bool> {
-  static bool fromJs(jsi::Runtime&, const jsi::Value& value) {
+  static bool fromJs(jsi::Runtime & /*unused*/, const jsi::Value &value)
+  {
     return value.asBool();
   }
 
-  static bool toJs(jsi::Runtime& /*unused*/, bool value) {
+  static bool toJs(jsi::Runtime & /*unused*/, bool value)
+  {
     return value;
   }
 };