From 6bed2ea30f20927f62ace9da333d4fc5921a36bd Mon Sep 17 00:00:00 2001 From: "github-actions[bot]" Date: Fri, 29 May 2026 22:18:55 +0000 Subject: [PATCH 1/4] fix: configure npm to use Azure Artifacts feed in GenerateReleaseArtifacts job (CFS) Configure npm to use the internal Azure Artifacts npm feed instead of the public registry.npmjs.org in the GenerateReleaseArtifacts job. This resolves network isolation failures when installing moxygen. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --- eng/pipelines/templates/jobs/archetype-sdk-client.yml | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/eng/pipelines/templates/jobs/archetype-sdk-client.yml b/eng/pipelines/templates/jobs/archetype-sdk-client.yml index b6039aad3d..37dcdda3e8 100644 --- a/eng/pipelines/templates/jobs/archetype-sdk-client.yml +++ b/eng/pipelines/templates/jobs/archetype-sdk-client.yml @@ -162,6 +162,10 @@ jobs: -DBUILD_TRANSPORT_CURL=OFF -DBUILD_DOCUMENTATION=YES + - template: /eng/common/pipelines/templates/steps/create-authenticated-npmrc.yml + parameters: + npmrcPath: $(UserProfile)/.npmrc + - pwsh: npm install -g moxygen displayName: Install Moxygen to generate markdown for learn.microsoft.com From b6cefed5595bd50fa2dbd6b6605fc8ae1961bac4 Mon Sep 17 00:00:00 2001 From: Chidozie Ononiwu <31145988+chidozieononiwu@users.noreply.github.com> Date: Fri, 29 May 2026 16:37:42 -0700 Subject: [PATCH 2/4] Apply suggestion from @chidozieononiwu --- eng/pipelines/templates/jobs/archetype-sdk-client.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/eng/pipelines/templates/jobs/archetype-sdk-client.yml b/eng/pipelines/templates/jobs/archetype-sdk-client.yml index 37dcdda3e8..13b84edbd9 100644 --- a/eng/pipelines/templates/jobs/archetype-sdk-client.yml +++ b/eng/pipelines/templates/jobs/archetype-sdk-client.yml @@ -164,7 +164,7 @@ jobs: - template: /eng/common/pipelines/templates/steps/create-authenticated-npmrc.yml parameters: - npmrcPath: $(UserProfile)/.npmrc + npmrcPath: $(Agent.TempDirectory)/generate-release/.npmrc - pwsh: npm install -g moxygen displayName: Install Moxygen to generate markdown for learn.microsoft.com From 811216ab077172d9e4f2214398d9f563170fc3b2 Mon Sep 17 00:00:00 2001 From: Chidozie Ononiwu <31145988+chidozieononiwu@users.noreply.github.com> Date: Fri, 29 May 2026 16:37:50 -0700 Subject: [PATCH 3/4] Apply suggestion from @chidozieononiwu --- eng/pipelines/templates/jobs/archetype-sdk-client.yml | 2 ++ 1 file changed, 2 insertions(+) diff --git a/eng/pipelines/templates/jobs/archetype-sdk-client.yml b/eng/pipelines/templates/jobs/archetype-sdk-client.yml index 13b84edbd9..d2cdee1dbf 100644 --- a/eng/pipelines/templates/jobs/archetype-sdk-client.yml +++ b/eng/pipelines/templates/jobs/archetype-sdk-client.yml @@ -168,6 +168,8 @@ jobs: - pwsh: npm install -g moxygen displayName: Install Moxygen to generate markdown for learn.microsoft.com + env: + npm_config_userconfig: "$(Agent.TempDirectory)/generate-release/.npmrc" - pwsh: | Write-Host "Using apiview parser version ${{ variables.apiviewParserVersion }}" From 32821e23578def7a1a5655b21b861854fa8b522d Mon Sep 17 00:00:00 2001 From: Chidozie Ononiwu Date: Tue, 2 Jun 2026 17:38:47 -0700 Subject: [PATCH 4/4] Yse authenticated npmrc in check spelling --- .../pipelines/templates/steps/check-spelling.yml | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/eng/common/pipelines/templates/steps/check-spelling.yml b/eng/common/pipelines/templates/steps/check-spelling.yml index d5faccdbd7..6c8f6a8b62 100644 --- a/eng/common/pipelines/templates/steps/check-spelling.yml +++ b/eng/common/pipelines/templates/steps/check-spelling.yml @@ -15,7 +15,8 @@ # npm commands in the context of this step. If specified, this # will be set as the npm_config_userconfig environment variable # so that npm uses the provided config file instead of the default -# user config. +# user config. If not specified, this template creates and +# authenticates a temporary .npmrc and uses that file. # This check recognizes the setting of variable "Skip.SpellCheck" # if set to 'true', spellchecking will not be invoked. @@ -35,6 +36,11 @@ parameters: steps: - ${{ if eq(variables['Build.Reason'], 'PullRequest') }}: + - ${{ if eq(parameters.NpmConfigUserConfig, '') }}: + - template: /eng/common/pipelines/templates/steps/create-authenticated-npmrc.yml + parameters: + npmrcPath: $(Agent.TempDirectory)/check-spelling/.npmrc + - task: PowerShell@2 displayName: Check spelling (cspell) condition: and(succeeded(), ne(variables['Skip.SpellCheck'],'true')) @@ -49,6 +55,8 @@ steps: env: ${{ if ne(parameters.NpmConfigUserConfig, '') }}: npm_config_userconfig: ${{ parameters.NpmConfigUserConfig }} + ${{ if eq(parameters.NpmConfigUserConfig, '') }}: + npm_config_userconfig: "$(Agent.TempDirectory)/check-spelling/.npmrc" - ${{ if ne('', parameters.ScriptToValidateUpgrade) }}: - pwsh: | $changedFiles = ./eng/common/scripts/get-changedfiles.ps1