Update app.py

app.py

@@ -180,22 +180,13 @@ def read_csv_to_dataframe() -> pd.DataFrame:
         return pd.DataFrame()
 
 def format_dataframe_for_display(df: pd.DataFrame) -> pd.DataFrame:
-    """Formats dataframe for display with truncated text while preserving original data."""
+    """Returns dataframe with full text (no truncation) for display."""
     if df.empty:
         return df
     
-    # Create a copy for display purposes
-    display_df = df.copy()
-    
-    # Apply formatting only for display
-    if not display_df.empty:
-        display_df['repo id'] = display_df['repo id'].apply(lambda x: format_text_for_dataframe(x, 50))
-        display_df['strength'] = display_df['strength'].apply(lambda x: format_text_for_dataframe(x, 180))
-        display_df['weaknesses'] = display_df['weaknesses'].apply(lambda x: format_text_for_dataframe(x, 180))
-        display_df['speciality'] = display_df['speciality'].apply(lambda x: format_text_for_dataframe(x, 150))
-        # Keep relevance rating as is since it should be short
-    
-    return display_df
+    # Return the dataframe as-is without any text truncation
+    # This will show the full text content in the CSV display
+    return df.copy()
 
 def analyze_and_update_single_repo(repo_id: str, user_requirements: str = "") -> Tuple[str, str, pd.DataFrame]:
     """
@@ -261,7 +252,7 @@ def analyze_and_update_single_repo(repo_id: str, user_requirements: str = "") ->
                 logger.error(f"Failed to write CSV for {repo_id} on retry: {retry_error}")
 
         logger.info(f"Successfully analyzed and updated CSV for {repo_id}")
-        return combined_content, summary, format_dataframe_for_display(df)
+        return combined_content, summary, df
 
     except Exception as e:
         logger.error(f"An error occurred during analysis of {repo_id}: {e}")
@@ -389,37 +380,41 @@ def create_ui() -> gr.Blocks:
         border-radius: 4px;
     }
     
-    /* Improved dataframe styling */
+    /* Improved dataframe styling for full text display */
     .gr-dataframe {
-        max-height: 400px;
+        max-height: 600px;
         overflow-y: auto;
+        overflow-x: auto;
     }
     
     .gr-dataframe table {
-        table-layout: fixed;
+        table-layout: auto;
         width: 100%;
+        min-width: 1200px;
     }
     
     .gr-dataframe th,
     .gr-dataframe td {
-        padding: 8px 12px;
+        padding: 12px 15px;
         vertical-align: top;
         word-wrap: break-word;
         overflow-wrap: break-word;
-        max-height: 100px;
+        max-height: 200px;
         overflow-y: auto;
+        white-space: pre-wrap;
+        line-height: 1.4;
     }
     
     .gr-dataframe th:nth-child(1),
-    .gr-dataframe td:nth-child(1) { width: 15%; }
+    .gr-dataframe td:nth-child(1) { width: 200px; min-width: 200px; }
     .gr-dataframe th:nth-child(2),
-    .gr-dataframe td:nth-child(2) { width: 25%; }
+    .gr-dataframe td:nth-child(2) { width: 300px; min-width: 300px; }
     .gr-dataframe th:nth-child(3),
-    .gr-dataframe td:nth-child(3) { width: 25%; }
+    .gr-dataframe td:nth-child(3) { width: 300px; min-width: 300px; }
     .gr-dataframe th:nth-child(4),
-    .gr-dataframe td:nth-child(4) { width: 20%; }
+    .gr-dataframe td:nth-child(4) { width: 250px; min-width: 250px; }
     .gr-dataframe th:nth-child(5),
-    .gr-dataframe td:nth-child(5) { width: 15%; }
+    .gr-dataframe td:nth-child(5) { width: 150px; min-width: 150px; }
     
     /* Make repository names clickable */
     .gr-dataframe td:nth-child(1) {
@@ -435,24 +430,13 @@ def create_ui() -> gr.Blocks:
         transform: scale(1.02);
     }
     
-    /* Make content columns (strengths, weaknesses, speciality) clickable for text expansion */
+    /* Content columns - readable styling with scroll for long text */
     .gr-dataframe td:nth-child(2),
     .gr-dataframe td:nth-child(3),
-    .gr-dataframe td:nth-child(4) {
-        cursor: pointer;
-        transition: all 0.3s ease;
-    }
-    
-    .gr-dataframe td:nth-child(2):hover,
-    .gr-dataframe td:nth-child(3):hover,
-    .gr-dataframe td:nth-child(4):hover {
-        background-color: rgba(102, 126, 234, 0.08);
-        box-shadow: inset 0 0 0 1px rgba(102, 126, 234, 0.2);
-    }
-    
-    /* Relevance column - not clickable */
+    .gr-dataframe td:nth-child(4),
     .gr-dataframe td:nth-child(5) {
         cursor: default;
+        font-size: 0.9rem;
     }
     
     .gr-dataframe tbody tr:hover {
@@ -494,15 +478,42 @@ def create_ui() -> gr.Blocks:
             });
         });
         
-        // Listen for programmatic tab changes (button-triggered navigation)
+        // Enhanced listener for programmatic tab changes (button-triggered navigation)
         let lastSelectedTab = null;
-        setInterval(function() {
+        const checkInterval = setInterval(function() {
             const currentSelectedTab = document.querySelector('.gr-tab-nav button.selected');
             if (currentSelectedTab && currentSelectedTab !== lastSelectedTab) {
                 lastSelectedTab = currentSelectedTab;
-                scrollToTop();
+                setTimeout(scrollToTop, 100);
             }
-        }, 200);
+        }, 100);
+        
+        // Additional scroll trigger for repo explorer navigation
+        window.addEventListener('repoExplorerNavigation', function() {
+            setTimeout(scrollToTop, 200);
+        });
+        
+        // Watch for specific tab transitions to repo explorer
+        const repoExplorerObserver = new MutationObserver(function(mutations) {
+            mutations.forEach(function(mutation) {
+                if (mutation.type === 'attributes' && mutation.attributeName === 'class') {
+                    const target = mutation.target;
+                    if (target.textContent && target.textContent.includes('🔍 Repo Explorer') && target.classList.contains('selected')) {
+                        setTimeout(scrollToTop, 150);
+                    }
+                }
+            });
+        });
+        
+        // Start observing for repo explorer specific changes
+        setTimeout(function() {
+            const repoExplorerTab = Array.from(document.querySelectorAll('.gr-tab-nav button')).find(btn => 
+                btn.textContent && btn.textContent.includes('🔍 Repo Explorer')
+            );
+            if (repoExplorerTab) {
+                repoExplorerObserver.observe(repoExplorerTab, { attributes: true });
+            }
+        }, 1000);
     });
     </script>
     """
