Update app.py

app.py

@@ -406,13 +406,13 @@ def create_ui() -> gr.Blocks:
                 # Progress bar for batch analysis
                 with gr.Row():
                     analysis_progress = gr.Progress()
-                    progress_display = gr.Textbox(
-                        label="📊 Batch Analysis Progress",
-                        interactive=False,
-                        lines=2,
-                        visible=False,
-                        info="Shows progress when analyzing all repositories"
-                    )
+                    # progress_display = gr.Textbox(
+                    #     label="📊 Batch Analysis Progress",
+                    #     interactive=False,
+                    #     lines=2,
+                    #     visible=False,
+                    #     info="Shows progress when analyzing all repositories"
+                    # )
 
                 with gr.Row(equal_height=True):
                     # with gr.Column():
@@ -422,13 +422,14 @@ def create_ui() -> gr.Blocks:
                     #         show_copy_button=True,
                     #         info="Raw content extracted from the repository"
                     #     )
-                    with gr.Column():
-                        summary_output = gr.Textbox(
-                            label="🎯 AI Analysis Summary", 
-                            lines=20, 
-                            show_copy_button=True,
-                            info="Detailed analysis and insights from AI"
-                        )
+                    # with gr.Column():
+                    #     summary_output = gr.Textbox(
+                    #         label="🎯 AI Analysis Summary", 
+                    #         lines=20, 
+                    #         show_copy_button=True,
+                    #         info="Detailed analysis and insights from AI"
+                    #     )
+                    pass
 
                 gr.Markdown("### 📊 Results Dashboard")
                 gr.Markdown("💡 **Tip:** Click on any repository name to explore it in detail!")
@@ -559,12 +560,12 @@ def create_ui() -> gr.Blocks:
             requirements = "\n".join([f"- {msg}" for msg in user_messages if msg.strip()])
             return requirements
 
-        def handle_analyze_next(repo_ids: List[str], current_idx: int, user_requirements: str) -> Tuple[str, pd.DataFrame, int, str]:
+        def handle_analyze_next(repo_ids: List[str], current_idx: int, user_requirements: str) -> Tuple[pd.DataFrame, int, str]:
             """Analyzes the next repository in the list."""
             if not repo_ids:
-                return "", pd.DataFrame(), 0, "Status: No repositories to analyze. Please submit repo IDs first."
+                return pd.DataFrame(), 0, "Status: No repositories to analyze. Please submit repo IDs first."
             if current_idx >= len(repo_ids):
-                return "", read_csv_to_dataframe(), current_idx, "Status: All repositories have been analyzed."
+                return read_csv_to_dataframe(), current_idx, "Status: All repositories have been analyzed."
             
             repo_id_to_analyze = repo_ids[current_idx]
             status = f"Status: Analyzing repository {current_idx + 1}/{len(repo_ids)}: {repo_id_to_analyze}"
@@ -577,7 +578,7 @@ def create_ui() -> gr.Blocks:
             if next_idx >= len(repo_ids):
                 status += "\n\nFinished all analyses."
 
-            return summary, df, next_idx, status
+            return df, next_idx, status
 
         def handle_user_message(user_message: str, history: List[Dict[str, str]]) -> Tuple[List[Dict[str, str]], str]:
             """Appends the user's message to the history, preparing for the bot's response."""
@@ -675,19 +676,17 @@ def create_ui() -> gr.Blocks:
             
             return "", gr.update(visible=False), gr.update()
 
-        def handle_analyze_all_repos(repo_ids: List[str], user_requirements: str, progress=gr.Progress()) -> Tuple[pd.DataFrame, str, str]:
+        def handle_analyze_all_repos(repo_ids: List[str], user_requirements: str, progress=gr.Progress()) -> Tuple[pd.DataFrame, str]:
             """Analyzes all repositories in the CSV file with progress tracking."""
             if not repo_ids:
-                return pd.DataFrame(), "Status: No repositories to analyze. Please submit repo IDs first.", ""
+                return pd.DataFrame(), "Status: No repositories to analyze. Please submit repo IDs first."
             
             total_repos = len(repo_ids)
-            progress_text = f"Starting batch analysis of {total_repos} repositories..."
             
             try:
                 # Start the progress tracking
                 progress(0, desc="Initializing batch analysis...")
                 
-                all_summaries = []
                 successful_analyses = 0
                 failed_analyses = 0
                 csv_update_failures = 0
@@ -718,7 +717,6 @@ def create_ui() -> gr.Blocks:
                                     break
                         
                         if repo_updated:
