updated

backend/services/report_generator.py

@@ -21,33 +21,34 @@ This module provides two primary helpers used by the recruiter dashboard:
 """
 
 from __future__ import annotations
-
 import json
 from io import BytesIO
 import textwrap
 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
+from matplotlib.patches import Rectangle, FancyBboxPatch, Circle
+import matplotlib.dates as mdates
+from datetime import datetime
+
 
 def generate_llm_interview_report(application) -> str:
     """Generate a human‑readable interview report for a candidate.
-
+    
     The report includes the candidate's name and email, job details,
     application date, a computed skills match summary and placeholder
     sections for future enhancements.  If server‑side storage of
     question/answer pairs is added later, this function can be updated
     to incorporate those details.
-
+    
     Parameters
     ----------
     application : backend.models.database.Application
         The SQLAlchemy Application instance representing the candidate's
         job application.  Assumed to have related ``job`` and
         ``date_applied`` attributes available.
-
+    
     Returns
     -------
     str
@@ -64,9 +65,10 @@ def generate_llm_interview_report(application) -> str:
         candidate_features = json.loads(application.extracted_features) if application.extracted_features else {}
     except Exception:
         candidate_features = {}
-    candidate_skills: List[str] = candidate_features.get('skills', []) or []
 
+    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:
@@ -95,12 +97,14 @@ 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}')
     else:
         lines.append('Job Applied: N/A')
         lines.append('Company: N/A')
+
     # Format date_applied if available
     try:
         date_str = application.date_applied.strftime('%Y-%m-%d') if application.date_applied else 'N/A'
@@ -108,18 +112,21 @@ def generate_llm_interview_report(application) -> str:
         date_str = 'N/A'
     lines.append(f'Date Applied: {date_str}')
     lines.append('')
+
     lines.append('Skills Match Summary:')
     # Represent required and candidate skills as comma‑separated lists.  Use
     # title‑case for presentation and handle empty lists gracefully.
     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}')
     lines.append(f'  Match Ratio: {ratio * 100:.0f}%')
     lines.append(f'  Score: {score_label}')
     lines.append('')
+
     lines.append('Interview Transcript & Evaluation:')
     try:
         if application.interview_log:
@@ -127,14 +134,14 @@ def generate_llm_interview_report(application) -> str:
                 qa_log = json.loads(application.interview_log)
             except Exception:
                 qa_log = []
-
+            
             if qa_log:
                 for idx, entry in enumerate(qa_log, 1):
                     q = entry.get("question", "N/A")
                     a = entry.get("answer", "N/A")
                     eval_score = entry.get("evaluation", {}).get("score", "N/A")
                     eval_feedback = entry.get("evaluation", {}).get("feedback", "N/A")
-
+                    
                     lines.append(f"\nQuestion {idx}: {q}")
                     lines.append(f"Answer: {a}")
                     lines.append(f"Score: {eval_score}")
@@ -185,137 +192,190 @@ def create_pdf_report(report_text: str) -> BytesIO:
         # Current Y position (start from top)
         y_pos = A4_HEIGHT - TOP_MARGIN
         
