Merge pull request #354 from microsoft/dev/andarno/dropUnnecessaryCode

Drop custom STJ converter that we no longer need

Author: AArnott

src/Microsoft.ServiceHub.Framework/JsonConverterWrapper`1.cs

@@ -4,7 +4,6 @@
 using System.Collections.Immutable;
 using System.Diagnostics;
 using System.Diagnostics.CodeAnalysis;
-using System.Globalization;
 using System.IO.Pipelines;
 using Microsoft.VisualStudio.Threading;
 using Nerdbank.Streams;
@@ -486,12 +485,6 @@ private IJsonRpcMessageFormatter CreateSystemTextJsonFormatter()
 				DictionaryKeyPolicy = null,
 				PropertyNamingPolicy = STJ.JsonNamingPolicy.CamelCase,
 				DefaultIgnoreCondition = STJ.Serialization.JsonIgnoreCondition.WhenWritingNull,
-				Converters =
-				{
-					// These converters are a workaround for https://github.com/dotnet/runtime/issues/85981
-					JsonConverterWrapper<Version>.Instance,
-					JsonConverterWrapper<Uri>.Instance,
-				},
 			},
 		};
 	}
@@ -501,8 +494,6 @@ private IJsonRpcMessageFormatter CreateSystemTextJsonFormatter()
 	/// </summary>
 	public class JsonRpcConnection : RpcConnection
 	{
-		private const string GetOptionalInterfacesMethodName = "__jsonrpc_getOptionalInterfaces";
-
 		private readonly ServiceJsonRpcDescriptor? owner;
 
 		/// <summary>