-                            all_summaries.append(f"✅ {repo_id}: Analysis completed & CSV updated")
                             successful_analyses += 1
                         else:
                             # CSV update failed - try once more
@@ -747,10 +745,8 @@ def create_ui() -> gr.Blocks:
                                             break
                             
                             if retry_success:
-                                all_summaries.append(f"✅ {repo_id}: Analysis completed & CSV updated (retry)")
                                 successful_analyses += 1
                             else:
-                                all_summaries.append(f"⚠️ {repo_id}: Analysis completed but CSV update failed")
                                 csv_update_failures += 1
                         
                         # Longer delay to prevent file conflicts
@@ -758,7 +754,6 @@ def create_ui() -> gr.Blocks:
                         
                     except Exception as e:
                         logger.error(f"Error analyzing {repo_id}: {e}")
-                        all_summaries.append(f"❌ {repo_id}: Error - {str(e)[:100]}...")
                         failed_analyses += 1
                         # Still wait to prevent rapid failures
                         time.sleep(0.2)
@@ -771,34 +766,23 @@ def create_ui() -> gr.Blocks:
                 if csv_update_failures > 0:
                     final_status += f"\n⚠️ CSV Update Issues: {csv_update_failures}/{total_repos}"
                 
-                # Create progress summary
-                progress_summary = "\n".join(all_summaries[-15:])  # Show last 15 entries
-                if len(all_summaries) > 15:
-                    progress_summary = f"... (showing last 15 of {len(all_summaries)} repositories)\n" + progress_summary
-                
                 # Get final updated dataframe
                 updated_df = read_csv_to_dataframe()
                 
                 logger.info(f"Batch analysis completed: {successful_analyses} successful, {failed_analyses} failed, {csv_update_failures} CSV update issues")
-                return updated_df, final_status, progress_summary
+                return updated_df, final_status
                 
             except Exception as e:
                 logger.error(f"Error in batch analysis: {e}")
                 error_status = f"❌ Batch analysis failed: {e}"
-                return read_csv_to_dataframe(), error_status, ""
+                return read_csv_to_dataframe(), error_status
 
         def handle_visit_repo(repo_id: str) -> Tuple[Any, str]:
             """Handle visiting the Hugging Face Space for the repository."""
             if repo_id and repo_id.strip():
                 hf_url = f"https://huggingface.co/spaces/{repo_id.strip()}"
                 logger.info(f"User chose to visit: {hf_url}")
-                # Use JavaScript to open URL in new tab
-                js_code = f"""
-                <script>
-                window.open('{hf_url}', '_blank');
-                </script>
-                """
-                return gr.update(visible=False), f"🌐 Opening: {hf_url}"
+                return gr.update(visible=False), hf_url
             return gr.update(visible=False), ""
 
         def handle_explore_repo(repo_id: str) -> Tuple[Any, Any, str]:
@@ -836,18 +820,15 @@ def create_ui() -> gr.Blocks:
         analyze_next_btn.click(
             fn=handle_analyze_next,
             inputs=[repo_ids_state, current_repo_idx_state, user_requirements_state],
-            outputs=[summary_output, df_output, current_repo_idx_state, status_box_analysis]
+            outputs=[df_output, current_repo_idx_state, status_box_analysis]
         )
         analyze_all_btn.click(
-            fn=lambda: gr.update(visible=True),  # Show progress display
-            outputs=[progress_display]
+            fn=lambda: None,  # No need to show progress display since it's commented out
+            outputs=[]
         ).then(
             fn=handle_analyze_all_repos,
             inputs=[repo_ids_state, user_requirements_state],
-            outputs=[df_output, status_box_analysis, progress_display]
-        ).then(
-            fn=lambda: gr.update(visible=True),  # Keep progress display visible with results
-            outputs=[progress_display]
+            outputs=[df_output, status_box_analysis]
         )
         
         # Chatbot Tab
@@ -891,7 +872,8 @@ def create_ui() -> gr.Blocks:
         visit_repo_btn.click(
             fn=handle_visit_repo,
             inputs=[selected_repo_display],
-            outputs=[repo_action_modal, selected_repo_display]
+            outputs=[repo_action_modal, selected_repo_display],
+            js="(repo_id) => { if(repo_id && repo_id.trim()) { window.open('https://huggingface.co/spaces/' + repo_id.trim(), '_blank'); } }"
         )
         explore_repo_btn.click(
             fn=handle_explore_repo,