Merge PR: Compatibilidad con macOS y correccion de brechas de seguridad

Author: GuilleBouix

.gitignore

@@ -74,4 +74,4 @@ assets/installer/
 installer/
 notes/
 Compilar-Buildear-Pasos.txt
-index.html
+index.html

app/app.py

@@ -20,7 +20,7 @@
 from app.utils.paths import resource_path
 from app.ui.main_window import MainWindow
 from app.translations import t
-from app.modules.image_utils import init_heif_support
+from app.utils.image_utils import init_heif_support
 
 logger = logging.getLogger(__name__)
 
@@ -60,16 +60,24 @@ def __init__(self):
         # Actualizar para asegurar que la geometria se aplique antes de maximizar
         self.update()
 
-        # Maximizar ventana al iniciar (Windows/Linux)
-        try:
-            # Windows y algunos WM
-            self.state('zoomed')
-        except Exception:
+        # Maximizar ventana al iniciar
+        if sys.platform == 'darwin':
+            self.update_idletasks()
+            w = self.winfo_screenwidth()
+            h = self.winfo_screenheight()
+            self.geometry(f'{w}x{h}+0+0')
+        elif sys.platform == 'win32':
+            try:
+                self.state('zoomed')
+            except Exception:
+                self.update_idletasks()
+                w = self.winfo_screenwidth()
+                h = self.winfo_screenheight()
+                self.geometry(f'{w}x{h}+0+0')
+        else:
             try:
-                # Linux (algunos window managers)
                 self.attributes('-zoomed', True)
             except Exception:
-                # Fallback: usar tamaño de pantalla
                 self.update_idletasks()
                 w = self.winfo_screenwidth()
                 h = self.winfo_screenheight()
@@ -78,42 +86,60 @@ def __init__(self):
         # Establecer tamano minimo de la ventana
         self.minsize(900, 600)
 
+        # En macOS, restaurar la ventana al hacer click en el icono del Dock
+        if sys.platform == 'darwin':
+            self._setup_dock_reopen()
+        
         # Configurar icono de la ventana segun plataforma
         self._setup_icon()
 
         # Crear ventana principal con sidebar y area de contenido
         self.main_window = MainWindow(self)
         self.main_window.pack(fill='both', expand=True)
 
+    def _setup_dock_reopen(self):
+        """Registra el handler para restaurar la ventana desde el Dock en macOS."""
+        try:
+            self.createcommand('tk::mac::ReopenApplication', self._on_dock_click)
+        except Exception as e:
+            logger.debug("No se pudo registrar ReopenApplication: %s", e)
+
+    def _on_dock_click(self):
+        """Restaura la ventana cuando el usuario hace click en el icono del Dock."""
+        self.deiconify()
+        self.lift()
+        self.focus_force()
+
     def _setup_icon(self):
         """
         Configura el icono de la ventana segun la plataforma.
         
-        En Windows usa icon.ico, en otras plataformas usa icon.png.
-        Registra warnings si los iconos no se encuentran.
+        Windows: icon.ico | macOS: icon.png (icon.icns para bundle) | Linux: icon.png
         """
-        # Rutas de los archivos de icono
         icon_ico = resource_path('assets/icon.ico')
+        icon_icns = resource_path('assets/icon.icns')
         icon_png = resource_path('assets/icon.png')
 
-        # Configuracion especifica para Windows
         if sys.platform == 'win32':
-            # Usar .ico en Windows (formato nativo)
             if icon_ico.exists():
                 self.iconbitmap(str(icon_ico))
-            # Fallback a PNG si .ico no existe
             elif icon_png.exists():
                 self._setup_icon_png(icon_png)
             else:
-                # Registrar warning si no hay icono disponible
+                logger.warning("Icono no encontrado en assets/")
+        elif sys.platform == 'darwin':
+            if icon_png.exists():
+                self._setup_icon_png(icon_png)
+            elif icon_icns.exists():
+                try:
+                    self.iconbitmap(str(icon_icns))
+                except Exception as e:
+                    logger.warning(f'No se pudo aplicar icon.icns: {e}')
+            else:
                 logger.warning("Icono no encontrado en assets/")
         else:
-            # En otras plataformas solo usar PNG
             if icon_png.exists():
                 self._setup_icon_png(icon_png)
-            elif icon_ico.exists():
-                # .ico no es compatible con macOS/Linux
-                logger.warning("icon.ico no es compatible con esta plataforma")
             else:
                 logger.warning("Icono no encontrado en assets/")
 

app/modules/compress.py

@@ -15,8 +15,8 @@
 
 from PIL import Image
 
-from app.modules.image_utils import normalize_common, ensure_rgb_for_jpeg
-from app.modules.output import unique_output_path
+from app.utils.image_utils import normalize_common, ensure_rgb_for_jpeg
+from app.utils.output import unique_output_path
 
 logger = logging.getLogger(__name__)
 

app/modules/convert.py

@@ -12,8 +12,8 @@
 
 from PIL import Image
 
