Update app.py

app.py

@@ -5,6 +5,10 @@ from reportlab.pdfgen import canvas
 import bleach
 from simple_salesforce import Salesforce, SalesforceError
 import os
+import logging
+
+# Set up logging for debugging
+logging.basicConfig(filename="app_errors.log", level=logging.ERROR, format="%(asctime)s - %(levelname)s - %(message)s")
 
 # Salesforce configuration (loaded from environment variables for security)
 SF_USERNAME = os.getenv("SF_USERNAME")
@@ -22,22 +26,23 @@ def init_salesforce():
             instance_url=SF_INSTANCE_URL
         )
         return sf, "Salesforce connected successfully"
-    except SalesforceError as e:
+    except Exception as e:
+        logging.error(f"Failed to initialize Salesforce connection: {str(e)}")
         return None, f"Salesforce connection failed: {str(e)}"
 
 # Create Salesforce record using Case object
 def create_salesforce_record(score, checklist_summary, missing_summary, status):
-    sf, connection_message = init_salesforce()
-    if not sf:
-        # Silently handle connection failure since UI feedback is removed
-        return
+    try:
+        sf, connection_message = init_salesforce()
+        if not sf:
+            logging.error(f"Skipping Salesforce record creation due to connection failure: {connection_message}")
+            return
 
-    # Map app status to Salesforce Case.Status values
-    sf_status = "New"  # Default for Pending and Escalated
-    if status == "Completed":
-        sf_status = "Closed"
+        # Map app status to Salesforce Case.Status values
+        sf_status = "New"  # Default for Pending and Escalated
+        if status == "Completed":
+            sf_status = "Closed"
 
-    try:
         # Create a Case record in Salesforce with required fields
         record = {
             "Subject": "Project Readiness Evaluation",
@@ -47,8 +52,8 @@ def create_salesforce_record(score, checklist_summary, missing_summary, status):
             "Priority": "Medium"  # Required field in many Salesforce orgs
         }
         sf.Case.create(record)
-    except SalesforceError:
-        # Silently handle errors since UI feedback is removed
+    except Exception as e:
+        logging.error(f"Failed to create Salesforce Case record: {str(e)}")
         pass
 
 # Clean input to prevent injection attacks
@@ -59,80 +64,88 @@ def sanitize_input(text):
 
 # Rule-based completeness engine to evaluate readiness and create Salesforce record
 def evaluate_readiness_and_save(logs, qa_report, punch_list_text):
