@socketsecurity/lib 5.9.0 → 5.9.1

package/CHANGELOG.md

@@ -5,6 +5,17 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [5.9.1](https://github.com/SocketDev/socket-lib/releases/tag/v5.9.1) - 2026-03-14
+
+### Fixed
+
+- **fs**: `safeDelete()` and `safeDeleteSync()` now properly implement retry logic
+  - Previously `maxRetries` was incorrectly passed as `concurrency` to del (parallelism, not retries)
+  - `safeDelete()` now wraps `deleteAsync()` with `pRetry()` for exponential backoff
+  - `safeDeleteSync()` implements sync retry loop with `Atomics.wait()` for non-blocking sleep
+  - Both use `backoffFactor: 2` (delay doubles each retry: 200ms → 400ms → 800ms by default)
+  - `maxRetries` and `retryDelay` options in `RemoveOptions` now work as documented
+
 ## [5.9.0](https://github.com/SocketDev/socket-lib/releases/tag/v5.9.0) - 2026-03-14
 
 ### Changed

package/dist/fs.js

@@ -53,6 +53,7 @@ module.exports = __toCommonJS(fs_exports);
 var import_process = require("./constants/process");
 var import_arrays = require("./arrays");
 var import_del = require("./external/del");
+var import_promises = require("./promises");
 var import_globs = require("./globs");
 var import_parse = require("./json/parse");
 var import_objects = require("./objects");
@@ -527,12 +528,23 @@ async function safeDelete(filepath, options) {
       shouldForce = true;
     }
   }
-  await (0, import_del.deleteAsync)(patterns, {
-    concurrency: opts.maxRetries || defaultRemoveOptions.maxRetries,
-    dryRun: false,
-    force: shouldForce,
-    onlyFiles: false
-  });
+  const maxRetries = opts.maxRetries ?? defaultRemoveOptions.maxRetries;
+  const retryDelay = opts.retryDelay ?? defaultRemoveOptions.retryDelay;
+  await (0, import_promises.pRetry)(
+    async () => {
+      await (0, import_del.deleteAsync)(patterns, {
+        dryRun: false,
+        force: shouldForce,
+        onlyFiles: false
+      });
+    },
+    {
+      retries: maxRetries,
+      baseDelayMs: retryDelay,
+      backoffFactor: 2,
+      signal: opts.signal
+    }
+  );
 }
 function safeDeleteSync(filepath, options) {
   const opts = { __proto__: null, ...options };
@@ -557,12 +569,30 @@ function safeDeleteSync(filepath, options) {
       shouldForce = true;
     }
   }
-  (0, import_del.deleteSync)(patterns, {
-    concurrency: opts.maxRetries || defaultRemoveOptions.maxRetries,
-    dryRun: false,
-    force: shouldForce,
-    onlyFiles: false
-  });
+  const maxRetries = opts.maxRetries ?? defaultRemoveOptions.maxRetries;
+  const retryDelay = opts.retryDelay ?? defaultRemoveOptions.retryDelay;
+  let lastError;
+  let delay = retryDelay;
+  for (let attempt = 0; attempt <= maxRetries; attempt++) {
+    try {
+      (0, import_del.deleteSync)(patterns, {
+        dryRun: false,
+        force: shouldForce,
+        onlyFiles: false
+      });
+      return;
+    } catch (error) {
+      lastError = error;
+      if (attempt < maxRetries) {
+        const waitMs = delay;
+        Atomics.wait(new Int32Array(new SharedArrayBuffer(4)), 0, 0, waitMs);
+        delay *= 2;
+      }
+    }
+  }
+  if (lastError) {
+    throw lastError;
+  }
 }
 async function safeMkdir(path, options) {
   const fs = /* @__PURE__ */ getFs();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@socketsecurity/lib",
-  "version": "5.9.0",
+  "version": "5.9.1",
   "packageManager": "pnpm@10.32.1",
   "license": "MIT",
   "description": "Core utilities and infrastructure for Socket.dev security tools",
@@ -734,7 +734,7 @@
     "@socketregistry/is-unicode-supported": "1.0.5",
     "@socketregistry/packageurl-js": "1.3.5",
     "@socketregistry/yocto-spinner": "1.0.25",
-    "@socketsecurity/lib-stable": "npm:@socketsecurity/lib@5.8.2",
+    "@socketsecurity/lib-stable": "npm:@socketsecurity/lib@5.9.0",
     "@types/node": "24.9.2",
     "@typescript/native-preview": "7.0.0-dev.20250920.1",
     "@vitest/coverage-v8": "4.0.3",