-from app.modules.image_utils import normalize_common, ensure_rgb_for_jpeg
-from app.modules.output import unique_output_path
+from app.utils.image_utils import normalize_common, ensure_rgb_for_jpeg
+from app.utils.output import unique_output_path
 
 logger = logging.getLogger(__name__)
 

app/modules/lqip.py

@@ -7,15 +7,39 @@
 """
 
 import base64
+import html
 import io
 import json
 import logging
+import re
+import unicodedata
 from pathlib import Path
 
 from PIL import Image, ImageFilter
 
 logger = logging.getLogger(__name__)
 
+
+def _nombre_css_seguro(nombre_archivo):
+    """Convierte el nombre original a una clase CSS segura."""
+    nombre = Path(nombre_archivo).stem.strip().lower()
+    nombre = unicodedata.normalize('NFKD', nombre).encode('ascii', 'ignore').decode('ascii')
+    nombre = re.sub(r'\s+', '-', nombre)
+    nombre = re.sub(r'[^a-z0-9_-]+', '-', nombre)
+    nombre = re.sub(r'-{2,}', '-', nombre).strip('-_')
+
+    if not nombre:
+        return 'img'
+    if nombre[0].isdigit():
+        return f'img-{nombre}'
+    return nombre
+
+
+def _comentario_seguro(nombre_archivo):
+    """Evita cierres de comentario o saltos de linea en exportes TXT."""
+    return str(nombre_archivo).replace('*/', '* /').replace('\r', ' ').replace('\n', ' ')
+
+
 def imagen_a_base64(ruta, calidad=85):
     """
     Convierte una imagen completa a string base64.
@@ -89,13 +113,14 @@ def _construir_resultado(datos_b64, nombre_archivo):
         Diccionario con data_uri, html_tag, css_bg.
     """
     data_uri = f'data:image/jpeg;base64,{datos_b64}'
-    nombre_css = Path(nombre_archivo).stem.replace(' ', '-').lower()
+    nombre_css = _nombre_css_seguro(nombre_archivo)
+    alt = html.escape(Path(nombre_archivo).stem, quote=True)
 
     return {
         'nombre': nombre_archivo,
         'base64': datos_b64,
         'data_uri': data_uri,
-        'html_tag': f'<img src="{data_uri}" alt="{Path(nombre_archivo).stem}" />',
+        'html_tag': f'<img src="{data_uri}" alt="{alt}" />',
         'css_bg': f'.{nombre_css} {{\n  background-image: url("{data_uri}");\n}}',
     }
 
@@ -151,7 +176,7 @@ def exportar_txt(resultados, ruta_salida, campo='data_uri'):
     logger.info("Exportar LQIP TXT: %s resultados -> %s", len(resultados), ruta_salida)
     lineas = []
     for res in resultados:
-        lineas.append(f'/* {res["nombre"]} */')
+        lineas.append(f'/* {_comentario_seguro(res["nombre"])} */')
         lineas.append(res.get(campo, ''))
         lineas.append('')
 

app/modules/metadata.py

@@ -14,7 +14,7 @@
 from PIL import Image, ImageOps
 import piexif
 
-from app.modules.output import unique_output_path
+from app.utils.output import unique_output_path
 
 logger = logging.getLogger(__name__)
 

app/modules/ocr.py

@@ -14,11 +14,12 @@
 import cv2
 from PIL import features
 
-from app.modules.image_utils import load_cv_image
+from app.utils.image_utils import load_cv_image
 
 logger = logging.getLogger(__name__)
 _READER_CACHE = {}
 _READER_LOCK = threading.Lock()
+MAX_IMAGENES = 10
 
 
 def _idiomas_key(idiomas):
@@ -184,7 +185,7 @@ def batch_procesar(rutas, idiomas=['es', 'en']):
     textos = {}
     ok = 0
     errores = 0
-    for ruta in rutas[:10]:  # L?mite de 10 im?genes
+    for ruta in rutas[:MAX_IMAGENES]:
         try:
             texto = procesar_imagen(ruta, reader)
             textos[str(ruta)] = texto

app/modules/remove_bg.py

@@ -14,14 +14,16 @@
 
 from PIL import Image, ImageOps
 
-from app.modules.output import unique_output_path
+from app.utils.output import unique_output_path
 
 logger = logging.getLogger(__name__)
 
 MODELO = 'u2netp'
 FORMATOS_SALIDA = ['PNG', 'WEBP', 'TIFF']
 _FMT_A_EXT = {'PNG': '.png', 'WEBP': '.webp', 'TIFF': '.tiff'}
+INSTALL_COMMAND = 'pip install --upgrade rembg onnxruntime pymatting'
 _MODEL_READY = False
+MAX_IMAGENES = 10
 
 
 def _ensure_stdio():
@@ -53,16 +55,17 @@ def quitar_fondo(ruta_entrada, ruta_salida, formato_salida='PNG'):
         try:
             from rembg import remove as rembg_remove, new_session
         except ImportError:
-            raise ImportError('rembg no est? instalado. Ejecut?: pip install rembg')
+            raise ImportError(f'rembg no está disponible. Ejecutá: {INSTALL_COMMAND}')
 
         logger.info(
             "remove_bg: quitar_fondo inicio (entrada=%s, salida=%s, formato=%s)",
             ruta_entrada, ruta_salida, formato_salida
         )
         _ensure_stdio()
+        tam_original = Path(ruta_entrada).stat().st_size
+
         imagen = Image.open(ruta_entrada)
         imagen = ImageOps.exif_transpose(imagen)
-        tam_original = Path(ruta_entrada).stat().st_size
 
         logger.info("remove_bg: crear_sesion (modelo=%s)", MODELO)
         sesion = new_session(MODELO)
@@ -111,7 +114,7 @@ def batch_quitar_fondo(rutas, carpeta_salida, formato_salida='PNG'):
     try:
         from rembg import remove as rembg_remove, new_session
     except ImportError:
-        raise ImportError('rembg no está instalado.')
+        raise ImportError(f'rembg no está disponible. Ejecutá: {INSTALL_COMMAND}')
 
     logger.info(
         "remove_bg: batch_inicio (cantidad=%s, carpeta=%s, formato=%s)",
@@ -126,12 +129,13 @@ def batch_quitar_fondo(rutas, carpeta_salida, formato_salida='PNG'):
     errores = 0
     conflictos = 0
 
-    for ruta in rutas:
+    for ruta in list(rutas)[:MAX_IMAGENES]:
         try:
             logger.debug("remove_bg: procesar_imagen (ruta=%s)", ruta)
+            tam_original = Path(ruta).stat().st_size
+
             imagen = Image.open(ruta)
             imagen = ImageOps.exif_transpose(imagen)
-            tam_original = Path(ruta).stat().st_size
 
             resultado_raw = rembg_remove(imagen, session=sesion)
             if isinstance(resultado_raw, Image.Image):
@@ -193,15 +197,21 @@ def ensure_model():
     logger.info("remove_bg: ensure_model listo")
 
 
-def rembg_disponible():
-    """Verifica si rembg esta instalado."""
+def estado_rembg():
+    """Verifica si rembg puede importarse y retorna detalle del error si falla."""
     try:
         import rembg  # noqa: F401
-        logger.debug("remove_bg: rembg_disponible=True")
-        return True
+        return True, None
     except Exception as exc:
-        logger.warning("rembg no disponible: %s", exc)
-        return False
+        detalle = str(exc).strip() or type(exc).__name__
+        logger.warning("rembg no disponible: %s", detalle)
+        return False, detalle
+
+
+def rembg_disponible():
+    """Verifica si rembg esta disponible en el entorno actual."""
+    disponible, _ = estado_rembg()
+    return disponible
 
 
 def modelo_descargado():

app/modules/rename.py

@@ -7,11 +7,34 @@
 """
 
 import logging
