iatoolkit 0.59.0__tar.gz → 0.64.0__tar.gz

{iatoolkit-0.59.0 → iatoolkit-0.64.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: iatoolkit
-Version: 0.59.0
+Version: 0.64.0
 Summary: IAToolkit
 Author: Fernando Libedinsky
 License-Expression: MIT

{iatoolkit-0.59.0 → iatoolkit-0.64.0}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "iatoolkit"
-version = "0.59.0"
+version = "0.64.0"
 requires-python = ">=3.12"
 description = "IAToolkit"
 readme = "readme.md"

{iatoolkit-0.59.0 → iatoolkit-0.64.0}/src/iatoolkit/base_company.py

@@ -29,11 +29,13 @@ class BaseCompany(ABC):
     def _create_company(self,
                         short_name: str,
                         name: str,
+                        parameters: dict | None = None,
                         branding: dict | None = None,
-                        onboarding_cards: dict | None = None
+                        onboarding_cards: dict | None = None,
                         ) -> Company:
         company_obj = Company(short_name=short_name,
                               name=name,
+                              parameters=parameters,
                               branding=branding,
                               onboarding_cards=onboarding_cards)
         self.company = self.profile_repo.create_company(company_obj)

{iatoolkit-0.59.0 → iatoolkit-0.64.0}/src/iatoolkit/common/routes.py

@@ -3,17 +3,8 @@
 #
 # IAToolkit is open source software.
 
-from flask import render_template, redirect, flash, url_for,send_from_directory, current_app, abort
-from iatoolkit.common.session_manager import SessionManager
+from flask import render_template, redirect, url_for,send_from_directory, current_app, abort
 from flask import jsonify
-from iatoolkit.views.history_api_view import HistoryApiView
-import os
-
-
-def logout(company_short_name: str):
-    SessionManager.clear()
-    flash("Has cerrado sesión correctamente", "info")
-    return redirect(url_for('index', company_short_name=company_short_name))
 
 
 # this function register all the views
@@ -22,8 +13,8 @@ def register_views(injector, app):
     from iatoolkit.views.index_view import IndexView
     from iatoolkit.views.init_context_api_view import InitContextApiView
     from iatoolkit.views.llmquery_api_view import LLMQueryApiView
-    from iatoolkit.views.tasks_view import TaskView
-    from iatoolkit.views.tasks_review_view import TaskReviewView
+    from iatoolkit.views.tasks_api_view import TaskApiView
+    from iatoolkit.views.tasks_review_api_view import TaskReviewApiView
     from iatoolkit.views.login_simulation_view import LoginSimulationView
     from iatoolkit.views.signup_view import SignupView
     from iatoolkit.views.verify_user_view import VerifyAccountView
@@ -32,12 +23,19 @@ def register_views(injector, app):
     from iatoolkit.views.file_store_api_view import FileStoreApiView
     from iatoolkit.views.user_feedback_api_view import UserFeedbackApiView
     from iatoolkit.views.prompt_api_view import PromptApiView
-    from iatoolkit.views.chat_token_request_view import ChatTokenRequestView
+    from iatoolkit.views.history_api_view import HistoryApiView
+    from iatoolkit.views.help_content_api_view import HelpContentApiView
+
     from iatoolkit.views.login_view import LoginView, FinalizeContextView
     from iatoolkit.views.external_login_view import ExternalLoginView, RedeemTokenApiView
+    from iatoolkit.views.logout_api_view import LogoutApiView
+    from iatoolkit.views.home_view import HomeView
 
     # iatoolkit home page
-    app.add_url_rule('/<company_short_name>', view_func=IndexView.as_view('index'))
+    app.add_url_rule('/', view_func=IndexView.as_view('index'))
+
+    # company home view
+    app.add_url_rule('/<company_short_name>/home', view_func=HomeView.as_view('home'))
 
     # login for the iatoolkit integrated frontend
     app.add_url_rule('/<company_short_name>/login', view_func=LoginView.as_view('login'))
@@ -57,23 +55,21 @@ def register_views(injector, app):
         view_func=FinalizeContextView.as_view('finalize_with_token')
     )
 
