JsValue::to_json fix integer property keys

[Nikita-str]

This Pull Request fixes #3923

[codecov]

Codecov Report

Attention: Patch coverage is 91.66667% with 1 line in your changes missing coverage. Please review.

Project coverage is 52.79%. Comparing base (6ddc2b4) to head (b0800c4).
Report is 268 commits behind head on main.

Files with missing lines	Patch %	Lines
core/engine/src/value/conversions/serde_json.rs	91.66%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #4011      +/-   ##
==========================================
+ Coverage   47.24%   52.79%   +5.54%     
==========================================
  Files         476      480       +4     
  Lines       46892    46684     -208     
==========================================
+ Hits        22154    24645    +2491     
+ Misses      24738    22039    -2699     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[CrazyboyQCD]

Maybe a bit off, but we could avoid the clone here with extra map:

     .and_then(|x| x.value().map(|v| v.to_json(context)))
{
    Some(val) => val?,
    None => Value::Null,
};

And similar thing can be done here

[Nikita-str]

In the worst case it's still just cloning of a smart pointer so I don't think that it has much overhead costs, but ok, done

[raskad]

Thank you for the fix!