Skip to content

Remove obsolete transitive NuGet overrides from typespec-vs#10824

Open
Copilot wants to merge 5 commits into
mainfrom
copilot/remove-unnecessary-nuget-package
Open

Remove obsolete transitive NuGet overrides from typespec-vs#10824
Copilot wants to merge 5 commits into
mainfrom
copilot/remove-unnecessary-nuget-package

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 28, 2026

The Visual Studio extension was carrying explicit NuGet references that were only needed to suppress older transitive vulnerability and NU1603 issues. Those overrides are no longer necessary with the currently resolved Visual Studio package set.

  • Dependency cleanup

    • Remove explicit PackageReferences for:
      • MessagePack
      • Microsoft.VisualStudio.Composition
      • Microsoft.VisualStudio.Shell.15.0
  • Project file simplification

    • Drop the associated explanatory/TODO comments tied to those temporary overrides.
    • Keep the direct Visual Studio SDK/workspace dependencies unchanged so the extension continues to pick up these packages transitively.
  • Result

    • typespec-vs now relies on the Visual Studio packages to supply these dependencies instead of pinning them redundantly in the project file.
<!-- removed -->
<PackageReference Include="MessagePack" Version="2.5.192" />
<PackageReference Include="Microsoft.VisualStudio.Composition" Version="17.13.41" />
<PackageReference Include="Microsoft.VisualStudio.Shell.15.0" Version="17.14.40264" />

Co-authored-by: timotheeguerin <1031227+timotheeguerin@users.noreply.github.com>
@microsoft-github-policy-service microsoft-github-policy-service Bot added the ide Issues for VS, VSCode, Monaco, etc. label May 28, 2026
Copilot AI changed the title [WIP] Remove unnecessary nuget package in vs-extension Remove obsolete transitive NuGet overrides from typespec-vs May 28, 2026
Copilot AI requested a review from timotheeguerin May 28, 2026 12:36
@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented May 28, 2026

Open in StackBlitz

npm i https://pkg.pr.new/typespec-vs@10824

commit: fc0d818

@timotheeguerin
Copy link
Copy Markdown
Member

@copilot add changelog

Co-authored-by: timotheeguerin <1031227+timotheeguerin@users.noreply.github.com>
Copy link
Copy Markdown
Contributor Author

Copilot AI commented May 28, 2026

@copilot add changelog

Addressed in 4d6e325.

Comment thread packages/typespec-vs/CHANGELOG.md Outdated
Co-authored-by: timotheeguerin <1031227+timotheeguerin@users.noreply.github.com>
@github-actions
Copy link
Copy Markdown
Contributor

All changed packages have been documented.

  • typespec-vs
Show changes

typespec-vs - fix ✏️

Remove obsolete explicit Visual Studio transitive NuGet package overrides.

Comment thread packages/typespec-vs/src/Microsoft.TypeSpec.VS.csproj
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ide Issues for VS, VSCode, Monaco, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove unnecessary nuget package in vs-extension

3 participants