Update app.py

app.py

@@ -1,3 +1,11 @@
+import sys
+
+# Silenciador del error fantasma de Linux (No congela, solo oculta el texto rojo)
+def silenciador_errores_basura(unraisable):
+    if unraisable.exc_type == ValueError and "Invalid file descriptor: -1" in str(unraisable.exc_value): pass
+    else: sys.__unraisablehook__(unraisable)
+sys.unraisablehook = silenciador_errores_basura
+
 import os
 import subprocess
 import pandas as pd
@@ -11,7 +19,7 @@ import random
 import requests
 from PIL import Image
 import io
-from concurrent.futures import ThreadPoolExecutor # <-- LA LIBRERÍA SALVAVIDAS
+import traceback
 
 # --- INSTALACIÓN DE DEPENDENCIAS ---
 try:
@@ -24,9 +32,9 @@ except ImportError:
     subprocess.run(["pip", "install", "fake-useragent"], check=True)
     from fake_useragent import UserAgent
 
-# --- FUNCIÓN PARA DESCARGAR Y PREPARAR FOTOS ---
+# --- DESCARGA SEGURA DE FOTOS ---
 def descargar_imagen(url, idx):
-    if not url or url.startswith("data:"): 
+    if not url or len(url) < 10 or url.startswith("data:"): 
         return None
     try:
         r = requests.get(url, timeout=5, headers={"User-Agent": "Mozilla/5.0"})
@@ -38,7 +46,7 @@ def descargar_imagen(url, idx):
             img.save(path, format="JPEG")
             return path
     except:
-        pass
+        return None # Si falla, devuelve None silenciosamente
     return None
 
 # --- 1. GENERADOR DE URLS ---
@@ -79,188 +87,188 @@ def extraer_ubicacion(texto):
     lineas = [l.strip() for l in texto.split('\n') if len(l.strip()) > 5]
     if len(lineas) > 1:
         for linea in lineas[1:4]:
-            if "$" not in linea and not linea.isdigit():
+            if "$" not in linea and not linea.isdigit() and "m²" not in linea:
                 return linea[:60]
     return "Ubicación en zona solicitada"
 
-# --- 3. MOTOR DE EXTRACCIÓN SÍNCRONO (NÚCLEO) ---
+# --- 3. MOTOR DE EXTRACCIÓN (SÍNCRONO Y ESTABLE) ---
 def motor_tramitia_visual(operacion, barrio, ciudad, area, m2_min, m2_max, tipo, hab, ban, park, antiguedad, ascensor, piscina):
     resultados = []
-    url_fr, url_mc = construir_urls_final(operacion, barrio, ciudad, tipo, hab, ban, park, antiguedad, m2_min, m2_max, ascensor, piscina)
+    log_visible = ""
     
-    log_visible = f"✅ INICIANDO EXTRACCIÓN AISLADA:\nFR: {url_fr}\nMC: {url_mc}\n\n"
-    ua = UserAgent()
+    try:
+        url_fr, url_mc = construir_urls_final(operacion, barrio, ciudad, tipo, hab, ban, park, antiguedad, m2_min, m2_max, ascensor, piscina)
+        log_visible = f"✅ INICIANDO EXTRACCIÓN:\nFR: {url_fr}\nMC: {url_mc}\n\n"
+        ua = UserAgent()
 
-    with sync_playwright() as p:
-        browser = p.chromium.launch(
-            headless=True,
-            args=['--disable-blink-features=AutomationControlled', '--no-sandbox', '--disable-dev-shm-usage', f'--user-agent={ua.random}']
-        )
-        context = browser.new_context(viewport={'width': 1366, 'height': 768})
+        with sync_playwright() as p:
+            browser = p.chromium.launch(headless=True, args=['--disable-blink-features=AutomationControlled', '--no-sandbox'])
+            context = browser.new_context(viewport={'width': 1366, 'height': 768}, user_agent=ua.random)
 
