@pushpalsdev/cli 1.0.98 → 1.0.99

package/package.json

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

package/runtime/sandbox/apps/workerpals/src/common/sandbox_env.ts

@@ -53,6 +53,12 @@ function ensureSandboxGitConfig(homeDir: string): void {
   }
 }
 
+function withNodeDnsIpv4First(value: string | undefined): string {
+  const existing = (value ?? "").trim();
+  if (/(^|\s)--dns-result-order=/.test(existing)) return existing;
+  return [existing, "--dns-result-order=ipv4first"].filter(Boolean).join(" ");
+}
+
 function resolveOriginalHome(env: Record<string, string>): string {
   return env.HOME || env.USERPROFILE || homedir();
 }
@@ -100,6 +106,8 @@ export function buildWorkerSandboxWritableEnv(
     EXPO_NO_INTERACTIVE: env.EXPO_NO_INTERACTIVE ?? "1",
     CI: env.CI ?? "1",
     BROWSER: env.BROWSER ?? "none",
+    NODE_OPTIONS: withNodeDnsIpv4First(env.NODE_OPTIONS),
+    REACT_NATIVE_PACKAGER_HOSTNAME: env.REACT_NATIVE_PACKAGER_HOSTNAME ?? "127.0.0.1",
     EXPO_DEV_SERVER_PORT: env.EXPO_DEV_SERVER_PORT ?? defaultExpoPort,
     RCT_METRO_PORT: env.RCT_METRO_PORT ?? defaultExpoPort,
     PUSHPALS_VALIDATION_REPO: repo,

package/runtime/sandbox/apps/workerpals/src/execute_job.ts

@@ -647,7 +647,49 @@ async function terminateValidationProcessTree(
   }
 }
 
-async function runValidationArgv(
+function captureValidationStream(stream: ReadableStream<Uint8Array> | null) {
+  let text = "";
+  let done = false;
+  const reader = stream?.getReader();
+  const promise = reader
+    ? (async () => {
+        try {
+          while (true) {
+            const result = await reader.read();
+            if (result.done) break;
+            text += Buffer.from(result.value).toString("utf8");
+          }
+        } catch {
+          // Stream cancellation after process exit is expected when descendants
+          // inherit pipes from failed browser/dev-server launchers.
+        } finally {
+          done = true;
+          try {
+            reader.releaseLock();
+          } catch {
+            // ignore
+          }
+        }
+      })()
+    : Promise.resolve().then(() => {
+        done = true;
+      });
+
+  return {
+    cancel: async () => {
+      try {
+        await reader?.cancel();
+      } catch {
+        // ignore
+      }
+    },
+    isDone: () => done,
+    promise,
+    text: () => text,
+  };
+}
+
+export async function runValidationArgv(
   repo: string,
   command: string,
   argv: string[],
@@ -665,31 +707,52 @@ async function runValidationArgv(
     stderr: "pipe",
     detached: process.platform !== "win32",
   });
+  const stdoutCapture = captureValidationStream(proc.stdout);
+  const stderrCapture = captureValidationStream(proc.stderr);
   let timedOut = false;
-  const timer = setTimeout(
-    () => {
+  const timeout = Math.max(1_000, timeoutMs);
+  let timeoutTimer: ReturnType<typeof setTimeout> | null = null;
+  const timeoutPromise = new Promise<"timeout">((resolveTimeout) => {
+    timeoutTimer = setTimeout(() => {
       timedOut = true;
       void terminateValidationProcessTree(proc);
-    },
-    Math.max(1_000, timeoutMs),
-  );
+      resolveTimeout("timeout");
+    }, timeout);
+  });
+  const exitOrTimeout = await Promise.race([
+    proc.exited.then((code) => ({ type: "exit" as const, code })),
+    timeoutPromise,
+  ]);
+  if (timeoutTimer) clearTimeout(timeoutTimer);
+  const exitCode = exitOrTimeout === "timeout" ? 124 : exitOrTimeout.code;
+
+  if (!timedOut) {
+    await Promise.race([
+      Promise.all([stdoutCapture.promise, stderrCapture.promise]),
+      Bun.sleep(1_000),
+    ]);
+    if (!stdoutCapture.isDone() || !stderrCapture.isDone()) {
+      await terminateValidationProcessTree(proc);
+      await Promise.all([stdoutCapture.cancel(), stderrCapture.cancel()]);
+    }
+  } else {
+    await Promise.all([stdoutCapture.cancel(), stderrCapture.cancel()]);
+  }
 
-  const [stdout, stderr, exitCode] = await Promise.all([
-    new Response(proc.stdout).text(),
-    new Response(proc.stderr).text(),
-    proc.exited,
+  await Promise.race([
+    Promise.all([stdoutCapture.promise, stderrCapture.promise]),
+    Bun.sleep(500),
   ]);
-  clearTimeout(timer);
 
   return {
     step: command,
     command,
     ok: !timedOut && exitCode === 0,
-    exitCode: timedOut ? 124 : exitCode,
-    stdout: compactJobOutput(stdout.trim(), outputPolicy),
+    exitCode,
+    stdout: compactJobOutput(stdoutCapture.text().trim(), outputPolicy),
     stderr: compactJobOutput(
       [
-        stderr.trim(),
+        stderrCapture.text().trim(),
         timedOut ? timeoutMessage : "",
       ]
         .filter(Boolean)
@@ -1242,6 +1305,10 @@ function detectValidationBlocker(runs: ValidationExecutionResult[]): ValidationB
     combined.includes("network access") ||
     combined.includes("connection refused") ||
     combined.includes("getaddrinfo") ||
+    combined.includes("err_socket_bad_port") ||
+    combined.includes("expo exited early") ||
+    combined.includes("eperm") ||
+    combined.includes("operation not permitted") ||
     combined.includes("eacces")
   ) {
     return {