qazen-cli 0.1.7 → 0.1.9

package/dist/lib/visionNavigator.js

@@ -104,6 +104,18 @@ export class VisionNavigator {
                 onEvent({ type: "error", message: `Redirected to ${currentHostname} — skipping` });
                 return;
             }
+            // Post-goto dedup: if redirect landed on an already-mapped URL, skip.
+            const normalizedFinal = this.normalizeUrl(currentUrl);
+            if (normalizedFinal !== normalizedUrl && this.visitedUrls.has(normalizedFinal)) {
+                onEvent({
+                    type: "vision_analysis",
+                    message: `Skipping ${currentUrl} — already mapped`,
+                });
+                return;
+            }
+            this.visitedUrls.add(normalizedFinal);
+            // Mark original URL as visited too — prevents re-queuing via redirect.
+            this.visitedUrls.add(normalizedUrl);
             const screenshot = await this.takeScreenshot(url);
             const analysis = await this.analyzeScreenshot(screenshot, url, context, onEvent);
             if (!analysis)
@@ -122,8 +134,26 @@ export class VisionNavigator {
                 message: `${analysis.pageDescription} — ${analysis.elements.length} elements found`,
             });
             const highPriority = analysis.elements
-                .filter((e) => e.priority === "high" &&
-                (e.elementType === "link" || e.elementType === "tab" || e.elementType === "menu"))
+                .filter((e) => {
+                if (e.priority !== "high")
+                    return false;
+                if (!["link", "tab", "menu"].includes(e.elementType))
+                    return false;
+                // Skip if this action URL is already visited
+                if (e.action && e.action.startsWith("http")) {
+                    const normalized = this.normalizeUrl(e.action);
+                    if (this.visitedUrls.has(normalized))
+                        return false;
+                }
+                // Skip if element description indicates it's the current page
+                const desc = e.description.toLowerCase();
+                if (desc.includes("currently active") ||
+                    desc.includes("current page") ||
+                    desc.includes("(active)")) {
+                    return false;
+                }
+                return true;
+            })
                 .slice(0, 4);
             for (const element of highPriority) {
                 try {
@@ -237,13 +267,9 @@ Return ONLY the JSON object. No markdown, no explanation.`,
             return null;
         }
     }
-    async clickElement(element, pageMap, onEvent) {
-        onEvent({
-            type: "action",
-            message: `Clicking "${element.description}" (${element.visualLocation || element.elementType})`,
-        });
-        const urlBefore = this.page.url();
+    async getElementCoordinates(description, visualLocation) {
         const screenshot = await this.takeScreenshot("click-target");
+        const hint = visualLocation ? ` (${visualLocation})` : "";
         const coordResponse = await this.anthropic.messages.create({
             model: MODEL,
             max_tokens: 200,
@@ -257,7 +283,7 @@ Return ONLY the JSON object. No markdown, no explanation.`,
                         },
                         {
                             type: "text",
-                            text: `Find "${element.description}" on screen.
+                            text: `Find "${description}"${hint} on screen.
 Return ONLY JSON: {"x": number, "y": number, "found": boolean}
 x and y are pixel coordinates (image is 1280x720).
 If not found, return {"x": 0, "y": 0, "found": false}`,
@@ -267,24 +293,50 @@ If not found, return {"x": 0, "y": 0, "found": false}`,
             ],
         });
         const coordText = firstTextBlock(coordResponse.content);
-        let coords;
         try {
-            coords = JSON.parse(stripCodeFences(coordText));
+            return JSON.parse(stripCodeFences(coordText));
         }
         catch {
             return null;
         }
-        if (!coords.found)
+    }
+    async clickElement(element, pageMap, onEvent) {
+        onEvent({
+            type: "action",
+            message: `Clicking "${element.description}"`,
+        });
+        const coords = await this.getElementCoordinates(element.description, element.visualLocation);
+        if (!coords?.found)
             return null;
+        // Set up navigation listener BEFORE clicking so we catch the redirect.
+        let navigationUrl = null;
+        const navigationPromise = this.page.waitForNavigation({
+            timeout: 3000,
+            waitUntil: "domcontentloaded",
+        })
+            .then(() => {
+            navigationUrl = this.page.url();
+        })
+            .catch(() => {
+            /* no navigation — fine */
+        });
         await this.page.mouse.click(coords.x, coords.y);
         this.totalActions++;
-        pageMap.actions_taken.push(`Clicked "${element.description}" at (${coords.x}, ${coords.y})`);
-        await this.page.waitForTimeout(2000);
-        const urlAfter = this.page.url();
-        if (urlAfter !== urlBefore) {
-            const newScreenshot = await this.takeScreenshot(urlAfter);
-            this.screenshots.push(newScreenshot);
-            return urlAfter;
+        await Promise.race([navigationPromise, this.page.waitForTimeout(2000)]);
+        if (navigationUrl) {
+            const normalizedNew = this.normalizeUrl(navigationUrl);
+            // If we landed on an already-mapped page, go back and skip.
+            if (this.visitedUrls.has(normalizedNew)) {
+                onEvent({
+                    type: "vision_analysis",
+                    message: `Skipping ${normalizedNew} — already mapped`,
+                });
+                await this.page.goBack({ waitUntil: "domcontentloaded", timeout: 10000 }).catch(() => { });
+                await this.page.waitForTimeout(1000);
+                return null;
+            }
+            pageMap.actions_taken.push(`Navigated to ${navigationUrl}`);
+            return navigationUrl;
         }
         return null;
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "qazen-cli",
-  "version": "0.1.7",
+  "version": "0.1.9",
   "description": "QAZen CLI — capture authenticated browser sessions for enterprise SSO testing",
   "license": "MIT",
   "author": "QAZen",