-        # --- FINCA RAÍZ ---
-        try:
-            page = context.new_page()
-            log_visible += "🔄 FR: Cargando imágenes y datos...\n"
-            page.goto(url_fr, wait_until="domcontentloaded", timeout=60000)
-            
-            for _ in range(4):
-                page.mouse.wheel(0, 800)
-                time.sleep(1.5)
+            # --- FINCA RAÍZ ---
+            try:
+                page = context.new_page()
+                log_visible += "🔄 FR: Buscando inmuebles...\n"
+                page.goto(url_fr, wait_until="domcontentloaded", timeout=60000)
+                
+                for _ in range(3):
+                    page.mouse.wheel(0, 1000)
+                    time.sleep(1.5)
 
-            cards = page.query_selector_all("article, div[class*='card']")
-            for card in cards[:15]:
-                txt = card.inner_text()
-                precio = extraer_precio(txt, operacion)
-                if precio > 0:
-                    enlace_el = card.query_selector("a")
-                    img_el = card.query_selector("img")
-                    
-                    href = enlace_el.get_attribute("href") if enlace_el else ""
-                    img_url = ""
-                    if img_el:
-                        img_url = img_el.get_attribute("src") or img_el.get_attribute("data-src") or ""
+                cards = page.query_selector_all("article, div[class*='card']")
+                for card in cards[:12]:
+                    try:
+                        txt = card.inner_text()
+                        precio = extraer_precio(txt, operacion)
+                        if precio > 0:
+                            enlace_el = card.query_selector("a")
+                            img_el = card.query_selector("img")
+                            
+                            href = enlace_el.get_attribute("href") if enlace_el else ""
+                            img_url = img_el.get_attribute("src") if img_el else ""
 
-                    if href:
-                        resultados.append({
-                            "Portal": "Finca Raiz",
-                            "Precio": precio,
-                            "Precio_M2": precio / area,
-                            "Ubicacion": extraer_ubicacion(txt),
-                            "Descripcion": txt.replace('\n', ' | ')[:120] + "...",
-                            "URL": f"https://www.fincaraiz.com.co{href}" if href.startswith("/") else href,
-                            "Imagen": img_url
-                        })
-            page.close()
-        except Exception as e: log_visible += f"⚠️ Error FR: {e}\n"
+                            if href:
+                                resultados.append({
+                                    "Portal": "Finca Raiz",
+                                    "Precio": precio,
+                                    "Precio_M2": precio / area,
+                                    "Ubicacion": extraer_ubicacion(txt),
+                                    "Descripcion": txt.replace('\n', ' | ')[:120] + "...",
+                                    "URL": f"https://www.fincaraiz.com.co{href}" if href.startswith("/") else href,
+                                    "Imagen": img_url
+                                })
+                    except: continue # Si una tarjeta falla, salta a la siguiente
+                page.close()
+            except Exception as e: log_visible += f"⚠️ Error buscando en FR.\n"
 
-        # --- METROCUADRADO ---
-        try:
-            page = context.new_page()
-            log_visible += "🔄 MC: Cargando imágenes y datos...\n"
-            page.goto(url_mc, wait_until="domcontentloaded", timeout=60000)
-            
-            for _ in range(5):
-                page.mouse.wheel(0, 800)
-                time.sleep(1.5)
+            # --- METROCUADRADO ---
+            try:
+                page = context.new_page()
+                log_visible += "🔄 MC: Buscando inmuebles...\n"
+                page.goto(url_mc, wait_until="domcontentloaded", timeout=60000)
+                
+                for _ in range(4):
+                    page.mouse.wheel(0, 1000)
+                    time.sleep(1.5)
+                
+                cards = page.query_selector_all("li, div.property-card")
+                for card in cards[:12]:
+                    try:
+                        txt = card.inner_text()
+                        precio = extraer_precio(txt, operacion)
+                        if precio > 0:
+                            enlace_el = card.query_selector("a")
+                            img_el = card.query_selector("img")
+                            
+                            href = enlace_el.get_attribute("href") if enlace_el else ""
+                            img_url = img_el.get_attribute("src") if img_el else ""
+
+                            if href:
+                                resultados.append({
+                                    "Portal": "Metrocuadrado",
+                                    "Precio": precio,
+                                    "Precio_M2": precio / area,
+                                    "Ubicacion": extraer_ubicacion(txt),
+                                    "Descripcion": txt.replace('\n', ' | ')[:120] + "...",
+                                    "URL": f"https://www.metrocuadrado.com{href}" if href.startswith("/") else href,
+                                    "Imagen": img_url
+                                })
+                    except: continue
+                page.close()
+            except Exception as e: log_visible += f"⚠️ Error buscando en MC.\n"
             