-        # Header background
+        # Professional Header with gradient-like effect
         header_rect = FancyBboxPatch(
-            (LEFT_MARGIN, y_pos - 1.5), CONTENT_WIDTH, 1.5,
-            boxstyle="round,pad=0.02",
+            (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 (circle)
-        logo_circle = plt.Circle((LEFT_MARGIN + 0.5, y_pos - 0.75), 0.35, 
-                                color='white', alpha=0.3)
+        # 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.2, y_pos - 0.5, 'INTERVIEW REPORT',
-                fontsize=20, fontweight='bold', color='white',
+        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.2, y_pos - 1.0, 
+        ax.text(LEFT_MARGIN + 1.4, y_pos - 0.9, 
                 f"{report_data['job_role']} • {report_data['company']}",
-                fontsize=12, color='white', alpha=0.9,
+                fontsize=11, color='white', alpha=0.9,
                 verticalalignment='center')
         
-        y_pos -= 2.0
+        # 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,
+                horizontalalignment='right')
+        
+        y_pos -= 2.2
         
-        # Overall Score Section (Prominent)
+        # Overall Score Section (Very Prominent)
         overall_score = _calculate_overall_score(report_data)
         score_color = _get_score_color(overall_score['label'])
         
-        # Score box
+        # Large score display box
         score_box = FancyBboxPatch(
-            (LEFT_MARGIN, y_pos - 1.2), CONTENT_WIDTH, 1.2,
-            boxstyle="round,pad=0.05",
+            (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=2
+            linewidth=3
         )
         ax.add_patch(score_box)
         
-        # Score text
-        ax.text(LEFT_MARGIN + CONTENT_WIDTH/2, y_pos - 0.3,
-                'OVERALL ASSESSMENT',
-                fontsize=10, color='#6b7280',
-                horizontalalignment='center')
+        # 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)
         
-        ax.text(LEFT_MARGIN + CONTENT_WIDTH/2, y_pos - 0.7,
-                overall_score['label'].upper(),
-                fontsize=28, fontweight='bold', color=score_color,
-                horizontalalignment='center')
+        # 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,
+                horizontalalignment='center', verticalalignment='center')
         
-        ax.text(LEFT_MARGIN + CONTENT_WIDTH/2, y_pos - 1.0,
-                f"{overall_score['percentage']}%",
-                fontsize=16, color=score_color,
-                horizontalalignment='center')
+        # Score label
+        ax.text(LEFT_MARGIN + 1.8, y_pos - 0.4, 'OVERALL ASSESSMENT',
+                fontsize=10, color='#6b7280', fontweight='bold')
         
-        y_pos -= 1.7
+        ax.text(LEFT_MARGIN + 1.8, y_pos - 0.8, overall_score['label'].upper(),
+                fontsize=24, fontweight='bold', color=score_color)
+        
+        # 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')
+        
+        y_pos -= 2.0
         
         # Candidate Information Section
-        _add_section_header(ax, LEFT_MARGIN, y_pos, 'Candidate Information', CONTENT_WIDTH)
-        y_pos -= 0.4
+        _add_section_header(ax, LEFT_MARGIN, y_pos, 'CANDIDATE PROFILE', CONTENT_WIDTH)
+        y_pos -= 0.5
         
-        # Info grid
+        # Professional info grid with better spacing
         info_items = [
-            ('Name', report_data['candidate_name']),
-            ('Email', report_data['candidate_email']),
-            ('Position', report_data['job_role']),
+            ('Full Name', report_data['candidate_name']),
+            ('Email Address', report_data['candidate_email']),
+            ('Applied Position', report_data['job_role']),
             ('Company', report_data['company']),
-            ('Date Applied', report_data['date_applied'])
+            ('Application Date', report_data['date_applied'])
         ]
         
+        # Create info boxes
         for i, (label, value) in enumerate(info_items):
-            y_offset = y_pos - (i * 0.3)
-            ax.text(LEFT_MARGIN + 0.2, y_offset, f"{label}:",
-                    fontsize=10, color='#6b7280')
-            ax.text(LEFT_MARGIN + 2.0, y_offset, value,
+            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')
         
-        y_pos -= 1.8
+        y_pos -= 2.2
         
-        # Skills Match Summary
-        _add_section_header(ax, LEFT_MARGIN, y_pos, 'Skills Analysis', CONTENT_WIDTH)
-        y_pos -= 0.4
+        # Skills Analysis Section
+        _add_section_header(ax, LEFT_MARGIN, y_pos, 'SKILLS ANALYSIS', CONTENT_WIDTH)
+        y_pos -= 0.5
         
-        # Skills match visualization
         skills_data = report_data['skills_match']
         
-        # Progress bar for match ratio
-        bar_x = LEFT_MARGIN + 0.2
-        bar_y = y_pos - 0.3
-        bar_width = CONTENT_WIDTH - 0.4
-        bar_height = 0.3
+        # 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
-        bg_bar = Rectangle((bar_x, bar_y), bar_width, bar_height,
-                          facecolor='#e5e7eb', edgecolor='none')
+        # 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)
         
         # Progress bar
-        progress_width = bar_width * (skills_data['ratio'] / 100)
+        progress_width = max(0.3, bar_width * (skills_data['ratio'] / 100))  # Minimum visible width
         progress_color = _get_score_color(skills_data['score'])
-        progress_bar = Rectangle((bar_x, bar_y), progress_width, bar_height,
-                                facecolor=progress_color, edgecolor='none')
+        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)
         
-        # Percentage text
+        # Percentage text with better positioning
         ax.text(bar_x + bar_width/2, bar_y + bar_height/2,
-                f"{skills_data['ratio']:.0f}% Match",
+                f"{skills_data['ratio']:.0f}% SKILLS MATCH",
                 fontsize=12, fontweight='bold', color='white',
                 horizontalalignment='center', verticalalignment='center')
         
-        y_pos -= 0.8
+        y_pos -= 0.9
         
-        # Skills details
+        # Skills details with better formatting
         skills_items = [
-            ('Required Skills', skills_data['required']),
-            ('Candidate Skills', skills_data['candidate']),
-            ('Matching Skills', skills_data['common'])
+            ('Required Skills', skills_data['required'], '#dc2626'),
+            ('Candidate Skills', skills_data['candidate'], '#2563eb'),
+            ('Matching Skills', skills_data['common'], '#059669')
         ]
         
-        for i, (label, value) in enumerate(skills_items):
-            y_offset = y_pos - (i * 0.4)
-            ax.text(LEFT_MARGIN + 0.2, y_offset, f"{label}:",
-                    fontsize=9, color='#6b7280')
-            # Wrap long skill lists
-            wrapped_value = textwrap.fill(value, width=60)
-            lines = wrapped_value.split('\n')
-            for j, line in enumerate(lines):
-                ax.text(LEFT_MARGIN + 0.2, y_offset - 0.2 - (j * 0.2), line,
-                        fontsize=9, color='#374151', style='italic')
+        for i, (label, value, color) in enumerate(skills_items):
+            y_offset = y_pos - (i * 0.6)
+            
+            # Color indicator
+            indicator = Circle((LEFT_MARGIN + 0.15, y_offset), 0.08,
+                             facecolor=color, edgecolor='none')
+            ax.add_patch(indicator)
+            
+            # Label
+            ax.text(LEFT_MARGIN + 0.35, y_offset, f"{label}:",
+                    fontsize=10, color='#374151', fontweight='bold')
+            
+            # 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')
         
         # Save first page
         pdf.savefig(fig, bbox_inches='tight', pad_inches=0)
@@ -353,6 +413,7 @@ 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:'):
@@ -407,13 +468,13 @@ def _calculate_overall_score(report_data: Dict[str, Any]) -> Dict[str, Any]:
     qa_scores = []
     for qa in report_data['qa_log']:
         score_text = qa['score'].lower()
-        if 'excellent' in score_text or '5' in score_text:
+        if 'excellent' in score_text or '5' in score_text or '10' in score_text:
             qa_scores.append(1.0)
-        elif 'good' in score_text or '4' in score_text:
+        elif 'good' in score_text or '4' in score_text or '8' in score_text or '9' in score_text:
             qa_scores.append(0.8)
-        elif 'satisfactory' in score_text or '3' in score_text:
+        elif 'satisfactory' in score_text or 'medium' in score_text or '3' in score_text or '6' in score_text or '7' in score_text:
             qa_scores.append(0.6)
-        elif 'needs improvement' in score_text or '2' in score_text:
+        elif 'needs improvement' in score_text or 'poor' in score_text or '2' in score_text or '4' in score_text or '5' in score_text:
             qa_scores.append(0.4)
         else:
             qa_scores.append(0.2)
@@ -426,9 +487,9 @@ def _calculate_overall_score(report_data: Dict[str, Any]) -> Dict[str, Any]:
     
     if overall >= 0.8:
         label = 'Excellent'
-    elif overall >= 0.6:
+    elif overall >= 0.65:
         label = 'Good'
-    elif overall >= 0.4:
+    elif overall >= 0.45:
         label = 'Satisfactory'
     else:
         label = 'Needs Improvement'
@@ -442,35 +503,54 @@ def _get_score_color(score_label: str) -> str:
     if 'excellent' in score_label:
         return '#059669'  # Green
     elif 'good' in score_label:
-        return '#3b82f6'  # Blue
+        return '#2563eb'  # Blue
     elif 'medium' in score_label or 'satisfactory' in score_label:
-        return '#f59e0b'  # Amber
+        return '#d97706'  # Orange
     else:
-        return '#ef4444'  # Red
+        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 section header with separator line."""
+    """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, y, title.upper(), fontsize=12, fontweight='bold', 
-            color='#1e293b')
+    ax.text(x + 0.1, y + 0.1, title, fontsize=12, fontweight='bold', color='#1e293b')
     
