Skip to content

refactor: adding allowScripts for npm v12#468

Merged
spoletijuan merged 1 commit into
v3.4.0from
refactor/npmv12
Jun 26, 2026
Merged

refactor: adding allowScripts for npm v12#468
spoletijuan merged 1 commit into
v3.4.0from
refactor/npmv12

Conversation

@annipi

@annipi annipi commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

Copilot AI review requested due to automatic review settings June 23, 2026 13:49
@github-actions

Copy link
Copy Markdown

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

OpenSSF Scorecard

PackageVersionScoreDetails

Scanned Files

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the project’s npm metadata/config to support newer npm behavior by explicitly disabling install scripts for @scarf/scarf, aligning Node engine metadata, and making minor package metadata adjustments.

Changes:

  • Bumps the package version from 4.0.0 to 4.0.1.
  • Adds allowScripts to disable @scarf/scarf install scripts (commonly used for telemetry) and removes .npmrc’s scripts-prepend-node-path.
  • Updates package-lock.json metadata (including Node engine requirement) and reformats overrides for readability.

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.

File Description
package.json Version bump + adds allowScripts to disable @scarf/scarf scripts; reformats overrides.
package-lock.json Updates lockfile metadata to 4.0.1 and aligns the root package Node engine to >=20.0.0.
.npmrc Removes scripts-prepend-node-path, leaving only package-lock=true.

@spoletijuan spoletijuan merged commit 9c3a295 into v3.4.0 Jun 26, 2026
4 checks passed
@spoletijuan spoletijuan deleted the refactor/npmv12 branch June 26, 2026 13:32
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.

5 participants