@qawolf/cli 1.0.0 → 1.0.1

package/dist/cli.js

@@ -155308,7 +155308,6 @@ Hint: pass a pattern scoped to one package, e.g \`qawolf flows run '.qawolf/<env
 ${stderr}`
   },
   dotenv: {
-    invalidKey: (key) => `Cannot serialize env var with invalid key: ${JSON.stringify(key)}`,
     unparseableLine: (line) => `Cannot parse .env line: ${JSON.stringify(line)}`
   }
 };
@@ -155370,6 +155369,8 @@ var runnerMessages = {
   flowsFailed: (count) => `${count} flow(s) failed`,
   invalidRetries: (retries) => `retries must be a non-negative integer, got ${String(retries)}`,
   manifestStampReadFailed: (file, message) => `failed to read manifest stamp for ${file}: ${message}`,
+  harCleanupFailed: (file, message) => `failed to delete HAR file ${file} (retain-on-failure); remove it manually: ${message}`,
+  traceCleanupFailed: (file, message) => `failed to delete trace file ${file} (retain-on-failure); remove it manually: ${message}`,
   notSupportedInCli: (name) => `${name} is not supported in the CLI runner`,
   notAvailableLocally: (name) => `${name} is not available in local runs yet.`,
   noDefaultExport: (flowPath) => `No default export found in "${flowPath}"`,
@@ -155465,6 +155466,8 @@ function envPaths(name, { suffix = "nodejs" } = {}) {
 }
 
 // src/core/paths.ts
+var qawolfDir = ".qawolf";
+var defaultOutputDir = `${qawolfDir}/output`;
 var _paths;
 function getConfigDir() {
   _paths ??= envPaths("qawolf");
@@ -173643,7 +173646,7 @@ function registerAuthCommand(program2, signals) {
 // package.json
 var package_default = {
   name: "@qawolf/cli",
-  version: "1.0.0",
+  version: "1.0.1",
   description: "Run and manage QA Wolf flows from the terminal, CI, or an AI agent",
   keywords: [
     "automation",
@@ -173665,7 +173668,7 @@ var package_default = {
     url: "git+https://github.com/qawolf/cli.git"
   },
   bin: {
-    qawolf: "./dist/cli.js"
+    qawolf: "dist/cli.js"
   },
   files: [
     "LICENSE",
@@ -173679,7 +173682,7 @@ var package_default = {
   },
   scripts: {
     build: "bun run generate && bun scripts/build.ts",
-    "build:binary": "bun scripts/buildBinary.ts",
+    "build:binary": "bun run generate && bun scripts/buildBinary.ts",
     dev: "bun run src/main.ts",
     format: "oxfmt .",
     "format:check": "oxfmt --check .",
@@ -173690,7 +173693,7 @@ var package_default = {
     lint: "oxlint",
     "lint:fix": "oxlint --fix",
     postbuild: "./scripts/postbuild.sh",
-    prepare: "git rev-parse --is-inside-work-tree >/dev/null 2>&1 && git config core.hooksPath .githooks || true",
+    prepare: "git config core.hooksPath .githooks || true",
     test: "bun run generate && bun test",
     "test:watch": "bun test --watch",
     typecheck: "bun run generate && tsc --noEmit",
@@ -175929,19 +175932,14 @@ async function applyTeamStorageRewrite(rootDir, fs2 = makeDefaultFs()) {
 // src/domains/flows/pull/envVars.ts
 import { join as join19 } from "node:path";
 
-// src/domains/flows/dotenv.ts
-var envKeyPattern = /^[A-Za-z_][A-Za-z0-9_]*$/;
-var lineRe = /^([A-Za-z_][A-Za-z0-9_]*)="((?:[^"\\]|\\.)*)"$/;
+// src/core/dotenv.ts
+var bareKeyPattern = /^[A-Za-z_][A-Za-z0-9_]*$/;
+var lineRe = /^(?:([A-Za-z_][A-Za-z0-9_]*)|"((?:[^"\\]|\\.)*)")="((?:[^"\\]|\\.)*)"$/;
 function serializeDotenv(vars) {
   const keys = Object.keys(vars).sort();
-  for (const key of keys) {
-    if (!envKeyPattern.test(key)) {
-      throw new Error(flowsMessages.dotenv.invalidKey(key));
-    }
-  }
   if (keys.length === 0)
     return "";
-  return `${keys.map((key) => `${key}=${quote(vars[key] ?? "")}`).join(`
+  return `${keys.map((key) => `${bareKeyPattern.test(key) ? key : quote(key)}=${quote(vars[key] ?? "")}`).join(`
 `)}
 `;
 }
@@ -175961,7 +175959,7 @@ function parseDotenv(content) {
     if (!match) {
       throw new Error(flowsMessages.dotenv.unparseableLine(line));
     }
-    out[match[1]] = unquote(match[2]);
+    out[match[1] ?? unquote(match[2])] = unquote(match[3]);
   }
   return out;
 }
@@ -180050,14 +180048,30 @@ function createLaunch({
   contextSetup,
   launchBrowserOpts,
   signals,
-  timeout
+  timeout,
+  traceMode = "off",
+  tracePath
 }) {
   const openBrowsers = [];
   const openContexts = [];
-  const unregisters = [];
-  const trackContext = (ctx) => {
+  const tracingEnabled = traceMode !== "off";
+  let closed = false;
+  const closeAll = async () => {
+    if (closed)
+      return;
+    closed = true;
+    if (tracingEnabled && tracePath !== undefined) {
+      await Promise.allSettled(openContexts.map((c2) => c2.tracing.stop({ path: tracePath })));
+    }
+    await Promise.allSettled(openContexts.map((c2) => c2.close()));
+    await Promise.allSettled(openBrowsers.map((b3) => b3.close()));
+  };
+  const unregister = signals.register(closeAll);
+  const trackContext = async (ctx) => {
     openContexts.push(ctx);
-    unregisters.push(signals.register(() => ctx.close()));
+    if (tracingEnabled) {
+      await ctx.tracing.start({ screenshots: true, snapshots: true });
+    }
   };
   const launch = async (launchOpts) => {
     const browserName = normalizeBrowserName(launchOpts?.browser);
@@ -180069,25 +180083,20 @@ function createLaunch({
         ...contextSetup
       });
       context2.setDefaultTimeout(timeout);
-      trackContext(context2);
+      await trackContext(context2);
       const page = await context2.newPage();
       return { browserType: browserName, context: context2, page };
     }
     const browser = await bt2.launch(launchBrowserOpts);
     openBrowsers.push(browser);
-    unregisters.push(signals.register(() => browser.close()));
     const context = await browser.newContext(contextSetup);
     context.setDefaultTimeout(timeout);
-    trackContext(context);
+    await trackContext(context);
     return { browser, browserType: browserName, context };
   };
   const cleanup = async () => {
-    for (const unreg of unregisters)
-      unreg();
-    await Promise.allSettled([
-      ...openContexts.map((c2) => c2.close()),
-      ...openBrowsers.map((b3) => b3.close())
-    ]);
+    unregister();
+    await closeAll();
   };
   return { launch, cleanup };
 }
@@ -180231,7 +180240,7 @@ async function findFlowsRunnerPath(flowPath, fs3) {
   }
 }
 var initCache = new Map;
-async function doInit(flowPath, fs3) {
+async function doInit(flowPath, timeout, fs3) {
   const runnerPath = await findFlowsRunnerPath(flowPath, fs3);
   const mod = await import(pathToFileURL2(runnerPath).href);
   if (typeof mod.configureFlowRuntime !== "function")
@@ -180242,14 +180251,15 @@ async function doInit(flowPath, fs3) {
       schemaVersion: 1,
       runnerName: "node20WithPlaywright",
       meta: "legacy"
-    }
+    },
+    webExpectAttributes: { defaultExpectTimeoutMs: timeout }
   });
 }
-function initFlowRuntime(flowPath, fs3 = makeDefaultFs()) {
+function initFlowRuntime(flowPath, options, fs3 = makeDefaultFs()) {
   const startDir = path6.dirname(flowPath);
   let p3 = initCache.get(startDir);
   if (!p3) {
-    p3 = doInit(flowPath, fs3);
+    p3 = doInit(flowPath, options.timeout, fs3);
     initCache.set(startDir, p3);
   }
   return p3;
@@ -180260,6 +180270,9 @@ import path7 from "node:path";
 function harFlowPath(outputDir, flowName) {
   return path7.join(outputDir, "har", `${flowName}.har`);
 }
+function traceFlowPath(outputDir, flowName) {
+  return path7.join(outputDir, "trace", `${flowName}.zip`);
+}
 function buildHarContextOpts(harPath, harContent) {
   return {
     recordHar: {
@@ -180277,6 +180290,23 @@ async function initHar(fs3, options, flowName) {
   await fs3.mkdir(path7.dirname(harPath), { recursive: true });
   return { harMode, harPath };
 }
+async function initTrace(fs3, options, flowName) {
+  const traceMode = options.trace ?? "off";
+  if (traceMode === "off")
+    return { traceMode, tracePath: undefined };
+  const tracePath = traceFlowPath(options.outputDir, flowName);
+  await fs3.mkdir(path7.dirname(tracePath), { recursive: true });
+  return { traceMode, tracePath };
+}
+async function maybeCleanupTrace(fs3, tracePath, passed, traceMode, warn) {
+  if (traceMode !== "retain-on-failure" || !passed)
+    return;
+  try {
+    await fs3.unlink(tracePath);
+  } catch (err) {
+    warn?.(runnerMessages.traceCleanupFailed(tracePath, errorMessage(err)));
+  }
+}
 function buildContextSetup(videoSize, options, harPath) {
   const videosDir = options.artifactDir ?? path7.join(options.outputDir, "videos");
   const harOpts = harPath !== undefined ? buildHarContextOpts(harPath, options.harContent ?? "omit") : {};
@@ -180287,12 +180317,14 @@ function buildContextSetup(videoSize, options, harPath) {
     ...harOpts
   } : { viewport: videoSize, screen: videoSize, ...harOpts };
 }
-async function maybeCleanupHar(fs3, harPath, passed, harMode) {
+async function maybeCleanupHar(fs3, harPath, passed, harMode, warn) {
   if (harMode !== "retain-on-failure" || !passed)
     return;
   try {
     await fs3.unlink(harPath);
-  } catch {}
+  } catch (err) {
+    warn?.(runnerMessages.harCleanupFailed(harPath, errorMessage(err)));
+  }
 }
 
 // src/domains/runner/runWebFlow.ts
@@ -180301,12 +180333,13 @@ async function runWebFlow({
   options,
   flowPath
 }) {
-  await initFlowRuntime(flowPath);
+  await initFlowRuntime(flowPath, { timeout: options.timeout });
   const exported = await loadFlowDefault(flowPath);
   const isLegacy = typeof exported === "function";
   const flowName = isLegacy ? path8.basename(flowPath, path8.extname(flowPath)) : exported.name;
   const runFn = isLegacy ? exported : exported.run;
   const { harMode, harPath } = await initHar(deps.fs, options, flowName);
+  const { traceMode, tracePath } = await initTrace(deps.fs, options, flowName);
   const videoSize = { width: 1280, height: 720 };
   const contextSetup = buildContextSetup(videoSize, options, harPath);
   const launchBrowserOpts = {
@@ -180323,7 +180356,9 @@ async function runWebFlow({
     contextSetup,
     launchBrowserOpts,
     signals: deps.signals,
-    timeout: options.timeout
+    timeout: options.timeout,
+    traceMode,
+    tracePath
   });
   const flowDef = {
     name: flowName,
@@ -180352,8 +180387,12 @@ async function runWebFlow({
     return result;
   } finally {
     await cleanup();
+    const warn = (message) => deps.logger?.warn(message);
     if (harPath !== undefined) {
-      await maybeCleanupHar(deps.fs, harPath, passed, harMode);
+      await maybeCleanupHar(deps.fs, harPath, passed, harMode, warn);
+    }
+    if (tracePath !== undefined) {
+      await maybeCleanupTrace(deps.fs, tracePath, passed, traceMode, warn);
     }
   }
 }
@@ -180631,6 +180670,7 @@ function buildRunOptions(flags) {
       headed: flags.headed,
       slowMo: 0,
       video: flags.video,
+      trace: flags.trace,
       timeout: flags.timeout,
       har: flags.har,
       harContent: flags.harContent
@@ -181401,7 +181441,7 @@ Examples:
 function registerFlowsRunCommand(flows, signals) {
   declareCommandKind(flows.command("run [pattern]"), "local", {
     kindNote: "read with --env"
-  }).description("Run flows matching [pattern], or every flow when omitted; with --env, pull missing flows from that QA Wolf environment").option("--retries <n>", "Retry each failing flow up to this many times", parseInteger("--retries", { min: 0 }), 0).option("--bail", "Stop the run after the first failure", false).option("--workers <n>", "Parallel worker count for web flows", parseInteger("--workers", { min: 1 }), 1).option("--timeout <ms>", "Per-flow timeout in milliseconds", parseInteger("--timeout", { min: 0 }), 30000).option("--video <mode>", "Record video: on | off | retain-on-failure", parseEnum("--video", videoModes), videoDefault).option("--trace <mode>", "Record Playwright trace: on | off | retain-on-failure", parseEnum("--trace", traceModes), traceDefault).option("--har <mode>", "Record HAR network log: on | off | retain-on-failure", parseEnum("--har", harModes), harDefault).option("--har-content <mode>", "HAR response bodies: omit | full (full uses more memory)", parseEnum("--har-content", harContentModes), harContentDefault).option("--output-dir <path>", "Directory for run artifacts (videos, traces, HAR)", "qawolf-output").option("--junit [path]", "Write a JUnit XML report (default: <output-dir>/junit-report.xml)").option("--headed", "Show the browser window instead of headless", false).option("--env <env>", "Pull and run a flow from this environment (UUID or slug) if not cached locally").addHelpText("after", runExamples).action((pattern, opts, command) => {
+  }).description("Run flows matching [pattern], or every flow when omitted; with --env, pull missing flows from that QA Wolf environment").option("--retries <n>", "Retry each failing flow up to this many times", parseInteger("--retries", { min: 0 }), 0).option("--bail", "Stop the run after the first failure", false).option("--workers <n>", "Parallel worker count for web flows", parseInteger("--workers", { min: 1 }), 1).option("--timeout <ms>", "Default timeout for actions and assertions, in milliseconds", parseInteger("--timeout", { min: 0 }), 30000).option("--video <mode>", "Record video: on | off | retain-on-failure", parseEnum("--video", videoModes), videoDefault).option("--trace <mode>", "Record Playwright trace: on | off | retain-on-failure", parseEnum("--trace", traceModes), traceDefault).option("--har <mode>", "Record HAR network log: on | off | retain-on-failure", parseEnum("--har", harModes), harDefault).option("--har-content <mode>", "HAR response bodies: omit | full (full uses more memory)", parseEnum("--har-content", harContentModes), harContentDefault).option("--output-dir <path>", "Directory for run artifacts (videos, traces, HAR)", defaultOutputDir).option("--junit [path]", "Write a JUnit XML report (default: <output-dir>/junit-report.xml)").option("--headed", "Show the browser window instead of headless", false).option("--env <env>", "Pull and run a flow from this environment (UUID or slug) if not cached locally").addHelpText("after", runExamples).action((pattern, opts, command) => {
     if (opts.env !== undefined) {
       const hybridFlags = { ...opts, env: opts.env };
       return withAuthContext(signals, (ctx) => handleHybridFlowsRun(ctx, pattern, hybridFlags))(opts, command);
@@ -181499,7 +181539,7 @@ import { dirname as dirname9, join as join26, relative as relative7 } from "node
 
 // src/domains/init/templates.ts
 var qawolfConfigTs = `export default {
-  outputDir: ".qawolf",
+  outputDir: "${defaultOutputDir}",
   timeout: 60_000,
   retries: 0,
   video: "retain-on-failure",
@@ -182178,4 +182218,4 @@ process.on("SIGINT", onSignal("SIGINT"));
 process.on("SIGTERM", onSignal("SIGTERM"));
 createProgram({ signals }).parse();
 
-//# debugId=53F1EE42E10CEF1764756E2164756E21
+//# debugId=61BE7631A00FDFEA64756E2164756E21

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@qawolf/cli",
-  "version": "1.0.0",
+  "version": "1.0.1",
   "description": "Run and manage QA Wolf flows from the terminal, CI, or an AI agent",
   "keywords": [
     "automation",
@@ -22,7 +22,7 @@
     "url": "git+https://github.com/qawolf/cli.git"
   },
   "bin": {
-    "qawolf": "./dist/cli.js"
+    "qawolf": "dist/cli.js"
   },
   "files": [
     "LICENSE",
@@ -36,7 +36,7 @@
   },
   "scripts": {
     "build": "bun run generate && bun scripts/build.ts",
-    "build:binary": "bun scripts/buildBinary.ts",
+    "build:binary": "bun run generate && bun scripts/buildBinary.ts",
     "dev": "bun run src/main.ts",
     "format": "oxfmt .",
     "format:check": "oxfmt --check .",
@@ -47,7 +47,7 @@
     "lint": "oxlint",
     "lint:fix": "oxlint --fix",
     "postbuild": "./scripts/postbuild.sh",
-    "prepare": "git rev-parse --is-inside-work-tree >/dev/null 2>&1 && git config core.hooksPath .githooks || true",
+    "prepare": "git config core.hooksPath .githooks || true",
     "test": "bun run generate && bun test",
     "test:watch": "bun test --watch",
     "typecheck": "bun run generate && tsc --noEmit",