Remove Microsoft.Extensions.AI.Ollama
#6655
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
There was a problem hiding this comment.
Pull Request Overview
This PR removes the Microsoft.Extensions.AI.Ollama package in favor of the OllamaSharp package, addressing technical debt and consolidating the .NET AI ecosystem around a maintained third-party solution.
- Eliminates deprecated
Microsoft.Extensions.AI.Ollamalibrary and all associated test files - Migrates
OllamaSharp.Integration.Teststo useOllamaSharpdirectly instead of inheriting from removed Ollama test classes - Updates project references to point to the generic integration tests rather than the removed Ollama-specific tests
Reviewed Changes
Copilot reviewed 30 out of 32 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| Microsoft.Extensions.AI.Ollama.Tests/*.cs | Complete removal of unit test files for deprecated library |
| Microsoft.Extensions.AI.Ollama.Tests/*.csproj | Removal of test project file |
| Microsoft.Extensions.AI.OllamaSharp.Integration.Tests/*.cs | Migration to use OllamaSharp directly with new integration tests |
| Microsoft.Extensions.AI.OllamaSharp.Integration.Tests/*.csproj | Updated project reference to core integration tests |
| src/Libraries/Microsoft.Extensions.AI.Ollama/*.cs | Complete removal of deprecated source library |
...crosoft.Extensions.AI.OllamaSharp.Integration.Tests/OllamaSharpChatClientIntegrationTests.cs
Show resolved
Hide resolved
...crosoft.Extensions.AI.OllamaSharp.Integration.Tests/OllamaSharpChatClientIntegrationTests.cs
Show resolved
Hide resolved
MackinnonBuck
commented
Jul 24, 2025
...crosoft.Extensions.AI.OllamaSharp.Integration.Tests/OllamaSharpChatClientIntegrationTests.cs
Show resolved
Hide resolved
jeffhandley
approved these changes
Jul 24, 2025
This was referenced Aug 13, 2025
Merged
Closed
This was referenced Aug 20, 2025
joakimia
pushed a commit
to heimdallpower/api-sdk
that referenced
this pull request
Aug 21, 2025
Updated [Microsoft.Extensions.Http.Resilience](https://github.com/dotnet/extensions) from 9.7.0 to 9.8.0. <details> <summary>Release notes</summary> _Sourced from [Microsoft.Extensions.Http.Resilience's releases](https://github.com/dotnet/extensions/releases)._ ## 9.8.0 ## What's Changed * Align EventId generation with M.E.Logging source-gen by @xakep139 in dotnet/extensions#6566 * Add resiliency mechanism to CPU and memory utilization checks by @amadeuszl in dotnet/extensions#6528 * Suppress flaky test until fixed by @joelverhagen in dotnet/extensions#6568 * Create a project template for an MCP server by @joelverhagen in dotnet/extensions#6547 * Use dnx instead of dotnet tool exec in MCP server template README by @joelverhagen in dotnet/extensions#6571 * Add reporting tests that show NLP results. by @peterwald in dotnet/extensions#6574 * Branding updates for 9.8.0 by @joperezr in dotnet/extensions#6573 * Fix `ConfigureEvaluationTests.ps1` script when `$ConfigRoot` is not supplied by @peterwald in dotnet/extensions#6575 * Refactor Resource Monitoring by @evgenyfedorov2 in dotnet/extensions#6554 * Update McpServer template for 0.3.0-preview.2 by @stephentoub in dotnet/extensions#6578 * Add container.cpu.time metric by @evgenyfedorov2 in dotnet/extensions#5806 * Add netstandard2.0 compatibility to Microsoft.Extensions.Diagnostics.Testing and dependencies by @bdovaz in dotnet/extensions#6219 * Add netstandard2.0 compatibility to Microsoft.Extensions.Telemetry and dependencies by @bdovaz in dotnet/extensions#6218 * Simplify Http.Diagnostics by @pentp in dotnet/extensions#6174 * Add netstandard2.0 compatibility to Microsoft.Extensions.Http.Resilience and dependencies by @bdovaz in dotnet/extensions#6582 * Ingore null loggers returned by LogProviders in ELoggerFactory by @petrroll in dotnet/extensions#6585 * Merging changes from internal by @joperezr in dotnet/extensions#6588 * [Logging] Fixes LogProperties and LogPropertyIgnore attributes to work if an object being logged resides in a different assembly than the logging method by @iliar-turdushev in dotnet/extensions#6600 * Add memory usage metric by @evgenyfedorov2 in dotnet/extensions#6586 * Add schema version to server.json in MCP template by @joelverhagen in dotnet/extensions#6606 * Merge release/9.7 into main by @joperezr in dotnet/extensions#6589 * Update MCP server template readme to show both VS Code and Visual Studio notes by @Copilot in dotnet/extensions#6591 * Update Azure Open AI package referenced by eval integration tests by @shyamnamboodiripad in dotnet/extensions#6609 * Target .NET 8 for more stable runtime requirement by @joelverhagen in dotnet/extensions#6617 * Add support for new Azure AI Foundry project type for Safety evals by @shyamnamboodiripad in dotnet/extensions#6621 * M.E.AI.AzureAIInference - Azure.AI.Inference Package Bump by @rogerbarreto in dotnet/extensions#6624 * Add DataContent.Name property by @stephentoub in dotnet/extensions#6616 * Fix handling of multiple responses messages by @stephentoub in dotnet/extensions#6627 * Expose additional chat model conversion helpers by @stephentoub in dotnet/extensions#6629 * [main] Update dependencies from dotnet/arcade by @dotnet-maestro[bot] in dotnet/extensions#6633 * [Http.Resilience] The DisableFor method should try to retrieve the request object from the resilience context by @iliar-turdushev in dotnet/extensions#6618 * Bump Package validation baseline version to 9.7.0 by @Copilot in dotnet/extensions#6650 * Fix internal package version detection for project templates by @MackinnonBuck in dotnet/extensions#6651 * Remove `Microsoft.Extensions.AI.Ollama` by @MackinnonBuck in dotnet/extensions#6655 * Fix M.E.AI package refs by @stephentoub in dotnet/extensions#6654 * Add [Description] to DataContent.Uri by @stephentoub in dotnet/extensions#6615 * Fix duplicate solution file when creating an AI Chat Web app from VS by @MackinnonBuck in dotnet/extensions#6653 * Add ChatMessage.CreatedAt by @stephentoub in dotnet/extensions#6657 * Add TextContent.Annotations by @stephentoub in dotnet/extensions#6619 * Add FunctionInvokingChatClient.AdditionalTools by @stephentoub in dotnet/extensions#6661 * Fix unintentional test env var change by @stephentoub in dotnet/extensions#6660 * Add more openai conversion helpers by @stephentoub in dotnet/extensions#6662 * Add OriginalRepoCommitHash to assemblies by @BrennanConroy in dotnet/extensions#6667 * Include a trivial items keyword if missing. by @eiriktsarpalis in dotnet/extensions#6669 * Add resolution of function parameter level data annotation attributes. by @eiriktsarpalis in dotnet/extensions#6671 * Fix issue with NetSourceIndexStage1 for dependency conflict versions by @joperezr in dotnet/extensions#6672 * Expose streaming conversion utility methods by @stephentoub in dotnet/extensions#6636 * Couple of fixes for MEAI.Evaluation by @shyamnamboodiripad in dotnet/extensions#6673 * Fix one more version conflict on the docs transport package by @joperezr in dotnet/extensions#6675 ... (truncated) ## 9.7.2 ## Packages Released * [Microsoft.Extensions.AI.Templates 9.7.2-preview.3.25366.2](https://www.nuget.org/packages/Microsoft.Extensions.AI.Templates/9.7.2-preview.3.25366.2) ## What's Changed * Target .NET 8 for more stable runtime requirement by @joelverhagen in #6617 **Full Changelog**: dotnet/extensions@v9.7.1...v9.7.2 ## 9.7.1 ## Packages Released * [Microsoft.Extensions.AI 9.7.1](https://www.nuget.org/packages/Microsoft.Extensions.AI/9.7.1) * [Microsoft.Extensions.AI.Abstractions 9.7.1](https://www.nuget.org/packages/Microsoft.Extensions.AI.Abstractions/9.7.1) * [Microsoft.Extensions.AI.OpenAI 9.7.1-preview.1.25365.4](https://www.nuget.org/packages/Microsoft.Extensions.AI.OpenAI/9.7.1-preview.1.25365.4) * [Microsoft.Extensions.AI.AzureAIInference 9.7.1-preview.1.25365.4](https://www.nuget.org/packages/Microsoft.Extensions.AI.AzureAIInference/9.7.1-preview.1.25365.4) * [Microsoft.Extensions.AI.Templates 9.7.1-preview.3.25365.4](https://www.nuget.org/packages/Microsoft.Extensions.AI.Templates/9.7.1-preview.3.25365.4) ## What's Changed * Merging changes from internal by @joperezr in #6588 * Bump FunctionInvokingChatClient.MaximumIterationsPerRequest from 10 to 40 by @stephentoub in #6599 * Expose M.E.AI.OpenAI input message conversions by @stephentoub in #6601 * Add schema version to server.json in MCP template by @joelverhagen in #6606 * Update MCP server template readme to show both VS Code and Visual Studio notes by @jeffhandley in #6591 * Fix schema generation for Nullable<T> function parameters. by @eiriktsarpalis in #6596 * Branding updates for 9.7.1 by @joperezr in dotnet/extensions#6611 * Add DelegatingAIFunction by @stephentoub in dotnet/extensions#6565 * Add FunctionInvokingChatClient.FunctionInvoker delegate by @stephentoub in dotnet/extensions#6564 * Enable specifying "strict" for OpenAI clients via ChatOptions by @stephentoub in dotnet/extensions#6552 * AIFunctionFactory: tolerate JSON string function parameters. by @eiriktsarpalis in dotnet/extensions#6572 * AIFunctionFactory: add test coverage for JSON comments. by @eiriktsarpalis in dotnet/extensions#6576 * Update M.E.AI.OpenAI for latest OpenAI release by @stephentoub in dotnet/extensions#6577 * Update OpenTelemetry semantic conventions version from 1.35 to 1.36 by @Copilot in dotnet/extensions#6579 * AIFunctionFactory: add a flag for disabling return schema generation. by @eiriktsarpalis in dotnet/extensions#6551 * Bump FunctionInvokingChatClient.MaximumIterationsPerRequest from 10 to 40 by @stephentoub in dotnet/extensions#6599 * Expose M.E.AI.OpenAI input message conversions by @stephentoub in dotnet/extensions#6601 * Fix schema generation for Nullable<T> function parameters. by @eiriktsarpalis in dotnet/extensions#6596 **Full Changelog**: dotnet/extensions@v9.7.0...v9.7.1 Commits viewable in [compare view](dotnet/extensions@v9.7.0...v9.8.0). </details> [](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores) Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting `@dependabot rebase`. [//]: # (dependabot-automerge-start) [//]: # (dependabot-automerge-end) --- <details> <summary>Dependabot commands and options</summary> <br /> You can trigger Dependabot actions by commenting on this PR: - `@dependabot rebase` will rebase this PR - `@dependabot recreate` will recreate this PR, overwriting any edits that have been made to it - `@dependabot merge` will merge this PR after your CI passes on it - `@dependabot squash and merge` will squash and merge this PR after your CI passes on it - `@dependabot cancel merge` will cancel a previously requested merge and block automerging - `@dependabot reopen` will reopen this PR if it is closed - `@dependabot close` will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually - `@dependabot show <dependency name> ignore conditions` will show all of the ignore conditions of the specified dependency - `@dependabot ignore this major version` will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself) - `@dependabot ignore this minor version` will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself) - `@dependabot ignore this dependency` will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself) </details> Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
This was referenced Aug 22, 2025
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR removes
Microsoft.Extensions.AI.Ollamain favor ofOllamaSharp.Fixes #6631
Microsoft Reviewers: Open in CodeFlow