Update services/agentic_handlers.py

services/agentic_handlers.py

@@ -49,8 +49,8 @@ class AgenticHandlers:
 
         # Initial "waiting" status updates
         initial_report_status = "Pipeline AI: In attesa dei dati necessari..."
-        initial_okr_choices = []
-        initial_okr_value = []
+        # 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_details = "Pipeline AI: In attesa dei dati necessari..."
         initial_orchestration_results = self.orchestration_raw_results_st.value # Preserve if re-running
@@ -61,7 +61,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
         
-        okrs_cbg_update = gr.update(choices=initial_okr_choices, value=initial_okr_value, interactive=initial_okr_interactive) if self.okrs_components.get("key_results_cbg") else gr.update()
+        # If key_results_cbg is the Markdown component causing the error, its update should not have 'choices'.
+        # 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_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"):
@@ -69,8 +71,8 @@ class AgenticHandlers:
             yield (
                 report_status_md_update, # For agentic_pipeline_status_md
                 report_display_md_update, # For agentic_report_display_md (no change yet)
-                okrs_cbg_update,         # For key_results_cbg
-                okrs_detail_md_update,   # For okr_detail_display_md
+                okrs_cbg_update,          # For key_results_cbg (now updated as if it's Markdown)
+                okrs_detail_md_update,    # For okr_detail_display_md
                 initial_orchestration_results, # For orchestration_raw_results_st
                 initial_selected_krs,          # For selected_key_result_ids_st
                 initial_krs_for_selection      # For key_results_for_selection_st
@@ -84,6 +86,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)
         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
@@ -96,6 +99,12 @@ class AgenticHandlers:
                 report_status_md_update = gr.update(value=error_status)
             if self.report_components.get("agentic_report_display_md"):
                 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.
+            if self.okrs_components.get("key_results_cbg"):
+                 okrs_cbg_update = gr.update(value=error_status, interactive=False) # Value is string, interactive is False
+            
             if self.okrs_components.get("okr_detail_display_md"):
                  okrs_detail_md_update = gr.update(value=error_status) # Update OKR detail too
 
@@ -137,9 +146,19 @@ 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]
+                # 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"):
-                    okrs_cbg_update = gr.update(choices=kr_choices_for_cbg, value=[], interactive=True)
+                    # 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) 
+                                                # Example string value if it's Markdown.
+                                                # Or simply value=""
                 
                 # Display all OKRs initially (before any KR selection)
                 all_okrs_md_parts = []
@@ -159,8 +178,11 @@ class AgenticHandlers:
                 final_status_text = "Pipeline AI (Sempre): Nessun risultato prodotto."
                 if self.report_components.get("agentic_report_display_md"):
                     report_display_md_update = gr.update(value="Nessun report generato dalla pipeline AI (Sempre).")
+                
+                # If key_results_cbg is Markdown
                 if self.okrs_components.get("key_results_cbg"):
-                    okrs_cbg_update = gr.update(choices=[], value=[], interactive=False)
+                    okrs_cbg_update = gr.update(value="Nessun Key Result (Markdown display)", interactive=False) # 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).")
             
@@ -179,8 +201,11 @@ class AgenticHandlers:
                 report_status_md_update = gr.update(value=error_status_text)
             if self.report_components.get("agentic_report_display_md"):
                  report_display_md_update = gr.update(value=f"Errore generazione report AI (Sempre): {str(e)}")
+            
+            # If key_results_cbg is Markdown
             if self.okrs_components.get("key_results_cbg"):
-                okrs_cbg_update = gr.update(choices=[], value=[], interactive=False)
+                okrs_cbg_update = gr.update(value=f"Errore Key Results (Markdown): {str(e)}", interactive=False) # 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)}")
 
@@ -262,12 +287,17 @@ class AgenticHandlers:
             return
 
         if self.okrs_components.get("key_results_cbg"):
+            # The input 'key_results_cbg' provides the selected IDs.
+            # If it's truly a Markdown component now, this .change handler might not behave as expected
+            # for selection, as Markdown isn't typically an input component this way.
+            # However, the function signature of update_okr_display_on_kr_selection expects selected_kr_unique_ids.
+            # This setup might need further review if key_results_cbg is indeed Markdown and meant to be interactive.
             self.okrs_components['key_results_cbg'].change(
                 fn=self.update_okr_display_on_kr_selection,
                 inputs=[
-                    self.okrs_components['key_results_cbg'], # Current value of checkbox group
-                    self.orchestration_raw_results_st,      # State
-                    self.key_results_for_selection_st       # State
+                    self.okrs_components['key_results_cbg'], # Current value of checkbox group (or Markdown if changed)
+                    self.orchestration_raw_results_st,     # State
+                    self.key_results_for_selection_st      # State
                 ],
                 outputs=[self.okrs_components['okr_detail_display_md']],
                 api_name="update_okr_display_on_kr_selection"
@@ -275,4 +305,3 @@ class AgenticHandlers:
             logging.info("Agentic OKR selection handler setup complete.")
         else:
             logging.warning("key_results_cbg component not found for agentic OKR handler setup.")
-