From 81e4bc9b160094df7c302160a5581084c507935b Mon Sep 17 00:00:00 2001 From: ramsessanchez <63934382+ramsessanchez@users.noreply.github.com> Date: Thu, 4 Jun 2026 16:55:23 -0700 Subject: [PATCH 1/6] fix: resolve daily CI build failures (1ES pool image + npm auth) - Add missing 'image: ubuntu-latest' to main pool configuration - Add .npmrc pointing to GraphDeveloperExperiences_Public Azure Artifacts feed - Add npmAuthenticate@0 task before npm ci in the pipeline Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --- .azure-pipelines/daily-ci-build.yml | 6 ++++++ .npmrc | 2 ++ 2 files changed, 8 insertions(+) create mode 100644 .npmrc diff --git a/.azure-pipelines/daily-ci-build.yml b/.azure-pipelines/daily-ci-build.yml index 5a76b71f43..1184d682ce 100644 --- a/.azure-pipelines/daily-ci-build.yml +++ b/.azure-pipelines/daily-ci-build.yml @@ -21,6 +21,7 @@ extends: parameters: pool: name: Azure-Pipelines-1ESPT-ExDShared + image: ubuntu-latest os: linux sdl: sourceAnalysisPool: @@ -47,6 +48,11 @@ extends: inputs: versionSpec: '20.x' + - task: npmAuthenticate@0 + displayName: Authenticate npm + inputs: + workingFile: $(Build.SourcesDirectory)/.npmrc + - script: npm ci displayName: Install dependencies workingDirectory: $(Build.SourcesDirectory) diff --git a/.npmrc b/.npmrc new file mode 100644 index 0000000000..6aa1836c55 --- /dev/null +++ b/.npmrc @@ -0,0 +1,2 @@ +registry=https://microsoftgraph.pkgs.visualstudio.com/0985d294-5762-4bc2-a565-161ef349ca3e/_packaging/GraphDeveloperExperiences_Public/npm/registry/ +always-auth=true From 7f500697094c47a5bcddca9da537e030570a8b06 Mon Sep 17 00:00:00 2001 From: ramsessanchez <63934382+ramsessanchez@users.noreply.github.com> Date: Thu, 4 Jun 2026 17:03:20 -0700 Subject: [PATCH 2/6] fix: generate .npmrc inline in pipeline instead of committing it Aligns with updated approach in microsoftgraph/msgraph-sdk-typescript-core#583. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --- .azure-pipelines/daily-ci-build.yml | 6 ++++++ .npmrc | 2 -- 2 files changed, 6 insertions(+), 2 deletions(-) delete mode 100644 .npmrc diff --git a/.azure-pipelines/daily-ci-build.yml b/.azure-pipelines/daily-ci-build.yml index 1184d682ce..38604e8d9f 100644 --- a/.azure-pipelines/daily-ci-build.yml +++ b/.azure-pipelines/daily-ci-build.yml @@ -48,6 +48,12 @@ extends: inputs: versionSpec: '20.x' + - script: | + echo "registry=https://microsoftgraph.pkgs.visualstudio.com/0985d294-5762-4bc2-a565-161ef349ca3e/_packaging/GraphDeveloperExperiences_Public/npm/registry/" > .npmrc + echo "always-auth=true" >> .npmrc + displayName: Create .npmrc for Azure Artifacts feed + workingDirectory: $(Build.SourcesDirectory) + - task: npmAuthenticate@0 displayName: Authenticate npm inputs: diff --git a/.npmrc b/.npmrc deleted file mode 100644 index 6aa1836c55..0000000000 --- a/.npmrc +++ /dev/null @@ -1,2 +0,0 @@ -registry=https://microsoftgraph.pkgs.visualstudio.com/0985d294-5762-4bc2-a565-161ef349ca3e/_packaging/GraphDeveloperExperiences_Public/npm/registry/ -always-auth=true From b27da2416204a1063a6bbf803b6e8d9d8712bab5 Mon Sep 17 00:00:00 2001 From: ramsessanchez <63934382+ramsessanchez@users.noreply.github.com> Date: Fri, 5 Jun 2026 12:21:25 -0700 Subject: [PATCH 3/6] fix: remove test step from daily CI build pipeline Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --- .azure-pipelines/daily-ci-build.yml | 3 --- 1 file changed, 3 deletions(-) diff --git a/.azure-pipelines/daily-ci-build.yml b/.azure-pipelines/daily-ci-build.yml index 38604e8d9f..77bcb6c329 100644 --- a/.azure-pipelines/daily-ci-build.yml +++ b/.azure-pipelines/daily-ci-build.yml @@ -67,6 +67,3 @@ extends: displayName: Build SDK workingDirectory: $(Build.SourcesDirectory) - - script: npm test - displayName: Run unit tests - workingDirectory: $(Build.SourcesDirectory) From fe37d1a9a66a1e9373a214fbe8eb0a9314305286 Mon Sep 17 00:00:00 2001 From: ramsessanchez <63934382+ramsessanchez@users.noreply.github.com> Date: Fri, 5 Jun 2026 15:04:34 -0700 Subject: [PATCH 4/6] fix: use ADO variable for npm registry URL in daily CI build Replace hardcoded Azure Artifacts registry URL with ADO pipeline variable, matching the approach in msgraph-sdk-typescript-core PR #583. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --- .azure-pipelines/daily-ci-build.yml | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/.azure-pipelines/daily-ci-build.yml b/.azure-pipelines/daily-ci-build.yml index 77bcb6c329..7a578774f5 100644 --- a/.azure-pipelines/daily-ci-build.yml +++ b/.azure-pipelines/daily-ci-build.yml @@ -49,8 +49,10 @@ extends: versionSpec: '20.x' - script: | - echo "registry=https://microsoftgraph.pkgs.visualstudio.com/0985d294-5762-4bc2-a565-161ef349ca3e/_packaging/GraphDeveloperExperiences_Public/npm/registry/" > .npmrc + echo "registry=$(NPM_REGISTRY_URL)" > .npmrc echo "always-auth=true" >> .npmrc + env: + NPM_REGISTRY_URL: $(npmRegistryUrl) displayName: Create .npmrc for Azure Artifacts feed workingDirectory: $(Build.SourcesDirectory) From 0b2a7ff3559e59d4cb2e65ea145774ebf1fc8a51 Mon Sep 17 00:00:00 2001 From: ramsessanchez <63934382+ramsessanchez@users.noreply.github.com> Date: Fri, 5 Jun 2026 15:09:17 -0700 Subject: [PATCH 5/6] fix: use ADO variable directly without env mapping Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --- .azure-pipelines/daily-ci-build.yml | 2 -- 1 file changed, 2 deletions(-) diff --git a/.azure-pipelines/daily-ci-build.yml b/.azure-pipelines/daily-ci-build.yml index 7a578774f5..bf00da635b 100644 --- a/.azure-pipelines/daily-ci-build.yml +++ b/.azure-pipelines/daily-ci-build.yml @@ -51,8 +51,6 @@ extends: - script: | echo "registry=$(NPM_REGISTRY_URL)" > .npmrc echo "always-auth=true" >> .npmrc - env: - NPM_REGISTRY_URL: $(npmRegistryUrl) displayName: Create .npmrc for Azure Artifacts feed workingDirectory: $(Build.SourcesDirectory) From fa0167186a150fdfb00434066551a745f5c06b74 Mon Sep 17 00:00:00 2001 From: ramsessanchez <63934382+ramsessanchez@users.noreply.github.com> Date: Fri, 5 Jun 2026 16:27:02 -0700 Subject: [PATCH 6/6] fix: use npm@1 task with Azure Artifacts feed for daily CI build Replace .npmrc creation + npmAuthenticate@0 + script npm ci with a single npm@1 task using customRegistry/customFeed, matching the approach from microsoftgraph/msgraph-sdk-typescript-core#583. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --- .azure-pipelines/daily-ci-build.yml | 20 +++++++------------- 1 file changed, 7 insertions(+), 13 deletions(-) diff --git a/.azure-pipelines/daily-ci-build.yml b/.azure-pipelines/daily-ci-build.yml index bf00da635b..2089fd8131 100644 --- a/.azure-pipelines/daily-ci-build.yml +++ b/.azure-pipelines/daily-ci-build.yml @@ -48,20 +48,14 @@ extends: inputs: versionSpec: '20.x' - - script: | - echo "registry=$(NPM_REGISTRY_URL)" > .npmrc - echo "always-auth=true" >> .npmrc - displayName: Create .npmrc for Azure Artifacts feed - workingDirectory: $(Build.SourcesDirectory) - - - task: npmAuthenticate@0 - displayName: Authenticate npm - inputs: - workingFile: $(Build.SourcesDirectory)/.npmrc - - - script: npm ci + - task: npm@1 displayName: Install dependencies - workingDirectory: $(Build.SourcesDirectory) + inputs: + command: 'ci' + verbose: true + customRegistry: 'useFeed' + customFeed: 'Graph Developer Experiences/GraphDeveloperExperiences_Public' + workingDir: '$(Build.SourcesDirectory)' - script: npm run build --workspaces displayName: Build SDK