@pnpm/fs.graceful-fs 1100.2.0 → 1100.2.1

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # @pnpm/fs.graceful-fs
 
+## 1100.2.1
+
+### Patch Changes
+
+- Windows filesystem operations now retry permission errors for up to one second. Permanent permission errors previously delayed failure by a minute. Sharing and lock violations retain their one-minute retry budget [pnpm/pnpm#14682](https://github.com/pnpm/pnpm/issues/14682).
+
 ## 1100.2.0
 
 ### Minor Changes

package/lib/index.d.ts

@@ -21,9 +21,10 @@ declare const _default: {
 };
 export default _default;
 /**
- * Renames `src` over `dest`, waiting out a Windows sharing violation — an
- * EPERM, EACCES or EBUSY from whoever else holds the file open — for up to a
- * minute before rethrowing it. Every other error is thrown right away.
+ * Renames `src` over `dest`, retrying Windows EBUSY errors for up to a minute.
+ * EPERM and EACCES have a one-second budget because they can also indicate
+ * permanent permission or destination conflicts. Other errors are thrown
+ * right away.
  *
  * `dest` is never removed to make room for the rename: a concurrent install may
  * still be reading that dirent, and a reader has to see either the whole file

package/lib/index.js

@@ -2,6 +2,7 @@ import fs from 'node:fs';
 import util, { promisify } from 'node:util';
 import gfs from 'graceful-fs';
 const RENAME_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));
 export default {
@@ -46,9 +47,10 @@ function withEagainRetry(fn, maxRetries = 15) {
     };
 }
 /**
- * Renames `src` over `dest`, waiting out a Windows sharing violation — an
- * EPERM, EACCES or EBUSY from whoever else holds the file open — for up to a
- * minute before rethrowing it. Every other error is thrown right away.
+ * Renames `src` over `dest`, retrying Windows EBUSY errors for up to a minute.
+ * EPERM and EACCES have a one-second budget because they can also indicate
+ * permanent permission or destination conflicts. Other errors are thrown
+ * right away.
  *
  * `dest` is never removed to make room for the rename: a concurrent install may
  * still be reading that dirent, and a reader has to see either the whole file
@@ -57,16 +59,24 @@ function withEagainRetry(fn, maxRetries = 15) {
 export function renameFileWithRetry(src, dest) {
     const startedAt = Date.now();
     let backoffMs = 0;
+    let budgetMs = RENAME_RETRY_BUDGET_MS;
     for (;;) {
         try {
             fs.renameSync(src, dest);
             return;
         }
         catch (err) {
-            if (!isTransientRenameError(err) || Date.now() - startedAt >= RENAME_RETRY_BUDGET_MS)
+            if (!isTransientRenameError(err))
+                throw err;
+            if (err.code === 'EPERM' || err.code === 'EACCES')
+                budgetMs = Math.min(budgetMs, PERMISSION_DENIED_RETRY_BUDGET_MS);
+            const remainingMs = budgetMs - (Date.now() - startedAt);
+            if (remainingMs <= 0)
                 throw err;
             if (backoffMs > 0)
-                Atomics.wait(renameRetrySleepBuffer, 0, 0, backoffMs);
+                Atomics.wait(renameRetrySleepBuffer, 0, 0, Math.min(backoffMs, remainingMs));
+            if (Date.now() - startedAt >= budgetMs)
+                throw err;
             backoffMs = Math.min(backoffMs + 10, RENAME_RETRY_BACKOFF_CAP_MS);
         }
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pnpm/fs.graceful-fs",
-  "version": "1100.2.0",
+  "version": "1100.2.1",
   "description": "Promisified graceful-fs",
   "keywords": [
     "pnpm",
@@ -30,7 +30,8 @@
     "graceful-fs": "^4.2.11"
   },
   "devDependencies": {
-    "@pnpm/fs.graceful-fs": "1100.2.0",
+    "@jest/globals": "30.4.1",
+    "@pnpm/fs.graceful-fs": "1100.2.1",
     "@types/graceful-fs": "^4.1.9"
   },
   "engines": {
@@ -40,8 +41,9 @@
     "preset": "@pnpm/jest-config"
   },
   "scripts": {
-    "lint": "eslint \"src/**/*.ts\"",
-    "test": "pn compile",
-    "compile": "tsgo --build && pn lint --fix"
+    "lint": "eslint \"src/**/*.ts\" \"test/**/*.ts\"",
+    "test": "pn compile && pn .test",
+    "compile": "tsgo --build && pn lint --fix",
+    ".test": "cross-env NODE_OPTIONS=\"$NODE_OPTIONS --experimental-vm-modules --disable-warning=ExperimentalWarning --disable-warning=DEP0169\" jest"
   }
 }