Enable servers to log to the clients via ILogger

[stephentoub]

Fixes #59
Replaces #71

cc: @Redth

[Copilot]

Copilot reviewed 8 out of 8 changed files in this pull request and generated 1 comment.

Comments suppressed due to low confidence (1)

tests/ModelContextProtocol.Tests/Client/McpClientExtensionsTests.cs:414

• The list 'data' is incorrectly initialized; please use 'new List()' to create a new list instance.

List<string> data = [];

[Meir017]

Copilot reviewed 8 out of 8 changed files in this pull request and generated 1 comment.

Comments suppressed due to low confidence (1)
tests/ModelContextProtocol.Tests/Client/McpClientExtensionsTests.cs:414

• The list 'data' is incorrectly initialized; please use 'new List()' to create a new list instance.

List<string> data = [];

I wonder about this one - Copilot doesn't seem to know about collection expressions

[stephentoub]

Copilot reviewed 8 out of 8 changed files in this pull request and generated 1 comment.
Comments suppressed due to low confidence (1)
tests/ModelContextProtocol.Tests/Client/McpClientExtensionsTests.cs:414

• The list 'data' is incorrectly initialized; please use 'new List()' to create a new list instance.

List<string> data = [];

I wonder about this one - Copilot doesn't seem to know about collection expressions

It's a known issue with copilot; the model was trained prior to C# 12.

[Meir017]

Copilot reviewed 8 out of 8 changed files in this pull request and generated 1 comment.
Comments suppressed due to low confidence (1)
tests/ModelContextProtocol.Tests/Client/McpClientExtensionsTests.cs:414

• The list 'data' is incorrectly initialized; please use 'new List()' to create a new list instance.

List<string> data = [];

I wonder about this one - Copilot doesn't seem to know about collection expressions

It's a known issue with copilot; the model was trained prior to C# 12.

hopefully the next model will leverage the source code under github.com/dotnet and github.com/azure to learn about new features :)