+    # logout
+    app.add_url_rule('/<company_short_name>/api/logout',
+                     view_func=LogoutApiView.as_view('logout'))
+
     # this endpoint is called by the JS for changing the token for a session
     app.add_url_rule('/<string:company_short_name>/api/redeem_token',
                      view_func = RedeemTokenApiView.as_view('redeem_token'))
 
-    # this endpoint is for requesting a chat token for external users
-    app.add_url_rule('/auth/chat_token',
-                     view_func=ChatTokenRequestView.as_view('chat-token'))
-
-    # init (reset) the company context (with api-key)
+    # init (reset) the company context
     app.add_url_rule('/<company_short_name>/api/init-context',
                      view_func=InitContextApiView.as_view('init-context'),
                      methods=['POST', 'OPTIONS'])
 
     # register new user, account verification and forgot password
     app.add_url_rule('/<company_short_name>/signup',view_func=SignupView.as_view('signup'))
-    app.add_url_rule('/<company_short_name>/logout', 'logout', logout)
-    app.add_url_rule('/logout', 'logout', logout)
     app.add_url_rule('/<company_short_name>/verify/<token>', view_func=VerifyAccountView.as_view('verify_account'))
     app.add_url_rule('/<company_short_name>/forgot-password', view_func=ForgotPasswordView.as_view('forgot_password'))
     app.add_url_rule('/<company_short_name>/change-password/<token>', view_func=ChangePasswordView.as_view('change_password'))
@@ -85,13 +81,14 @@ def register_views(injector, app):
     # open the promt directory
     app.add_url_rule('/<company_short_name>/api/prompts', view_func=PromptApiView.as_view('prompt'))
 
-    # feedback and history
+    # toolbar buttons
     app.add_url_rule('/<company_short_name>/api/feedback', view_func=UserFeedbackApiView.as_view('feedback'))
     app.add_url_rule('/<company_short_name>/api/history', view_func=HistoryApiView.as_view('history'))
+    app.add_url_rule('/<company_short_name>/api/help-content', view_func=HelpContentApiView.as_view('help-content'))
 
     # tasks management endpoints: create task, and review answer
-    app.add_url_rule('/tasks', view_func=TaskView.as_view('tasks'))
-    app.add_url_rule('/tasks/review/<int:task_id>', view_func=TaskReviewView.as_view('tasks-review'))
+    app.add_url_rule('/tasks', view_func=TaskApiView.as_view('tasks'))
+    app.add_url_rule('/tasks/review/<int:task_id>', view_func=TaskReviewApiView.as_view('tasks-review'))
 
     # this endpoint is for upload documents into the vector store (api-key)
     app.add_url_rule('/api/load', view_func=FileStoreApiView.as_view('load_api'))
@@ -133,6 +130,6 @@ def register_views(injector, app):
     # hacer que la raíz '/' vaya al home de iatoolkit
     @app.route('/')
     def root_redirect():
-        return redirect(url_for('index', company_short_name='sample_company'))
+        return redirect(url_for('index'))
 
 

{iatoolkit-0.59.0 → iatoolkit-0.64.0}/src/iatoolkit/common/util.py

@@ -21,10 +21,8 @@ class Utility:
     def __init__(self):
         self.encryption_key = os.getenv('FERNET_KEY')
 
-
     def render_prompt_from_template(self,
                                     template_pathname: str,
-                                    query: str = None,
                                     client_data: dict = {},
                                     **kwargs) -> str:
 
@@ -37,8 +35,6 @@ class Utility:
             env = Environment(loader=FileSystemLoader(template_dir))
             template = env.get_template(template_file)
 
-            kwargs["query"] = query
-
             # add all the keys in client_data to kwargs
             kwargs.update(client_data)
 