@@ -634,9 +645,9 @@ def create_ui() -> gr.Blocks:
                         interactive=False
                     )
                 
-                gr.Markdown("💡 **Tip:** Click on any repository name to explore it in detail!")
+                gr.Markdown("💡 **Tip:** Full text is displayed directly in the table. Click on repository names to explore or visit them!")
                 
-                # Text expansion modal for showing full content
+                # Text expansion modal for showing full content (kept for backwards compatibility)
                 with gr.Row():
                     with gr.Column():
                         text_expansion_modal = gr.Column(visible=False)
@@ -860,7 +871,7 @@ def create_ui() -> gr.Blocks:
             return final_keywords_str, status, user_requirements
 
         def handle_dataframe_select(evt: gr.SelectData, df_data) -> Tuple[str, Any, Any, str, str, Any]:
-            """Handle dataframe row selection - repo ID shows modal, content columns show full text."""
+            """Handle dataframe row selection - only repo ID (column 0) shows modal since full text is now displayed directly."""
             print(f"DEBUG: Selection event triggered!")
             print(f"DEBUG: evt = {evt}")
             print(f"DEBUG: df_data type = {type(df_data)}")
@@ -877,31 +888,7 @@ def create_ui() -> gr.Blocks:
                 # Handle pandas DataFrame
                 if isinstance(df_data, pd.DataFrame) and not df_data.empty and row_idx < len(df_data):
                     
-                    # For text expansion, read full data from CSV to get untruncated text
-                    if col_idx in [1, 2, 3]:  # Strength, Weakness, or Speciality columns
-                        repo_name = str(df_data.iloc[row_idx, 0])
-                        
-                        # Read full data from CSV file to get untruncated text
-                        full_df = read_csv_to_dataframe()
-                        
-                        # Find the matching row in the full data
-                        matching_rows = full_df[full_df['repo id'] == repo_name]
-                        if not matching_rows.empty:
-                            full_row = matching_rows.iloc[0]
-                            
-                            if col_idx == 1:  # Strengths column
-                                full_text = str(full_row['strength'])
-                                title = f"Strengths - {repo_name}"
-                            elif col_idx == 2:  # Weaknesses column
-                                full_text = str(full_row['weaknesses'])
-                                title = f"Weaknesses - {repo_name}"
-                            elif col_idx == 3:  # Speciality column
-                                full_text = str(full_row['speciality'])
-                                title = f"Speciality - {repo_name}"
-                            
-                            return "", gr.update(visible=False), gr.update(), title, full_text, gr.update(visible=True)
-                    
-                    elif col_idx == 0:  # Repository name column - show action modal
+                    if col_idx == 0:  # Repository name column - show action modal
                         repo_id = df_data.iloc[row_idx, 0]
                         print(f"DEBUG: Extracted repo_id = '{repo_id}'")
                         
@@ -910,9 +897,9 @@ def create_ui() -> gr.Blocks:
                             logger.info(f"Showing modal for repository: {clean_repo_id}")
                             return clean_repo_id, gr.update(visible=True), gr.update(), "", "", gr.update(visible=False)
                     
-                    # For other columns (like relevance), do nothing
+                    # For content columns (1,2,3) and relevance (4), do nothing since full text is shown directly
                     else:
-                        print(f"DEBUG: Clicked on column {col_idx}, no action defined")
+                        print(f"DEBUG: Clicked on column {col_idx}, full text already shown in table")
                         return "", gr.update(visible=False), gr.update(), "", "", gr.update(visible=False)
                 else:
                     print(f"DEBUG: df_data is not a DataFrame or row_idx {row_idx} out of range")
@@ -1200,7 +1187,8 @@ def create_ui() -> gr.Blocks:
         explore_repo_btn.click(
             fn=handle_explore_repo,
             inputs=[selected_repo_display],
-            outputs=[repo_action_modal, tabs, repo_components["repo_explorer_input"]]
+            outputs=[repo_action_modal, tabs, repo_components["repo_explorer_input"]],
+            js="() => { setTimeout(() => { window.scrollTo({top: 0, behavior: 'smooth'}); window.dispatchEvent(new Event('repoExplorerNavigation')); }, 150); }"
         )
         cancel_modal_btn.click(
             fn=handle_cancel_modal,