@pnpm/fs.graceful-fs 1100.2.1 → 1100.2.2

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # @pnpm/fs.graceful-fs
 
+## 1100.2.2
+
+### Patch Changes
+
+- Installs in different projects that share a global virtual store no longer fail on Windows with `Access is denied` while repairing the same slot [#15114](https://github.com/pnpm/pnpm/issues/15114).
+
 ## 1100.2.1
 
 ### Patch Changes

package/lib/index.d.ts

@@ -31,3 +31,17 @@ export default _default;
  * that was there or the whole file replacing it.
  */
 export declare function renameFileWithRetry(src: string, dest: string): void;
+/**
+ * Reads `target`'s stats without following it, with the retry policy of
+ * {@link renameFileWithRetry}.
+ *
+ * A Windows path another process has just unlinked stays delete-pending until
+ * the last handle on it closes, and inspecting it fails with EPERM for as long
+ * as that lasts. Retrying lets the unlink land, so a caller that reads ENOENT
+ * as an absent entry sees the same absence POSIX shows it at once.
+ */
+export declare function lstatWithRetry(target: string): fs.Stats;
+/**
+ * Removes a file, with the retry policy of {@link renameFileWithRetry}.
+ */
+export declare function unlinkWithRetry(target: string): void;

package/lib/index.js

@@ -1,10 +1,10 @@
 import fs from 'node:fs';
 import util, { promisify } from 'node:util';
 import gfs from 'graceful-fs';
-const RENAME_RETRY_BUDGET_MS = 60_000;
+const FILE_LOCK_RETRY_BUDGET_MS = 60_000;
 const PERMISSION_DENIED_RETRY_BUDGET_MS = 1_000;
-const RENAME_RETRY_BACKOFF_CAP_MS = 100;
-const renameRetrySleepBuffer = new Int32Array(new SharedArrayBuffer(4));
+const FILE_LOCK_RETRY_BACKOFF_CAP_MS = 100;
+const fileLockRetrySleepBuffer = new Int32Array(new SharedArrayBuffer(4));
 export default {
     chmod: promisify(gfs.chmod),
     copyFile: promisify(gfs.copyFile),
@@ -57,16 +57,40 @@ function withEagainRetry(fn, maxRetries = 15) {
  * that was there or the whole file replacing it.
  */
 export function renameFileWithRetry(src, dest) {
+    withFileLockRetry(() => {
+        fs.renameSync(src, dest);
+    });
+}
+/**
+ * Reads `target`'s stats without following it, with the retry policy of
+ * {@link renameFileWithRetry}.
+ *
+ * A Windows path another process has just unlinked stays delete-pending until
+ * the last handle on it closes, and inspecting it fails with EPERM for as long
+ * as that lasts. Retrying lets the unlink land, so a caller that reads ENOENT
+ * as an absent entry sees the same absence POSIX shows it at once.
+ */
+export function lstatWithRetry(target) {
+    return withFileLockRetry(() => fs.lstatSync(target));
+}
+/**
+ * Removes a file, with the retry policy of {@link renameFileWithRetry}.
+ */
+export function unlinkWithRetry(target) {
+    withFileLockRetry(() => {
+        fs.unlinkSync(target);
+    });
+}
+function withFileLockRetry(operation) {
     const startedAt = Date.now();
     let backoffMs = 0;
-    let budgetMs = RENAME_RETRY_BUDGET_MS;
+    let budgetMs = FILE_LOCK_RETRY_BUDGET_MS;
     for (;;) {
         try {
-            fs.renameSync(src, dest);
-            return;
+            return operation();
         }
         catch (err) {
-            if (!isTransientRenameError(err))
+            if (!isTransientFileLockError(err))
                 throw err;
             if (err.code === 'EPERM' || err.code === 'EACCES')
                 budgetMs = Math.min(budgetMs, PERMISSION_DENIED_RETRY_BUDGET_MS);
@@ -74,14 +98,14 @@ export function renameFileWithRetry(src, dest) {
             if (remainingMs <= 0)
                 throw err;
             if (backoffMs > 0)
-                Atomics.wait(renameRetrySleepBuffer, 0, 0, Math.min(backoffMs, remainingMs));
+                Atomics.wait(fileLockRetrySleepBuffer, 0, 0, Math.min(backoffMs, remainingMs));
             if (Date.now() - startedAt >= budgetMs)
                 throw err;
-            backoffMs = Math.min(backoffMs + 10, RENAME_RETRY_BACKOFF_CAP_MS);
+            backoffMs = Math.min(backoffMs + 10, FILE_LOCK_RETRY_BACKOFF_CAP_MS);
         }
     }
 }
-function isTransientRenameError(err) {
+function isTransientFileLockError(err) {
     return process.platform === 'win32' &&
         util.types.isNativeError(err) &&
         'code' in err &&

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pnpm/fs.graceful-fs",
-  "version": "1100.2.1",
+  "version": "1100.2.2",
   "description": "Promisified graceful-fs",
   "keywords": [
     "pnpm",
@@ -31,7 +31,7 @@
   },
   "devDependencies": {
     "@jest/globals": "30.4.1",
-    "@pnpm/fs.graceful-fs": "1100.2.1",
+    "@pnpm/fs.graceful-fs": "1100.2.2",
     "@types/graceful-fs": "^4.1.9"
   },
   "engines": {