JsonValue primitives

[SGStino]

Version used

3.1.0

Describe the bug

JsonNodes get converted to JsValues, but if a JsonValue originates from a .NET integer, the conversion results in undefined

To Reproduce
roslynpad:

#r "nuget: Jint, 3.1.0"
using System.Text.Json.Nodes;
using Jint;
var engine = new Engine();

engine.SetValue("int", JsonValue.Create(15));
engine.GetValue("int").Dump(); // JsUndefined
engine.SetValue("double", JsonValue.Create(15.0));
engine.GetValue("double").Dump(); // JsNumber(15)
engine.SetValue("float", JsonValue.Create(15.0f));
engine.GetValue("float").Dump(); // JsUndefined

Expected behavior
It should take over the JsonValue number into the JsValue for all primitive types,
it works already correct for string based json values like JsonValue.Parse("15")

Additional context
Problem seems to be situated here:

  case JsonValueKind.Number:
      result = jsonValue.TryGetValue<double>(out var doubleValue) ? JsNumber.Create(doubleValue) : JsValue.Undefined;
      break;

.TryGetValue<double>() returns false if the JsonValuePrimitive<T>'s T isn't a double.

Suggested fix

  case JsonValueKind.Number:
      var doubleValue = JsonSerializer.Deserialize<double>(jsonValue);   
      result = JsNumber.Create(doubleValue);
      break;

[lahma]

Thanks for the detailed report and repro, fix is shipping in 3.1.1.