Update process_interview.py

process_interview.py

@@ -592,13 +592,11 @@ def calculate_acceptance_probability(analysis_data: Dict) -> float:
 
 
 
-
-
 def generate_report(analysis_data: Dict) -> str:
     try:
         voice = analysis_data.get('voice_analysis', {})
         voice_interpretation = generate_voice_interpretation(voice)
-        interviewee_responses = [f"- {u['text']}" for u in analysis_data['transcript'] if u['role'] == 'Interviewee'][:5]
+        interviewee_responses = [f"- {u['text']}" for u in analysis_data['transcript'] if u.get('role') == 'Interviewee'][:5]
         acceptance_prob = analysis_data.get('acceptance_probability', 50.0)
         acceptance_line = f"\n**Suitability Score: {acceptance_prob:.2f}%**\n"
         if acceptance_prob >= 80:
@@ -609,71 +607,181 @@ def generate_report(analysis_data: Dict) -> str:
             acceptance_line += "HR Verdict: Moderate potential, needs additional assessment."
         else:
             acceptance_line += "HR Verdict: Limited fit, significant improvement required."
+        
+        # Enhanced prompt with detailed recommendations
         prompt = f"""
-        You are EvalBot, a senior HR consultant delivering a concise, professional interview analysis report. Use clear headings, bullet points ('-'), and avoid redundancy. Ensure text is clean and free of special characters that could break formatting.
+        You are EvalBot, a senior HR consultant delivering a comprehensive, professional interview analysis report. 
+        Use clear headings, bullet points ('-'), and avoid redundancy. 
+        Ensure text is clean, professional, and free of special characters that could break formatting.
+        The interview involves two roles: Interviewer and Interviewee, assigned alternately to utterances.
         {acceptance_line}
         **1. Executive Summary**
-        - Summarize performance, key metrics, and hiring potential.
-        - Duration: {analysis_data['text_analysis']['total_duration']:.2f} seconds
-        - Speaker Turns: {analysis_data['text_analysis']['speaker_turns']}
-        - Participants: {', '.join(sorted(set(u['speaker'] for u in analysis_data['transcript'])))}
+        - Summarize the candidate's overall performance, key strengths, and hiring potential in 2-3 bullets.
+        - Include metrics: Duration ({analysis_data['text_analysis']['total_duration']:.2f} seconds), Speaker Turns ({analysis_data['text_analysis']['speaker_turns']}).
+        - Mention roles: Interviewer and Interviewee.
         **2. Communication and Vocal Dynamics**
-        - Evaluate vocal delivery (rate, fluency, confidence).
-        - Provide HR insights on workplace alignment.
-        {voice_interpretation}
+        - Evaluate the Interviewee's vocal delivery (speaking rate, fluency, confidence, anxiety) in 2-3 bullets.
+        - Provide one HR insight on workplace communication alignment.
+        - Voice Analysis: {voice_interpretation}
         **3. Competency and Content**
-        - Assess leadership, problem-solving, communication, adaptability.
-        - List strengths and growth areas separately with examples.
-        - Sample responses:
+        - List 2-3 key strengths (e.g., leadership, problem-solving, communication) with specific examples from responses.
+        - List 1-2 weaknesses (growth areas) with actionable feedback to address them.
+        - Sample Interviewee responses (for context):
         {chr(10).join(interviewee_responses)}
         **4. Role Fit and Potential**
-        - Analyze cultural fit, role readiness, and growth potential.
+        - Analyze the Interviewee's cultural fit, role readiness, and long-term growth potential in 2-3 bullets.
         **5. Recommendations**
-        - Provide prioritized strategies for growth (communication, technical skills, presence).
-        - Suggest next steps for hiring managers (advance, train, assess).
+        - Provide 3-4 specific suggestions for improvement focusing on:
+          - Communication skills (e.g., clarity, fluency, reducing filler words).
+          - Content delivery (e.g., structuring responses, emphasizing key points).
+          - Professional presentation (e.g., confidence, tone, engagement).
+        - Include practical strategies or examples for each suggestion.
+        - Suggest 2-3 specific next steps for hiring managers (e.g., advance to technical round, provide training, conduct behavioral assessment).
         """
         response = gemini_model.generate_content(prompt)
