@weavetab/mcp 2.5.0 → 2.5.1

package/dist/tools/pointer.js

@@ -46,12 +46,12 @@ async function resolveRefToCoords(session, targetId, ref, currentUrl) {
         return null;
     // Scroll element into view before resolving coordinates
     await session.Runtime.evaluate({
-        expression: `(function(sel) {
-      const el = document.querySelector(sel);
-      if (el) {
-        if (el.scrollIntoViewIfNeeded) el.scrollIntoViewIfNeeded();
-        else el.scrollIntoView({behavior:'instant',block:'center'});
-      }
+        expression: `(function(sel) {
+      const el = document.querySelector(sel);
+      if (el) {
+        if (el.scrollIntoViewIfNeeded) el.scrollIntoViewIfNeeded();
+        else el.scrollIntoView({behavior:'instant',block:'center'});
+      }
     })(${JSON.stringify(selector)})`,
         returnByValue: true
     }).catch(() => { });
@@ -68,25 +68,25 @@ async function resolveRefToCoords(session, targetId, ref, currentUrl) {
     }
     // Fallback 1: use element bounding rect directly via CDP evaluate
     const bboxRes = await session.Runtime.evaluate({
-        expression: `(function(sel) {
-      const el = document.querySelector(sel);
-      if (!el) return null;
-      // Primary: getBoundingClientRect
-      const r = el.getBoundingClientRect();
-      if (r.width > 0 || r.height > 0) {
-        return { x: Math.round(r.left + r.width / 2), y: Math.round(r.top + r.height / 2) };
-      }
-      // Fallback for inline text elements (e.g. Wikipedia lang links).
-      // Inline elements can have zero-size bounding rects but non-zero client rects
-      // for their first text-box fragment.
-      const rects = el.getClientRects();
-      if (rects && rects.length > 0) {
-        const fr = rects[0];
-        if (fr.width > 0 || fr.height > 0) {
-          return { x: Math.round(fr.left + fr.width / 2), y: Math.round(fr.top + fr.height / 2) };
-        }
-      }
-      return null;
+        expression: `(function(sel) {
+      const el = document.querySelector(sel);
+      if (!el) return null;
+      // Primary: getBoundingClientRect
+      const r = el.getBoundingClientRect();
+      if (r.width > 0 || r.height > 0) {
+        return { x: Math.round(r.left + r.width / 2), y: Math.round(r.top + r.height / 2) };
+      }
+      // Fallback for inline text elements (e.g. Wikipedia lang links).
+      // Inline elements can have zero-size bounding rects but non-zero client rects
+      // for their first text-box fragment.
+      const rects = el.getClientRects();
+      if (rects && rects.length > 0) {
+        const fr = rects[0];
+        if (fr.width > 0 || fr.height > 0) {
+          return { x: Math.round(fr.left + fr.width / 2), y: Math.round(fr.top + fr.height / 2) };
+        }
+      }
+      return null;
     })(${JSON.stringify(selector)})`,
         returnByValue: true
     }).catch(() => null);
@@ -167,12 +167,12 @@ export async function browserPointer(session, args, config) {
             // Dispatch mouseenter/mouseover events for dynamic flyout/hover menus
             if (stepCoords.selector) {
                 await session.Runtime.evaluate({
-                    expression: `(function(sel, cx, cy) {
-            const el = document.querySelector(sel) || (window.__Weavetab_shadowQuery ? window.__Weavetab_shadowQuery(sel) : null);
-            if (!el) return;
-            el.dispatchEvent(new MouseEvent('mouseenter', { bubbles: false, cancelable: true, clientX: cx, clientY: cy }));
-            el.dispatchEvent(new MouseEvent('mouseover',  { bubbles: true,  cancelable: true, clientX: cx, clientY: cy }));
-            el.dispatchEvent(new MouseEvent('mousemove',  { bubbles: true,  cancelable: true, clientX: cx, clientY: cy }));
+                    expression: `(function(sel, cx, cy) {
+            const el = document.querySelector(sel) || (window.__Weavetab_shadowQuery ? window.__Weavetab_shadowQuery(sel) : null);
+            if (!el) return;
+            el.dispatchEvent(new MouseEvent('mouseenter', { bubbles: false, cancelable: true, clientX: cx, clientY: cy }));
+            el.dispatchEvent(new MouseEvent('mouseover',  { bubbles: true,  cancelable: true, clientX: cx, clientY: cy }));
+            el.dispatchEvent(new MouseEvent('mousemove',  { bubbles: true,  cancelable: true, clientX: cx, clientY: cy }));
           })(${JSON.stringify(stepCoords.selector)}, ${stepCoords.x}, ${stepCoords.y})`,
                     returnByValue: false
                 }).catch(() => { });
@@ -286,11 +286,11 @@ export async function browserPointer(session, args, config) {
         // Dispatch both mouseenter AND mouseover (some sites only listen to one)
         if (startSelector) {
             await session.Runtime.evaluate({
-                expression: `(function(sel) {
-          const el = document.querySelector(sel) || (window.__Weavetab_shadowQuery ? window.__Weavetab_shadowQuery(sel) : null);
-          if (!el) return;
-          el.dispatchEvent(new MouseEvent('mouseenter', { bubbles: false, cancelable: true }));
-          el.dispatchEvent(new MouseEvent('mouseover',  { bubbles: true,  cancelable: true }));
+                expression: `(function(sel) {
+          const el = document.querySelector(sel) || (window.__Weavetab_shadowQuery ? window.__Weavetab_shadowQuery(sel) : null);
+          if (!el) return;
+          el.dispatchEvent(new MouseEvent('mouseenter', { bubbles: false, cancelable: true }));
+          el.dispatchEvent(new MouseEvent('mouseover',  { bubbles: true,  cancelable: true }));
         })(${JSON.stringify(startSelector)})`,
                 returnByValue: true
             }).catch(() => { });
@@ -326,11 +326,11 @@ export async function browserPointer(session, args, config) {
         }
         // ── Pre-drag: snapshot the element bounding box for verification ──────
         const preDragBBox = await session.Runtime.evaluate({
-            expression: `(function(sx, sy, sel) {
-        var el = sel ? document.querySelector(sel) : document.elementFromPoint(sx, sy);
-        if (!el) return null;
-        var r = el.getBoundingClientRect();
-        return { x: Math.round(r.left), y: Math.round(r.top), w: Math.round(r.width), h: Math.round(r.height), tag: el.tagName };
+            expression: `(function(sx, sy, sel) {
+        var el = sel ? document.querySelector(sel) : document.elementFromPoint(sx, sy);
+        if (!el) return null;
+        var r = el.getBoundingClientRect();
+        return { x: Math.round(r.left), y: Math.round(r.top), w: Math.round(r.width), h: Math.round(r.height), tag: el.tagName };
       })(${startX}, ${startY}, ${startSelector ? JSON.stringify(startSelector) : 'null'})`,
             returnByValue: true,
         }).catch(() => null);
@@ -391,37 +391,37 @@ export async function browserPointer(session, args, config) {
         // Skipped for canvas targets — they don't use HTML5 DnD.
         if (!cdpDragWorked && !isCanvasTarget) {
             await session.Runtime.evaluate({
-                expression: `(function(sx, sy, dx, dy, srcSel) {
-          var src = srcSel ? document.querySelector(srcSel) : document.elementFromPoint(sx, sy);
-          var dest = document.elementFromPoint(dx, dy);
-          if (!src) return;
-
-          var dt;
-          try { dt = new DataTransfer(); } catch(e) {
-            dt = { items: [], types: [], files: [],
-              setData: function(k,v){ this.types.push(k); this._data = this._data||{}; this._data[k]=v; },
-              getData: function(k){ return (this._data||{})[k]||''; },
-              clearData: function(){}, dropEffect:'move', effectAllowed:'all' };
-          }
-          dt.effectAllowed = 'all';
-
-          function fire(el, type, related) {
-            if (!el) return true;
-            var evt = new DragEvent(type, {
-              bubbles: true, cancelable: true, view: window,
-              clientX: type.startsWith('drag') && (type==='dragstart'||type==='drag'||type==='dragend') ? sx : dx,
-              clientY: type.startsWith('drag') && (type==='dragstart'||type==='drag'||type==='dragend') ? sy : dy,
-              dataTransfer: dt,
-              relatedTarget: related || null
-            });
-            return el.dispatchEvent(evt);
-          }
-
-          fire(src, 'dragstart', null);
-          fire(dest, 'dragenter', src);
-          for (var i = 0; i < 3; i++) { fire(dest, 'dragover', src); }
-          fire(dest, 'drop', src);
-          fire(src, 'dragend', dest);
+                expression: `(function(sx, sy, dx, dy, srcSel) {
+          var src = srcSel ? document.querySelector(srcSel) : document.elementFromPoint(sx, sy);
+          var dest = document.elementFromPoint(dx, dy);
+          if (!src) return;
+
+          var dt;
+          try { dt = new DataTransfer(); } catch(e) {
+            dt = { items: [], types: [], files: [],
+              setData: function(k,v){ this.types.push(k); this._data = this._data||{}; this._data[k]=v; },
+              getData: function(k){ return (this._data||{})[k]||''; },
+              clearData: function(){}, dropEffect:'move', effectAllowed:'all' };
+          }
+          dt.effectAllowed = 'all';
+
+          function fire(el, type, related) {
+            if (!el) return true;
+            var evt = new DragEvent(type, {
+              bubbles: true, cancelable: true, view: window,
+              clientX: type.startsWith('drag') && (type==='dragstart'||type==='drag'||type==='dragend') ? sx : dx,
+              clientY: type.startsWith('drag') && (type==='dragstart'||type==='drag'||type==='dragend') ? sy : dy,
+              dataTransfer: dt,
+              relatedTarget: related || null
+            });
+            return el.dispatchEvent(evt);
+          }
+
+          fire(src, 'dragstart', null);
+          fire(dest, 'dragenter', src);
+          for (var i = 0; i < 3; i++) { fire(dest, 'dragover', src); }
+          fire(dest, 'drop', src);
+          fire(src, 'dragend', dest);
         })(${startX}, ${startY}, ${destX}, ${destY}, ${startSelector ? JSON.stringify(startSelector) : 'null'})`,
                 returnByValue: false,
             }).catch(() => { });
@@ -458,16 +458,16 @@ export async function browserPointer(session, args, config) {
             // for belt-and-suspenders coverage.
             if (isCanvasTarget) {
                 await session.Runtime.evaluate({
-                    expression: `(function(sx, sy, mx, my) {
-            var el = document.elementFromPoint(sx, sy) || document.elementFromPoint(mx, my);
-            if (!el) return;
-            el.dispatchEvent(new PointerEvent('pointermove', {
-              bubbles: true, cancelable: true, view: window,
-              pointerId: 1, pointerType: 'mouse', isPrimary: true,
-              width: 1, height: 1, pressure: 0.5,
-              button: -1, buttons: 1,
-              clientX: ${mx}, clientY: ${my}
-            }));
+                    expression: `(function(sx, sy, mx, my) {
+            var el = document.elementFromPoint(sx, sy) || document.elementFromPoint(mx, my);
+            if (!el) return;
+            el.dispatchEvent(new PointerEvent('pointermove', {
+              bubbles: true, cancelable: true, view: window,
+              pointerId: 1, pointerType: 'mouse', isPrimary: true,
+              width: 1, height: 1, pressure: 0.5,
+              button: -1, buttons: 1,
+              clientX: ${mx}, clientY: ${my}
+            }));
           })(${startX}, ${startY}, ${mx}, ${my})`,
                     returnByValue: false,
                 }).catch(() => { });
@@ -481,30 +481,30 @@ export async function browserPointer(session, args, config) {
         // Augment with synthetic pointerup for canvas apps
         if (isCanvasTarget) {
             await session.Runtime.evaluate({
-                expression: `(function(sx, sy, dx, dy) {
-          var el = document.elementFromPoint(sx, sy) || document.elementFromPoint(dx, dy);
-          if (!el) return;
-          el.dispatchEvent(new PointerEvent('pointerup', {
-            bubbles: true, cancelable: true, view: window,
-            pointerId: 1, pointerType: 'mouse', isPrimary: true,
-            width: 1, height: 1, pressure: 0, button: 0, buttons: 0,
-            clientX: ${destX}, clientY: ${destY}
-          }));
+                expression: `(function(sx, sy, dx, dy) {
+          var el = document.elementFromPoint(sx, sy) || document.elementFromPoint(dx, dy);
+          if (!el) return;
+          el.dispatchEvent(new PointerEvent('pointerup', {
+            bubbles: true, cancelable: true, view: window,
+            pointerId: 1, pointerType: 'mouse', isPrimary: true,
+            width: 1, height: 1, pressure: 0, button: 0, buttons: 0,
+            clientX: ${destX}, clientY: ${destY}
+          }));
         })(${startX}, ${startY}, ${destX}, ${destY})`,
                 returnByValue: false,
             }).catch(() => { });
         }
         // Clean up stuck drag CSS classes
         await session.Runtime.evaluate({
-            expression: `(function() {
-        const classes = ['dragging', 'drag-active', 'ui-draggable-dragging', 'is-dragging', 'being-dragged'];
-        const attrs = ['data-dragging'];
-        classes.forEach(cls => {
-          document.querySelectorAll('.' + cls).forEach(el => el.classList.remove(cls));
-        });
-        attrs.forEach(attr => {
-          document.querySelectorAll('[' + attr + ']').forEach(el => el.removeAttribute(attr));
-        });
+            expression: `(function() {
+        const classes = ['dragging', 'drag-active', 'ui-draggable-dragging', 'is-dragging', 'being-dragged'];
+        const attrs = ['data-dragging'];
+        classes.forEach(cls => {
+          document.querySelectorAll('.' + cls).forEach(el => el.classList.remove(cls));
+        });
+        attrs.forEach(attr => {
+          document.querySelectorAll('[' + attr + ']').forEach(el => el.removeAttribute(attr));
+        });
       })()`,
             returnByValue: false,
         }).catch(() => { });
@@ -529,23 +529,23 @@ export async function browserPointer(session, args, config) {
         await new Promise(r => setTimeout(r, 200)); // let animations settle
         // Check 1: bounding-box delta on the originally-clicked element
         const postDragBBox = await session.Runtime.evaluate({
-            expression: `(function(dx, dy, sel) {
-        var el = sel ? document.querySelector(sel) : document.elementFromPoint(dx, dy);
-        if (!el) return null;
-        var r = el.getBoundingClientRect();
-        return { x: Math.round(r.left), y: Math.round(r.top), w: Math.round(r.width), h: Math.round(r.height) };
+            expression: `(function(dx, dy, sel) {
+        var el = sel ? document.querySelector(sel) : document.elementFromPoint(dx, dy);
+        if (!el) return null;
+        var r = el.getBoundingClientRect();
+        return { x: Math.round(r.left), y: Math.round(r.top), w: Math.round(r.width), h: Math.round(r.height) };
       })(${end_x}, ${end_y}, ${startSelector ? JSON.stringify(startSelector) : 'null'})`,
             returnByValue: true,
         }).catch(() => null);
         const postBox = postDragBBox?.result?.value;
         // Check 2: probe for drop-zone indicators (text change, class change)
         const dropProbe = await session.Runtime.evaluate({
-            expression: `(function(dx, dy) {
-        var el = document.elementFromPoint(dx, dy);
-        if (!el) return { found: false };
-        var text = (el.textContent || '').trim().substring(0, 100);
-        var classes = el.className || '';
-        return { found: true, text: text, classes: typeof classes === 'string' ? classes : '' };
+            expression: `(function(dx, dy) {
+        var el = document.elementFromPoint(dx, dy);
+        if (!el) return { found: false };
+        var text = (el.textContent || '').trim().substring(0, 100);
+        var classes = el.className || '';
+        return { found: true, text: text, classes: typeof classes === 'string' ? classes : '' };
       })(${end_x}, ${end_y})`,
             returnByValue: true,
         }).catch(() => null);
@@ -553,11 +553,11 @@ export async function browserPointer(session, args, config) {
         if (postBox && startSelector) {
             // DOM element with a selector — check if it moved
             const curBBoxRes = await session.Runtime.evaluate({
-                expression: `(function(sel) {
-          var el = document.querySelector(sel);
-          if (!el) return null;
-          var r = el.getBoundingClientRect();
-          return { x: Math.round(r.left), y: Math.round(r.top) };
+                expression: `(function(sel) {
+          var el = document.querySelector(sel);
+          if (!el) return null;
+          var r = el.getBoundingClientRect();
+          return { x: Math.round(r.left), y: Math.round(r.top) };
         })(${JSON.stringify(startSelector)})`,
                 returnByValue: true,
             }).catch(() => null);

package/dist/tools/read.js

@@ -205,15 +205,15 @@ async function buildVisualGrounding(session, targetId, elements) {
         }
         // Also discover canvas elements that may not be in the DOM map
         const { result: canvasRes } = await session.Runtime.evaluate({
-            expression: `(function(){
-        var canvases = [...document.querySelectorAll('canvas')];
-        return JSON.stringify(canvases.map(function(c){
-          var r = c.getBoundingClientRect();
-          return r.width > 0 && r.height > 0 ? {
-            x: Math.round(r.left + r.width/2), y: Math.round(r.top + r.height/2),
-            w: Math.round(r.width), h: Math.round(r.height)
-          } : null;
-        }).filter(Boolean));
+            expression: `(function(){
+        var canvases = [...document.querySelectorAll('canvas')];
+        return JSON.stringify(canvases.map(function(c){
+          var r = c.getBoundingClientRect();
+          return r.width > 0 && r.height > 0 ? {
+            x: Math.round(r.left + r.width/2), y: Math.round(r.top + r.height/2),
+            w: Math.round(r.width), h: Math.round(r.height)
+          } : null;
+        }).filter(Boolean));
       })()`,
             returnByValue: true,
         }).catch(() => ({ result: { value: '[]' } }));
@@ -228,19 +228,19 @@ async function buildVisualGrounding(session, targetId, elements) {
         if (domMarks.length === 0)
             return null;
         // Inject badges — fixed-position, shadow-safe, above everything
-        const injectExpr = `(function(marks){
-      var BADGE_KEY = '__wt_som_badges__';
-      if (window[BADGE_KEY]) window[BADGE_KEY].forEach(function(d){ d.parentNode && d.parentNode.removeChild(d); });
-      window[BADGE_KEY] = [];
-      marks.forEach(function(m){
-        var d = document.createElement('div');
-        d.setAttribute('data-wt-som', String(m.mark));
-        d.style.cssText = 'position:fixed;left:'+(m.x - 10)+'px;top:'+(m.y - 10)+'px;width:20px;height:20px;z-index:2147483647;pointer-events:none;background:#7c3aed;color:#fff;font:bold 11px/20px monospace;text-align:center;border-radius:50%;box-shadow:0 1px 4px rgba(0,0,0,.5);';
-        d.textContent = String(m.mark);
-        document.documentElement.appendChild(d);
-        window[BADGE_KEY].push(d);
-      });
-      return true;
+        const injectExpr = `(function(marks){
+      var BADGE_KEY = '__wt_som_badges__';
+      if (window[BADGE_KEY]) window[BADGE_KEY].forEach(function(d){ d.parentNode && d.parentNode.removeChild(d); });
+      window[BADGE_KEY] = [];
+      marks.forEach(function(m){
+        var d = document.createElement('div');
+        d.setAttribute('data-wt-som', String(m.mark));
+        d.style.cssText = 'position:fixed;left:'+(m.x - 10)+'px;top:'+(m.y - 10)+'px;width:20px;height:20px;z-index:2147483647;pointer-events:none;background:#7c3aed;color:#fff;font:bold 11px/20px monospace;text-align:center;border-radius:50%;box-shadow:0 1px 4px rgba(0,0,0,.5);';
+        d.textContent = String(m.mark);
+        document.documentElement.appendChild(d);
+        window[BADGE_KEY].push(d);
+      });
+      return true;
     })(${JSON.stringify(domMarks)})`;
         await session.Runtime.evaluate({ expression: injectExpr, returnByValue: true }).catch(() => { });
         // Hide Weavetab overlay so badges are the only overlay in the shot
@@ -272,14 +272,14 @@ export async function browserMap(session, options = {}, targetId = "default", co
     // ── Wait for DOM and Visual Stability ──────────────────────────────────────
     try {
         await session.Runtime.evaluate({
-            expression: `
-        new Promise(resolve => {
-          if (document.readyState === 'complete') {
-            setTimeout(resolve, 300); // 300ms visual stability delay
-          } else {
-            window.addEventListener('load', () => setTimeout(resolve, 300));
-          }
-        })
+            expression: `
+        new Promise(resolve => {
+          if (document.readyState === 'complete') {
+            setTimeout(resolve, 300); // 300ms visual stability delay
+          } else {
+            window.addEventListener('load', () => setTimeout(resolve, 300));
+          }
+        })
       `,
             awaitPromise: true
         });
@@ -324,18 +324,21 @@ export async function browserMap(session, options = {}, targetId = "default", co
         catch (e) {
             logDebug(`[browserMap] Failed to compute DOM hash: ${e}`);
         }
-        // Try strictly validated cache first
-        const cachedMap = getCachedActionMap(targetId, currentUrl);
-        if (cachedMap) {
-            logDebug(`[browserMap] Cache Hit (strict)! Returning cached action map with ${cachedMap.elements.length} elements.`);
-            map = cachedMap;
-        }
-        else {
-            // Fallback: check raw cache with domHash
-            const rawMap = getRawCachedActionMap(targetId, currentUrl);
-            if (rawMap && rawMap.domHash === domHash) {
-                logDebug(`[browserMap] Cache Hit (domHash=${domHash})! MutationObserver invalidated but DOM content length unchanged.`);
-                map = rawMap;
+        const hasFilterOptions = Boolean(options.scope || options.query || options.limit);
+        if (!hasFilterOptions) {
+            // Try strictly validated cache first
+            const cachedMap = getCachedActionMap(targetId, currentUrl);
+            if (cachedMap) {
+                logDebug(`[browserMap] Cache Hit (strict)! Returning cached action map with ${cachedMap.elements.length} elements.`);
+                map = cachedMap;
+            }
+            else {
+                // Fallback: check raw cache with domHash
+                const rawMap = getRawCachedActionMap(targetId, currentUrl);
+                if (rawMap && rawMap.domHash === domHash) {
+                    logDebug(`[browserMap] Cache Hit (domHash=${domHash})! MutationObserver invalidated but DOM content length unchanged.`);
+                    map = rawMap;
+                }
             }
         }
     }
@@ -403,11 +406,11 @@ export async function browserMap(session, options = {}, targetId = "default", co
                 map.domHash = result.value ?? 0;
             }
             catch (e) { }
-            // Never write an AX-tree result into the primary DOM map cache.
+            // Never write an AX-tree result or a filtered map into the primary DOM map cache.
             // AX tree uses sequential w:1…w:N IDs; DOM walker uses stable content-addressable hashes.
-            // Caching the AX result would make the next plain browser_map() call return AX-tree refs,
-            // causing different tools to see different ref numbering for the same element.
-            if (!usedAXTree) {
+            // Caching a filtered or AX result would make subsequent plain browser_map() calls return incomplete refs.
+            const hasFilterOptions = Boolean(options.scope || options.query || options.limit);
+            if (!usedAXTree && !hasFilterOptions) {
                 cacheActionMap(targetId, map);
             }
             logAction("browser_map", map.url, `${map.elements.length} elements`);
@@ -461,21 +464,22 @@ export async function browserMap(session, options = {}, targetId = "default", co
     const activeTabId = sessionRegistry.getActive(getConfigAgentId(config))?.targetId ?? targetId;
     // ── Visual fallback when AX tree is empty ──────────────────────────────────
     let visualFallback = null;
-    if (map.elements.length === 0) {
+    const hasFilter = Boolean(options.query || options.scope);
+    if (map.elements.length === 0 && !hasFilter) {
         try {
             // Canvas detection — check before full screenshot analysis
             const { result: canvasCheck } = await session.Runtime.evaluate({
-                expression: `(function() {
-          const canvases = [...document.querySelectorAll('canvas')];
-          return JSON.stringify(canvases.map(c => {
-            const rect = c.getBoundingClientRect();
-            return {
-              id: c.id || '', width: Math.round(rect.width), height: Math.round(rect.height),
-              x: Math.round(rect.left + rect.width / 2),
-              y: Math.round(rect.top + rect.height / 2),
-              hasWebGL: !!(c.getContext('webgl') || c.getContext('webgl2'))
-            };
-          }));
+                expression: `(function() {
+          const canvases = [...document.querySelectorAll('canvas')];
+          return JSON.stringify(canvases.map(c => {
+            const rect = c.getBoundingClientRect();
+            return {
+              id: c.id || '', width: Math.round(rect.width), height: Math.round(rect.height),
+              x: Math.round(rect.left + rect.width / 2),
+              y: Math.round(rect.top + rect.height / 2),
+              hasWebGL: !!(c.getContext('webgl') || c.getContext('webgl2'))
+            };
+          }));
         })()`,
                 returnByValue: true,
             }).catch(() => ({ result: { value: '[]' } }));
@@ -539,22 +543,41 @@ export async function browserMap(session, options = {}, targetId = "default", co
                     lines.push("stable — no DOM changes since last map (use existing refs)");
                 }
                 else {
-                    for (const r of richDiff.added)
+                    const maxLines = 40;
+                    let printed = 0;
+                    for (const r of richDiff.added) {
+                        if (printed >= maxLines)
+                            break;
                         lines.push(`+ [${r.id}] ${r.role} "${r.label}"`);
+                        printed++;
+                    }
                     for (const r of richDiff.changed) {
+                        if (printed >= maxLines)
+                            break;
                         const changes = Object.keys(r.changes);
                         const changeHints = changes.map(c => `${c}: "${r.changes[c].from}" → "${r.changes[c].to}"`).join(', ');
                         const actionable = r.actionable ? ` ⚡${r.recommended_action ? ` ${r.recommended_action}` : ''}` : '';
                         lines.push(`~ [${r.ref}] ${r.role} "${r.label}"${changeHints ? ` (${changeHints})` : ''}${actionable}`);
+                        printed++;
+                    }
+                    for (const id of richDiff.removed) {
+                        if (printed >= maxLines)
+                            break;
+                        lines.push(`- [${id}] (removed)`);
+                        printed++;
                     }
-                    for (const id of richDiff.removed)
-                        lines.push(`- ${id}`);
                     if (richDiff.iframes) {
                         for (const f of richDiff.iframes) {
+                            if (printed >= maxLines)
+                                break;
                             const icon = f.status === "added" ? "+" : f.status === "removed" ? "-" : "~";
                             lines.push(`${icon} [iframe] ${f.frameUrl} (${f.status}, ${f.elementCount} elements)`);
+                            printed++;
                         }
                     }
+                    if (totalChanges > printed) {
+                        lines.push(`... (+${totalChanges - printed} more changes omitted)`);
+                    }
                 }
                 if (visualFallback)
                     lines.push(visualFallback);
@@ -601,8 +624,17 @@ export async function browserMap(session, options = {}, targetId = "default", co
     const lines = [];
     if (hint)
         lines.push(`> ${hint}`);
-    // Cure optional-login blindness by explicitly pointing out dismiss buttons
-    const dismissButton = map.elements.find(el => /(not now|skip|close|dismiss|no thanks|^x$)/i.test(el.label) &&
+    // Cure optional-login blindness by explicitly pointing out dismiss buttons on true modals
+    const isTitlebarControl = (el) => {
+        const sel = (el.selector || "").toLowerCase();
+        const lbl = (el.label || "").toLowerCase();
+        const isTopCorner = (el.y !== undefined && el.y < 45 && el.x !== undefined && el.x > 800);
+        return isTopCorner || sel.includes("win-close") || sel.includes("titlebar") || sel.includes("window-controls") || lbl === "close window";
+    };
+    const hasModalContext = map.elements.some(el => el.isModal || el.role === "dialog" || el.role === "alertdialog");
+    const dismissButton = map.elements.find(el => (hasModalContext || el.isModal) &&
+        !isTitlebarControl(el) &&
+        /(not now|skip|close|dismiss|no thanks|^x$)/i.test(el.label) &&
         /(button|link)/i.test(el.role));
     if (dismissButton) {
         lines.push(`\n🚨 [POPUP DETECTED]: If an optional modal/login is blocking the page, try clicking [${dismissButton.id}] ("${dismissButton.label}") to dismiss it.\n`);
@@ -620,10 +652,10 @@ export async function browserMap(session, options = {}, targetId = "default", co
     let activeElementId;
     try {
         const { result } = await session.Runtime.evaluate({
-            expression: `(function() {
-        const ae = document.activeElement;
-        if (!ae || ae === document.body) return null;
-        return ae.getAttribute('data-wt-id') || null;
+            expression: `(function() {
+        const ae = document.activeElement;
+        if (!ae || ae === document.body) return null;
+        return ae.getAttribute('data-wt-id') || null;
       })()`,
             returnByValue: true
         });

package/dist/tools/recording.js

@@ -102,6 +102,15 @@ export async function browserRecording(session, options, config) {
             keepFrames: options.keepFrames ?? false,
         });
         logAction("browser_recording", sessionId, `stopped (${count} frames, compiled: ${compileRes.videoPath || "none"})`);
+        if (!compileRes.success) {
+            return {
+                success: false,
+                recordings: Array.from(activeRecordings.values()),
+                frameCount: count,
+                dir: state.dir,
+                error: compileRes.error,
+            };
+        }
         return {
             success: true,
             recordings: Array.from(activeRecordings.values()),