-            cards = page.query_selector_all("li, div.property-card")
-            for card in cards[:15]:
-                txt = card.inner_text()
-                precio = extraer_precio(txt, operacion)
-                if precio > 0:
-                    enlace_el = card.query_selector("a")
-                    img_el = card.query_selector("img")
-                    
-                    href = enlace_el.get_attribute("href") if enlace_el else ""
-                    img_url = ""
-                    if img_el:
-                        img_url = img_el.get_attribute("src") or img_el.get_attribute("data-src") or ""
+            browser.close()
 
-                    if href:
-                        resultados.append({
-                            "Portal": "Metrocuadrado",
-                            "Precio": precio,
-                            "Precio_M2": precio / area,
-                            "Ubicacion": extraer_ubicacion(txt),
-                            "Descripcion": txt.replace('\n', ' | ')[:120] + "...",
-                            "URL": f"https://www.metrocuadrado.com{href}" if href.startswith("/") else href,
-                            "Imagen": img_url
-                        })
-            page.close()
-        except Exception as e: log_visible += f"⚠️ Error MC: {e}\n"
-        
-        browser.close()
+        if not resultados:
+            return f"{log_visible}\n❌ NO SE ENCONTRARON DATOS.", pd.DataFrame(), None, "---"
 
-    if not resultados:
-        return f"{log_visible}\n❌ NO SE ENCONTRARON DATOS.", None, None, "---"
+        # --- LIMPIEZA ---
+        df_crudo = pd.DataFrame(resultados).drop_duplicates(subset=['URL'])
+        df_fr = df_crudo[df_crudo['Portal'] == 'Finca Raiz'].head(6)
+        df_mc = df_crudo[df_crudo['Portal'] == 'Metrocuadrado'].head(6)
+        df_final = pd.concat([df_fr, df_mc]).reset_index(drop=True)
 
-    df_crudo = pd.DataFrame(resultados).drop_duplicates(subset=['URL'])
-    df_fr = df_crudo[df_crudo['Portal'] == 'Finca Raiz'].head(6)
-    df_mc = df_crudo[df_crudo['Portal'] == 'Metrocuadrado'].head(6)
-    df_final = pd.concat([df_fr, df_mc]).reset_index(drop=True)
+        log_visible += f"\n✨ PROCESANDO PDF CON {len(df_final)} INMUEBLES..."
 
-    # --- GENERACIÓN DEL PDF VISUAL ---
-    pdf_path = f"Reporte_Visual_{int(time.time())}.pdf"
-    pdf = FPDF()
-    pdf.add_page()
-    
-    pdf.set_font("Arial", 'B', 16)
-    pdf.set_fill_color(40, 53, 147)
-    pdf.set_text_color(255, 255, 255)
-    pdf.cell(0, 15, f" REPORTE INMOBILIARIO: {barrio.upper()} ({operacion.upper()})", ln=True, fill=True)
-    pdf.set_text_color(0, 0, 0)
-    pdf.ln(5)
+        # --- PDF ---
+        pdf_path = f"Reporte_Visual_{int(time.time())}.pdf"
+        pdf = FPDF()
+        pdf.add_page()
+        
+        pdf.set_font("Arial", 'B', 16)
+        pdf.set_fill_color(40, 53, 147)
+        pdf.set_text_color(255, 255, 255)
+        pdf.cell(0, 15, f" REPORTE INMOBILIARIO: {barrio.upper()} ({operacion.upper()})", ln=True, fill=True)
+        pdf.set_text_color(0, 0, 0)
+        pdf.ln(5)
 
-    for idx, r in df_final.iterrows():
-        if pdf.get_y() > 220: 
-            pdf.add_page()
+        for idx, r in df_final.iterrows():
+            if pdf.get_y() > 220: pdf.add_page()
+                
+            y_start = pdf.get_y()
+            img_path = descargar_imagen(r['Imagen'], idx)
+            text_x = 10
             
