Fix "pattern is too long" error when running np (#633)

sindresorhus · Mar 19, 2022 · 20117a4 · 20117a4
1 parent b1939c7
commit 20117a4
Showing 1 changed file with 7 additions and 3 deletions.
diff --git a/source/npm/util.js b/source/npm/util.js
@@ -154,16 +154,20 @@ function npmignoreExistsInPackageRootDir() {
 	return fs.existsSync(path.resolve(rootDir, '.npmignore'));
 }
 
+function excludeGitAndNodeModulesPaths(singlePath) {
+	return !singlePath.startsWith('.git/') && !singlePath.startsWith('node_modules/');
+}
+
 async function getFilesIgnoredByDotnpmignore(pkg, fileList) {
-	const allowList = await ignoreWalker({
+	const allowList = (await ignoreWalker({
 		path: pkgDir.sync(),
 		ignoreFiles: ['.npmignore']
-	});
+	})).filter(singlePath => excludeGitAndNodeModulesPaths(singlePath));
 	return fileList.filter(minimatch.filter(getIgnoredFilesGlob(allowList, pkg.directories), {matchBase: true, dot: true}));
 }
 
 function filterFileList(globArray, fileList) {
-	const globString = globArray.length > 1 ? `{${globArray}}` : globArray[0];
+	const globString = globArray.length > 1 ? `{${globArray.filter(singlePath => excludeGitAndNodeModulesPaths(singlePath))}}` : globArray[0];
 	return fileList.filter(minimatch.filter(globString, {matchBase: true, dot: true})); // eslint-disable-line unicorn/no-fn-reference-in-iterator
 }