Fix JSValue leak in Detach causing JS_FreeRuntime assert on Android

Napi::Detach now calls JS_FreeValue on all remaining JSValues in the
handle scope stack before deleting the env. Previously unique_ptr only
freed heap memory without decrementing QuickJS reference counts.

Co-authored-by: Copilot <[email protected]>

Author: CedricGuillemet

Core/Node-API/Source/env_quickjs.cc

@@ -52,7 +52,16 @@ namespace Napi
             if (!JS_IsUndefined(env_ptr->has_own_property_function))
             {
                 JS_FreeValue(env_ptr->context, env_ptr->has_own_property_function);
+                env_ptr->has_own_property_function = JS_UNDEFINED;
             }
+
+            // Free all remaining JSValues in the handle scope stack
+            for (auto& ptr : env_ptr->handle_scope_stack)
+            {
+                JS_FreeValue(env_ptr->context, *ptr);
+            }
+            env_ptr->handle_scope_stack.clear();
+
             delete env_ptr;
         }
     }