+import re
 from datetime import datetime
 from pathlib import Path
 
 logger = logging.getLogger(__name__)
 
+_INVALID_FILENAME_CHARS = re.compile(r'[<>:"/\\|?*\x00-\x1f]+')
+
+
+def _sanitizar_prefijo(prefijo):
+    """Limpia el prefijo para evitar rutas inseguras en el renombrado."""
+    if not isinstance(prefijo, str):
+        return ''
+
+    prefijo = _INVALID_FILENAME_CHARS.sub('_', prefijo.strip())
+    prefijo = re.sub(r'_+', '_', prefijo).strip(' ._')
+    return prefijo
+
+
+def _nombre_es_seguro(nombre):
+    """Verifica que el nombre no contenga separadores ni componentes de ruta."""
+    if not nombre or '\x00' in nombre:
+        return False
+    if '/' in nombre or '\\' in nombre:
+        return False
+    return Path(nombre).name == nombre
+
+
 def generar_nombres_preview(rutas, opciones):
     """
     Genera preview de nuevos nombres sin modificar archivos.
@@ -49,7 +72,7 @@ def _aplicar_transformaciones(nombre, indice, opciones):
 
     # Paso 1 - Prefijo + numeracion
     if opciones.get('numeracion_activa'):
-        prefijo = opciones.get('prefijo', '').strip()
+        prefijo = _sanitizar_prefijo(opciones.get('prefijo', ''))
         numero = str(indice + inicio).zfill(3)
         if prefijo:
             nombre_resultado = f'{prefijo}_{numero}'
@@ -108,6 +131,11 @@ def renombrar_archivos(rutas, opciones):
                 ok += 1
                 continue
 
+            if not _nombre_es_seguro(nombre_nuevo):
+                logger.warning("Nombre de destino inseguro rechazado: %s", nombre_nuevo)
+                errores += 1
+                continue
+
             ruta_nueva = ruta_archivo.parent / (nombre_nuevo + extension)
 
             if ruta_nueva.exists() and ruta_nueva != ruta_archivo:

app/modules/resize.py

@@ -12,8 +12,8 @@
 
 from PIL import Image
 
-from app.modules.image_utils import normalize_common
-from app.modules.output import unique_output_path
+from app.utils.image_utils import normalize_common
+from app.utils.output import unique_output_path
 
 from math import gcd