@pushpalsdev/cli 1.1.39 → 1.1.41

package/dist/pushpals-cli.js

@@ -58,11 +58,13 @@ function shouldRestartService(attempts, maxAttempts = DEFAULT_SERVICE_MANAGER_MA
   return normalizedAttempts < normalizedMax;
 }
 function pipeProcessStreamToLines(stream, onLine) {
-  if (!stream || typeof stream === "number" || typeof stream.getReader !== "function")
-    return;
+  if (!stream || typeof stream === "number" || typeof stream.getReader !== "function") {
+    return { cancel: () => {} };
+  }
   const reader = stream.getReader();
   const decoder = new TextDecoder;
   let pending = "";
+  let cancelled = false;
   (async () => {
     try {
       while (true) {
@@ -86,9 +88,21 @@ function pipeProcessStreamToLines(stream, onLine) {
       if (tail)
         onLine?.(tail);
     } catch {} finally {
-      reader.releaseLock();
+      try {
+        reader.releaseLock();
+      } catch {}
     }
   })();
+  return {
+    cancel: () => {
+      if (cancelled)
+        return;
+      cancelled = true;
+      try {
+        reader.cancel().catch(() => {});
+      } catch {}
+    }
+  };
 }
 function spawnManagedService(spec) {
   const env = { ...spec.env ?? {} };
@@ -98,8 +112,8 @@ function spawnManagedService(spec) {
     stdout: "pipe",
     stderr: "pipe"
   });
-  pipeProcessStreamToLines(proc.stdout, spec.onStdoutLine);
-  pipeProcessStreamToLines(proc.stderr, spec.onStderrLine);
+  const stdoutPipe = pipeProcessStreamToLines(proc.stdout, spec.onStdoutLine);
+  const stderrPipe = pipeProcessStreamToLines(proc.stderr, spec.onStderrLine);
   const service = {
     name: spec.name,
     proc,
@@ -109,7 +123,11 @@ function spawnManagedService(spec) {
     exited: false,
     exitCode: null,
     launchedAtMs: Date.now(),
-    logPath: spec.logPath
+    logPath: spec.logPath,
+    stopOutputPipes: () => {
+      stdoutPipe.cancel();
+      stderrPipe.cancel();
+    }
   };
   proc.exited.then((code) => {
     service.exited = true;
@@ -161,6 +179,7 @@ class ServiceManager {
     const existing = this.services.get(spec.name);
     if (existing && !existing.exited) {
       try {
+        existing.stopOutputPipes?.();
         const pid = existing.proc.pid;
         if (process.platform === "win32" && typeof pid === "number" && pid > 0) {
           Bun.spawnSync(["taskkill", "/PID", String(pid), "/T", "/F"], {
@@ -224,6 +243,7 @@ class ServiceManager {
     this.stopped = true;
     for (const service of this.services.values()) {
       try {
+        service.stopOutputPipes?.();
         const pid = service.proc.pid;
         if (process.platform === "win32" && typeof pid === "number" && pid > 0) {
           Bun.spawnSync(["taskkill", "/PID", String(pid), "/T", "/F"], {
@@ -3221,6 +3241,7 @@ function buildServiceStopCommand(pid, platform = process.platform) {
 function stopRuntimeServices(services) {
   for (const service of services) {
     try {
+      service.stopOutputPipes?.();
       const stopCommand = buildServiceStopCommand(service.proc.pid, process.platform);
       if (stopCommand) {
         Bun.spawnSync(stopCommand, {
@@ -3236,6 +3257,52 @@ function stopRuntimeServices(services) {
     } catch {}
   }
 }
+async function runWindowsServiceStopCommand(command, timeoutMs) {
+  const proc = Bun.spawn(command, {
+    stdin: "ignore",
+    stdout: "ignore",
+    stderr: "ignore"
+  });
+  let timeout = null;
+  let timedOut = false;
+  const exitCode = await Promise.race([
+    proc.exited,
+    new Promise((resolveTimeout) => {
+      timeout = setTimeout(() => {
+        timedOut = true;
+        try {
+          proc.kill("SIGKILL");
+        } catch {}
+        resolveTimeout(-1);
+      }, Math.max(250, timeoutMs));
+    })
+  ]);
+  if (timeout) {
+    clearTimeout(timeout);
+    timeout = null;
+  }
+  if (timedOut) {
+    await Promise.race([proc.exited.catch(() => -1), Bun.sleep(250)]);
+    return false;
+  }
+  return exitCode === 0;
+}
+async function stopRuntimeServicesOnWindows(services, timeoutMs) {
+  const deadline = Date.now() + Math.max(1000, timeoutMs);
+  for (const service of services) {
+    service.stopOutputPipes?.();
+    const stopCommand = buildServiceStopCommand(service.proc.pid, "win32");
+    if (stopCommand) {
+      const remainingMs = Math.max(250, deadline - Date.now());
+      const stopped = await runWindowsServiceStopCommand(stopCommand, Math.min(WINDOWS_TASKKILL_TIMEOUT_MS2, remainingMs));
+      if (stopped)
+        continue;
+    }
+    try {
+      service.proc.kill("SIGKILL");
+    } catch {}
+  }
+}
 function resolveGracefulShutdownPriority(name) {
   if (name === "source_control_manager")
     return 0;
@@ -3264,7 +3331,7 @@ async function stopRuntimeServicesGracefully(services, timeoutMs = 1e4) {
     return;
   const ordered = [...running].sort((a, b) => resolveGracefulShutdownPriority(a.name) - resolveGracefulShutdownPriority(b.name));
   if (process.platform === "win32") {
-    stopRuntimeServices(ordered);
+    await stopRuntimeServicesOnWindows(ordered, timeoutMs);
     await waitForRuntimeServicesExit(ordered, Math.min(1000, timeoutMs));
     return;
   }
@@ -3295,6 +3362,7 @@ async function shutdownEmbeddedServiceManagerGracefully(options) {
     reason,
     requestShutdown = requestLocalRuntimeShutdown,
     shutdownAcceptedDelayMs = 1500,
+    serviceStopTimeoutMs = 1e4,
     onLog = (line) => console.log(line),
     onWarn = (line) => console.warn(line),
     cleanupTasks = []
@@ -3310,7 +3378,7 @@ async function shutdownEmbeddedServiceManagerGracefully(options) {
   } else if (shutdown.detail) {
     onWarn(`[pushpals] ${shutdown.detail}`);
   }
-  await stopRuntimeServicesGracefully(services);
+  await stopRuntimeServicesGracefully(services, serviceStopTimeoutMs);
   for (const task of cleanupTasks) {
     await task();
   }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pushpalsdev/cli",
-  "version": "1.1.39",
+  "version": "1.1.41",
   "description": "PushPals terminal CLI for LocalBuddy -> RemoteBuddy orchestration",
   "license": "MIT",
   "repository": {

package/runtime/sandbox/apps/workerpals/src/backends/openai_codex/openai_codex_executor.py

@@ -119,6 +119,7 @@ _DEFAULT_NO_EDIT_RECHECK_S = 120
 _NO_EDIT_RECOVERY_RECHECK_S = 30
 _DEFAULT_NO_EDIT_COMMAND_GRACE_S = 240
 _DEFAULT_NO_EDIT_COMMAND_PROGRESS_CAP_S = 360
+_BACKGROUND_NO_EDIT_COMMAND_PROGRESS_CAP_S = 120
 _NO_EDIT_RECOVERY_COMMAND_PROGRESS_CAP_S = 120
 _DEFAULT_STARTUP_STALL_WATCHDOG_S = 210
 _RECOVERY_STARTUP_STALL_WATCHDOG_S = 150
@@ -817,6 +818,7 @@ def _resolve_no_edit_command_progress_cap_seconds(
     communicate_timeout_s: Optional[int],
     no_edit_command_grace_s: Optional[int],
     recovery_attempt: int = 0,
+    prompt: str = "",
 ) -> Optional[int]:
     if not communicate_timeout_s or no_edit_command_grace_s is None:
         return None
@@ -834,11 +836,12 @@ def _resolve_no_edit_command_progress_cap_seconds(
         else:
             return max(1, min(parsed, max(1, communicate_timeout_s - 1)))
 
-    default_s = (
-        _NO_EDIT_RECOVERY_COMMAND_PROGRESS_CAP_S
-        if recovery_attempt > 0
-        else _DEFAULT_NO_EDIT_COMMAND_PROGRESS_CAP_S
-    )
+    if recovery_attempt > 0:
+        default_s = _NO_EDIT_RECOVERY_COMMAND_PROGRESS_CAP_S
+    elif _looks_like_background_autonomy_prompt(prompt):
+        default_s = _BACKGROUND_NO_EDIT_COMMAND_PROGRESS_CAP_S
+    else:
+        default_s = _DEFAULT_NO_EDIT_COMMAND_PROGRESS_CAP_S
     upper = max(1, communicate_timeout_s - 1)
     return max(1, min(default_s, upper))
 
@@ -2713,6 +2716,7 @@ def _run_codex_task(
                 communicate_timeout_s,
                 no_edit_command_grace_s,
                 recovery_attempt=recovery_depth,
+                prompt=prompt,
             )
             startup_stall_watchdog_s = _resolve_startup_stall_watchdog_seconds(
                 communicate_timeout_s,

package/runtime/sandbox/apps/workerpals/src/backends/openai_codex/test_openai_codex_runtime_config.py

@@ -45,6 +45,7 @@ from openai_codex_executor import (
     _has_credible_shell_wrapper_progress,
     _load_prompt_template,
     _mask_repo_local_codex_files,
+    _minimum_recovery_attempt_seconds,
     _repo_root_for_prompt_loading,
     _restore_repo_local_codex_files,
     _resolve_codex_command_prefix,
@@ -249,6 +250,41 @@ class OpenAICodexRuntimeConfigTests(unittest.TestCase):
         )
         self.assertEqual(_resolve_rollout_watchdog_seconds(prompt, 1200, no_edit), 90)
 
+    def test_background_autonomy_caps_patchless_command_progress_before_recovery_reserve(self) -> None:
+        prompt = (
+            "Task planning contract from PushPals:\n"
+            "- Planning summary: intent=code_change, risk=low, priority=background\n"
+            "- Origin=autonomy targetPaths=[app/__tests__/opportunity-graph.contract.test.ts]\n"
+            "Add focused contract coverage without broad discovery.\n"
+        )
+        child_budget_s = 570
+
+        with mock.patch.dict(
+            os.environ,
+            {
+                "WORKERPALS_OPENAI_CODEX_NO_EDIT_COMMAND_GRACE_S": "",
+                "WORKERPALS_OPENAI_CODEX_NO_EDIT_COMMAND_PROGRESS_CAP_S": "",
+                "WORKERPALS_OPENAI_CODEX_STARTUP_STALL_WATCHDOG_S": "",
+            },
+            clear=False,
+        ):
+            command_grace_s = _resolve_no_edit_command_grace_seconds(child_budget_s)
+            command_cap_s = _resolve_no_edit_command_progress_cap_seconds(
+                child_budget_s,
+                command_grace_s,
+                prompt=prompt,
+            )
+            startup_stall_s = _resolve_startup_stall_watchdog_seconds(child_budget_s)
+
+        self.assertEqual(command_grace_s, 240)
+        self.assertEqual(command_cap_s, 120)
+        self.assertEqual(startup_stall_s, 210)
+        first_attempt_patchless_ceiling_s = startup_stall_s + command_cap_s
+        self.assertGreaterEqual(
+            child_budget_s - first_attempt_patchless_ceiling_s,
+            2 * _minimum_recovery_attempt_seconds(child_budget_s),
+        )
+
     def test_runtime_config_prefers_explicit_config_dir_override(self) -> None:
         import executor_base