@@ -53,7 +49,6 @@ class Utility:
     def render_prompt_from_string(self,
                                   template_string: str,
                                   searchpath: str | list[str] = None,
-                                  query: str = None,
                                   client_data: dict = {},
                                   **kwargs) -> str:
         """
@@ -76,7 +71,6 @@ class Utility:
             env = Environment(loader=loader)
             template = env.from_string(template_string)
 
-            kwargs["query"] = query
             kwargs.update(client_data)
 
             prompt = template.render(**kwargs)

{iatoolkit-0.59.0 → iatoolkit-0.64.0}/src/iatoolkit/iatoolkit.py

@@ -16,10 +16,10 @@ import os
 from typing import Optional, Dict, Any
 from iatoolkit.repositories.database_manager import DatabaseManager
 from werkzeug.middleware.proxy_fix import ProxyFix
-from injector import Binder, singleton, Injector
+from injector import Binder, Injector, singleton
 from importlib.metadata import version as _pkg_version, PackageNotFoundError
 
-IATOOLKIT_VERSION = "0.59.0"
+IATOOLKIT_VERSION = "0.64.0"
 
 # global variable for the unique instance of IAToolkit
 _iatoolkit_instance: Optional['IAToolkit'] = None
@@ -52,7 +52,7 @@ class IAToolkit:
         self.app = None
         self.db_manager = None
         self._injector = None
-        self.version = IATOOLKIT_VERSION
+        self.version = IATOOLKIT_VERSION    # default version
 
     @classmethod
     def get_instance(cls) -> 'IAToolkit':
@@ -88,7 +88,7 @@ class IAToolkit:
         # and other integrations, as views are handled manually.
         FlaskInjector(app=self.app, injector=self._injector)
 
-        # Step 6: initialize dispatcher and registered compaies
+        # Step 6: initialize dispatcher and registered companies
         self._init_dispatcher_and_company_instances()
 
         # Step 7: Finalize setup within the application context
@@ -101,8 +101,6 @@ class IAToolkit:
         # Step 8: define the download_dir for excel's
         self._setup_download_dir()
 
-
-
         logging.info(f"🎉 IAToolkit v{self.version} inicializado correctamente")
         self._initialized = True
         return self.app
@@ -117,7 +115,7 @@ class IAToolkit:
         log_level = getattr(logging, log_level_name, logging.INFO)
 
         logging.basicConfig(
-            level=logging.INFO,  # log_level,
+            level=log_level,
             format="%(asctime)s - IATOOLKIT - %(name)s - %(levelname)s - %(message)s",
             handlers=[logging.StreamHandler()],
             force=True
@@ -226,20 +224,19 @@ class IAToolkit:
 
     def _setup_cors(self):
         """🌐 Configura CORS"""
-        # Origins por defecto para desarrollo
+        from iatoolkit.company_registry import get_company_registry
+
+        # default CORS origin
         default_origins = [
-            "http://localhost:5001",
-            "http://127.0.0.1:5001",
-            "https://portal-interno.maxxa.cl",
             os.getenv('IATOOLKIT_BASE_URL')
         ]
 
-        # Obtener origins adicionales desde configuración/env
+        # Iterate through the registered company names
         extra_origins = []
-        for i in range(1, 11):  # Soporte para CORS_ORIGIN_1 a CORS_ORIGIN_10
-            origin = self._get_config_value(f'CORS_ORIGIN_{i}')
-            if origin:
-                extra_origins.append(origin)
+        all_company_instances = get_company_registry().get_all_company_instances()
+        for company_name, company_instance in all_company_instances.items():
+            cors_origin = company_instance.company.parameters.get('cors_origin', [])
+            extra_origins += cors_origin
 
         all_origins = default_origins + extra_origins
 
@@ -254,7 +251,6 @@ class IAToolkit:
 
         logging.info(f"✅ CORS configurado para: {all_origins}")
 
-
     def _configure_core_dependencies(self, binder: Binder):
         """⚙️ Configures all system dependencies."""
         try:
@@ -325,8 +321,8 @@ class IAToolkit:
         from iatoolkit.services.auth_service import AuthService
         from iatoolkit.common.util import Utility
 
-        binder.bind(LLMProxy, to=LLMProxy, scope=singleton)
-        binder.bind(llmClient, to=llmClient, scope=singleton)
+        binder.bind(LLMProxy, to=LLMProxy)
+        binder.bind(llmClient, to=llmClient)
         binder.bind(GoogleChatApp, to=GoogleChatApp)
         binder.bind(MailApp, to=MailApp)
         binder.bind(AuthService, to=AuthService)
@@ -385,6 +381,7 @@ class IAToolkit:
                 'iatoolkit_base_url': os.environ.get('IATOOLKIT_BASE_URL', ''),
             }
 
+
     def _get_default_static_folder(self) -> str:
         try:
             current_dir = os.path.dirname(os.path.abspath(__file__))  # .../src/iatoolkit

{iatoolkit-0.59.0 → iatoolkit-0.64.0}/src/iatoolkit/repositories/models.py

@@ -60,7 +60,7 @@ class Company(Base):
 
     branding = Column(JSON, nullable=True)
     onboarding_cards = Column(JSON, nullable=True)
-    parameters = Column(JSON, nullable=True, default={})
+    parameters = Column(JSON, nullable=True)
     created_at = Column(DateTime, default=datetime.now)
     allow_jwt = Column(Boolean, default=True, nullable=True)
 

{iatoolkit-0.59.0 → iatoolkit-0.64.0}/src/iatoolkit/services/auth_service.py

@@ -91,9 +91,10 @@ class AuthService:
             )
             return {'success': False, 'error': 'No se pudo crear la sesión del usuario'}
 
-    def verify(self) -> dict:
+    def verify(self, anonymous: bool = False) -> dict:
         """
         Verifies the current request and identifies the user.
