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

@manzoorwanijkmanzoorwanijk 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

BEFOREAFTER
ImageScreenshot 2026-06-29 at 11 10 38 PM

@manzoorwanijk

Copy link
Copy Markdown
ContributorAuthor

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

@JamieMageeJamieMagee 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:latestJun 29, 2026
27 checks passed
@github-actionsgithub-actionsBot mentioned this pull request Jun 29, 2026
@manzoorwanijk
manzoorwanijk deleted the feat/install-scripts-log-title branch June 30, 2026 01:31
Sign up for freeto 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

@manzoorwanijk@owlstronaut@JamieMagee