SergeyMenshykh
SergeyMenshykh
> We should not create another DI just because we dislike what .NET offers, or because we want to isolate SK from the rest of the applications where it is...
_This change appears to add approximately 1K LOC without introducing new functionality. Is this the right approach?_ > The larges part of the change is code refactoring - changing code...
Closing the issue since all SK components have been modified to work with either external instance of HttpClient or the internal one, which is created by default if no external...
@stephentoub , we have the [JsonOptionsCache](https://github.com/microsoft/semantic-kernel/blob/main/dotnet/src/InternalUtilities/src/Text/JsonOptionsCache.cs) cache, which is used by many SK components. I think it should be removed in favor of the `Kernel.SerializerOptions` property. The configuration of the...
Hi @AleRoe, your thinking may be right, and the `parsedDescriptionUrl` should be used as a base URL - `BaseUrl = parsedDescriptionUrl`. CC: @baywet, @andrueastman, @musale
@AleRoe, please do so, and add a few tests to check the base URL for both remote and local cases as well.
Expect the fix released sometime this week.
Hi @hemantkathuria, it's quite difficult to tell what is going on without having the code. Please create a console app that reproduces this behavior and share it with us so...
Link to the same issue in MCP C# repo - https://github.com/modelcontextprotocol/csharp-sdk/issues/333
Hi @hemantkathuria, was able to fix the issue? If not, try to move MCP client creation one level up to the root of the main method: `await using IMcpClient mcpClient...