Skip to content

fix(security): add .npmrc with ignore-scripts=true#1083

Merged
yogeshchoudhary147 merged 2 commits intomainfrom
ci_ignore_scripts
Apr 10, 2026
Merged

fix(security): add .npmrc with ignore-scripts=true#1083
yogeshchoudhary147 merged 2 commits intomainfrom
ci_ignore_scripts

Conversation

@yogeshchoudhary147
Copy link
Copy Markdown
Contributor

@yogeshchoudhary147 yogeshchoudhary147 commented Apr 10, 2026

This prevents malicious postinstall scripts from running during npm install, both in CI and locally.

Changes

  • Added .npmrc with ignore-scripts=true
  • Removed .npmrc from .gitignore (auth tokens belong in ~/.npmrc, not project-level)
  • Added explicit npx cypress install in integration workflow (Cypress needs its binary downloaded via postinstall)

@yogeshchoudhary147 yogeshchoudhary147 requested a review from a team as a code owner April 10, 2026 05:25
@yogeshchoudhary147 yogeshchoudhary147 changed the title fix(security): add --ignore-scripts to CI npm installs fix(security): add .npmrc with ignore-scripts=true Apr 10, 2026
@yogeshchoudhary147 yogeshchoudhary147 merged commit 658bfca into main Apr 10, 2026
12 checks passed
@yogeshchoudhary147 yogeshchoudhary147 deleted the ci_ignore_scripts branch April 10, 2026 14:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants