fix(jerryscript): fix memory leak in JS function handle deleter

The custom deleter for wrapped JS functions was not deleting the
jerry_value_t handle memory in some edge cases (context cleaned up,
wrong thread, etc.). Fixed by ensuring delete h is always called,
only skipping jerry_value_free when appropriate.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <[email protected]>

Author: alandefreitas

src/lib/Support/JavaScript.cpp

@@ -2021,15 +2021,14 @@ toDomValue(jerry_value_t v, std::shared_ptr<Context::Impl> const& impl)
             // If Impl is gone or cleanup() was called, the value is already freed.
             if (auto locked = weak_impl.lock())
             {
-                if (!locked->alive || !locked->jerry_ctx || locked->cleaning_up)
-                    return;
-
-                if (locked->owner_thread != std::this_thread::get_id())
-                    return; // Different thread: skip to avoid UB
-
-                auto lock = lockContext(locked);
-                jerry_value_free(*h);
+                if (locked->alive && locked->jerry_ctx && !locked->cleaning_up
+                    && locked->owner_thread == std::this_thread::get_id())
+                {
+                    auto lock = lockContext(locked);
+                    jerry_value_free(*h);
+                }
             }
+            // Always delete the handle memory, even if we skipped jerry_value_free
             delete h;
         });