@pushpalsdev/cli 1.1.16 → 1.1.18

package/dist/pushpals-cli.js

@@ -1641,6 +1641,7 @@ var DEFAULT_REMOTEBUDDY_CONSUMER_STARTUP_GRACE_MS = 8000;
 var DEFAULT_COMMAND_OUTPUT_DRAIN_TIMEOUT_MS = 2000;
 var DEFAULT_COMMAND_OUTPUT_MAX_CHARS = 512000;
 var DEFAULT_REMOTEBUDDY_SILENT_STARTUP_FALLBACK_MS = 20000;
+var DEFAULT_WORKERPAL_STARTUP_STATUS_FETCH_TIMEOUT_MS = 2000;
 var WINDOWS_TASKKILL_TIMEOUT_MS2 = 5000;
 var RUNTIME_BINARY_DOWNLOAD_ATTEMPTS = 3;
 var DEFAULT_STARTUP_GIT_PROBE_TIMEOUT_MS = 5000;
@@ -1648,6 +1649,7 @@ var DEFAULT_STARTUP_GIT_REMOTE_TIMEOUT_MS = 1e4;
 var DEFAULT_EMBEDDED_SERVICE_LAUNCH_WARN_MS = 5000;
 var EMBEDDED_SERVICE_RESTART_MAX_ATTEMPTS = 4;
 var WORKERPAL_STARTUP_READINESS_PROBE_MAX_MS = 15000;
+var BLOCKING_WORKERPAL_IMAGE_BUILD_ENV = "PUSHPALS_BLOCKING_WORKERPAL_IMAGE_BUILD";
 var CLI_SESSION_JOB_LOG_MAX_CHARS = 700;
 var CLI_SESSION_SHOW_JOB_EVENTS_ENV = "PUSHPALS_CLI_SHOW_JOB_EVENTS";
 var EMBEDDED_RUNTIME_SAFETY_CAP_DISABLE_ENV = "PUSHPALS_DISABLE_EMBEDDED_SAFETY_CAPS";
@@ -3900,6 +3902,13 @@ function resolveWorkerpalCapacityTimeoutMs(config) {
 function resolveWorkerpalStartupReadinessProbeTimeoutMs(config) {
   return Math.max(5000, Math.min(resolveWorkerpalCapacityTimeoutMs(config), WORKERPAL_STARTUP_READINESS_PROBE_MAX_MS));
 }
+function shouldPrepareEmbeddedWorkerpalDockerImageBlocking(opts = {}) {
+  const env = opts.env ?? process.env;
+  const explicit = String(env[BLOCKING_WORKERPAL_IMAGE_BUILD_ENV] ?? "").trim();
+  if (explicit)
+    return isTruthyCliEnvValue(explicit);
+  return (opts.platform ?? process.platform) !== "win32";
+}
 function shouldRunEmbeddedRuntimeStartupPrechecks(opts) {
   return !opts.serverHealthy && !opts.noAutoStart;
 }
@@ -4287,8 +4296,8 @@ async function probeSourceControlManager(port) {
     return false;
   }
 }
-async function fetchWorkerStatusRows(serverUrl, ttlMs) {
-  const payload = await fetchJsonWithTimeout(`${serverUrl}/workers?ttlMs=${Math.max(1000, Math.floor(ttlMs))}`, {}, 1e4);
+async function fetchWorkerStatusRows(serverUrl, ttlMs, timeoutMs = 1e4) {
+  const payload = await fetchJsonWithTimeout(`${serverUrl}/workers?ttlMs=${Math.max(1000, Math.floor(ttlMs))}`, {}, Math.max(250, Math.floor(timeoutMs)));
   if (!payload?.ok || !Array.isArray(payload.workers)) {
     return [];
   }
@@ -4298,7 +4307,8 @@ async function waitForWorkerpalCapacity(opts) {
   const deadline = Date.now() + Math.max(1000, opts.timeoutMs);
   let lastObservedOnline = 0;
   while (Date.now() < deadline) {
-    const workers = await (opts.fetchWorkersFn ?? fetchWorkerStatusRows)(opts.serverUrl, opts.ttlMs);
+    const remainingMs = Math.max(250, deadline - Date.now());
+    const workers = await (opts.fetchWorkersFn ?? fetchWorkerStatusRows)(opts.serverUrl, opts.ttlMs, Math.min(DEFAULT_WORKERPAL_STARTUP_STATUS_FETCH_TIMEOUT_MS, remainingMs));
     const summary = summarizeWorkerStatusRows(workers);
     if (summary.onlineWorkers > 0) {
       lastObservedOnline = Math.max(lastObservedOnline, summary.onlineWorkers);
@@ -5726,7 +5736,10 @@ async function main() {
     console.error("[pushpals] Precheck failed: start Docker Desktop or the Docker daemon, then retry pushpals.");
     process.exit(1);
   }
-  if (runEmbeddedRuntimeStartupPrechecks && workerpalDockerPrecheck.status !== "failed") {
+  if (runEmbeddedRuntimeStartupPrechecks && workerpalDockerPrecheck.status !== "failed" && shouldPrepareEmbeddedWorkerpalDockerImageBlocking({
+    platform: process.platform,
+    env: process.env
+  })) {
     const workerpalImagePrecheck = await prepareEmbeddedWorkerpalDockerImageIfNeeded({
       preparedRuntime,
       config,
@@ -5740,6 +5753,8 @@ async function main() {
     if (workerpalImagePrecheck.runtimeTag) {
       resolvedRuntimeTagForAutoStart = workerpalImagePrecheck.runtimeTag;
     }
+  } else if (runEmbeddedRuntimeStartupPrechecks && workerpalDockerPrecheck.status !== "failed") {
+    console.log(`[pushpals] Skipping blocking WorkerPal sandbox image build during CLI startup; WorkerPal warmup will prepare it in the background. Set ${BLOCKING_WORKERPAL_IMAGE_BUILD_ENV}=1 to force the old foreground behavior.`);
   }
   let remoteBuddyConsumerHealth = {
     ok: false,
@@ -6064,6 +6079,7 @@ export {
   shouldShowCliSessionOperationalEvents,
   shouldRunEmbeddedRuntimeStartupPrechecks,
   shouldRestartEmbeddedService,
+  shouldPrepareEmbeddedWorkerpalDockerImageBlocking,
   shouldDeferRemoteBuddySessionConsumerReadiness,
   runCommandWithEnv,
   resolveWorkerpalDockerProbe,

package/package.json

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

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

@@ -1213,16 +1213,7 @@ export class DockerExecutor {
     );
     await this.ensureWorktreeDependencyArtifacts(containerWorktreePath, onLog);
 
-    const args: string[] = [
-      "exec",
-      "-w",
-      containerWorktreePath,
-      this.warmContainerName,
-      "bun",
-      "run",
-      "/workspace/apps/workerpals/src/job_runner.ts",
-      "--spec-stdin",
-    ];
+    const args = this.buildWarmContainerExecArgs(containerWorktreePath);
 
     console.log(
       `[DockerExecutor] Running job in warm container: ${this.warmContainerName} (${this.executionConfigSummary()})`,
@@ -1321,26 +1312,62 @@ export class DockerExecutor {
     return result;
   }
 
+  private buildWarmContainerExecArgs(containerWorktreePath: string): string[] {
+    return [
+      "exec",
+      "-i",
+      "-w",
+      containerWorktreePath,
+      this.warmContainerName,
+      "bun",
+      "run",
+      "/workspace/apps/workerpals/src/job_runner.ts",
+      "--spec-stdin",
+    ];
+  }
+
   private async writeJobSpecToStdin(
     proc: ReturnType<typeof Bun.spawn>,
     base64Spec: string,
   ): Promise<void> {
-    const stdin = proc.stdin as WritableStream<Uint8Array> | undefined;
+    const stdin = proc.stdin as
+      | WritableStream<Uint8Array>
+      | {
+          write?: (chunk: Uint8Array | string) => unknown;
+          end?: () => unknown;
+          flush?: () => unknown;
+        }
+      | undefined;
     if (!stdin) {
       throw new Error("docker exec stdin pipe was not available");
     }
-    const writer = stdin.getWriter();
-    try {
-      await writer.write(new TextEncoder().encode(base64Spec));
-      await writer.close();
-    } catch (err) {
+    const bytes = new TextEncoder().encode(base64Spec);
+    if ("getWriter" in stdin && typeof stdin.getWriter === "function") {
+      const writer = stdin.getWriter();
       try {
-        await writer.abort(err);
-      } catch {
-        // Ignore abort failures; the original write error is more useful.
+        await writer.write(bytes);
+        await writer.close();
+      } catch (err) {
+        try {
+          await writer.abort(err);
+        } catch {
+          // Ignore abort failures; the original write error is more useful.
+        }
+        throw err;
       }
-      throw err;
+      return;
     }
+
+    if (typeof stdin.write === "function" && typeof stdin.end === "function") {
+      await stdin.write(bytes);
+      if (typeof stdin.flush === "function") {
+        await stdin.flush();
+      }
+      await stdin.end();
+      return;
+    }
+
+    throw new Error("docker exec stdin pipe does not support write/end or getWriter");
   }
 
   private async ensureWorktreeDependencyArtifacts(