Merge pull request #1 from uwe-schwarz/feature/fix-dev-path-exclusion

fix: prevent false positives when excluding /dev paths

Author: codingstark-dev

index.ts

@@ -682,11 +682,8 @@ class BunKill {
   }
 
   private shouldSkipDirectory(dirPath: string): boolean {
-    const skipPatterns = [
+    const absoluteSkipRoots = [
       "/System",
-      "/Library/Application Support",
-      "/Library/Frameworks",
-      "/Applications",
       "/private",
       "/dev",
       "/proc",
@@ -705,6 +702,15 @@ class BunKill {
       "/opt/homebrew",
       "/usr/local/bin",
       "/usr/local/sbin",
+    ];
+
+    const pathContainsSkipPatterns = [
+      "/Library/Application Support",
+      "/Library/Frameworks",
+      "/Applications",
+    ];
+
+    const nameSkipPatterns = [
       ".photolibrary",
       ".photoslibrary",
       ".photoboothlibrary",
@@ -743,9 +749,27 @@ class BunKill {
       return false;
     }
 
-    return skipPatterns.some((pattern) =>
-      dirPath.includes(pattern) ||
-      dirPath.toLowerCase().includes(pattern.toLowerCase())
+    const normalizedPath = dirPath.toLowerCase();
+    const isAbsoluteSkip = absoluteSkipRoots.some((root) => {
+      const normalizedRoot = root.toLowerCase();
+      return normalizedPath === normalizedRoot ||
+        normalizedPath.startsWith(`${normalizedRoot}/`);
+    });
+
+    if (isAbsoluteSkip) {
+      return true;
+    }
+
+    if (
+      pathContainsSkipPatterns.some((pattern) =>
+        normalizedPath.includes(pattern.toLowerCase())
+      )
+    ) {
+      return true;
+    }
+
+    return nameSkipPatterns.some((pattern) =>
+      normalizedPath.includes(pattern.toLowerCase())
     );
   }