Microsoft.VisualStudio.Shared.VSCodeDebugProtocol should not parse ISO 8601 compliant strings as date object

[stertingen]

The nuget package's Microsoft.VisualStudio.Shared.VSCodeDebugProtocol project URL lists this repo as its homepage, so that's why I'm reporting the issue here.

The MIEngine, used by the vscode-cpptools extension, changes the to-be-debugged application's CLI args if they contain ISO 8601 strings. I have boiled down this issue of MIEngine using the default behavior of the Newtonsoft.Json library. For whatever reason, it sometimes parses strings as date objects, which may serialize to a different string than originally used.

MIEngine seems to use Microsoft.VisualStudio.Shared.VSCodeDebugProtocol for parsing the JSON document, leaving no room for adjusting the JSON parser's behavior by itself.

I investigated the Microsoft.VisualStudio.Shared.VSCodeDebugProtocol package and came to the conclusion that Microsoft.VisualStudio.Shared.VSCodeDebugProtocol.Protocol.DebugProtocol might needed to be modified to include the DateParseHandling = DateParseHandling.None setting in its jsonSettings.

However, I'm not sure if this is sufficient to fix the issue everywhere because I cannot find the source code to that package to verify it.

Related issues:

• Debugger rewrites date-times being passed through the args array vscode#238514
• Debugging application modifies arguments when passing an ISO 8601 compliant string vscode-cpptools#13241
• Debugging from VS code rewrites args if containing ISO 8601 compliant date/time string MIEngine#1491
• Json.NET interprets and modifies ISO dates when deserializing to JObject JamesNK/Newtonsoft.Json#862