Update services/agentic_handlers.py

services/agentic_handlers.py

@@ -49,9 +49,8 @@ class AgenticHandlers:
 
         # Initial "waiting" status updates
         initial_report_status = "Pipeline AI: In attesa dei dati necessari..."
-        # initial_okr_choices = [] # No longer used directly for okrs_cbg_update if it's Markdown
         initial_okr_value_for_markdown = "" # Assuming this component is Markdown, value should be string
-        initial_okr_interactive = False
+        # initial_okr_interactive = False # Markdown does not take 'interactive' in gr.update
         initial_okr_details = "Pipeline AI: In attesa dei dati necessari..."
         initial_orchestration_results = self.orchestration_raw_results_st.value # Preserve if re-running
         initial_selected_krs = self.selected_key_result_ids_st.value
@@ -61,9 +60,9 @@ class AgenticHandlers:
         report_status_md_update = gr.update(value=initial_report_status) if self.report_components.get("agentic_pipeline_status_md") else gr.update()
         report_display_md_update = gr.update() # No change to report display yet
         
-        # If key_results_cbg is the Markdown component causing the error, its update should not have 'choices'.
+        # If key_results_cbg is the Markdown component causing the error, its update should not have 'interactive'.
         # Its value should be a string.
-        okrs_cbg_update = gr.update(value=initial_okr_value_for_markdown, interactive=initial_okr_interactive) if self.okrs_components.get("key_results_cbg") else gr.update()
+        okrs_cbg_update = gr.update(value=initial_okr_value_for_markdown) if self.okrs_components.get("key_results_cbg") else gr.update()
         okrs_detail_md_update = gr.update(value=initial_okr_details) if self.okrs_components.get("okr_detail_display_md") else gr.update()
 
         if not current_token_state_val or not current_token_state_val.get("token"):
@@ -86,7 +85,7 @@ class AgenticHandlers:
         if self.okrs_components.get("okr_detail_display_md"): # Also update OKR detail placeholder
              okrs_detail_md_update = gr.update(value="Dettagli OKR (Sempre) in corso di generazione...")
         
-        # okrs_cbg_update remains as initially defined (waiting state, no choices)
+        # okrs_cbg_update remains as initially defined (waiting state, no choices/interactive)
         yield (
             report_status_md_update, report_display_md_update, okrs_cbg_update, okrs_detail_md_update,
             initial_orchestration_results, initial_selected_krs, initial_krs_for_selection
@@ -101,9 +100,9 @@ class AgenticHandlers:
                 report_display_md_update = gr.update(value=error_status) # Update report display too
             
             # If key_results_cbg is Markdown, update its value to reflect the error.
-            # No choices argument here.
+            # No choices or interactive argument here.
             if self.okrs_components.get("key_results_cbg"):
-                 okrs_cbg_update = gr.update(value=error_status, interactive=False) # Value is string, interactive is False
+                 okrs_cbg_update = gr.update(value=error_status) # Value is string
             
             if self.okrs_components.get("okr_detail_display_md"):
                  okrs_detail_md_update = gr.update(value=error_status) # Update OKR detail too
@@ -146,19 +145,11 @@ class AgenticHandlers:
                 krs_for_ui_selection_list = extract_key_results_for_selection(actionable_okrs)
                 krs_for_selection_update_val = krs_for_ui_selection_list # Store for CBG change handler
                 
-                # kr_choices_for_cbg = [(kr['kr_description'], kr['unique_kr_id']) for kr in krs_for_ui_selection_list] # Original
-                # If key_results_cbg is Markdown, we don't set choices.
-                # We might set its value to a string representation of these KRs, or leave it to be handled by selection logic.
-                # For now, setting value to a generic message or empty if it's Markdown.
-                # If it were a CBG, value=[] would be appropriate for selection reset.
-                # For Markdown, value should be a string.
                 if self.okrs_components.get("key_results_cbg"):
-                    # If this component is Markdown, it cannot take choices.
-                    # Its value should be a string. For now, an empty string or a placeholder.
-                    # If you want to display the KRs as text, you'd format them here.
-                    okrs_cbg_update = gr.update(value="Key Results available for selection (display as Markdown)", interactive=True) 
+                    # If this component is Markdown, it cannot take choices or interactive.
+                    # Its value should be a string.
+                    okrs_cbg_update = gr.update(value="Key Results available for selection (display as Markdown)") 
                                                 # Example string value if it's Markdown.
-                                                # Or simply value=""
                 
                 # Display all OKRs initially (before any KR selection)
                 all_okrs_md_parts = []
@@ -181,7 +172,7 @@ class AgenticHandlers:
                 
                 # If key_results_cbg is Markdown
                 if self.okrs_components.get("key_results_cbg"):
-                    okrs_cbg_update = gr.update(value="Nessun Key Result (Markdown display)", interactive=False) # String value
+                    okrs_cbg_update = gr.update(value="Nessun Key Result (Markdown display)") # String value
                 
                 if self.okrs_components.get("okr_detail_display_md"):
                     okrs_detail_md_update = gr.update(value="Nessun OKR generato o errore nella pipeline AI (Sempre).")
@@ -204,7 +195,7 @@ class AgenticHandlers:
             
             # If key_results_cbg is Markdown
             if self.okrs_components.get("key_results_cbg"):
-                okrs_cbg_update = gr.update(value=f"Errore Key Results (Markdown): {str(e)}", interactive=False) # String value
+                okrs_cbg_update = gr.update(value=f"Errore Key Results (Markdown): {str(e)}") # String value
 
             if self.okrs_components.get("okr_detail_display_md"):
                 okrs_detail_md_update = gr.update(value=f"Errore generazione OKR AI (Sempre): {str(e)}")