@youdie006/prodex 0.40.2 → 0.40.4

package/dist/blocker-report.js

@@ -53,8 +53,14 @@ export function buildBlockerReport(input) {
         const existing = groups.get(key);
         if (existing) {
             existing.count += 1;
-            if (at > existing.lastSeen)
+            // The example belongs to the record the row dates itself by. Keeping the
+            // first one seen printed a cause's oldest wording next to its newest
+            // timestamp: a group holding both `has no "Pro" step` and a pre-fix
+            // `has no "<a model>" step` showed the fixed one as what is failing now.
+            if (at > existing.lastSeen) {
                 existing.lastSeen = at;
+                existing.example = consult.blocker.message;
+            }
             existing.repos.set(consult.repo, (existing.repos.get(consult.repo) ?? 0) + 1);
         }
         else {

package/dist/browser-send-lock.js

@@ -1,4 +1,4 @@
-import { link, mkdir, open, readFile, rm } from "node:fs/promises";
+import { link, mkdir, open, readFile, rm, stat } from "node:fs/promises";
 import os from "node:os";
 import path from "node:path";
 // One visible-browser send at a time per machine: the dedicated Chrome is a
@@ -91,6 +91,15 @@ async function tryAcquire(file) {
         await rm(temp, { force: true }).catch(() => undefined);
     }
 }
+async function lockFileExists(file) {
+    try {
+        await stat(file);
+        return true;
+    }
+    catch {
+        return false;
+    }
+}
 /**
  * Serialize visible-browser sends across processes. Waits up to waitMs for a
  * live holder to finish (0 = fail fast); a lock whose holder process is dead
@@ -112,6 +121,17 @@ export async function withBrowserSendLock(waitMs, onWait, fn) {
             if (current?.pid === holder?.pid) {
                 await rm(file, { force: true }).catch(() => undefined);
             }
+            // A reap that did not actually remove the file - another user's lock in a
+            // shared directory, or a directory sitting in its place - used to retry
+            // immediately, skipping both the sleep and the deadline: a hot loop that
+            // never returned and never timed out. Waiting here costs a reap that
+            // raced nothing; not waiting costs the process.
+            if (await lockFileExists(file)) {
+                if (Date.now() >= deadline) {
+                    throw new Error(`A prodex browser send lock at ${file} is held by nothing and could not be removed, so no send can start. Delete that file and retry.`);
+                }
+                await new Promise((resolve) => setTimeout(resolve, 2_000));
+            }
             continue;
         }
         if (Date.now() >= deadline) {