+        If anonymous is True the non-presence of use_identifier is ignored
 
         Returns a dictionary with:
         - success: bool
@@ -109,7 +110,7 @@ class AuthService:
             return {
                 "success": True,
                 "company_short_name": session_info['company_short_name'],
-                "user_identifier": session_info['user_identifier']
+                "user_identifier": session_info['user_identifier'],
             }
 
         # --- Priority 2: Check for a valid API Key in headers ---
@@ -118,31 +119,37 @@ class AuthService:
         if isinstance(auth, str) and auth.lower().startswith('bearer '):
             api_key =  auth.split(' ', 1)[1].strip()
 
-        if api_key:
-            api_key_entry = self.profile_service.get_active_api_key_entry(api_key)
-            if not api_key_entry:
-                logging.info(f"Invalid or inactive API Key {api_key}")
-                return {"success": False, "error_message": "Invalid or inactive API Key", "status_code": 401}
+        if not api_key:
+            # --- Failure: No valid credentials found ---
+            logging.info(f"Authentication required. No session cookie or API Key provided.")
+            return {"success": False,
+                    "error_message": "Authentication required. No session cookie or API Key provided.",
+                    "status_code": 401}
+
+        # check if the api-key is valid and active
+        api_key_entry = self.profile_service.get_active_api_key_entry(api_key)
+        if not api_key_entry:
+            logging.info(f"Invalid or inactive API Key {api_key}")
+            return {"success": False, "error_message": "Invalid or inactive API Key",
+                    "status_code": 402}
+
+        # get the company from the api_key_entry
+        company = api_key_entry.company
+
+        # For API calls, the external_user_id must be provided in the request.
+        data = request.get_json(silent=True) or {}
+        user_identifier = data.get('user_identifier', '')
+        if not anonymous and not user_identifier:
+            logging.info(f"No user_identifier provided for API call.")
+            return {"success": False, "error_message": "No user_identifier provided for API call.",
+                    "status_code": 403}
+
+        return {
+            "success": True,
+            "company_short_name": company.short_name,
+            "user_identifier": user_identifier
+        }
 
