CONFIG: Update NuGet dependencies to latest stable (Component Versions dashboard green) - #562
CONFIG: Update NuGet dependencies to latest stable (Component Versions dashboard green)#562perirrs wants to merge 4 commits into
Conversation
…te form (child <Version> element defeats the Component Versions dashboard parser - version unchanged at 10.0.10)
…patible with the .NET 10 AspNetCore.OpenApi source generator (dependabot #560 Build failure); bracket pin + dependabot ignore so the dashboard treats it as intentionally held. Direct ref must stay to keep the NU1903-vulnerable transitive 2.4.1 out
…2.0.0.1 and adapt to the 2.x API (SendSmsAsync mobileNumber param, SendLetterAsync recipient/address params, GovUkNotifyProvider rename - coupled, bump alone fails CS7036/CS0246 per dependabot #559)
…HelperBroker internally to the 3.x Stream/IAsyncEnumerable API (broker interface unchanged - services and tests unaffected; coupled, bump alone fails to compile - the break that forced the ISL.Providers.ReIdentification revert)
|
Status: all checks green EXCEPT Build - acceptance suites only, needs a pair of eyesWhat passes: full solution build, ISL.ReIdentification.Core.Tests.Unit 964/964 (including the #561 hex tests over the CSV path), CodeQL/Sonar/Linting/prLinter. What fails (reproduced on a rerun, so not flake): Configurations AuditsApiTests x5 + Portals ReIdentificationApiTests x9. Symptom is identical everywhere: the API returns audits stamped with the authenticated test user (65b5ccfb-...) and server-side timestamps, while the tests expect their seeded CreatedBy/UpdatedBy/dates (e.g. 2c00cf04-...). main is green at the same base commit (#561 merge). Why this is odd: none of the four commits here touches audits, auth, or the coordination service - the unit suite is untouched and green. Chief suspect is a transitive shift: ISL.Providers.Notifications.GovukNotify 2.0.0.1 pulls newer GovukNotify/Microsoft.IdentityModel dependencies than 0.0.0.3, which can change claims/user-resolution behaviour in the test hosts. The CsvHelperClient 3.1.0 broker adaptation is interface-preserving and covered green by the unit + hex tests. Suggested next step: run the Configurations acceptance suite locally against this branch with a breakpoint on SecurityBroker.GetCurrentUserAsync / ApplyAddAuditAsync to see which claim the EntraUserId resolves from, and compare resolved Microsoft.IdentityModel.* versions vs main (dotnet list package --include-transitive diff). Happy to split the GovukNotify commit out to its own PR if that isolates it. |



Summary
Clears all 15 "behind" references on the Component Versions dashboard for ISL.ReIdentification. Branched on top of #561. Supersedes dependabot #559/#560 (both fail Build by design - see below; they auto-close/should be closed on merge) and #558 stays closed per the FluentAssertions policy.
closes AB#29743
(Follow-up under the same repo task; AZDO closure is handled manually.)
Changes (one per commit, each build-gated)
Validation (local)