-        return re.sub(r'[^\x00-\x7F]+', '', response.text)  # Sanitize non-ASCII characters
+        # Robust sanitization
+        clean_text = re.sub(r'[^\x20-\x7E\n]+', '', response.text)
+        return clean_text
     except Exception as e:
         logger.error(f"Report generation failed: {str(e)}")
         return f"Error generating report: {str(e)}"
 
 def create_pdf_report(analysis_data: Dict, output_path: str, gemini_report_text: str) -> bool:
     try:
-        doc = SimpleDocTemplate(output_path, pagesize=letter,
-                                rightMargin=0.75*inch, leftMargin=0.75*inch,
-                                topMargin=1*inch, bottomMargin=1*inch)
+        doc = SimpleDocTemplate(
+            output_path,
+            pagesize=letter,
+            rightMargin=0.75*inch,
+            leftMargin=0.75*inch,
+            topMargin=1*inch,
+            bottomMargin=1*inch
+        )
         styles = getSampleStyleSheet()
-        h1 = ParagraphStyle(name='Heading1', fontSize=20, leading=24, spaceAfter=18, alignment=1, textColor=colors.HexColor('#003087'), fontName='Helvetica-Bold')
-        h2 = ParagraphStyle(name='Heading2', fontSize=14, leading=16, spaceBefore=12, spaceAfter=8, textColor=colors.HexColor('#0050BC'), fontName='Helvetica-Bold')
-        h3 = ParagraphStyle(name='Heading3', fontSize=10, leading=12, spaceBefore=8, spaceAfter=6, textColor=colors.HexColor('#3F7CFF'), fontName='Helvetica')
-        body_text = ParagraphStyle(name='BodyText', fontSize=9, leading=12, spaceAfter=6, fontName='Helvetica', textColor=colors.HexColor('#333333'))
-        bullet_style = ParagraphStyle(name='Bullet', parent=body_text, leftIndent=18, bulletIndent=8, fontName='Helvetica', bulletFontName='Helvetica', bulletFontSize=9)
+        
+        # Custom styles
+        cover_title = ParagraphStyle(
+            name='CoverTitle',
+            fontSize=24,
+            leading=28,
+            spaceAfter=20,
+            alignment=1,
+            textColor=colors.HexColor('#003087'),
+            fontName='Helvetica-Bold'
+        )
+        h1 = ParagraphStyle(
+            name='Heading1',
+            fontSize=16,
+            leading=20,
+            spaceAfter=14,
+            alignment=1,
+            textColor=colors.HexColor('#003087'),
+            fontName='Helvetica-Bold'
+        )
+        h2 = ParagraphStyle(
+            name='Heading2',
+            fontSize=12,
+            leading=15,
+            spaceBefore=10,
+            spaceAfter=8,
+            textColor=colors.HexColor('#0050BC'),
+            fontName='Helvetica-Bold'
+        )
+        h3 = ParagraphStyle(
+            name='Heading3',
+            fontSize=10,
+            leading=12,
+            spaceBefore=8,
+            spaceAfter=6,
+            textColor=colors.HexColor('#3F7CFF'),
+            fontName='Helvetica-Bold'
+        )
+        body_text = ParagraphStyle(
+            name='BodyText',
+            fontSize=9,
+            leading=12,
+            spaceAfter=6,
+            fontName='Helvetica',
+            textColor=colors.HexColor('#333333')
+        )
+        bullet_style = ParagraphStyle(
+            name='Bullet',
+            parent=body_text,
+            leftIndent=18,
+            bulletIndent=8,
+            fontName='Helvetica',
+            bulletFontName='Helvetica',
+            bulletFontSize=9
+        )
+        table_header = ParagraphStyle(
+            name='TableHeader',
+            fontSize=9,
+            leading=11,
+            textColor=colors.white,
+            fontName='Helvetica-Bold'
+        )
+        table_body = ParagraphStyle(
+            name='TableBody',
+            fontSize=9,
+            leading=11,
+            fontName='Helvetica'
+        )
         
         story = []
 
         def header_footer(canvas, doc):
             canvas.saveState()
