Skip to content

chore: use msft approved npm feed - #2992

Merged
JasonYeMSFT (JasonYeMSFT) merged 2 commits into
microsoft:mainfrom
JasonYeMSFT:chuye/use-msft-npm-feed
Aug 5, 2026
Merged

chore: use msft approved npm feed#2992
JasonYeMSFT (JasonYeMSFT) merged 2 commits into
microsoft:mainfrom
JasonYeMSFT:chuye/use-msft-npm-feed

Conversation

@JasonYeMSFT

@JasonYeMSFT JasonYeMSFT (JasonYeMSFT) commented Jul 29, 2026

Copy link
Copy Markdown
Member

Description

MSFT policy requires all dev machines to use a screened package feed to protect people from supply chain attacks. This PR sets the repo level .npmrc to use this screened feed so everyone working in this repo get the same view of the available packages.

Checklist

  • Tests pass locally (cd tests && npm test)
  • Title has one of the prefixes: fix:, feat:, feature:, chore:, misc:, test:, eval:
  • If modifying skill descriptions: verified routing correctness with integration tests (In tests/, npm run test:integration -- <skill> or npm run test:vally -- --skill <skill>)

Related Issues

#2984

@JasonYeMSFT
JasonYeMSFT (JasonYeMSFT) requested a review from a team as a code owner July 29, 2026 22:06
Copilot AI review requested due to automatic review settings July 29, 2026 22:06

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR aligns npm configuration with Microsoft’s screened package feed policy by setting the npm registry to the approved proxy feed so installs resolve through the same vetted source across the repo.

Changes:

  • Configure the npm registry to https://packagefeedproxy.microsoft.io/npm/ at the repo root.
  • Mirror the same registry setting in the scripts/ and tests/ subproject .npmrc files for consistent behavior when running npm from those directories.
Show a summary per file
File Description
.npmrc Sets the repo-wide npm registry to the Microsoft package feed proxy.
scripts/.npmrc Ensures the scripts workspace uses the same approved npm registry.
tests/.npmrc Ensures the tests workspace uses the same approved npm registry.

Review details

  • Files reviewed: 3/3 changed files
  • Comments generated: 0
  • Review effort level: Low

Comment thread scripts/.npmrc Outdated
@JasonYeMSFT
JasonYeMSFT (JasonYeMSFT) merged commit 01a3c81 into microsoft:main Aug 5, 2026
11 checks passed
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.

4 participants