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

package/dist/cli-cmds.mjs

@@ -1304,15 +1304,20 @@ function setAtPath(obj, dottedKey, value) {
 }
 function deleteAtPath(obj, dottedKey) {
   const parts = splitKey(dottedKey);
-  let cur = obj;
+  const chain = [obj];
   for (let i2 = 0; i2 < parts.length - 1; i2++) {
-    const next = cur[parts[i2]];
+    const next = chain[i2][parts[i2]];
     if (!isPlainObject(next)) return false;
-    cur = next;
+    chain.push(next);
   }
+  const parent = chain[parts.length - 1];
   const leaf = parts[parts.length - 1];
-  if (!Object.hasOwn(cur, leaf)) return false;
-  delete cur[leaf];
+  if (!Object.hasOwn(parent, leaf)) return false;
+  delete parent[leaf];
+  for (let i2 = chain.length - 1; i2 >= 1; i2--) {
+    if (Object.keys(chain[i2]).length > 0) break;
+    delete chain[i2 - 1][parts[i2 - 1]];
+  }
   return true;
 }
 var LOCK_STALE_MS2 = 1e4;
@@ -1447,6 +1452,9 @@ function asString(raw) {
   const trimmed = raw.trim();
   return trimmed === "" ? void 0 : trimmed;
 }
+function asNumber(raw) {
+  return typeof raw === "number" && Number.isFinite(raw) ? raw : void 0;
+}
 function asStringArray(raw) {
   if (!Array.isArray(raw)) return void 0;
   const out = [];
@@ -1455,6 +1463,15 @@ function asStringArray(raw) {
   }
   return out;
 }
+var PARSER_EXPECTATIONS = /* @__PURE__ */ new Map([
+  [asBoolean, "a boolean (true or false)"],
+  [asString, "a non-empty string"],
+  [asNumber, "a number"],
+  [asStringArray, "an array of strings"]
+]);
+function describeExpectedValue(def) {
+  return def.expected ?? PARSER_EXPECTATIONS.get(def.parse);
+}
 var CONFIG_SCHEMA = [
   {
     key: "telemetry.enabled",
@@ -1560,12 +1577,18 @@ var ConfigScopeError = class extends Error {
   allowed;
 };
 var ConfigValidationError = class extends Error {
-  constructor(key) {
-    super(`Invalid value for config key "${key}".`);
+  constructor(key, expected, example) {
+    super(
+      expected ? `Invalid value for config key "${key}": expected ${expected}.` : `Invalid value for config key "${key}".`
+    );
     this.key = key;
+    this.expected = expected;
+    this.example = example;
     this.name = "ConfigValidationError";
   }
   key;
+  expected;
+  example;
 };
 var ConfigManagedElsewhereError = class extends Error {
   constructor(key, command) {
@@ -1582,7 +1605,8 @@ function setConfigValue(key, rawValue, scope = "global", options = {}, registry
   if (def.manageCommand) throw new ConfigManagedElsewhereError(key, def.manageCommand);
   if (!def.scopes.includes(scope)) throw new ConfigScopeError(key, scope, def.scopes);
   const parsed = def.parse(rawValue);
-  if (parsed === void 0) throw new ConfigValidationError(key);
+  if (parsed === void 0)
+    throw new ConfigValidationError(def.key, describeExpectedValue(def), def.example);
   updateConfig((config2) => setAtPath(config2, key, parsed), scope, options);
   return parsed;
 }
@@ -1607,6 +1631,8 @@ function listConfig(options = {}, registry = CONFIG_SCHEMA) {
     description: def.description,
     scopes: def.scopes,
     ...def.manageCommand ? { manageCommand: def.manageCommand } : {},
+    ...describeExpectedValue(def) ? { expected: describeExpectedValue(def) } : {},
+    ...def.example ? { example: def.example } : {},
     effective: getConfigValue(def, options),
     project: readScopeValue(def, "project", options),
     global: readScopeValue(def, "global", options)
@@ -9153,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 = [];
@@ -9222,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}`);
   }
@@ -11055,6 +11083,7 @@ the raw value stored at each scope.`);
 }
 function scopeDetail(e) {
   const parts = [`scopes: ${e.scopes.join(", ")}`];
+  if (e.expected) parts.push(`value: ${e.expected}${e.example ? `, e.g. ${e.example}` : ""}`);
   if (e.project !== void 0) parts.push(`project=${formatValuePlain(e.project)}`);
   if (e.global !== void 0) parts.push(`global=${formatValuePlain(e.global)}`);
   return parts.join("  \xB7  ");
@@ -11119,7 +11148,7 @@ parsed (e.g. \`true\`, \`42\`, \`["a","b"]\`); anything else is stored as a stri
     if (warning) console.error(import_picocolors2.default.yellow(warning));
     console.log(`Set ${import_picocolors2.default.bold(key)} = ${formatValuePlain(stored)} (${scopeLabel(targetScope)}).`);
   } catch (err) {
-    reportError(err);
+    reportError(err, () => suggestCorrectedSet(err, key, rawValue, scope));
   }
 }
 function cmdUnset(argv) {
@@ -11212,7 +11241,20 @@ function formatValue(value) {
   }
   return formatValuePlain(value);
 }
-function reportError(err) {
+function quoteForShell(value) {
+  if (/^[A-Za-z0-9._/@:+-]+$/.test(value)) return value;
+  return `'${value.replace(/'/g, `'\\''`)}'`;
+}
+function suggestCorrectedSet(err, key, rawValue, scope) {
+  if (!(err instanceof ConfigValidationError)) return null;
+  const def = getConfigDefinition(key);
+  if (!def) return null;
+  const wrapped = def.parse([rawValue]);
+  if (wrapped === void 0) return null;
+  const scopeFlag = scope ? ` --scope ${scope}` : "";
+  return `argent config set ${key} ${quoteForShell(JSON.stringify([rawValue]))}${scopeFlag}`;
+}
+function reportError(err, suggest) {
   if (err instanceof ConfigManagedElsewhereError) {
     console.error(`Error: ${err.message} Use \`${err.command}\` instead.`);
   } else if (err instanceof UnknownConfigKeyError || err instanceof ConfigScopeError || err instanceof ConfigValidationError) {
@@ -11220,6 +11262,15 @@ function reportError(err) {
     if (err instanceof UnknownConfigKeyError) {
       console.error(`Run \`argent config list\` to see available keys.`);
     }
+    if (err instanceof ConfigValidationError) {
+      const corrected = suggest?.() ?? null;
+      if (corrected) {
+        console.error(`Did you mean: ${corrected}`);
+      } else if (err.example) {
+        console.error(`Example: argent config set ${err.key} ${quoteForShell(err.example)}`);
+      }
+      console.error(`Run \`argent config list\` to see each key's expected value.`);
+    }
   } else {
     console.error(`Error: ${err instanceof Error ? err.message : String(err)}`);
   }

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.14",
+  "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",