Fix TS and initialization order for executeUIRuntimeSync. (#5516)

## Summary

Minor changes to #4300.

## Test plan

🚀

Common/cpp/ReanimatedRuntime/WorkletRuntime.cpp

@@ -67,7 +67,10 @@ WorkletRuntime::WorkletRuntime(
 }
 
 std::unique_lock<std::recursive_mutex> WorkletRuntime::lock() {
-  assert(supportsLocking_ && "Runtime doesn't support locking");
+  assert(
+      supportsLocking_ &&
+      ("[Reanimated] Runtime \"" + name_ + "\" doesn't support locking.")
+          .c_str());
   return std::unique_lock<std::recursive_mutex>(runtimeMutex_);
 }
 

Common/cpp/ReanimatedRuntime/WorkletRuntime.h

@@ -65,10 +65,10 @@ class WorkletRuntime : public jsi::HostObject,
   std::vector<jsi::PropNameID> getPropertyNames(jsi::Runtime &rt) override;
 
  private:
+  std::recursive_mutex runtimeMutex_;
   const std::shared_ptr<jsi::Runtime> runtime_;
   const std::string name_;
   std::shared_ptr<AsyncQueue> queue_;
-  std::recursive_mutex runtimeMutex_;
   bool supportsLocking_;
 };
 

src/reanimated2/initializers.ts

@@ -14,10 +14,10 @@ const SHOULD_BE_USE_WEB = shouldBeUseWeb();
 const IS_CHROME_DEBUGGER = isChromeDebugger();
 
 // callGuard is only used with debug builds
-export function callGuardDEV<T extends Array<unknown>, U extends void>(
-  fn: (...args: T) => U,
-  ...args: T
-): void {
+export function callGuardDEV<Args extends unknown[], ReturnValue>(
+  fn: (...args: Args) => ReturnValue,
+  ...args: Args
+): ReturnValue | void {
   'worklet';
   try {
     return fn(...args);

src/reanimated2/threads.ts

@@ -142,8 +142,8 @@ export function executeOnUIRuntimeSync<Args extends unknown[], ReturnValue>(
     return NativeReanimatedModule.executeOnUIRuntimeSync(
       makeShareableCloneRecursive(() => {
         'worklet';
-        const ret = worklet(...args);
-        return makeShareableCloneOnUIRecursive(ret);
+        const result = worklet(...args);
+        return makeShareableCloneOnUIRecursive(result);
       })
     );
   };