@swmansion/argent 0.31.2-next.1 → 0.31.2-next.3

package/dist/mcp-server.mjs

@@ -25593,6 +25593,36 @@ var TOOL_ID_ALIASES = Object.freeze({
 function canonicalToolId(id) {
   return Object.hasOwn(TOOL_ID_ALIASES, id) ? TOOL_ID_ALIASES[id] : id;
 }
+var TOOL_PARAM_ALIASES = Object.freeze({
+  "e2e-execute": Object.freeze({ flow_path: "e2e_test_path", flow_file: "e2e_test_file" }),
+  "e2e-read-prerequisite": Object.freeze({
+    flow_path: "e2e_test_path",
+    flow_file: "e2e_test_file"
+  })
+});
+var ToolParamAliasError = class extends Error {
+  constructor(former, current) {
+    super(`Pass ${current} or its former name ${former}, not both.`);
+    this.name = "ToolParamAliasError";
+  }
+};
+function toolParamAliases(tool) {
+  const id = canonicalToolId(tool);
+  return Object.hasOwn(TOOL_PARAM_ALIASES, id) ? TOOL_PARAM_ALIASES[id] : {};
+}
+function canonicalToolArgs(tool, args) {
+  if (typeof args !== "object" || args === null || Array.isArray(args)) return args;
+  const record2 = args;
+  let out;
+  for (const [former, current] of Object.entries(toolParamAliases(tool))) {
+    if (record2[former] === void 0) continue;
+    if (record2[current] !== void 0) throw new ToolParamAliasError(former, current);
+    out ??= { ...record2 };
+    out[current] = out[former];
+    delete out[former];
+  }
+  return out ?? args;
+}
 
 // ../registry/src/flow-file-refs.ts
 async function canonicalFlowPath(p) {
@@ -25662,7 +25692,13 @@ async function classifyOnDiskSpelling(dir, base, addressable = FLOW_FILE_NAME_PA
 }
 function nestedFlowTarget(args) {
   if (typeof args !== "object" || args === null || Array.isArray(args)) return void 0;
-  const { name, project_root: projectRoot, flow_path: flowPath } = args;
+  let canonical;
+  try {
+    canonical = canonicalToolArgs("e2e-execute", args);
+  } catch {
+    return void 0;
+  }
+  const { name, project_root: projectRoot, e2e_test_path: flowPath } = canonical;
   if (flowPath === void 0) {
     if (typeof name !== "string" || !FLOW_NAME_PATTERN.test(name)) return void 0;
     if (!isResolvedAbsolute(projectRoot)) return void 0;
@@ -25671,7 +25707,7 @@ function nestedFlowTarget(args) {
   }
   if (name !== void 0 || !isResolvedAbsolute(flowPath)) return void 0;
   if (!FLOW_FILE_NAME_PATTERN.test(path8.basename(flowPath))) return void 0;
-  return { kind: "flow_path", path: flowPath };
+  return { kind: "e2e_test_path", path: flowPath };
 }
 function isResolvedAbsolute(value) {
   return typeof value === "string" && path8.isAbsolute(value) && !value.split(/[\\/]+/).includes("..");
@@ -26611,12 +26647,18 @@ function isResolvedAbsolute2(value) {
   return typeof value === "string" && path11.isAbsolute(value) && !value.split(/[\\/]+/).includes("..");
 }
 function namesValidFlow(args) {
-  const { project_root, flow_path, name } = args;
+  let canonical;
+  try {
+    canonical = canonicalToolArgs("e2e-execute", args);
+  } catch {
+    return false;
+  }
+  const { project_root, e2e_test_path: testPath, name } = canonical;
   if (!isResolvedAbsolute2(project_root)) return false;
   if (name === void 0) {
-    return isResolvedAbsolute2(flow_path) && FLOW_FILE_NAME_PATTERN.test(path11.basename(flow_path));
+    return isResolvedAbsolute2(testPath) && FLOW_FILE_NAME_PATTERN.test(path11.basename(testPath));
   }
-  return flow_path === void 0 && typeof name === "string" && FLOW_NAME_PATTERN.test(name);
+  return testPath === void 0 && typeof name === "string" && FLOW_NAME_PATTERN.test(name);
 }
 function savedFlowProject(file) {
   if (!path11.isAbsolute(file)) return null;
@@ -26790,8 +26832,8 @@ function memberCollector(roots, opts) {
 `));
   const emit = (member, sent) => {
     members2.push(member);
-    const subject = member.role === "flow" ? member.canonical : member.key;
-    if (logging) log(printable(`[flow-files] ${member.role} ${subject}: ${sent}`));
+    const [label, subject] = member.role === "flow" ? ["e2e test", member.canonical] : [member.role, member.key];
+    if (logging) log(printable(`[e2e-files] ${label} ${subject}: ${sent}`));
   };
   async function flow(anchorDir, target, onlyPresent) {
     const key = flowMemberKey(anchorDir, target);
@@ -26957,7 +26999,7 @@ function recordedStep(args) {
   if (typeof text !== "string") return void 0;
   let parsed;
   try {
-    parsed = JSON.parse(text);
+    parsed = canonicalToolArgs(command, JSON.parse(text));
   } catch {
     return void 0;
   }
@@ -26980,7 +27022,7 @@ async function collectStepMembers(args, opts, recordingDirFor = e2eDirFor) {
     const self = await collector.flow(path11.dirname(recording), path11.basename(recording));
     let runArgs = step.args;
     const named = nestedFlowTarget(runArgs);
-    if (named?.kind === "flow_path") {
+    if (named?.kind === "e2e_test_path") {
       const stem = path11.basename(named.path, ".yaml");
       const base = path11.basename(named.path);
       const dir = path11.resolve(path11.dirname(named.path));
@@ -26989,7 +27031,7 @@ async function collectStepMembers(args, opts, recordingDirFor = e2eDirFor) {
       if (dirs.some((d) => path11.resolve(d) === dir)) {
         await collector.flow(path11.dirname(named.path), base);
         runArgs = { ...runArgs, name: stem };
-        delete runArgs.flow_path;
+        delete runArgs.e2e_test_path;
       }
     }
     const target = nestedFlowTarget(runArgs);
@@ -27017,7 +27059,21 @@ async function nameToE2ePathForOlderServer(tool, fileInputs, args) {
   const picked = await pickE2eFile(target.paths);
   if (!picked.exists || picked.legacy) return args;
   const { name: _name, ...rest } = args;
-  return { ...rest, flow_path: picked.path };
+  return { ...rest, e2e_test_path: picked.path };
+}
+function formerParamsForOlderServer(meta2, args) {
+  const listed = meta2.inputSchema.properties;
+  if (typeof args !== "object" || args === null || Array.isArray(args)) return args;
+  if (typeof listed !== "object" || listed === null) return args;
+  const record2 = args;
+  let out;
+  for (const [former, current] of Object.entries(toolParamAliases(meta2.name))) {
+    if (record2[current] === void 0 || current in listed || !(former in listed)) continue;
+    out ??= { ...record2 };
+    out[former] = out[current];
+    delete out[current];
+  }
+  return out ?? args;
 }
 async function stepForOlderServer(args, listedId) {
   if (typeof args !== "object" || args === null || Array.isArray(args)) return args;
@@ -27275,12 +27331,13 @@ function createToolsClient(options = {}) {
   async function callTool(name, args, opts) {
     opts?.signal?.throwIfAborted();
     const { url, token, remote } = await route();
-    let finalArgs = args;
+    let finalArgs = canonicalToolArgs(name, args);
     const tools = await fetchTools({ signal: opts?.signal });
     const meta2 = findTool(tools, name);
     const olderServer = meta2 !== null && !tools.some((t) => t.name === canonicalToolId(meta2.name));
     if (meta2) {
-      finalArgs = await nameToE2ePathForOlderServer(meta2.name, meta2.fileInputs, args);
+      finalArgs = await nameToE2ePathForOlderServer(meta2.name, meta2.fileInputs, finalArgs);
+      finalArgs = formerParamsForOlderServer(meta2, finalArgs);
     }
     if (olderServer && canonicalToolId(meta2.name) === "e2e-add-step") {
       finalArgs = await stepForOlderServer(finalArgs, (tool) => findTool(tools, tool)?.name);
@@ -27308,7 +27365,7 @@ function createToolsClient(options = {}) {
       if (filesLogEnabled()) {
         for (const file of applied.written) {
           if (baselineDirs.includes(path12.dirname(file))) {
-            diagnose(`[flow-files] baseline ${file}: written`);
+            diagnose(`[e2e-files] baseline ${file}: written`);
           }
         }
       }
@@ -28678,12 +28735,13 @@ function stepLabel(step) {
 }
 async function flowRunToMcpContent(result, ctx) {
   const blocks = [];
+  const name = result.e2eTest ?? result.flow;
   if (result.executionPrerequisite) {
     blocks.push({ type: "text", text: `Prerequisite: ${result.executionPrerequisite}` });
   }
   blocks.push({
     type: "text",
-    text: `Running e2e test "${result.flow}"${result.device ? ` on ${result.device}` : ""} (${result.steps.length} steps)`
+    text: `Running e2e test "${name}"${result.device ? ` on ${result.device}` : ""} (${result.steps.length} steps)`
   });
   if (typeof result.warning === "string")
     blocks.push({ type: "text", text: `\u26A0 ${result.warning}` });
@@ -28728,7 +28786,7 @@ async function flowRunToMcpContent(result, ctx) {
       text: `${result.ok ? "PASS" : "FAIL"} \u2014 ${result.passed ?? 0} passed, ${result.failed ?? 0} failed, ${result.errored ?? 0} errored, ${result.skipped ?? 0} skipped${note}${durationSuffix(result.durationMs)}`
     });
   } else {
-    blocks.push({ type: "text", text: `E2E test "${result.flow}" complete.` });
+    blocks.push({ type: "text", text: `E2E test "${name}" complete.` });
   }
   return blocks;
 }
@@ -29339,7 +29397,7 @@ ${desc}` }];
         deviceId: getDeviceIdFromArgs(params.arguments)
       };
       let content;
-      if (canonicalToolId(params.name) === "e2e-execute" && result && typeof result === "object" && "flow" in result && "steps" in result) {
+      if (canonicalToolId(params.name) === "e2e-execute" && result && typeof result === "object" && ("e2eTest" in result || "flow" in result) && "steps" in result) {
         content = await flowRunToMcpContent(result, ctx);
       } else if (params.name === "screenshot-diff" && isScreenshotDiffResult(result)) {
         content = await screenshotDiffToMcpContent(result, ctx);

package/dist/tool-args.generated.d.ts

@@ -223,14 +223,14 @@ export interface ArgentToolArgs {
         delayMs?: number;
     };
     "e2e-execute": {
-        /** Name of a saved e2e test to run from `.argent/e2e`, or `.argent/flows` when it is not there (e.g. "settings-explore"). Omit when flow_path is set. */
+        /** Name of a saved e2e test to run from `.argent/e2e`, or `.argent/flows` when it is not there (e.g. "settings-explore"). Omit when e2e_test_path is set. */
         name?: string;
-        /** Absolute path to the calling agent's project root: the cwd it is working in. With name, the saved e2e test is read from `.argent/e2e/<name>.yaml`, else `.argent/flows/<name>.yaml`, under this root; with flow_path, the e2e test, its run: siblings, its script: paths and baselines all resolve beside the YAML instead, so pass the agent's cwd. A script still RUNS in this root whichever source was used. */
+        /** Absolute path to the calling agent's project root: the cwd it is working in. With name, the saved e2e test is read from `.argent/e2e/<name>.yaml`, else `.argent/flows/<name>.yaml`, under this root; with e2e_test_path, the e2e test, its run: siblings, its script: paths and baselines all resolve beside the YAML instead, so pass the agent's cwd. A script still RUNS in this root whichever source was used. */
         project_root: string;
         /** Path to the e2e test .yaml as readable by the tool-server. Internal: the argent client derives it from project_root and name automatically; leave unset. */
-        flow_file?: string;
+        e2e_test_file?: string;
         /** Omit when name is set. Absolute path to an e2e test .yaml on the client. Over a link (argent link or ARGENT_TOOLS_URL), the argent client uploads the file, its run: fragments and the snapshot baselines of each run, and the tool-server runs those copies. Keep run: fragments under project_root or beside the e2e test file. With updateBaselines, the client writes the new baselines when the run ends. The client also uploads each absolute .png or .yaml path that a tool: step takes as a file argument, when the file is under project_root or beside the e2e test file. A tool: e2e-execute step that names an e2e test with name and an absolute project_root without a .. segment runs: the client also uploads that e2e test and its own files. Over a link, the uploaded e2e tests must not have: script: steps; tool: steps that run an e2e test another way or record an e2e test; tool: steps that take a directory, an app or an output directory; tool: steps whose tool builds a file path from several arguments; tool: steps that name a relative path or a file other than .png or .yaml. Without a link, the tool-server reads the files in place and all step kinds run. */
-        flow_path?: string;
+        e2e_test_path?: string;
         /** Device id to run against (iOS UDID, Android/Vega serial, Chromium id), as list-devices reports it. Auto-detected when omitted, but only when exactly one booted device matches (optionally narrowed by `platform`); with several booted the run fails and lists them, so pass this explicitly whenever more than one device is up. */
         device?: string;
         /** Restrict auto-detection to this platform when several devices are booted. `ios` selects local simulators only; pass `ios-remote` to select a remote one. `chromium` does more than filter: with no `device` it SELECTS the self-boot branch for a standalone e2e test - the runner boots an Electron instance from the `launch` step's chromium value and tears it down after the run (a single-key `launch: { chromium: … }` map selects it on its own, without this parameter). When it selects that branch it never falls back to device auto-detection (a fragment, or a standalone e2e test's launch map with no `chromium` key, still does), and the launch value must be a real Electron app path on the tool-server host: a bare-string `launch:` - what the recorder writes - holds an installed-app bundle id, so passing `chromium` for one fails the whole run with `Electron boot: path does not exist`. Edit the launch to `{ chromium: <app path> }` first. */
@@ -239,6 +239,10 @@ export interface ArgentToolArgs {
         updateBaselines?: boolean;
         /** Set to true to confirm the execution prerequisite has been met. Required (LLM path) when a fragment defines an executionPrerequisite. */
         prerequisiteAcknowledged?: boolean;
+        /** @deprecated Renamed to `e2e_test_path`. */
+        flow_path?: string;
+        /** @deprecated Renamed to `e2e_test_file`. */
+        flow_file?: string;
     };
     "e2e-finish-recording": {
         /** Name of the e2e test being recorded: the one passed to e2e-start-recording. */
@@ -247,14 +251,18 @@ export interface ArgentToolArgs {
         project_root: string;
     };
     "e2e-read-prerequisite": {
-        /** Name of a saved e2e test to inspect from `.argent/e2e`, or `.argent/flows` when it is not there (e.g. "settings-explore"). Omit when flow_path is set. */
+        /** Name of a saved e2e test to inspect from `.argent/e2e`, or `.argent/flows` when it is not there (e.g. "settings-explore"). Omit when e2e_test_path is set. */
         name?: string;
-        /** Absolute path to the calling agent's project root: the cwd it is working in. With name, the saved e2e test is read from `.argent/e2e/<name>.yaml`, else `.argent/flows/<name>.yaml`, under this root; with flow_path, the prerequisite is read from that YAML instead, so pass the agent's cwd. */
+        /** Absolute path to the calling agent's project root: the cwd it is working in. With name, the saved e2e test is read from `.argent/e2e/<name>.yaml`, else `.argent/flows/<name>.yaml`, under this root; with e2e_test_path, the prerequisite is read from that YAML instead, so pass the agent's cwd. */
         project_root: string;
         /** Path to the e2e test .yaml as readable by the tool-server. Internal: the argent client derives it from project_root and name automatically; leave unset. */
-        flow_file?: string;
+        e2e_test_file?: string;
         /** Omit when name is set. Absolute path to an e2e test .yaml on the client. Over a link (argent link or ARGENT_TOOLS_URL), the argent client uploads the file and the tool-server reads the uploaded copy; without a link, the tool-server reads the file in place. Pass the path through the argent client; a raw path that the file-input boundary did not verify is rejected. Pass the same e2e test source here as to e2e-execute, so the prerequisite you read belongs to the e2e test that will run. */
+        e2e_test_path?: string;
+        /** @deprecated Renamed to `e2e_test_path`. */
         flow_path?: string;
+        /** @deprecated Renamed to `e2e_test_file`. */
+        flow_file?: string;
     };
     "e2e-start-recording": {
         /** Name for this e2e test (e.g. "settings-explore"): letters, digits, underscore and hyphen only. */