updated

backend/services/report_generator.py

@@ -28,8 +28,7 @@ from typing import List, Dict, Any, Tuple
 import matplotlib.pyplot as plt
 from matplotlib.backends.backend_pdf import PdfPages
 import matplotlib.patches as mpatches
-from matplotlib.patches import Rectangle, FancyBboxPatch, Circle
-import matplotlib.dates as mdates
+from matplotlib.patches import Rectangle, FancyBboxPatch
 from datetime import datetime
 
 
@@ -68,7 +67,6 @@ def generate_llm_interview_report(application) -> str:
 
     candidate_skills: List[str] = candidate_features.get('skills', []) or []
     job_skills: List[str] = []
-    
     try:
         job_skills = json.loads(application.job.skills) if application.job and application.job.skills else []
     except Exception:
@@ -97,7 +95,6 @@ def generate_llm_interview_report(application) -> str:
     lines.append('')
     lines.append(f'Candidate Name: {application.name}')
     lines.append(f'Candidate Email: {application.email}')
-    
     if application.job:
         lines.append(f'Job Applied: {application.job.role}')
         lines.append(f'Company: {application.job.company}')
@@ -119,7 +116,7 @@ def generate_llm_interview_report(application) -> str:
     formatted_job_skills = ', '.join(job_skills) if job_skills else 'N/A'
     formatted_candidate_skills = ', '.join(candidate_skills) if candidate_skills else 'N/A'
     formatted_common = ', '.join(sorted(common)) if common else 'None'
-    
+
     lines.append(f'  Required Skills: {formatted_job_skills}')
     lines.append(f'  Candidate Skills: {formatted_candidate_skills}')
     lines.append(f'  Skills in Common: {formatted_common}')
@@ -172,19 +169,25 @@ def create_pdf_report(report_text: str) -> BytesIO:
     # Margins in inches
     LEFT_MARGIN = 0.75
     RIGHT_MARGIN = 0.75
-    TOP_MARGIN = 1.0
-    BOTTOM_MARGIN = 1.0
+    TOP_MARGIN = 0.75
+    BOTTOM_MARGIN = 0.75
     
     # Calculate content area
     CONTENT_WIDTH = A4_WIDTH - LEFT_MARGIN - RIGHT_MARGIN
     CONTENT_HEIGHT = A4_HEIGHT - TOP_MARGIN - BOTTOM_MARGIN
     
+    # Professional color scheme - single accent color
+    ACCENT_COLOR = '#1e40af'  # Dark blue
+    TEXT_COLOR = '#111827'    # Dark gray/black
+    LIGHT_GRAY = '#f8fafc'    # Light background
+    BORDER_COLOR = '#e2e8f0'  # Light border
+    
     # Parse report data
     report_data = _parse_report_text(report_text)
     
     # Create PDF
     with PdfPages(buffer) as pdf:
-        # Page 1: Header and Summary
+        # Page 1: Header, Candidate Info, and Skills Summary
         fig = plt.figure(figsize=(A4_WIDTH, A4_HEIGHT))
         fig.patch.set_facecolor('white')
         
@@ -197,199 +200,181 @@ def create_pdf_report(report_text: str) -> BytesIO:
         # Current Y position (start from top)
         y_pos = A4_HEIGHT - TOP_MARGIN
         
-        # Professional Header with gradient-like effect
-        header_rect = FancyBboxPatch(
-            (LEFT_MARGIN, y_pos - 1.8), CONTENT_WIDTH, 1.8,
-            boxstyle="round,pad=0.05",
-            facecolor='#1e40af',
-            edgecolor='none'
-        )
-        ax.add_patch(header_rect)
-        
-        # Company Logo placeholder (professional circle)
-        logo_circle = Circle((LEFT_MARGIN + 0.6, y_pos - 0.9), 0.4, 
-                           color='white', alpha=0.15, linewidth=2, 
-                           edgecolor='white')
-        ax.add_patch(logo_circle)
-        
-        # Logo text
-        ax.text(LEFT_MARGIN + 0.6, y_pos - 0.9, 'IR', 
-                fontsize=16, fontweight='bold', color='white',
-                horizontalalignment='center', verticalalignment='center')
-        
-        # Header text
-        ax.text(LEFT_MARGIN + 1.4, y_pos - 0.5, 'INTERVIEW ASSESSMENT REPORT',
-                fontsize=18, fontweight='bold', color='white',
-                verticalalignment='center')
-        
-        ax.text(LEFT_MARGIN + 1.4, y_pos - 0.9, 
-                f"{report_data['job_role']} • {report_data['company']}",
-                fontsize=11, color='white', alpha=0.9,
-                verticalalignment='center')
+        # === HEADER SECTION ===
+        # Clean header with company info
+        ax.text(LEFT_MARGIN, y_pos, 'INTERVIEW ASSESSMENT REPORT', 
+                fontsize=20, fontweight='bold', color=ACCENT_COLOR, fontfamily='sans-serif')
         
         # Date
         current_date = datetime.now().strftime('%B %d, %Y')
