[BUG] npm ci fails with ERR_FS_EISDIR for node_modules/.bin #6051
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
Can't npm ci if there's an existing .bin directory in node_modules
Expected Behavior
It deletes node_modules without crashing
Steps To Reproduce
Use this package.json
{
"name": "npm-test",
"version": "1.0.0",
"description": "",
"main": "index.js",
"scripts": {
"test": "echo \"Error: no test specified\" && exit 1"
},
"author": "",
"license": "ISC",
"dependencies": {
"eslint": "^8.31.0"
}
}npm cinpm ci- It fails the second time
npm ERR! code ERR_FS_EISDIR
npm ERR! syscall rm
npm ERR! path /Users/justintoman/code/npm-test/node_modules/.bin
npm ERR! errno 21
npm ERR! Path is a directory: rm returned EISDIR (is a directory) /Users/justintoman/code/npm-test/node_modules/.bin
npm ERR! A complete log of this run can be found in:
npm ERR! /Users/justintoman/.npm/_logs/2023-01-12T21_14_19_211Z-debug-0.log
Environment
- npm: 9.3.0
- Node.js: 18.12.1
- OS Name: MacOS Venture 13.1
- System Model Name: Macbook Pro
- npm config:
; node bin location = /Users/justintoman/.nvm/versions/node/v18.12.1/bin/node
; node version = v18.12.1
; npm local prefix = /Users/justintoman/code/npm-test
; npm version = 9.3.0
; cwd = /Users/justintoman/code/npm-test
; HOME = /Users/justintoman
; Run `npm config ls -l` to show all defaults.
Activity
eleonoratennyson commentedon Jan 13, 2023
👍 also having this issue when running
npm cion circleci with v9.3.0. 9.2.0 runs clean on the same machineeleonoratennyson commentedon Jan 13, 2023
PR with a fix here #6052
iSuslov commentedon Jan 13, 2023
+1 I'm surprised dev team doesn't have a test for it.
Pin npm to 9.2.0 because of the issue npm/cli#6051
Pin CircleCI npm to 9.2.0 (#2334)
jamesshaw1987 commentedon Jan 13, 2023
Agreed. Added a PR (#6054) that adds a test to @burkel24's PR (#6052).
Pin npm to 9.2.0 because of the issue npm/cli#6051
Pin npm to 9.2.0 because of the issue npm/cli#6051 (#466)
Pin npm to 9.2.0 based on Tom's suggestion as a fix for build issues.
Pin npm to 9.2.0 based on Tom's suggestion as a fix for build issues. (…
Pinned npm to 9.2.0 due to npm ci issue npm/cli#6051
Revert "Pin npm to 9.2.0 because of the issue npm/cli#6051 (#466)"
Revert "Pin npm to 9.2.0 because of the issue npm/cli#6051 (#466)" (#471