Skip to content

SK-2839: Pin dependency versions and add --ignore-scripts to harden against supply chain attacks#321

Open
saileshwar-skyflow wants to merge 1 commit into
release/26.6.0from
saileshwar/SK-2839-pin-versions-and-add-ignore-scripts
Open

SK-2839: Pin dependency versions and add --ignore-scripts to harden against supply chain attacks#321
saileshwar-skyflow wants to merge 1 commit into
release/26.6.0from
saileshwar/SK-2839-pin-versions-and-add-ignore-scripts

Conversation

@saileshwar-skyflow
Copy link
Copy Markdown
Collaborator

@saileshwar-skyflow saileshwar-skyflow commented Jun 3, 2026

Summary

  • Pin all package.json versions to exact lock-file versions — removes all ^ and ~ range specifiers from dependencies and devDependencies so the registry can never serve a different version than what was tested
  • Add --ignore-scripts to every CI npm install/npm ci call in .github/workflows/ — blocks lifecycle scripts (postinstall, preinstall, etc.) from executing at install time in CI
  • Create .npmrc with ignore-scripts=true as a project-level default, applying the same protection for local installs

Hardens the repo against the Mini Shai-Hulud / Shai-Hulud npm supply chain attack campaign.

Files changed

File Change
package.json 31 range specifiers (^/~) replaced with exact lock-file versions
.github/workflows/common-ci.yml npm installnpm install --ignore-scripts
.github/workflows/contract-tests.yml npm cinpm ci --ignore-scripts
.github/workflows/common-release.yml npm installnpm install --ignore-scripts
.npmrc Created with ignore-scripts=true

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 3, 2026

Gitleaks Findings: No secrets detected. Safe to proceed!

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 3, 2026

Semgrep Findings: Issues with Error level severity are found (Error is Highest severity in Semgrep), Please resolve the issues before merging.

@saileshwar-skyflow saileshwar-skyflow changed the base branch from main to release/26.6.0 June 3, 2026 08:41
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