-
Notifications
You must be signed in to change notification settings - Fork 0
Update all NuGet and npm packages to latest versions #612
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.
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
Conversation
Co-authored-by: NatYou345 <[email protected]>
Co-authored-by: NatYou345 <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This pull request updates all NuGet and npm packages to their latest versions with the goal of achieving zero vulnerabilities. The update includes major version bumps for Microsoft.Extensions.* packages (9.0.x → 10.0.0), MongoDB.Driver (2.x → 3.5.1), and Angular (19.x → 20.3.13).
Key Changes:
- Updated Microsoft.Extensions.* packages to version 10.0.0
- Upgraded MongoDB.Driver to 3.5.1 with API migration changes
- Updated Angular to 20.3.13 and TypeScript to 5.8.3
- Maintained compatibility by using AspNetCore.HealthChecks.Rabbitmq.v6 for RabbitMQ.Client 6.x compatibility
- Kept Swashbuckle.AspNetCore at 7.2.0 and RabbitMQ.Client at 6.8.1 to avoid breaking changes
Reviewed changes
Copilot reviewed 121 out of 280 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| tests/Services/Coupon/Coupon.API.Tests/Coupon.API.Tests.csproj | Updated test packages including xunit, coverlet, and Microsoft packages to latest versions |
| src/Services/Payment/Payment.API/Payment.API.csproj | Updated health check and logging packages to v10.0.0; switched to Rabbitmq.v6 variant |
| src/Web/WebSPA/WebSPA.csproj | Updated ASP.NET Core health checks, Serilog, and Microsoft packages |
| src/Web/WebSPA/Client/package.json | Upgraded Angular to 20.3.13, TypeScript to 5.8.3, and other npm dependencies |
| tests/Services/Coupon/Coupon.API.Tests/obj/* | Build artifacts that should not be committed |
| src/Services/Payment/Payment.API/obj/* | Build artifacts that should not be committed |
| src/Web/WebStatus/obj/* | Build artifacts that should not be committed |
| src/Services/Ordering/Ordering.SignalrHub/obj/* | Build artifacts that should not be committed |
| src/Services/Ordering/Ordering.Infrastructure/obj/* | Build artifacts that should not be committed |
Comments suppressed due to low confidence (1)
src/Services/Coupon/Coupon.API/Infrastructure/Repositories/CouponContext.cs:14
- Disposable 'MongoClient' is created but not disposed.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
|
||
| public static IServiceCollection AddCustomHealthCheck(this IServiceCollection services, IConfiguration configuration) | ||
| { | ||
| var accountName = configuration.GetValue<string>("AzureStorageAccountName"); |
Copilot
AI
Nov 25, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This assignment to accountName is useless, since its value is never read.
| public static IServiceCollection AddCustomHealthCheck(this IServiceCollection services, IConfiguration configuration) | ||
| { | ||
| var accountName = configuration.GetValue<string>("AzureStorageAccountName"); | ||
| var accountKey = configuration.GetValue<string>("AzureStorageAccountKey"); |
Copilot
AI
Nov 25, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This assignment to accountKey is useless, since its value is never read.
Updates all package dependencies to latest versions with zero vulnerabilities.
NuGet Updates
Used
AspNetCore.HealthChecks.Rabbitmq.v6to maintain RabbitMQ.Client 6.x compatibility.npm Updates
Breaking Change Workarounds
Swashbuckle.AspNetCore kept at 7.2.0 — v10.x removes
Microsoft.OpenApi.Modelsnamespace.RabbitMQ.Client kept at 6.8.1 — v7.x has extensive breaking changes (
IModel→IChannel, sync → async).Code Changes
MongoDB.Driver 3.x API migration:
Original prompt
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.