[BUG] Production prune removes peerDeps that are also devDeps
Nobody has claimed this yet.
- Dominant language
- JavaScript
- Stars
- 10.1k
- Forks
- 4.7k
- Avg merge
- 2d 2h
- Merged PRs (30d)
- 19
Description
Is there an existing issue for this?
- I have searched the existing issues
This issue exists in the latest npm version
- I am using the latest npm
Current Behavior
The packages that are both peer and development packages are removed when pruning due to the omit=dev option being applied to all development dependencies.
I.E. the below script emits Expected dev and peer package not found was pruned when it was not supposd to be.
Expected Behavior
The packages that are both peer and development packages are kept due to the include=peer option overriding the omit=dev option.
I.E. the below script emits Test passed.
Steps To Reproduce
Here's a shell script that reproduces the problem:
#!/bin/sh
set -e
rm -rf node_modules package.json package-lock.json
npm -v # 10.2.4
# Part 1: Set up a package for demonstration.
npm init --init-author-name bugdemo --init-license UNLICENSED --init-module index.js --init-version 0.0.0 --yes
## Install a package as a development dependency.
npm i -D --package-lock-only @slidewave/gitignore-include@2 c8
## Add the one package as peer dependency.
jq '. + {peerDependencies:{"@slidewave/gitignore-include": ">=2", "chai": ">=1"}}' package.json > t && mv t package.json
## Update the package-lock to include the peer dependency data.
npm i -D --package-lock-only @slidewave/gitignore-include
# Part 2: Demonstrate the bug.
## Install all the packages.
ENV_NAME=build npm ci --unsafe-perm --no-fund --no-audit
## Verify existence of the expected packages.
if [ ! -e 'node_modules/c8/package.json' ]; then
echo "Expected dev-only package not found." >&2
exit 1
fi
if [ ! -e 'node_modules/@slidewave/gitignore-include' ]; then
echo "Expected dev and peer package not found." >&2
exit 1
fi
if [ ! -e 'node_modules/chai' ]; then
echo "Expected peer-only package not found." >&2
exit 1
fi
## Remove the dev packages, but leave the peer packages.
ENV_NAME= npm prune --omit=dev --include=peer --no-fund --no-audit
hadError=
if [ -e 'node_modules/c8/package.json' ]; then
echo "Expected dev-only package found when it should have been pruned." >&2
hadError=1
fi
if [ ! -e 'node_modules/chai/package.json' ]; then
echo "Expected peer-only package was pruned when it was not supposd to be." >&2
hadError=1
fi
if [ ! -e 'node_modules/@slidewave/gitignore-include/package.json' ]; then
echo "Expected dev and peer package not found was pruned when it was not supposd to be." >&2
hadError=1
fi
if [ ! "$hadError" ]; then
echo "Test passed."
fi
Environment
- npm: 10.2.4
- Node.js: v20.11.1
- OS Name: macOS 14.1.1
- System Model Name: Apple M1 Pro
- npm config:
; "user" config from /root/.npmrc
//npm.pkg.github.com/:_authToken = (protected)
//registry.npmjs.org/:_authToken = (protected)
; node bin location = /usr/local/share/nvm/versions/node/v20.11.1/bin/node
; node version = v20.11.1
; npm local prefix = /npmbug
; npm version = 10.2.4
; cwd = /npmbug
; HOME = /root
; Run `npm config ls -l` to show all defaults.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by running the shell reproduction in the issue with npm 10.2.4, focusing on the npm prune --omit=dev --include=peer command. Trace the npm CLI's pruning and dependency-omitting entry points to determine why a package classified as both peer and dev is removed. Done means dev-only packages are pruned while peer-only and peer-plus-dev packages remain.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript, nodejs, shell
- Domain
- cli, tooling
- Issue type
- Bug
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100