Commit Graph

662 Commits

Author SHA1 Message Date
dependabot[bot] 6a252b13fe Bump actions/download-artifact from 7 to 8 (#312121)
Bumps [actions/download-artifact](https://github.com/actions/download-artifact) from 7 to 8.
- [Release notes](https://github.com/actions/download-artifact/releases)
- [Commits](https://github.com/actions/download-artifact/compare/v7...v8)

---
updated-dependencies:
- dependency-name: actions/download-artifact
  dependency-version: '8'
  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-05-12 18:59:03 +00:00
Rob Lourens 37ad80f8e5 Fix the node_modules cache (#315585) 2026-05-10 18:34:39 -07:00
Dmitriy Vasyura 44669e93e8 Run copilot extension lint in PR CI (#315368) 2026-05-08 21:40:38 +00:00
Henning Dieterichs 94197da951 Deletes unneeded screenshot-test.yml 2026-05-08 13:19:23 +02:00
Henning Dieterichs 66bdcf97c7 Renames component fixture ci workflow to "component-fixtures.yml" / Component Fixture Screenshots & Tests (#315218)
Renames component fixture ci workflow to "component-fixtures.yml" / Component Fixture Screenshots & Tests
2026-05-08 12:47:58 +02:00
Henning Dieterichs ca2ded47a8 updates component explorer 2026-05-07 18:43:10 +02:00
Henning Dieterichs 8ff4538da0 Fail CI if a component fixture throws & enable node_module caching 2026-05-07 12:44:16 +02:00
Henning Dieterichs 8f4ed8d714 Do no fail component explorer check if fixture had errors 2026-05-05 01:51:38 +02:00
Henning Dieterichs 83b3a75bba Fixes component fixture errors (#314164)
* Fixes component fixture errors

* refactor: replace empty mock with MockChatModeService in chat fixture services
2026-05-04 22:43:25 +02:00
Henning Dieterichs b4399bee94 Fixes 2026-05-04 19:15:19 +02:00
Henning Dieterichs bb81c9c9bb Improves screenshot experience for forks 2026-05-04 19:15:19 +02:00
Raymond Zhao d84f2e3008 fix: save extensions cache on misses (#313560)
* fix: restore built-in extensions cache on main workflow

* Add prepare step
2026-04-30 19:04:07 +00:00
Matt Bierner ac56466f75 Merge branch 'main' into dev/mjbvz/copilot-ts-go 2026-04-29 14:56:18 -07:00
Michael Lively 41499ac00a ci: drop VSCODE_OSS token conditional from pr.yml compile job (#313314)
Follow-up to #313128. The VSCODE_OSS fallback isn't needed for the
api.github.com calls in core-ci — secrets.GITHUB_TOKEN already
authenticates those reads with permissions: contents: read (added in
#304929), so we don't hit the anonymous rate limit on 1ES.
2026-04-29 14:41:25 -07:00
Matt Bierner 062d81fead Make sure we install root build tools 2026-04-29 13:53:28 -07:00
Michael Lively 7a833e177b ci: switch PR workflows back to 1ES self-hosted runners with JobId (#313128)
* ci: switch PR workflows back to 1ES self-hosted runners with JobId

Re-applies #311975 (reverted in #312033). Adds per-run+attempt JobId
labels to scope 1ES agents to specific GitHub Actions runs and prevent
intermittent runner cancellations.

Also switches the pr.yml compile job's GITHUB_TOKEN from the
ephemeral repo-scoped runner token to secrets.VSCODE_OSS so cross-repo
GitHub API release fetches (vscode-js-debug, vscode-js-debug-companion,
vscode-js-profile-visualizer, etc.) authenticate properly. On 1ES pools
the shared egress IPs hit the anonymous 60/hr api.github.com rate limit
and produced 403 fan-out across PRs last time.

* ci: fall back to GITHUB_TOKEN for fork PRs

Match the historical pattern from before #255987 — fork PRs can't
access secrets.VSCODE_OSS, so use the conditional to pick GITHUB_TOKEN
for forks.
2026-04-29 10:51:12 -07:00
Matt Bierner 371c4a0a73 Use root eslint for copilot 2026-04-28 15:20:59 -07:00
Henning Dieterichs a080227496 Improve screenshot diff logic for pull requests and pushes; add job summary step 2026-04-24 19:59:43 +02:00
Matt Bierner 1a54933643 Revert "Use main eslint config for copilot extension too" 2026-04-23 15:17:22 -07:00
Matt Bierner 74fca4377e Merge pull request #311606 from microsoft/dev/mjbvz/advisory-swallow
Use main eslint config for copilot extension too
2026-04-23 14:29:25 -07:00
dependabot[bot] fd6a74635f Bump actions/cache from 4 to 5 (#312122)
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>
2026-04-23 08:30:49 -07:00
Henning Dieterichs 6e04312650 Add detailed diff output for blocks-ci screenshot mismatch errors 2026-04-22 19:55:25 +02:00
Paul cc33dc6e7c Fixes for the chat perf pipeline (#311605) 2026-04-21 19:11:30 -07:00
Matt Bierner a261ac606a Remove extra copilot lint check
These lint checks should already be handled by the full pr lint checks
2026-04-20 23:21:22 -07:00
Ben Villalobos 79e5111feb Allow cherry-pick bot PRs in engineering system changes check (#311475)
* Allow cherry-pick bot PRs in engineering system changes check

Add an exception for PRs created by vs-code-engineering[bot] whose title
starts with [cherry-pick] and that carry the cherry-pick-artifact label.

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

* Fetch cherry-pick-artifact label via API at runtime

The label is applied ~2s after PR creation, so the webhook payload may
not include it. Fetch current labels from the API instead, gated behind
cheap event-payload checks to avoid extra API calls on unrelated PRs.

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

* Add label retry loop and consolidate guard expressions

Retry the cherry-pick-artifact label check up to 3 times (2s apart) to
handle the ~2s delay between PR creation and label application.

Consolidate the repeated exception guards into a single 'allowed' step
with a 'blocked' output, simplifying downstream conditions.

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

---------

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
2026-04-20 21:23:32 +00:00
Ben Villalobos 6159ac50a6 ci: allow copilot extension version bump in engineering system check (#311122) 2026-04-18 00:09:51 +00:00
Paul ec992baa49 Add performance tests (#309700) 2026-04-17 21:23:43 +00:00
Henning Dieterichs 41770c58f2 Fixes screenshots attributed to the wrong PR. 2026-04-17 22:48:28 +02:00
dependabot[bot] de423905d4 Bump actions/github-script from 8 to 9 (#310716)
Bumps [actions/github-script](https://github.com/actions/github-script) from 8 to 9.
- [Release notes](https://github.com/actions/github-script/releases)
- [Commits](https://github.com/actions/github-script/compare/v8...v9)

---
updated-dependencies:
- dependency-name: actions/github-script
  dependency-version: '9'
  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-04-16 07:43:39 -07:00
Henning Dieterichs 88797f66a0 skip screenshot upload/diff for fork PRs (OIDC unavailable) 2026-04-14 11:38:10 +02:00
Matt Bierner f8e1706349 Merge branch 'main' into dev/mjbvz/potential-narwhal 2026-04-13 10:24:52 -07:00
Matt Bierner ec1fc105d7 Merge branch 'main' into dev/mjbvz/potential-narwhal 2026-04-13 10:24:11 -07:00
Robo 06f94d567a ci: fix intermittent fontconfig SIGSEGV on Linux CI (#309451)
The Electron main process intermittently crashes during startup on the
`[pango] FcInit` thread with a NULL pointer dereference in expat's XML
string processing, triggered by fontconfig parsing `<include>` directives
in fonts.conf via `XML_ExternalEntityParserCreate`.

Set FONTCONFIG_FILE to a minimal config based on upstream
fontconfig 2.15.0 fonts.conf.in with `<include>` directives removed and
generic family aliases inlined. This avoids the external entity parser
codepath entirely. A version check will fail the build once the runner
ships expat >= 2.7.5, prompting removal of the workaround.
2026-04-13 07:08:42 -07:00
Alex Ross 027a4d3ce4 Bump version to 1.117.0 (#309394)
* Bump version to 1.117.0

* npm i

* wait to do engine version bump

* Revert "wait to do engine version bump"

This reverts commit 9db1c0feb6.

* Add Copilot extension tests to Linux/Windows Electron integration test runs

* Remove failing step that we moved to the main build

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
2026-04-13 14:52:32 +02:00
Henning Dieterichs 1117499308 fix: use fake time when rendering fixtures 2026-04-13 14:26:40 +02:00
Osvaldo Ortega a75ae92ef5 Sessions test on property (#309218) 2026-04-12 17:51:24 -07:00
João Moreno 148cf3a1c4 fix: remove duplicate test runs (#309015) 2026-04-10 16:29:56 +02:00
Matt Bierner 15f509dc32 Remove extract script 2026-04-09 11:14:14 -07:00
Henning Dieterichs 7892126910 Report all screenshot changes 2026-04-09 18:47:23 +02:00
Christof Marti 16be18337a Move chat-lib workflow up 2026-04-09 16:27:21 +00:00
dependabot[bot] 11f3c7f744 build(deps): bump actions/github-script from 7 to 8 (#308763)
Bumps [actions/github-script](https://github.com/actions/github-script) from 7 to 8.
- [Release notes](https://github.com/actions/github-script/releases)
- [Commits](https://github.com/actions/github-script/compare/v7...v8)

---
updated-dependencies:
- dependency-name: actions/github-script
  dependency-version: '8'
  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-04-09 08:44:57 -07:00
dependabot[bot] 97c3f4a4df build(deps): bump actions/cache from 4 to 5 (#308762)
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>
2026-04-09 08:44:55 -07:00
dependabot[bot] b56446b212 build(deps): bump actions/upload-artifact from 5 to 7 (#308761)
Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 5 to 7.
- [Release notes](https://github.com/actions/upload-artifact/releases)
- [Commits](https://github.com/actions/upload-artifact/compare/v5...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-04-09 08:44:53 -07:00
Robo 1f40e568c5 ci: bump github runner to ubuntu-24.04 to address fontconfig crash (#308495)
* ci: debug fontconfig crash during app launch

* chore: update commands

* ci: bump to ubuntu-24.04 to fix fontconfig crash

Fixes an intermittent SIGSEGV on the [pango] FcInit thread during
Electron startup in CI integration tests.

Root cause: Chromium's InitializeGlobalFontConfigAsync() posts FcInit()
to a thread pool worker (crbug.com/404311), while pango's pangoft2
backend independently calls FcInit() from its own thread during GTK
initialization. fontconfig 2.13.1 (shipped in ubuntu-22.04) lacks
thread-safe initialization — concurrent first-time FcInit() calls
race and both enter FcConfigParse(), corrupting shared global state.
This causes expat (called by fontconfig to parse fonts.conf) to
dereference a NULL pointer.

ubuntu-24.04 ships fontconfig 2.15.0 which includes the thread-safe
initialization from 2.14+.

* ci: enable namespace sandbox
2026-04-08 14:46:58 +00:00
Henning Dieterichs 389903d431 Temporarily allow screenshot test failures to avoid blocking the pipeline until the screenshot test is stable. 2026-04-07 23:45:14 +02:00
João Moreno 55a24659d9 chore: Add unstaged and staged diff output to git state check (#308300)
Co-authored-by: Copilot <copilot@github.com>
2026-04-07 19:10:34 +00:00
Henning Dieterichs da1fd60941 fixes flaky fixtures 2026-04-07 20:54:19 +02:00
Michael Lively 5267dfdc95 ci: switch Copilot PR checks to GitHub-hosted runners (#308267)
* switch to gh hosted runners (#308241)

Move the Copilot PR checks off the 1ES self-hosted pools and onto the same GitHub-hosted Ubuntu and Windows images used by the rest of the workflow.

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

* fix: install xvfb for copilot linux tests on gh-hosted runner

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

---------

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
2026-04-07 11:01:06 -07:00
Henning Dieterichs 706c2ced9e Fix blocks-ci screenshot diff output and PR comment content 2026-04-07 18:38:44 +02:00
Henning Dieterichs a199142a6f Add error handling for blocks-ci screenshot suggestion posting 2026-04-07 18:38:44 +02:00