@test-bro/cli 0.1.3 → 0.1.4

package/dist/index.js

@@ -6621,13 +6621,14 @@ var HybridTestExecutor = class {
     const strategies = generateSelectors(aiLog.element);
     if (strategies.length === 0) return void 0;
     const elementDescription = aiLog.action.elementDescription || "";
-    if (this.page) {
-      const pageUrl = this.page.url();
+    const pageUrl = this.page?.url();
+    if (pageUrl) {
       const tagName = aiLog.element.tagName?.toLowerCase();
       this.learnToLibrary(pageUrl, elementDescription, tagName, strategies).catch(() => {
       });
+      this.addToLocalCache(pageUrl, elementDescription, strategies);
     }
-    return { stepId, elementDescription, strategies };
+    return { stepId, elementDescription, strategies, pageUrl };
   }
   /**
    * Call the selector-learn API to persist learned selectors to the library.
@@ -6660,6 +6661,37 @@ var HybridTestExecutor = class {
     } catch {
     }
   }
+  /**
+   * Add a newly learned selector to the in-memory library index so subsequent
+   * steps in the same run can use it without a fresh API fetch.
+   */
+  addToLocalCache(pageUrl, elementDescription, strategies) {
+    if (!this.libraryIndex) return;
+    let urlPath;
+    try {
+      urlPath = new URL(pageUrl).pathname;
+    } catch {
+      urlPath = pageUrl;
+    }
+    urlPath = urlPath.replace(/\/$/, "") || "/";
+    let page = this.libraryIndex.pages.find((p) => p.urlPattern === urlPath);
+    if (!page) {
+      page = { urlPattern: urlPath, elements: [] };
+      this.libraryIndex.pages.push(page);
+    }
+    page.elements.push({
+      name: elementDescription,
+      aliases: [],
+      elementRole: null,
+      strategies: strategies.map((s, i) => ({
+        type: s.type,
+        value: s.value,
+        playwrightLocator: s.playwrightLocator,
+        confidence: s.confidence,
+        rank: i
+      }))
+    });
+  }
   /**
    * Fetch the full library export and cache it for the duration of plan execution.
    * Called once before executeTestPlan; failures are non-fatal.
@@ -8384,24 +8416,31 @@ async function handleHybridSelectorLearning(learnedSelectors, options) {
 async function saveToLibrary(projectId, learnedSelectors, targetUrl, isPretty, verbose) {
   const saveSpinner = isPretty ? createSpinner("Saving selectors to library...").start() : null;
   try {
-    const elements = learnedSelectors.map((ls) => ({
-      elementDescription: ls.elementDescription || "Unknown element",
-      strategies: ls.strategies.map((s) => ({
-        type: s.type || "css",
-        value: s.value || "",
-        playwrightLocator: s.playwrightLocator || s.value || "",
-        confidence: s.confidence ?? 0.8
-      }))
-    }));
-    const validElements = elements.filter((e) => e.strategies.length > 0);
-    const result = await learnSelectors(
-      projectId,
-      targetUrl || "",
-      validElements
-    );
-    const count = result.created.elements + result.updated.elements;
+    const groups = /* @__PURE__ */ new Map();
+    for (const ls of learnedSelectors) {
+      const url = ls.pageUrl || targetUrl || "";
+      if (!url) continue;
+      if (!groups.has(url)) groups.set(url, []);
+      groups.get(url).push(ls);
+    }
+    let totalCount = 0;
+    for (const [pageUrl, selectors] of groups) {
+      const elements = selectors.map((ls) => ({
+        elementDescription: ls.elementDescription || "Unknown element",
+        strategies: ls.strategies.map((s) => ({
+          type: s.type || "css",
+          value: s.value || "",
+          playwrightLocator: s.playwrightLocator || s.value || "",
+          confidence: s.confidence ?? 0.8
+        }))
+      }));
+      const validElements = elements.filter((e) => e.strategies.length > 0);
+      if (validElements.length === 0) continue;
+      const result = await learnSelectors(projectId, pageUrl, validElements);
+      totalCount += result.created.elements + result.updated.elements;
+    }
     saveSpinner?.succeed(
-      `Saved ${count} selector${count === 1 ? "" : "s"} to library`
+      `Saved ${totalCount} selector${totalCount === 1 ? "" : "s"} to library`
     );
   } catch (saveError) {
     saveSpinner?.fail("Failed to save selectors to library");
@@ -8435,6 +8474,7 @@ async function runCommand(options) {
     recordTrace = false,
     env: envName,
     credential: credentialName,
+    feature: featureFlag,
     sequential = true
   } = mergedOptions;
   const mode = file && !requestedMode ? "ai" : requestedMode || "selector";
@@ -8551,6 +8591,34 @@ async function runCommand(options) {
       }
     }
   }