-            # obtain the company from the api_key_entry
-            company = api_key_entry.company
-
-            # For API calls, the external_user_id must be provided in the request.
-            user_identifier = ''
-            if request.is_json:
-                data = request.get_json() or {}
-                user_identifier = data.get('user_identifier', '')
-
-            return {
-                "success": True,
-                "company_short_name": company.short_name,
-                "user_identifier": user_identifier
-            }
-
-        # --- Failure: No valid credentials found ---
-        logging.info(f"Authentication required. No session cookie or API Key provided.")
-        return {"success": False, "error_message": "Authentication required. No session cookie or API Key provided.",
-                "status_code": 402}
 
     def log_access(self,
                    company_short_name: str,

{iatoolkit-0.59.0 → iatoolkit-0.64.0}/src/iatoolkit/services/branding_service.py

@@ -4,6 +4,7 @@
 # IAToolkit is open source software.
 
 from iatoolkit.repositories.models import Company
+from injector import inject
 
 
 class BrandingService:
@@ -11,6 +12,7 @@ class BrandingService:
     Servicio centralizado que gestiona la configuración de branding.
     """
 
+    @inject
     def __init__(self):
         """
         Define los estilos de branding por defecto para la aplicación.
@@ -19,13 +21,16 @@ class BrandingService:
             # --- Estilos del Encabezado Principal ---
             "header_background_color": "#FFFFFF",
             "header_text_color": "#6C757D",
-            "primary_font_weight": "bold",
-            "primary_font_size": "1rem",
-            "secondary_font_weight": "600",
-            "secondary_font_size": "0.875rem",
-            "tertiary_font_weight": "normal",
-            "tertiary_font_size": "0.75rem",
-            "tertiary_opacity": "0.8",
+            "primary_font_weight": "600",
+            "primary_font_size": "1.2rem",
+            "secondary_font_weight": "400",
+            "secondary_font_size": "0.9rem",
+            "tertiary_font_weight": "300",
+            "tertiary_font_size": "0.8rem",
+            "tertiary_opacity": "0.7",
+
+            # headings
+            "brand_text_heading_color": "#334155",  # Gris pizarra por defecto
 
             # Estilos Globales de la Marca ---
             "brand_primary_color": "#0d6efd",  # Azul de Bootstrap por defecto
@@ -40,25 +45,27 @@ class BrandingService:
             "brand_danger_border": "#f5c2c7",  # Borde rojo intermedio
 
             # Estilos para Alertas Informativas ---
-            "brand_info_bg": "#cff4fc",  # Fondo celeste pálido
-            "brand_info_text": "#055160",  # Texto azul oscuro
-            "brand_info_border": "#b6effb",
+            "brand_info_bg": "#F0F4F8",         # Un fondo de gris azulado muy pálido
+            "brand_info_text": "#0d6efd",       # Texto en el color primario
+            "brand_info_border": "#D9E2EC",     # Borde de gris azulado pálido
 
             # Estilos para el Asistente de Prompts ---
             "prompt_assistant_bg": "#f8f9fa",
             "prompt_assistant_border": "#dee2e6",
-            "prompt_assistant_icon_color": "#6c757d",
             "prompt_assistant_button_bg": "#FFFFFF",
             "prompt_assistant_button_text": "#495057",
             "prompt_assistant_button_border": "#ced4da",
             "prompt_assistant_dropdown_bg": "#f8f9fa",
             "prompt_assistant_header_bg": "#e9ecef",
             "prompt_assistant_header_text": "#495057",
-            "prompt_assistant_item_hover_bg": None,  # Usará el primario por defecto
-            "prompt_assistant_item_hover_text": None,  # Usará el texto sobre primario
+
+            # this use the primary by default
+            "prompt_assistant_icon_color": None,
+            "prompt_assistant_item_hover_bg": None,
+            "prompt_assistant_item_hover_text": None,
 
             # Color para el botón de Enviar ---
-            "send_button_color": "#212529"  # Gris oscuro/casi negro por defecto
+            "send_button_color": "#212529"          # Gris oscuro/casi negro por defecto
         }
 
     def get_company_branding(self, company: Company | None) -> dict:
@@ -80,10 +87,6 @@ class BrandingService:
         secondary_rgb = hex_to_rgb(final_branding_values['brand_secondary_color'])
 
         # --- CONSTRUCCIÓN DE ESTILOS Y VARIABLES CSS ---
