oh-my-node-modules 1.2.9 → 1.3.2

package/README.md

@@ -17,7 +17,7 @@ npm install -g oh-my-node-modules
 ### Interactive mode
 
 ```bash
-onm                    # Start in current directory
+onm .                  # Start in current directory
 onm ~/projects         # Scan specific directory
 ```
 
@@ -37,16 +37,16 @@ Keyboard shortcuts:
 
 ```bash
 # Quick scan report
-onm --scan
+onm --scan .
 
 # JSON output
-onm --scan --json
+onm --scan --json .
 
 # Auto-delete large node_modules
-onm --auto --min-size 500mb --yes
+onm --auto --min-size 500mb --yes .
 
 # Preview what would be deleted
-onm --auto --min-size 1gb --dry-run
+onm --auto --min-size 1gb --dry-run .
 ```
 
 ## Features

package/dist/cli.js

@@ -3080,6 +3080,9 @@ function calculateStatistics(items) {
 }
 function shouldExcludePath(path, patterns) {
   return patterns.some((pattern) => {
+    if (pattern === "**/.*" || pattern === ".*") {
+      return /(^|\/)\.[^\/]+($|\/)/.test(path);
+    }
     const regexPattern = pattern.replace(/\*\*/g, "{{GLOBSTAR}}").replace(/\*/g, "[^/]*").replace(/\?/g, ".").replace(/\{\{GLOBSTAR\}\}/g, ".*");
     const regex = new RegExp(regexPattern, "i");
     return regex.test(path);
@@ -3189,8 +3192,7 @@ async function getFastDirectorySize(dirPath) {
 }
 
 // src/scanner.ts
-var DEFAULT_CONCURRENCY = 5;
-var SIZE_CALCULATION_CONCURRENCY = 3;
+var SIZE_CALCULATION_CONCURRENCY = 4;
 async function findRepoRoot(startPath) {
   let currentPath = startPath;
   const root = process.platform === "win32" ? "C:\\" : "/";
@@ -3343,59 +3345,69 @@ async function scanForNodeModules(options, onProgress, lazy = false) {
   const pathsToScan = [
     { path: options.rootPath, depth: 0 }
   ];
-  const scanLimit = pLimit(DEFAULT_CONCURRENCY);
-  const sizeLimit = pLimit(SIZE_CALCULATION_CONCURRENCY);
+  const foundNodeModules = [];
   let processedCount = 0;
   let totalEstimate = 1;
-  let foundCount = 0;
   while (pathsToScan.length > 0) {
-    const batch = pathsToScan.splice(0, Math.min(pathsToScan.length, DEFAULT_CONCURRENCY * 2));
-    const scanPromises = batch.map(({ path: currentPath, depth }) => scanLimit(async () => {
-      if (visitedPaths.has(currentPath))
-        return;
-      if (options.maxDepth !== undefined && depth > options.maxDepth)
-        return;
-      if (shouldExcludePath(currentPath, options.excludePatterns))
-        return;
-      visitedPaths.add(currentPath);
-      result.directoriesScanned++;
+    const { path: currentPath, depth } = pathsToScan.shift();
+    if (visitedPaths.has(currentPath))
+      continue;
+    if (options.maxDepth !== undefined && depth > options.maxDepth)
+      continue;
+    if (shouldExcludePath(currentPath, options.excludePatterns))
+      continue;
+    visitedPaths.add(currentPath);
+    result.directoriesScanned++;
+    try {
+      const entries = await fs2.readdir(currentPath, { withFileTypes: true });
+      const hasNodeModules = entries.some((entry) => entry.isDirectory() && entry.name === "node_modules");
+      if (hasNodeModules) {
+        const nodeModulesPath = join2(currentPath, "node_modules");
+        foundNodeModules.push({ nodeModulesPath, projectPath: currentPath });
+        if (onProgress) {
+          onProgress(Math.min(100, Math.round(processedCount / totalEstimate * 100)), foundNodeModules.length);
+        }
+      }
+      for (const entry of entries) {
+        if (entry.isDirectory() && entry.name !== "node_modules" && !entry.name.startsWith(".")) {
+          const subPath = join2(currentPath, entry.name);
+          if (!shouldExcludePath(subPath, options.excludePatterns)) {
+            pathsToScan.push({ path: subPath, depth: depth + 1 });
+            totalEstimate++;
+          }
+        }
+      }
+    } catch (error) {
+      const errorMessage = error instanceof Error ? error.message : String(error);
+      result.errors.push(`Error scanning ${currentPath}: ${errorMessage}`);
+    }
+    processedCount++;
+    if (onProgress && foundNodeModules.length === 0) {
+      onProgress(Math.min(100, Math.round(processedCount / totalEstimate * 100)), 0);
+    }
+  }
+  if (foundNodeModules.length > 0) {
+    const sizeLimit = pLimit(SIZE_CALCULATION_CONCURRENCY);
+    const analysisPromises = foundNodeModules.map(({ nodeModulesPath, projectPath }) => sizeLimit(async () => {
       try {
-        const entries = await fs2.readdir(currentPath, { withFileTypes: true });
-        const hasNodeModules = entries.some((entry) => entry.isDirectory() && entry.name === "node_modules");
-        if (hasNodeModules) {
-          const nodeModulesPath = join2(currentPath, "node_modules");
-          const info = await sizeLimit(() => analyzeNodeModules(nodeModulesPath, currentPath, lazy));
+        const info = await analyzeNodeModules(nodeModulesPath, projectPath, lazy);
+        if (!lazy) {
           const passesFilter = (!options.minSizeBytes || info.sizeBytes >= options.minSizeBytes) && (!options.olderThanDays || getAgeInDays2(info.lastModified) >= options.olderThanDays);
-          if (passesFilter || lazy) {
+          if (passesFilter) {
             result.nodeModules.push(info);
-            foundCount++;
-            if (onProgress) {
-              onProgress(Math.min(100, Math.round(processedCount / totalEstimate * 100)), foundCount);
-            }
-          }
-        }
-        for (const entry of entries) {
-          if (entry.isDirectory() && entry.name !== "node_modules" && !entry.name.startsWith(".")) {
-            const subPath = join2(currentPath, entry.name);
-            if (!shouldExcludePath(subPath, options.excludePatterns)) {
-              pathsToScan.push({ path: subPath, depth: depth + 1 });
-              totalEstimate++;
-            }
           }
+        } else {
+          result.nodeModules.push(info);
         }
       } catch (error) {
         const errorMessage = error instanceof Error ? error.message : String(error);
-        result.errors.push(`Error scanning ${currentPath}: ${errorMessage}`);
-      }
-      processedCount++;
-      if (onProgress) {
-        onProgress(Math.min(100, Math.round(processedCount / totalEstimate * 100)), foundCount);
+        result.errors.push(`Error analyzing ${nodeModulesPath}: ${errorMessage}`);
       }
     }));
-    await Promise.all(scanPromises);
+    await Promise.all(analysisPromises);
   }
   if (onProgress) {
-    onProgress(100, foundCount);
+    onProgress(100, result.nodeModules.length);
   }
   return result;
 }

package/dist/index.js

@@ -306,6 +306,9 @@ function calculateStatistics(items) {
 }
 function shouldExcludePath(path, patterns) {
   return patterns.some((pattern) => {
+    if (pattern === "**/.*" || pattern === ".*") {
+      return /(^|\/)\.[^\/]+($|\/)/.test(path);
+    }
     const regexPattern = pattern.replace(/\*\*/g, "{{GLOBSTAR}}").replace(/\*/g, "[^/]*").replace(/\?/g, ".").replace(/\{\{GLOBSTAR\}\}/g, ".*");
     const regex = new RegExp(regexPattern, "i");
     return regex.test(path);
@@ -423,8 +426,7 @@ async function getFastDirectorySize(dirPath) {
 }
 
 // src/scanner.ts
-var DEFAULT_CONCURRENCY = 5;
-var SIZE_CALCULATION_CONCURRENCY = 3;
+var SIZE_CALCULATION_CONCURRENCY = 4;
 async function findRepoRoot(startPath) {
   let currentPath = startPath;
   const root = process.platform === "win32" ? "C:\\" : "/";
@@ -577,59 +579,69 @@ async function scanForNodeModules(options, onProgress, lazy = false) {
   const pathsToScan = [
     { path: options.rootPath, depth: 0 }
   ];
-  const scanLimit = pLimit(DEFAULT_CONCURRENCY);
-  const sizeLimit = pLimit(SIZE_CALCULATION_CONCURRENCY);
+  const foundNodeModules = [];
   let processedCount = 0;
   let totalEstimate = 1;
-  let foundCount = 0;
   while (pathsToScan.length > 0) {
-    const batch = pathsToScan.splice(0, Math.min(pathsToScan.length, DEFAULT_CONCURRENCY * 2));
-    const scanPromises = batch.map(({ path: currentPath, depth }) => scanLimit(async () => {
-      if (visitedPaths.has(currentPath))
-        return;
-      if (options.maxDepth !== undefined && depth > options.maxDepth)
-        return;
-      if (shouldExcludePath(currentPath, options.excludePatterns))
-        return;
-      visitedPaths.add(currentPath);
-      result.directoriesScanned++;
+    const { path: currentPath, depth } = pathsToScan.shift();
+    if (visitedPaths.has(currentPath))
+      continue;
+    if (options.maxDepth !== undefined && depth > options.maxDepth)
+      continue;
+    if (shouldExcludePath(currentPath, options.excludePatterns))
+      continue;
+    visitedPaths.add(currentPath);
+    result.directoriesScanned++;
+    try {
+      const entries = await fs2.readdir(currentPath, { withFileTypes: true });
+      const hasNodeModules = entries.some((entry) => entry.isDirectory() && entry.name === "node_modules");
+      if (hasNodeModules) {
+        const nodeModulesPath = join2(currentPath, "node_modules");
+        foundNodeModules.push({ nodeModulesPath, projectPath: currentPath });
+        if (onProgress) {
+          onProgress(Math.min(100, Math.round(processedCount / totalEstimate * 100)), foundNodeModules.length);
+        }
+      }
+      for (const entry of entries) {
+        if (entry.isDirectory() && entry.name !== "node_modules" && !entry.name.startsWith(".")) {
+          const subPath = join2(currentPath, entry.name);
+          if (!shouldExcludePath(subPath, options.excludePatterns)) {
+            pathsToScan.push({ path: subPath, depth: depth + 1 });
+            totalEstimate++;
+          }
+        }
+      }
+    } catch (error) {
+      const errorMessage = error instanceof Error ? error.message : String(error);
+      result.errors.push(`Error scanning ${currentPath}: ${errorMessage}`);
+    }
+    processedCount++;
+    if (onProgress && foundNodeModules.length === 0) {
+      onProgress(Math.min(100, Math.round(processedCount / totalEstimate * 100)), 0);
+    }
+  }
+  if (foundNodeModules.length > 0) {
+    const sizeLimit = pLimit(SIZE_CALCULATION_CONCURRENCY);
+    const analysisPromises = foundNodeModules.map(({ nodeModulesPath, projectPath }) => sizeLimit(async () => {
       try {
-        const entries = await fs2.readdir(currentPath, { withFileTypes: true });
-        const hasNodeModules = entries.some((entry) => entry.isDirectory() && entry.name === "node_modules");
-        if (hasNodeModules) {
-          const nodeModulesPath = join2(currentPath, "node_modules");
-          const info = await sizeLimit(() => analyzeNodeModules(nodeModulesPath, currentPath, lazy));
+        const info = await analyzeNodeModules(nodeModulesPath, projectPath, lazy);
+        if (!lazy) {
           const passesFilter = (!options.minSizeBytes || info.sizeBytes >= options.minSizeBytes) && (!options.olderThanDays || getAgeInDays2(info.lastModified) >= options.olderThanDays);
-          if (passesFilter || lazy) {
+          if (passesFilter) {
             result.nodeModules.push(info);
-            foundCount++;
-            if (onProgress) {
-              onProgress(Math.min(100, Math.round(processedCount / totalEstimate * 100)), foundCount);
-            }
-          }
-        }
-        for (const entry of entries) {
-          if (entry.isDirectory() && entry.name !== "node_modules" && !entry.name.startsWith(".")) {
-            const subPath = join2(currentPath, entry.name);
-            if (!shouldExcludePath(subPath, options.excludePatterns)) {
-              pathsToScan.push({ path: subPath, depth: depth + 1 });
-              totalEstimate++;
-            }
           }
+        } else {
+          result.nodeModules.push(info);
         }
       } catch (error) {
         const errorMessage = error instanceof Error ? error.message : String(error);
-        result.errors.push(`Error scanning ${currentPath}: ${errorMessage}`);
-      }
-      processedCount++;
-      if (onProgress) {
-        onProgress(Math.min(100, Math.round(processedCount / totalEstimate * 100)), foundCount);
+        result.errors.push(`Error analyzing ${nodeModulesPath}: ${errorMessage}`);
       }
     }));
-    await Promise.all(scanPromises);
+    await Promise.all(analysisPromises);
   }
   if (onProgress) {
-    onProgress(100, foundCount);
+    onProgress(100, result.nodeModules.length);
   }
   return result;
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "oh-my-node-modules",
-  "version": "1.2.9",
+  "version": "1.3.2",
   "description": "Visualize, analyze, and clean up node_modules directories to reclaim disk space",
   "type": "module",
   "main": "./dist/index.js",