github/meteor
1
1
Fork 0
mirror of https://github.com/meteor/meteor.git synced 2026-05-02 03:01:46 -04:00
Code Issues Packages Projects Releases Wiki Activity

fix a test supporting non-npm node_modules watching

Browse Source
This commit is contained in:
Nacho Codoñer
2025-04-24 11:03:25 +02:00
parent 1995445b7b
commit eaf78bca81
1 changed files with 17 additions and 16 deletions
Download Patch File Download Diff File Expand all files Collapse all files

33
tools/fs/safe-watcher.ts
View File

@@ -153,24 +153,25 @@ function shouldIgnorePath(absPath: string): boolean {
}
// For project node_modules: check if it's a direct node_modules/<package>
if (isWithinCwd && absPath.includes(`${cwd}/node_modules`)) {
// Check if it's a direct node_modules/<package> path
const relPath = absPath.substring(cwd.length + 1); // +1 for the slash
const relParts = relPath.split('/');
if (relParts.length >= 2 && relParts[0] === 'node_modules') {
// If it's a direct node_modules/<package>, check if it's a symlink
// We'll return false here (don't ignore) so that the code can later decide to use polling
// based on isSymbolicLink check in the watch function
if (relParts.length === 2 && isSymbolicLink(absPath)) {
return false;
if (isWithinCwd) {
if (absPath.includes(`${cwd}/node_modules`)) {
// Check if it's a direct node_modules/<package> path
const relPath = absPath.substring(cwd.length + 1); // +1 for the slash
const relParts = relPath.split('/');
if (relParts.length >= 2 && relParts[0] === 'node_modules') {
// If it's a direct node_modules/<package>, check if it's a symlink
// We'll return false here (don't ignore) so that the code can later decide to use polling
// based on isSymbolicLink check in the watch function
if (relParts.length === 2 && isSymbolicLink(absPath)) {
return false;
}
// Check if it's within a symlink root to not ignore
if (isWithinSymlinkRoot(absPath)) {
return false;
}
}
// Check if it's within a symlink root to not ignore
if (isWithinSymlinkRoot(absPath)) {
return false;
}
return true;
}
return true;
return false;
}
// For external node_modules: check if it's a direct node_modules/<package>