Skip to content

.NET: chore: upgrades messagepack to a safe version to unblock CI - #9455

Merged
Vincent Biret (baywet) merged 1 commit into
mainfrom
chore/upgrade-message-pack
Oct 16, 2026
Merged

Vincent Biret (baywet) merged 1 commit into
mainfrom
chore/upgrade-message-pack

Conversation

@baywet

Copy link
Copy Markdown
Member

Copilot AI balanced review requested due to automatic review settings October 16, 2026 11:34
@agent-framework-automation agent-framework-automation Bot added the .NET Usage: [Issues, PRs], Target: .Net label Oct 16, 2026
@github-actions github-actions Bot changed the title chore: upgrades messagepack to a safe version to unblock CI .NET: chore: upgrades messagepack to a safe version to unblock CI Oct 16, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

The narrowly scoped patch upgrade has no identified blocking issues.

0 open findings

What changed in this PR

Updates the centrally pinned .NET MessagePack dependency to address the vulnerability blocking CI.

Changes:

  • Upgrades MessagePack from 3.1.7 to 3.1.12 without changing application code.
File Description
dotnet/​Directory.Packages.props Updates the MessagePack version pin.

🧠 Review effort: Balanced


💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

MAF Automated Review — Iteration 1

Result: No findings
Scope: full PR (1 commit(s)): bf78ac64c969
Model: gpt-5.6-sol

Overview

This PR updates the centrally managed MessagePack transitive pin used by Aspire dependency graphs from 3.1.7 to 3.1.12 without changing application APIs or serialization code. Central transitive pinning, representative Aspire consumers and tests in the main solution, warnings-as-errors builds, and transitive NuGet auditing provide concrete guardrails. No publishable regression was established.

Reviewed the supplied pull-request change set across correctness, security/reliability, architecture, and failure behavior.
No publishable findings remained after source verification for this scope.

@github-code-quality

Copy link
Copy Markdown
Contributor

Code coverage overview

Languages: C#

C# / code-coverage/dotnet

main deecbcb chore/upgrade-messag... bf78ac6 +/-
86% 86% 0%

ℹ️ Calculations represent line coverage. Results may change as calculations are updated.

@baywet
Vincent Biret (baywet) added this pull request to the merge queue Oct 16, 2026
Merged via the queue into main with commit c2d2930 Oct 16, 2026
41 checks passed
@baywet
Vincent Biret (baywet) deleted the chore/upgrade-message-pack branch October 16, 2026 12:29

This branch was previously deployed

2 inactive deployments
integration — bf78ac64 Deployed Oct 16, 2026 by baywet via dotnet-foundry-hosted-it #30550
github-app-auth — bf78ac64 Deployed Oct 16, 2026 by baywet via team_check #6941
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

.NET Usage: [Issues, PRs], Target: .Net

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants