.NET: chore: updates microsoft extensions and related packages to their latest version, as well as global.json dotnet sdk version to align - #8190
Conversation
There was a problem hiding this comment.
🟡 Changes recommended
A few net10-pinned packages remain at 10.0.0 despite the PR’s “latest” update intent, so the version set looks incomplete/inconsistent for the stated goal.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
Pull request overview
Updates centrally-managed .NET package versions (via dotnet/Directory.Packages.props) to newer Microsoft.* patch/minor releases, intended to unblock Dependabot by moving Microsoft Extensions and related packages forward.
Changes:
- Bumps several .NET 10-related packages from
10.0.11→10.0.12(e.g.,System.Text.Json,System.Threading.Channels,DiagnosticSource, variousMicrosoft.Extensions.*). - Updates some Microsoft.Extensions feature packages to
10.9.0(e.g.,Microsoft.Extensions.Http.Resilience,ServiceDiscovery,VectorData.Abstractions,Compliance.Abstractions). - Updates
Microsoft.Data.SqliteandMicrosoft.Bcl.AsyncInterfacesto10.0.12.
File summaries
| File | Description |
|---|---|
| dotnet/Directory.Packages.props | Central package version bumps for Microsoft.Extensions and related .NET 10 packages. |
Review details
Suppressed comments (1)
dotnet/Directory.Packages.props:55
- System.Net.Http.Json remains pinned to 10.0.0 while other net10 System.* packages were updated to 10.0.12 in this PR; if the goal is “latest” patch levels to unblock dependabot, consider updating this pin too for consistency.
<PackageVersion Include="System.Net.Http.Json" Version="10.0.0" />
- Files reviewed: 1/1 changed files
- Comments generated: 2
- Review effort level: Lite
💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
🟢 Approval recommended
The changes are limited to central package version updates and align with the repo’s .NET 10 SDK usage without introducing any evident repo-local incompatibilities.
Review details
- Files reviewed: 1/1 changed files
- Comments generated: 0 new
- Review effort level: Lite
There was a problem hiding this comment.
MAF Automated Review — Iteration 1
Result: Findings reported
Scope: full PR (1 commit(s)): feb01e0a8fb5
Model: gpt-5.6-sol-fast
Overview
The PR consistently aligns Microsoft runtime packages on 10.0.12 and related Extensions packages on 10.9.0 through the repository's central package manifest. Central transitive pinning, broad multi-target CI, package validation, and dependency auditing provide strong guardrails, and no security-specific regression was identified. However, a pinning-disabled net472 test project resolves the prior System.Text.Json assembly while a referenced project compiles against the new one, causing the supported net472 build to fail.
Reviewed the supplied pull-request change set across correctness, security/reliability, architecture, and failure behavior.
1 verified finding remained after source verification (1 high) across 1 file. Details are attached to the affected lines below.
Affected areas: dotnet/Directory.Packages.props
There was a problem hiding this comment.
🔵 Needs a closer look
Broad dependency + SDK pin updates can have wide impact and should be validated by CI/build results and manual review of any breaking changes.
Review details
- Files reviewed: 3/3 changed files
- Comments generated: 1
- Review effort level: Lite
Code Coverage OverviewLanguages: C# C# / code-coverage/dotnetThe overall line coverage in commit f15d611 in the Show a line coverage summary of the most covered files.
|
Updates all microsoft extensions packages to their latest version in hope of unblocking dependabot