Microsoft/vscode
1
0
Fork 0
mirror of https://github.com/microsoft/vscode.git synced 2026-07-01 12:06:24 +01:00
Code Issues Packages Projects Releases Wiki Activity
Files
dev/dmitriv/agent-host-implicit-context
vscode/.eslint-allowed-javascript-files
T
Alexandru Dima 37e7e85b10 Optimize node_modules caching for CI & PR checks (#322074) 
* CI: speed up node_modules cache with zstd + shared scripts

Switch the Linux/macOS node_modules cache from single-threaded gzip
(tar -czf) to multi-threaded zstd. The "Create node_modules archive"
step was spending ~5min of single-core gzip on a multi-GB tree on every
cache miss; zstd -T0 uses all cores and decompresses much faster, so
cache-hit jobs benefit too. Windows stays on 7-Zip (already threaded).

Extract the archive/extract commands into shared per-platform scripts
under .github/workflows/node_modules_cache/ (cache.sh / cache.ps1, each
dispatching on an archive|extract argument) so the format and flags live
in one place instead of being duplicated across ~8 workflows. Bump
build/.cachesalt to invalidate existing gzip caches.

Also remove the obsolete extensions/copilot CI workflows
(copilot-setup-steps.yml, ensure-node-modules-cache.yml, pr.yml) and the
unused build/listBuildCacheFiles.js, and drop their now-stale entries
(plus lit-html and signals-core) from .eslint-allowed-javascript-files.

* ci: seed copilot node_modules cache on main and rename cache keys

Add copilot-linux and copilot-windows jobs to pr-node-modules.yml so the
copilot node_modules cache is populated on main. Rename the copilot cache
keys to copilot-node_modules-linux / copilot-node_modules-windows in pr.yml.

* ci: extract node_modules cache into composite actions

Factor the repeated node_modules cache plumbing into two local composite
actions, restore-node-modules and save-node-modules, and migrate all
workflows that used the cache.sh/cache.ps1 archive flow (pr, pr-node-modules,
pr-{linux,darwin,win32}-test, copilot-setup-steps, component-fixtures,
css-order-scan).

- restore-node-modules computes the key, restores the cache, optionally
  extracts on a hit, and exports the resolved key via $GITHUB_ENV.
- save-node-modules archives node_modules and saves it to the cache, reusing
  the key exported by restore so callers don't repeat the prefix.
- Bespoke install steps stay in the workflows, so per-job env/secrets never
  cross the action boundary.
- Only seed the cache on branch pushes (component-fixtures skips PRs, whose
  caches aren't shared).

* save the node_modules cache for now to test it

* ci: fix node_modules cache save dropping the archive

cache.sh wrote its archive as cache.tzst, but actions/cache reserves that
name for its own tarball and passes --exclude cache.tzst, so our archive was
excluded and an empty (~200 B) cache was saved on Linux/macOS. Rename the
archive to node-modules.tzst and bump build/.cachesalt to invalidate the
broken cache entries.

* empty commit

* Remove again saving to the node modules cache from PR steps
2026-06-19 17:21:43 +02:00

160 lines
8.0 KiB
Plaintext
Raw Permalink Blame History

# Allowlist of pre-existing JavaScript files (.js, .cjs, .mjs).
#
# This file is consumed by eslint.config.js and gates the
# `local/code-no-new-javascript-files` lint rule.
#
# DO NOT ADD NEW ENTRIES. Write new files in TypeScript instead.
# Modifications to this file require sign-off from the CODEOWNERS.
#
# Lines starting with `#` are comments. Blank lines are ignored.
.github/skills/heap-snapshot-analysis/helpers/streamSnapshot.mjs
.vscode-test.js
build/azure-pipelines/common/installPlaywright.js
build/azure-pipelines/github-check-run.js
build/builtin/browser-main.js
build/builtin/main.js
build/codex/generate-protocol.mjs
eslint.config.js
extensions/copilot/.mocha-multi-reporters.js
extensions/copilot/.mocharc.js
extensions/copilot/.vscode-test.mjs
extensions/copilot/.vscode/extensions/visualization-runner/entry.js
extensions/copilot/script/electron/simulationWorkbenchMain.js
extensions/copilot/src/extension/completions-core/vscode-node/extension/test/run.js
extensions/copilot/src/extension/test/node/fixtures/gitdiff/generate-diffs.js
extensions/copilot/src/platform/parser/test/node/fixtures/test.js
extensions/copilot/test/scenarios/test-cli/wkspc1/sample.js
extensions/copilot/test/scenarios/test-cli/wkspc1/stringUtils.js
extensions/copilot/test/scenarios/test-cli/wkspc1/utils.js
extensions/copilot/test/scenarios/test-cli/wkspc2/foobar.js
extensions/copilot/test/scenarios/test-scenario-1/bar.js
extensions/copilot/test/scenarios/test-scenario-1/fib.js
extensions/copilot/test/scenarios/test-system/puppeteer.js
extensions/copilot/test/simulation/fixtures/edit/issue-6329/math.js
extensions/copilot/test/simulation/fixtures/edit/issue-7282/math.js
extensions/copilot/test/simulation/fixtures/editing/math.js
extensions/copilot/test/simulation/fixtures/gen-twice-issue-3597/new.js
extensions/copilot/test/simulation/fixtures/generate/issue-6956/.eslintrc.js
extensions/copilot/test/simulation/fixtures/multiFile/unicode-string-sequences/example.js
extensions/copilot/test/simulation/fixtures/multiFileEdit/two-edits/generate-command-ts.js
extensions/copilot/test/simulation/fixtures/review/binary-search-1.js
extensions/copilot/test/simulation/fixtures/review/binary-search-2.js
extensions/copilot/test/simulation/fixtures/tests/generate-jest/some/app.js
extensions/copilot/test/simulation/fixtures/tests/generate-jest/some/sum.js
extensions/copilot/test/simulation/fixtures/tests/generate-jest/some/sum.test.js
extensions/copilot/test/simulation/fixtures/tests/simple-js-proj copy/src/index.js
extensions/copilot/test/simulation/fixtures/tests/simple-js-proj/src/index.js
extensions/copilot/test/simulationExtension/extension.js
extensions/cpp/build/update-grammars.js
extensions/css-language-features/server/test/index.js
extensions/css-language-features/server/test/pathCompletionFixtures/.foo.js
extensions/css-language-features/server/test/pathCompletionFixtures/src/feature.js
extensions/css-language-features/server/test/pathCompletionFixtures/src/test.js
extensions/git-base/build/update-grammars.js
extensions/git/build/update-emoji.js
extensions/html-language-features/build/bundleTypeScriptLibraries.js
extensions/html-language-features/server/src/test/pathCompletionFixtures/.foo.js
extensions/html-language-features/server/src/test/pathCompletionFixtures/src/feature.js
extensions/html-language-features/server/src/test/pathCompletionFixtures/src/test.js
extensions/html-language-features/server/test/index.js
extensions/html/build/update-grammar.mjs
extensions/json/build/update-grammars.js
extensions/latex/build/update-grammars.js
extensions/less/build/update-grammar.js
extensions/media-preview/media/audioPreview.js
extensions/media-preview/media/imagePreview.js
extensions/media-preview/media/videoPreview.js
extensions/microsoft-authentication/packageMocks/dpapi/dpapi.js
extensions/microsoft-authentication/packageMocks/keytar/index.js
extensions/objective-c/build/update-grammars.js
extensions/php/build/update-grammar.mjs
extensions/postinstall.mjs
extensions/razor/build/update-grammar.mjs
extensions/rust/build/update-grammar.mjs
extensions/search-result/syntaxes/generateTMLanguage.js
extensions/sql/build/update-grammar.mjs
extensions/terminal-suggest/scripts/update-specs.js
extensions/theme-seti/build/update-icon-theme.js
extensions/typescript-basics/build/update-grammars.mjs
extensions/typescript-language-features/test-workspace/foojs.js
extensions/vscode-api-tests/testWorkspace/debug.js
extensions/vscode-api-tests/testWorkspace/far.js
extensions/vscode-api-tests/testWorkspace/worker.js
extensions/vscode-colorize-tests/test/colorize-fixtures/test.js
extensions/vscode-colorize-tests/test/colorize-fixtures/test6916.js
extensions/yaml/build/update-grammar.js
gulpfile.mjs
scripts/chat-simulation/common/perf-scenarios.js
scripts/chat-simulation/common/utils.js
scripts/chat-simulation/merge-ci-summary.js
scripts/chat-simulation/test-chat-mem-leaks.js
scripts/chat-simulation/test-chat-perf-regression.js
scripts/code-agent-host.js
scripts/code-perf.js
scripts/code-server.js
scripts/code-sessions-web.js
scripts/code-web.js
scripts/xterm-update.js
src/vs/base/browser/dompurify/dompurify.js
src/vs/base/common/marked/marked.js
src/vs/base/common/semver/semver.js
src/vs/base/test/common/filters.perf.data.js
src/vs/editor/test/node/diffing/fixtures/difficult-move/1.js
src/vs/editor/test/node/diffing/fixtures/difficult-move/2.js
src/vs/editor/test/node/diffing/fixtures/just-whitespace/1.js
src/vs/editor/test/node/diffing/fixtures/just-whitespace/2.js
src/vs/platform/files/test/node/fixtures/resolver/examples/company.js
src/vs/platform/files/test/node/fixtures/resolver/examples/conway.js
src/vs/platform/files/test/node/fixtures/resolver/examples/employee.js
src/vs/platform/files/test/node/fixtures/resolver/examples/small.js
src/vs/platform/files/test/node/fixtures/resolver/other/deep/company.js
src/vs/platform/files/test/node/fixtures/resolver/other/deep/conway.js
src/vs/platform/files/test/node/fixtures/resolver/other/deep/employee.js
src/vs/platform/files/test/node/fixtures/resolver/other/deep/small.js
src/vs/platform/files/test/node/fixtures/service/deep/company.js
src/vs/platform/files/test/node/fixtures/service/deep/conway.js
src/vs/platform/files/test/node/fixtures/service/deep/employee.js
src/vs/platform/files/test/node/fixtures/service/deep/small.js
src/vs/sessions/test/e2e/common.cjs
src/vs/sessions/test/e2e/extensions/sessions-e2e-mock/extension.js
src/vs/sessions/test/e2e/generate.cjs
src/vs/sessions/test/e2e/test.cjs
src/vs/workbench/contrib/webview/browser/pre/service-worker.js
src/vs/workbench/services/keybinding/test/node/linux_de_ch.js
src/vs/workbench/services/keybinding/test/node/linux_en_uk.js
src/vs/workbench/services/keybinding/test/node/linux_en_us.js
src/vs/workbench/services/keybinding/test/node/linux_ru.js
src/vs/workbench/services/keybinding/test/node/mac_de_ch.js
src/vs/workbench/services/keybinding/test/node/mac_en_us.js
src/vs/workbench/services/keybinding/test/node/mac_zh_hant.js
src/vs/workbench/services/keybinding/test/node/mac_zh_hant2.js
src/vs/workbench/services/keybinding/test/node/win_de_ch.js
src/vs/workbench/services/keybinding/test/node/win_en_us.js
src/vs/workbench/services/keybinding/test/node/win_por_ptb.js
src/vs/workbench/services/keybinding/test/node/win_ru.js
src/vs/workbench/services/search/test/node/fixtures/examples/NullPoinderException.js
src/vs/workbench/services/search/test/node/fixtures/examples/company.js
src/vs/workbench/services/search/test/node/fixtures/examples/employee.js
src/vs/workbench/services/search/test/node/fixtures/examples/small.js
test/automation/tools/copy-driver-definition.js
test/automation/tools/copy-package-version.js
test/integration/electron/testrunner.js
test/monaco/core.js
test/monaco/esm-check/esm-check.js
test/monaco/esm-check/index.js
test/monaco/runner.js
test/monaco/webpack.config.js
test/smoke/extensions/vscode-smoketest-ext-host/extension.js
test/smoke/test/index.js
test/unit/analyzeSnapshot.js
test/unit/assert.js
test/unit/browser/index.js
test/unit/coverage.js
test/unit/electron/index.js
test/unit/electron/preload.js
test/unit/electron/renderer.js
test/unit/fullJsonStreamReporter.js
test/unit/node/index.js
test/unit/reporter.js
Reference in New Issue View Git Blame Copy Permalink