Update src/graph/graph_loader.py

src/graph/graph_loader.py

@@ -7,26 +7,26 @@ from dotenv import load_dotenv
 load_dotenv()
 
 class KnowledgeGraphPersister:
-    
     def __init__(self):
         """
-        Inizializza il driver Neo4j usando le variabili d'ambiente per sicurezza.
+        Inizializza il driver Neo4j e crea i vincoli necessari per le performance.
         """
-        uri = os.getenv("NEO4J_URI", "")
+        uri = os.getenv("NEO4J_URI", "neo4j+s://748d6c94.databases.neo4j.io")
         user = os.getenv("NEO4J_USER", "neo4j")
-        password = os.getenv("NEO4J_PASSWORD", "")
+        password = os.getenv("NEO4J_PASSWORD", "t1bT1DiXwDOGMYfX89qR20loSN8FXurB3Dfg8bPQcTI")
         
         try:
             self.driver = GraphDatabase.driver(uri, auth=(user, password))
             self.driver.verify_connectivity()
             print(f"✅ Connesso a Neo4j ({uri}).")
+            
             # Creazione indici all'avvio (Fondamentale per la velocità dei MERGE)
             self._create_constraints()
+            
         except Exception as e:
             print(f"❌ Errore critico connessione Neo4j: {e}")
             self.driver = None
 
-    
     def close(self):
         if self.driver:
             self.driver.close()
@@ -43,6 +43,7 @@ class KnowledgeGraphPersister:
                 session.run(query)
                 print("⚡ Vincoli/Indici Neo4j verificati.")
             except Exception as e:
+                # Spesso fallisce se l'utente non ha permessi admin o se esiste già con nome diverso
                 print(f"⚠️ Warning creazione indici: {e}")
 
     def sanitize_name(self, name):
@@ -51,15 +52,25 @@ class KnowledgeGraphPersister:
         """
         if not name: return "Unknown"
         # Rimuove spazi extra e normalizza.
-        # Nota: In produzione usare slugify o urllib.parse.quote per URI robusti
         return name.strip().replace(" ", "_").replace("'", "").replace('"', "")
 
     def sanitize_predicate(self, pred):
         """
-        Pulisce il predicato per evitare Cypher Injection, dato che non può essere parametrizzato.
+        Pulisce il predicato per evitare Cypher Injection.
+        FIX: Gestisce meglio i separatori (:, -, spazio) sostituendoli con underscore
+        per evitare predicati illeggibili come XCHEHASOBJECT.
+        Es. xche:has_object -> XCHE_HAS_OBJECT
         """
-        # Accetta solo caratteri alfanumerici e underscore. Upper case per convenzione Neo4j.
+        if not pred: return "RELATED_TO"
+        
+        # 1. Normalizzazione preliminare dei separatori comuni
+        # Sostituisce i due punti dei namespace e trattini con underscore
+        pred = pred.replace(":", "_").replace("-", "_").replace(" ", "_")
+        
+        # 2. Rimozione caratteri non sicuri (mantiene solo alfanumerici e underscore)
         clean = "".join(x for x in pred if x.isalnum() or x == "_")
+        
+        # 3. Conversione in uppercase (convenzione Neo4j per Relationships)
         return clean.upper() if clean else "RELATED_TO"
 
     def save_triples(self, triples):
@@ -73,7 +84,6 @@ class KnowledgeGraphPersister:
         print(f"💾 Preparazione Batch di {len(triples)} triple...")
 
         # 1. Raggruppamento per Predicato
-        # Struttura: { "LOCATED_IN": [ {subj:..., obj:..., ...}, ... ], "HAS_TYPE": [...] }
         batched_by_pred = defaultdict(list)
         
         for t in triples:
@@ -84,7 +94,7 @@ class KnowledgeGraphPersister:
                 "subj_label": t.subject,
                 "obj_uri": self.sanitize_name(t.object),
                 "obj_label": t.object,
-                "conf": float(t.confidence), # Assicura float nativo
+                "conf": float(t.confidence),
                 "src": t.source or "unknown"
             }
             batched_by_pred[safe_pred].append(item)
@@ -104,10 +114,7 @@ class KnowledgeGraphPersister:
     def _unwind_write_tx(tx, predicate, batch_data):
         """
         Usa UNWIND per inserire migliaia di righe in un colpo solo.
-        Molto più performante su rete cloud.
         """
-        # La query è dinamica SOLO sul tipo di relazione (sanitizzato prima),
-        # tutto il resto passa come parametro lista ($batch).
         query = (
             f"UNWIND $batch AS row "
             f"MERGE (s:Resource {{uri: row.subj_uri}}) "