Skip to content

Commit 0bccb94

Browse files
committed
Fix build
1 parent cf45d2d commit 0bccb94

File tree

1 file changed

+5
-2
lines changed

1 file changed

+5
-2
lines changed

src/ModelContextProtocol/Shared/McpSession.cs

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,9 @@
1010
using System.Diagnostics.Metrics;
1111
using System.Text.Json;
1212
using System.Text.Json.Nodes;
13+
#if !NET
14+
using System.Threading.Channels;
15+
#endif
1316

1417
namespace ModelContextProtocol.Shared;
1518

@@ -515,7 +518,7 @@ private void AddTags(ref TagList tags, Activity? activity, IJsonRpcMessage messa
515518
tags.Add("mcp.method.name", method);
516519
tags.Add("network.transport", _transportKind);
517520

518-
// TODO (lmolkova), when using SSE transport, add:
521+
// TODO: When using SSE transport, add:
519522
// - server.address and server.port on client spans and metrics
520523
// - client.address and client.port on server spans (not metrics because of cardinality) when using SSE transport
521524
if (activity is { IsAllDataRequested: true })
@@ -646,7 +649,7 @@ public void Dispose()
646649
TagList tags = default;
647650
tags.Add("network.transport", _transportKind);
648651

649-
// TODO (lmolkova): add server.address and server.port on client-side when using SSE transport,
652+
// TODO: Add server.address and server.port on client-side when using SSE transport,
650653
// client.* attributes are not added to metrics because of cardinality
651654
durationMetric.Record(GetElapsed(_sessionStartingTimestamp).TotalSeconds, tags);
652655
}

0 commit comments

Comments
 (0)