[dotnet] [bidi] Avoid using JsonInclude attribute to include optional property for DTO - #16413

Description: The new primary-constructor record NodeRemoteValue(string? SharedId, NodeProperties?
Value) makes SharedId and Value optional, which can lead to null dereference or logic
errors if downstream code assumes these were always populated by deserialization when
previously guarded by JsonInclude.
RemoteValue.cs [255-260]

public sealed record NodeRemoteValue(string? SharedId, NodeProperties? Value) : RemoteValue, ISharedReference
{
    public Handle? Handle { get; set; }

    public InternalId? InternalId { get; set; }
}

• Create ticket/issue

Follow the guide to enable codebase context checks.

Follow the guide to enable custom compliance check.

Move the mutable Handle and InternalId properties of the NodeRemoteValue record
to its primary constructor to ensure the object is immutable.

dotnet/src/webdriver/BiDi/Script/RemoteValue.cs [255-260]

-public sealed record NodeRemoteValue(string? SharedId, NodeProperties? Value) : RemoteValue, ISharedReference
-{
-    public Handle? Handle { get; set; }
+public sealed record NodeRemoteValue(string? SharedId, NodeProperties? Value, Handle? Handle, InternalId? InternalId) : RemoteValue, ISharedReference;
 
-    public InternalId? InternalId { get; set; }
-}
-

• Apply / Chat