-        y_start = pdf.get_y()
-        img_path = descargar_imagen(r['Imagen'], idx)
-        text_x = 10
-        
-        if img_path and os.path.exists(img_path):
-            try:
-                pdf.image(img_path, x=10, y=y_start, w=45, h=30)
-                text_x = 60
-            except: pass
+            if img_path and os.path.exists(img_path):
+                try:
+                    pdf.image(img_path, x=10, y=y_start, w=45, h=30)
+                    text_x = 60
+                except: pass
 
-        pdf.set_xy(text_x, y_start)
-        pdf.set_font("Arial", 'B', 12)
-        pdf.cell(0, 6, f"${r['Precio']:,.0f} COP", ln=True)
-        
-        pdf.set_x(text_x)
-        pdf.set_font("Arial", 'B', 9)
-        pdf.set_text_color(100, 100, 100)
-        pdf.cell(0, 5, f"📍 {r['Ubicacion']} | Fuente: {r['Portal']}", ln=True)
-        
-        pdf.set_x(text_x)
-        pdf.set_font("Arial", '', 8)
-        pdf.set_text_color(0, 0, 0)
-        pdf.multi_cell(0, 4, f"{r['Descripcion']}")
-        
-        pdf.set_x(text_x)
-        pdf.set_font("Arial", 'U', 8)
-        pdf.set_text_color(0, 102, 204)
-        pdf.cell(0, 5, "🔗 Hacer clic aquí para ver publicación original", link=r['URL'], ln=True)
-        pdf.set_text_color(0, 0, 0)
+            pdf.set_xy(text_x, y_start)
+            pdf.set_font("Arial", 'B', 12)
+            pdf.cell(0, 6, f"${r['Precio']:,.0f} COP", ln=True)
+            
+            pdf.set_x(text_x)
+            pdf.set_font("Arial", 'B', 9)
+            pdf.set_text_color(100, 100, 100)
+            pdf.cell(0, 5, f"📍 {r['Ubicacion']} | Fuente: {r['Portal']}", ln=True)
+            
+            pdf.set_x(text_x)
+            pdf.set_font("Arial", '', 8)
+            pdf.set_text_color(0, 0, 0)
+            pdf.multi_cell(0, 4, f"{r['Descripcion']}")
+            
+            pdf.set_x(text_x)
+            pdf.set_font("Arial", 'U', 8)
+            pdf.set_text_color(0, 102, 204)
+            pdf.cell(0, 5, "🔗 Hacer clic aquí para ver publicación", link=r['URL'], ln=True)
+            pdf.set_text_color(0, 0, 0)
+            
+            y_end = pdf.get_y()
+            pdf.set_y(max(y_start + 35, y_end + 5))
+            
+            pdf.set_draw_color(200, 200, 200)
+            pdf.line(10, pdf.get_y(), 200, pdf.get_y())
+            pdf.ln(5)
+            
+            if img_path and os.path.exists(img_path):
+                try: os.remove(img_path)
+                except: pass
+
+        pdf.output(pdf_path)
         
-        y_end = pdf.get_y()
-        pdf.set_y(max(y_start + 35, y_end + 5))
+        # --- CÁLCULOS ---
+        promedio = df_final['Precio_M2'].mean() * area
+        resumen = (
+            f"💰 **ESTIMACIÓN DE {operacion.upper()}**\n"
+            f"🔹 **Valor Sugerido:** ${promedio:,.0f}\n"
+            f"📉 Mínimo Zona: ${df_final['Precio'].min():,.0f}\n"
+            f"📈 Máximo Zona: ${df_final['Precio'].max():,.0f}"
+        )
         
-        pdf.set_draw_color(200, 200, 200)
-        pdf.line(10, pdf.get_y(), 200, pdf.get_y())
-        pdf.ln(5)
+        df_mostrar = df_final[['Portal', 'Precio', 'Ubicacion', 'Descripcion', 'URL']]
         
