Skip to content

Conversation

@antongolub
Copy link
Collaborator

@antongolub antongolub commented Aug 20, 2025

  • Tests pass
  • Appropriate changes to README are included in PR

@antongolub antongolub merged commit 67743df into google:main Aug 20, 2025
29 checks passed
@antongolub antongolub deleted the ci-npm-env-preset branch August 20, 2025 12:56
@antongolub
Copy link
Collaborator Author

antongolub commented Aug 20, 2025

Hey @souhailaS,

This case might be useful for your research. If multiple npm i / npm ci calls are performed in the same environment combinations, there is no point in auditing vulnerabilities each time (npm install does this by default). It is enough in one place, we do this as a separate step, so we deactivated implicit auditing for the remaining calls via npm config preset. Internally, the audit operation is quite resource-intensive: advisories are requested in several batches and applied to the current dependency tree, matching CVE-affected version ranges for each nested package.

@souhailaS
Copy link

thanks a lot for the insight @antongolub !

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.

2 participants