Skip to content

ci: bump autorest.powershell submodule for CFSClean pnpm-lock v9 fix - #3731

Merged
Ramses Sanchez-Hernandez (ramsessanchez) merged 1 commit into
mainfrom
gavinbarron/bump-autorest-pnpm-lockfile
Aug 19, 2026
Merged

ci: bump autorest.powershell submodule for CFSClean pnpm-lock v9 fix#3731
Ramses Sanchez-Hernandez (ramsessanchez) merged 1 commit into
mainfrom
gavinbarron/bump-autorest-pnpm-lockfile

Conversation

@gavinbarron

Copy link
Copy Markdown
Member

What

Advance the autorest.powershell submodule c42e7f8eb1669283 (powershell-v2 tip), picking up:

Why (CFSClean / 1ES network isolation)

Pipelines 187 / 221 / 663 kept leaking a stable registry.npmjs.org per generation job (PermissiveCFSClean = False). Root cause: the committed Rush lockfile was lockfileVersion 6.0 while rush.json pins pnpm 10.34.4 (v9), so pnpm logged "Ignoring not compatible lockfile" and re-resolved + re-fetched every run — following the feed's upstream dist.tarball to public npm on first-request cache misses. Neither #3718 (npm_config_registry) nor #3729 (.pnpmfile.cjs afterAllResolved rewrite) could fix it while the lockfile was ignored.

The submodule fix regenerates the lockfile to v9 so rush install runs frozen and deterministic, resolving every tarball through the authenticated private feed. Verified locally: 0 registry.npmjs.org URLs in the lockfile; frozen install reports "Already up to date."

Validation after merge

Once merged, the first individualCI run of def 187 should report registry.npmjs.org = 0 (the strict CFSClean tier). The PowerShell Gallery (CFSClean2) egress is handled separately by draft PR #3720; PermissiveCFSClean flips True once both land.

Note: build-stage egress can only be validated via individualCI/scheduled runs — manual queues skip the generation stage on this 1ES Official pipeline.

Advance the autorest.powershell submodule c42e7f8e -> b1669283 to pick up
powershell-v2 PRs #21 and #22:
- #21 regenerates common/config/rush/pnpm-lock.yaml to lockfileVersion 9
  (matching the pinned pnpm 10.34.4), so the Rush Build step runs a
  frozen, deterministic `rush install` instead of ignoring the
  incompatible v6 lockfile and re-resolving/re-fetching from public npm.
  This eliminates the residual 2x registry.npmjs.org egress that kept
  PermissiveCFSClean = False on pipelines 187 / 221 / 663.
- #22 bumps js-yaml 3.15.0 -> 3.15.1.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 6ea00b50-c719-469b-bf9c-72120184bfe0
@gavinbarron
Gavin Barron (gavinbarron) requested a review from a team as a code owner August 19, 2026 18:57
@ramsessanchez
Ramses Sanchez-Hernandez (ramsessanchez) merged commit 65532f3 into main Aug 19, 2026
6 of 8 checks passed
@ramsessanchez
Ramses Sanchez-Hernandez (ramsessanchez) deleted the gavinbarron/bump-autorest-pnpm-lockfile branch August 19, 2026 19:32
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