+            # Footer
             canvas.setFont('Helvetica', 8)
             canvas.setFillColor(colors.HexColor('#666666'))
             canvas.drawString(doc.leftMargin, 0.5*inch, f"Page {doc.page} | EvalBot HR Interview Report | Confidential")
+            canvas.drawRightString(doc.width + doc.leftMargin, 0.5*inch, time.strftime('%B %d, %Y'))
+            # Header
             canvas.setStrokeColor(colors.HexColor('#0050BC'))
             canvas.setLineWidth(0.8)
             canvas.line(doc.leftMargin, doc.height + 0.9*inch, doc.width + doc.leftMargin, doc.height + 0.9*inch)
             canvas.setFont('Helvetica-Bold', 9)
             canvas.drawString(doc.leftMargin, doc.height + 0.95*inch, "Candidate Interview Analysis")
-            canvas.drawRightString(doc.width + doc.leftMargin, doc.height + 0.95*inch, time.strftime('%B %d, %Y'))
             canvas.restoreState()
 
+        # Cover Page
+        story.append(Spacer(1, 2*inch))
+        logo_path = 'logo.png'
+        if os.path.exists(logo_path):
+            story.append(Image(logo_path, width=2*inch, height=0.75*inch))
+            story.append(Spacer(1, 0.3*inch))
+        story.append(Paragraph("Candidate Interview Analysis Report", cover_title))
+        story.append(Spacer(1, 0.2*inch))
+        story.append(Paragraph(f"Candidate ID: {analysis_data.get('user_id', 'N/A')}", body_text))
+        story.append(Paragraph(f"Generated: {time.strftime('%B %d, %Y')}", body_text))
+        story.append(Spacer(1, 0.5*inch))
+        story.append(Paragraph("Confidential", ParagraphStyle(
+            name='Confidential',
+            fontSize=10,
+            alignment=1,
+            textColor=colors.HexColor('#D32F2F'),
+            fontName='Helvetica-Bold'
+        )))
+        story.append(PageBreak())
+
         # Title Page
-        story.append(Paragraph("Candidate Interview Analysis", h1))
-        story.append(Paragraph(f"Generated: {time.strftime('%B %d, %Y')}", ParagraphStyle(name='Date', alignment=1, fontSize=9, textColor=colors.HexColor('#666666'), fontName='Helvetica')))
-        story.append(Spacer(1, 0.4*inch))
+        story.append(Paragraph("Interview Evaluation Summary", h1))
+        story.append(Spacer(1, 0.3*inch))
+        # Suitability Score
         acceptance_prob = analysis_data.get('acceptance_probability', 50.0)
-        story.append(Paragraph("Hiring Suitability Snapshot", h2))
-        prob_color = colors.HexColor('#2E7D32') if acceptance_prob >= 80 else (colors.HexColor('#F57C00') if acceptance_prob >= 60 else colors.HexColor('#D32F2F'))
-        story.append(Paragraph(f"Suitability Score: <font size=15 color='{prob_color.hexval()}'><b>{acceptance_prob:.2f}%</b></font>",
-                             ParagraphStyle(name='Prob', fontSize=11, spaceAfter=10, alignment=1, fontName='Helvetica-Bold')))
+        prob_color = colors.HexColor('#2E7D32') if acceptance_prob >= 80 else (
+            colors.HexColor('#F57C00') if acceptance_prob >= 60 else colors.HexColor('#D32F2F')
+        )
+        story.append(Paragraph(
+            f"Suitability Score: <font size=14 color='{prob_color.hexval()}'><b>{acceptance_prob:.2f}%</b></font>",
+            ParagraphStyle(
+                name='Score',
+                fontSize=14,
+                spaceAfter=12,
+                alignment=1,
+                fontName='Helvetica-Bold'
+            )
+        ))
+        # HR Verdict
         if acceptance_prob >= 80:
             story.append(Paragraph("<b>HR Verdict:</b> Outstanding candidate, recommended for immediate advancement.", body_text))
         elif acceptance_prob >= 60:
