fix(arborist): match allowScripts keys for local paths #9929

Merged
github-actions[bot] opened 5:21pm on August 27, 2026 wanted to merge 1 commit into npm/cli release/v11 from
backport/v11/9914

Is there an existing issue for this?

  • [x] I have searched the existing issues

This issue exists in the latest npm version

  • [x] I am using the latest npm

This is not just a request to bump a dependency for a CVE

  • [x] This is not solely a request to bump a dependency for a CVE

Current Behavior

I have a local tgz package (a local fork of electron-winstaller) with install scripts in devDependencies. npm v12.0.2 always blocks its install scripts no matter what I have added in allowScirpts inside package.json. I have tried json { "allowScripts": { "electron-winstaller": true, "[email protected]": true, "file:path/to/electron-winstaller-5.3.0.tgz": true, "file:C:\\absolute\\path\\to\\electron-winstaller-5.3.0.tgz": true, "file:C:/absolute/path/to/electron-winstaller-5.3.0.tgz": true } } which still results in `` npm warn install-scripts 1 package had install scripts blocked because they are not covered by allowScripts: npm warn install-scripts [email protected] (install: node ./script/select-7z-arch.js) npm warn install-scripts npm warn install-scripts Runnpm install-scripts lsto review, ornpm install-scripts approve <pkg>` to allow.

```

Below is Claude's analysis:

npm 12's new install-scripts gate matches policy keys against node.resolved using npm-package-arg (npa). For a file: dependency, your lockfile stores the relative form (file:path/to/electron-winstaller-5.3.0.tgz), but when npm loads the actual installed tree (what install/install-scripts ls actually checks against), it re-derives resolved through consistentResolve() (consistent-resolve.js:22-26), which on Windows produces:

file:C:\absolute\path\to\electron-winstaller-5.3.0.tgz โ€” absolute, with backslashes, "file:" glued directly onto them.

The matcher (script-allowed.js:330-333) requires an exact string match against a key parsed by npa(). But npa() always normalizes its saveSpec to forward slashes, and its fetchSpec never carries the file: prefix.

Expected Behavior

npm should allow the scripts to run. Ideally with just the package name match, or relative path match.

Steps To Reproduce

run npm install to install a local tgz npm package with install scripts on Windows.

Environment

  • npm: 12.0.2
  • Node.js: 22.23.2
  • OS Name: Windows 10
  • System Model Name: Intel NUC
  • npm config: ini strict-ssl = false allow-git = "all" min-release-age = 7
github-actions[bot]
Diff Delta:
0
Classified as:  Bug Fix

github-actions[bot]'s Description of Work #9900 [BUG] allowScripts does not work for local tarball on Windows

Backport of #9914 to release/v11.

0 total changed files
You've made it to the end of the scroll.
Thus concludes todayโ€™s episode of โ€œJust One Small Changeโ€