Skip to content

module: add --experimental-strip-private-modules #63869

Open
marco-ippolito wants to merge 2 commits into
nodejs:mainfrom
marco-ippolito:strip-private-node-modules
Open

module: add --experimental-strip-private-modules #63869
marco-ippolito wants to merge 2 commits into
nodejs:mainfrom
marco-ippolito:strip-private-node-modules

Conversation

@marco-ippolito

@marco-ippolito marco-ippolito commented Jun 12, 2026

Copy link
Copy Markdown
Member

Followup of #63853
Adds a flag to allow type stripping inside node_modules ONLY if private.
I think I covered #63853 (comment) concerns about smuggling private package.json's in a non private package

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/config
  • @nodejs/loaders
  • @nodejs/typescript

@nodejs-github-bot nodejs-github-bot added c++ Issues and PRs that require attention from people who are familiar with C++. lib / src Issues and PRs related to general changes in the lib or src directory. needs-ci PRs that need a full CI run. labels Jun 12, 2026
Signed-off-by: Marco Ippolito <marcoippolito54@gmail.com>
@marco-ippolito marco-ippolito force-pushed the strip-private-node-modules branch from ac9e626 to 003fda2 Compare June 12, 2026 08:59
@marco-ippolito marco-ippolito force-pushed the strip-private-node-modules branch from 003fda2 to 88c3881 Compare June 12, 2026 09:02

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c++ Issues and PRs that require attention from people who are familiar with C++. lib / src Issues and PRs related to general changes in the lib or src directory. needs-ci PRs that need a full CI run.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants