Commit Graph

569 Commits

Author SHA1 Message Date
Osvaldo Ortega
da5b9e899c CI 2026-03-10 15:53:16 -07:00
Matt Bierner
0cd1e5976a Disable api-version-check for now
Needed to unblock #300477
2026-03-10 10:22:07 -07:00
Alex Ross
4198c6b4c4 Try fix workflow list (#300458)
* Try fix workflow list

* Fix API name again.
2026-03-10 16:30:39 +01:00
Alex Ross
54b762dd84 Rerun the failed API version check (#300443)
* Rerun the failed API version check

* CCR comments
2026-03-10 14:59:01 +01:00
Alex Ross
cba3c7aac7 Add API proposal version check (#300392)
* Add API proposal version check

* CCR feedback

* Test with version bump

* Comment improvements

* Undo test version bump

* More comment improvement
2026-03-10 12:52:52 +01:00
Henning Dieterichs
74c765017e Disable screenshots until time out bug is fixed (#300293) 2026-03-10 00:52:12 +00:00
dependabot[bot]
3eab7b9d79 Bump actions/upload-artifact from 4 to 7 (#298952)
Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 4 to 7.
- [Release notes](https://github.com/actions/upload-artifact/releases)
- [Commits](https://github.com/actions/upload-artifact/compare/v4...v7)

---
updated-dependencies:
- dependency-name: actions/upload-artifact
  dependency-version: '7'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
2026-03-03 09:32:24 -08:00
João Moreno
d2f1f7ce97 Inline Compile & CompileCLI stages into platform specific jobs (#296006)
* chore: ralph loop checkpoint - 2026-02-17 12:48

* fix shell script

* Inline compile stage into platform jobs

Remove the standalone Compile stage from the ADO pipeline. Each
platform job (Windows, Linux, macOS, Alpine, Web) now compiles
TypeScript itself instead of downloading a shared Compilation artifact.

- Add VSCODE_RUN_CHECKS parameter to Linux jobs for hygiene/lint/CG
- Add VSCODE_RUN_COMPILE_EXTRAS parameter to macOS jobs for telemetry
  extraction and sourcemap upload
- Remove VSCODE_COMPILE_ONLY parameter entirely
- Delete product-compile.yml (no longer referenced)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* Always run telemetry extraction on macOS builds

The macOS Universal app merge requires both x64 and arm64 builds to have
identical file sets. Telemetry extraction was only running on arm64 (via
VSCODE_RUN_COMPILE_EXTRAS), causing the universal merge to fail due to
missing telemetry-core.json and telemetry-extensions.json in the x64 build.

Move telemetry extraction outside the VSCODE_RUN_COMPILE_EXTRAS gate so
it runs on all macOS builds. Sourcemap upload remains gated.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* Run telemetry extraction on all client builds

All client builds (Linux, Windows, macOS) need telemetry-core.json and
telemetry-extensions.json. Previously only macOS arm64 ran extract-telemetry.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* Only run telemetry extraction on Linux x64 (client build)

Linux arm64 and armhf are server-only builds, no need for telemetry files.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* Remove .ralph scaffolding files

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* Fix telemetry extraction on Windows with native PowerShell

The bash extract-telemetry.sh script fails on Windows because the Unix
bin shim gets interpreted as Node.js code. Use a native PowerShell
implementation that calls the extractor JS entry point directly.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* Handle missing telemetry files gracefully on Windows

The telemetry extractor may skip emitting declarations-resolved.json
when no events are found. Handle this case with a warning instead of
failing the build.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* Use deterministic build date from git commit timestamp

When each platform compiles independently, the build date embedded in
cli.js (via INSERT_PRODUCT_CONFIGURATION) differs between machines
because each uses new Date().toISOString(). This causes the macOS
Universal app merge to fail since cli.js SHA differs between x64/arm64.

Fix: use the git commit date (git log -1 --format=%cI HEAD) instead
of the current wall-clock time. This ensures all independent builds
on different machines produce identical timestamps.

Updated in:
- build/lib/date.ts: writeISODate() uses git commit date
- build/next/index.ts: bundle(), transpile, and readISODate fallback
  all use git commit date

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* Remove redundant extensions-ci task from CI pipelines

core-ci (esbuild path) already includes cleanExtensionsBuildTask,
compileNonNativeExtensionsBuildTask, and compileExtensionMediaBuildTask.
Running extensions-ci in parallel caused a race condition where
core-ci's rimraf of .build/extensions clashed with extensions-ci
writing to the same directory.

Also removes dead code:
- extensions-ci and extensions-ci-pr task definitions (fully subsumed)
- core-ci-old task (useEsbuildTranspile is always true)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* remove VSCODE_RUN_COMPILE_EXTRAS

* address PR feedback regarding code duplication of `getGitCommitDate()` function by exporting a single helper function from `build/lib/date.ts` and importing it in `build/next/index.ts` to ensure consistent behavior and improve code maintainability.

* update readISODate function to return git commit date instead of current date

* add telemetry sorting script and integrate into build process for consistent output

* refactor telemetry extraction process: replace shell script with TypeScript implementation

* update skill

* update telemetry-extractor dependency to version 1.19.0

* fix build

* fix more duplicate telemetry definition issues

* cleanup

* refactor: consolidate validation checks into quality checks and remove obsolete tasks

* bust the cache

* undo cache bust

* fix expression

* fix

* fix: update Azure storage account name in quality checks

* fix: initialize stages set with 'Quality'

* fix: add VSCODE_BUILD_TYPE parameter with options for Product and CI builds

* fix: update Azure Pipeline CLI to use parameters instead of variables for queueing builds

* fix: update VSCODE_BUILD_TYPE parameter values for clarity

* fix: update default value for VSCODE_BUILD_TYPE parameter to 'Product'

* leaner

* even leaner

* only npm ci in build

* 💄

* run entire npm ci

* fix

* fix

* fix it

* Inline CompileCLI into platform stages

- Remove centralized CompileCLI stage
- Move CLI jobs into Windows, Linux, macOS stages as independent jobs
- CLI jobs now compile, publish unsigned mid-job, sign, and publish signed
- Platform compile jobs use deemon + waitForArtifacts for async CLI download
- Delete separate CLI sign jobs (now merged into CLI compile jobs)
- Remove CompileCLI from publish.ts stage tracking

* fix: macOS CLI signing - use proper directory structure for ESRP

* fix: add BUILDS_API_URL to Windows and Linux job templates

* fix: label Linux CLI jobs clearly

---------

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: João Moreno <22350+joaomoreno@users.noreply.github.com>
2026-03-01 08:48:08 +01:00
Henning Dieterichs
5428850858 fixes screenshot status (#298303) 2026-02-27 18:04:49 +00:00
Henning Dieterichs
3272ea62f3 updates screenshot pipeline & baseline 2026-02-27 17:58:11 +01:00
dependabot[bot]
f59869f7c6 Bump actions/checkout from 4 to 6 (#297973)
Bumps [actions/checkout](https://github.com/actions/checkout) from 4 to 6.
- [Release notes](https://github.com/actions/checkout/releases)
- [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](https://github.com/actions/checkout/compare/v4...v6)

---
updated-dependencies:
- dependency-name: actions/checkout
  dependency-version: '6'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Raymond Zhao <7199958+rzhao271@users.noreply.github.com>
2026-02-27 00:29:06 +00:00
dependabot[bot]
cfc0b71fbf Bump actions/setup-node from 4 to 6 (#297974)
Bumps [actions/setup-node](https://github.com/actions/setup-node) from 4 to 6.
- [Release notes](https://github.com/actions/setup-node/releases)
- [Commits](https://github.com/actions/setup-node/compare/v4...v6)

---
updated-dependencies:
- dependency-name: actions/setup-node
  dependency-version: '6'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
2026-02-26 15:48:15 -08:00
Henning Dieterichs
0a01b4f5ae Fixes vite warnings & improves pipeline 2026-02-26 21:03:52 +01:00
Henning Dieterichs
07e303015d dont fail ci for external PRs (#297124) 2026-02-23 15:00:11 -08:00
Henning Dieterichs
5c4204e60d updates component explorer 2026-02-23 22:44:44 +01:00
Henning Dieterichs
89cd7ec4bb adds screenshot status with url, updates skill 2026-02-23 13:28:43 +01:00
Henning Dieterichs
e50a372498 Post url to review screenshots 2026-02-23 12:50:38 +01:00
Henning Dieterichs
155c18ecfc updates component explorer (#296938)
* updates component explorer

* updates package.json

* fix screenshot CI: install build/ dependencies for jsonc-parser

* sets relative base

* updates lockfile
2026-02-23 11:44:35 +01:00
Henning Dieterichs
a9c01a08a3 updates github action check 2026-02-20 19:57:07 +01:00
Henning Dieterichs
e8f4c461f1 updates screenshot-test.yml 2026-02-20 19:57:07 +01:00
Henning Dieterichs
a74a86e1f9 fix: rm stale lockfile for build/vite in CI (platform-specific bindings) 2026-02-20 19:57:07 +01:00
Henning Dieterichs
669def0eca update 2026-02-20 19:57:07 +01:00
Henning Dieterichs
424ea5b801 fixes ci 2026-02-20 19:57:07 +01:00
Henning Dieterichs
2751fb4d2f adds component explorer 2026-02-20 19:57:07 +01:00
Johannes Rieken
10d738ca3c Add cyclic dependency check script and tests (#296035)
* eng: add cyclic dependency check script and update workflow

* refactor: export classes and functions in checkCyclicDependencies for better accessibility; add comprehensive tests for cycle detection and file processing

* Update build/lib/checkCyclicDependencies.ts

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
2026-02-18 10:16:25 -08:00
Raymond Zhao
965d950c25 chore: run cargo update (#295062) 2026-02-13 10:58:40 -08:00
Raymond Zhao
000e601e5e Revert "engineering: use tar+zstd for win32 node_modules cache" (#291740)
Revert "engineering: use tar+zstd for win32 node_modules cache (#291624)"

This reverts commit 70529729e6.
2026-01-29 21:48:29 +00:00
João Moreno
70529729e6 engineering: use tar+zstd for win32 node_modules cache (#291624)
* engineering: use tar+zstd for win32 node_modules cache

* bump cache

* Update .github/workflows/pr-node-modules.yml

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Update build/azure-pipelines/win32/product-build-win32-node-modules.yml

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Update build/azure-pipelines/win32/steps/product-build-win32-compile.yml

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Update .github/workflows/pr-win32-test.yml

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* bump cache salt

* more fixes

---------

Co-authored-by: Aman Karmani <aman@tmm1.net>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
2026-01-29 21:16:43 +01:00
Ladislau Szomoru
dd65f0c5ef Engineering - switch from npm-run-all to npm-run-all2 (#284087)
* Introduce compilation error

* Engineering - limit the tasks that we run

* Limit available memory to simulate an OOM

* Try to update the task

* Remove the use of npm-run-all

* Fix script

* Another try

* Try npm-run-all2

* Restore tasks, keep npm-run-all2

* Switch from npm-run-all to npm-run-all2

* Revert changes that were used for testing
2026-01-05 15:49:42 +01:00
dependabot[bot]
843d264a93 Bump actions/cache from 4 to 5 (#283559)
Bumps [actions/cache](https://github.com/actions/cache) from 4 to 5.
- [Release notes](https://github.com/actions/cache/releases)
- [Changelog](https://github.com/actions/cache/blob/main/RELEASES.md)
- [Commits](https://github.com/actions/cache/compare/v4...v5)

---
updated-dependencies:
- dependency-name: actions/cache
  dependency-version: '5'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
2025-12-15 11:46:42 +00:00
dependabot[bot]
25dcf325d2 Bump actions/upload-artifact from 5 to 6 (#283557)
Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 5 to 6.
- [Release notes](https://github.com/actions/upload-artifact/releases)
- [Commits](https://github.com/actions/upload-artifact/compare/v5...v6)

---
updated-dependencies:
- dependency-name: actions/upload-artifact
  dependency-version: '6'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
2025-12-15 12:25:45 +01:00
Ladislau Szomoru
15ac5b6a4b Engineering - delete unused GitHub action (#280915) 2025-12-03 11:04:26 +00:00
Ladislau Szomoru
e1341b09d3 Engineering - add GitHub action to prevent changes to the engineering system (#280261) 2025-12-02 14:00:45 +01:00
Ladislau Szomoru
90bc9ea6ad Revert "Engineering - don't use hashFiles on macOS (#279015)" (#279993)
This reverts commit be127fdd32.
2025-12-01 09:02:27 +01:00
Matt Bierner
643ae5a998 Merge pull request #279258 from mjbvz/dev/mjbvz/outside-crow
Remove build-script no new files PR check
2025-11-25 10:09:48 -08:00
dependabot[bot]
ffc052c913 Bump actions/checkout from 5 to 6 (#279152)
Bumps [actions/checkout](https://github.com/actions/checkout) from 5 to 6.
- [Release notes](https://github.com/actions/checkout/releases)
- [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](https://github.com/actions/checkout/compare/v5...v6)

---
updated-dependencies:
- dependency-name: actions/checkout
  dependency-version: '6'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Raymond Zhao <7199958+rzhao271@users.noreply.github.com>
2025-11-25 14:29:41 +00:00
Matt Bierner
475d0c85af Remove build-script PR check
For #277526

Our build scripts no longer require being compiled so we can skip the extra check
2025-11-24 17:19:45 -08:00
Ladislau Szomoru
be127fdd32 Engineering - don't use hashFiles on macOS (#279015) 2025-11-23 21:16:37 +01:00
Matt Bierner
2648263d3e Run our build scripts directly as typescript (#277567)
* Run our build scripts directly as typescript #277567

Follow up on #276864
For #277526

* Remove a few more ts-node references

* Fix linux and script reference

* Remove `_build-script` ref

* Fix script missing closing quote

* use type only import

* Fix export

* Make sure to run copy-policy-dto

* Make sure we run the copy-policy-dto script

* Enable `verbatimModuleSyntax`

* Pipelines fixes

* Try adding explicit ext to path

* Fix bad edit

* Revert extra `--`

---------

Co-authored-by: João Moreno <joaomoreno@users.noreply.github.com>
2025-11-21 14:56:00 +01:00
Paul
2b76aa1113 Add build folder tests to CI (#272564) 2025-10-31 09:23:04 +01:00
Ladislau Szomoru
0aead435a0 Engineering - try to run mangling in the PR check (#268428) 2025-10-29 09:04:43 +00:00
dependabot[bot]
ab2cef5ab1 Bump actions/upload-artifact from 4 to 5 (#273539)
Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 4 to 5.
- [Release notes](https://github.com/actions/upload-artifact/releases)
- [Commits](https://github.com/actions/upload-artifact/compare/v4...v5)

---
updated-dependencies:
- dependency-name: actions/upload-artifact
  dependency-version: '5'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
2025-10-27 07:11:05 -07:00
dependabot[bot]
7a90c2cf94 Bump actions/setup-node from 5 to 6 (#272258)
Bumps [actions/setup-node](https://github.com/actions/setup-node) from 5 to 6.
- [Release notes](https://github.com/actions/setup-node/releases)
- [Commits](https://github.com/actions/setup-node/compare/v5...v6)

---
updated-dependencies:
- dependency-name: actions/setup-node
  dependency-version: '6'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
2025-10-20 15:14:12 +02:00
Peng Lyu
eeca8c6a90 use large runners 2025-09-19 08:19:11 -07:00
Peng Lyu
67ed568d37 ubuntu-latest-4-cores 2025-09-17 21:19:54 -07:00
Peng Lyu
d0fd753cdc Merge remote-tracking branch 'origin/main' into rebornix/minimum-newt 2025-09-17 09:43:29 -07:00
Peng Lyu
9aba23589d maybe macos 2025-09-17 09:31:07 -07:00
Alexandru Dima
aead51d8a2 Define GitHub Copilot coding agent environment (#267117) 2025-09-17 01:35:53 +02:00
Peng Lyu
c34285217c experiment ubuntu-4-core 2025-09-16 10:07:31 -07:00
Peng Lyu
f39ac34be8 Update .github/workflows/copilot-setup-steps.yml
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
2025-09-15 22:30:14 -07:00