-        ax.text(A4_WIDTH - RIGHT_MARGIN, y_pos - 1.4, f'Generated: {current_date}',
-                fontsize=9, color='white', alpha=0.8,
+        ax.text(A4_WIDTH - RIGHT_MARGIN, y_pos, current_date,
+                fontsize=10, color=TEXT_COLOR, fontfamily='sans-serif',
                 horizontalalignment='right')
         
-        y_pos -= 2.2
+        y_pos -= 0.8
         
-        # Overall Score Section (Very Prominent)
+        # === CANDIDATE INFO AND OVERALL SCORE ===
+        # Large overall score box (prominent)
         overall_score = _calculate_overall_score(report_data)
         score_color = _get_score_color(overall_score['label'])
         
-        # Large score display box
-        score_box = FancyBboxPatch(
-            (LEFT_MARGIN, y_pos - 1.5), CONTENT_WIDTH, 1.5,
-            boxstyle="round,pad=0.08",
-            facecolor=score_color,
-            alpha=0.1,
-            edgecolor=score_color,
-            linewidth=3
+        # Score box on the right
+        score_box_width = 2.5
+        score_box_height = 1.8
+        score_x = A4_WIDTH - RIGHT_MARGIN - score_box_width
+        
+        # Score background
+        score_rect = FancyBboxPatch(
+            (score_x, y_pos - score_box_height), score_box_width, score_box_height,
+            boxstyle="round,pad=0.1",
+            facecolor=LIGHT_GRAY,
+            edgecolor=ACCENT_COLOR,
+            linewidth=2
         )
-        ax.add_patch(score_box)
+        ax.add_patch(score_rect)
         
-        # Score circle indicator
-        score_circle = Circle((LEFT_MARGIN + 0.8, y_pos - 0.75), 0.5,
-                            facecolor=score_color, alpha=0.2,
-                            edgecolor=score_color, linewidth=3)
-        ax.add_patch(score_circle)
-        
-        # Score percentage in circle
-        ax.text(LEFT_MARGIN + 0.8, y_pos - 0.75, f"{overall_score['percentage']:.0f}%",
-                fontsize=20, fontweight='bold', color=score_color,
+        # Large score percentage
+        ax.text(score_x + score_box_width/2, y_pos - 0.6, f"{overall_score['percentage']:.0f}%",
+                fontsize=32, fontweight='bold', color=ACCENT_COLOR, fontfamily='sans-serif',
                 horizontalalignment='center', verticalalignment='center')
         
         # Score label
-        ax.text(LEFT_MARGIN + 1.8, y_pos - 0.4, 'OVERALL ASSESSMENT',
-                fontsize=10, color='#6b7280', fontweight='bold')
+        ax.text(score_x + score_box_width/2, y_pos - 1.2, 'OVERALL SCORE',
+                fontsize=10, color=TEXT_COLOR, fontfamily='sans-serif',
+                horizontalalignment='center', fontweight='bold')
         
-        ax.text(LEFT_MARGIN + 1.8, y_pos - 0.8, overall_score['label'].upper(),
-                fontsize=24, fontweight='bold', color=score_color)
+        ax.text(score_x + score_box_width/2, y_pos - 1.5, overall_score['label'].upper(),
+                fontsize=14, fontweight='bold', color=score_color, fontfamily='sans-serif',
+                horizontalalignment='center')
         
-        # Score description
-        score_desc = _get_score_description(overall_score['label'])
-        ax.text(LEFT_MARGIN + 1.8, y_pos - 1.1, score_desc,
-                fontsize=10, color='#6b7280', style='italic')
+        # Candidate information on the left
+        info_width = CONTENT_WIDTH - score_box_width - 0.5
         
-        y_pos -= 2.0
+        # Candidate name (large)
+        ax.text(LEFT_MARGIN, y_pos - 0.3, report_data['candidate_name'],
+                fontsize=18, fontweight='bold', color=TEXT_COLOR, fontfamily='sans-serif')
         
-        # Candidate Information Section
-        _add_section_header(ax, LEFT_MARGIN, y_pos, 'CANDIDATE PROFILE', CONTENT_WIDTH)
-        y_pos -= 0.5
+        # Position and company
+        ax.text(LEFT_MARGIN, y_pos - 0.7, f"{report_data['job_role']} • {report_data['company']}",
+                fontsize=12, color=TEXT_COLOR, fontfamily='sans-serif')
         
-        # Professional info grid with better spacing
-        info_items = [
-            ('Full Name', report_data['candidate_name']),
-            ('Email Address', report_data['candidate_email']),
-            ('Applied Position', report_data['job_role']),
-            ('Company', report_data['company']),
-            ('Application Date', report_data['date_applied'])
-        ]
-        
-        # Create info boxes
-        for i, (label, value) in enumerate(info_items):
-            y_offset = y_pos - (i * 0.35)
-            
-            # Label
-            ax.text(LEFT_MARGIN + 0.1, y_offset, f"{label}:",
-                    fontsize=9, color='#6b7280', fontweight='bold')
-            
-            # Value with background
-            value_rect = FancyBboxPatch(
-                (LEFT_MARGIN + 2.2, y_offset - 0.1), CONTENT_WIDTH - 2.3, 0.25,
-                boxstyle="round,pad=0.02",
-                facecolor='#f8fafc',
-                edgecolor='#e2e8f0',
-                linewidth=0.5
-            )
-            ax.add_patch(value_rect)
-            
-            ax.text(LEFT_MARGIN + 2.3, y_offset, value,
-                    fontsize=10, color='#111827', fontweight='bold')
+        # Email and date
+        ax.text(LEFT_MARGIN, y_pos - 1.0, f"Email: {report_data['candidate_email']}",
+                fontsize=10, color=TEXT_COLOR, fontfamily='sans-serif')
+        
+        ax.text(LEFT_MARGIN, y_pos - 1.3, f"Application Date: {report_data['date_applied']}",
+                fontsize=10, color=TEXT_COLOR, fontfamily='sans-serif')
         
-        y_pos -= 2.2
+        y_pos -= 2.5
+        
+        # === SKILLS MATCH SUMMARY ===
+        # Section header
+        ax.text(LEFT_MARGIN, y_pos, 'SKILLS MATCH SUMMARY',
+                fontsize=14, fontweight='bold', color=ACCENT_COLOR, fontfamily='sans-serif')
+        
+        # Underline
+        ax.plot([LEFT_MARGIN, LEFT_MARGIN + 3], [y_pos - 0.1, y_pos - 0.1], 
+                color=ACCENT_COLOR, linewidth=2)
         
-        # Skills Analysis Section
-        _add_section_header(ax, LEFT_MARGIN, y_pos, 'SKILLS ANALYSIS', CONTENT_WIDTH)
         y_pos -= 0.5
         
         skills_data = report_data['skills_match']
         
-        # Skills match visualization with better design
-        bar_x = LEFT_MARGIN + 0.1
-        bar_y = y_pos - 0.4
-        bar_width = CONTENT_WIDTH - 0.2
-        bar_height = 0.4
-        
-        # Background bar with rounded corners
-        bg_bar = FancyBboxPatch(
-            (bar_x, bar_y), bar_width, bar_height,
-            boxstyle="round,pad=0.02",
-            facecolor='#f1f5f9',
-            edgecolor='#cbd5e1',
-            linewidth=1
-        )
-        ax.add_patch(bg_bar)
+        # Skills match percentage bar
+        bar_width = CONTENT_WIDTH - 1
+        bar_height = 0.3
         
-        # Progress bar
-        progress_width = max(0.3, bar_width * (skills_data['ratio'] / 100))  # Minimum visible width
-        progress_color = _get_score_color(skills_data['score'])
-        progress_bar = FancyBboxPatch(
-            (bar_x, bar_y), progress_width, bar_height,
-            boxstyle="round,pad=0.02",
-            facecolor=progress_color,
-            edgecolor='none'
-        )
-        ax.add_patch(progress_bar)
+        # Background bar
+        bg_rect = Rectangle((LEFT_MARGIN + 0.5, y_pos - bar_height), bar_width, bar_height,
+                           facecolor=LIGHT_GRAY, edgecolor=BORDER_COLOR)
+        ax.add_patch(bg_rect)
         
-        # Percentage text with better positioning
-        ax.text(bar_x + bar_width/2, bar_y + bar_height/2,
-                f"{skills_data['ratio']:.0f}% SKILLS MATCH",
-                fontsize=12, fontweight='bold', color='white',
+        # Progress bar
+        progress_width = bar_width * (skills_data['ratio'] / 100)
+        progress_rect = Rectangle((LEFT_MARGIN + 0.5, y_pos - bar_height), progress_width, bar_height,
+                                 facecolor=ACCENT_COLOR, edgecolor='none')
+        ax.add_patch(progress_rect)
+        
+        # Percentage text
+        ax.text(LEFT_MARGIN + 0.5 + bar_width/2, y_pos - bar_height/2, 
+                f"{skills_data['ratio']:.0f}% Skills Match",
+                fontsize=11, fontweight='bold', color='white', fontfamily='sans-serif',
                 horizontalalignment='center', verticalalignment='center')
         
-        y_pos -= 0.9
+        y_pos -= 0.8
+        
+        # Skills details (simplified)
+        ax.text(LEFT_MARGIN, y_pos, f"Required Skills: {skills_data['required'][:80]}{'...' if len(skills_data['required']) > 80 else ''}",
+                fontsize=10, color=TEXT_COLOR, fontfamily='sans-serif')
+        
+        y_pos -= 0.3
+        
+        ax.text(LEFT_MARGIN, y_pos, f"Candidate Skills: {skills_data['candidate'][:80]}{'...' if len(skills_data['candidate']) > 80 else ''}",
+                fontsize=10, color=TEXT_COLOR, fontfamily='sans-serif')
+        
+        y_pos -= 0.3
+        
+        ax.text(LEFT_MARGIN, y_pos, f"Matching Skills: {skills_data['common'][:80]}{'...' if len(skills_data['common']) > 80 else ''}",
+                fontsize=10, color=TEXT_COLOR, fontfamily='sans-serif')
         
-        # Skills details with better formatting
-        skills_items = [
-            ('Required Skills', skills_data['required'], '#dc2626'),
-            ('Candidate Skills', skills_data['candidate'], '#2563eb'),
-            ('Matching Skills', skills_data['common'], '#059669')
-        ]
+        y_pos -= 0.8
         
-        for i, (label, value, color) in enumerate(skills_items):
-            y_offset = y_pos - (i * 0.6)
+        # === INTERVIEW TRANSCRIPT PREVIEW ===
+        if report_data['qa_log']:
+            ax.text(LEFT_MARGIN, y_pos, 'INTERVIEW TRANSCRIPT',
+                    fontsize=14, fontweight='bold', color=ACCENT_COLOR, fontfamily='sans-serif')
             
-            # Color indicator
-            indicator = Circle((LEFT_MARGIN + 0.15, y_offset), 0.08,
-                             facecolor=color, edgecolor='none')
-            ax.add_patch(indicator)
+            # Underline
+            ax.plot([LEFT_MARGIN, LEFT_MARGIN + 3], [y_pos - 0.1, y_pos - 0.1], 
+                    color=ACCENT_COLOR, linewidth=2)
             
-            # Label
-            ax.text(LEFT_MARGIN + 0.35, y_offset, f"{label}:",
-                    fontsize=10, color='#374151', fontweight='bold')
+            y_pos -= 0.5
             
-            # Wrap and display skills
-            if value and value != 'N/A':
-                wrapped_value = textwrap.fill(value, width=65)
-                lines = wrapped_value.split('\n')
-                for j, line in enumerate(lines[:2]):  # Max 2 lines
-                    ax.text(LEFT_MARGIN + 0.35, y_offset - 0.25 - (j * 0.2), line,
-                            fontsize=9, color='#6b7280')
-            else:
-                ax.text(LEFT_MARGIN + 0.35, y_offset - 0.25, 'None specified',
-                        fontsize=9, color='#9ca3af', style='italic')
+            # Show first 2-3 Q&As on first page
+            max_qa_on_page1 = min(3, len(report_data['qa_log']))
+            
+            for i in range(max_qa_on_page1):
+                qa = report_data['qa_log'][i]
+                
+                # Check if we have space
+                if y_pos < BOTTOM_MARGIN + 2:
+                    break
+                
+                # Question
+                ax.text(LEFT_MARGIN, y_pos, f"Q{i+1}: {qa['question'][:90]}{'...' if len(qa['question']) > 90 else ''}",
+                        fontsize=11, fontweight='bold', color=ACCENT_COLOR, fontfamily='sans-serif')
+                
+                y_pos -= 0.4
+                
+                # Answer
+                answer_text = qa['answer']
+                if "salary" in qa['question'].lower() and (answer_text == "0$" or answer_text == "0" or answer_text == "$0"):
+                    answer_text = "Prefer not to disclose"
+                
+                wrapped_answer = textwrap.fill(answer_text, width=85)
+                answer_lines = wrapped_answer.split('\n')[:2]  # Max 2 lines
+                
+                for line in answer_lines:
+                    ax.text(LEFT_MARGIN + 0.3, y_pos, line,
+                            fontsize=10, color=TEXT_COLOR, fontfamily='sans-serif')
+                    y_pos -= 0.25
+                
+                # Evaluation
+                eval_color = _get_score_color(qa['score'])
+                ax.text(LEFT_MARGIN + 0.3, y_pos, f"Evaluation: {qa['score']}",
+                        fontsize=10, fontweight='bold', color=eval_color, fontfamily='sans-serif')
+                
+                y_pos -= 0.6
         
         # Save first page
         pdf.savefig(fig, bbox_inches='tight', pad_inches=0)
         plt.close(fig)
         
-        # Page 2+: Interview Transcript
-        if report_data['qa_log']:
-            _create_transcript_pages(pdf, report_data['qa_log'], A4_WIDTH, A4_HEIGHT,
-                                   LEFT_MARGIN, RIGHT_MARGIN, TOP_MARGIN, BOTTOM_MARGIN)
+        # === PAGE 2: REMAINING TRANSCRIPT ===
+        if report_data['qa_log'] and len(report_data['qa_log']) > 3:
+            _create_transcript_page(pdf, report_data['qa_log'][3:], A4_WIDTH, A4_HEIGHT,
+                                  LEFT_MARGIN, RIGHT_MARGIN, TOP_MARGIN, BOTTOM_MARGIN,
+                                  ACCENT_COLOR, TEXT_COLOR, start_index=4)
     
     buffer.seek(0)
     return buffer
@@ -418,7 +403,6 @@ def _parse_report_text(report_text: str) -> Dict[str, Any]:
     
     for line in lines:
         line = line.strip()
-        
         if line.startswith('Candidate Name:'):
             data['candidate_name'] = line.split(':', 1)[1].strip()
         elif line.startswith('Candidate Email:'):
@@ -515,54 +499,12 @@ def _get_score_color(score_label: str) -> str:
         return '#dc2626'  # Red
 
 
-def _get_score_description(score_label: str) -> str:
-    """Get description based on score label."""
-    descriptions = {
-        'Excellent': 'Outstanding candidate with strong alignment',
-        'Good': 'Solid candidate with good potential',
-        'Satisfactory': 'Adequate candidate with room for growth',
-        'Needs Improvement': 'Candidate requires significant development'
-    }
-    return descriptions.get(score_label, 'Assessment completed')
-
-
-def _add_section_header(ax, x: float, y: float, title: str, width: float):
-    """Add a professional section header with separator line."""
-    # Background for header
-    header_bg = FancyBboxPatch(
-        (x - 0.05, y - 0.05), width + 0.1, 0.35,
-        boxstyle="round,pad=0.02",
-        facecolor='#f8fafc',
-        edgecolor='#e2e8f0',
-        linewidth=1
-    )
-    ax.add_patch(header_bg)
-    
-    # Title
-    ax.text(x + 0.1, y + 0.1, title, fontsize=12, fontweight='bold', color='#1e293b')
-    
-    # Decorative line
-    line = plt.Line2D([x + 0.1, x + width - 0.1], [y - 0.15, y - 0.15],
-                      color='#3b82f6', linewidth=2)
-    ax.add_line(line)
-
-
-def _calculate_dynamic_box_height(text: str, width_chars: int = 75, base_height: float = 0.8) -> float:
-    """Calculate dynamic height for text box based on content."""
-    wrapped_lines = textwrap.wrap(text, width=width_chars)
-    num_lines = len(wrapped_lines)
-    # Base height + additional height per line
-    return base_height + (max(0, num_lines - 2) * 0.15)
-
-
-def _create_transcript_pages(pdf, qa_log: List[Dict], page_width: float, page_height: float,
-                           left_margin: float, right_margin: float,
-                           top_margin: float, bottom_margin: float):
-    """Create professional pages for interview transcript with dynamic text wrapping."""
+def _create_transcript_page(pdf, qa_log: List[Dict], page_width: float, page_height: float,
+                          left_margin: float, right_margin: float, top_margin: float, bottom_margin: float,
+                          accent_color: str, text_color: str, start_index: int = 1):
+    """Create a clean page for remaining interview transcript."""
     content_width = page_width - left_margin - right_margin
-    wrapper = textwrap.TextWrapper(width=75)
     
-    # Create pages dynamically based on content
     fig = plt.figure(figsize=(page_width, page_height))
     fig.patch.set_facecolor('white')
     ax = fig.add_subplot(111)
@@ -570,311 +512,80 @@ def _create_transcript_pages(pdf, qa_log: List[Dict], page_width: float, page_he
     ax.set_ylim(0, page_height)
     ax.axis('off')
     
-    # Initialize page
+    # Start from top
     y_pos = page_height - top_margin
-    page_num = 0
     
-    # Header for first transcript page
-    _add_transcript_header(ax, left_margin, y_pos, content_width, page_num + 2)
-    y_pos -= 1.0
+    # Page header
+    ax.text(left_margin, y_pos, 'INTERVIEW TRANSCRIPT (CONTINUED)',
+            fontsize=14, fontweight='bold', color=accent_color, fontfamily='sans-serif')
     
-    for idx, qa in enumerate(qa_log):
-        # Calculate space needed for this Q&A
-        q_height = _calculate_dynamic_box_height(qa['question'])
-        a_height = _calculate_dynamic_box_height(qa['answer'], width_chars=70)
-        total_height = q_height + a_height + 2.5  # Include spacing
-        
-        # Check if we need a new page
-        if y_pos - total_height < bottom_margin + 1.0:
-            # Save current page
-            pdf.savefig(fig, bbox_inches='tight', pad_inches=0)
-            plt.close(fig)
-            
-            # Create new page
-            fig = plt.figure(figsize=(page_width, page_height))
-            fig.patch.set_facecolor('white')
-            ax = fig.add_subplot(111)
-            ax.set_xlim(0, page_width)
-            ax.set_ylim(0, page_height)
-            ax.axis('off')
-            
-            y_pos = page_height - top_margin
-            page_num += 1
-            _add_transcript_header(ax, left_margin, y_pos, content_width, page_num + 2)
-            y_pos -= 1.0
-        
-        # Question section with dynamic height
-        q_box = FancyBboxPatch(
-            (left_margin, y_pos - q_height), content_width, q_height,
-            boxstyle="round,pad=0.05",
-            facecolor='#eff6ff',
-            edgecolor='#3b82f6',
-            linewidth=2
-        )
-        ax.add_patch(q_box)
-        
-        # Question number badge
-        q_badge = Circle((left_margin + 0.4, y_pos - q_height/2), 0.2,
-                       facecolor='#3b82f6', edgecolor='white', linewidth=2)
-        ax.add_patch(q_badge)
-        
-        ax.text(left_margin + 0.4, y_pos - q_height/2, f'{idx+1}',
-                fontsize=12, fontweight='bold', color='white',
-                horizontalalignment='center', verticalalignment='center')
-        
-        # Question text with proper wrapping
-        ax.text(left_margin + 0.8, y_pos - 0.2, 'QUESTION',
-                fontsize=9, fontweight='bold', color='#1e40af')
-        
-        q_wrapped = wrapper.wrap(qa['question'])
-        for j, line in enumerate(q_wrapped):
-            ax.text(left_margin + 0.8, y_pos - 0.4 - (j * 0.15), line,
-                    fontsize=11, fontweight='bold', color='#1e293b')
+    # Underline
+    ax.plot([left_margin, left_margin + 4], [y_pos - 0.1, y_pos - 0.1], 
+            color=accent_color, linewidth=2)
+    
+    y_pos -= 0.8
+    
+    # Process remaining Q&As
+    for i, qa in enumerate(qa_log):
+        # Check if we have space for this Q&A
+        if y_pos < bottom_margin + 1.5:
+            break
+        
+        # Question
+        question_text = f"Q{start_index + i}: {qa['question']}"
+        wrapped_question = textwrap.fill(question_text, width=85)
+        question_lines = wrapped_question.split('\n')
+        
+        for line in question_lines:
+            ax.text(left_margin, y_pos, line,
+                    fontsize=11, fontweight='bold', color=accent_color, fontfamily='sans-serif')
+            y_pos -= 0.3
         
-        y_pos -= (q_height + 0.3)
+        y_pos -= 0.1
         
-        # Answer section with dynamic height
+        # Answer
         answer_text = qa['answer']
-        # Handle salary question replacement
         if "salary" in qa['question'].lower() and (answer_text == "0$" or answer_text == "0" or answer_text == "$0"):
             answer_text = "Prefer not to disclose"
-            
-        answer_box = FancyBboxPatch(
-            (left_margin + 0.2, y_pos - a_height), content_width - 0.4, a_height,
-            boxstyle="round,pad=0.05",
-            facecolor='#f9fafb',
-            edgecolor='#d1d5db',
-            linewidth=1
-        )
-        ax.add_patch(answer_box)
-        
-        ax.text(left_margin + 0.4, y_pos - 0.2, 'CANDIDATE RESPONSE',
-                fontsize=9, fontweight='bold', color='#6b7280')
-        
-        a_wrapped = wrapper.wrap(answer_text)
-        for j, line in enumerate(a_wrapped):
-            if j * 0.15 < a_height - 0.4:  # Ensure text fits in box
-                ax.text(left_margin + 0.4, y_pos - 0.4 - (j * 0.15), line,
-                        fontsize=10, color='#374151')
-        
-        y_pos -= (a_height + 0.4)
-        
-        # Evaluation section
-        eval_box = FancyBboxPatch(
-            (left_margin + 0.4, y_pos - 0.8), content_width - 0.8, 0.8,
-            boxstyle="round,pad=0.05",
-            facecolor='#fefefe',
-            edgecolor='#e5e7eb',
-            linewidth=1
-        )
-        ax.add_patch(eval_box)
-        
-        # Score badge
-        score_color = _get_score_color(qa['score'])
-        score_badge = FancyBboxPatch(
-            (left_margin + 0.6, y_pos - 0.35), 1.2, 0.25,
-            boxstyle="round,pad=0.02",
-            facecolor=score_color,
-            alpha=0.2,
-            edgecolor=score_color,
-            linewidth=1
-        )
-        ax.add_patch(score_badge)
         
-        ax.text(left_margin + 1.2, y_pos - 0.225, qa['score'],
-                fontsize=10, fontweight='bold', color=score_color,
-                horizontalalignment='center', verticalalignment='center')
+        wrapped_answer = textwrap.fill(answer_text, width=80)
+        answer_lines = wrapped_answer.split('\n')
         
-        # Feedback with proper wrapping
-        if qa['feedback'] and qa['feedback'] != 'N/A':
-            ax.text(left_margin + 2.2, y_pos - 0.15, 'Feedback:',
-                    fontsize=9, fontweight='bold', color='#6b7280')
-            
-            # Calculate feedback area width
-            feedback_width = content_width - 2.6
-            feedback_wrapper = textwrap.TextWrapper(width=int(feedback_width * 10))
-            f_wrapped = feedback_wrapper.wrap(qa['feedback'])
+        for line in answer_lines[:3]:  # Max 3 lines per answer
+            ax.text(left_margin + 0.3, y_pos, line,
+                    fontsize=10, color=text_color, fontfamily='sans-serif')
+            y_pos -= 0.25
+        
+        # Evaluation
+        eval_color = _get_score_color(qa['score'])
+        ax.text(left_margin + 0.3, y_pos, f"Evaluation: {qa['score']}",
+                fontsize=10, fontweight='bold', color=eval_color, fontfamily='sans-serif')
+        
+        y_pos -= 0.2
+        
+        # Feedback (if available and space permits)
+        if qa['feedback'] and qa['feedback'] != 'N/A' and y_pos > bottom_margin + 0.8:
+            feedback_text = f"Feedback: {qa['feedback']}"
+            wrapped_feedback = textwrap.fill(feedback_text, width=75)
+            feedback_lines = wrapped_feedback.split('\n')[:2]  # Max 2 lines
             
-            for j, line in enumerate(f_wrapped[:2]):  # Max 2 lines
-                ax.text(left_margin + 2.2, y_pos - 0.35 - (j * 0.15), line,
-                        fontsize=9, color='#6b7280', style='italic')
-        
-        y_pos -= 1.2
-        
-        # Add separator between questions (except last)
-        if idx < len(qa_log) - 1:
-            separator = plt.Line2D([left_margin + 1, left_margin + content_width - 1],
-                                 [y_pos + 0.3, y_pos + 0.3],
-                                 color='#e5e7eb', linewidth=1, linestyle='--')
-            ax.add_line(separator)
+            for line in feedback_lines:
+                ax.text(left_margin + 0.3, y_pos, line,
+                        fontsize=9, color='#6b7280', fontfamily='sans-serif', style='italic')
+                y_pos -= 0.2
+        
+        y_pos -= 0.4
+        
+        # Add separator line between questions
+        if i < len(qa_log) - 1 and y_pos > bottom_margin + 1:
+            ax.plot([left_margin + 0.5, left_margin + content_width - 0.5], 
+                   [y_pos + 0.1, y_pos + 0.1], 
+                   color='#e5e7eb', linewidth=0.5, linestyle='--')
             y_pos -= 0.3
     
-    # Save last page
+    # Save page
     pdf.savefig(fig, bbox_inches='tight', pad_inches=0)
     plt.close(fig)
 
 
-def _add_transcript_header(ax, left_margin: float, y_pos: float, content_width: float, page_num: int):
-    """Add header for transcript pages."""
-    # Header background
-    header_rect = FancyBboxPatch(
-        (left_margin, y_pos - 0.6), content_width, 0.6,
-        boxstyle="round,pad=0.02",
-        facecolor='#1e40af',
-        edgecolor='none'
-    )
-    ax.add_patch(header_rect)
-    
-    ax.text(left_margin + 0.2, y_pos - 0.3, 'INTERVIEW TRANSCRIPT',
-            fontsize=14, fontweight='bold', color='white')
-    
-    # Page number
-    ax.text(left_margin + content_width - 0.2, y_pos - 0.3, f'Page {page_num}',
-            fontsize=10, color='white', horizontalalignment='right')
-
-
-# Keep the original advanced version as fallback
-def create_pdf_report_advanced(report_text: str) -> BytesIO:
-    """
-    Alternative implementation using reportlab for professional PDF generation.
-    This creates cleaner, more consistent PDFs with better text handling.
-    Install with: pip install reportlab
-    """
-    try:
-        from reportlab.lib.pagesizes import A4
-        from reportlab.lib.styles import getSampleStyleSheet, ParagraphStyle
-        from reportlab.lib.units import inch
-        from reportlab.platypus import SimpleDocTemplate, Paragraph, Spacer, PageBreak
-        from reportlab.lib.colors import HexColor
-        from reportlab.lib.enums import TA_LEFT, TA_CENTER
-
-        buffer = BytesIO()
-        
-        # Create document with consistent margins
-        doc = SimpleDocTemplate(
-            buffer,
-            pagesize=A4,
-            rightMargin=0.75*inch,
-            leftMargin=0.75*inch,
-            topMargin=1*inch,
-            bottomMargin=1*inch
-        )
-        
-        # Define consistent styles
-        styles = getSampleStyleSheet()
-        
-        # Title style
-        title_style = ParagraphStyle(
-            'CustomTitle',
-            parent=styles['Heading1'],
-            fontSize=16,
-            textColor=HexColor('#111827'),
-            spaceAfter=12,
-            alignment=TA_CENTER
-        )
-        
-        # Header style for metadata
-        header_style = ParagraphStyle(
-            'Header',
-            parent=styles['Normal'],
-            fontSize=10,
-            textColor=HexColor('#111827'),
-            spaceAfter=4
-        )
-        
-        # Question style
-        question_style = ParagraphStyle(
-            'Question',
-            parent=styles['Heading2'],
-            fontSize=11,
-            textColor=HexColor('#1e3a8a'),
-            spaceBefore=12,
-            spaceAfter=6,
-            fontName='Helvetica-Bold'
-        )
-        
-        # Answer style
-        answer_style = ParagraphStyle(
-            'Answer',
-            parent=styles['Normal'],
-            fontSize=10,
-            textColor=HexColor('#374151'),
-            leftIndent=20,
-            spaceAfter=4
-        )
-        
-        # Score style
-        score_style = ParagraphStyle(
-            'Score',
-            parent=styles['Normal'],
-            fontSize=10,
-            textColor=HexColor('#059669'),
-            leftIndent=20,
-            fontName='Helvetica-Bold',
-            spaceAfter=4
-        )
-        
-        # Feedback style
-        feedback_style = ParagraphStyle(
-            'Feedback',
-            parent=styles['Normal'],
-            fontSize=10,
-            textColor=HexColor('#dc2626'),
-            leftIndent=20,
-            spaceAfter=8
-        )
-        
-        # Build document content
-        story = []
-        lines = report_text.split('\n')
-        
-        for line in lines:
-            stripped = line.strip()
-            if not stripped:
-                story.append(Spacer(1, 6))
-            elif 'Interview Report' in stripped:
-                story.append(Paragraph(stripped, title_style))
-                story.append(Spacer(1, 12))
-            elif any(stripped.startswith(x) for x in ['Candidate Name:', 'Candidate Email:', 'Job Applied:', 'Company:', 'Date Applied:']):
-                story.append(Paragraph(stripped, header_style))
-            elif stripped.startswith('Skills Match Summary:') or stripped.startswith('Interview Transcript'):
-                story.append(Spacer(1, 12))
-                story.append(Paragraph(f"<b>{stripped}</b>", header_style))
-                story.append(Spacer(1, 6))
-            elif stripped.startswith('Question'):
-                story.append(Paragraph(stripped, question_style))
-            elif stripped.startswith('Answer:'):
-                # Handle salary replacement
-                if "0$" in stripped or " 0 " in stripped or "$0" in stripped:
-                    stripped = stripped.replace("0$", "Prefer not to disclose").replace(" 0 ", " Prefer not to disclose ").replace("$0", "Prefer not to disclose")
-                story.append(Paragraph(stripped, answer_style))
-            elif stripped.startswith('Score:'):
-                story.append(Paragraph(stripped, score_style))
-            elif stripped.startswith('Feedback:'):
-                story.append(Paragraph(stripped, feedback_style))
-            else:
-                # Regular text with proper indentation for sub-items
-                if stripped.startswith('  '):
-                    indent_style = ParagraphStyle(
-                        'Indented',
-                        parent=styles['Normal'],
-                        fontSize=10,
-                        leftIndent=20,
-                        spaceAfter=2
-                    )
-                    story.append(Paragraph(stripped, indent_style))
-                else:
-                    story.append(Paragraph(stripped, styles['Normal']))
-        
-        # Build PDF
-        doc.build(story)
-        buffer.seek(0)
-        return buffer
-        
-    except ImportError:
-        # Fallback to matplotlib version if reportlab not available
-        print("Reportlab not installed. Using matplotlib version.")
-        return create_pdf_report(report_text)
-
-
-__all__ = ['generate_llm_interview_report', 'create_pdf_report']
+__all__ = ['generate_llm_interview_report', 'create_pdf_report']