@swmansion/argent 0.18.1-next.15 → 0.18.1-next.16

package/dist/cli-cmds.mjs

@@ -9179,8 +9179,10 @@ function renderUnderStepLine(s, n2, text2) {
 function renderSummary(report, opts = {}) {
   const warnings = report.steps.filter((s) => s.warning).length;
   const warningsNote = warnings ? `, ${warnings} warning${warnings === 1 ? "" : "s"}` : "";
-  const where = opts.withDevice ? ` on ${report.device}` : "";
-  return `${report.ok ? "PASS" : "FAIL"}${where} \u2014 ${report.passed} passed, ${report.failed} failed, ${report.errored} errored, ${report.skipped} skipped${warningsNote}`;
+  const where = opts.withDevice && report.device ? ` on ${report.device}` : "";
+  const nothingCounted = report.ok && report.passed + report.failed + report.errored + report.skipped === 0;
+  const note = nothingCounted ? " (no test steps)" : "";
+  return `${report.ok ? "PASS" : "FAIL"}${where} \u2014 ${report.passed} passed, ${report.failed} failed, ${report.errored} errored, ${report.skipped} skipped${warningsNote}${note}`;
 }
 function renderArtifactLines(report) {
   const lines = [];
@@ -9248,7 +9250,7 @@ function exitAfterFlush(code, streams = [process.stdout, process.stderr]) {
 }
 function renderReport(report) {
   const lines = [];
-  lines.push(`Flow "${report.flow}" on ${report.device}`);
+  lines.push(`Flow "${report.flow}"${report.device ? ` on ${report.device}` : ""}`);
   if (report.executionPrerequisite) {
     lines.push(`  assumes: ${report.executionPrerequisite}`);
   }

package/dist/mcp-server.mjs

@@ -19293,9 +19293,11 @@ async function flowRunToMcpContent(result, ctx) {
     }
   }
   if (result.ok !== void 0) {
+    const counted = (result.passed ?? 0) + (result.failed ?? 0) + (result.errored ?? 0) + (result.skipped ?? 0);
+    const note = result.ok && counted === 0 ? " (no test steps)" : "";
     blocks.push({
       type: "text",
-      text: `${result.ok ? "PASS" : "FAIL"} \u2014 ${result.passed ?? 0} passed, ${result.failed ?? 0} failed, ${result.errored ?? 0} errored, ${result.skipped ?? 0} skipped`
+      text: `${result.ok ? "PASS" : "FAIL"} \u2014 ${result.passed ?? 0} passed, ${result.failed ?? 0} failed, ${result.errored ?? 0} errored, ${result.skipped ?? 0} skipped${note}`
     });
   } else {
     blocks.push({ type: "text", text: `Flow "${result.flow}" complete.` });

package/dist/tool-server.cjs

@@ -148170,6 +148170,7 @@ var path31 = __toESM(require("node:path"));
 // ../tool-server/src/tools/flows/flow-device.ts
 init_src();
 var DEVICE_BIND_KEYS = ["udid", "device_id"];
+var DEVICE_ARG_KEYS = [...DEVICE_BIND_KEYS, "device"];
 function deviceEntryId(d) {
   if (d.platform === "ios") return d.udid;
   if (d.platform === "chromium") return d.id;
@@ -148225,6 +148226,43 @@ function stripDeviceKeys(args) {
   for (const k of DEVICE_BIND_KEYS) delete out[k];
   return out;
 }
+function stepRequiresDevice(registry2, step) {
+  switch (step.kind) {
+    case "echo":
+    case "wait":
+      return false;
+    case "tool":
+      return toolRequiresDevice(registry2, step.name);
+    case "when":
+    case "run":
+    case "launch":
+    case "tap":
+    case "long-press":
+    case "type":
+    case "await":
+    case "assert":
+    case "scroll-to":
+    case "pinch":
+    case "rotate":
+    case "snapshot":
+      return true;
+    default: {
+      const unclassified = step;
+      void unclassified;
+      return true;
+    }
+  }
+}
+function flowRequiresDevice(registry2, steps) {
+  return steps.some((step) => stepRequiresDevice(registry2, step));
+}
+function toolRequiresDevice(registry2, toolName) {
+  const toolDef = registry2.getTool(toolName);
+  if (!toolDef) return true;
+  const props = toolDef.inputSchema?.properties;
+  if (!props) return false;
+  return DEVICE_ARG_KEYS.some((k) => k in props);
+}
 function bindDeviceArgs(registry2, toolName, deviceId, args) {
   const toolDef = registry2.getTool(toolName);
   const props = toolDef?.inputSchema?.properties;
@@ -151834,7 +151872,8 @@ async function treeSourceGate(registry2, device, bundleId, signal) {
   return null;
 }
 async function runLaunch(state3, app) {
-  const { registry: registry2, device, signal } = state3;
+  const env = deviceEnv(state3);
+  const { registry: registry2, device, signal } = env;
   if (device.platform === "chromium") {
     if (state3.chromiumLaunched) {
       return {
@@ -151871,7 +151910,7 @@ async function runLaunch(state3, app) {
     };
   }
   try {
-    await invokeOnDevice(state3, "restart-app", { bundleId });
+    await invokeOnDevice(env, "restart-app", { bundleId });
   } catch (err) {
     if (signal?.aborted) return ABORTED_OUTCOME;
     return { ok: false, reason: `restart-app failed: ${errMsg2(err)}` };
@@ -151882,6 +151921,12 @@ async function runLaunch(state3, app) {
   if (gate) return { ok: false, reason: gate };
   return { ok: true };
 }
+function deviceEnv(state3) {
+  if (!state3.device) {
+    throw new Error("internal: a step that acts on a device ran in a flow resolved as device-free");
+  }
+  return { ...state3, device: state3.device };
+}
 function createRunFlowTool(registry2) {
   return {
     id: "flow-execute",
@@ -151940,11 +151985,13 @@ returns a notice with the prerequisite instead of running.`,
       }
       const resolved = await resolveRunDevice(registry2, ctx, flow, params, flowsDir);
       const device = resolved.device;
-      const env = { registry: registry2, ctx, device, signal };
-      const statusBarPinned = await pinStatusBar(device);
-      if (device.platform === "chromium") await frontChromiumPage(registry2, device);
+      const statusBarPinned = device !== null && await pinStatusBar(device);
+      if (device?.platform === "chromium") await frontChromiumPage(registry2, device);
       const state3 = {
-        ...env,
+        registry: registry2,
+        ctx,
+        device,
+        signal,
         flowsDir,
         topFlowName: params.name,
         updateBaselines: Boolean(params.updateBaselines),
@@ -151964,10 +152011,16 @@ returns a notice with the prerequisite instead of running.`,
         });
       } finally {
         aborted2 = state3.signal?.aborted === true;
-        if (state3.pinned) await restoreStatusBar(device);
+        if (state3.pinned && device) await restoreStatusBar(device);
         if (resolved.booted) await teardownBootedChromium(registry2, resolved.booted);
       }
-      return summarize(params.name, device.id, flow.executionPrerequisite, state3.reports, aborted2);
+      return summarize(
+        params.name,
+        device?.id ?? "",
+        flow.executionPrerequisite,
+        state3.reports,
+        aborted2
+      );
     }
   };
 }
@@ -151978,6 +152031,9 @@ async function resolveRunDevice(registry2, ctx, flow, params, flowDir) {
       const booted = await bootChromiumForFlow(spec, flowDir);
       return { device: resolveDevice(booted.deviceId), booted };
     }
+    if (!flowRequiresDevice(registry2, flow.steps)) {
+      return { device: null, booted: null };
+    }
   }
   const device = await resolveFlowDevice(registry2, ctx, {
     device: params.device,
@@ -152133,6 +152189,20 @@ async function execSteps(state3, steps, scope) {
       if (step.kind === "when") reportBlockSkipped(state3, step.steps, childScope(scope));
       continue;
     }
+    if (!state3.device && stepRequiresDevice(state3.registry, step)) {
+      state3.stopped = true;
+      pushReport(state3, {
+        index,
+        kind: step.kind,
+        status: "error",
+        flow: scope.flow,
+        target: stepTarget(step),
+        ...depthOf(scope),
+        reason: `step needs a device but the flow was resolved as device-free \u2014 pass an explicit device`
+      });
+      if (step.kind === "when") reportBlockSkipped(state3, step.steps, childScope(scope));
+      continue;
+    }
     if (state3.signal?.aborted) {
       state3.stopped = true;
       pushReport(state3, {
@@ -152193,10 +152263,11 @@ async function execWhenStep(state3, step, scope) {
   const inner = childScope(scope);
   let met;
   if (step.condition.kind === "platform") {
-    const platform = state3.device.platform === "ios-remote" ? "ios" : state3.device.platform;
+    const guardEnv = deviceEnv(state3);
+    const platform = guardEnv.device.platform === "ios-remote" ? "ios" : guardEnv.device.platform;
     met = platform === step.condition.platform;
   } else {
-    const probe3 = await probeWhenCondition(state3, step.condition);
+    const probe3 = await probeWhenCondition(deviceEnv(state3), step.condition);
     if (probe3.aborted) {
       pushReport(state3, { ...marker, status: "skip", reason: "run aborted" });
       reportBlockSkipped(state3, step.steps, inner, "run aborted");
@@ -152288,7 +152359,7 @@ async function execLeafStep(state3, step, index, scope) {
     case "pinch":
     case "rotate": {
       try {
-        const r = await runDirective(state3, step);
+        const r = await runDirective(deviceEnv(state3), step);
         if (r.aborted) return { ...base, status: "skip", reason: r.reason };
         return { ...base, status: r.ok ? "pass" : "fail", reason: r.reason };
       } catch (err) {
@@ -152303,7 +152374,7 @@ async function execLeafStep(state3, step, index, scope) {
     }
     case "snapshot": {
       try {
-        const r = await runSnapshot(state3, {
+        const r = await runSnapshot(deviceEnv(state3), {
           flowsDir: state3.flowsDir,
           flowName: state3.topFlowName,
           name: step.name,
@@ -152323,7 +152394,7 @@ async function execLeafStep(state3, step, index, scope) {
       }
     }
     case "tool": {
-      const args = bindDeviceArgs(registry2, step.name, device.id, step.args);
+      const args = bindDeviceArgs(registry2, step.name, device?.id ?? "", step.args);
       const outputHint = registry2.getTool(step.name)?.outputHint;
       if (step.delayMs && !await sleepOrAbort(step.delayMs, signal)) {
         return { ...base, status: "skip", tool: step.name, reason: "run aborted during delay" };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@swmansion/argent",
-  "version": "0.18.1-next.15",
+  "version": "0.18.1-next.16",
   "mcpName": "io.github.software-mansion/argent",
   "description": "MCP server for iOS Simulator and Android Emulator control",
   "license": "Apache-2.0",