Skip to content

ci: replace direct npm flags with env presets - #1327

Merged
antongolub merged 2 commits into
google:mainfrom
antongolub:ci-npm-env-preset
Aug 20, 2025
Merged

ci: replace direct npm flags with env presets#1327
antongolub merged 2 commits into
google:mainfrom
antongolub:ci-npm-env-preset

Conversation

@antongolub

@antongolub antongolub commented Aug 20, 2025

Copy link
Copy Markdown
Collaborator
  • 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

antongolub commented Aug 20, 2025

Copy link
Copy Markdown
Collaborator Author

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
Copy Markdown

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