-        header_style = (
-            f"background-color: {final_branding_values['header_background_color']}; "
-            f"color: {final_branding_values['header_text_color']};"
-        )
         primary_text_style = (
             f"font-weight: {final_branding_values['primary_font_weight']}; "
             f"font-size: {final_branding_values['primary_font_size']};"
@@ -103,6 +106,10 @@ class BrandingService:
             :root {{
                 --brand-primary-color: {final_branding_values['brand_primary_color']};
                 --brand-secondary-color: {final_branding_values['brand_secondary_color']};
+                --brand-header-bg: {final_branding_values['header_background_color']};
+                --brand-header-text: {final_branding_values['header_text_color']};
+                --brand-text-heading-color: {final_branding_values['brand_text_heading_color']};
+
                 --brand-primary-color-rgb: {', '.join(map(str, primary_rgb))};
                 --brand-secondary-color-rgb: {', '.join(map(str, secondary_rgb))};
                 --brand-text-on-primary: {final_branding_values['brand_text_on_primary']};
@@ -114,11 +121,11 @@ class BrandingService:
                 --brand-danger-text: {final_branding_values['brand_danger_text']};
                 --brand-danger-border: {final_branding_values['brand_danger_border']};
                 --brand-info-bg: {final_branding_values['brand_info_bg']};
-                --brand-info-text: {final_branding_values['brand_info_text']};
+                --brand-info-text: {final_branding_values['brand_info_text'] or final_branding_values['brand_primary_color']};
                 --brand-info-border: {final_branding_values['brand_info_border']};
                 --brand-prompt-assistant-bg: {final_branding_values['prompt_assistant_bg']};
                 --brand-prompt-assistant-border: {final_branding_values['prompt_assistant_border']};
-                --brand-prompt-assistant-icon-color: {final_branding_values['prompt_assistant_icon_color']};
+                --brand-prompt-assistant-icon-color: {final_branding_values['prompt_assistant_icon_color'] or final_branding_values['brand_primary_color']};
                 --brand-prompt-assistant-button-bg: {final_branding_values['prompt_assistant_button_bg']};
                 --brand-prompt-assistant-button-text: {final_branding_values['prompt_assistant_button_text']};
                 --brand-prompt-assistant-button-border: {final_branding_values['prompt_assistant_button_border']};
@@ -133,11 +140,10 @@ class BrandingService:
 
         return {
             "name": company.name if company else "IAToolkit",
-            "header_style": header_style,
             "primary_text_style": primary_text_style,
             "secondary_text_style": secondary_text_style,
             "tertiary_text_style": tertiary_text_style,
             "header_text_color": final_branding_values['header_text_color'],
             "css_variables": css_variables,
-            "send_button_color": final_branding_values['send_button_color']
+            "send_button_color": final_branding_values['brand_primary_color']
         }

iatoolkit-0.64.0/src/iatoolkit/services/help_content_service.py

@@ -0,0 +1,30 @@
+# Copyright (c) 2024 Fernando Libedinsky
+# Product: IAToolkit
+#
+# IAToolkit is open source software.
+
+from iatoolkit.common.util import Utility
+from iatoolkit.common.exceptions import IAToolkitException
+import os
+from injector import inject
+import logging
+
+
+class HelpContentService:
+    @inject
+    def __init__(self, util: Utility):
+        self.util = util
+
+    def get_content(self, company_short_name: str | None) -> dict:
+        filepath = f'companies/{company_short_name}/help_content.yaml'
+        if not os.path.exists(filepath):
+            return {}
+
+        # read the file
+        try:
+            help_content = self.util.load_schema_from_yaml(filepath)
+            return help_content
+        except Exception as e:
+            logging.exception(e)
+            raise IAToolkitException(IAToolkitException.ErrorType.CONFIG_ERROR,
+                            f"Error obteniendo help de {company_short_name}: {str(e)}") from e

{iatoolkit-0.59.0 → iatoolkit-0.64.0}/src/iatoolkit/services/profile_service.py

@@ -131,6 +131,7 @@ class ProfileService:
             "profile": profile
         }
 
+
     def get_profile_by_identifier(self, company_short_name: str, user_identifier: str) -> dict:
         """
         Fetches a user profile directly by their identifier, bypassing the Flask session.

iatoolkit-0.64.0/src/iatoolkit/services/user_feedback_service.py

@@ -0,0 +1,103 @@
+# Copyright (c) 2024 Fernando Libedinsky
+# Product: IAToolkit
+#
+# IAToolkit is open source software.
+
+from iatoolkit.repositories.models import UserFeedback, Company
+from injector import inject
+from iatoolkit.repositories.profile_repo import ProfileRepo
+from iatoolkit.infra.google_chat_app import GoogleChatApp
+from iatoolkit.infra.mail_app import MailApp  # <-- 1. Importar MailApp
+import logging
+
+
+class UserFeedbackService:
+    @inject
+    def __init__(self,
+                 profile_repo: ProfileRepo,
+                 google_chat_app: GoogleChatApp,
+                 mail_app: MailApp):
+        self.profile_repo = profile_repo
+        self.google_chat_app = google_chat_app
+        self.mail_app = mail_app
+
+    def _send_google_chat_notification(self, space_name: str, message_text: str):
+        """Envía una notificación de feedback a un espacio de Google Chat."""
+        try:
+            chat_data = {
+                "type": "MESSAGE_TRIGGER",
+                "space": {"name": space_name},
+                "message": {"text": message_text}
+            }
+            chat_result = self.google_chat_app.send_message(message_data=chat_data)
+            if not chat_result.get('success'):
+                logging.warning(f"Error al enviar notificación a Google Chat: {chat_result.get('message')}")
+        except Exception as e:
+            logging.exception(f"Fallo inesperado al enviar notificación a Google Chat: {e}")
+
+    def _send_email_notification(self, destination_email: str, company_name: str, message_text: str):
+        """Envía una notificación de feedback por correo electrónico."""
+        try:
+            subject = f"Nuevo Feedback de {company_name}"
+            # Convertir el texto plano a un HTML simple para mantener los saltos de línea
+            html_body = message_text.replace('\n', '<br>')
+            self.mail_app.send_email(to=destination_email, subject=subject, body=html_body)
+        except Exception as e:
+            logging.exception(f"Fallo inesperado al enviar email de feedback: {e}")
+
+    def _handle_notification(self, company: Company, message_text: str):
+        """Lee la configuración de la empresa y envía la notificación al canal correspondiente."""
+        feedback_params = company.parameters.get('user_feedback')
+        if not isinstance(feedback_params, dict):
+            logging.warning(f"No se encontró configuración de 'user_feedback' para la empresa {company.short_name}.")
+            return
+
+        # get channel and destination
+        channel = feedback_params.get('channel')
+        destination = feedback_params.get('destination')
+        if not channel or not destination:
+            logging.warning(f"Configuración 'user_feedback' incompleta para {company.short_name}. Faltan 'channel' o 'destination'.")
+            return
+
+        if channel == 'google_chat':
+            self._send_google_chat_notification(space_name=destination, message_text=message_text)
+        elif channel == 'email':
+            self._send_email_notification(destination_email=destination, company_name=company.short_name, message_text=message_text)
+        else:
+            logging.warning(f"Canal de feedback '{channel}' no reconocido para la empresa {company.short_name}.")
+
+    def new_feedback(self,
+                     company_short_name: str,
+                     message: str,
+                     user_identifier: str,
+                     rating: int = None) -> dict:
+        try:
+            # 1. Validar empresa
+            company = self.profile_repo.get_company_by_short_name(company_short_name)
+            if not company:
+                return {'error': f'No existe la empresa: {company_short_name}'}
+
+            # 2. Enviar notificación según la configuración de la empresa
+            notification_text = (f"*Nuevo feedback de {company_short_name}*:\n"
+                                 f"*Usuario:* {user_identifier}\n"
+                                 f"*Mensaje:* {message}\n"
+                                 f"*Calificación:* {rating if rating is not None else 'N/A'}")
+            self._handle_notification(company, notification_text)
+
+            # 3. Guardar el feedback en la base de datos (independientemente del éxito de la notificación)
+            new_feedback_obj = UserFeedback(
+                company_id=company.id,
+                message=message,
+                user_identifier=user_identifier,
+                rating=rating
+            )
+            saved_feedback = self.profile_repo.save_feedback(new_feedback_obj)
+            if not saved_feedback:
+                logging.error(f"No se pudo guardar el feedback para el usuario {user_identifier} en la empresa {company_short_name}")
+                return {'error': 'No se pudo guardar el feedback'}
+
+            return {'success': True, 'message': 'Feedback guardado correctamente'}
+
+        except Exception as e:
+            logging.exception(f"Error crítico en el servicio de feedback: {e}")
+            return {'error': str(e)}