-        if img_path and os.path.exists(img_path):
-            os.remove(img_path)
-
-    pdf.output(pdf_path)
-    
-    # --- CÁLCULOS ---
-    promedio = df_final['Precio_M2'].mean() * area
-    resumen = (
-        f"💰 **ESTIMACIÓN DE {operacion.upper()}**\n"
-        f"🔹 **Valor Sugerido:** ${promedio:,.0f}\n"
-        f"📉 Mínimo Zona: ${df_final['Precio'].min():,.0f}\n"
-        f"📈 Máximo Zona: ${df_final['Precio'].max():,.0f}"
-    )
-    
-    df_mostrar = df_final[['Portal', 'Precio', 'Ubicacion', 'Descripcion', 'URL']]
-    
-    return f"{log_visible}\n✅ PDF Visual Generado con Éxito.", df_mostrar, pdf_path, resumen
+        return f"{log_visible}\n✅ PDF Generado Exitosamente.", df_mostrar, pdf_path, resumen
 
-# --- 4. FUNCIÓN ENVOLTORIO (LA BURBUJA SALVAVIDAS) ---
-def ejecutor_aislado(*args):
-    # Ejecuta el motor en un hilo completamente separado para que no choque con Gradio
-    with ThreadPoolExecutor(max_workers=1) as executor:
-        futuro = executor.submit(motor_tramitia_visual, *args)
-        return futuro.result()
+    except Exception as error_fatal:
+        # PARACAÍDAS: Si todo colapsa, te mostrará exactamente DÓNDE y POR QUÉ en la pantalla
+        traza = traceback.format_exc()
+        return f"❌ ERROR GRAVE DEL SISTEMA:\n{str(error_fatal)}\n\nDetalles:\n{traza}", pd.DataFrame(), None, "⚠️ Falló la ejecución"
 
 # --- INTERFAZ GRÁFICA ---
 with gr.Blocks(theme=gr.themes.Soft()) as demo:
-    gr.Markdown("## 📸 TramitIA Pro: Generador Visual Anti-Errores")
+    gr.Markdown("## 📸 TramitIA Pro: Generador Visual (Versión Estable)")
     
     with gr.Row():
         with gr.Column(scale=1):
@@ -270,7 +278,7 @@ with gr.Blocks(theme=gr.themes.Soft()) as demo:
             
             with gr.Row():
                 t = gr.Dropdown(["Apartamento", "Casa", "Bodega", "Lote", "Oficina"], label="Tipo", value="Apartamento")
-                a = gr.Number(label="Área M2 (Tu Cliente)", value=70)
+                a = gr.Number(label="Área M2", value=70)
                 
             with gr.Row():
                 m2_min = gr.Number(label="M2 Mínimo", value=10)
@@ -286,16 +294,15 @@ with gr.Blocks(theme=gr.themes.Soft()) as demo:
                 p = gr.Number(label="Park", value=1)
                 
             e = gr.Dropdown(["Menos de 1 año", "1 a 8 años", "9 a 15 años", "16 a 30 años", "Más de 30 años"], label="Antigüedad", value="1 a 8 años")
-            btn = gr.Button("GENERAR REPORTE CON FOTOS", variant="primary")
+            btn = gr.Button("GENERAR REPORTE", variant="primary")
             
         with gr.Column(scale=2):
-            res_fin = gr.Markdown("### 💰 Resultado...")
+            res_fin = gr.Markdown("### 💰 El resultado aparecerá aquí...")
             with gr.Tabs():
-                with gr.TabItem("Descargar Reporte PDF"): out_pdf = gr.File()
+                with gr.TabItem("Descargar PDF"): out_pdf = gr.File()
                 with gr.TabItem("Tabla de Datos"): out_df = gr.Dataframe()
                 with gr.TabItem("Auditoría"): msg = gr.Textbox(lines=10)
 
-    # AQUÍ ESTÁ EL TRUCO: Llamamos al ejecutor_aislado, no directamente al motor
-    btn.click(ejecutor_aislado, [op, b, c, a, m2_min, m2_max, t, h, ban, p, e, ascensor, piscina], [msg, out_df, out_pdf, res_fin])
+    btn.click(motor_tramitia_visual, [op, b, c, a, m2_min, m2_max, t, h, ban, p, e, ascensor, piscina], [msg, out_df, out_pdf, res_fin])
 
 demo.launch()