Skip to content

[SFI] Bump semantic kernel package and dependencies #227

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
gwharris7 wants to merge 1 commit into
base: main
Choose a base branch
from
+12 −12
Open

[SFI] Bump semantic kernel package and dependencies #227

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
24 changes: 12 additions & 12 deletions src/Directory.Packages.props
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,11 +5,11 @@
</PropertyGroup>
<ItemGroup>
<!-- Azure packages -->
<PackageVersion Include="Azure.Core" Version="1.47.3" />
<PackageVersion Include="Azure.Core" Version="1.50.0" />
<PackageVersion Include="Azure.Identity" Version="1.14.2" />
<PackageVersion Include="Azure.Monitor.Ingestion" Version="1.2.0" />
<PackageVersion Include="Azure.Monitor.OpenTelemetry.Exporter" Version="1.4.0" />
<PackageVersion Include="Azure.AI.OpenAI" Version="2.3.0-beta.2" />
<PackageVersion Include="Azure.AI.OpenAI" Version="2.7.0-beta.2" />

<!-- OpenTelemetry packages -->
<PackageVersion Include="OpenTelemetry" Version="1.11.2" />
Expand All @@ -27,21 +27,21 @@
<PackageVersion Include="Newtonsoft.Json" Version="13.0.3" />
<PackageVersion Include="System.Security.Claims" Version="4.3.0" />
<!-- Semantic Kernel packages -->
<PackageVersion Include="Microsoft.SemanticKernel.Agents.Core" Version="1.65.0" />
<PackageVersion Include="Microsoft.SemanticKernel" Version="1.65.0" />
<PackageVersion Include="Microsoft.SemanticKernel.Abstractions" Version="1.65.0" />
<PackageVersion Include="Microsoft.SemanticKernel.Agents.Core" Version="1.71.0" />
<PackageVersion Include="Microsoft.SemanticKernel" Version="1.71.0" />
<PackageVersion Include="Microsoft.SemanticKernel.Abstractions" Version="1.71.0" />
<!-- OpenAI packages -->
<PackageVersion Include="OpenAI" Version="2.4.0" />
<PackageVersion Include="OpenAI" Version="2.7.0" />
<!-- Azure Foundry related package -->
<PackageVersion Include="Azure.AI.Agents.Persistent" Version="1.2.0-beta.4" />
<!-- Agent Framework packages -->
<PackageVersion Include="Microsoft.Agents.AI" Version="1.0.0-preview.251016.1" />
<PackageVersion Include="Microsoft.Agents.AI.OpenAI" Version="1.0.0-preview.251016.1" />
<PackageVersion Include="Microsoft.Extensions.AI" Version="9.10.0" />
<PackageVersion Include="Microsoft.Extensions.AI" Version="10.2.0" />
<PackageVersion Include="Microsoft.Extensions.AI.AzureAIInference" Version="9.9.0-preview.1.25458.4" />
Copy link

Copilot AI Mar 28, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Microsoft.Extensions.AI was bumped to 10.2.0 and Microsoft.Extensions.AI.OpenAI to 10.0.1-preview., but Microsoft.Extensions.AI.AzureAIInference remains on 9.9.0-preview.. If these packages are used together (as in Tooling.Extensions.AgentFramework), keeping them on different major versions can create restore-time version range conflicts or subtle API mismatches. Consider upgrading AzureAIInference to a 10.x-compatible version (or add a note explaining why it must stay on 9.x).

Suggested change
<PackageVersion Include="Microsoft.Extensions.AI.AzureAIInference" Version="9.9.0-preview.1.25458.4" />
<PackageVersion Include="Microsoft.Extensions.AI.AzureAIInference" Version="10.0.1-preview.1.25571.5" />

Copilot uses AI. Check for mistakes.
<PackageVersion Include="Microsoft.Extensions.AI.OpenAI" Version="9.9.0-preview.1.25458.4" />
<PackageVersion Include="Microsoft.Extensions.AI.OpenAI" Version="10.0.1-preview.1.25571.5" />
<!-- Json packages -->
<PackageVersion Include="System.Text.Json" Version="9.0.2" />
<PackageVersion Include="System.Text.Json" Version="10.0.2" />
Copy link

Copilot AI Mar 28, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

System.Text.Json is pinned to 10.0.2 while the repo targets net8.0 and also has netstandard2.0 projects. Please verify that this System.Text.Json major version is compatible with netstandard2.0 (and all net8.0 projects) and does not introduce restore conflicts with other pinned Microsoft.Extensions.* 9.x dependencies; otherwise consider keeping it on the 9.x line or aligning the rest of the dependency graph to the same major.

Suggested change
<PackageVersion Include="System.Text.Json" Version="10.0.2" />
<PackageVersion Include="System.Text.Json" Version="9.0.0" />

Copilot uses AI. Check for mistakes.
<!-- Source Link Support -->
<PackageVersion Include="Microsoft.SourceLink.GitHub" Version="8.0.0" />
<!-- Versioning -->
Expand All @@ -61,11 +61,11 @@
<PackageVersion Include="Microsoft.CodeAnalysis.CSharp.Workspaces" Version="4.11.0" />
<PackageVersion Include="System.Composition" Version="8.0.0" />
<!-- Microsoft Extensions packages - using highest versions to avoid conflicts -->
<PackageVersion Include="Microsoft.Extensions.DependencyInjection" Version="9.0.1" />
<PackageVersion Include="Microsoft.Extensions.DependencyInjection.Abstractions" Version="9.0.10" />
<PackageVersion Include="Microsoft.Extensions.DependencyInjection" Version="10.0.2" />
<PackageVersion Include="Microsoft.Extensions.DependencyInjection.Abstractions" Version="10.0.2" />
<PackageVersion Include="Microsoft.Extensions.Logging" Version="9.0.1" />
<PackageVersion Include="Microsoft.Extensions.Logging.Console" Version="9.0.1" />
<PackageVersion Include="Microsoft.Extensions.Logging.Abstractions" Version="9.0.10" />
<PackageVersion Include="Microsoft.Extensions.Logging.Abstractions" Version="10.0.2" />
Comment on lines +64 to +68
Copy link

Copilot AI Mar 28, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The Microsoft.Extensions.* versions are now mixed across major versions (DependencyInjection/DependencyInjection.Abstractions/Logging.Abstractions at 10.0.2, but Logging and Logging.Console remain at 9.0.1). With central package management + transitive pinning enabled, this increases the likelihood of NuGet restore conflicts (major-version upper bounds) and/or runtime binding issues. Consider aligning all Microsoft.Extensions.* packages to the same major/minor (e.g., bump Logging/Console/Configuration/Hosting/etc. to 10.x as well, or keep the whole group on 9.x).

Copilot uses AI. Check for mistakes.
<PackageVersion Include="Microsoft.Extensions.Configuration" Version="9.0.1" />
<PackageVersion Include="Microsoft.Extensions.Configuration.Abstractions" Version="9.0.3" />
<PackageVersion Include="Microsoft.Extensions.Hosting" Version="9.0.1" />
Expand Down
Loading