Skip to content

Allow custom npm registry URL for downloading Copilot from dotnet SDK#463

Merged
SteveSandersonMS merged 3 commits intogithub:mainfrom
nayanshah:custom-npm-registry
Feb 16, 2026
Merged

Allow custom npm registry URL for downloading Copilot from dotnet SDK#463
SteveSandersonMS merged 3 commits intogithub:mainfrom
nayanshah:custom-npm-registry

Conversation

@nayanshah
Copy link
Contributor

Our repos enforce using private feeds and npm registry to fetch packages. But the .NET SDK attempts to download Copilot package from public npm registry.

This change allows customizing the registry url by defined a MSBuild property in the csproj or props file - CopilotNpmRegistryUrl.

Note: an improvement could be attempting to find and parse any existing .npmrc files in any parent directories, but that ends up complicating this target a bit, so skipping for now

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@nayanshah nayanshah requested a review from a team as a code owner February 13, 2026 15:57
Copilot AI review requested due to automatic review settings February 13, 2026 15:57
Copy link
Contributor

Copilot AI left a 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 adds support for customizing the npm registry URL in the .NET SDK to accommodate organizations that use private npm registries. The SDK automatically downloads the Copilot CLI binary from npm during the build process, and this change makes the registry URL configurable via an MSBuild property.

Changes:

  • Added CopilotNpmRegistryUrl MSBuild property with a default value of https://registry.npmjs.org
  • Updated the _CopilotDownloadUrl property to use the configurable registry URL instead of the hardcoded public npm registry

nayanshah and others added 2 commits February 13, 2026 08:02
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@nayanshah
Copy link
Contributor Author

@SteveSandersonMS thoughts?

@SteveSandersonMS
Copy link
Contributor

Excellent - thanks!

@SteveSandersonMS SteveSandersonMS merged commit 273c425 into github:main Feb 16, 2026
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.

2 participants