+  let resolvedFeatureId;
+  if (featureFlag && projectId) {
+    try {
+      const { features } = await listFeatures(projectId);
+      const byId = features.find((f) => f.id === featureFlag);
+      if (byId) {
+        resolvedFeatureId = byId.id;
+        if (isPretty) log.info(`Using feature "${byId.name}" ${chalk16.dim(`(${byId.id.slice(0, 8)}...)`)}`);
+      } else {
+        const byName = features.find((f) => f.name.toLowerCase() === featureFlag.toLowerCase());
+        if (byName) {
+          resolvedFeatureId = byName.id;
+          if (isPretty) log.info(`Using feature "${byName.name}" ${chalk16.dim(`(${byName.id.slice(0, 8)}...)`)}`);
+        } else {
+          if (isPretty) log.info(`Creating feature "${featureFlag}"...`);
+          const { feature: newFeature } = await createFeature(projectId, { name: featureFlag });
+          resolvedFeatureId = newFeature.id;
+          if (isPretty) log.info(`Created feature "${newFeature.name}" ${chalk16.dim(`(${newFeature.id.slice(0, 8)}...)`)}`);
+        }
+      }
+      if (isPretty) log.newline();
+    } catch (error) {
+      if (isPretty) {
+        const message = error instanceof Error ? error.message : String(error);
+        log.warn(`Failed to resolve feature "${featureFlag}": ${message}`);
+      }
+    }
+  }
   const { generatedFromFile, fileGeneratedTestCases } = await handleFileOption(
     file,
     projectId,
@@ -8580,7 +8648,8 @@ async function runCommand(options) {
       projectConfig,
       isPretty,
       environment: resolvedEnvName,
-      authEnabled: !!envAuth
+      authEnabled: !!envAuth,
+      featureId: resolvedFeatureId
     });
     await runTestExecution({
       testPlan,
@@ -10758,7 +10827,7 @@ var package_default = {
   publishConfig: {
     access: "public"
   },
-  version: "0.1.3",
+  version: "0.1.4",
   description: "TestBro CLI - AI-powered browser testing from your terminal",
   type: "module",
   bin: {
@@ -11086,7 +11155,7 @@ program.command("generate").description("Generate test cases from a file or desc
     process.exit(1);
   }
 });
-program.command("run").description("Run tests against a URL").option("-u, --url <url>", "Target URL to test (or use baseUrl from test-bro.config.json)").option("--file <path>", "Path to file (AC/PRD) to generate and run tests from").option("-d, --description <text>", "Feature description to test").option("-p, --project <id>", "Project ID to use test cases from (overrides active)").option("-t, --test-case <id>", "Run specific test case only").option("-m, --mode <mode>", "Execution mode: selector (default), ai, or hybrid", "selector").option("--headed", "Run browser in headed mode (visible)", false).option("--remote", "Run tests on remote staging environment", false).option("-f, --format <format>", "Output format: pretty or json", "pretty").option("-v, --verbose", "Show verbose output", false).option("--timeout <ms>", "Timeout per step in milliseconds", "30000").option("--learn-selectors", "Learn selectors from AI actions", false).option("--yes", "Auto-confirm selector learning", false).option("--no-dedup", "Skip deduplication check when using --file").option("--auto-merge", "Auto-apply smart merge without prompting when using --file").option("--record-video", "Record video of the test execution", false).option("--record-trace", "Capture Playwright trace for debugging", false).option("-e, --env <name>", "Use a named environment (fetches baseUrl from API)").option("--credential <name>", "Use a named credential from the vault").option("--no-sequential", "Run test cases independently (re-navigate for each)").action(async (options) => {
+program.command("run").description("Run tests against a URL").option("-u, --url <url>", "Target URL to test (or use baseUrl from test-bro.config.json)").option("--file <path>", "Path to file (AC/PRD) to generate and run tests from").option("-d, --description <text>", "Feature description to test").option("-p, --project <id>", "Project ID to use test cases from (overrides active)").option("-t, --test-case <id>", "Run specific test case only").option("-m, --mode <mode>", "Execution mode: selector (default), ai, or hybrid", "selector").option("--headed", "Run browser in headed mode (visible)", false).option("--remote", "Run tests on remote staging environment", false).option("-f, --format <format>", "Output format: pretty or json", "pretty").option("-v, --verbose", "Show verbose output", false).option("--timeout <ms>", "Timeout per step in milliseconds", "30000").option("--learn-selectors", "Learn selectors from AI actions", false).option("--yes", "Auto-confirm selector learning", false).option("--no-dedup", "Skip deduplication check when using --file").option("--auto-merge", "Auto-apply smart merge without prompting when using --file").option("--record-video", "Record video of the test execution", false).option("--record-trace", "Capture Playwright trace for debugging", false).option("-e, --env <name>", "Use a named environment (fetches baseUrl from API)").option("--credential <name>", "Use a named credential from the vault").option("--feature <id|name>", "Associate this run with a feature (by ID or name)").option("--no-sequential", "Run test cases independently (re-navigate for each)").action(async (options) => {
   if (options.url && !validateUrl(options.url)) {
     displayError("Invalid URL format", `Provided URL: ${options.url}`);
     console.log(chalk29.dim('  Example: testbro run --url https://example.com --description "..."'));
@@ -11133,6 +11202,7 @@ program.command("run").description("Run tests against a URL").option("-u, --url
       recordTrace: options.recordTrace,
       env: options.env,
       credential: options.credential,
+      feature: options.feature,
       sequential: options.sequential
     });
   } catch (error) {

package/package.json

@@ -3,7 +3,7 @@
   "publishConfig": {
     "access": "public"
   },
-  "version": "0.1.3",
+  "version": "0.1.4",
   "description": "TestBro CLI - AI-powered browser testing from your terminal",
   "type": "module",
   "bin": {