feat(browser): auto-commit con 44 cambios

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

functions/browser/cdp_click.go

@@ -14,11 +14,19 @@ func CdpClick(c *CDPConn, selector string) error {
 		return fmt.Errorf("cdp click: conexion nula")
 	}
 
-	// Obtener coordenadas del centro del elemento
+	// Obtener coordenadas del centro del elemento, tras hacer scroll para que sea
+	// visible. Verificamos visibilidad: un elemento existente pero oculto
+	// (display:none, visibility:hidden, opacity 0 o tamaño 0) daria un rect en
+	// (0,0) y clicariamos en la esquina sin efecto — devolvemos error en su lugar.
 	js := fmt.Sprintf(`(function() {
 		var el = document.querySelector(%q);
 		if (!el) return null;
+		el.scrollIntoView({block:'center'});
 		var r = el.getBoundingClientRect();
+		var s = window.getComputedStyle(el);
+		var visible = r.width > 0 && r.height > 0 &&
+			s.visibility !== 'hidden' && s.display !== 'none' && s.opacity !== '0';
+		if (!visible) return '__HIDDEN__';
 		return JSON.stringify({x: r.left + r.width/2, y: r.top + r.height/2});
 	})()`, selector)
 
@@ -29,6 +37,9 @@ func CdpClick(c *CDPConn, selector string) error {
 	if coordStr == "" || coordStr == "null" {
 		return fmt.Errorf("cdp click: elemento %q no encontrado en el DOM", selector)
 	}
+	if strings.Contains(coordStr, "__HIDDEN__") {
+		return fmt.Errorf("cdp click: elemento %q existe pero no es visible/clickable (display:none, oculto, opacity 0 o tamaño 0)", selector)
+	}
 
 	// Parsear "{x:...,y:...}" — CdpEvaluate ya retorna el JSON como string
 	coordStr = strings.Trim(coordStr, `"`)
@@ -37,13 +48,6 @@ func CdpClick(c *CDPConn, selector string) error {
 		return fmt.Errorf("cdp click: parsear coordenadas %q: %w", coordStr, err)
 	}
 
-	// Hacer scroll al elemento para que este visible
-	scrollJS := fmt.Sprintf(`document.querySelector(%q).scrollIntoView({block:'center'})`, selector)
-	if _, err := CdpEvaluate(c, scrollJS); err != nil {
-		// No es fatal si el scroll falla
-		_ = err
-	}
-
 	// Despachar mousedown
 	mouseParams := map[string]any{
 		"type":       "mousePressed",