Skip to content

docs(contributing): note npm CFS proxy for Microsoft-managed devices#638

Merged
corinagum merged 2 commits into
mainfrom
cg/npm-registry-docs
Jul 14, 2026
Merged

docs(contributing): note npm CFS proxy for Microsoft-managed devices#638
corinagum merged 2 commits into
mainfrom
cg/npm-registry-docs

Conversation

@corinagum

Copy link
Copy Markdown
Collaborator

What

Adds an "npm Registry (Microsoft-managed devices)" section to CONTRIBUTING.MD.

Why

Direct access to registry.npmjs.org is now blocked on Microsoft-managed devices; npm/pnpm/yarn must resolve through the org's Central Feed Services (CFS) proxy. Contributors on managed devices hitting install failures had no guidance in-repo.

Notes

  • Points to aka.ms/CFS rather than hardcoding an internal feed URL — per CFS guidance, onboarding config stays private to the build and must not be committed to a public repo.
  • Clarifies external contributors are unaffected (public npm still works for them).
  • Distinguishes the device-level reroute (this note) from repo-level CFS onboarding (handled in the build pipeline, tracked separately).

Copilot AI review requested due to automatic review settings July 10, 2026 21:45

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

Adds contributor-facing documentation to help developers on Microsoft-managed devices successfully install npm dependencies when direct access to registry.npmjs.org is blocked, by pointing them to Central Feed Services (CFS) guidance.

Changes:

  • Adds a new “npm Registry (Microsoft-managed devices)” section to clarify managed-device behavior vs. external contributors.
  • Documents how to verify the current npm registry configuration and where to find CFS setup instructions.
  • Notes the distinction between device-level rerouting and repo-level build pipeline onboarding.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread CONTRIBUTING.MD Outdated
Corina Gum and others added 2 commits July 10, 2026 15:22
Direct access to registry.npmjs.org is blocked on Microsoft-managed
devices. Add guidance pointing contributors to the CFS proxy setup at
aka.ms/CFS, while clarifying external contributors are unaffected and
that repo-level CFS onboarding is handled in the build pipeline.

Co-authored-by: Copilot <[email protected]>
Copilot-Session: 75223766-eb7f-4146-9d06-2c4e1e22f42e
Co-authored-by: Copilot <[email protected]>
Copilot-Session: 75223766-eb7f-4146-9d06-2c4e1e22f42e
@corinagum
corinagum force-pushed the cg/npm-registry-docs branch from 1e118d7 to 7623e79 Compare July 10, 2026 22:23
@corinagum
corinagum added this pull request to the merge queue Jul 14, 2026
Merged via the queue into main with commit 2a03281 Jul 14, 2026
6 checks passed
@corinagum
corinagum deleted the cg/npm-registry-docs branch July 14, 2026 23:25
@lilyydu lilyydu mentioned this pull request Jul 16, 2026
lilyydu added a commit that referenced this pull request Jul 16, 2026
## Release v2.0.14

Prepares the stable **2.0.14** release by merging `main` into `release`
and setting `version.json` to the stable version (removing the
`-preview.{height}` suffix).

- Previous release: `2.0.13` (published 2026-06-15)
- Only functional diff vs `main` is `version.json`:
`2.0.14-preview.{height}` → `2.0.14`

## What's in this release

### 🚀 Features
* Flatten client method chains to reduce call hops by @lilyydu in
#634
* Support resetting response streams and streaming error handling by
@lilyydu in #635
* Add `extendedMarkdown` text format value by @singhk97 in
#615

### 🐛 Fixes
* Set `replyToId` for all streaming activities by @lilyydu in
#646
* Use `artifactName` for pipelineArtifact inputs in publish pipeline by
@corinagum in #625

### ♻️ Refactors
* Move `apps` from file-based mixins to real classes by @heyitsaamir in
#633
* Split up Outbound and Inbound Activities by @heyitsaamir in
#639
* Remove `ExperimentalTeamsQuotedReplies` markers by @corinagum in
#621

### 🧪 Tests
* Add integration tests for Teams TypeScript SDK by @corinagum in
#622

### 📚 Docs & Examples
* Migrate examples to the flattened conversation API by @lilyydu in
#637
* Note npm CFS proxy for Microsoft-managed devices by @corinagum in
#638
* Update integration test runbook link to ADO wiki by @corinagum in
#624

### 🔧 Chores & Dependencies
* Bump version to 2.0.14-preview and fix pipeline release job by
@corinagum in #620
* Bump hono from 4.12.21 to 4.12.25 by @dependabot in
#623
* Bump react-router from 7.15.0 to 7.15.1 by @dependabot in
#619
* Bump vite from 6.4.2 to 6.4.3 by @dependabot in
#617

**Full Changelog**:
v2.0.13...prep-release/2.0.14

### After merge
1. Trigger the [release
pipeline](https://dev.azure.com/DomoreexpGithub/Github_Pipelines/_build?definitionId=52&_a=summary)
on `release` with **Public** publish type
2. Bump `main` → `2.0.15-preview.{height}`
3. Create the `v2.0.14` git tag + GitHub Release

Co-authored-by: Copilot <[email protected]>
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.

3 participants