Address PR review comments

Author: mikekistler

src/ModelContextProtocol.Core/Client/McpClient.Methods.cs

@@ -83,9 +83,8 @@ public static async Task<McpClient> ResumeSessionAsync(
     /// </summary>
     /// <param name="options">Optional request options including metadata, serialization settings, and progress tracking.</param>
     /// <param name="cancellationToken">The <see cref="CancellationToken"/> to monitor for cancellation requests. The default is <see cref="CancellationToken.None"/>.</param>
-    /// <returns>A task that completes when the ping is successful.</returns>
+    /// <returns>A task containing the ping result.</returns>
     /// <exception cref="McpException">The server cannot be reached or returned an error response.</exception>
-    /// <exception cref="McpException">Thrown when the server cannot be reached or returns an error response.</exception>
     public ValueTask<PingResult> PingAsync(RequestOptions? options = null, CancellationToken cancellationToken = default)
     {
         return SendRequestAsync(

src/ModelContextProtocol.Core/RequestOptions.cs

@@ -33,8 +33,8 @@ public JsonObject? Meta
             // Add progress token to metadata
             meta["progressToken"] = ProgressToken.Value.Token switch
             {
-                string s => JsonValue.Create(s),
-                long l => JsonValue.Create(l),
+                string s => s,
+                long l => l,
                 _ => null
             };
 

tests/ModelContextProtocol.Tests/Client/McpClientMetaTests.cs

@@ -103,7 +103,7 @@ public async Task ResourceReadWithMetaFields()
 
 
     [Fact]
-    public async Task PromptGettWithMetaFields()
+    public async Task PromptGetWithMetaFields()
     {
         Server.ServerOptions.PromptCollection?.Add(McpServerPrompt.Create(
             (RequestContext<GetPromptRequestParams> context) =>