Skip to content

Nprmc - #182

Open
matdave wants to merge 2 commits into
mainfrom
nprmc
Open

Nprmc#182
matdave wants to merge 2 commits into
mainfrom
nprmc

Conversation

@matdave

@matdave matdave commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

What does it do?

Add npmrc file

Why is it needed?

Prevents automated script running

@matdave
matdave requested review from theboxer and a lite review from Copilot August 12, 2026 19:10

Copilot AI left a comment

Copy link
Copy Markdown

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 introduces repository-level npm configuration intended to harden installs (notably by preventing lifecycle scripts from running) and also updates package-lock.json with a broad set of transitive dependency/version changes.

Changes:

  • Add a new .npmrc with security-focused npm configuration (e.g., ignore-scripts=true, engine-strict=true).
  • Update package-lock.json with widespread dependency resolution/version changes (including multiple @babel/* packages and related transitive packages).

Reviewed changes

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

File Description
.npmrc Adds npm install policy settings aimed at reducing risk from lifecycle scripts and enforcing engines constraints.
package-lock.json Large lockfile refresh with many transitive dependency updates beyond the .npmrc addition.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread .npmrc
Comment on lines +4 to +5
# Do not run any lifecycle hook scripts such as postinstall for packages
ignore-scripts=true
Comment thread .npmrc
Comment on lines +13 to +14
# Require strict npm version to comply with the security policies in this file
engine-strict=true
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.

2 participants