@@ -682,12 +790,15 @@ def create_pdf_report(analysis_data: Dict, output_path: str, gemini_report_text:
             story.append(Paragraph("<b>HR Verdict:</b> Moderate potential, needs additional assessment.", body_text))
         else:
             story.append(Paragraph("<b>HR Verdict:</b> Limited fit, significant improvement required.", body_text))
-        story.append(Spacer(1, 0.3*inch))
+        story.append(Spacer(1, 0.2*inch))
+        
+        # Summary Table with Roles
+        roles = sorted(set(u.get('role', 'Unknown') for u in analysis_data.get('transcript', [])))
         table_data = [
-            ['Metric', 'Value'],
-            ['Interview Duration', f"{analysis_data['text_analysis']['total_duration']:.2f} seconds"],
-            ['Speaker Turns', f"{analysis_data['text_analysis']['speaker_turns']}"],
-            ['Participants', ', '.join(sorted(set(u['speaker'] for u in analysis_data['transcript'])))],
+            [Paragraph('Metric', table_header), Paragraph('Value', table_header)],
+            [Paragraph('Interview Duration', table_body), Paragraph(f"{analysis_data['text_analysis'].get('total_duration', 0):.2f} seconds", table_body)],
+            [Paragraph('Speaker Turns', table_body), Paragraph(f"{analysis_data['text_analysis'].get('speaker_turns', 0)}", table_body)],
+            [Paragraph('Roles', table_body), Paragraph(', '.join(roles), table_body)],
         ]
         table = Table(table_data, colWidths=[2.3*inch, 3.7*inch])
         table.setStyle(TableStyle([
@@ -700,10 +811,10 @@ def create_pdf_report(analysis_data: Dict, output_path: str, gemini_report_text:
             ('BOTTOMPADDING', (0,0), (-1,0), 8),
             ('TOPPADDING', (0,0), (-1,0), 8),
             ('BACKGROUND', (0,1), (-1,-1), colors.HexColor('#F5F6FA')),
-            ('GRID', (0,0), (-1,-1), 0.5, colors.HexColor('#DDE4EB')),
+            ('GRID', (0,0), (-1,-1), 0.5, colors.HexColor('#DDE4EE')),
         ]))
         story.append(table)
-        story.append(Spacer(1, 0.4*inch))
+        story.append(Spacer(1, 0.3*inch))
         story.append(Paragraph("Prepared by: EvalBot - AI-Powered HR Analysis", body_text))
         story.append(PageBreak())
 
@@ -715,12 +826,12 @@ def create_pdf_report(analysis_data: Dict, output_path: str, gemini_report_text:
         voice_analysis = analysis_data.get('voice_analysis', {})
         if voice_analysis and 'error' not in voice_analysis:
             table_data = [
-                ['Metric', 'Value', 'HR Insight'],
-                ['Speaking Rate', f"{voice_analysis.get('speaking_rate', 0):.2f} words/sec", 'Benchmark: 2.0-3.0 wps; impacts clarity'],
-                ['Filler Words', f"{voice_analysis.get('filler_ratio', 0) * 100:.1f}%", 'High usage reduces credibility'],
-                ['Anxiety', voice_analysis.get('interpretation', {}).get('anxiety_level', 'N/A'), f"Score: {voice_analysis.get('composite_scores', {}).get('anxiety', 0):.3f}"],
-                ['Confidence', voice_analysis.get('interpretation', {}).get('confidence_level', 'N/A'), f"Score: {voice_analysis.get('composite_scores', {}).get('confidence', 0):.3f}"],
-                ['Fluency', voice_analysis.get('interpretation', {}).get('fluency_level', 'N/A'), 'Drives engagement'],
+                [Paragraph('Metric', table_header), Paragraph('Value', table_header), Paragraph('HR Insight', table_header)],
+                [Paragraph('Speaking Rate', table_body), Paragraph(f"{voice_analysis.get('speaking_rate', 0):.2f} words/sec", table_body), Paragraph('Benchmark: 2.0-3.0 wps; impacts clarity', table_body)],
+                [Paragraph('Filler Words', table_body), Paragraph(f"{voice_analysis.get('filler_ratio', 0) * 100:.1f}%", table_body), Paragraph('High usage may reduce credibility', table_body)],
+                [Paragraph('Anxiety', table_body), Paragraph(voice_analysis.get('interpretation', {}).get('anxiety_level', 'N/A').title(), table_body), Paragraph(f"Score: {voice_analysis.get('composite_scores', {}).get('anxiety', 0):.3f}", table_body)],
+                [Paragraph('Confidence', table_body), Paragraph(voice_analysis.get('interpretation', {}).get('confidence_level', 'N/A').title(), table_body), Paragraph(f"Score: {voice_analysis.get('composite_scores', {}).get('confidence', 0):.3f}", table_body)],
+                [Paragraph('Fluency', table_body), Paragraph(voice_analysis.get('interpretation', {}).get('fluency_level', 'N/A').title(), table_body), Paragraph('Drives engagement', table_body)],
             ]
             table = Table(table_data, colWidths=[1.6*inch, 1.2*inch, 3.2*inch])
             table.setStyle(TableStyle([
@@ -730,10 +841,10 @@ def create_pdf_report(analysis_data: Dict, output_path: str, gemini_report_text:
                 ('VALIGN', (0,0), (-1,-1), 'MIDDLE'),
                 ('FONTNAME', (0,0), (-1,0), 'Helvetica-Bold'),
                 ('FONTSIZE', (0,0), (-1,-1), 9),
-                ('BOTTOMPADDING', (0,0), (-1,0), 8),
-                ('TOPPADDING', (0,0), (-1,0), 8),
+                ('BOTTOMPADDING', (0,0), (-1,-1), 8),
+                ('TOPPADDING', (0,0), (-1,-1), 8),
                 ('BACKGROUND', (0,1), (-1,-1), colors.HexColor('#F5F6FA')),
-                ('GRID', (0,0), (-1,-1), 0.5, colors.HexColor('#DDE4EB')),
+                ('GRID', (0,0), (-1,-1), 0.5, colors.HexColor('#DDE4EE')),
             ]))
             story.append(table)
             story.append(Spacer(1, 0.2*inch))
@@ -744,16 +855,16 @@ def create_pdf_report(analysis_data: Dict, output_path: str, gemini_report_text:
             img.hAlign = 'CENTER'
             story.append(img)
         else:
-            story.append(Paragraph("Vocal analysis unavailable.", body_text))
+            story.append(Paragraph(f"Vocal analysis unavailable: {voice_analysis.get('error', 'No data available')}", body_text))
         story.append(Spacer(1, 0.2*inch))
 
         # Parse Gemini Report
         sections = {
             "Executive Summary": [],
             "Communication": [],
-            "Competency": {"Strengths": [], "Growth Areas": []},
-            "Recommendations": {"Development": [], "Next Steps": []},
+            "Competency": {"Strengths": [], "Weaknesses": []},
             "Role Fit": [],
+            "Recommendations": {"Development": [], "Next Steps": []},
         }
         current_section = None
         current_subsection = None
@@ -761,11 +872,9 @@ def create_pdf_report(analysis_data: Dict, output_path: str, gemini_report_text:
         for line in lines:
             line = line.strip()
             if not line: continue
-            # Simplified regex to avoid parenthesis issues
-            if line.startswith('**') and line.endswith('**'):
-                section_title = line.strip('**').strip()
-                if section_title.startswith(('1.', '2.', '3.', '4.', '5.')):
-                    section_title = section_title[2:].strip()
+            heading_match = re.match(r'^\**(\d+\.\s+)?([^\*]+)\**$', line)
+            if heading_match:
+                section_title = heading_match.group(2).strip()
                 if 'Executive Summary' in section_title:
                     current_section = 'Executive Summary'
                     current_subsection = None
@@ -781,27 +890,51 @@ def create_pdf_report(analysis_data: Dict, output_path: str, gemini_report_text:
                 elif 'Recommendations' in section_title:
                     current_section = 'Recommendations'
                     current_subsection = None
-            elif line.startswith(('-', '*', '•')) and current_section:
-                clean_line = line.lstrip('-*• ').strip()
+            elif re.match(r'^[-*•]\s+', line) and current_section:
+                clean_line = re.sub(r'^[-*•]\s+', '', line).strip()
                 if not clean_line: continue
-                clean_line = re.sub(r'[()]', '', clean_line)  # Remove parentheses
+                clean_line = re.sub(r'[()\[\]{}]', '', clean_line)
                 if current_section == 'Competency':
-                    if any(k in clean_line.lower() for k in ['leader', 'problem', 'commun', 'adapt', 'strength']):
+                    if any(k in clean_line.lower() for k in ['leader', 'problem', 'commun', 'adapt', 'strength', 'effective', 'skill']):
                         current_subsection = 'Strengths'
-                    elif any(k in clean_line.lower() for k in ['improv', 'grow', 'depth']):
-                        current_subsection = 'Growth Areas'
+                    elif any(k in clean_line.lower() for k in ['improv', 'grow', 'weak', 'depth', 'challenge']):
+                        current_subsection = 'Weaknesses'
                     if current_subsection:
                         sections[current_section][current_subsection].append(clean_line)
                 elif current_section == 'Recommendations':
-                    if any(k in clean_line.lower() for k in ['commun', 'tech', 'depth', 'pres']):
+                    if any(k in clean_line.lower() for k in ['commun', 'tech', 'depth', 'pres', 'improve', 'enhance', 'clarity', 'structur', 'tone']):
                         current_subsection = 'Development'
-                    elif any(k in clean_line.lower() for k in ['adv', 'train', 'assess', 'next', 'mentor']):
+                    elif any(k in clean_line.lower() for k in ['adv', 'train', 'assess', 'next', 'mentor', 'round']):
                         current_subsection = 'Next Steps'
                     if current_subsection:
                         sections[current_section][current_subsection].append(clean_line)
                 else:
                     sections[current_section].append(clean_line)
 
+        # Summary Box
+        story.append(Paragraph("Key Highlights", h2))
+        summary_data = [
+            [Paragraph("Category", table_header), Paragraph("Detail", table_header)],
+            [Paragraph("Top Strength", table_body), Paragraph(sections['Competency']['Strengths'][0] if sections['Competency']['Strengths'] else "N/A", table_body)],
+            [Paragraph("Key Weakness", table_body), Paragraph(sections['Competency']['Weaknesses'][0] if sections['Competency']['Weaknesses'] else "N/A", table_body)],
+            [Paragraph("Top Recommendation", table_body), Paragraph(sections['Recommendations']['Development'][0] if sections['Recommendations']['Development'] else "N/A", table_body)],
+        ]
+        summary_table = Table(summary_data, colWidths=[2*inch, 4*inch])
+        summary_table.setStyle(TableStyle([
+            ('BACKGROUND', (0,0), (-1,0), colors.HexColor('#0050BC')),
+            ('TEXTCOLOR', (0,0), (-1,0), colors.white),
+            ('ALIGN', (0,0), (-1,-1), 'LEFT'),
+            ('VALIGN', (0,0), (-1,-1), 'MIDDLE'),
+            ('FONTNAME', (0,0), (-1,0), 'Helvetica-Bold'),
+            ('FONTSIZE', (0,0), (-1,-1), 9),
+            ('BOTTOMPADDING', (0,0), (-1,-1), 6),
+            ('TOPPADDING', (0,0), (-1,-1), 6),
+            ('BACKGROUND', (0,1), (-1,-1), colors.HexColor('#E8F0FE')),
+            ('GRID', (0,0), (-1,-1), 0.5, colors.HexColor('#DDE4EE')),
+        ]))
+        story.append(summary_table)
+        story.append(Spacer(1, 0.3*inch))
+
         # Executive Summary
         story.append(Paragraph("2. Executive Summary", h2))
         if sections['Executive Summary']:
@@ -815,17 +948,27 @@ def create_pdf_report(analysis_data: Dict, output_path: str, gemini_report_text:
         story.append(Paragraph("3. Competency & Evaluation", h2))
         story.append(Paragraph("Strengths", h3))
         if sections['Competency']['Strengths']:
-            for line in sections['Competency']['Strengths']:
-                story.append(Paragraph(line, bullet_style))
+            strength_table = Table([[Paragraph(line, bullet_style)] for line in sections['Competency']['Strengths']], colWidths=[6*inch])
+            strength_table.setStyle(TableStyle([
+                ('BACKGROUND', (0,0), (-1,-1), colors.HexColor('#E6FFE6')),
+                ('VALIGN', (0,0), (-1,-1), 'TOP'),
+                ('LEFTPADDING', (0,0), (-1,-1), 6),
+            ]))
+            story.append(strength_table)
         else:
             story.append(Paragraph("No strengths identified.", body_text))
         story.append(Spacer(1, 0.1*inch))
-        story.append(Paragraph("Growth Areas", h3))
-        if sections['Competency']['Growth Areas']:
-            for line in sections['Competency']['Growth Areas']:
-                story.append(Paragraph(line, bullet_style))
+        story.append(Paragraph("Weaknesses", h3))
+        if sections['Competency']['Weaknesses']:
+            weakness_table = Table([[Paragraph(line, bullet_style)] for line in sections['Competency']['Weaknesses']], colWidths=[6*inch])
+            weakness_table.setStyle(TableStyle([
+                ('BACKGROUND', (0,0), (-1,-1), colors.HexColor('#FFF0F0')),
+                ('VALIGN', (0,0), (-1,-1), 'TOP'),
+                ('LEFTPADDING', (0,0), (-1,-1), 6),
+            ]))
+            story.append(weakness_table)
         else:
-            story.append(Paragraph("No growth areas identified; maintain current strengths.", body_text))
+            story.append(Paragraph("No weaknesses identified; maintain current strengths.", body_text))
         story.append(Spacer(1, 0.2*inch))
 
         # Role Fit
@@ -841,8 +984,13 @@ def create_pdf_report(analysis_data: Dict, output_path: str, gemini_report_text:
         story.append(Paragraph("5. Recommendations", h2))
         story.append(Paragraph("Development Priorities", h3))
         if sections['Recommendations']['Development']:
-            for line in sections['Recommendations']['Development']:
-                story.append(Paragraph(line, bullet_style))
+            dev_table = Table([[Paragraph(line, bullet_style)] for line in sections['Recommendations']['Development']], colWidths=[6*inch])
+            dev_table.setStyle(TableStyle([
+                ('BACKGROUND', (0,0), (-1,-1), colors.HexColor('#E8F0FE')),
+                ('VALIGN', (0,0), (-1,-1), 'TOP'),
+                ('LEFTPADDING', (0,0), (-1,-1), 6),
+            ]))
+            story.append(dev_table)
         else:
             story.append(Paragraph("No development priorities specified.", body_text))
         story.append(Spacer(1, 0.1*inch))
@@ -862,6 +1010,7 @@ def create_pdf_report(analysis_data: Dict, output_path: str, gemini_report_text:
         return False
 
 
+
 def convert_to_serializable(obj):
     if isinstance(obj, np.generic):
         return obj.item()