build: reduce official build latency - #335953
build: reduce official build latency#335953Dmitriy Vasyura (dmitrivMS) wants to merge 3 commits into
Conversation
Pipeline macOS signing into per-artifact notarization, cache API Scan dependencies, and retry failed sanity extension installs as soon as the UI becomes actionable. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Copilot review overview
🟡 Changes recommended
The cache key can cross registries, and the scoped installation-success locator cannot find its target.
Get a fresh assessment by requesting another Copilot review.
Review tier: Balanced
Findings: 2
Open findings (2)
What changed in this PR
Reduces official build latency through parallelized macOS signing, cached API Scan dependencies, and faster sanity-test retries.
Changes:
- Chains notarization per macOS artifact.
- Adds API Scan
node_modulescaching. - Retries extension installation when it becomes actionable.
| File | Description |
|---|---|
test/sanity/src/uiTest.ts |
Improves extension-install retry detection. |
build/azure-pipelines/win32/sdl-scan-win32.yml |
Adds API Scan dependency caching. |
build/azure-pipelines/darwin/codesign.ts |
Parallelizes per-artifact signing and notarization. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Include the selected npm registry in the API Scan cache identity and retain page-level extension installation success detection. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Shorten the registry setup comment and call the PowerShell directory cmdlet directly while retaining native command exit checks. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
|
Investigated official Insider build 20260912.7 (
The run used the initial PR commit; the current head also contains the two review-fix commits. I did not queue or rerun any Azure pipeline. |
|
Closing in favor of three focused replacement PRs: macOS signing, API Scan caching, and the sanity-test retry change. The sanity-test PR will be created only after local Windows validation against an archive ZIP. |

Summary
Improves official build latency based on analysis of build 20260912.1, which completed in 1h 46m 41s.
Measurements
Validation
npm run eslintcd build && npm run typecheckcd test/sanity && npm run compilebuild/azure-pipelines/win32/sdl-scan-win32.ymlwithjs-yamlgit diff --checkNo Azure DevOps pipelines were started as part of this investigation.