vibeview 0.32.0 → 0.34.0

package/dist/cli.js

@@ -24,7 +24,7 @@ import {
   readDevStateFile,
   readMetroState,
   runAgentVerb
-} from "./chunk-T6H5XTBX.js";
+} from "./chunk-TPZG7NCU.js";
 import {
   ApiClient,
   ApiStatusError,
@@ -45,7 +45,7 @@ import {
   resolveAuth,
   saveConfig,
   uploadApp
-} from "./chunk-ZZK7YQSP.js";
+} from "./chunk-4D2E2PE4.js";
 
 // src/cli.ts
 import { createRequire } from "module";
@@ -60,10 +60,25 @@ import chalk2 from "chalk";
 // src/lib/formatters/human.ts
 import chalk from "chalk";
 
+// src/lib/formatters/not-checked.ts
+function notCheckedSteps(steps) {
+  return (steps ?? []).filter((s) => s.status === "not_checked");
+}
+function runStepsNotChecked(result) {
+  if (typeof result.steps_not_checked === "number") return result.steps_not_checked;
+  return notCheckedSteps(result.steps).length;
+}
+function stepsNotCheckedLabel(n) {
+  return `${n} step${n === 1 ? "" : "s"} not checked`;
+}
+function notCheckedSuffix(status, n) {
+  return status === "passed" && n && n > 0 ? ` (${stepsNotCheckedLabel(n)})` : "";
+}
+
 // src/lib/formatters/cases.ts
 function resultCases(result) {
   if (Array.isArray(result.test_runs)) return result.test_runs;
-  const failedStep = result.steps?.find((s) => s.status !== "passed");
+  const failedStep = result.steps?.find((s) => s.status === "failed" || s.status === "error");
   return [
     {
       case_name: result.case_name || result.run_id,
@@ -73,7 +88,12 @@ function resultCases(result) {
       error_kind: failedStep?.error_kind ?? null,
       error_kind_description: failedStep?.error_kind_description ?? null,
       screenshot_signed_url: failedStep?.screenshot_signed_url ?? null,
-      url: result.url
+      url: result.url,
+      steps_not_checked: runStepsNotChecked(result),
+      not_checked_steps: notCheckedSteps(result.steps).map((s) => ({
+        step: s.step,
+        note: s.note ?? null
+      }))
     }
   ];
 }
@@ -100,11 +120,16 @@ function formatHuman(result, verbose) {
   const duration = result.duration_ms ? `${(result.duration_ms / 1e3).toFixed(1)}s` : "-";
   lines.push("");
   const title = result.suite_name || result.case_name || result.run_id;
-  lines.push(`  ${icon}  ${chalk.bold(title)}  ${chalk.dim(duration)}`);
   const cases = resultCases(result);
+  const notChecked = cases.reduce((n, t) => n + (t.steps_not_checked ?? 0), 0);
+  const runSuffix = notCheckedSuffix(result.status, notChecked);
+  lines.push(
+    `  ${icon}${runSuffix ? chalk.yellow(runSuffix) : ""}  ${chalk.bold(title)}  ${chalk.dim(duration)}`
+  );
   const passed = cases.filter((t) => t.status === "passed").length;
   const total = cases.length;
-  lines.push(`  ${chalk.dim(`${passed}/${total} cases passed`)}`);
+  const notCheckedNote = notChecked > 0 ? `, ${stepsNotCheckedLabel(notChecked)}` : "";
+  lines.push(`  ${chalk.dim(`${passed}/${total} cases passed${notCheckedNote}`)}`);
   if (result.url) {
     lines.push(`  ${chalk.dim(result.url)}`);
   }
@@ -114,7 +139,10 @@ function formatHuman(result, verbose) {
     for (const tr of suiteCases) {
       const caseIcon = tr.status === "passed" ? chalk.green("  PASS") : tr.status === "cancelled" || tr.status === "pending" || tr.status === "running" ? chalk.yellow("  SKIP") : chalk.red("  FAIL");
       const caseDuration = tr.duration_ms ? `${(tr.duration_ms / 1e3).toFixed(1)}s` : "-";
-      lines.push(`  ${caseIcon}  ${tr.case_name}  ${chalk.dim(caseDuration)}`);
+      const caseSuffix = notCheckedSuffix(tr.status, tr.steps_not_checked);
+      lines.push(
+        `  ${caseIcon}${caseSuffix ? chalk.yellow(caseSuffix) : ""}  ${tr.case_name}  ${chalk.dim(caseDuration)}`
+      );
       if (tr.url) {
         lines.push(`        ${chalk.dim(tr.url)}`);
       }
@@ -129,7 +157,7 @@ function formatHuman(result, verbose) {
       lines.push(`  ${chalk.bold(result.case_name)}`);
     }
     for (const step of result.steps) {
-      const stepIcon = step.status === "passed" ? chalk.green("  PASS") : chalk.red("  FAIL");
+      const stepIcon = step.status === "passed" ? chalk.green("  PASS") : step.status === "not_checked" ? chalk.yellow("  NOT CHECKED") : chalk.red("  FAIL");
       const stepDuration = step.duration_ms ? `${(step.duration_ms / 1e3).toFixed(1)}s` : "-";
       lines.push(`  ${stepIcon}  ${step.step}  ${chalk.dim(stepDuration)}`);
       if (verbose && step.actions && step.actions.length > 0) {
@@ -137,7 +165,9 @@ function formatHuman(result, verbose) {
           lines.push(`        ${chalk.dim(formatAction(action))}`);
         }
       }
-      if (step.status !== "passed") {
+      if (step.status === "not_checked") {
+        if (step.note) lines.push(`        ${chalk.dim(step.note)}`);
+      } else if (step.status !== "passed") {
         lines.push(...failureLines(step));
       }
     }
@@ -156,13 +186,33 @@ function formatHuman(result, verbose) {
 
 // src/lib/formatters/json.ts
 function formatJson(result) {
-  return JSON.stringify(result, null, 2);
+  const doc = Array.isArray(result.test_runs) ? result : { ...result, steps_not_checked: runStepsNotChecked(result) };
+  return JSON.stringify(doc, null, 2);
 }
 
 // src/lib/formatters/junit.ts
 function escapeXml(s) {
   return s.replace(/&/g, "&amp;").replace(/</g, "&lt;").replace(/>/g, "&gt;").replace(/"/g, "&quot;").replace(/'/g, "&apos;");
 }
+function caseDetailLines(c) {
+  const n = c.status === "passed" ? c.steps_not_checked ?? 0 : 0;
+  const lines = [];
+  if (n > 0) {
+    lines.push("      <properties>");
+    lines.push(`        <property name="steps_not_checked" value="${n}" />`);
+    lines.push("      </properties>");
+  }
+  const out = [];
+  if (c.url) out.push(c.url);
+  if (n > 0) {
+    out.push(`${stepsNotCheckedLabel(n)}:`);
+    for (const s of c.not_checked_steps ?? []) {
+      out.push(`- ${s.step}${s.note ? ` (${s.note})` : ""}`);
+    }
+  }
+  if (out.length) lines.push(`      <system-out>${escapeXml(out.join("\n"))}</system-out>`);
+  return lines;
+}
 function formatJunit(result) {
   const suiteName = escapeXml(result.suite_name || result.case_name || result.run_id);
   const cases = resultCases(result);
@@ -194,8 +244,8 @@ function formatJunit(result) {
   for (const tr of cases) {
     const caseName = escapeXml(tr.case_name);
     const caseTime = tr.duration_ms ? (tr.duration_ms / 1e3).toFixed(1) : "0";
-    const out = tr.url ? `      <system-out>${escapeXml(tr.url)}</system-out>` : null;
-    if (tr.status === "passed" && !out) {
+    const detail = caseDetailLines(tr);
+    if (tr.status === "passed" && detail.length === 0) {
       lines.push(`    <testcase name="${caseName}" classname="${suiteName}" time="${caseTime}" />`);
       continue;
     }
@@ -207,7 +257,7 @@ function formatJunit(result) {
       const msg = escapeXml(tr.error_message || "Infrastructure error");
       lines.push(`      <error message="${msg}" type="InfraError">${msg}</error>`);
     }
-    if (out) lines.push(out);
+    lines.push(...detail);
     lines.push("    </testcase>");
   }
   lines.push("  </testsuite>");
@@ -240,8 +290,12 @@ var MULTI_TERMINAL = /* @__PURE__ */ new Set(["passed", "failed", "error", "canc
 function caseLines(c, classname) {
   const name = escapeXml(c.name);
   const head = `    <testcase name="${name}" classname="${classname}" time="${c.time}"`;
-  const out = c.url ? `      <system-out>${escapeXml(c.url)}</system-out>` : null;
-  if (c.verdict === "passed" && !out) return [`${head} />`];
+  const detail = caseDetailLines({
+    status: c.verdict,
+    url: c.url,
+    steps_not_checked: c.steps_not_checked
+  });
+  if (c.verdict === "passed" && detail.length === 0) return [`${head} />`];
   const lines = [`${head}>`];
   const msg = escapeXml(c.message ?? "");
   if (c.verdict === "failure") {
@@ -251,7 +305,7 @@ function caseLines(c, classname) {
   } else if (c.verdict === "skipped") {
     lines.push(`      <skipped message="${msg}" />`);
   }
-  if (out) lines.push(out);
+  lines.push(...detail);
   lines.push("    </testcase>");
   return lines;
 }
@@ -263,7 +317,12 @@ function formatMultiSuiteJunit(result, opts = {}) {
     const cases = (sr.case_runs ?? []).map((c) => {
       const name = c.case_name || c.run_id;
       const time = c.duration_ms ? (c.duration_ms / 1e3).toFixed(1) : "0";
-      const base = { name, time, url: c.url || void 0 };
+      const base = {
+        name,
+        time,
+        url: c.url || void 0,
+        steps_not_checked: c.steps_not_checked
+      };
       switch (c.status) {
         case "passed":
           return { ...base, verdict: "passed" };
@@ -1039,8 +1098,13 @@ ${chalk5.dim("Matched:")} ${totalMatched} suite(s)${matchedNames.length ? " \u20
   } else if (opts.output === "junit") {
     process.stdout.write(formatMultiSuiteJunit(last) + "\n");
   } else {
+    const totalNotChecked = last.suite_runs.reduce(
+      (n, sr) => n + (sr.case_runs ?? []).reduce((m, cr) => m + (cr.steps_not_checked ?? 0), 0),
+      0
+    );
+    const aggregateNote = totalNotChecked > 0 ? chalk5.yellow(` (${stepsNotCheckedLabel(totalNotChecked)})`) : "";
     process.stdout.write(`
-Aggregate: ${last.status}
+Aggregate: ${last.status}${aggregateNote}
 `);
     for (const sr of last.suite_runs) {
       const mark = sr.status === "passed" ? chalk5.green("\u2713") : chalk5.red("\u2717");
@@ -1056,8 +1120,9 @@ Aggregate: ${last.status}
       for (const cr of sr.case_runs ?? []) {
         const caseMark = cr.status === "passed" ? chalk5.green("\u2713") : cr.status === "cancelled" || cr.status === "pending" || cr.status === "running" ? chalk5.yellow("-") : chalk5.red("\u2717");
         const dur = cr.duration_ms ? chalk5.dim(` (${(cr.duration_ms / 1e3).toFixed(1)}s)`) : "";
+        const unchecked = notCheckedSuffix(cr.status, cr.steps_not_checked);
         process.stdout.write(
-          `      ${caseMark} ${cr.case_name ?? "Case"} ${cr.status ?? "?"}${dur} ${chalk5.dim(cr.url)}
+          `      ${caseMark} ${cr.case_name ?? "Case"} ${cr.status ?? "?"}${unchecked ? chalk5.yellow(unchecked) : ""}${dur} ${chalk5.dim(cr.url)}
 `
         );
         if (cr.error_message && cr.status !== "passed") {
@@ -4079,7 +4144,7 @@ program.command("dev-reload").description(
 program.command("mcp").description(
   "Run an MCP stdio server exposing the agent verbs (tap, swipe, type, ...) plus dev_start/dev_stop/dev_reload as tools"
 ).option("--token <token>", "API token (or set VIBEVIEW_API_TOKEN)").action(async (opts) => {
-  const { runMcpServer } = await import("./server-FJSBWEHZ.js");
+  const { runMcpServer } = await import("./server-RDGEVJD4.js");
   await runMcpServer({ ...opts, version: pkg.version });
 });
 program.command("agent-setup").description(