agentic-workflow-manager 3.13.2 → 3.13.3

package/dist/src/core/journal/process.js

@@ -275,17 +275,51 @@ function spawnStructured(argv, cwd, nonce, extraEnv = {}) {
  *  rompia el invariante "JAMAS safe sin evidencia" (safeToReplace
  *  devolvia 'safe' para un proceso vivo). process.kill(pid,0) evita la
  *  capa de emulacion por completo. */
+/** Sleep sincronico REAL, multiplataforma, sin depender de un binario externo
+ *  (`sleep` no existe nativamente en win32 — ver el comentario de sleepSync
+ *  mas arriba). `Atomics.wait` sobre un buffer compartido bloquea el hilo
+ *  actual durante `ms` sin abrir ningun subproceso; funciona identico en
+ *  cualquier plataforma que soporte Node. Usado SOLO por pidExistsNative
+ *  para el reintento acotado de abajo — nunca en un hot-path de alta
+ *  frecuencia. */
+function sleepMsSync(ms) {
+    Atomics.wait(new Int32Array(new SharedArrayBuffer(4)), 0, 0, ms);
+}
+/** Reintento acotado (R6, post-mortem de CI real): dos corridas reales de
+ *  windows-latest sobre el MISMO commit (uno vía `release.yml`, uno vía
+ *  `ci.yml`, ambos re-corriendo el suite completo desde cero) dieron
+ *  resultados DISTINTOS para el mismo test — `pidExistsNative` reportando
+ *  ESRCH para un proceso recien spawneado por el propio test, genuinamente
+ *  vivo — evidencia directa de una condicion de carrera transitoria
+ *  especifica de esta plataforma/runner, no un bug determinista (el codigo
+ *  no cambio entre ambas corridas). Un solo intento en el hot-path exacto
+ *  "acabo de spawnear esto, ¿esta vivo?" no le da tiempo al SO a que el pid
+ *  sea consultable via OpenProcess bajo carga pesada del runner. Reintentar
+ *  con una pausa breve ANTES de declarar ESRCH definitivo no debilita el
+ *  invariante "jamas muerte sin evidencia" — un proceso genuinamente muerto
+ *  sigue reportando ESRCH en el reintento; esto solo absorbe el falso
+ *  negativo transitorio. Costo maximo: ~150ms, y SOLO en la rama que ya iba
+ *  a declarar "no existe" — el camino feliz (proceso vivo, exito inmediato)
+ *  no paga nada. */
+const PID_EXISTS_RETRY_ATTEMPTS = 3;
+const PID_EXISTS_RETRY_DELAY_MS = 50;
 function pidExistsNative(pid) {
-    try {
-        process.kill(pid, 0);
-        return true;
-    }
-    catch (error) {
-        // ESRCH = el SO confirmo que el pid no existe. Cualquier otro
-        // codigo (ej. EPERM: existe pero sin permiso de senializarlo) NO
-        // es evidencia de muerte — falla a favor de "vivo" (R2.1).
-        return error.code !== 'ESRCH';
+    for (let attempt = 0; attempt < PID_EXISTS_RETRY_ATTEMPTS; attempt++) {
+        try {
+            process.kill(pid, 0);
+            return true;
+        }
+        catch (error) {
+            // ESRCH = el SO confirmo que el pid no existe. Cualquier otro
+            // codigo (ej. EPERM: existe pero sin permiso de senializarlo) NO
+            // es evidencia de muerte — falla a favor de "vivo" (R2.1).
+            if (error.code !== 'ESRCH')
+                return true;
+            if (attempt < PID_EXISTS_RETRY_ATTEMPTS - 1)
+                sleepMsSync(PID_EXISTS_RETRY_DELAY_MS);
+        }
     }
+    return false;
 }
 /** Vivo Y con la MISMA identidad — tupla completa, nunca PID solo (R2.1,
  *  bloqueador 6): startTime + pgid + digest de ps args.

package/dist/tests/core/journal/process.test.js

@@ -237,6 +237,22 @@ describe('process identity (win32, mockeado — sin windows real disponible en e
         expect((0, process_1.refIsAlive)(fakeRef)).toBe(false);
         expect(killSpy).toHaveBeenCalledWith(999999, 0);
     });
+    test('pidExistsNative reintenta un ESRCH transitorio antes de declarar muerte — no confia en un unico intento (regresion: CI real windows-latest, dos corridas del mismo commit dieron resultados distintos para un proceso recien spawneado)', () => {
+        Object.defineProperty(process, 'platform', { value: 'win32' });
+        let calls = 0;
+        const killSpy = jest.spyOn(process, 'kill').mockImplementation(() => {
+            calls++;
+            if (calls < 3) {
+                const err = new Error('transient');
+                err.code = 'ESRCH';
+                throw err;
+            }
+            return true; // el pid "aparece" recien al tercer intento — simula la carrera real observada
+        });
+        const fakeRef = { pid: 424242, startTime: 'x', spawnNonce: 'n', argvDigest: 'd', processGroup: 424242, psArgsDigest: 'x' };
+        expect((0, process_1.refIsAlive)(fakeRef)).toBe(true); // NUNCA declara muerte por el ESRCH transitorio de los primeros 2 intentos
+        expect(killSpy).toHaveBeenCalledTimes(3);
+    });
     test('refIsAlive en win32 NUNCA declara muerte por un error que no sea ESRCH (ej. EPERM: el pid existe pero sin permiso de senializarlo) (R2.1)', () => {
         Object.defineProperty(process, 'platform', { value: 'win32' });
         jest.spyOn(process, 'kill').mockImplementation(() => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agentic-workflow-manager",
-  "version": "3.13.2",
+  "version": "3.13.3",
   "main": "dist/src/index.js",
   "bin": {
     "awm": "./dist/src/index.js"