-    # Separator line
-    line = plt.Line2D([x, x + width], [y - 0.15, y - 0.15],
-                      color='#e5e7eb', linewidth=1)
+    # 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 _create_transcript_pages(pdf, qa_log: List[Dict], 
-                           page_width: float, page_height: float,
+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 pages for interview transcript."""
+    """Create professional pages for interview transcript."""
     content_width = page_width - left_margin - right_margin
-    wrapper = textwrap.TextWrapper(width=80)
+    wrapper = textwrap.TextWrapper(width=75)
     
     # Group questions for pagination
-    questions_per_page = 3
+    questions_per_page = 2  # Reduced for better spacing
     total_pages = (len(qa_log) + questions_per_page - 1) // questions_per_page
     
     for page_num in range(total_pages):
@@ -483,14 +563,24 @@ def _create_transcript_pages(pdf, qa_log: List[Dict],
         
         # Page header
         y_pos = page_height - top_margin
-        ax.text(left_margin, y_pos, 'INTERVIEW TRANSCRIPT',
-                fontsize=14, fontweight='bold', color='#1e293b')
+        
+        # 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(page_width - right_margin, y_pos, f'Page {page_num + 2}',
-                fontsize=9, color='#9ca3af', horizontalalignment='right')
+        ax.text(page_width - right_margin - 0.2, y_pos - 0.3, f'Page {page_num + 2}',
+                fontsize=10, color='white', horizontalalignment='right')
         
-        y_pos -= 0.5
+        y_pos -= 1.0
         
         # Questions for this page
         start_idx = page_num * questions_per_page
@@ -499,45 +589,70 @@ def _create_transcript_pages(pdf, qa_log: List[Dict],
         for i in range(start_idx, end_idx):
             qa = qa_log[i]
             
-            # Question box
+            # Question section
             q_box = FancyBboxPatch(
-                (left_margin, y_pos - 0.8), content_width, 0.8,
-                boxstyle="round,pad=0.02",
+                (left_margin, y_pos - 1.0), content_width, 1.0,
+                boxstyle="round,pad=0.05",
                 facecolor='#eff6ff',
                 edgecolor='#3b82f6',
-                linewidth=1
+                linewidth=2
             )
             ax.add_patch(q_box)
             
-            # Question number and text
-            ax.text(left_margin + 0.1, y_pos - 0.2, f'Q{i+1}.',
-                    fontsize=14, fontweight='bold', color='#1e40af')
+            # Question number badge
+            q_badge = Circle((left_margin + 0.4, y_pos - 0.5), 0.2,
+                           facecolor='#3b82f6', edgecolor='white', linewidth=2)
+            ax.add_patch(q_badge)
+            
+            ax.text(left_margin + 0.4, y_pos - 0.5, f'{i+1}',
+                    fontsize=12, fontweight='bold', color='white',
+                    horizontalalignment='center', verticalalignment='center')
+            
+            # Question text
+            ax.text(left_margin + 0.8, y_pos - 0.3, 'QUESTION',
+                    fontsize=9, fontweight='bold', color='#1e40af')
             
-            # Wrap question text
             q_wrapped = wrapper.wrap(qa['question'])
-            for j, line in enumerate(q_wrapped[:2]):  # Max 2 lines
-                ax.text(left_margin + 0.5, y_pos - 0.2 - (j * 0.25), line,
+            for j, line in enumerate(q_wrapped[:3]):  # Max 3 lines
+                ax.text(left_margin + 0.8, y_pos - 0.5 - (j * 0.15), line,
                         fontsize=11, fontweight='bold', color='#1e293b')
             
-            y_pos -= 1.2
+            y_pos -= 1.4
             
-            # Answer
-            ax.text(left_margin + 0.2, y_pos, 'Answer:',
-                    fontsize=10, fontweight='bold', color='#6b7280')
+            # Answer section
+            answer_box = FancyBboxPatch(
+                (left_margin + 0.2, y_pos - 1.2), content_width - 0.4, 1.2,
+                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(qa['answer'])
-            for j, line in enumerate(a_wrapped[:3]):  # Max 3 lines
-                ax.text(left_margin + 0.2, y_pos - 0.3 - (j * 0.2), line,
+            for j, line in enumerate(a_wrapped[:4]):  # Max 4 lines
+                ax.text(left_margin + 0.4, y_pos - 0.4 - (j * 0.15), line,
                         fontsize=10, color='#374151')
             
-            y_pos -= 1.0
+            y_pos -= 1.6
             
-            # Score and Feedback row
-            score_color = _get_score_color(qa['score'])
+            # 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.2, y_pos - 0.3), 1.5, 0.3,
+                (left_margin + 0.6, y_pos - 0.35), 1.2, 0.25,
                 boxstyle="round,pad=0.02",
                 facecolor=score_color,
                 alpha=0.2,
@@ -546,28 +661,29 @@ def _create_transcript_pages(pdf, qa_log: List[Dict],
             )
             ax.add_patch(score_badge)
             
-            ax.text(left_margin + 0.95, y_pos - 0.15, qa['score'],
-                    fontsize=9, fontweight='bold', color=score_color,
+            ax.text(left_margin + 1.2, y_pos - 0.225, qa['score'],
+                    fontsize=10, fontweight='bold', color=score_color,
                     horizontalalignment='center', verticalalignment='center')
             
             # Feedback
             if qa['feedback'] and qa['feedback'] != 'N/A':
-                ax.text(left_margin + 2.0, y_pos, 'Feedback:',
-                        fontsize=9, style='italic', color='#6b7280')
+                ax.text(left_margin + 2.2, y_pos - 0.15, 'Feedback:',
+                        fontsize=9, fontweight='bold', color='#6b7280')
                 
                 f_wrapped = wrapper.wrap(qa['feedback'])
                 for j, line in enumerate(f_wrapped[:2]):  # Max 2 lines
-                    ax.text(left_margin + 2.0, y_pos - 0.25 - (j * 0.2), line,
-                            fontsize=9, style='italic', color='#6b7280')
+                    ax.text(left_margin + 2.2, y_pos - 0.35 - (j * 0.15), line,
+                            fontsize=9, color='#6b7280', style='italic')
             
-            y_pos -= 1.5
+            y_pos -= 1.2
             
             # Add separator between questions (except last)
             if i < end_idx - 1:
-                line = plt.Line2D([left_margin, left_margin + content_width], 
-                                [y_pos + 0.5, y_pos + 0.5],
-                                color='#e5e7eb', linewidth=0.5)
-                ax.add_line(line)
+                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)
+                y_pos -= 0.3
         
         pdf.savefig(fig, bbox_inches='tight', pad_inches=0)
         plt.close(fig)
@@ -587,7 +703,7 @@ def create_pdf_report_advanced(report_text: str) -> BytesIO:
         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
@@ -670,14 +786,12 @@ def create_pdf_report_advanced(report_text: str) -> BytesIO:
         
         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:']):
+            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))
@@ -693,7 +807,7 @@ def create_pdf_report_advanced(report_text: str) -> BytesIO:
                 story.append(Paragraph(stripped, feedback_style))
             else:
                 # Regular text with proper indentation for sub-items
-                if stripped.startswith(' '):
+                if stripped.startswith('  '):
                     indent_style = ParagraphStyle(
                         'Indented',
                         parent=styles['Normal'],
@@ -714,5 +828,6 @@ def create_pdf_report_advanced(report_text: str) -> BytesIO:
         # 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']