oh-my-node-modules 1.3.2 → 1.3.3

package/dist/cli.js

@@ -3484,7 +3484,10 @@ async function isNodeModulesInUse(path) {
 
 // src/deletion.ts
 import { promises as fs3 } from "fs";
-import { join as join3 } from "path";
+import { join as join3, resolve } from "path";
+import { exec as exec2 } from "child_process";
+import { promisify as promisify2 } from "util";
+var execAsync2 = promisify2(exec2);
 async function deleteSelectedNodeModules(nodeModules, options, onProgress) {
   const selected = nodeModules.filter((nm) => nm.selected);
   const result = {
@@ -3547,8 +3550,26 @@ async function deleteNodeModules(nodeModules, options) {
     if (options.dryRun) {
       detail.success = true;
     } else {
-      await fs3.rm(nodeModules.path, { recursive: true, force: true });
-      detail.success = true;
+      try {
+        if (options.force) {
+          await forceDelete(nodeModules.path);
+        } else {
+          await fs3.rm(nodeModules.path, { recursive: true, force: true });
+        }
+        detail.success = true;
+      } catch (rmError) {
+        const errorCode = rmError.code;
+        const errorMessage = rmError.message;
+        if (errorCode === "EPERM" || errorCode === "EACCES") {
+          detail.error = "Permission denied - run as Administrator or check file permissions";
+        } else if (errorCode === "EBUSY") {
+          detail.error = "Directory in use - close any programs using these files";
+        } else if (errorMessage?.includes("ENOTEMPTY")) {
+          detail.error = "Directory not empty - may contain read-only files. Try using --force";
+        } else {
+          detail.error = errorMessage || "Unknown error during deletion";
+        }
+      }
     }
     detail.durationMs = Date.now() - startTime;
   } catch (error) {
@@ -3557,16 +3578,55 @@ async function deleteNodeModules(nodeModules, options) {
   }
   return detail;
 }
-async function verifyNodeModules(path) {
+async function forceDelete(dirPath) {
+  const isWindows = process.platform === "win32";
+  try {
+    await fs3.rm(dirPath, { recursive: true, force: true });
+    return;
+  } catch {}
+  try {
+    await makeWritableRecursive(dirPath);
+    await fs3.rm(dirPath, { recursive: true, force: true });
+    return;
+  } catch {}
+  try {
+    if (isWindows) {
+      const windowsPath = dirPath.length > 240 ? `\\\\?\\${resolve(dirPath)}` : dirPath;
+      await execAsync2(`rd /s /q "${windowsPath}"`, { timeout: 30000 });
+    } else {
+      await execAsync2(`rm -rf "${dirPath}"`, { timeout: 30000 });
+    }
+    return;
+  } catch {}
+  const tempPath = `${dirPath}.old.${Date.now()}`;
+  await fs3.rename(dirPath, tempPath);
+  try {
+    await fs3.rm(tempPath, { recursive: true, force: true });
+  } catch {}
+}
+async function makeWritableRecursive(dirPath) {
   try {
-    const parts = path.split("/");
-    if (parts[parts.length - 1] !== "node_modules") {
+    const entries = await fs3.readdir(dirPath, { withFileTypes: true });
+    for (const entry of entries) {
+      const fullPath = join3(dirPath, entry.name);
+      if (entry.isDirectory()) {
+        await makeWritableRecursive(fullPath);
+      }
+      await fs3.chmod(fullPath, 511).catch(() => {});
+    }
+    await fs3.chmod(dirPath, 511).catch(() => {});
+  } catch {}
+}
+async function verifyNodeModules(dirPath) {
+  try {
+    const baseName = dirPath.replace(/\\/g, "/").split("/").pop();
+    if (baseName !== "node_modules") {
       return false;
     }
-    const entries = await fs3.readdir(path);
+    const entries = await fs3.readdir(dirPath);
     let hasSubdirs = false;
     for (const entry of entries) {
-      const entryPath = join3(path, entry);
+      const entryPath = join3(dirPath, entry);
       try {
         const stats = await fs3.stat(entryPath);
         if (stats.isDirectory()) {
@@ -3575,7 +3635,8 @@ async function verifyNodeModules(path) {
         }
       } catch {}
     }
-    const parentPath = path.replace(/\/node_modules$/, "").replace(/\\node_modules$/, "");
+    const normalizedPath = dirPath.replace(/\\/g, "/");
+    const parentPath = normalizedPath.replace(/\/node_modules$/, "");
     const hasPackageJson = await fileExists(join3(parentPath, "package.json"));
     return hasSubdirs || hasPackageJson;
   } catch {
@@ -3606,7 +3667,7 @@ function formatItem(item) {
   const warning = !isPending && item.sizeCategory === "huge" ? import_picocolors2.default.red(" ⚠") : "";
   return `${size}  ${item.projectName}${warning}  [${age}]`;
 }
-async function interactiveMode(rootPath) {
+async function interactiveMode(rootPath, force = false) {
   pe(import_picocolors2.default.cyan("oh-my-node-modules"));
   const s = _2();
   s.start("Scanning for node_modules...");
@@ -3682,10 +3743,22 @@ ${import_picocolors2.default.gray("Total:")} ${import_picocolors2.default.white(
         const result2 = await deleteSelectedNodeModules(items, {
           dryRun: false,
           yes: true,
+          force,
           checkRunningProcesses: false,
           showProgress: false
         });
         ds.stop(`Deleted ${result2.successful}/${result2.totalAttempted} directories`);
+        const failedDeletions = result2.details.filter((d2) => !d2.success && d2.error);
+        if (failedDeletions.length > 0) {
+          console.log(import_picocolors2.default.red(`
+Failed to delete ${failedDeletions.length} directories:`));
+          for (const detail of failedDeletions.slice(0, 5)) {
+            console.log(import_picocolors2.default.red(`  • ${detail.nodeModules.projectName}: ${detail.error}`));
+          }
+          if (failedDeletions.length > 5) {
+            console.log(import_picocolors2.default.red(`  ... and ${failedDeletions.length - 5} more`));
+          }
+        }
         ge(import_picocolors2.default.green(`Freed ${result2.formattedBytesFreed}`));
         return;
       }
@@ -3735,7 +3808,7 @@ ${import_picocolors2.default.gray("Total size:")} ${import_picocolors2.default.c
     process.exit(1);
   }
 }
-async function autoDeleteMode(rootPath, minSize, dryRun = false) {
+async function autoDeleteMode(rootPath, minSize, dryRun = false, force = false) {
   const s = _2();
   s.start("Scanning...");
   try {
@@ -3781,8 +3854,9 @@ Dry run - no files deleted.`));
       const ds = _2();
       ds.start("Deleting...");
       const result2 = await deleteSelectedNodeModules(items, {
-        dryRun: false,
+        dryRun,
         yes: true,
+        force,
         checkRunningProcesses: false,
         showProgress: false
       });
@@ -3796,13 +3870,13 @@ Dry run - no files deleted.`));
   }
 }
 var program2 = new Command;
-program2.name("onm").description("Find and clean up node_modules directories").version(getVersion()).argument("<path>", "Directory to scan").option("--scan", "quick scan mode (no interactive UI)").option("--auto", "auto-delete mode with filters").option("--min-size <size>", "minimum size in bytes for auto mode").option("--dry-run", "simulate deletion without actually deleting").option("--json", "output as JSON").action(async (path, options) => {
+program2.name("onm").description("Find and clean up node_modules directories").version(getVersion()).argument("<path>", "Directory to scan").option("--scan", "quick scan mode (no interactive UI)").option("--auto", "auto-delete mode with filters").option("--min-size <size>", "minimum size in bytes for auto mode").option("--dry-run", "simulate deletion without actually deleting").option("--force", "force delete (handles read-only files and long paths)").option("--json", "output as JSON").action(async (path, options) => {
   if (options.scan) {
     await quickScanMode(path, options.json);
   } else if (options.auto) {
-    await autoDeleteMode(path, options.minSize, options.dryRun);
+    await autoDeleteMode(path, options.minSize, options.dryRun, options.force);
   } else {
-    await interactiveMode(path);
+    await interactiveMode(path, options.force);
   }
 });
 program2.parse();

package/dist/index.js

@@ -765,7 +765,10 @@ async function quickScan(rootPath) {
 }
 // src/deletion.ts
 import { promises as fs3 } from "fs";
-import { join as join3 } from "path";
+import { join as join3, resolve } from "path";
+import { exec as exec2 } from "child_process";
+import { promisify as promisify2 } from "util";
+var execAsync2 = promisify2(exec2);
 async function deleteSelectedNodeModules(nodeModules, options, onProgress) {
   const selected = nodeModules.filter((nm) => nm.selected);
   const result = {
@@ -828,8 +831,26 @@ async function deleteNodeModules(nodeModules, options) {
     if (options.dryRun) {
       detail.success = true;
     } else {
-      await fs3.rm(nodeModules.path, { recursive: true, force: true });
-      detail.success = true;
+      try {
+        if (options.force) {
+          await forceDelete(nodeModules.path);
+        } else {
+          await fs3.rm(nodeModules.path, { recursive: true, force: true });
+        }
+        detail.success = true;
+      } catch (rmError) {
+        const errorCode = rmError.code;
+        const errorMessage = rmError.message;
+        if (errorCode === "EPERM" || errorCode === "EACCES") {
+          detail.error = "Permission denied - run as Administrator or check file permissions";
+        } else if (errorCode === "EBUSY") {
+          detail.error = "Directory in use - close any programs using these files";
+        } else if (errorMessage?.includes("ENOTEMPTY")) {
+          detail.error = "Directory not empty - may contain read-only files. Try using --force";
+        } else {
+          detail.error = errorMessage || "Unknown error during deletion";
+        }
+      }
     }
     detail.durationMs = Date.now() - startTime;
   } catch (error) {
@@ -838,16 +859,55 @@ async function deleteNodeModules(nodeModules, options) {
   }
   return detail;
 }
-async function verifyNodeModules(path) {
+async function forceDelete(dirPath) {
+  const isWindows = process.platform === "win32";
+  try {
+    await fs3.rm(dirPath, { recursive: true, force: true });
+    return;
+  } catch {}
+  try {
+    await makeWritableRecursive(dirPath);
+    await fs3.rm(dirPath, { recursive: true, force: true });
+    return;
+  } catch {}
+  try {
+    if (isWindows) {
+      const windowsPath = dirPath.length > 240 ? `\\\\?\\${resolve(dirPath)}` : dirPath;
+      await execAsync2(`rd /s /q "${windowsPath}"`, { timeout: 30000 });
+    } else {
+      await execAsync2(`rm -rf "${dirPath}"`, { timeout: 30000 });
+    }
+    return;
+  } catch {}
+  const tempPath = `${dirPath}.old.${Date.now()}`;
+  await fs3.rename(dirPath, tempPath);
+  try {
+    await fs3.rm(tempPath, { recursive: true, force: true });
+  } catch {}
+}
+async function makeWritableRecursive(dirPath) {
+  try {
+    const entries = await fs3.readdir(dirPath, { withFileTypes: true });
+    for (const entry of entries) {
+      const fullPath = join3(dirPath, entry.name);
+      if (entry.isDirectory()) {
+        await makeWritableRecursive(fullPath);
+      }
+      await fs3.chmod(fullPath, 511).catch(() => {});
+    }
+    await fs3.chmod(dirPath, 511).catch(() => {});
+  } catch {}
+}
+async function verifyNodeModules(dirPath) {
   try {
-    const parts = path.split("/");
-    if (parts[parts.length - 1] !== "node_modules") {
+    const baseName = dirPath.replace(/\\/g, "/").split("/").pop();
+    if (baseName !== "node_modules") {
       return false;
     }
-    const entries = await fs3.readdir(path);
+    const entries = await fs3.readdir(dirPath);
     let hasSubdirs = false;
     for (const entry of entries) {
-      const entryPath = join3(path, entry);
+      const entryPath = join3(dirPath, entry);
       try {
         const stats = await fs3.stat(entryPath);
         if (stats.isDirectory()) {
@@ -856,7 +916,8 @@ async function verifyNodeModules(path) {
         }
       } catch {}
     }
-    const parentPath = path.replace(/\/node_modules$/, "").replace(/\\node_modules$/, "");
+    const normalizedPath = dirPath.replace(/\\/g, "/");
+    const parentPath = normalizedPath.replace(/\/node_modules$/, "");
     const hasPackageJson = await fileExists(join3(parentPath, "package.json"));
     return hasSubdirs || hasPackageJson;
   } catch {

package/package.json

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