-    # Evaluate readiness
-    score = 0  # Base score set to 0 for clearer range
-    missing_items = []
-    checklist_details = []
-
-    # Sanitize inputs
-    logs = sanitize_input(logs)
-    qa_report = sanitize_input(qa_report)
-    punch_list_text = sanitize_input(punch_list_text)
-
-    # Process Project Logs
-    log_keywords = r"complete|handover done|finished|closed|successful"
-    if logs and re.search(log_keywords, logs.lower()):
-        score += 30
-        checklist_details.append("Logs: Completed")
-    else:
-        missing_items.append("Project Logs Incomplete")
-        checklist_details.append("Logs: Pending")
-
-    # Process QA Report
-    qa_keywords = r"approved|passed|cleared"
-    if qa_report and re.search(qa_keywords, qa_report.lower()):
-        score += 40
-        checklist_details.append("QA Report: Approved")
-    else:
-        missing_items.append("QA Approval Missing")
-        checklist_details.append("QA Report: Pending")
-
-    # Process Punch List
-    punch_keywords = r"none|resolved|closed|no issues"
-    if punch_list_text and re.search(punch_keywords, punch_list_text.lower()):
-        score += 30
-        checklist_details.append("Punch List: Resolved")
-    else:
-        missing_items.append("Open Punch Points Detected")
-        checklist_details.append("Punch List: Pending")
-
-    # Handle critical issues (escalation)
-    escalated = any("incomplete" in item.lower() or "missing" in item.lower() for item in missing_items)
-    status = "Escalated" if escalated else ("Completed" if not missing_items else "Pending")
-
-    # Build summaries
-    checklist_summary = "\n".join(checklist_details)
-    missing_summary = "None" if not missing_items else ", ".join(missing_items)
-
-    # Generate progress bar HTML (alert indicator)
-    color = "red" if score < 70 else "yellow" if score < 90 else "green"
-    progress_bar = f'<progress value="{score}" max="100" style="width:100%;height:20px;background-color:{color}">{score}%</progress>'
-
-    # Automatically create Salesforce record
-    create_salesforce_record(score, checklist_summary, missing_summary, status)
-
-    return score, checklist_summary, missing_summary, status, progress_bar
+    try:
+        # Evaluate readiness
+        score = 0  # Base score set to 0 for clearer range
+        missing_items = []
+        checklist_details = []
+
+        # Sanitize inputs
+        logs = sanitize_input(logs)
+        qa_report = sanitize_input(qa_report)
+        punch_list_text = sanitize_input(punch_list_text)
+
+        # Process Project Logs
+        log_keywords = r"complete|handover done|finished|closed|successful"
+        if logs and re.search(log_keywords, logs.lower()):
+            score += 30
+            checklist_details.append("Logs: Completed")
+        else:
+            missing_items.append("Project Logs Incomplete")
+            checklist_details.append("Logs: Pending")
+
+        # Process QA Report
+        qa_keywords = r"approved|passed|cleared"
+        if qa_report and re.search(qa_keywords, qa_report.lower()):
+            score += 40
+            checklist_details.append("QA Report: Approved")
+        else:
+            missing_items.append("QA Approval Missing")
+            checklist_details.append("QA Report: Pending")
+
+        # Process Punch List
+        punch_keywords = r"none|resolved|closed|no issues"
+        if punch_list_text and re.search(punch_keywords, punch_list_text.lower()):
+            score += 30
+            checklist_details.append("Punch List: Resolved")
+        else:
+            missing_items.append("Open Punch Points Detected")
+            checklist_details.append("Punch List: Pending")
+
+        # Handle critical issues (escalation)
+        escalated = any("incomplete" in item.lower() or "missing" in item.lower() for item in missing_items)
+        status = "Escalated" if escalated else ("Completed" if not missing_items else "Pending")
+
+        # Build summaries
+        checklist_summary = "\n".join(checklist_details)
+        missing_summary = "None" if not missing_items else ", ".join(missing_items)
+
+        # Generate progress bar HTML (alert indicator)
+        color = "red" if score < 70 else "yellow" if score < 90 else "green"
+        progress_bar = f'<progress value="{score}" max="100" style="width:100%;height:20px;background-color:{color}">{score}%</progress>'
+
+        # Automatically create Salesforce record
+        create_salesforce_record(score, checklist_summary, missing_summary, status)
+
+        return score, checklist_summary, missing_summary, status, progress_bar
+    except Exception as e:
+        logging.error(f"Error in evaluate_readiness_and_save: {str(e)}")
+        raise
 
 # Generate PDF report with signature slots
 def generate_pdf(score, checklist_summary, missing_summary, status):
-    pdf_file = "readiness_report.pdf"
-    c = canvas.Canvas(pdf_file, pagesize=letter)
-    c.setFont("Helvetica-Bold", 16)
-    c.drawString(50, 750, "Project Closure Readiness Report")
-    c.setFont("Helvetica", 12)
-    c.drawString(50, 720, f"Readiness Score: {score}%")
-    c.drawString(50, 700, f"Status: {status}")
-    c.drawString(50, 680, "Checklist Summary:")
-    y = 660
-    for line in checklist_summary.split("\n"):
-        c.drawString(50, y, line)
-        y -= 20
-    c.drawString(50, y-20, "Missing Items:")
-    c.drawString(50, y-40, missing_summary)
-    c.drawString(50, y-80, "Stakeholder Signature: ____________________")
-    c.drawString(50, y-100, "Date: ____________________")
-    c.save()
-    return pdf_file
+    try:
+        pdf_file = "readiness_report.pdf"
+        c = canvas.Canvas(pdf_file, pagesize=letter)
+        c.setFont("Helvetica-Bold", 16)
+        c.drawString(50, 750, "Project Closure Readiness Report")
+        c.setFont("Helvetica", 12)
+        c.drawString(50, 720, f"Readiness Score: {score}%")
+        c.drawString(50, 700, f"Status: {status}")
+        c.drawString(50, 680, "Checklist Summary:")
+        y = 660
+        for line in checklist_summary.split("\n"):
+            c.drawString(50, y, line)
+            y -= 20
+        c.drawString(50, y-20, "Missing Items:")
+        c.drawString(50, y-40, missing_summary)
+        c.drawString(50, y-80, "Stakeholder Signature: ____________________")
+        c.drawString(50, y-100, "Date: ____________________")
+        c.save()
+        return pdf_file
+    except Exception as e:
+        logging.error(f"Error in generate_pdf: {str(e)}")
+        raise
 
 # Gradio interface with updated UI
 with gr.Blocks(css="""progress { background-color: #f0f0f0; } .red { background-color: red; }""") as demo: