wowdump 0.2.0 → 0.2.1

package/dist/process-log-lock.js

@@ -2,7 +2,7 @@ import { createHash, randomUUID } from "node:crypto";
 import { mkdir, readFile, readdir, rename, rm, stat, utimes, writeFile } from "node:fs/promises";
 const DEFAULT_RETRY_DELAY_MS = 20;
 const DEFAULT_STALE_AFTER_MS = 2_000;
-const DEFAULT_TRANSIENT_RETRIES = 3;
+const DEFAULT_TRANSIENT_RETRIES = 50;
 const defaultFileSystem = { mkdir, readFile, readdir, rename, rm, stat, utimes, writeFile };
 export async function acquireProcessLogLock(file, options = {}) {
     const fileSystem = { ...defaultFileSystem, ...options.fileSystem };
@@ -11,6 +11,7 @@ export async function acquireProcessLogLock(file, options = {}) {
     const retryDelayMs = options.retryDelayMs ?? DEFAULT_RETRY_DELAY_MS;
     const staleAfterMs = Math.max(options.staleAfterMs ?? DEFAULT_STALE_AFTER_MS, 2_000);
     const transientRetries = options.transientRetries ?? DEFAULT_TRANSIENT_RETRIES;
+    const operationTimeoutMs = options.timeoutMs ?? 10_000;
     const platform = options.platform ?? process.platform;
     const ownerPid = options.ownerPid ?? process.pid;
     const token = (options.tokenFactory ?? randomUUID)();
@@ -18,7 +19,7 @@ export async function acquireProcessLogLock(file, options = {}) {
     const ownerName = `owner.${tokenHash}.json`;
     const ownerPath = `${file}/${ownerName}`;
     const candidate = `${file}.candidate-${ownerPid}-${tokenHash}`;
-    const deadline = now() + (options.timeoutMs ?? 10_000);
+    const deadline = now() + operationTimeoutMs;
     const isProcessAlive = options.isProcessAlive ?? processIsAlive;
     while (now() <= deadline) {
         if (await publishCandidate(file, candidate, ownerName, ownerPid, token, fileSystem, now, platform)) {
@@ -31,15 +32,16 @@ export async function acquireProcessLogLock(file, options = {}) {
                 released = true;
                 clearInterval(heartbeat);
                 const claimName = `claim.${tokenHash}.json`;
+                const releaseDeadline = now() + operationTimeoutMs;
                 try {
-                    await fileSystem.rename(ownerPath, `${file}/${claimName}`);
+                    await renameWithTransientWindowsRetry(ownerPath, `${file}/${claimName}`, fileSystem, platform, retryDelayMs, transientRetries, releaseDeadline, now, sleep);
                 }
                 catch (error) {
                     if (errorCode(error) === "ENOENT")
                         return;
                     throw error;
                 }
-                await quarantineClaimed(file, ownerPid, tokenHash, "release", fileSystem, platform, retryDelayMs, transientRetries, sleep);
+                await quarantineClaimed(file, ownerPid, tokenHash, "release", fileSystem, platform, retryDelayMs, transientRetries, releaseDeadline, now, sleep);
             };
         }
         const state = await inspectState(file, fileSystem);
@@ -49,7 +51,7 @@ export async function acquireProcessLogLock(file, options = {}) {
             const claimed = now() - state.mtimeMs >= staleAfterMs
                 && await claimLegacy(file, ownerPid, tokenHash, fileSystem);
             if (claimed) {
-                await quarantineClaimed(file, ownerPid, tokenHash, "legacy", fileSystem, platform, retryDelayMs, transientRetries, sleep);
+                await quarantineClaimed(file, ownerPid, tokenHash, "legacy", fileSystem, platform, retryDelayMs, transientRetries, deadline, now, sleep);
                 continue;
             }
         }
@@ -59,8 +61,8 @@ export async function acquireProcessLogLock(file, options = {}) {
             if (reclaimable) {
                 const claimName = `claim.${tokenHash}.json`;
                 try {
-                    await fileSystem.rename(`${file}/${state.name}`, `${file}/${claimName}`);
-                    await quarantineClaimed(file, ownerPid, tokenHash, "stale", fileSystem, platform, retryDelayMs, transientRetries, sleep);
+                    await renameWithTransientWindowsRetry(`${file}/${state.name}`, `${file}/${claimName}`, fileSystem, platform, retryDelayMs, transientRetries, deadline, now, sleep);
+                    await quarantineClaimed(file, ownerPid, tokenHash, "stale", fileSystem, platform, retryDelayMs, transientRetries, deadline, now, sleep);
                     continue;
                 }
                 catch (error) {
@@ -141,22 +143,34 @@ async function claimLegacy(file, ownerPid, tokenHash, fileSystem) {
         throw error;
     }
 }
-async function quarantineClaimed(file, ownerPid, tokenHash, reason, fileSystem, platform, retryDelayMs, transientRetries, sleep) {
+async function quarantineClaimed(file, ownerPid, tokenHash, reason, fileSystem, platform, retryDelayMs, transientRetries, deadline, now, sleep) {
     const quarantine = `${file}.${reason}-${ownerPid}-${tokenHash}`;
+    try {
+        await renameWithTransientWindowsRetry(file, quarantine, fileSystem, platform, retryDelayMs, transientRetries, deadline, now, sleep);
+    }
+    catch (error) {
+        if (errorCode(error) === "ENOENT")
+            return;
+        throw error;
+    }
+    await fileSystem.rm(quarantine, { recursive: true, force: true });
+}
+async function renameWithTransientWindowsRetry(oldPath, newPath, fileSystem, platform, retryDelayMs, transientRetries, deadline, now, sleep) {
+    let firstTransientError;
     for (let attempt = 0;; attempt += 1) {
         try {
-            await fileSystem.rename(file, quarantine);
-            break;
+            await fileSystem.rename(oldPath, newPath);
+            return;
         }
         catch (error) {
-            if (errorCode(error) === "ENOENT")
-                return;
-            if (!isTransientWindowsLockError(error, platform) || attempt >= transientRetries)
+            if (!isTransientWindowsLockError(error, platform))
                 throw error;
+            firstTransientError ??= error;
+            if (attempt >= transientRetries || now() + retryDelayMs > deadline)
+                throw firstTransientError;
             await sleep(retryDelayMs);
         }
     }
-    await fileSystem.rm(quarantine, { recursive: true, force: true });
 }
 async function touch(path, fileSystem, now) {
     const date = new Date(now());

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "wowdump",
-  "version": "0.2.0",
+  "version": "0.2.1",
   "description": "Build-aware World of Warcraft runtime analysis MCP server backed by Frida",
   "type": "module",
   "bin": {