Skip to content

Fix audit alerts and enforce frozen lockfile (#33266)#33302

Merged
alexslavr merged 1 commit into24_2from
lavrov/audit-alerts-24_2
Apr 16, 2026
Merged

Fix audit alerts and enforce frozen lockfile (#33266)#33302
alexslavr merged 1 commit into24_2from
lavrov/audit-alerts-24_2

Conversation

@alexslavr
Copy link
Copy Markdown
Contributor

Cherry-pick of #33266

@alexslavr alexslavr requested a review from Copilot April 16, 2026 10:25
@alexslavr alexslavr self-assigned this Apr 16, 2026
@alexslavr alexslavr requested a review from a team as a code owner April 16, 2026 10:25
@alexslavr alexslavr added the 24_2 label Apr 16, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Cherry-pick of #33266 to make dependency installs deterministic in CI (frozen lockfile) and reduce audit/lockfile drift by switching to lockfile-enforcing install commands.

Changes:

  • Replace pnpm install with pnpm install --frozen-lockfile across multiple GitHub Actions workflows.
  • Update the documentation build step to use npm ci instead of npm install.
  • Remove the custom setup-chrome-headless-shell composite action and adjust workflow/job wiring (notably in lint.yml).

Reviewed changes

Copilot reviewed 23 out of 23 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
tools/scripts/build-all.ts Uses npm ci for deterministic installs when building/injecting docs.
apps/angular/src/polyfills.ts Removes outdated “run npm install” hints from commented polyfill imports.
.github/workflows/wrapper_tests_e2e.yml Enforces frozen lockfile for wrapper e2e jobs.
.github/workflows/update_version.yml Enforces frozen lockfile during version bump automation.
.github/workflows/ts_declarations.yml Enforces frozen lockfile for declaration generation/validation jobs.
.github/workflows/themebuilder_tests.yml Enforces frozen lockfile for themebuilder tests.
.github/workflows/testcafe_tests.yml Enforces frozen lockfile for TestCafe pipelines.
.github/workflows/styles.yml Enforces frozen lockfile for style/lint jobs.
.github/workflows/run-testcafe-on-gh-pages.yml Enforces frozen lockfile before running GH Pages TestCafe suite.
.github/workflows/renovation.yml Enforces frozen lockfile for Renovation build steps.
.github/workflows/qunit_tests-renovation.yml Enforces frozen lockfile for QUnit renovation workflow.
.github/workflows/qunit_tests-additional-renovation.yml Enforces frozen lockfile for additional renovation QUnit runs.
.github/workflows/publish-demos.yml Enforces frozen lockfile during demos publishing workflow steps.
.github/workflows/playgrounds_tests.yml Enforces frozen lockfile for playground builds/tests.
.github/workflows/packages_publishing.yml Enforces frozen lockfile for package publishing jobs.
.github/workflows/lint.yml Enforces frozen lockfile and removes the dedicated pnpm_lock job from the workflow.
.github/workflows/demos_visual_tests_frameworks.yml Enforces frozen lockfile for multi-framework visual test workflow.
.github/workflows/demos_unit_tests.yml Enforces frozen lockfile for demos unit tests.
.github/workflows/default_workflow.yml Enforces frozen lockfile in default workflow installs.
.github/workflows/codeql.yml Enforces frozen lockfile prior to CodeQL-related steps.
.github/workflows/build_all.yml Enforces frozen lockfile for the build-all workflow.
.github/actions/setup-chrome-headless-shell/action.yml Removes the composite action that installed chrome-headless-shell.
.github/actions/run-qunit-tests/action.yml Enforces frozen lockfile for QUnit composite action installs.

Comment thread .github/actions/run-qunit-tests/action.yml
@alexslavr alexslavr force-pushed the lavrov/audit-alerts-24_2 branch from c4b4480 to f80d88c Compare April 16, 2026 10:39
IlyaKhD
IlyaKhD previously approved these changes Apr 16, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 22 out of 22 changed files in this pull request and generated no new comments.

@alexslavr alexslavr merged commit ec31dfb into 24_2 Apr 16, 2026
74 of 77 checks passed
@alexslavr alexslavr deleted the lavrov/audit-alerts-24_2 branch April 16, 2026 14:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants