vibeview 0.12.0 → 0.12.1

package/dist/{chunk-NJU4AKHN.js → chunk-NFFLJNUI.js}

@@ -835,7 +835,24 @@ function describeAllExclusions(configured) {
   return parts.join("; ");
 }
 var POLL_INTERVAL_MS = 2e3;
+var QUEUED_POLL_INTERVAL_MS = 1e4;
+function nextPollDelayMs(pendingStatuses, baseMs, queuedMs) {
+  if (pendingStatuses.length > 0 && pendingStatuses.every((s) => s === "queued")) {
+    return queuedMs;
+  }
+  return baseMs;
+}
 var POLL_SAFETY_MARGIN_MS = 10 * 60 * 1e3;
+function runningCapDeadline(startedAt, timeoutS) {
+  return startedAt + timeoutS * 1e3 + POLL_SAFETY_MARGIN_MS;
+}
+var DEFAULT_QUEUE_WAIT_MS = 30 * 60 * 1e3;
+function queueWaitMs() {
+  const raw = process.env.VIBEVIEW_QUEUE_WAIT_MS;
+  if (!raw) return DEFAULT_QUEUE_WAIT_MS;
+  const parsed = Number(raw);
+  return Number.isFinite(parsed) && parsed > 0 ? parsed : DEFAULT_QUEUE_WAIT_MS;
+}
 function defaultCloudBuildDeps(client) {
   return {
     listSourceFiles,
@@ -927,7 +944,7 @@ ${offenders}`);
       const message = err instanceof Error ? err.message : String(err);
       fail(opts, message);
     }
-    uploadSpinner?.succeed(`uploaded \u2014 ${created.length} build(s) started`);
+    uploadSpinner?.succeed(`uploaded \u2014 ${created.length} build(s) submitted`);
     for (const b of created) {
       emitCloud(opts.json, { event: "build_created", id: b.id, platform: b.platform });
     }
@@ -985,75 +1002,179 @@ var MAX_CONSECUTIVE_POLL_FAILURES = 5;
 async function waitForCloudBuilds(builds, opts, deps) {
   const now = deps.now ?? Date.now;
   const pollIntervalMs = deps.pollIntervalMs ?? POLL_INTERVAL_MS;
+  const queuedPollIntervalMs = deps.queuedPollIntervalMs ?? (deps.pollIntervalMs !== void 0 ? pollIntervalMs : QUEUED_POLL_INTERVAL_MS);
   const multi = builds.length > 1;
   const state = new Map(builds.map((b) => [b.id, b]));
   const offsets = new Map(builds.map((b) => [b.id, 0]));
   const failures = new Map(builds.map((b) => [b.id, 0]));
+  const finalized = /* @__PURE__ */ new Set();
   const abandoned = /* @__PURE__ */ new Set();
+  const pollAbandoned = /* @__PURE__ */ new Set();
+  const transitions = new Map(
+    builds.map((b) => [b.id, newTransitionState()])
+  );
   let lastError;
-  const maxTimeoutS = Math.max(...builds.map((b) => b.timeout_s), 0);
-  const deadline = now() + maxTimeoutS * 1e3 + POLL_SAFETY_MARGIN_MS;
+  const runningStartedAt = /* @__PURE__ */ new Map();
+  const queueStartedAt = new Map(builds.map((b) => [b.id, now()]));
+  const queueCapMs = queueWaitMs();
+  const waitStartedAt = now();
+  const maxTimeoutMs = Math.max(0, ...builds.map((b) => b.timeout_s * 1e3));
+  const initialBackstopDeadline = waitStartedAt + queueCapMs + maxTimeoutMs + POLL_SAFETY_MARGIN_MS;
+  const backstopDeadlineFor = (id, timeoutS) => {
+    const startedAt = runningStartedAt.get(id);
+    if (startedAt === void 0) return initialBackstopDeadline;
+    return Math.max(initialBackstopDeadline, runningCapDeadline(startedAt, timeoutS));
+  };
   const printLine = (id, platform, line) => {
     const prefix = multi ? `[${platform}] ` : "";
     if (!opts.json) console.error(prefix + line);
   };
-  const stillWaiting = () => Array.from(state.values()).some(
-    (b) => !BUILD_TERMINAL_STATUSES.has(b.status) && !abandoned.has(b.id)
-  );
-  while (stillWaiting()) {
-    if (now() > deadline) {
-      throw new Error(
-        `cloud build poll timed out after exceeding the ${Math.round((maxTimeoutS * 1e3 + POLL_SAFETY_MARGIN_MS) / 1e3)}s safety limit \u2014 the backend never reported a terminal status`
-      );
+  const stillWaiting = () => Array.from(state.values()).some((b) => !finalized.has(b.id) && !abandoned.has(b.id));
+  const buildDeadline = (build) => {
+    const startedAt = runningStartedAt.get(build.id);
+    if (startedAt === void 0) return null;
+    return runningCapDeadline(startedAt, build.timeout_s);
+  };
+  const giveUpLocally = (id, message) => {
+    abandoned.add(id);
+    emitCloud(opts.json, { event: "error", message });
+    if (!opts.json) console.error(chalk.red(message));
+  };
+  const pollAndProcess = async (build) => {
+    const offset = offsets.get(build.id) ?? 0;
+    const logs = await deps.getCloudBuildLogs(build.id, offset);
+    const updated = { ...build, status: logs.status };
+    state.set(build.id, updated);
+    if (logs.status === "running" && !runningStartedAt.has(build.id)) {
+      runningStartedAt.set(build.id, now());
+    }
+    offsets.set(build.id, logs.offset);
+    const transitionState = transitions.get(build.id);
+    printTransition(build.id, build.platform, logs.status, transitionState, opts.json, printLine);
+    if (logs.content) {
+      emitCloud(opts.json, { event: "build_log", id: build.id, content: logs.content });
+      for (const line of logs.content.split("\n")) {
+        if (line.length === 0) continue;
+        printLine(build.id, build.platform, line);
+      }
+    }
+    if (BUILD_TERMINAL_STATUSES.has(logs.status)) {
+      const final = await fetchFinalSummary(build.id, updated, deps);
+      state.set(build.id, final);
+      if (!opts.json) {
+        printOutcomeLine(final, multi);
+      }
+      emitCloud(opts.json, {
+        event: "build_finished",
+        id: final.id,
+        status: final.status,
+        cost_micro: final.cost_micro,
+        error_summary: final.error_summary
+      });
+      finalized.add(build.id);
     }
+    failures.set(build.id, 0);
+    return logs.status;
+  };
+  const finalPollBeforeGivingUp = async (build) => {
+    try {
+      const status = await pollAndProcess(build);
+      return BUILD_TERMINAL_STATUSES.has(status) ? "terminal" : "nonterminal";
+    } catch {
+      return "poll_failed";
+    }
+  };
+  while (stillWaiting()) {
     for (const build of state.values()) {
-      if (BUILD_TERMINAL_STATUSES.has(build.status) || abandoned.has(build.id)) continue;
-      try {
-        const offset = offsets.get(build.id) ?? 0;
-        const logs = await deps.getCloudBuildLogs(build.id, offset);
-        if (logs.content) {
-          emitCloud(opts.json, { event: "build_log", id: build.id, content: logs.content });
-          for (const line of logs.content.split("\n")) {
-            if (line.length === 0) continue;
-            printLine(build.id, build.platform, line);
+      if (abandoned.has(build.id)) continue;
+      if (finalized.has(build.id)) continue;
+      if (BUILD_TERMINAL_STATUSES.has(build.status)) {
+        try {
+          await pollAndProcess(build);
+        } catch (err) {
+          lastError = err;
+          const count = (failures.get(build.id) ?? 0) + 1;
+          failures.set(build.id, count);
+          if (count >= MAX_CONSECUTIVE_POLL_FAILURES) {
+            pollAbandoned.add(build.id);
+            const message = err instanceof Error ? err.message : String(err);
+            giveUpLocally(
+              build.id,
+              `giving up finishing ${build.id} after ${count} consecutive failures: ${message} \u2014 the build likely finished server-side; check its outcome with \`vibeview builds logs ${build.id}\``
+            );
           }
         }
-        offsets.set(build.id, logs.offset);
-        const updated = { ...build, status: logs.status };
-        state.set(build.id, updated);
-        failures.set(build.id, 0);
-        if (BUILD_TERMINAL_STATUSES.has(logs.status)) {
-          const final = await fetchFinalSummary(build.id, updated, deps);
-          state.set(build.id, final);
-          emitCloud(opts.json, {
-            event: "build_finished",
-            id: final.id,
-            status: final.status,
-            cost_micro: final.cost_micro,
-            error_summary: final.error_summary
-          });
-          if (!opts.json) {
-            printOutcomeLine(final, multi);
-          }
+        continue;
+      }
+      if (build.status === "queued") {
+        const queuedSince = queueStartedAt.get(build.id) ?? now();
+        if (now() - queuedSince > queueCapMs) {
+          const rescue = await finalPollBeforeGivingUp(build);
+          if (state.get(build.id).status !== "queued") continue;
+          giveUpLocally(
+            build.id,
+            rescue === "poll_failed" ? (
+              // PR-445 review F6: the rescue poll failed, so "queued" is the
+              // last status observed BEFORE the gap — not verified server
+              // truth. Say so instead of asserting the build stayed queued.
+              `${build.id} still queued after ${Math.round(queueCapMs / 6e4)}m \u2014 builders busy; giving up waiting on it. Could not re-check its final status (the last poll failed) \u2014 it may have started or finished; check on it with \`vibeview builds\``
+            ) : `${build.id} still queued after ${Math.round(queueCapMs / 6e4)}m \u2014 builders busy; giving up waiting on it. The build stays queued server-side \u2014 check on it later with \`vibeview builds\``
+          );
+          continue;
+        }
+      } else {
+        const deadline = buildDeadline(build);
+        if (deadline !== null && now() > deadline) {
+          if (await finalPollBeforeGivingUp(build) === "terminal") continue;
+          giveUpLocally(
+            build.id,
+            `${build.id} exceeded its ${Math.round((build.timeout_s * 1e3 + POLL_SAFETY_MARGIN_MS) / 1e3)}s time limit client-side \u2014 giving up waiting on it. The build may still be running server-side \u2014 check its outcome with \`vibeview builds logs ${build.id}\``
+          );
+          continue;
         }
+      }
+      try {
+        await pollAndProcess(build);
       } catch (err) {
         lastError = err;
         const count = (failures.get(build.id) ?? 0) + 1;
         failures.set(build.id, count);
         if (count >= MAX_CONSECUTIVE_POLL_FAILURES) {
-          abandoned.add(build.id);
+          pollAbandoned.add(build.id);
           const message = err instanceof Error ? err.message : String(err);
-          const giveUp = `giving up polling ${build.id} after ${count} consecutive failures: ${message} \u2014 the build may still be running server-side; resume watching it with \`vibeview builds logs ${build.id} --follow\``;
-          emitCloud(opts.json, { event: "error", message: giveUp });
-          if (!opts.json) console.error(chalk.red(giveUp));
+          giveUpLocally(
+            build.id,
+            `giving up polling ${build.id} after ${count} consecutive failures: ${message} \u2014 the build may still be running server-side; resume watching it with \`vibeview builds logs ${build.id} --follow\``
+          );
         }
       }
     }
     if (stillWaiting()) {
-      await new Promise((r) => setTimeout(r, pollIntervalMs));
+      for (const build of state.values()) {
+        if (BUILD_TERMINAL_STATUSES.has(build.status) || abandoned.has(build.id)) continue;
+        const deadline = backstopDeadlineFor(build.id, build.timeout_s);
+        if (now() <= deadline) continue;
+        if (await finalPollBeforeGivingUp(build) === "terminal") continue;
+        const postRescueDeadline = backstopDeadlineFor(build.id, build.timeout_s);
+        if (now() <= postRescueDeadline) continue;
+        const fresh = state.get(build.id);
+        const capMinutes = Math.round((postRescueDeadline - waitStartedAt) / 6e4);
+        giveUpLocally(
+          build.id,
+          `giving up waiting on ${build.id}: still \`${fresh.status}\` with no terminal status after the overall ${capMinutes}m safety limit \u2014 the backend never reported the build finished; check on it later with \`vibeview builds\``
+        );
+      }
+    }
+    if (stillWaiting()) {
+      const pendingStatuses = Array.from(state.values()).filter(
+        (b) => !abandoned.has(b.id) && !finalized.has(b.id) && !BUILD_TERMINAL_STATUSES.has(b.status)
+      ).map((b) => b.status);
+      await new Promise(
+        (r) => setTimeout(r, nextPollDelayMs(pendingStatuses, pollIntervalMs, queuedPollIntervalMs))
+      );
     }
   }
-  if (abandoned.size > 0 && !Array.from(state.values()).some((b) => BUILD_TERMINAL_STATUSES.has(b.status))) {
+  if (pollAbandoned.size > 0 && pollAbandoned.size === abandoned.size && !Array.from(state.values()).some((b) => BUILD_TERMINAL_STATUSES.has(b.status))) {
     const message = lastError instanceof Error ? lastError.message : String(lastError);
     throw new Error(
       `cloud build poll failed ${MAX_CONSECUTIVE_POLL_FAILURES} times in a row, giving up: ${message}
@@ -1069,6 +1190,26 @@ async function fetchFinalSummary(id, fallback, deps) {
     return fallback;
   }
 }
+function newTransitionState() {
+  return { printedQueued: false, printedRunning: false };
+}
+function printTransition(id, platform, status, state, json, printLine) {
+  if (status === "queued" && !state.printedQueued) {
+    state.printedQueued = true;
+    if (json) {
+      emitCloud(json, { event: "status", id, status: "queued" });
+    } else {
+      printLine(id, platform, chalk.dim("\u23F3 queued \u2014 waiting for a build slot"));
+    }
+  } else if (status === "running" && !state.printedRunning) {
+    state.printedRunning = true;
+    if (json) {
+      emitCloud(json, { event: "status", id, status: "running" });
+    } else {
+      printLine(id, platform, `${chalk.cyan("\u25B6")} building`);
+    }
+  }
+}
 function printOutcomeLine(build, multi) {
   const prefix = multi ? `[${build.platform}] ` : "";
   const ok = build.status === "succeeded";
@@ -2958,6 +3099,8 @@ export {
   buildCommand,
   BUILD_TERMINAL_STATUSES,
   fetchFinalSummary,
+  newTransitionState,
+  printTransition,
   printOutcomeLine,
   startDevSession,
   PLATFORM_DEVICE,

package/dist/cli.js

@@ -7,11 +7,13 @@ import {
   dev,
   fetchFinalSummary,
   killOwnedMetro,
+  newTransitionState,
   printOutcomeLine,
+  printTransition,
   readDevState,
   readMetroState,
   runAgentVerb
-} from "./chunk-NJU4AKHN.js";
+} from "./chunk-NFFLJNUI.js";
 import {
   ApiClient,
   EXIT_INFRA_ERROR,
@@ -1247,9 +1249,14 @@ async function followLogs(id, opts, deps) {
   let offset = 0;
   let status = "queued";
   let consecutiveFailures = 0;
+  const transitionState = newTransitionState();
+  const printLine = (_id, _platform, line) => {
+    if (!opts.json) console.error(line);
+  };
   while (!BUILD_TERMINAL_STATUSES.has(status)) {
     try {
       const logs = await deps.getCloudBuildLogs(id, offset);
+      printTransition(id, "", logs.status, transitionState, opts.json, printLine);
       if (logs.content) {
         if (opts.json) {
           console.log(JSON.stringify({ event: "build_log", id, content: logs.content }));
@@ -1710,7 +1717,7 @@ program.command("dev-stop").description("Stop a `vibeview dev --detach` session
 program.command("mcp").description(
   "Run an MCP stdio server exposing the agent verbs (tap, swipe, type, ...) plus dev_start/dev_stop as tools"
 ).option("--token <token>", "API token (or set VIBEVIEW_API_TOKEN)").action(async (opts) => {
-  const { runMcpServer } = await import("./server-LZRNKGVA.js");
+  const { runMcpServer } = await import("./server-A44Z2HUK.js");
   await runMcpServer(opts);
 });
 program.command("agent-setup").description(

package/dist/{server-LZRNKGVA.js → server-A44Z2HUK.js}

@@ -11,7 +11,7 @@ import {
   readDevState,
   renderEnvelope,
   startDevSession
-} from "./chunk-NJU4AKHN.js";
+} from "./chunk-NFFLJNUI.js";
 import {
   ApiClient,
   resolveAuth

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "vibeview",
-  "version": "0.12.0",
+  "version": "0.12.1",
   "description": "VibeView CLI -- run mobile tests from CI/CD pipelines",
   "type": "module",
   "bin": {