Skip to content

Use npm ci --ignore-scripts in shared workflows#98

Merged
fumikito merged 2 commits into
mainfrom
fix/npm-ci-ignore-scripts
Apr 8, 2026
Merged

Use npm ci --ignore-scripts in shared workflows#98
fumikito merged 2 commits into
mainfrom
fix/npm-ci-ignore-scripts

Conversation

@fumikito
Copy link
Copy Markdown
Contributor

Summary

  • Replace npm install with npm ci --ignore-scripts in npm.yml, lint.yml
  • Prevents execution of postinstall/lifecycle scripts during CI, mitigating supply chain attacks (e.g. axios vulnerability)
  • No functional impact: CI doesn't need husky hooks or other lifecycle scripts
  • Note: playground-preview-build.yml only exists on the feature/playground-preview branch, so the --ignore-scripts change for that file should be applied there separately

Test plan

  • npm.yml: Verify lint/build jobs still pass in consuming plugins
  • lint.yml: Verify YAML lint job still passes

…rity

Co-authored-by: Claude <noreply@anthropic.com>
@fumikito fumikito enabled auto-merge March 31, 2026 08:52
@fumikito fumikito merged commit e66fcbe into main Apr 8, 2026
9 checks passed
@fumikito fumikito deleted the fix/npm-ci-ignore-scripts branch April 8, 2026 07:35
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.

1 participant