Skip to content

feat(install-scripts): use install-scripts as the warning log title#9697

Merged
owlstronaut merged 1 commit into
npm:latestfrom
manzoorwanijk:feat/install-scripts-log-title
Jun 29, 2026
Merged

feat(install-scripts): use install-scripts as the warning log title#9697
owlstronaut merged 1 commit into
npm:latestfrom
manzoorwanijk:feat/install-scripts-log-title

Conversation

@manzoorwanijk

@manzoorwanijk manzoorwanijk commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

Renames the warning log title for install-script-policy messages from allow-scripts to install-scripts, so npm install prints npm warn install-scripts … to match the npm install-scripts command namespace.

Only the log title changes. The allowScripts package.json field, the allow-scripts config (and the command static params that declare it), and message-body text that names the config are all left as-is.

Affected emitters: reify-output.js (the post-install blocked-scripts summary), resolve-allow-scripts.js (policy precedence/validation warnings), warn-workspace-allow-scripts.js, and allow-scripts-writer.js.

References

Follows up #9629

BEFORE AFTER
Image Screenshot 2026-06-29 at 11 10 38 PM

@manzoorwanijk

Copy link
Copy Markdown
Contributor Author

@owlstronaut, it won't apply cleanly to release/v11, so here is the backport change ready #9698.

@JamieMagee JamieMagee 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.

LGTM 👍

@owlstronaut owlstronaut merged commit 42b12c2 into npm:latest Jun 29, 2026
27 checks passed
@manzoorwanijk manzoorwanijk deleted the feat/install-scripts-log-title branch June 30, 2026 01:31
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.

3 participants