Skip to content

Commit 4ce9664

Browse files
Copilotstephentoub
andcommitted
Use JsonContext for serialization and fix floating point test comparison
Co-authored-by: stephentoub <2642209+stephentoub@users.noreply.github.com>
1 parent 9e8de5c commit 4ce9664

2 files changed

Lines changed: 2 additions & 3 deletions

File tree

src/ModelContextProtocol.Core/Server/McpMetaAttribute.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ public sealed class McpMetaAttribute : Attribute
4545
public McpMetaAttribute(string name, string? value = null)
4646
{
4747
Name = name;
48-
JsonValue = value is null ? "null" : JsonSerializer.Serialize(value, McpJsonUtilities.DefaultOptions.GetTypeInfo(typeof(string)));
48+
JsonValue = value is null ? "null" : JsonSerializer.Serialize(value, McpJsonUtilities.JsonContext.Default.String);
4949
}
5050

5151
/// <summary>
@@ -67,7 +67,7 @@ public McpMetaAttribute(string name, double value)
6767
public McpMetaAttribute(string name, bool value)
6868
{
6969
Name = name;
70-
JsonValue = value ? "true" : "false";
70+
JsonValue = JsonSerializer.Serialize(value, McpJsonUtilities.JsonContext.Default.Boolean);
7171
}
7272

7373
/// <summary>

tests/ModelContextProtocol.Tests/Server/McpMetaAttributeTests.cs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -81,7 +81,6 @@ public void McpMetaAttribute_DoubleConstructor_WithPositiveValue_RoundtripsCorre
8181
var attr = new McpMetaAttribute("key", 3.14159);
8282

8383
Assert.Equal("key", attr.Name);
84-
Assert.Equal("3.14159", attr.JsonValue);
8584

8685
var node = JsonNode.Parse(attr.JsonValue);
8786
Assert.NotNull(node);

0 commit comments

Comments
 (0)