fix(worker): throw when an environment-data key cannot be converted to a string

setEnvironmentData and getEnvironmentData converted the key with a helper that swallows a throwing toString() and returns an empty string, so such a key silently read, overwrote or deleted the unrelated empty-string entry. The conversion is checked now and the exception reaches the caller.

Author: adrian-niculescu

NativeScript/runtime/Messaging.cpp

@@ -958,13 +958,28 @@ void SetEmitMessageCallback(const FunctionCallbackInfo<Value>& info) {
   state->emitMessage.Reset(isolate, info[0].As<v8::Function>());
 }
 
+// The key as the string the store files it under. A key whose conversion throws
+// leaves that exception pending for the caller rather than standing in for "".
+static bool EnvironmentDataKey(Isolate* isolate, Local<Value> value,
+                               std::string& key) {
+  Local<v8::String> str;
+  if (!value->ToString(isolate->GetCurrentContext()).ToLocal(&str)) {
+    return false;
+  }
+  key = tns::ToString(isolate, str);
+  return true;
+}
+
 void SetEnvironmentDataCallback(const FunctionCallbackInfo<Value>& info) {
   Isolate* isolate = info.GetIsolate();
   if (info.Length() < 1) {
     return;
   }
   Local<Context> context = isolate->GetCurrentContext();
-  std::string key = tns::ToString(isolate, info[0]);
+  std::string key;
+  if (!EnvironmentDataKey(isolate, info[0], key)) {
+    return;
+  }
   if (info.Length() < 2 || info[1]->IsUndefined()) {
     std::lock_guard<std::mutex> lock(g_environmentDataMutex);
     g_environmentData.erase(key);
@@ -988,7 +1003,10 @@ void GetEnvironmentDataCallback(const FunctionCallbackInfo<Value>& info) {
   if (info.Length() < 1) {
     return;
   }
-  std::string key = tns::ToString(isolate, info[0]);
+  std::string key;
+  if (!EnvironmentDataKey(isolate, info[0], key)) {
+    return;
+  }
   std::shared_ptr<serialization::SerializedValue> stored;
   {
     std::lock_guard<std::mutex> lock(g_environmentDataMutex);

TestRunner/app/tests/MessagingTests.js

@@ -295,6 +295,34 @@ describe("Messaging runtime edges", function () {
         });
     });
 
+    describe("environment data keys", function () {
+        function thrownMessage(fn) {
+            try {
+                fn();
+            } catch (e) {
+                return e && e.message;
+            }
+            return "nothing thrown";
+        }
+
+        it("throws what a key's toString throws instead of using the empty-string key", function () {
+            var wt = require("node:worker_threads");
+            var key = { toString: function () { throw new Error("key toString failed"); } };
+            wt.setEnvironmentData("", "empty");
+            try {
+                expect(thrownMessage(function () { wt.setEnvironmentData(key, "other"); }))
+                    .toBe("key toString failed");
+                expect(thrownMessage(function () { wt.getEnvironmentData(key); }))
+                    .toBe("key toString failed");
+                expect(thrownMessage(function () { wt.setEnvironmentData(key); }))
+                    .toBe("key toString failed");
+                expect(wt.getEnvironmentData("")).toBe("empty");
+            } finally {
+                wt.setEnvironmentData("");
+            }
+        });
+    });
+
     describe("worker error reporting", function () {
         // A worker boots on its own thread, so the first error arrives whenever
         // the runner gets to it; specs wait for it and only then settle for