Skip to content

[NSJ -> STJ]Migrate ServiceIndexResource deserialization#7297

Merged
Nigusu-Allehu merged 14 commits into
devfrom
dev-nyenework-serevice-deserialization-stj
May 6, 2026
Merged

[NSJ -> STJ]Migrate ServiceIndexResource deserialization#7297
Nigusu-Allehu merged 14 commits into
devfrom
dev-nyenework-serevice-deserialization-stj

Conversation

@Nigusu-Allehu
Copy link
Copy Markdown
Member

@Nigusu-Allehu Nigusu-Allehu commented Apr 17, 2026

Bug

Fixes: NuGet/Home#14886
Related: NuGet/Home#14846

Description

Adds ServiceIndexModel POCOs and migrates ConsumeServiceIndexStreamAsync to support STJ source generation. The existing JObject constructor and callers are unchanged.

PR Checklist

  • Meaningful title, helpful description and a linked NuGet/Home issue
  • Added tests
  • Link to an issue or pull request to update docs if this PR changes settings, environment variables, new feature, etc.

Comment thread src/NuGet.Core/NuGet.Protocol/Providers/ServiceIndexResourceV3Provider.cs Outdated
Comment thread src/NuGet.Core/NuGet.Protocol/Resources/ServiceIndexResourceV3.cs Outdated
Comment thread src/NuGet.Core/NuGet.Protocol/Resources/ServiceIndexResourceV3.cs Outdated
Comment thread src/NuGet.Core/NuGet.Protocol/Resources/ServiceIndexResourceV3.cs
Comment thread src/NuGet.Core/NuGet.Protocol/Resources/ServiceIndexResourceV3.cs
Comment thread src/NuGet.Core/NuGet.Protocol/Resources/ServiceIndexResourceV3.cs Outdated
@Nigusu-Allehu Nigusu-Allehu self-assigned this Apr 18, 2026
@Nigusu-Allehu Nigusu-Allehu force-pushed the dev-nyenework-serevice-deserialization-stj branch from eec8cde to 0bd78f3 Compare April 20, 2026 16:35
@dotnet-policy-service dotnet-policy-service Bot added the Status:No recent activity PRs that have not had any recent activity and will be closed if the label is not removed label Apr 27, 2026
@Nigusu-Allehu Nigusu-Allehu force-pushed the dev-nyenework-serevice-deserialization-stj branch from 0bd78f3 to ec0c0f7 Compare April 28, 2026 14:54
@Nigusu-Allehu Nigusu-Allehu changed the base branch from dev-feature-nsj-stj-migration to dev April 28, 2026 14:55
@dotnet-policy-service dotnet-policy-service Bot removed the Status:No recent activity PRs that have not had any recent activity and will be closed if the label is not removed label Apr 28, 2026
@Nigusu-Allehu Nigusu-Allehu marked this pull request as ready for review April 28, 2026 18:38
@Nigusu-Allehu Nigusu-Allehu requested a review from a team as a code owner April 28, 2026 18:38
Comment thread src/NuGet.Core/NuGet.Protocol/Model/ServiceIndexModel.cs Outdated
Comment thread src/NuGet.Core/NuGet.Protocol/Model/ServiceIndexModel.cs Outdated
Comment thread src/NuGet.Core/NuGet.Protocol/Resources/ServiceIndexResourceV3.cs
@Nigusu-Allehu Nigusu-Allehu requested a review from nkolev92 April 28, 2026 23:36
Comment thread src/NuGet.Core/NuGet.Protocol/Model/ServiceIndexModel.cs
Comment thread src/NuGet.Core/NuGet.Protocol/Model/ServiceIndexModel.cs
Comment thread src/NuGet.Core/NuGet.Protocol/Converters/StringOrStringArrayConverter.cs Outdated
Comment thread src/NuGet.Core/NuGet.Protocol/Model/ServiceIndexModel.cs
Comment thread test/NuGet.Core.Tests/NuGet.Protocol.Tests/ServiceIndexResourceV3Tests.cs Outdated
@Nigusu-Allehu Nigusu-Allehu requested a review from zivkan April 30, 2026 06:22
Copy link
Copy Markdown
Member

@nkolev92 nkolev92 left a comment

Choose a reason for hiding this comment

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

I think this is good now.
I have 2 comments, 1 should be addressed, the other one is an idea.

Comment thread src/NuGet.Core/NuGet.Protocol/Resources/ServiceIndexResourceV3.cs
Comment thread src/NuGet.Core/NuGet.Protocol/Resources/ServiceIndexResourceV3.cs
@Nigusu-Allehu Nigusu-Allehu requested a review from nkolev92 May 1, 2026 00:12
nkolev92
nkolev92 previously approved these changes May 1, 2026
jeffkl
jeffkl previously approved these changes May 5, 2026
@Nigusu-Allehu Nigusu-Allehu dismissed stale reviews from jeffkl and nkolev92 via 9b74364 May 5, 2026 17:22
@Nigusu-Allehu Nigusu-Allehu requested a review from jeffkl May 5, 2026 17:23
@Nigusu-Allehu Nigusu-Allehu requested a review from jeffkl May 5, 2026 23:53
@Nigusu-Allehu
Copy link
Copy Markdown
Member Author

rebased

@Nigusu-Allehu Nigusu-Allehu force-pushed the dev-nyenework-serevice-deserialization-stj branch from f4bb7c3 to 9cbf288 Compare May 6, 2026 15:24
@Nigusu-Allehu Nigusu-Allehu enabled auto-merge (squash) May 6, 2026 15:32
@Nigusu-Allehu Nigusu-Allehu merged commit 08062f3 into dev May 6, 2026
17 of 18 checks passed
@Nigusu-Allehu Nigusu-Allehu deleted the dev-nyenework-serevice-deserialization-stj branch May 6, 2026 16:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Migrate ServiceIndexResource deserialization to System.Text.Json

5 participants