Update app.py

app.py

@@ -94,25 +94,46 @@ def run_query(driver, query, params=None):
 # --- UI: SIDEBAR ---
 st.sidebar.title("⚙️ Configurazione")
 
+# Recuperiamo le variabili d'ambiente (Server Side)
+# NOTA: Queste variabili contengono i veri segreti ma NON vengono passate al frontend
 env_uri = os.getenv("NEO4J_URI", "")
 env_user = os.getenv("NEO4J_USER", "neo4j")
 env_password = os.getenv("NEO4J_PASSWORD", "")
 env_hf_token = os.getenv("HF_TOKEN", "")
 
 st.sidebar.subheader("Backend AI")
-hf_token_input = st.sidebar.text_input("HF Token (Opzionale)", value=env_hf_token, type="password")
-if hf_token_input: os.environ["HF_TOKEN"] = hf_token_input
+# LOGICA SICURA: Se il token esiste nei secrets, mostriamo solo un badge verde.
+# Non mostriamo mai il token nel campo input (value=...).
+if env_hf_token:
+    st.sidebar.success("✅ HF Token: Configurato da Secrets")
+    # Se l'utente vuole sovrascriverlo, può usare questo campo opzionale
+    hf_token_input = st.sidebar.text_input("Sovrascrivi Token (Opzionale)", type="password", key="hf_token_override")
+    if hf_token_input:
+        os.environ["HF_TOKEN"] = hf_token_input
+else:
+    hf_token_input = st.sidebar.text_input("Inserisci HF Token", type="password")
+    if hf_token_input:
+        os.environ["HF_TOKEN"] = hf_token_input
 
 st.sidebar.subheader("Knowledge Graph")
+# URI e User non sono segreti critici, possiamo mostrarli pre-compilati
 uri = st.sidebar.text_input("URI", value=env_uri)
 user = st.sidebar.text_input("User", value=env_user)
-password = st.sidebar.text_input("Password", value=env_password, type="password")
+
+# LOGICA SICURA: Gestione Password
+# Non usiamo 'value=env_password' per evitare che finisca nell'HTML.
+pwd_placeholder = "✅ Configurato da Secrets (Lascia vuoto)" if env_password else "Inserisci Password"
+password_input = st.sidebar.text_input("Password", type="password", placeholder=pwd_placeholder)
+
+# Determiniamo quale password usare (Input Utente > Secret Env)
+password = password_input if password_input else env_password
 
 driver = None
 if uri and password:
     driver = get_driver(uri, user, password)
     if driver:
         st.sidebar.success("🟢 Connesso a Neo4j")
+        # Aggiorniamo l'ambiente per i moduli backend che usano os.getenv
         os.environ["NEO4J_URI"] = uri
         os.environ["NEO4J_USER"] = user
         os.environ["NEO4J_PASSWORD"] = password
@@ -221,8 +242,8 @@ with tab_gen:
     is_step_b_unlocked = st.session_state.pipeline_stage >= 1
     
     with st.container():
-        # Header grigio se bloccato, colorato se attivo
-        color = "black" if is_step_b_unlocked else "gray"
+        # Header grigio se bloccato, bianco (per dark mode) se attivo
+        color = "white" if is_step_b_unlocked else "gray"
         icon = "✅" if st.session_state.pipeline_stage >= 2 else ("2️⃣" if is_step_b_unlocked else "🔒")
         st.markdown(f"<h3 style='color:{color}'>{icon} Fase B: Information Extraction</h3>", unsafe_allow_html=True)
         
@@ -275,7 +296,7 @@ with tab_gen:
     is_step_c_unlocked = st.session_state.pipeline_stage >= 2
     
     with st.container():
-        color = "black" if is_step_c_unlocked else "gray"
+        color = "white" if is_step_c_unlocked else "gray"
         icon = "✅" if st.session_state.pipeline_stage >= 3 else ("3️⃣" if is_step_c_unlocked else "🔒")
         st.markdown(f"<h3 style='color:{color}'>{icon} Fase C: Graph Construction</h3>", unsafe_allow_html=True)
 
@@ -289,6 +310,7 @@ with tab_gen:
                 Puoi esplorarli nei tab "Validazione" e "Visualizzazione".
             </div>
             """, unsafe_allow_html=True)
+            st.balloons()
             if st.button("Riavvia con nuovo file"):
                 reset_pipeline()
                 st.rerun()
@@ -355,32 +377,39 @@ with tab_val:
 with tab_vis:
     st.header("Esplorazione Topologica")
     if driver:
-        physics = st.checkbox("Abilita Fisica (Gravità)", value=True)
-        if st.button("Aggiorna Visualizzazione"):
-            cypher_vis = """
-            MATCH (s)-[r]->(o) 
-            RETURN COALESCE(s.label, s.name, head(labels(s))) as src, 
-                   type(r) as rel, 
-                   COALESCE(o.label, o.name, head(labels(o))) as dst 
-            LIMIT 100
-            """
-            graph_data = run_query(driver, cypher_vis)
+        col_ctrl, col_info = st.columns([1, 4])
+        with col_ctrl:
+            physics = st.checkbox("Abilita Fisica (Gravità)", value=True)
+            if st.button("🔄 Ricarica Dati"):
+                st.rerun()
+        
+        # Logica di visualizzazione automatica (non dipendente da un bottone)
+        cypher_vis = """
+        MATCH (s)-[r]->(o) 
+        RETURN COALESCE(s.label, s.name, head(labels(s))) as src, 
+                type(r) as rel, 
+                COALESCE(o.label, o.name, head(labels(o))) as dst 
+        LIMIT 100
+        """
+        graph_data = run_query(driver, cypher_vis)
+        
+        if graph_data:
+            net = Network(height="600px", width="100%", bgcolor="#222222", font_color="white", notebook=False)
+            for item in graph_data:
+                src, dst, rel = str(item['src']), str(item['dst']), str(item['rel'])
+                net.add_node(src, label=src, color="#4facfe", title=src)
+                net.add_node(dst, label=dst, color="#00f2fe", title=dst)
+                net.add_edge(src, dst, title=rel, label=rel)
             
-            if graph_data:
-                net = Network(height="600px", width="100%", bgcolor="#222222", font_color="white", notebook=False)
-                for item in graph_data:
-                    src, dst, rel = str(item['src']), str(item['dst']), str(item['rel'])
-                    net.add_node(src, label=src, color="#4facfe", title=src)
-                    net.add_node(dst, label=dst, color="#00f2fe", title=dst)
-                    net.add_edge(src, dst, title=rel, label=rel)
-                
-                net.toggle_physics(physics)
-                path = "data/processed/graph_viz.html"
-                os.makedirs("data/processed", exist_ok=True)
-                net.save_graph(path)
-                
-                with open(path, 'r', encoding='utf-8') as f:
-                    html_string = f.read()
-                components.html(html_string, height=600, scrolling=True)
+            net.toggle_physics(physics)
+            path = "data/processed/graph_viz.html"
+            os.makedirs("data/processed", exist_ok=True)
+            net.save_graph(path)
+            
+            with open(path, 'r', encoding='utf-8') as f:
+                html_string = f.read()
+            components.html(html_string, height=600, scrolling=True)
+        else:
+            st.info("Il grafo è attualmente vuoto o non raggiungibile.")
     else:
-        st.warning("Database non connesso.")
+        st.warning("Database non connesso. Configura le credenziali nella sidebar.")