n-api: refactor napi_set_property function for improved performance

Author: mert.altin

README.md

@@ -204,6 +204,8 @@ For information about the governance of the Node.js project, see
   **Michaël Zasso** <<[email protected]>> (he/him)
 * [tniessen](https://github.com/tniessen) -
   **Tobias Nießen** <<[email protected]>> (he/him)
+* [Trott](https://github.com/Trott) -
+  **Rich Trott** <<[email protected]>> (he/him)
 
 #### TSC regular members
 
@@ -231,8 +233,6 @@ For information about the governance of the Node.js project, see
   **Rod Vagg** <<[email protected]>>
 * [TimothyGu](https://github.com/TimothyGu) -
   **Tiancheng "Timothy" Gu** <<[email protected]>> (he/him)
-* [Trott](https://github.com/Trott) -
-  **Rich Trott** <<[email protected]>> (he/him)
 
 <details>
 

lib/buffer.js

@@ -475,6 +475,7 @@ function fromString(string, encoding) {
     if (string.length === 0)
       return new FastBuffer();
     ops = encodingOps.utf8;
+    encoding = undefined;
   } else {
     ops = getEncodingOps(encoding);
     if (ops === undefined)

src/js_native_api_v8.cc

@@ -1109,6 +1109,31 @@ napi_status NAPI_CDECL napi_set_property(napi_env env,
   return GET_RETURN_STATUS(env);
 }
 
+napi_status NAPI_CDECL napi_set_property_utf16(napi_env env,
+                                                napi_value object,
+                                                const uint16_t* utf16name,
+                                                napi_value value) {
+  NAPI_PREAMBLE(env);
+  CHECK_ARG(env, utf16name);
+  CHECK_ARG(env, value);
+
+  v8::Local<v8::Context> context = env->context();
+  v8::Local<v8::Object> obj;
+
+  CHECK_TO_OBJECT(env, context, obj, object);
+
+  v8::Local<v8::Value> k = v8::String::NewFromTwoByte(context->GetIsolate(),
+                                                     utf16name,
+                                                     v8::NewStringType::kNormal)
+                               .ToLocalChecked();
+  v8::Local<v8::Value> val = v8impl::V8LocalValueFromJsValue(value);
+
+  v8::Maybe<bool> set_maybe = obj->Set(context, k, val);
+
+  RETURN_STATUS_IF_FALSE(env, set_maybe.FromMaybe(false), napi_generic_failure);
+  return GET_RETURN_STATUS(env);
+}
+
 napi_status NAPI_CDECL napi_has_property(napi_env env,
                                          napi_value object,
                                          napi_value key,

test/pummel/pummel.status

@@ -9,8 +9,6 @@ prefix pummel
 [$system==win32]
 # https://github.com/nodejs/node/issues/40728
 test-fs-watch-non-recursive: PASS,FLAKY
-# https://github.com/nodejs/node/issues/50260
-test-structuredclone-jstransferable: PASS,FLAKY
 
 [$system==linux]
 # https://github.com/nodejs/node/issues/38226